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

[Marek Vasut]

On 6/16/23 14:23, Patrice CHOTARD wrote:
> 
> 
> 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.

Can you send another MR ?