[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date: Mon, 6 Jun 2022 13:37:10 +1000
From: Greg Ungerer <gerg@...ux-m68k.org>
To: Randy Dunlap <rdunlap@...radead.org>, linux-kernel@...r.kernel.org
Cc: Geert Uytterhoeven <geert@...ux-m68k.org>,
linux-m68k@...ts.linux-m68k.org, uclinux-dev@...inux.org,
Angelo Dureghello <angelo@...nel-space.org>
Subject: Re: [PATCH] m68k: coldfire/device.c: protect FLEXCAN blocks
Hi Randy,
On 31/5/22 12:17, Randy Dunlap wrote:
> When CAN_FLEXCAN=y and M5441x is not set/enabled, there are build
> errors in coldfire/device.c:
>
> ../arch/m68k/coldfire/device.c:595:26: error: 'MCFFLEXCAN_BASE0' undeclared here (not in a function); did you mean 'MCFDMA_BASE0'?
> 595 | .start = MCFFLEXCAN_BASE0,
> ../arch/m68k/coldfire/device.c:596:43: error: 'MCFFLEXCAN_SIZE' undeclared here (not in a function)
> 596 | .end = MCFFLEXCAN_BASE0 + MCFFLEXCAN_SIZE,
> ../arch/m68k/coldfire/device.c:600:26: error: 'MCF_IRQ_IFL0' undeclared here (not in a function); did you mean 'MCF_IRQ_I2C0'?
> 600 | .start = MCF_IRQ_IFL0,
> ../arch/m68k/coldfire/device.c:605:26: error: 'MCF_IRQ_BOFF0' undeclared here (not in a function); did you mean 'MCF_IRQ_I2C0'?
> 605 | .start = MCF_IRQ_BOFF0,
> ../arch/m68k/coldfire/device.c:610:26: error: 'MCF_IRQ_ERR0' undeclared here (not in a function); did you mean 'MCF_IRQ_I2C0'?
> 610 | .start = MCF_IRQ_ERR0,
>
> Protect the FLEXCAN code blocks by checking if MCFFLEXCAN_SIZE
> is defined.
>
> Fixes: 35a9f9363a89 ("m68k: m5441x: add flexcan support")
> Signed-off-by: Randy Dunlap <rdunlap@...radead.org>
> Cc: Greg Ungerer <gerg@...ux-m68k.org>
> Cc: Geert Uytterhoeven <geert@...ux-m68k.org>
> Cc: linux-m68k@...ts.linux-m68k.org
> Cc: uclinux-dev@...inux.org
> Cc: Angelo Dureghello <angelo@...nel-space.org>
Thanks. Added to the m68knommu git tree, for-next branch.
Regards
Greg
> ---
> arch/m68k/coldfire/device.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> --- a/arch/m68k/coldfire/device.c
> +++ b/arch/m68k/coldfire/device.c
> @@ -581,7 +581,7 @@ static struct platform_device mcf_esdhc
> };
> #endif /* MCFSDHC_BASE */
>
> -#if IS_ENABLED(CONFIG_CAN_FLEXCAN)
> +#ifdef MCFFLEXCAN_SIZE
>
> #include <linux/can/platform/flexcan.h>
>
> @@ -620,7 +620,7 @@ static struct platform_device mcf_flexca
> .resource = mcf5441x_flexcan0_resource,
> .dev.platform_data = &mcf5441x_flexcan_info,
> };
> -#endif /* IS_ENABLED(CONFIG_CAN_FLEXCAN) */
> +#endif /* MCFFLEXCAN_SIZE */
>
> static struct platform_device *mcf_devices[] __initdata = {
> &mcf_uart,
> @@ -657,7 +657,7 @@ static struct platform_device *mcf_devic
> #ifdef MCFSDHC_BASE
> &mcf_esdhc,
> #endif
> -#if IS_ENABLED(CONFIG_CAN_FLEXCAN)
> +#ifdef MCFFLEXCAN_SIZE
> &mcf_flexcan0,
> #endif
> };
Powered by blists - more mailing lists