[PATCH v3] ARM: imx: add smp support for imx7d

[Fabio Estevam]

Hi Shawn,

On 25/05/2021 21:14, Fabio Estevam wrote:
> From: Anson Huang <b20788 at freescale.com>
> 
> Add SMP support for i.MX7D, including CPU hotplug support, for
> systems where TFA is not present.
> 
> The motivation for bringing up the second i.MX7D core inside the kernel
> is that legacy vendor bootloaders usually do not implement PSCI 
> support.
> 
> This is a significant blocker for systems in the field that are running 
> old
> bootloader versions to upgrade to a modern mainline kernel version, as 
> only
> one CPU of the i.MX7D would be brought up.
> 
> Bring up the second i.MX7D core inside the kernel to make the migration
> path to mainline kernel easier for the existing iMX7D users.
> 
> Signed-off-by: Anson Huang <b20788 at freescale.com>
> Signed-off-by: Arulpandiyan Vadivel <arulpandiyan_vadivel at mentor.com>
> # Fix merge conflicts
> Signed-off-by: Leonard Crestez <leonard.crestez at nxp.com>
> Signed-off-by: Marek Vasut <marex at denx.de> # heavy cleanup
> Signed-off-by: Fabio Estevam <festevam at denx.de>
> ---
> Changes since v2:
> - Removed the DT comment part from the commit log - Shawn
> - Added comments in the code explaining the motivation of bringing
> up the second core inside the kernel - Shawn
> - Fixed checkpatch warnings, except the one regarding NR_CPUS,
> which Russell King explained that the current usage is correct - Shawn
> - Made imx_gpcv2_set_m_core_pgc() static - Shawn
> - Read PDN/PUP register after imx_gpcv2_set_m_core_pgc() - Shawn
> - Removed WARN_ON() occurrences - Shawn
> - Added comments in the commit log explaining the motivation of 
> bringing
> up the second core inside the kernel.

I tried to address all your previous comments about this patch.

Do you think this version is OK? Could it land in 5.14?

Thanks,

Fabio Estevam