Re: [RFC PATCH] soc: imx: Try harder to get imq8mq SoC revisions

From: Lucas Stach
Date: Wed May 22 2019 - 09:43:36 EST


Am Mittwoch, den 22.05.2019, 13:30 +0000 schrieb Leonard Crestez:
> On 22.05.2019 16:13, Guido GÃnther wrote:
> > Subject: Re: [RFC PATCH] soc: imx: Try harder to get imq8mq SoC revisions
>
> Fixed subject
>
> > On Wed, May 08, 2019 at 02:40:18PM +0200, Guido GÃnther wrote:
> > > Thanks for your comments. Let's try s.th. different then: identify by
> > > bootrom, ocotop and anatop and fall back to ATF afterwards (I'll split
> > > out the DT part and add binding docs if this makes sense). I'm also
> > > happy to drop the whole ATF logic until mailine ATF catched up:
> > >
> > > The mainline ATF doesn't currently support the FSL_SIP_GET_SOC_INFO call
> > > nor does it have the code to identify different imx8mq SOC revisions so
> > > mimic what NXPs ATF does here.
> >
> > Does this makes sense? If so I'll send this out as a series.
>
> Mainline ATF has recently caught up:
>
> https://git.trustedfirmware.org/TF-A/trusted-firmware-a.git/tree/plat/imx/imx8m/imx8mq/imx8mq_bl31_setup.c#n52
>
> > > As a fallback use ATF so we can identify new revisions once it gains
> > > support or when using NXPs ATF.
> >
> > I'm also fine with dropping the ATF part if we don't want to depend on
> > it in mainline.
>
> Linux arm64 depends on ATF to implement power management via PSCI:Â
> hotplug cpuidle and suspend.
>
> It is not clear why Linux would avoid other services and insist onÂ
> reimplementing hardware workarounds.

I fully agree. We should not duplicate functionality between ATF and
Linux kernel. If the mainline ATF provides the facilities to do the SoC
identification the kernel should use them as the sole source to get
this information.

Regards,
Lucas