[PATCH] SPL: stm32mp1: fix spl_mmc_boot_partition not defined

[Patrick DELAUNAY]

Hi,

> From: Richard Genoud <richard.genoud at posteo.net>
> Sent: mardi 13 octobre 2020 14:25
> 
> Hi Patrick,
> 
> Le 13/10/2020 à 14:18, Patrick DELAUNAY a écrit :
> > Hi Richard,
> >
> >> From: Richard Genoud <richard.genoud at posteo.net>
> >> Sent: lundi 12 octobre 2020 16:11
> >>
> >> spl_mmc_boot_partition is only defined when
> >> CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_PARTITION is defined.
> >>
> >> Signed-off-by: Richard Genoud <richard.genoud at posteo.net>
> >> ---
> >>   arch/arm/mach-stm32mp/spl.c | 2 ++
> >>   1 file changed, 2 insertions(+)
> >>
> >
> > Reviewed-by: Patrick Delaunay <patrick.delaunay at st.com>
> >
> > Thanks for this missing check in spl.
> >
> > NB: after check, it is possible to IS_ENABLED to prevent #ifdef here
> >
> > <unknown>:0: warning: Use 'if (IS_ENABLED(CONFIG...))' instead of '#if
> > or #ifdef' where possible
> 
> I don't think it's possible to use "if (IS_ENABLED(CONFIG...))"
> since we have to remove the whole function, not just some code in it.

Normally the unused function is remove by linker without compilation flag...

But after check on my side it is not possible to use IS_ENABLED(CONFIG_ here...  
because when the flags is absent, the 2 used CONFIG_ are also absent:
- CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_PARTITION
- CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_PARTITION_MMC2

And that cause compilation issue

Sorry for first feedback, I will take the patch in my next pull request.

 
> Regards,
> Richard

Regards
Patrick