[PATCH] ARM: stm32: Use __section(".data") with dot in the section name on DHSOM

Marek Vasut marex at denx.de
Fri Jun 16 14:18:57 CEST 2023


On 6/16/23 13:46, Patrice CHOTARD wrote:
> 
> 
> On 5/4/23 21:52, Marek Vasut wrote:
>> The correct specifier of the section is ".data" and not "data",
>> use the former to place the variables in ".data" section.
>>
>> Fixes: 731fd50e27f ("ARM: stm32: Implement board coding on AV96")
>> Fixes: 92ca0f7446c ("ARM: dts: stm32: Synchronize DDR setttings on DH SoMs")
>> Signed-off-by: Marek Vasut <marex at denx.de>
>> ---
>> Cc: Patrice Chotard <patrice.chotard at foss.st.com>
>> Cc: Patrick Delaunay <patrick.delaunay at foss.st.com>
>> Cc: u-boot at dh-electronics.com
>> Cc: uboot-stm32 at st-md-mailman.stormreply.com
>> ---
>>   board/dhelectronics/dh_stm32mp1/board.c | 6 +++---
>>   1 file changed, 3 insertions(+), 3 deletions(-)
>>
>> diff --git a/board/dhelectronics/dh_stm32mp1/board.c b/board/dhelectronics/dh_stm32mp1/board.c
>> index a9a2965a386..a0cdf83ea71 100644
>> --- a/board/dhelectronics/dh_stm32mp1/board.c
>> +++ b/board/dhelectronics/dh_stm32mp1/board.c
>> @@ -185,9 +185,9 @@ int checkboard(void)
>>   }
>>   
>>   #ifdef CONFIG_BOARD_EARLY_INIT_F
>> -static u8 brdcode __section("data");
>> -static u8 ddr3code __section("data");
>> -static u8 somcode __section("data");
>> +static u8 brdcode __section(".data");
>> +static u8 ddr3code __section(".data");
>> +static u8 somcode __section(".data");
>>   static u32 opp_voltage_mv __section(".data");
>>   
>>   static void board_get_coding_straps(void)
> 
> Applied to u-boot-stm/next

Since this is a bugfix, should be for current.


More information about the U-Boot mailing list