[U-Boot] [PATCH V2 29/32] fsl_esdhc: Update usdhc driver to support i.MX8

[Peng Fan]

Hi Troy,

> -----Original Message-----
> From: Troy Kisky [mailto:troy.kisky at boundarydevices.com]
> Sent: 2018年7月25日 2:27
> To: Fabio Estevam <festevam at gmail.com>; Peng Fan <peng.fan at nxp.com>
> Cc: Fabio Estevam <fabio.estevam at nxp.com>; U-Boot-Denx
> <u-boot at lists.denx.de>; dl-linux-imx <linux-imx at nxp.com>
> Subject: Re: [U-Boot] [PATCH V2 29/32] fsl_esdhc: Update usdhc driver to
> support i.MX8
> 
> On 7/23/2018 5:51 PM, Fabio Estevam wrote:
> > Hi Peng,
> >
> > On Mon, Jul 23, 2018 at 9:45 PM, Peng Fan <peng.fan at nxp.com> wrote:
> >
> >> In V1, I use CONFIG_IMX8, but I think there is no need to use it, because
> >> CONFIG_ARCH_IMX8 could do same thing.
> >
> > Yes, I prefer to drop "ARCH" for consistency.
> >
> >> I could revert to use CONFIG_IMX8, since you prefer it.
> >
> > Yes, that would be better.
> 
> 
> 
> Didn't we already discuss this and decide to be consistent with CONFIG_MX6 /7
> and the rest ?

NXP marketing requires us to use IMX for i.MX family for i.MX8. So
I choose CONFIG_IMX[x]. You could see NXP vendor tree all switch to
use CONFIG_XX_IMX[x].

I understand CONFIG_MX6/7, but this change only impacts i.MX8,
The MX6/7 could also be converted to use IMX I think, just a large patch by
some scripts.

Also no talking this config, all the device tree names has 'i' before mx, for i.MX family, including i.MX6/7.

So I prefer to use _IMX[x]_

Stefano, What do you think?

Thanks,
Peng.

> 
> 
> >
> >>> Also, for consistency: should CONFIG_MX8M be changed to
> CONFIG_IMX8M?
> >>
> >> Yes. CONFIG_MX8M also need to be changed to CONFIG_IMX8M, but not in
> this patchset.
> >
> > Correct.
> >
> > Thanks
> > _______________________________________________
> > U-Boot mailing list
> > U-Boot at lists.denx.de
> >
> https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.de
> nx.de%2Flistinfo%2Fu-boot&data=02%7C01%7Cpeng.fan%40nxp.com%7C
> 4c27504551844e8386a108d5f1931308%7C686ea1d3bc2b4c6fa92cd99c5c3016
> 35%7C0%7C0%7C636680536334072741&sdata=uEGuO5BIo2SI5v%2FSY6y
> aYnd2Q9I8ZMUzyhitJbgXQ34%3D&reserved=0
> >