[U-Boot] [PATCH v1] LS2080ardb: MMC: disable esdhc for dspi

[Y.B. LU]

Hi Yinbo,

> -----Original Message-----
> From: Yinbo Zhu <yinbo.zhu at nxp.com>
> Sent: Thursday, November 22, 2018 4:17 PM
> To: yorksun at freescale.com; Y.B. LU <yangbo.lu at nxp.com>;
> u-boot at lists.denx.de
> Cc: Yinbo Zhu <yinbo.zhu at nxp.com>; Xiaobo Xie <xiaobo.xie at nxp.com>;
> Chuanhua Han <chuanhua.han at nxp.com>
> Subject: [PATCH v1] LS2080ardb: MMC: disable esdhc for dspi
> 
> esdhc and dspi have Multiplexed pins, if dspi is enabled and esdhc need to be
> disabled, This patch is to disable esdhc when rcw config dspi enabled.

[Y.b. Lu] The title and commit message is confusing. From the code changes, I think you can rephrase like,

armv8: ls2080ardb: enable eSDHC pin mux support

CONFIG_FSL_ESDHC_PIN_MUX should be enabled for ls2080ardb whose eSDHC has pin mux with DSPI.
This config makes sure esdhc dts node would be fixed with disabled status if DSPI is used, in case of any
mmc issue in linux. Also, esdhc should be added in default hwconfig since eSDHC is used in default.


> 
> Signed-off-by: Yinbo Zhu <yinbo.zhu at nxp.com>
> ---
>  include/configs/ls2080ardb.h |    5 +++--
>  1 files changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/include/configs/ls2080ardb.h b/include/configs/ls2080ardb.h index
> f6aa4ed..d2e80a8 100644
> --- a/include/configs/ls2080ardb.h
> +++ b/include/configs/ls2080ardb.h
> @@ -331,6 +331,7 @@ unsigned long get_board_sys_clk(void);
>  /*  MMC  */
>  #ifdef CONFIG_MMC
>  #define CONFIG_SYS_FSL_MMC_HAS_CAPBLT_VS33
> +#define CONFIG_FSL_ESDHC_PIN_MUX
>  #endif
> 
>  #define BOOT_TARGET_DEVICES(func) \
> @@ -390,7 +391,7 @@ unsigned long get_board_sys_clk(void);  #undef
> CONFIG_EXTRA_ENV_SETTINGS  #ifdef CONFIG_TFABOOT
>  #define CONFIG_EXTRA_ENV_SETTINGS		\
> -	"hwconfig=fsl_ddr:bank_intlv=auto\0"	\
> +	"hwconfig=fsl_ddr:bank_intlv=auto;esdhc\0"	\
>  	"ramdisk_addr=0x800000\0"		\
>  	"ramdisk_size=0x2000000\0"		\
>  	"fdt_high=0xa0000000\0"			\
> @@ -459,7 +460,7 @@ unsigned long get_board_sys_clk(void);
>  		"bootm $load_addr#$board\0"
>  #else
>  #define CONFIG_EXTRA_ENV_SETTINGS		\
> -	"hwconfig=fsl_ddr:bank_intlv=auto\0"	\
> +	"hwconfig=fsl_ddr:bank_intlv=auto;esdhc\0"	\
>  	"ramdisk_addr=0x800000\0"		\
>  	"ramdisk_size=0x2000000\0"		\
>  	"fdt_high=0xa0000000\0"			\
> --
> 1.7.1