Re: [PATCH v6 03/10] ARM: EXYNOS4: Modify platform data for pl330 driver

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

 



Dear Boojin Kim,

2011/10/12 Boojin Kim <boojin.kim@xxxxxxxxxxx>:
> 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 ?

Thank you for letting me know about these event numbers.

Yes. I can modify this patch and s5pc100 patch for Anca's comment and
send these two updated patches after testing.

Thanks,
Thomas.

>
> 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


[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux