Re: [PATCH] ARM: dts: imx50: imx50-esdhc use imx53-esdhc

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

 



Hi Alexander,

On 09.10.2016 18:34, Alexander Kurz wrote:
> According to the reference manuals, both imx50/imx53 SOC seem to share
> the same eSDHC controller, especially the section on "Multi-block Read"
> mentioned in commit 361b8482026c ("mmc: sdhci-esdhc-imx: fix multiblock
> reads on i.MX53") is identical for both SOC.
> Hence, let imx50 use imx53-esdhc.
> 
> Signed-off-by: Alexander Kurz <akurz@xxxxxxxx>
> ---
>  arch/arm/boot/dts/imx50.dtsi | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/imx50.dtsi b/arch/arm/boot/dts/imx50.dtsi
> index e245713..769eac7 100644
> --- a/arch/arm/boot/dts/imx50.dtsi
> +++ b/arch/arm/boot/dts/imx50.dtsi
> @@ -99,7 +99,7 @@
>  				ranges;
>  
>  				esdhc1: esdhc@50004000 {
> -					compatible = "fsl,imx50-esdhc";
> +					compatible = "fsl,imx50-esdhc", "fsl,imx53-esdhc";

Instead of this change please add "fsl,imx50-esdhc" compatible to a list found in
drivers/mmc/host/sdhci-esdhc-imx.c

Regarding DTS change if the controllers are the same I think it may
have sense to add "fsl,imx50-esdhc" compatible to imx53.dtsi definitions,
but not the other way round.

--
With best wishes,
Vladimir
--
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