Re: [PATCH 1/6] ARM: SAMSUNG: Consolidation of SPI platform devices to plat-samsung

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

 



Hi Kukjin,

On Tue, Nov 15, 2011 at 12:38 PM, Kukjin Kim <kgene.kim@xxxxxxxxxxx> wrote:
> Padmavathi Venna wrote:
>>
>> SPI platform device definitions consolidated from respective machine
>> folder to plat-samsung
>>
>> Signed-off-by: Padmavathi Venna <padma.v@xxxxxxxxxxx>
>> ---
>>  arch/arm/mach-s3c64xx/Makefile                   |    1 -
>>  arch/arm/mach-s3c64xx/dev-spi.c                  |  172 -----------------
>>  arch/arm/mach-s5p64x0/Makefile                   |    1 -
>>  arch/arm/mach-s5p64x0/dev-spi.c                  |  218
> ---------------------
>>  arch/arm/mach-s5pc100/Makefile                   |    1 -
>>  arch/arm/mach-s5pc100/dev-spi.c                  |  220
> ----------------------
>>  arch/arm/mach-s5pv210/Makefile                   |    1 -
>>  arch/arm/mach-s5pv210/dev-spi.c                  |  169 -----------------
>>  arch/arm/plat-samsung/Kconfig                    |   14 ++-
>>  arch/arm/plat-samsung/devs.c                     |  112 +++++++++++
>>  arch/arm/plat-samsung/include/plat/devs.h        |    8 +-
>>  arch/arm/plat-samsung/include/plat/s3c64xx-spi.h |   13 +-
>>  12 files changed, 133 insertions(+), 797 deletions(-)
>>  delete mode 100644 arch/arm/mach-s3c64xx/dev-spi.c
>>  delete mode 100644 arch/arm/mach-s5p64x0/dev-spi.c
>>  delete mode 100644 arch/arm/mach-s5pc100/dev-spi.c
>>  delete mode 100644 arch/arm/mach-s5pv210/dev-spi.c
>>
>
> Hi Padma,
>
> Basically, looks good and there are some comments below.
>
> (snip)
>
>> diff --git a/arch/arm/plat-samsung/Kconfig b/arch/arm/plat-samsung/Kconfig
>> index 313eb26..8a2b23a 100644
>> --- a/arch/arm/plat-samsung/Kconfig
>> +++ b/arch/arm/plat-samsung/Kconfig
>> @@ -226,7 +226,19 @@ config SAMSUNG_DEV_IDE
>>       help
>>         Compile in platform device definitions for IDE
>>
>> -config S3C64XX_DEV_SPI
>> +config S3C64XX_DEV_SPI0
>> +     bool
>> +     help
>> +       Compile in platform device definitions for S3C64XX's type
>> +       SPI controllers.
>> +
>> +config S3C64XX_DEV_SPI1
>> +     bool
>> +     help
>> +       Compile in platform device definitions for S3C64XX's type
>> +       SPI controllers.
>> +
>> +config S3C64XX_DEV_SPI2
>
> Yeah, needs above config to check the channel now...but I'm thinking need to
> sort out it anyway.
>
>>       bool
>>       help
>>         Compile in platform device definitions for S3C64XX's type
>> diff --git a/arch/arm/plat-samsung/devs.c b/arch/arm/plat-samsung/devs.c
>> index 4ca8b57..76e374d 100644
>> --- a/arch/arm/plat-samsung/devs.c
>> +++ b/arch/arm/plat-samsung/devs.c
>> @@ -61,6 +61,7 @@
>>  #include <plat/regs-iic.h>
>>  #include <plat/regs-serial.h>
>>  #include <plat/regs-spi.h>
>> +#include <plat/s3c64xx-spi.h>
>>
>>  static u64 samsung_device_dma_mask = DMA_BIT_MASK(32);
>>
>> @@ -1461,3 +1462,114 @@ struct platform_device s3c_device_wdt = {
>>       .resource       = s3c_wdt_resource,
>>  };
>>  #endif /* CONFIG_S3C_DEV_WDT */
>> +
>> +#ifdef CONFIG_S3C64XX_DEV_SPI0
>> +static struct resource s3c64xx_spi0_resource[] = {
>> +     [0] = DEFINE_RES_MEM(S3C_PA_SPI0, SZ_256),
>> +     [1] = DEFINE_RES_DMA(DMACH_SPI0_TX),
>> +     [2] = DEFINE_RES_DMA(DMACH_SPI0_RX),
>> +     [3] = DEFINE_RES_IRQ(IRQ_SPI0),
>> +};
>> +
>> +struct platform_device s3c64xx_device_spi0 = {
>> +     .name           = "s3c64xx-spi",
>> +     .id             = 0,
>> +     .num_resources  = ARRAY_SIZE(s3c64xx_spi0_resource),
>> +     .resource       = s3c64xx_spi0_resource,
>> +     .dev = {
>> +             .dma_mask               = &samsung_device_dma_mask,
>> +             .coherent_dma_mask      = DMA_BIT_MASK(32),
>> +     },
>> +};
>> +
>> +void __init s3c64xx_spi0_set_info(struct s3c64xx_spi_info *pd,
>> +                                     int src_clk_nr, int num_cs)
>
> How about 's3c64xx_spi_set_platdata' like others in plat-samsung/devs.c?
>
>> +{
>> +     if (!pd) {
>> +             pr_err("%s:Need to pass platform data\n", __func__);
>> +             return;
>> +     }
>
> Needs empty line here.
Okey.
>
>> +     /* Reject invalid configuration */
>> +     if (!num_cs || src_clk_nr < 0) {
>> +             pr_err("%s: Invalid SPI configuration\n", __func__);
>> +             return;
>> +     }
>
> Same as above.
Okey.
>
>> +     pd->num_cs = num_cs;
>> +     pd->src_clk_nr = src_clk_nr;
>> +     s3c_set_platdata(pd, sizeof(*pd), &s3c64xx_device_spi0);
>
> If we know the SPI controller number here, we can merge this function to
> just one. So how about to add it to the 'struct s3c64xx_spi_info'?
Yes. It is good to have the controller number in the SPI Controller
defining structure.
So I will rebase the patches based on latest for-next branch and
submit the patches.

Thanks&Regards
Padma
>> +}
>
> (snip)
>
> Thanks.
>
> Best regards,
> Kgene.
> --
> Kukjin Kim <kgene.kim@xxxxxxxxxxx>, Senior Engineer,
> SW Solution Development Team, Samsung Electronics Co., Ltd.
>
>
--
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