On Wed, Oct 21, 2026 at 01:42:00AM +0000, [email protected] wrote: > Hi Tom, > > > On Fri, Oct 16, 2026 at 10:43:41AM +0900, Takahiro Kuwano wrote: > > > > > The spi-nor-core.c contains optional logics like SFDP and manufcaturer > > > specific fixups. The spi-nor-ids.c has single large ID database. There > > > are many #ifdef directives in those files and that negatively affects > > > readability and maintainability. > > > > > > This series, as the first step of cleanup, move SFDP logic, IDs, and > > > fixups to newly source files. Reworking, renaming, and reformating > > > existing code is not a scope of this series and further cleanup will be > > > taken place in follow-up patches. > > > > > > Here is the code size impact when applying this series. > > > Measured by my AMD Zynq-7000 (ARM32) environment with SFDP and all > > > manufcaturers are enabled. > > > > > > SPI-NOR: > > > text data bss dec hex filename > > > BEFORE 720685 14084 0 734769 b3631 u-boot > > > AFTER 721827 14156 0 735983 b3aef u-boot > > > DELTA +1142 +72 > > > > > > TINY: > > > text data bss dec hex filename > > > BEFORE 77399 3128 248 80775 13b87 spl/u-boot-spl > > > AFTER 77589 3200 248 81037 13c8d spl/u-boot-spl > > > DELTA +190 +72 > > > > So this leads to imx28_btt3 and imx28_xea overflowing in CI on link. > > I don't see any errors in my local. Something different from CI config. > > imx28_btt3_defconfig: > text data bss dec hex filename > BEFORE 37947 2896 0 40843 9f8b spl/u-boot-spl > AFTER 38003 2912 0 40915 9fd3 spl/u-boot-spl > +56 +16 > > imx28_xea_defconfig: > text data bss dec hex filename > BEFORE 38004 2816 0 40820 9f74 spl/u-boot-spl > AFTER 38060 2832 0 40892 9fbc spl/u-boot-spl > +56 +16 > > Could you share the exact CI error or size report?
What toolchain are you using? CI uses the 14.2.0 ones from kernel.org: https://www.kernel.org/pub/tools/crosstool/files/bin/x86_64/14.2.0/ and the CI run is: https://git.u-boot-project.org/u-boot/u-boot/-/jobs/115650#L3163 -- Tom
signature.asc
Description: PGP signature
