diff options
author | Sascha Hauer <s.hauer@pengutronix.de> | 2022-11-07 11:33:40 +0100 |
---|---|---|
committer | Sascha Hauer <s.hauer@pengutronix.de> | 2022-11-08 11:05:13 +0100 |
commit | 6ccdd298ab6925749803533a6ce27db47b45adf2 (patch) | |
tree | b3200601c42f55ab145b146cde3028b75e654840 | |
parent | d974b8e8f0ecd67507a40f97d1da59f0e4f9ec9d (diff) | |
download | barebox-6ccdd298ab6925749803533a6ce27db47b45adf2.tar.gz barebox-6ccdd298ab6925749803533a6ce27db47b45adf2.tar.xz |
ARM: i.MX: romapi: Hide using NULL pointers from gcc
The i.MX ROM API lies in the NULL page. Accessing that results in
compiler warnings like:
include/asm-generic/io.h:27:51: warning: array subscript 0 is outside array bounds of 'volatile unsigned char[0]' [-Warray-bounds]
Use OPTIMIZER_HIDE_VAR() to hide using NULL pointers from the compiler.
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Link: https://lore.barebox.org/20221107103340.3913098-1-s.hauer@pengutronix.de
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
-rw-r--r-- | arch/arm/mach-imx/include/mach/imx8mq.h | 7 | ||||
-rw-r--r-- | arch/arm/mach-imx/romapi.c | 2 |
2 files changed, 7 insertions, 2 deletions
diff --git a/arch/arm/mach-imx/include/mach/imx8mq.h b/arch/arm/mach-imx/include/mach/imx8mq.h index d02c5f23cf..e58a3935f7 100644 --- a/arch/arm/mach-imx/include/mach/imx8mq.h +++ b/arch/arm/mach-imx/include/mach/imx8mq.h @@ -58,10 +58,13 @@ static inline int imx8mq_cpu_revision(void) { void __iomem *anatop = IOMEM(MX8MQ_ANATOP_BASE_ADDR); void __iomem *ocotp = IOMEM(MX8MQ_OCOTP_BASE_ADDR); + void __iomem *rom = IOMEM(0x0); uint32_t revision = FIELD_GET(DIGPROG_MINOR, readl(anatop + MX8MQ_ANATOP_DIGPROG)); uint32_t rom_version; + OPTIMIZER_HIDE_VAR(rom); + if (revision != IMX_CHIP_REV_1_0) return revision; /* @@ -74,9 +77,9 @@ static inline int imx8mq_cpu_revision(void) * For B0 chip, the DIGPROG is not updated, still TO1.0. * we have to check ROM version further */ - rom_version = readb(IOMEM(IMX8MQ_ROM_VERSION_A0)); + rom_version = readb(IOMEM(rom + IMX8MQ_ROM_VERSION_A0)); if (rom_version != IMX_CHIP_REV_1_0) { - rom_version = readb(IOMEM(IMX8MQ_ROM_VERSION_B0)); + rom_version = readb(IOMEM(rom + IMX8MQ_ROM_VERSION_B0)); if (rom_version >= IMX_CHIP_REV_2_0) revision = IMX_CHIP_REV_2_0; } diff --git a/arch/arm/mach-imx/romapi.c b/arch/arm/mach-imx/romapi.c index 5d00d71154..56bf7e135f 100644 --- a/arch/arm/mach-imx/romapi.c +++ b/arch/arm/mach-imx/romapi.c @@ -35,6 +35,8 @@ int imx8mp_bootrom_load_image(void) { struct rom_api *rom_api = (void *)0x980; + OPTIMIZER_HIDE_VAR(rom_api); + return imx8m_bootrom_load_image(rom_api); } |