linux-kernel - Re: [PATCH] serial: Add Milbeaut serial control

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAK8P3a0Sx5vxbserW27KMPYnpLabPkGJZ_2iuW9y082oHiO+aQ@mail.gmail.com>
Date:   Tue, 26 Mar 2019 10:57:08 +0100
From:   Arnd Bergmann <arnd@...db.de>
To:     Sugaya Taichi <sugaya.taichi@...ionext.com>
Cc:     Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Jiri Slaby <jslaby@...e.com>,
        Takao Orito <orito.takao@...ionext.com>,
        Kazuhiro Kasai <kasai.kazuhiro@...ionext.com>,
        Shinji Kanematsu <kanematsu.shinji@...ionext.com>,
        Jassi Brar <jaswinder.singh@...aro.org>,
        Masami Hiramatsu <masami.hiramatsu@...aro.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        linux-serial@...r.kernel.org
Subject: Re: [PATCH] serial: Add Milbeaut serial control

On Tue, Mar 26, 2019 at 10:13 AM Sugaya Taichi
<sugaya.taichi@...ionext.com> wrote:

>
> diff --git a/drivers/tty/serial/Kconfig b/drivers/tty/serial/Kconfig
> index 72966bc..961519b 100644
> --- a/drivers/tty/serial/Kconfig
> +++ b/drivers/tty/serial/Kconfig
> @@ -1582,6 +1582,32 @@ config SERIAL_RDA_CONSOLE
>           Say 'Y' here if you wish to use the RDA8810PL UART as the system
>           console. Only earlycon is implemented currently.
>
> +config SERIAL_MILBEAUT_USIO
> +       tristate "Milbeaut USIO/UART serial port support"
> +       depends on ARCH_MILBEAUT
> +       default y
> +       select SERIAL_CORE
> +       help
> +         This selects the USIO/UART IP found in Socionext Milbeaut SoCs.

In general, it would be good to make it possible to build this on
other architectures for better compile test coverage, e.g. like

config SERIAL_MILBEAUT_USIO
     tristate "Milbeaut USIO/UART serial port support"
     depends on ARCH_MILBEAUT || COMPILE_TEST
     default ARCH_MILBEAUT

You might need additional dependencies like 'depends on OF'
to make this work reliably.

        Arnd