Re: [PATCH] sdhci-s3c: add platform_8bit_width()

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

 



On Wed, Jan 12, 2011 at 12:27 PM, Chris Ball <cjb@xxxxxxxxxx> wrote:
> Hi Jaehoon,
>
> On Wed, Jan 12, 2011 at 11:59:12AM +0900, Jaehoon Chung wrote:
>> Hi Chris.
>>
>> Thanks for your comments.
>> So i fixed them, and resend the patch.
>
> Thanks!  I've pushed it to mmc-next.
>
>> We have 8-bit width support bu is not a v3 controller.
>> So we need platform_8bit_width() to support 8-bit buswidth.
>> Also we need MMC_CAP_8_BIT_DATA, so we add it in platdata.
>>
>> Signed-off-by: Jaehoon Chung <jh80.chung@xxxxxxxxxxx>
>
> I made one more change -- the rest of sdhci-s3c.c uses kerneldoc format
> function comments, but your patch doesn't have the correct kerneldoc syntax:
>
> diff --git a/drivers/mmc/host/sdhci-s3c.c b/drivers/mmc/host/sdhci-s3c.c
> index 89f019f..5309ab9 100644
> --- a/drivers/mmc/host/sdhci-s3c.c
> +++ b/drivers/mmc/host/sdhci-s3c.c
> @@ -277,12 +277,14 @@ static void sdhci_cmu_set_clock(struct sdhci_host *host, unsigned int clock)
>        host->clock = clock;
>  }
>
> -/*
> +/**
>  * sdhci_s3c_platform_8bit_width - support 8bit buswidth
> + * @host: The SDHCI host being queried
> + * @width: MMC_BUS_WIDTH_ macro for the bus width being requested
>  *
>  * We have 8-bit width support but is not a v3 controller.
>  * So we add platform_8bit_width() and support 8bit width.
> - **/
> + */
>  static int sdhci_s3c_platform_8bit_width(struct sdhci_host *host, int width)
>  {
>        u8 ctrl;
>
>
> Ben, feel free to yell if you object to the final version of the patch,
> else I'll merge it for .39.

Hi Chris,

Umm I think it's kind of bug fix since we used 8-bit buswidth previous
code but recent change don't make it working
So re-support the 8-bit buswidth. Can you merge it at 38.rc period?

How do you think?

Thank you,
Kyungmin Park
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux