[PATCH v4 0/6] imx8mn-smm-s2/pro: Add iMX8MN BSH SMM S2 boards

[Stefano Babic]

On 11.04.22 18:03, Tom Rini wrote:
> On Mon, Apr 11, 2022 at 05:56:52PM +0200, Stefano Babic wrote:
>> Hi Ariel,
>>
>> On 11.04.22 13:56, Ariel D'Alessandro wrote:
>>> Hi,
>>>
>>> Gentle re-ping. What's blocking this?
>>> This has been reviewed and queued for some time already. Please, be
>>> clear on the status or what's the plan for this patchset.
>>>
>>
>> After first run, build is blocked by CI due to CONFIG_ that should be se in
>> configs/imx8mn_bsh_smm_s2_defconfig instead of
>> include/configs/imx8mn_bsh_smm_s2_common.h. I can recognize at least
>> CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR (and family), but it looks like
>> there are some more.
>>
>> See error here:
>>
>> 	https://source.denx.de/u-boot/custodians/u-boot-imx/-/jobs/420878
> 
> Since this is going to pop up a bit I suspect,
> https://source.denx.de/u-boot/custodians/u-boot-imx/-/jobs/420878#L24
> shows the shell command to run that gives "Unmigrated symbols found in
> include/configs/imx8mn_bsh_smm_s2_common.h".  And if instead of:
> NUM=`comm -123 --total --output-delimiter=, ${KSYMLST} ${KUSEDLST} | cut
> -d , -f 3`; if [[ $NUM -ne 0 ]]; then echo "Unmigrated symbols found in
> $CFG"; exit 1; fi
> 
> You do:
> comm -12 ${KSYMLST} ${KUSEDLST}

It is much better because CI does not report which CONFIG_ were found.

> 
> You'll see the whole list of unmigrated symbols.  Many of which are easy
> to migrate and it's just a matter of ./tools/moveconfig.py -y FOO and
> then sanity checking the output.
> 
> And a currently failing patchset is also maybe motivation to expand the
> CI check to note them too?  As I can certainly see how it would be good
> to have the what failed more visible.

Sure, at least to get the whole list of CONFIG_ that clash.

Regards,
Stefano

> 



-- 
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================