[PATCH] configs: disable eMMC HS200 support on layerscape platforms

[Y.b. Lu]

Hi Jaehoon,

> -----Original Message-----
> From: Jaehoon Chung <jh80.chung at samsung.com>
> Sent: Thursday, March 19, 2020 5:01 PM
> To: Y.b. Lu <yangbo.lu at nxp.com>; u-boot at lists.denx.de; Priyanka Jain
> <priyanka.jain at nxp.com>
> Subject: Re: [PATCH] configs: disable eMMC HS200 support on layerscape
> platforms
> 
> Hi,
> 
> On 3/19/20 4:22 PM, Y.b. Lu wrote:
> > Any comments?
> > Thanks!
> >
> > Best regards,
> > Yangbo Lu
> >
> >> -----Original Message-----
> >> From: Yangbo Lu <yangbo.lu at nxp.com>
> >> Sent: Tuesday, March 3, 2020 10:33 AM
> >> To: u-boot at lists.denx.de; Priyanka Jain <priyanka.jain at nxp.com>
> >> Cc: Y.b. Lu <yangbo.lu at nxp.com>
> >> Subject: [PATCH] configs: disable eMMC HS200 support on layerscape
> >> platforms
> >>
> >> The eMMC HS200 speed mode on Layerscape platforms has not been
> >> supported properly. The eSDHC clock tuning has not been implemented
> >> by now. So disable it until it is supported properly in case of
> >> any potential issues.
> 
> 
> If it's not working fine, looks good to me.
> BTW, I didn't have tested on its platform. Do you have a plan to fix it?

[Yangbo Lu] Because we got issues with HS200 mode in testing, I sent this patch to disable it for now.
The tuning support with eSDHC tuning block is in my plan.

Thanks.

> 
> Reviewed-by: Jaehoon Chung <jh80.chung at samsung.com>
> 
> 
> >>
> >> Signed-off-by: Yangbo Lu <yangbo.lu at nxp.com>
> >> ---
> >>  configs/ls1012ardb_tfa_defconfig | 1 -
> >>  configs/ls1028ardb_tfa_defconfig | 1 -
> >>  configs/lx2160ardb_tfa_defconfig | 1 -
> >>  3 files changed, 3 deletions(-)
> >>
> >> diff --git a/configs/ls1012ardb_tfa_defconfig
> >> b/configs/ls1012ardb_tfa_defconfig
> >> index b47a47d..0e994f5 100644
> >> --- a/configs/ls1012ardb_tfa_defconfig
> >> +++ b/configs/ls1012ardb_tfa_defconfig
> >> @@ -38,7 +38,6 @@ CONFIG_NET_RANDOM_ETHADDR=y
> >>  CONFIG_DM=y
> >>  CONFIG_SATA_CEVA=y
> >>  CONFIG_DM_MMC=y
> >> -CONFIG_MMC_HS200_SUPPORT=y
> >>  CONFIG_FSL_ESDHC=y
> >>  CONFIG_MTD=y
> >>  CONFIG_DM_SPI_FLASH=y
> >> diff --git a/configs/ls1028ardb_tfa_defconfig
> >> b/configs/ls1028ardb_tfa_defconfig
> >> index 3ef5520..7ffd1c3 100644
> >> --- a/configs/ls1028ardb_tfa_defconfig
> >> +++ b/configs/ls1028ardb_tfa_defconfig
> >> @@ -45,7 +45,6 @@ CONFIG_I2C_DEFAULT_BUS_NUMBER=0
> >>  CONFIG_I2C_MUX=y
> >>  CONFIG_I2C_MUX_PCA954x=y
> >>  CONFIG_DM_MMC=y
> >> -CONFIG_MMC_HS200_SUPPORT=y
> >>  CONFIG_FSL_ESDHC=y
> >>  CONFIG_MTD=y
> >>  CONFIG_DM_SPI_FLASH=y
> >> diff --git a/configs/lx2160ardb_tfa_defconfig
> >> b/configs/lx2160ardb_tfa_defconfig
> >> index b998cb6..0799cd4 100644
> >> --- a/configs/lx2160ardb_tfa_defconfig
> >> +++ b/configs/lx2160ardb_tfa_defconfig
> >> @@ -41,7 +41,6 @@ CONFIG_DM_I2C=y
> >>  CONFIG_I2C_SET_DEFAULT_BUS_NUM=y
> >>  CONFIG_I2C_DEFAULT_BUS_NUMBER=0
> >>  CONFIG_DM_MMC=y
> >> -CONFIG_MMC_HS200_SUPPORT=y
> >>  CONFIG_FSL_ESDHC=y
> >>  CONFIG_MTD=y
> >>  CONFIG_DM_SPI_FLASH=y
> >> --
> >> 2.7.4
> >
> >
> >