Re: [PATCH] mmc: dw_mmc: fix misleading comment in dw_mci_rk3288_set_ios

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

 



On 5 April 2018 at 02:52, Shawn Lin <shawn.lin@xxxxxxxxxxxxxx> wrote:
> DDR52 with 8-bit mode should be handled in a different way when
> requesting ciu_clk. However DDR50 is used for SDMMC/SDIO and
> could never be possible with 8-bit mode. It's trival but misleading.
>
> Signed-off-by: Shawn Lin <shawn.lin@xxxxxxxxxxxxxx>
> ---
>
>  drivers/mmc/host/dw_mmc-rockchip.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/mmc/host/dw_mmc-rockchip.c b/drivers/mmc/host/dw_mmc-rockchip.c
> index 40d7de2..3fe4b9d 100644
> --- a/drivers/mmc/host/dw_mmc-rockchip.c
> +++ b/drivers/mmc/host/dw_mmc-rockchip.c
> @@ -45,7 +45,7 @@ static void dw_mci_rk3288_set_ios(struct dw_mci *host, struct mmc_ios *ios)
>          * ios->clock = (div == 0) ? bus_hz : (bus_hz / (2 * div))
>          *
>          * Note: div can only be 0 or 1
> -        *       if DDR50 8bit mode(only emmc work in 8bit mode),
> +        *       if DDR52 8bit mode(only emmc work in 8bit mode),

how about "eMMC DDR52 8-bit mode."

>          *       div must be set 1
>          */
>         if (ios->bus_width == MMC_BUS_WIDTH_8 &&
> --
> 1.9.1
>

Kind regards
Uffe
--
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