From: Marco Felsch <m.felsch@pengutronix.de>
To: Ahmad Fatoum <a.fatoum@pengutronix.de>
Cc: barebox@lists.infradead.org
Subject: Re: [PATCH 2/2] treewide: remove no-longer needed IMD_USED_OF
Date: Mon, 20 Feb 2023 18:31:34 +0100 [thread overview]
Message-ID: <20230220173134.ddunj3vgpaya6cf7@pengutronix.de> (raw)
In-Reply-To: <20230217203733.2926811-2-a.fatoum@pengutronix.de>
Hi Ahmad,
On 23-02-17, Ahmad Fatoum wrote:
> Now that a __dtb* reference automatically pulls in the IMD tag if
> support is compiled in, we can drop all IMD_USED_OF and turn the macro
> into a no-op until we remove it completely.
>
> Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
> ---
> arch/arm/boards/innocomm-imx8mm-wb15/lowlevel.c | 2 --
> arch/arm/boards/ls1046ardb/lowlevel.c | 2 --
> arch/arm/boards/nxp-imx8mm-evk/lowlevel.c | 2 --
> arch/arm/boards/nxp-imx8mp-evk/lowlevel.c | 2 --
> arch/arm/boards/nxp-imx8mq-evk/lowlevel.c | 2 --
> arch/arm/boards/phytec-som-imx6/lowlevel.c | 1 -
> arch/arm/boards/protonic-imx8m/lowlevel-prt8mm.c | 2 --
> arch/arm/boards/tqmls1046a/lowlevel.c | 2 --
> arch/arm/boards/zii-imx8mq-dev/lowlevel.c | 3 ---
> include/image-metadata.h | 5 +----
> 10 files changed, 1 insertion(+), 22 deletions(-)
>
> diff --git a/arch/arm/boards/innocomm-imx8mm-wb15/lowlevel.c b/arch/arm/boards/innocomm-imx8mm-wb15/lowlevel.c
> index f7753bd32671..f4c6239551f9 100644
> --- a/arch/arm/boards/innocomm-imx8mm-wb15/lowlevel.c
> +++ b/arch/arm/boards/innocomm-imx8mm-wb15/lowlevel.c
> @@ -74,8 +74,6 @@ void innocomm_wb15_power_init_board(void)
>
> ENTRY_FUNCTION(start_innocomm_wb15_evk, r0, r1, r2)
> {
> - IMD_USED_OF(imx8mm_innocomm_wb15_evk);
Nit: after removing this we can also remove the header: image-metadata.h.
Regards,
Marco
> -
> imx8mm_cpu_lowlevel_init();
>
> relocate_to_current_adr();
> diff --git a/arch/arm/boards/ls1046ardb/lowlevel.c b/arch/arm/boards/ls1046ardb/lowlevel.c
> index 9eae032377d7..fdb5202182db 100644
> --- a/arch/arm/boards/ls1046ardb/lowlevel.c
> +++ b/arch/arm/boards/ls1046ardb/lowlevel.c
> @@ -202,8 +202,6 @@ static noinline __noreturn void ls1046ardb_r_entry(unsigned long memsize)
> debug_ll_init();
> ls1046a_init_lowlevel();
>
> - IMD_USED_OF(fsl_ls1046a_rdb);
> -
> i2c = ls1046_i2c_init(IOMEM(LSCH2_I2C1_BASE_ADDR));
> ret = spd_read_eeprom(i2c, 0x51, &spd_eeprom, SPD_MEMTYPE_DDR4);
> if (ret) {
> diff --git a/arch/arm/boards/nxp-imx8mm-evk/lowlevel.c b/arch/arm/boards/nxp-imx8mm-evk/lowlevel.c
> index 5861d8d8bece..6381ed3a3690 100644
> --- a/arch/arm/boards/nxp-imx8mm-evk/lowlevel.c
> +++ b/arch/arm/boards/nxp-imx8mm-evk/lowlevel.c
> @@ -159,7 +159,5 @@ ENTRY_FUNCTION(start_nxp_imx8mm_evk, r0, r1, r2)
> relocate_to_current_adr();
> setup_c();
>
> - IMD_USED_OF(imx8mm_evk);
> -
> nxp_imx8mm_evk_start();
> }
> diff --git a/arch/arm/boards/nxp-imx8mp-evk/lowlevel.c b/arch/arm/boards/nxp-imx8mp-evk/lowlevel.c
> index d6fc32e65b61..88764b9f6549 100644
> --- a/arch/arm/boards/nxp-imx8mp-evk/lowlevel.c
> +++ b/arch/arm/boards/nxp-imx8mp-evk/lowlevel.c
> @@ -140,7 +140,5 @@ ENTRY_FUNCTION(start_nxp_imx8mp_evk, r0, r1, r2)
> relocate_to_current_adr();
> setup_c();
>
> - IMD_USED_OF(imx8mp_evk);
> -
> nxp_imx8mp_evk_start();
> }
> diff --git a/arch/arm/boards/nxp-imx8mq-evk/lowlevel.c b/arch/arm/boards/nxp-imx8mq-evk/lowlevel.c
> index 0c9f6345ff30..f0020e2a1cc3 100644
> --- a/arch/arm/boards/nxp-imx8mq-evk/lowlevel.c
> +++ b/arch/arm/boards/nxp-imx8mq-evk/lowlevel.c
> @@ -107,7 +107,5 @@ ENTRY_FUNCTION(start_nxp_imx8mq_evk, r0, r1, r2)
> relocate_to_current_adr();
> setup_c();
>
> - IMD_USED_OF(imx8mq_evk);
> -
> nxp_imx8mq_evk_start();
> }
> diff --git a/arch/arm/boards/phytec-som-imx6/lowlevel.c b/arch/arm/boards/phytec-som-imx6/lowlevel.c
> index 62a1c8de7309..31c1e056aa1f 100644
> --- a/arch/arm/boards/phytec-som-imx6/lowlevel.c
> +++ b/arch/arm/boards/phytec-som-imx6/lowlevel.c
> @@ -75,7 +75,6 @@ static void __noreturn start_imx6_phytec_common(uint32_t size,
> extern char __dtb_##fdt_name##_start[]; \
> \
> IMD_USED(physom_mx6_memsize_##memory_size); \
> - IMD_USED_OF(fdt_name); \
> \
> start_imx6_phytec_common(memory_size, do_early_uart_config, \
> __dtb_##fdt_name##_start); \
> diff --git a/arch/arm/boards/protonic-imx8m/lowlevel-prt8mm.c b/arch/arm/boards/protonic-imx8m/lowlevel-prt8mm.c
> index bfae39ea522e..56fc24b2d6b7 100644
> --- a/arch/arm/boards/protonic-imx8m/lowlevel-prt8mm.c
> +++ b/arch/arm/boards/protonic-imx8m/lowlevel-prt8mm.c
> @@ -87,7 +87,5 @@ ENTRY_FUNCTION(start_prt_prt8mm, r0, r1, r2)
> relocate_to_current_adr();
> setup_c();
>
> - IMD_USED_OF(imx8mm_prt8mm);
> -
> prt_prt8mm_start();
> }
> diff --git a/arch/arm/boards/tqmls1046a/lowlevel.c b/arch/arm/boards/tqmls1046a/lowlevel.c
> index 99dcf1eff7ab..17da09eb515d 100644
> --- a/arch/arm/boards/tqmls1046a/lowlevel.c
> +++ b/arch/arm/boards/tqmls1046a/lowlevel.c
> @@ -109,8 +109,6 @@ static noinline __noreturn void tqmls1046a_r_entry(void)
> udelay(500);
> putc_ll('>');
>
> - IMD_USED_OF(fsl_tqmls1046a_mbls10xxa);
> -
> fsl_ddr_set_memctl_regs(&ddrc[0], 0);
>
> ls1046a_errata_post_ddr();
> diff --git a/arch/arm/boards/zii-imx8mq-dev/lowlevel.c b/arch/arm/boards/zii-imx8mq-dev/lowlevel.c
> index cf53fb2defd6..d2b677d3890f 100644
> --- a/arch/arm/boards/zii-imx8mq-dev/lowlevel.c
> +++ b/arch/arm/boards/zii-imx8mq-dev/lowlevel.c
> @@ -201,8 +201,5 @@ ENTRY_FUNCTION(start_zii_imx8mq_dev, r0, r1, r2)
> relocate_to_current_adr();
> setup_c();
>
> - IMD_USED_OF(imx8mq_zii_ultra_rmb3);
> - IMD_USED_OF(imx8mq_zii_ultra_zest);
> -
> zii_imx8mq_dev_start();
> }
> diff --git a/include/image-metadata.h b/include/image-metadata.h
> index bf4e08d98a47..619154dcbe11 100644
> --- a/include/image-metadata.h
> +++ b/include/image-metadata.h
> @@ -156,10 +156,7 @@ static inline void imd_used(const void *unused)
>
> #define IMD_USED(_name) \
> imd_used(&__barebox_imd_##_name)
> -#define IMD_USED_OF(_name) ({ \
> - extern char __barebox_imd_OF_ ## _name[]; \
> - imd_used(&__barebox_imd_OF_ ## _name); \
> - })
> +#define IMD_USED_OF(_name) do { } while (0)
>
> #endif /* __BAREBOX__ */
>
> --
> 2.30.2
>
>
>
next prev parent reply other threads:[~2023-02-20 17:33 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-02-17 20:37 [PATCH 1/2] gen-dtb-s: reference OF IMD entries automatically Ahmad Fatoum
2023-02-17 20:37 ` [PATCH 2/2] treewide: remove no-longer needed IMD_USED_OF Ahmad Fatoum
2023-02-20 17:31 ` Marco Felsch [this message]
2023-02-21 9:36 ` [PATCH 1/2] gen-dtb-s: reference OF IMD entries automatically Sascha Hauer
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20230220173134.ddunj3vgpaya6cf7@pengutronix.de \
--to=m.felsch@pengutronix.de \
--cc=a.fatoum@pengutronix.de \
--cc=barebox@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox