* [U-Boot] [PATCH 1/1] imx8: cpu: fix warning for cpu_imx_get_temp
@ 2019-05-13 10:46 Igor Opaniuk
2019-05-13 13:56 ` Stefan Agner
0 siblings, 1 reply; 4+ messages in thread
From: Igor Opaniuk @ 2019-05-13 10:46 UTC (permalink / raw)
To: u-boot
cpu_imx_get_temp() definition is wrapped with a ifdef macro,
therefore all function references should be also wrapped the same way
instead IS_ENABLED() usage.
Fix warning:
arch/arm/mach-imx/imx8/cpu.c: In function ‘cpu_imx_get_desc’:
arch/arm/mach-imx/imx8/cpu.c:612:40: warning: implicit declaration of
function ‘cpu_imx_get_temp’; did you mean ‘cpu_imx_get_desc’?
[-Wimplicit-function-declaration]
ret = snprintf(buf, size, " at %dC", cpu_imx_get_temp());
^~~~~~~~~~~~~~~~
cpu_imx_get_desc
cpu_imx_get_desc
Fixes: 82467cb217 ("imx8: cpu: get temperature when print cpu desc")
Signed-off-by: Igor Opaniuk <igor.opaniuk@toradex.com>
---
arch/arm/mach-imx/imx8/cpu.c | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/arch/arm/mach-imx/imx8/cpu.c b/arch/arm/mach-imx/imx8/cpu.c
index 12596c6387..616baed7cc 100644
--- a/arch/arm/mach-imx/imx8/cpu.c
+++ b/arch/arm/mach-imx/imx8/cpu.c
@@ -606,11 +606,11 @@ int cpu_imx_get_desc(struct udevice *dev, char *buf, int size)
ret = snprintf(buf, size, "NXP i.MX8%s Rev%s %s at %u MHz",
plat->type, plat->rev, plat->name, plat->freq_mhz);
- if (IS_ENABLED(CONFIG_IMX_SCU_THERMAL)) {
- buf = buf + ret;
- size = size - ret;
- ret = snprintf(buf, size, " at %dC", cpu_imx_get_temp());
- }
+#if defined(CONFIG_IMX_SCU_THERMAL)
+ buf = buf + ret;
+ size = size - ret;
+ ret = snprintf(buf, size, " at %dC", cpu_imx_get_temp());
+#endif
snprintf(buf + ret, size - ret, "\n");
--
2.17.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH 1/1] imx8: cpu: fix warning for cpu_imx_get_temp
2019-05-13 10:46 [U-Boot] [PATCH 1/1] imx8: cpu: fix warning for cpu_imx_get_temp Igor Opaniuk
@ 2019-05-13 13:56 ` Stefan Agner
2019-05-14 1:08 ` Peng Fan
0 siblings, 1 reply; 4+ messages in thread
From: Stefan Agner @ 2019-05-13 13:56 UTC (permalink / raw)
To: u-boot
Hi Igor,
On 13.05.2019 12:46, Igor Opaniuk wrote:
> cpu_imx_get_temp() definition is wrapped with a ifdef macro,
> therefore all function references should be also wrapped the same way
> instead IS_ENABLED() usage.
>
> Fix warning:
> arch/arm/mach-imx/imx8/cpu.c: In function ‘cpu_imx_get_desc’:
> arch/arm/mach-imx/imx8/cpu.c:612:40: warning: implicit declaration of
> function ‘cpu_imx_get_temp’; did you mean ‘cpu_imx_get_desc’?
> [-Wimplicit-function-declaration]
> ret = snprintf(buf, size, " at %dC", cpu_imx_get_temp());
> ^~~~~~~~~~~~~~~~
> cpu_imx_get_desc
> cpu_imx_get_desc
>
Using IS_ENABLED is typically preferred over ifdef since it assures that
the code is compileable even if CONFIG_IMX_SCU_THERMAL is not enabled.
I'd rather prefer we drop the the ifdef around cpu_imx_get_temp(). The
linker will remove the function in case CONFIG_IMX_SCU_THERMAL is not
enabled.
--
Stefan
> Fixes: 82467cb217 ("imx8: cpu: get temperature when print cpu desc")
> Signed-off-by: Igor Opaniuk <igor.opaniuk@toradex.com>
> ---
> arch/arm/mach-imx/imx8/cpu.c | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/arch/arm/mach-imx/imx8/cpu.c b/arch/arm/mach-imx/imx8/cpu.c
> index 12596c6387..616baed7cc 100644
> --- a/arch/arm/mach-imx/imx8/cpu.c
> +++ b/arch/arm/mach-imx/imx8/cpu.c
> @@ -606,11 +606,11 @@ int cpu_imx_get_desc(struct udevice *dev, char
> *buf, int size)
> ret = snprintf(buf, size, "NXP i.MX8%s Rev%s %s at %u MHz",
> plat->type, plat->rev, plat->name, plat->freq_mhz);
>
> - if (IS_ENABLED(CONFIG_IMX_SCU_THERMAL)) {
> - buf = buf + ret;
> - size = size - ret;
> - ret = snprintf(buf, size, " at %dC", cpu_imx_get_temp());
> - }
> +#if defined(CONFIG_IMX_SCU_THERMAL)
> + buf = buf + ret;
> + size = size - ret;
> + ret = snprintf(buf, size, " at %dC", cpu_imx_get_temp());
> +#endif
>
> snprintf(buf + ret, size - ret, "\n");
^ permalink raw reply [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH 1/1] imx8: cpu: fix warning for cpu_imx_get_temp
2019-05-13 13:56 ` Stefan Agner
@ 2019-05-14 1:08 ` Peng Fan
2019-05-14 9:24 ` Igor Opaniuk
0 siblings, 1 reply; 4+ messages in thread
From: Peng Fan @ 2019-05-14 1:08 UTC (permalink / raw)
To: u-boot
Hi Igor,
Not found your original mail, so reply Stefan's mail here.
> -----Original Message-----
> From: Stefan Agner [mailto:stefan at agner.ch]
> Sent: 2019年5月13日 21:56
> To: Igor Opaniuk <igor.opaniuk@toradex.com>
> Cc: u-boot at lists.denx.de; festevam at gmail.com; dl-uboot-imx
> <uboot-imx@nxp.com>; albert.u.boot at aribaud.net; Peng Fan
> <peng.fan@nxp.com>; agust at denx.de; Marcel Ziswiler
> <marcel.ziswiler@toradex.com>; marcel at ziswiler.com; Max Krummenacher
> <max.krummenacher@toradex.com>; sbabic at denx.de
> Subject: Re: [PATCH 1/1] imx8: cpu: fix warning for cpu_imx_get_temp
>
> Hi Igor,
>
> On 13.05.2019 12:46, Igor Opaniuk wrote:
> > cpu_imx_get_temp() definition is wrapped with a ifdef macro, therefore
> > all function references should be also wrapped the same way instead
> > IS_ENABLED() usage.
Are you trying to fix https://patchwork.ozlabs.org/patch/1095430/ ?
Thanks,
Peng
> >
> > Fix warning:
> > arch/arm/mach-imx/imx8/cpu.c: In function ‘cpu_imx_get_desc’:
> > arch/arm/mach-imx/imx8/cpu.c:612:40: warning: implicit declaration of
> > function ‘cpu_imx_get_temp’; did you mean ‘cpu_imx_get_desc’?
> > [-Wimplicit-function-declaration]
> > ret = snprintf(buf, size, " at %dC", cpu_imx_get_temp());
> > ^~~~~~~~~~~~~~~~
> > cpu_imx_get_desc
> > cpu_imx_get_desc
> >
>
> Using IS_ENABLED is typically preferred over ifdef since it assures that the
> code is compileable even if CONFIG_IMX_SCU_THERMAL is not enabled.
>
> I'd rather prefer we drop the the ifdef around cpu_imx_get_temp(). The linker
> will remove the function in case CONFIG_IMX_SCU_THERMAL is not enabled.
>
> --
> Stefan
>
> > Fixes: 82467cb217 ("imx8: cpu: get temperature when print cpu desc")
> > Signed-off-by: Igor Opaniuk <igor.opaniuk@toradex.com>
> > ---
> > arch/arm/mach-imx/imx8/cpu.c | 10 +++++-----
> > 1 file changed, 5 insertions(+), 5 deletions(-)
> >
> > diff --git a/arch/arm/mach-imx/imx8/cpu.c
> > b/arch/arm/mach-imx/imx8/cpu.c index 12596c6387..616baed7cc 100644
> > --- a/arch/arm/mach-imx/imx8/cpu.c
> > +++ b/arch/arm/mach-imx/imx8/cpu.c
> > @@ -606,11 +606,11 @@ int cpu_imx_get_desc(struct udevice *dev, char
> > *buf, int size)
> > ret = snprintf(buf, size, "NXP i.MX8%s Rev%s %s at %u MHz",
> > plat->type, plat->rev, plat->name, plat->freq_mhz);
> >
> > - if (IS_ENABLED(CONFIG_IMX_SCU_THERMAL)) {
> > - buf = buf + ret;
> > - size = size - ret;
> > - ret = snprintf(buf, size, " at %dC", cpu_imx_get_temp());
> > - }
> > +#if defined(CONFIG_IMX_SCU_THERMAL)
> > + buf = buf + ret;
> > + size = size - ret;
> > + ret = snprintf(buf, size, " at %dC", cpu_imx_get_temp()); #endif
> >
> > snprintf(buf + ret, size - ret, "\n");
^ permalink raw reply [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH 1/1] imx8: cpu: fix warning for cpu_imx_get_temp
2019-05-14 1:08 ` Peng Fan
@ 2019-05-14 9:24 ` Igor Opaniuk
0 siblings, 0 replies; 4+ messages in thread
From: Igor Opaniuk @ 2019-05-14 9:24 UTC (permalink / raw)
To: u-boot
Hi Peng,
Seems this is a glitch, I was working with the local forked tree (a
copy of the latest U-boot mainline master) + manually applied patch
series for Toradex Colibri iMX8QXP [1]/Apalis iMX8QM [2] support on
top of it (that are still pending in the U-boot ML),
but seems that some of your patches were also applied and I wasn't
aware about this (I thought that they were already in `next`).
So just ignore this patch. Thanks
[1]: https://patchwork.ozlabs.org/project/uboot/list/?series=105770
[2]: https://patchwork.ozlabs.org/project/uboot/list/?series=105774
On Tue, May 14, 2019 at 4:09 AM Peng Fan <peng.fan@nxp.com> wrote:
>
> Hi Igor,
>
> Not found your original mail, so reply Stefan's mail here.
>
> > -----Original Message-----
> > From: Stefan Agner [mailto:stefan at agner.ch]
> > Sent: 2019年5月13日 21:56
> > To: Igor Opaniuk <igor.opaniuk@toradex.com>
> > Cc: u-boot at lists.denx.de; festevam at gmail.com; dl-uboot-imx
> > <uboot-imx@nxp.com>; albert.u.boot at aribaud.net; Peng Fan
> > <peng.fan@nxp.com>; agust at denx.de; Marcel Ziswiler
> > <marcel.ziswiler@toradex.com>; marcel at ziswiler.com; Max Krummenacher
> > <max.krummenacher@toradex.com>; sbabic at denx.de
> > Subject: Re: [PATCH 1/1] imx8: cpu: fix warning for cpu_imx_get_temp
> >
> > Hi Igor,
> >
> > On 13.05.2019 12:46, Igor Opaniuk wrote:
> > > cpu_imx_get_temp() definition is wrapped with a ifdef macro, therefore
> > > all function references should be also wrapped the same way instead
> > > IS_ENABLED() usage.
>
> Are you trying to fix https://patchwork.ozlabs.org/patch/1095430/ ?
>
> Thanks,
> Peng
>
> > >
> > > Fix warning:
> > > arch/arm/mach-imx/imx8/cpu.c: In function ‘cpu_imx_get_desc’:
> > > arch/arm/mach-imx/imx8/cpu.c:612:40: warning: implicit declaration of
> > > function ‘cpu_imx_get_temp’; did you mean ‘cpu_imx_get_desc’?
> > > [-Wimplicit-function-declaration]
> > > ret = snprintf(buf, size, " at %dC", cpu_imx_get_temp());
> > > ^~~~~~~~~~~~~~~~
> > > cpu_imx_get_desc
> > > cpu_imx_get_desc
> > >
> >
> > Using IS_ENABLED is typically preferred over ifdef since it assures that the
> > code is compileable even if CONFIG_IMX_SCU_THERMAL is not enabled.
> >
> > I'd rather prefer we drop the the ifdef around cpu_imx_get_temp(). The linker
> > will remove the function in case CONFIG_IMX_SCU_THERMAL is not enabled.
> >
> > --
> > Stefan
> >
> > > Fixes: 82467cb217 ("imx8: cpu: get temperature when print cpu desc")
> > > Signed-off-by: Igor Opaniuk <igor.opaniuk@toradex.com>
> > > ---
> > > arch/arm/mach-imx/imx8/cpu.c | 10 +++++-----
> > > 1 file changed, 5 insertions(+), 5 deletions(-)
> > >
> > > diff --git a/arch/arm/mach-imx/imx8/cpu.c
> > > b/arch/arm/mach-imx/imx8/cpu.c index 12596c6387..616baed7cc 100644
> > > --- a/arch/arm/mach-imx/imx8/cpu.c
> > > +++ b/arch/arm/mach-imx/imx8/cpu.c
> > > @@ -606,11 +606,11 @@ int cpu_imx_get_desc(struct udevice *dev, char
> > > *buf, int size)
> > > ret = snprintf(buf, size, "NXP i.MX8%s Rev%s %s at %u MHz",
> > > plat->type, plat->rev, plat->name, plat->freq_mhz);
> > >
> > > - if (IS_ENABLED(CONFIG_IMX_SCU_THERMAL)) {
> > > - buf = buf + ret;
> > > - size = size - ret;
> > > - ret = snprintf(buf, size, " at %dC", cpu_imx_get_temp());
> > > - }
> > > +#if defined(CONFIG_IMX_SCU_THERMAL)
> > > + buf = buf + ret;
> > > + size = size - ret;
> > > + ret = snprintf(buf, size, " at %dC", cpu_imx_get_temp()); #endif
> > >
> > > snprintf(buf + ret, size - ret, "\n");
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> https://lists.denx.de/listinfo/u-boot
--
Best regards - Freundliche Grüsse - Meilleures salutations
Senior Development Engineer,
Igor Opaniuk
Toradex AG
Altsagenstrasse 5 | 6048 Horw/Luzern | Switzerland | T: +41 41 500 48
00 (main line)
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-05-14 9:24 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-05-13 10:46 [U-Boot] [PATCH 1/1] imx8: cpu: fix warning for cpu_imx_get_temp Igor Opaniuk
2019-05-13 13:56 ` Stefan Agner
2019-05-14 1:08 ` Peng Fan
2019-05-14 9:24 ` Igor Opaniuk
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox