Hi Troy, > -----Original Message----- > From: Troy Kisky [mailto:troy.ki...@boundarydevices.com] > Sent: 2018年7月25日 2:27 > To: Fabio Estevam <feste...@gmail.com>; Peng Fan <peng....@nxp.com> > Cc: Fabio Estevam <fabio.este...@nxp.com>; U-Boot-Denx > <u-boot@lists.denx.de>; dl-linux-imx <linux-...@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....@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@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 > > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot