From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753168AbdBCOCy (ORCPT ); Fri, 3 Feb 2017 09:02:54 -0500 Received: from goliath.siemens.de ([192.35.17.28]:41312 "EHLO goliath.siemens.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752169AbdBCOCx (ORCPT ); Fri, 3 Feb 2017 09:02:53 -0500 Subject: Re: [PATCH v13 1/2] serial: exar: split out the exar code from 8250_pci To: Sudip Mukherjee , Greg Kroah-Hartman , Jiri Slaby , Andy Shevchenko References: <1485815302-5708-1-git-send-email-sudipm.mukherjee@gmail.com> Cc: linux-kernel@vger.kernel.org, linux-serial@vger.kernel.org From: Jan Kiszka Message-ID: Date: Fri, 3 Feb 2017 15:02:15 +0100 User-Agent: Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.8.1.12) Gecko/20080226 SUSE/2.0.0.12-1.1 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666 MIME-Version: 1.0 In-Reply-To: <1485815302-5708-1-git-send-email-sudipm.mukherjee@gmail.com> Content-Type: text/plain; charset=iso-8859-15 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2017-01-30 23:28, Sudip Mukherjee wrote: > From: Sudip Mukherjee > > Add the serial driver for the Exar chips. And also register the > platform device for the GPIO provided by the Exar chips. "Also" means you are doing two things in one patch - was this already discussed and accepted in previous review rounds? If so, ignore my comment, but I would have asked for two patches, one that just translates the existing code and another that adds this new feature. > > Reviewed-by: Andy Shevchenko > Signed-off-by: Sudip Mukherjee > --- > > Andy, > I have added the if (!board) check, but I am not sure how board can be > NULL here. If probe executes that will mean there was a match of the > device id and so in that case board can not be NULL. > > drivers/tty/serial/8250/8250_exar.c | 396 ++++++++++++++++++++++++++++++++++++ > drivers/tty/serial/8250/Kconfig | 4 + > drivers/tty/serial/8250/Makefile | 1 + > 3 files changed, 401 insertions(+) > create mode 100644 drivers/tty/serial/8250/8250_exar.c > > diff --git a/drivers/tty/serial/8250/8250_exar.c b/drivers/tty/serial/8250/8250_exar.c > new file mode 100644 > index 0000000..ba1f359 > --- /dev/null > +++ b/drivers/tty/serial/8250/8250_exar.c > @@ -0,0 +1,396 @@ > +/* > + * Probe module for 8250/16550-type Exar chips PCI serial ports. > + * > + * Based on drivers/tty/serial/8250/8250_pci.c, > + * > + * Copyright (C) 2017 Sudip Mukherjee, All Rights Reserved. It's legally cleaner to carry over the copyright notice from the original file, unless you rewrote everything (unlikely on first glance). You may still add yours to the list for the significant contributions. BTW, are you personally the copyright holder or your employer Codethink? Depends on your contractual situation, but the former is less common. Jan -- Siemens AG, Corporate Technology, CT RDA ITP SES-DE Corporate Competence Center Embedded Linux