Re: [PATCH] ARM: autcpu12: Fix config check in autcpu12_init_late()
Alexander Shiyan
shc_work at mail.ru
Sat Mar 23 01:22:20 EDT 2013
> The 'CONFIG_' prefix is not implicit in IS_ENABLED().
>
> Signed-off-by: Ben Hutchings <ben at decadent.org.uk>
> Cc: Alexander Shiyan <shc_work at mail.ru>
> Cc: Olof Johansson <olof at lixom.net>
> Cc: Paul Bolle <pebolle at tiscali.nl>
> Cc: <stable at vger.kernel.org> # 3.8+
> ---
> Found by inspection. This is not even compile-tested.
>
> Ben.
>
> arch/arm/mach-clps711x/board-autcpu12.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm/mach-clps711x/board-autcpu12.c b/arch/arm/mach-clps711x/board-autcpu12.c
> index f385847..8138523 100644
> --- a/arch/arm/mach-clps711x/board-autcpu12.c
> +++ b/arch/arm/mach-clps711x/board-autcpu12.c
> @@ -158,7 +158,8 @@ static void __init autcpu12_init(void)
>
> static void __init autcpu12_init_late(void)
> {
> - if (IS_ENABLED(MTD_NAND_GPIO) && IS_ENABLED(GPIO_GENERIC_PLATFORM)) {
> + if (IS_ENABLED(CONFIG_MTD_NAND_GPIO) &&
> + IS_ENABLED(CONFIG_GPIO_GENERIC_PLATFORM)) {
> /* We are need both drivers to handle NAND */
> platform_device_register(&autcpu12_nand_pdev);
> }
Oops. Right.
And I think checks can be removed completely.
---
More information about the linux-arm-kernel
mailing list