RE: [RFC] Moving files around in drivers/net

[Jeff Kirsher <jeffrey.t.kirsher@intel.com>]

[-- Attachment #1: Type: text/plain, Size: 2301 bytes --]

On Wed, 2011-06-15 at 22:05 -0700, Sathya.Perla@Emulex.Com wrote:
> >-----Original Message-----
> >From: netdev-owner@vger.kernel.org [mailto:netdev-owner@vger.kernel.org] On
> >Behalf Of Joe Perches
> >Sent: Thursday, June 16, 2011 6:40 AM
> >To: Jeff Kirsher
> >Cc: netdev; David Miller; Paul Gortmaker; Jan Engelhardt
> >Subject: Re: [RFC] Moving files around in drivers/net
> >
> >
> >commit 931da59411e60574a1235d0ad2d63ebd1638a859
> >Author: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
> >Date:   Fri May 13 00:37:27 2011 -0700
> >
> >    be2net: Move the BladeEngine 2 driver
> >
> >    Moves the Server Engine driver into drivers/net/ethernet/benet/ and
> >    make the necessary Kconfig and Makefile changes.
> >
> >    CC: Sathya Perla <sathya.perla@emulex.com>
> >    CC: Subbu Seetharaman <subbu.seetharaman@emulex.com>
> >    CC: Ajit Khaparde <ajit.khaparde@emulex.com>
> >    Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
> >
> > MAINTAINERS                                   |    2 +-
> > drivers/net/Kconfig                           |    2 --
> > drivers/net/Makefile                          |    1 -
> > drivers/net/ethernet/Kconfig                  |    1 +
> > drivers/net/ethernet/Makefile                 |    1 +
> > drivers/net/{ => ethernet}/benet/Kconfig      |    0
> > drivers/net/{ => ethernet}/benet/Makefile     |    0
> > drivers/net/{ => ethernet}/benet/be.h         |    0
> > drivers/net/{ => ethernet}/benet/be_cmds.c    |    0
> > drivers/net/{ => ethernet}/benet/be_cmds.h    |    0
> > drivers/net/{ => ethernet}/benet/be_ethtool.c |    0
> > drivers/net/{ => ethernet}/benet/be_hw.h      |    0
> > drivers/net/{ => ethernet}/benet/be_main.c    |    0
> > 13 files changed, 3 insertions(+), 4 deletions(-)
> >
> 
> If you are going to consistently have the path as drivers/net/ethernet/<company-name>/<driver-name>, then
> be2net driver must be under drivers/net/ethernet/emulex/be2net/
> 
> thanks,
> -Sathya

That sound fine.  I was not consistent about using
driver/net/ethernet/<company name>/ because companies have been
bought/sold and a driver may change hands one or more times.  So I tried
to avoid using company names unless I was confident about it.

I will make the requested change.

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 490 bytes --]