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

Patrice CHOTARD patrice.chotard at foss.st.com
Tue May 9 08:16:16 CEST 2023


Hi Marek

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)

Reviewed-by: Patrice Chotard <patrice.chotard at foss.st.com>

Thanks
Patrice


More information about the U-Boot mailing list