Re: [PATCH v2 1/2] ARM: SAMSUNG: Add pm_caps into platform data

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

 



Dear Mr. Lee,

On 16 November 2011 15:50, Sangwook Lee <sangwook.lee@xxxxxxxxxx> wrote:
> Add pm_caps into platform_data. This is power management, usually
> for SDIO device such as SDIO WLAN.
>
> Signed-off-by: Sangwook Lee <sangwook.lee@xxxxxxxxxxx>
> ---
>  arch/arm/plat-samsung/include/plat/sdhci.h |    2 ++
>  arch/arm/plat-samsung/platformdata.c       |    2 ++
>  2 files changed, 4 insertions(+), 0 deletions(-)
>
> diff --git a/arch/arm/plat-samsung/include/plat/sdhci.h b/arch/arm/plat-samsung/include/plat/sdhci.h
> index dcff7dd..fcd02d4 100644
> --- a/arch/arm/plat-samsung/include/plat/sdhci.h
> +++ b/arch/arm/plat-samsung/include/plat/sdhci.h
> @@ -40,6 +40,7 @@ enum clk_types {
>  * struct s3c_sdhci_platdata() - Platform device data for Samsung SDHCI
>  * @max_width: The maximum number of data bits supported.
>  * @host_caps: Standard MMC host capabilities bit field.
> + * @pm_caps: SDIO host PM capabilities bit field.
>  * @cd_type: Type of Card Detection method (see cd_types enum above)
>  * @clk_type: Type of clock divider method (see clk_types enum above)
>  * @ext_cd_init: Initialize external card detect subsystem. Called on
> @@ -63,6 +64,7 @@ enum clk_types {
>  struct s3c_sdhci_platdata {
>        unsigned int    max_width;
>        unsigned int    host_caps;
> +       unsigned int    pm_caps;
>        enum cd_types   cd_type;
>        enum clk_types  clk_type;
>
> diff --git a/arch/arm/plat-samsung/platformdata.c b/arch/arm/plat-samsung/platformdata.c
> index ceb9fa3..0f70718 100644
> --- a/arch/arm/plat-samsung/platformdata.c
> +++ b/arch/arm/plat-samsung/platformdata.c
> @@ -53,6 +53,8 @@ void s3c_sdhci_set_platdata(struct s3c_sdhci_platdata *pd,
>                set->cfg_gpio = pd->cfg_gpio;
>        if (pd->host_caps)
>                set->host_caps |= pd->host_caps;
> +       if (pd->pm_caps)
> +               set->pm_caps |= pd->pm_caps;


Are the 'pm_caps' specific to a (on-board) sdio host attached to the
sdhci controller? If yes, then this change should be fine. If no,
please let me know. I am kind of worried of any new addition to
platform data. Because, if that new addition in platform data is not
board specific data, then it becomes difficult to represent that
information in device tree.

Thanks,
Thomas.


>        if (pd->clk_type)
>                set->clk_type = pd->clk_type;
>  }
> --
> 1.7.4.1
>
> --
> 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