Re: [PATCH] ARM: dts: add AM33XX EDMA support

[Sergei Shtylyov]

Hello.

On 08/24/2013 10:33 PM, Joel Fernandes wrote:

> Updating CC with Matt's current email address.

> > > From: Matt Porter <mdp@xxxxxx>

> > > Adds AM33XX EDMA support to the am33xx.dtsi as documented in
> > > Documentation/devicetree/bindings/dma/ti-edma.txt

> > > Joel: Drop DT entries that are non-hardware-description for now as discussed
> > > in [1]

> > > [1] https://patchwork.kernel.org/patch/2226761/

> > > Signed-off-by: Matt Porter <mporter@xxxxxx>
> > > Signed-off-by: Joel A Fernandes <joelagnel@xxxxxx>
> > > Signed-off-by: Sebastian Andrzej Siewior <bigeasy@xxxxxxxxxxxxx>
> > > ---
> > > Could someone please pick this up?

> > >    arch/arm/boot/dts/am33xx.dtsi | 12 ++++++++++++
> > >    1 file changed, 12 insertions(+)

> > > diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi
> > > index 38b446b..784f774 100644
> > > --- a/arch/arm/boot/dts/am33xx.dtsi
> > > +++ b/arch/arm/boot/dts/am33xx.dtsi
> > > @@ -96,6 +96,18 @@
> > >                reg = <0x48200000 0x1000>;
> > >            };
> > >
> > > +        edma: edma@49000000 {

> >     The node should be named "dma-controller", not "edma",according to ePAPR
> > section 2.2.2:

> > http://www.power.org/resources/downloads/Power_ePAPR_APPROVED_v1.0.pdf

> So you mean something like the following?

> edma: dma-controller@49000000 {
>    ...
> }

   Yes, exactly.

> Thanks,

> -Joel

WBR, Sergei

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/