Hello Peng Fan
On Thu, Aug 08, 2024 at 03:16:54AM +0000, Peng Fan wrote:
> > Subject: [PATCH] imx: imx8: fix build when CONFIG_IMX_BOOTAUX is
> > set
> >
> > From: Max Krummenacher <max.krummenac...@toradex.com>
> >
> > Use correct function name.
> >
> > Fixes: e8cd1f60d964 ("imx: imx8: bootaux: Add i.MX8 M4 boot
> > support")
> > Signed-off-by: Max Krummenacher <max.krummenac...@toradex.com>
> > ---
> >
> > arch/arm/mach-imx/imx8/cpu.c | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/arch/arm/mach-imx/imx8/cpu.c b/arch/arm/mach-
> > imx/imx8/cpu.c index accba502e492..834aca82bcfd 100644
> > --- a/arch/arm/mach-imx/imx8/cpu.c
> > +++ b/arch/arm/mach-imx/imx8/cpu.c
> > @@ -258,14 +258,14 @@ int arch_auxiliary_core_up(u32 core_id,
> > ulong boot_private_data)
> > return -EIO;
> > }
> >
> > - if (!power_domain_lookup_name("audio_sai0", &pd))
> > {
> > + if (!imx8_power_domain_lookup_name("audio_sai0",
> > &pd)) {
>
> I think need to find a proper fix to this. This API is legacy API.
The current code does call a function which does not exist. Thus the
compile fails if enabling CONFIG_IMX_BOOTAUX for a i.MX 8X and likely
always did.
I think that would merrit fixing it now rather than waiting until the
more modern API is ported into U-Boot and the code using it changed to
the new API.
Alternatively I could remove the handling of the HiFi auxiliary core
from arch_auxiliary_core_up() and returning EINVAL if someone tried to
set that core up.
Regards
Max
>
> Thanks
> Peng
>
>
