[PATCH v7 1/2] ARM: davinci: dm365: add support for v4l2 video display

[nsekhar@ti.com (Sekhar Nori)]

On 3/25/2013 3:52 PM, Prabhakar Lad wrote:
> Hi Sekhar,
> 
> Thanks for the review!
> 
> On Mon, Mar 25, 2013 at 11:42 AM, Sekhar Nori <nsekhar@ti.com> wrote:
>> Prabhakar,
>>
>> On 3/22/2013 1:33 PM, Prabhakar lad wrote:
>>> From: Lad, Prabhakar <prabhakar.csengg@gmail.com>
>>>
>>> Create platform devices for various video modules like venc,osd,
>>> vpbe and v4l2 driver for dm365.
>>>
>>> Signed-off-by: Lad, Prabhakar <prabhakar.csengg@gmail.com>
>>> ---
>>>  arch/arm/mach-davinci/board-dm365-evm.c |    4 +-
>>>  arch/arm/mach-davinci/davinci.h         |    2 +-
>>>  arch/arm/mach-davinci/dm365.c           |  194 +++++++++++++++++++++++++++++--
>>>  3 files changed, 188 insertions(+), 12 deletions(-)
>>>
>>> diff --git a/arch/arm/mach-davinci/board-dm365-evm.c b/arch/arm/mach-davinci/board-dm365-evm.c
>>> index c2d4958..cf77c46 100644
>>> --- a/arch/arm/mach-davinci/board-dm365-evm.c
>>> +++ b/arch/arm/mach-davinci/board-dm365-evm.c
>>> @@ -564,8 +564,6 @@ static struct davinci_uart_config uart_config __initdata = {
>>>
>>>  static void __init dm365_evm_map_io(void)
>>>  {
>>> -     /* setup input configuration for VPFE input devices */
>>> -     dm365_set_vpfe_config(&vpfe_cfg);
>>>       dm365_init();
>>>  }
>>>
>>> @@ -597,6 +595,8 @@ static __init void dm365_evm_init(void)
>>>
>>>       davinci_setup_mmc(0, &dm365evm_mmc_config);
>>>
>>> +     dm365_init_video(&vpfe_cfg, NULL);
>>> +
>>>       /* maybe setup mmc1/etc ... _after_ mmc0 */
>>>       evm_init_cpld();
>>>
>>> diff --git a/arch/arm/mach-davinci/davinci.h b/arch/arm/mach-davinci/davinci.h
>>> index 12d544b..1c2670f 100644
>>> --- a/arch/arm/mach-davinci/davinci.h
>>> +++ b/arch/arm/mach-davinci/davinci.h
>>> @@ -84,7 +84,7 @@ void __init dm365_init_ks(struct davinci_ks_platform_data *pdata);
>>>  void __init dm365_init_rtc(void);
>>>  void dm365_init_spi0(unsigned chipselect_mask,
>>>                       const struct spi_board_info *info, unsigned len);
>>> -void dm365_set_vpfe_config(struct vpfe_config *cfg);
>>> +int __init dm365_init_video(struct vpfe_config *, struct vpbe_config *);
>>
>> No need of __init specifier in function declaration.
>>
> OK
> 
>>>
>>>  /* DM644x function declarations */
>>>  void __init dm644x_init(void);
>>> diff --git a/arch/arm/mach-davinci/dm365.c b/arch/arm/mach-davinci/dm365.c
>>> index f4c19f7..0a2dba2 100644
>>> --- a/arch/arm/mach-davinci/dm365.c
>>> +++ b/arch/arm/mach-davinci/dm365.c
>>> @@ -40,10 +40,16 @@
>>>
>>>  #define DM365_REF_FREQ               24000000        /* 24 MHz on the DM365 EVM */
>>>
>>> +#define DM365_VDAC_CONFIG            0x01c4002c
>>
>> Please define this in davinci.h as an offset from
>> DAVINCI_SYSTEM_MODULE_BASE.
>>
> OK
> 
>>> +
>>> +#define DM365_RTC_BASE                       0x01c69000
>>> +
>>>  /* Base of key scan register bank */
>>>  #define DM365_KEYSCAN_BASE           0x01c69400
>>>
>>> -#define DM365_RTC_BASE                       0x01c69000
>>> +#define DM365_OSD_BASE                       0x01c71c00
>>> +
>>> +#define DM365_VENC_BASE                      0x01c71e00
>>>
> Are you OK with this definitions or you want all of them in davinci.h ?

No please leave these here. The reason I asked you to use davinci.h for
SYSMOD is because I don't want those definitions to be strewn all over
the place - especially when there is already a consolidated place for them.

Thanks,
Sekhar