[U-Boot] [PATCH] test/py: Add option to skip SPL signature checking

Michal Simek michal.simek at xilinx.com
Tue Feb 23 10:37:49 CET 2016


On 22.2.2016 18:29, Stephen Warren wrote:
> On 02/22/2016 05:52 AM, Michal Simek wrote:
>> Provide user option to skip SPL signature verification for cases where
>> u-boot is build with SPL support but full U-Boot is also verified
>> without SPL.
>> If you want to support this feature please add env__spl_skipped = True
>> to your boardenv configuration file.
> 
> Nit: Blank line between those paragraphs?

fixed

> 
>>
>> For example Xilinx Zynq is using this feature where the same u-boot
>> binary is checked with SPL and without SPL(with FSBL).
> 
> Nit: U-Boot not u-boot?

fixed.

> 
>> diff --git a/test/py/u_boot_console_base.py
>> b/test/py/u_boot_console_base.py
> 
>>               config_spl = bcfg.get('config_spl', 'n') == 'y'
>>               config_spl_serial_support =
>> bcfg.get('config_spl_serial_support',
>>                                                    'n') == 'y'
>> -            if config_spl and config_spl_serial_support:
>> +            config_spl_skip = self.config.env.get('env__spl_skipped',
>> +                                                  False)
> 
> That's not a config file (.config or autoconf.mk), but a boardenv value.
> I'd suggest renaming it boardenv_spl_skipped or env_spl_skipped.

env_spl_skipped is used now.
Currently env__net_dhcp_server names are used. It is also board specific.
Maybe make sense to just rename all these to boardenv_XXX.

> Tested-by: Stephen Warren <swarren at nvidia.com>
> 
> Once renamed,
> Acked-by: Stephen Warren <swarren at nvidia.com>

I have added to v2 but feel free to reply if I should do any change.

Thanks,
Michal


More information about the U-Boot mailing list