Re: [PATCH v5 1/3] Image size checks: Use HAS_SIZE_LIMIT consistently

[Philip Oberfichtner <kallikles28@gmail.com>]

Hi Marek,

Thanks for the review, I'll get back to it asap. In the meantime please
take note of my new e-mail address.

Best regards,
Philip Oberfichtner

On Wed, Dec 31, 2025 at 10:27:44PM +0100, Marek Vasut wrote:
> On 11/10/25 10:12 AM, Philip Oberfichtner wrote:
> 
> [...]
> 
> > diff --git a/common/spl/Kconfig b/common/spl/Kconfig
> > index 8dade2b501e..97c7bfdad47 100644
> > --- a/common/spl/Kconfig
> > +++ b/common/spl/Kconfig
> > @@ -36,19 +36,33 @@ config SPL_FRAMEWORK_BOARD_INIT_F
> >   	  - initialize the serial (preloader_console_init)
> >   	  Unless you want to provide your own board_init_f, you should say Y.
> > +config HAS_SPL_SIZE_LIMIT
> > +	bool "Enable size limit check for the SPL image"
> > +	depends on !COMPILE_TEST
> > +	depends on SPL
> > +	default y if ARCH_MX6
> > +	default y if ARCH_MVEBU && ARMADA_32BIT
> > +	default y if ARMADA_38X
> > +	default y if K3_DM_FW
> 
> K3_DM_FW is some driver, what you likely want here is:
> 
> default y if ARCH_K3 && CPU_V7R && !SOC_K3_AM642 && !SOC_K3_AM654 &&
> !CLK_TI_SCI && !TI_SCI_POWER_DOMAIN
> 
> But it seems am64x_evm_r5_defconfig below enables HAS_SPL_SIZE_LIMIT too ,
> so likely remove !SOC_K3_AM642 from the above, which will allow you to
> reduce the amount of configs/ file changes.
> 
> > +	default y if SOC_K3_AM625
> > +	default y if SOC_MT7621
> > +	default y if TARGET_SOCFPGA_GEN5
> 
> You could also add Xilinx Zynq/ZynqMP into the defaults here, which will
> further reduce the configs/ file changes.
> 
> The rest looks fine, thanks !