RE: [PATCH 8/9] ARM: dts: iwg22d-sodimm: Sound DMA support via SRC on DTS

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




Hi Simon,

> -----Original Message-----
> From: Simon Horman [mailto:horms@xxxxxxxxxxxx]
> Sent: 21 December 2017 10:45
> To: Biju Das <biju.das@xxxxxxxxxxxxxx>
> Cc: Rob Herring <robh+dt@xxxxxxxxxx>; Mark Rutland
> <mark.rutland@xxxxxxx>; Magnus Damm <magnus.damm@xxxxxxxxx>;
> Chris Paterson <Chris.Paterson2@xxxxxxxxxxx>; Fabrizio Castro
> <fabrizio.castro@xxxxxxxxxxxxxx>; devicetree@xxxxxxxxxxxxxxx; linux-renesas-
> soc@xxxxxxxxxxxxxxx
> Subject: Re: [PATCH 8/9] ARM: dts: iwg22d-sodimm: Sound DMA support via
> SRC on DTS
>
> On Wed, Dec 20, 2017 at 08:02:04PM +0000, Biju Das wrote:
> > DMA transfer to/from SRC
> >
> >      DMA      DMApp
> > [MEM] -> [SRC] -> [SSIU] -> [SSI]
> >
> >      DMA      DMApp
> > [MEM] <- [SRC] <- [SSIU] <- [SSI]
> >
> > Current sound driver is supporting SSI/SRC random connection.
> > So, this patch is trying
> > SSI3 -> SRC3
> > SSI4 <- SRC4
>
> I am curious to know why the selection here differs from that present in
> iwg20d-q7-common. Is it an arbitrary decision?
>
> Likewise for other uses of ssi*, src* and dvc* in this patchset.

For ssi*,  on this board, SSI3/4 interface is connected to Audio In/Out.

For src*,  the original bsp from iwave is using src3/src4, also it is arbitrary as per the data sheet.

For dvc*,  Again it is arbitrary as per the data sheet.

> >
> > Signed-off-by: Biju Das <biju.das@xxxxxxxxxxxxxx>
> > Reviewed-by: Fabrizio Castro <fabrizio.castro@xxxxxxxxxxxxxx>
> > ---
> >  arch/arm/boot/dts/r8a7745-iwg22d-sodimm.dts | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/arch/arm/boot/dts/r8a7745-iwg22d-sodimm.dts
> > b/arch/arm/boot/dts/r8a7745-iwg22d-sodimm.dts
> > index 0f880c1..2cac57c 100644
> > --- a/arch/arm/boot/dts/r8a7745-iwg22d-sodimm.dts
> > +++ b/arch/arm/boot/dts/r8a7745-iwg22d-sodimm.dts
> > @@ -183,8 +183,8 @@
> >
> >  rcar_sound,dai {
> >  dai0 {
> > -playback = <&ssi3>;
> > -capture = <&ssi4>;
> > +playback = <&ssi3 &src3>;
> > +capture = <&ssi4 &src4>;
> >  };
> >  };
> >  };
> > --
> > 1.9.1
> >


[https://www2.renesas.eu/media/email/unicef_2017.jpg]

This Christmas, instead of sending out cards, Renesas Electronics Europe have decided to support Unicef with a donation. For further details click here<https://www.unicef.org/> to find out about the valuable work they do, helping children all over the world.
We would like to take this opportunity to wish you a Merry Christmas and a prosperous New Year.



Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered No. 04586709.
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux