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

[Patrice CHOTARD]

On 6/16/23 14:18, Marek Vasut wrote:
> 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.

Hi Marek

Unfortunately the pull request has already sent to the mailing list :-(
I will take care next time.

Patrice