Thomas Abraham wrote:
>
> Dear Mr. Kim,
>
> On 10 October 2011 23:45, Thomas Abraham <thomas.abraham@xxxxxxxxxx>
> wrote:
> > With the 'struct dma_pl330_peri' removed, the platfrom data for dma
> > driver can be simplified to a simple list of peripheral request ids.
> >
> > Cc: Jassi Brar <jassisinghbrar@xxxxxxxxx>
> > Cc: Boojin Kim <boojin.kim@xxxxxxxxxxx>
> > Signed-off-by: Thomas Abraham <thomas.abraham@xxxxxxxxxx>
> > Acked-by: Kukjin Kim <kgene.kim@xxxxxxxxxxx>
> > Acked-by: Grant Likely <grant.likely@xxxxxxxxxxxx>
> > ---
> > arch/arm/mach-exynos4/dma.c | 223 ++++++++++++---------------------
> ----------
> > 1 files changed, 62 insertions(+), 161 deletions(-)
> >
> > diff --git a/arch/arm/mach-exynos4/dma.c b/arch/arm/mach-
> exynos4/dma.c
> > index 9667c61..c3c0d17 100644
> > --- a/arch/arm/mach-exynos4/dma.c
> > +++ b/arch/arm/mach-exynos4/dma.c
> > @@ -35,95 +35,40 @@
> >
> > static u64 dma_dmamask = DMA_BIT_MASK(32);
> >
> > -struct dma_pl330_peri pdma0_peri[28] = {
> > - {
> > - .peri_id = (u8)DMACH_PCM0_RX,
> > - .rqtype = DEVTOMEM,
> > - }, {
> > - .peri_id = (u8)DMACH_PCM0_TX,
> > - .rqtype = MEMTODEV,
> > - }, {
> > - .peri_id = (u8)DMACH_PCM2_RX,
> > - .rqtype = DEVTOMEM,
> > - }, {
> > - .peri_id = (u8)DMACH_PCM2_TX,
> > - .rqtype = MEMTODEV,
> > - }, {
> > - .peri_id = (u8)DMACH_MSM_REQ0,
> > - }, {
> > - .peri_id = (u8)DMACH_MSM_REQ2,
> > - }, {
> > - .peri_id = (u8)DMACH_SPI0_RX,
> > - .rqtype = DEVTOMEM,
> > - }, {
> > - .peri_id = (u8)DMACH_SPI0_TX,
> > - .rqtype = MEMTODEV,
> > - }, {
> > - .peri_id = (u8)DMACH_SPI2_RX,
> > - .rqtype = DEVTOMEM,
> > - }, {
> > - .peri_id = (u8)DMACH_SPI2_TX,
> > - .rqtype = MEMTODEV,
> > - }, {
> > - .peri_id = (u8)DMACH_I2S0S_TX,
> > - .rqtype = MEMTODEV,
> > - }, {
> > - .peri_id = (u8)DMACH_I2S0_RX,
> > - .rqtype = DEVTOMEM,
> > - }, {
> > - .peri_id = (u8)DMACH_I2S0_TX,
> > - .rqtype = MEMTODEV,
> > - }, {
> > - .peri_id = (u8)DMACH_UART0_RX,
> > - .rqtype = DEVTOMEM,
> > - }, {
> > - .peri_id = (u8)DMACH_UART0_TX,
> > - .rqtype = MEMTODEV,
> > - }, {
> > - .peri_id = (u8)DMACH_UART2_RX,
> > - .rqtype = DEVTOMEM,
> > - }, {
> > - .peri_id = (u8)DMACH_UART2_TX,
> > - .rqtype = MEMTODEV,
> > - }, {
> > - .peri_id = (u8)DMACH_UART4_RX,
> > - .rqtype = DEVTOMEM,
> > - }, {
> > - .peri_id = (u8)DMACH_UART4_TX,
> > - .rqtype = MEMTODEV,
> > - }, {
> > - .peri_id = (u8)DMACH_SLIMBUS0_RX,
> > - .rqtype = DEVTOMEM,
> > - }, {
> > - .peri_id = (u8)DMACH_SLIMBUS0_TX,
> > - .rqtype = MEMTODEV,
> > - }, {
> > - .peri_id = (u8)DMACH_SLIMBUS2_RX,
> > - .rqtype = DEVTOMEM,
> > - }, {
> > - .peri_id = (u8)DMACH_SLIMBUS2_TX,
> > - .rqtype = MEMTODEV,
> > - }, {
> > - .peri_id = (u8)DMACH_SLIMBUS4_RX,
> > - .rqtype = DEVTOMEM,
> > - }, {
> > - .peri_id = (u8)DMACH_SLIMBUS4_TX,
> > - .rqtype = MEMTODEV,
> > - }, {
> > - .peri_id = (u8)DMACH_AC97_MICIN,
> > - .rqtype = DEVTOMEM,
> > - }, {
> > - .peri_id = (u8)DMACH_AC97_PCMIN,
> > - .rqtype = DEVTOMEM,
> > - }, {
> > - .peri_id = (u8)DMACH_AC97_PCMOUT,
> > - .rqtype = MEMTODEV,
> > - },
> > +u8 pdma0_peri[] = {
> > + DMACH_PCM0_RX,
> > + DMACH_PCM0_TX,
> > + DMACH_PCM2_RX,
> > + DMACH_PCM2_TX,
> > + DMACH_MSM_REQ0,
> > + DMACH_MSM_REQ2,
> > + DMACH_SPI0_RX,
> > + DMACH_SPI0_TX,
> > + DMACH_SPI2_RX,
> > + DMACH_SPI2_TX,
> > + DMACH_I2S0S_TX,
> > + DMACH_I2S0_RX,
> > + DMACH_I2S0_TX,
>
> As you have suggested, I have rechecked the platform data for all s5p
> platforms in this patchset. Here, DMACH_I2S2_RX (event id 13) and
> DMACH_I2S2_TX (event id 14) of DMAC[0] are missing as per the
> exynos4210 user manual. But these where missing in the original code
> as well which this patch modified. If possible, could you please let
> me know if DMACH_I2S2_RX and DMACH_I2S2_TX are indeed event id 13 and
> 14 in DMAC[0].
Yes, DMACH_I2S2_RX and DMACH_I2X2_TX are missed from my patch.
Can you modify it although you are inconvenient ?
Thanks..
Boojin
>
> I have rechecked the other s5p platform data for dmac (s5p64x0,
> s5pc100, s5pv210). Apart from the typo pointed out by Anca for
> s5pc100, there are no other issues.
>
> Thanks,
> Thomas.
>
> > + DMACH_UART0_RX,
> > + DMACH_UART0_TX,
> > + DMACH_UART2_RX,
> > + DMACH_UART2_TX,
> > + DMACH_UART4_RX,
> > + DMACH_UART4_TX,
> > + DMACH_SLIMBUS0_RX,
> > + DMACH_SLIMBUS0_TX,
> > + DMACH_SLIMBUS2_RX,
> > + DMACH_SLIMBUS2_TX,
> > + DMACH_SLIMBUS4_RX,
> > + DMACH_SLIMBUS4_TX,
> > + DMACH_AC97_MICIN,
> > + DMACH_AC97_PCMIN,
> > + DMACH_AC97_PCMOUT,
> > };
> >
> > struct dma_pl330_platdata exynos4_pdma0_pdata = {
> > .nr_valid_peri = ARRAY_SIZE(pdma0_peri),
> > - .peri = pdma0_peri,
> > + .peri_id = pdma0_peri,
> > };
> >
>
> [...]
> --
> To unsubscribe from this list: send the line "unsubscribe linux-
> samsung-soc" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
