Re: [PATCH] ARM: EXYNOS4: Fix card insert/removal event detection on smdkv310 board

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

 



Hi Marek,

On Mon, May 16, 2011 at 10:27 PM, Marek Szyprowski
<m.szyprowski@xxxxxxxxxxx> wrote:
> Hello,
>
> On Monday, May 16, 2011 4:06 PM Thomas Abraham wrote:
>
>> On SMDKV310 board, a card detect gpio pin is available that is directly
>> connected to the io pad of the sdhci controller. Fix incorrect value
>> of cd_type field in platform data for sdhci instance 0 and 2.
>>
>> Signed-off-by: Thomas Abraham <thomas.ab@xxxxxxxxxxx>
>> ---
>>  arch/arm/mach-exynos4/mach-smdkv310.c |    4 ++--
>>  1 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/arch/arm/mach-exynos4/mach-smdkv310.c b/arch/arm/mach-
>> exynos4/mach-smdkv310.c
>> index 1526764..f6b1c7e 100644
>> --- a/arch/arm/mach-exynos4/mach-smdkv310.c
>> +++ b/arch/arm/mach-exynos4/mach-smdkv310.c
>> @@ -78,7 +78,7 @@ static struct s3c2410_uartcfg smdkv310_uartcfgs[]
>> __initdata = {
>>  };
>>
>>  static struct s3c_sdhci_platdata smdkv310_hsmmc0_pdata __initdata = {
>> -     .cd_type                = S3C_SDHCI_CD_GPIO,
>> +     .cd_type                = S3C_SDHCI_CD_INTERNAL,
>>       .ext_cd_gpio            = EXYNOS4_GPK0(2),
>>       .ext_cd_gpio_invert     = 1,
>
> When you are changing the card detection type from GPIO to INTERNAL you should
> also remove 'ext_cd_gpio' and 'ext_cd_gpio_invert' definitions.

Thanks for your comments. I will fix this and submit another patch.

Thomas.

>
>>       .clk_type               = S3C_SDHCI_CLK_DIV_EXTERNAL,
>> @@ -96,7 +96,7 @@ static struct s3c_sdhci_platdata smdkv310_hsmmc1_pdata
>> __initdata = {
>>  };
>>
>>  static struct s3c_sdhci_platdata smdkv310_hsmmc2_pdata __initdata = {
>> -     .cd_type                = S3C_SDHCI_CD_GPIO,
>> +     .cd_type                = S3C_SDHCI_CD_INTERNAL,
>>       .ext_cd_gpio            = EXYNOS4_GPK2(2),
>>       .ext_cd_gpio_invert     = 1,
>
> The same as with hsmmc0.
>
>>       .clk_type               = S3C_SDHCI_CLK_DIV_EXTERNAL,
>> --
>
> Best regards
> --
> Marek Szyprowski
> Samsung Poland R&D Center
>
> --
> 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