[U-Boot] [PATCH 4/6] dma: ti: add driver to K3 UDMA

Tom Rini trini at konsulko.com
Wed Jan 23 13:52:59 UTC 2019


On Wed, Jan 23, 2019 at 12:35:26PM +0200, Peter Ujfalusi wrote:
> Tom,
> 
> On 22/01/2019 20.56, Tom Rini wrote:
> > On Tue, Jan 22, 2019 at 08:33:57PM +0530, Vignesh R wrote:
> > 
> >> The UDMA-P is intended to perform similar (but significantly upgraded) functions
> >> as the packet-oriented DMA used on previous SoC devices. The UDMA-P module
> >> supports the transmission and reception of various packet types.
> >> The UDMA-P also supports acting as both a UTC and UDMA-C for its internal
> >> channels. Channels in the UDMA-P can be configured to be either Packet-Based or
> >> Third-Party channels on a channel by channel basis.
> >>
> >> The initial driver supports:
> >> - MEM_TO_MEM (TR mode)
> >> - DEV_TO_MEM (Packet mode)
> >> - MEM_TO_DEV (Packet mode)
> >>
> >> Signed-off-by: Peter Ujfalusi <peter.ujfalusi at ti.com>
> >> Signed-off-by: Grygorii Strashko <grygorii.strashko at ti.com>
> >> Signed-off-by: Vignesh R <vigneshr at ti.com>
> > 
> > Reviewed-by: Tom Rini <trini at konsulko.com>
> > 
> > And the DT binding is common to Linux, and been reviewed there?  Or?
> 
> The binding is the same for Linux but unfortunately it has not went
> through a proper review yet due to the fact that I need to wait for the
> interrupt support to arrive to mainline.
> 
> However I have sent an earlier version as RFC:
> https://www.spinics.net/lists/dmaengine/msg16661.html
> 
> As for the bindings (and code):
> The linux bindings are different:

OK.  But we generally don't want the bindings to be different as,
putting my DT guy hat on, they aren't supposed to be OS centric and all
of that.

That's not to say that we might not end up needing to figure out ways to
do something more simply in U-Boot than in Linux.

So, really, do we want the bindings to not match?  Thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20190123/97cf5491/attachment.sig>


More information about the U-Boot mailing list