From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [PATCH net-next] net: mvpp2: Split the PPv2 driver to a dedicated directory Date: Fri, 01 Jun 2018 11:48:02 -0400 (EDT) Message-ID: <20180601.114802.1773849900748494270.davem@davemloft.net> References: <20180531080743.4218-1-maxime.chevallier@bootlin.com> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, antoine.tenart@bootlin.com, thomas.petazzoni@bootlin.com, gregory.clement@bootlin.com, miquel.raynal@bootlin.com, nadavh@marvell.com, stefanc@marvell.com, ymarkman@marvell.com, mw@semihalf.com To: maxime.chevallier@bootlin.com Return-path: In-Reply-To: <20180531080743.4218-1-maxime.chevallier@bootlin.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org From: Maxime Chevallier Date: Thu, 31 May 2018 10:07:43 +0200 > As the mvpp2 driver is growing, move this driver to a dedicated > directory and split it into several files. > > Since this driver has a lot of register defines and structure > definitions, it can benefit from having all of this into a dedicated > header file, named mvpp2.h. > > A good chunk of the mvpp2 code is dedicated to Header Parser handling, so > we introduce mvpp2_prs.h where all Header Parser definitions are located, > and mvpp2_prs.c containing the related code. > > In the same way, mvpp2_cls.h and mvpp2_cls.c are created to contain > Classifier and RSS related code. > > The former 'mvpp2.c' file is renamed 'mvpp2_main.c' so that we can keep > the driver binary named 'mvpp2'. > > This commit is only about spliting the driver into multiple files and > doesn't introduce any new function, feature or fix besides removing > 'static' keywords when needed. > > Signed-off-by: Maxime Chevallier > --- > This is based on the latest net-next, I expect no conflicts but due to > the amount of code this patchs touches, it can easily become un-appliable. > > Tell me if that's the case, I'll be happy to rebase it if needed. Applied, thank you.