Hi Stefano,
-----Original Message----- From: Stefano Babic [mailto:sbabic@denx.de] Sent: Thursday, April 13, 2017 2:24 PM To: Peng Fan peng.fan@nxp.com; Stefano Babic sbabic@denx.de Cc: van.freenix@gmail.com; u-boot@lists.denx.de; andre.przywara@arm.com; Albert ARIBAUD albert.u.boot@aribaud.net Subject: Re: [PATCH 1/2] imx-common: Enable ACTLR.SMP bit for i.MX cortex- a7 platforms
Hi Peng,
On 13/04/2017 05:05, Peng Fan wrote:
Anyway, it looks to me wrong to fix this in i.MX code, as this must be fixed for all Cortex-A7. It should be fixed in general ARM code.
And Andre has already sent patches for this (at least for sunxi), but with the opposite rule (default is set if not specific requested).
Thanks for correcting me. I see Andre's patch https://lists.denx.de/pipermail/u-boot/2017-February/279918.html
It only handles sunxi platform. If we need fix i.MX A7 cores, do you prefer I
set the SMP bit in s_init for i.MX6UL/7?
Or you prefer this should be handled by common ARM code?
As far as I understand, this issue is common to all A7 cores ==> fix should be in common ARM code.
Ok. Since there is no Cortex-A[x] revision check in U-Boot, need to first add the code. Then set ACTLR.SMP for A7 SMP.
Regards, Peng.