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 <ma...@denx.de> > --- > Cc: Patrice Chotard <patrice.chot...@foss.st.com> > Cc: Patrick Delaunay <patrick.delau...@foss.st.com> > Cc: u-b...@dh-electronics.com > Cc: uboot-st...@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.chot...@foss.st.com> Thanks Patrice