> Subject: Re: [PATCH V3 0/3] ARM: imx: move cpu code to drivers/soc/imx > > On 5/20/2020 9:01 AM, Peng Fan wrote: > > From: Peng Fan <peng....@nxp.com> > > > > V3: > > Rebased to latest next tree > > Resolved the conflicts with vf610 soc patch > > > > V2: > > Keep i.MX1/2/3/5 cpu type for completness Correct return value in > > patch 1/3 use CONFIG_ARM to guard compile soc-imx.c in patch 3/3 > > > > V1: > > https://patchwork.kernel.org/cover/11433689/ > > RFC version : > > https://patchwork.kernel.org/cover/11336433/ > > > > Nothing changed in v1, just rename to formal patches > > > > Shawn, > > The original concern has been eliminated in RFC discussion, so this > > patchset is ready to be in next. > > Thanks. > > > > Follow i.MX8, move the soc device register code to drivers/soc/imx to > > simplify arch/arm/mach-imx/cpu.c > > > > I planned to use similar logic as soc-imx8m.c to restructure soc-imx.c > > and merged the two files into one. But not sure, so still keep the > > logic in cpu.c. > > > > There is one change is the platform devices are not under > > /sys/devices/soc0 after patch 1/4. Actually ARM64 platform devices are > > not under /sys/devices/soc0, such as i.MX8/8M. > > So it should not hurt to let the platform devices under platform dir. > > > > Peng Fan (3): > > ARM: imx: use device_initcall for imx_soc_device_init > > ARM: imx: move cpu definitions into a header > > soc: imx: move cpu code to drivers/soc/imx > > > This patch series has the side effect of LS1021A platform now reporting that > it's part of "i.MX family". > > caam driver relies on the SoC bus / SoC attributes (ID, family) to determine > if > it's running on an i.MX SoC or other (Layerscape, QorIQ).
Sorry to break LS1021A. But I wonder why i.MX code would affect LS? Regards, Peng. > > With this patch set, driver fails to probe on LS1021A: > [ 5.998928] caam 1700000.crypto: No clock data provided for i.MX SoC > [ 6.005306] caam: probe of 1700000.crypto failed with error -22 > > Horia