Sekhar Nori <[email protected]> writes:

> This patch provides a function to help register cpuidle driver
> on da8xx/omap-l1xx platforms.
>
> Signed-off-by: Sekhar Nori <[email protected]>

Other than pdata struct change proposed in previous patch, this
apprach looks good.

This should also be rebased on top of your cpufreq changes which
I'll be pushing today.

Kevin

>
> ---
>  arch/arm/mach-davinci/devices-da8xx.c      |   26 ++++++++++++++++++++++++++
>  arch/arm/mach-davinci/include/mach/da8xx.h |    2 ++
>  2 files changed, 28 insertions(+), 0 deletions(-)
>
> diff --git a/arch/arm/mach-davinci/devices-da8xx.c 
> b/arch/arm/mach-davinci/devices-da8xx.c
> index dd0ea08..0ba6e5c 100644
> --- a/arch/arm/mach-davinci/devices-da8xx.c
> +++ b/arch/arm/mach-davinci/devices-da8xx.c
> @@ -488,3 +488,29 @@ int da8xx_register_rtc(void)
>  
>       return platform_device_register(&da8xx_rtc_device);
>  }
> +
> +static struct resource da8xx_cpuidle_resources[] = {
> +     {
> +             .start          = DA8XX_DDR2_CTL_BASE,
> +             .end            = DA8XX_DDR2_CTL_BASE + SZ_32K - 1,
> +             .flags          = IORESOURCE_MEM,
> +     },
> +};
> +
> +/* DA8XX devices support DDR2 power down */
> +static int ddr2_pdown = 1;
> +
> +static struct platform_device da8xx_cpuidle_device = {
> +     .name                   = "cpuidle-davinci",
> +     .num_resources          = ARRAY_SIZE(da8xx_cpuidle_resources),
> +     .resource               = da8xx_cpuidle_resources,
> +     .dev = {
> +             .platform_data  = &ddr2_pdown,
> +     },
> +};
> +int __init da8xx_register_cpuidle(void)
> +{
> +     return platform_device_register(&da8xx_cpuidle_device);
> +}
> +
> diff --git a/arch/arm/mach-davinci/include/mach/da8xx.h 
> b/arch/arm/mach-davinci/include/mach/da8xx.h
> index a152261..1de7957 100644
> --- a/arch/arm/mach-davinci/include/mach/da8xx.h
> +++ b/arch/arm/mach-davinci/include/mach/da8xx.h
> @@ -49,6 +49,7 @@ extern void __iomem *da8xx_syscfg_base;
>  #define DA8XX_AEMIF_CS2_BASE 0x60000000
>  #define DA8XX_AEMIF_CS3_BASE 0x62000000
>  #define DA8XX_AEMIF_CTL_BASE 0x68000000
> +#define DA8XX_DDR2_CTL_BASE  0xb0000000
>  
>  #define PINMUX0                      0x00
>  #define PINMUX1                      0x04
> @@ -82,6 +83,7 @@ int da8xx_register_lcdc(struct da8xx_lcdc_platform_data 
> *pdata);
>  int da8xx_register_mmcsd0(struct davinci_mmc_config *config);
>  void __init da8xx_register_mcasp(int id, struct snd_platform_data *pdata);
>  int da8xx_register_rtc(void);
> +int da8xx_register_cpuidle(void);
>  
>  extern struct platform_device da8xx_serial_device;
>  extern struct emac_platform_data da8xx_emac_pdata;
> -- 
> 1.6.2.4
>
> _______________________________________________
> Davinci-linux-open-source mailing list
> [email protected]
> http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

_______________________________________________
Davinci-linux-open-source mailing list
[email protected]
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to