From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752627AbdBCVb3 (ORCPT ); Fri, 3 Feb 2017 16:31:29 -0500 Received: from mail-wm0-f66.google.com ([74.125.82.66]:33299 "EHLO mail-wm0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752365AbdBCVb1 (ORCPT ); Fri, 3 Feb 2017 16:31:27 -0500 Message-ID: <5894F6A2.5000609@gmail.com> Date: Fri, 03 Feb 2017 21:31:14 +0000 From: Sudip Mukherjee User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.4.0 MIME-Version: 1.0 To: Jan Kiszka , Greg Kroah-Hartman , Jiri Slaby , Andy Shevchenko CC: linux-kernel@vger.kernel.org, linux-serial@vger.kernel.org Subject: Re: [PATCH v13 1/2] serial: exar: split out the exar code from 8250_pci References: <1485815302-5708-1-git-send-email-sudipm.mukherjee@gmail.com> In-Reply-To: Content-Type: text/plain; charset=iso-8859-15; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Friday 03 February 2017 02:02 PM, Jan Kiszka wrote: > 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. > Like Andy replied, this is already in tty-next. >> >> 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. Should i send a separate patch to modify those? Andy? > > BTW, are you personally the copyright holder or your employer Codethink? > Depends on your contractual situation, but the former is less common. Well, Codethink has nothing to do with this patch. This was a voluntary work started before I joined Codethink, but then I joined Codethink and found very little time to finish this. So finally now its done. https://lists.kernelnewbies.org/pipermail/kernelnewbies/2015-November/015372.html Regards Sudip