Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753082AbbKJOm7 (ORCPT ); Tue, 10 Nov 2015 09:42:59 -0500 Received: from mail-wm0-f47.google.com ([74.125.82.47]:34685 "EHLO mail-wm0-f47.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752544AbbKJOm5 (ORCPT ); Tue, 10 Nov 2015 09:42:57 -0500 Subject: Re: [PATCH 5/6] net: dsa: mv88e6060: add register defines header file To: Vivien Didelot References: <5641F04A.9060303@baylibre.com> <20151110142550.GB29568@ketchup.mtl.sfl> Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, "David S. Miller" , Florian Fainelli , Andrew Lunn , Guenter Roeck From: Neil Armstrong Organization: Baylibre Message-ID: <56420268.90806@baylibre.com> Date: Tue, 10 Nov 2015 15:42:48 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.2.0 MIME-Version: 1.0 In-Reply-To: <20151110142550.GB29568@ketchup.mtl.sfl> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1942 Lines: 58 On 11/10/2015 03:25 PM, Vivien Didelot wrote: > Hi Neil, > > On Nov. Tuesday 10 (46) 02:25 PM, Neil Armstrong wrote: >> To align with the mv88e6xxx code, add a similar header file >> with all the register defines. >> The file is based on the mv88e6xxx header for coherency. >> >> Signed-off-by: Neil Armstrong > > In the RFC patchset, Andrew mentioned that there is not that much things in > common with mv88e6xxx, so I don't really see a value to add a separate header > file. Would that make sense to you guys to add the defines directly in > mv88e6060.c and squash that in the last patch? > >> --- >> drivers/net/dsa/mv88e6060.h | 108 ++++++++++++++++++++++++++++++++++++++++++++ >> 1 file changed, 108 insertions(+) >> create mode 100644 drivers/net/dsa/mv88e6060.h >> >> diff --git a/drivers/net/dsa/mv88e6060.h b/drivers/net/dsa/mv88e6060.h >> new file mode 100644 >> index 0000000..adbc894 >> --- /dev/null >> +++ b/drivers/net/dsa/mv88e6060.h >> @@ -0,0 +1,108 @@ >> +/* >> + * net/dsa/mv88e6060.h - Marvell 88e6060 switch chip support >> + * Copyright (c) 2008 Marvell Semiconductor > > Also I don't think the copyright notice is correct here. > > Thanks, > -v > Vivien, Is something like this OK ? /* * drivers/net/dsa/mv88e6060.h - Marvell 88e6060 switch chip support * Copyright (c) 2015 Neil Armstrong * * Based on mv88e6xxx.h * Copyright (c) 2008 Marvell Semiconductor * * This program is free software; you can redistribute it and/or modify * it under the terms of the GNU General Public License as published by * the Free Software Foundation; either version 2 of the License, or * (at your option) any later version. */ Thanks, Neil -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/