Re: [PATCH] serial: samsung: earlycon support depends on CONFIG_SERIAL_SAMSUNG_CONSOLE

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Marek,

On Mon, Feb 2, 2015 at 3:17 PM, Marek Szyprowski
<m.szyprowski@xxxxxxxxxxx> wrote:
> EarlyCon support depends on serial console infrastructure, so the code
> implementing it should depend on CONFIG_SERIAL_SAMSUNG_CONSOLE.
>
> This patch fixes the following build break:
> CC [M]  drivers/tty/serial/samsung.o
> drivers/tty/serial/samsung.c:2468:1: error: expected declaration specifiers or ‘...’ before string constant
> drivers/tty/serial/samsung.c:2468:1: error: expected declaration specifiers or ‘...’ before ‘s3c2410_setup_earlycon’
> drivers/tty/serial/samsung.c:2487:1: error: expected declaration specifiers or ‘...’ before string constant
> drivers/tty/serial/samsung.c:2487:1: error: expected declaration specifiers or ‘...’ before ‘s3c2412_setup_earlycon’
> drivers/tty/serial/samsung.c:2488:1: error: expected declaration specifiers or ‘...’ before string constant
> drivers/tty/serial/samsung.c:2488:1: error: expected declaration specifiers or ‘...’ before ‘s3c2440_setup_earlycon’
> drivers/tty/serial/samsung.c:2489:1: error: expected declaration specifiers or ‘...’ before string constant
> drivers/tty/serial/samsung.c:2489:1: error: expected declaration specifiers or ‘...’ before ‘s3c6400_setup_earlycon’
> drivers/tty/serial/samsung.c:2506:1: error: expected declaration specifiers or ‘...’ before string constant
> drivers/tty/serial/samsung.c:2506:1: error: expected declaration specifiers or ‘...’ before ‘s5pv210_setup_earlycon’
> drivers/tty/serial/samsung.c:2507:1: error: expected declaration specifiers or ‘...’ before string constant
> drivers/tty/serial/samsung.c:2507:1: error: expected declaration specifiers or ‘...’ before ‘exynos4210_setup_earlycon’
> drivers/tty/serial/samsung.c:2468:1: warning: ‘s3c2410_setup_earlycon’ defined but not used [-Wunused-function]
> drivers/tty/serial/samsung.c:2487:1: warning: ‘s3c2412_setup_earlycon’ defined but not used [-Wunused-function]
> drivers/tty/serial/samsung.c:2488:1: warning: ‘s3c2440_setup_earlycon’ defined but not used [-Wunused-function]
> drivers/tty/serial/samsung.c:2489:1: warning: ‘s3c6400_setup_earlycon’ defined but not used [-Wunused-function]
> drivers/tty/serial/samsung.c:2506:1: warning: ‘s5pv210_setup_earlycon’ defined but not used [-Wunused-function]
> drivers/tty/serial/samsung.c:2507:1: warning: ‘exynos4210_setup_earlycon’ defined but not used [-Wunused-function]
> make[3]: *** [drivers/tty/serial/samsung.o] Error 1
>

Which config results in these build errors?

I just tried building arch/arm exynos_defconfig /wo
CONFIG_SERIAL_SAMSUNG_CONSOLE
and I do got below error [1]:

[1]:
drivers/built-in.o: In function `samsung_early_write':
/home/alim/linux-next/drivers/tty/serial/samsung.c:2442: undefined
reference to `uart_console_write'
make: *** [vmlinux] Error 1

and $SUBJECT patch does resolve [1],
So, Tested-by: Alim Akhtar <alim.akhtar@xxxxxxxxxxx>

but when I build arm64 _defconfig_ (with exynos7 enabled, manually)
/wo  CONFIG_SERIAL_SAMSUNG_CONSOLE,
build is successful, no build error like [1] above.

Also a build with multi_v7_defconfig does not result in any build error.
Probably this is to do with the way SERIAL_CORE_CONSOLE is handled in
arm64 defconfig and multi_v7_defconfig.
Not sure if there are other ways to fix this, lets see what others thinks.
In my view $SUBJECT patch is ok.


> Reported-by: kbuild test robot <fengguang.wu@xxxxxxxxx>
> Signed-off-by: Marek Szyprowski <m.szyprowski@xxxxxxxxxxx>
> ---
>  drivers/tty/serial/samsung.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/drivers/tty/serial/samsung.c b/drivers/tty/serial/samsung.c
> index a6bef8dcdc3c..bb73897809a1 100644
> --- a/drivers/tty/serial/samsung.c
> +++ b/drivers/tty/serial/samsung.c
> @@ -2403,6 +2403,7 @@ static struct platform_driver samsung_serial_driver = {
>
>  module_platform_driver(samsung_serial_driver);
>
> +#ifdef CONFIG_SERIAL_SAMSUNG_CONSOLE
>  /*
>   * Early console.
>   */
> @@ -2505,6 +2506,7 @@ OF_EARLYCON_DECLARE(exynos4210, "samsung,exynos4210-uart",
>                         s5pv210_early_console_setup);
>  EARLYCON_DECLARE(s5pv210, s5pv210_early_console_setup);
>  EARLYCON_DECLARE(exynos4210, s5pv210_early_console_setup);
> +#endif
>
>  MODULE_ALIAS("platform:samsung-uart");
>  MODULE_DESCRIPTION("Samsung SoC Serial port driver");
> --
> 1.9.2
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html



-- 
Regards,
Alim
--
To unsubscribe from this list: send the line "unsubscribe linux-serial" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux PPP]     [Linux FS]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Linmodem]     [Device Mapper]     [Linux Kernel for ARM]

  Powered by Linux