[PATCH v2 7/7] configs: stm32f746-disco: Remove CONFIG_SYS_UBOOT_START flag

Patrice CHOTARD patrice.chotard at foss.st.com
Mon Sep 26 17:56:53 CEST 2022



On 9/26/22 15:02, Patrick DELAUNAY wrote:
> Hi,
> 
> On 9/20/22 18:04, Patrice Chotard wrote:
>> By pressing "c" key during SPL execution, we force U-boot execution
>> instead of a kernel XIP image.
>>
>> This fixes a hard fault when booting stm32f746-disco in SPL with "c"
>> key pressed during SPL execution.
>>
>> U-Boot SPL 2022.10-rc5-00009-g40d02baa91 (Sep 20 2022 - 17:21:21 +0200)
>> Trying to boot from XIP
>> Hard fault
>> pc : 080083fc    lr : 08000d1b    xPSR : 21000000
>> r12 : 2004f108   r3 : 080083fd    r2 : 00000028
>> r1 : 2004f0c8    r0 : 2004f0e4
>> Resetting CPU ...
>>
>> This is due to SYS_UBOOT_START flag set to 0x080083FD which is not correct.
>> If unset, SYS_UBOOT_START is set by default to CONFIG_SYS_TEXT_BASE
>> which match with our requirement.
>>
>> Signed-off-by: Patrice Chotard <patrice.chotard at foss.st.com>
>> ---
>>
>> (no changes since v1)
>>
>>   include/configs/stm32f746-disco.h | 1 -
>>   1 file changed, 1 deletion(-)
>>
>> diff --git a/include/configs/stm32f746-disco.h b/include/configs/stm32f746-disco.h
>> index 64c1bc7d43..b0ec226114 100644
>> --- a/include/configs/stm32f746-disco.h
>> +++ b/include/configs/stm32f746-disco.h
>> @@ -34,7 +34,6 @@
>>               BOOTENV
>>     #define CONFIG_SYS_MONITOR_LEN        (512 * 1024)
>> -#define CONFIG_SYS_UBOOT_START        0x080083FD
>>   #define CONFIG_SYS_UBOOT_BASE        (CONFIG_SYS_FLASH_BASE + \
>>                        CONFIG_SPL_PAD_TO)
>>   
> 
> 
> 
> Reviewed-by: Patrick Delaunay <patrick.delaunay at foss.st.com>
> 
> Thanks
> Patrick
> 
Applied to u-boot-stm32/next

Thanks


More information about the U-Boot mailing list