[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20191211121335.GA513966@kroah.com>
Date: Wed, 11 Dec 2019 13:13:35 +0100
From: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
To: "Enrico Weigelt, metux IT consult" <lkml@...ux.net>
Cc: linux-serial@...r.kernel.org, Kukjin Kim <kgene@...nel.org>,
Krzysztof Kozlowski <krzk@...nel.org>,
Hyunki Koo <kkoos00@...er.com>,
HYUN-KI KOO <hyunki00.koo@...sung.com>,
Shinbeom Choi <sbeom.choi@...sung.com>,
Jiri Slaby <jslaby@...e.com>,
linux-arm-kernel@...ts.infradead.org,
linux-samsung-soc@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 02/10] tty: serial: samsung_tty: fix build warning
On Wed, Dec 11, 2019 at 12:01:28PM +0100, Enrico Weigelt, metux IT consult wrote:
> On 10.12.19 15:36, Greg Kroah-Hartman wrote:
>
> Hi,
>
> > diff --git a/drivers/tty/serial/samsung_tty.c b/drivers/tty/serial/samsung_tty.c
> > index 83fd51607741..67c5a84d0a26 100644
> > --- a/drivers/tty/serial/samsung_tty.c
> > +++ b/drivers/tty/serial/samsung_tty.c
> > @@ -1851,7 +1851,10 @@ static int s3c24xx_serial_init_port(struct s3c24xx_uart_port *ourport,
> >
> > /* Device driver serial port probe */
> >
> > +#ifdef CONFIG_OF
> > static const struct of_device_id s3c24xx_uart_dt_match[];
> > +#endif
> > +
>
> By the way: I've got some patch for conditionally declaring of match
> tables (including MODULE_DEVICE_TABE() call), so such ifdef's aren't
> needed anymore.
That's not why this #ifdef is needed.
greg k-h
Powered by blists - more mailing lists