From mboxrd@z Thu Jan  1 00:00:00 1970
From: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
Date: Thu, 2 Apr 2009 07:22:59 +0200
Subject: [U-Boot] [RFC PATCH] drivers/usb: regorganisation
In-Reply-To: <20090401221245.AA09783797DC@gemini.denx.de>
References: <200904011419.13396.sr@denx.de>
	<1238606479-914-1-git-send-email-plagnioj@jcrosoft.com>
	<20090401221245.AA09783797DC@gemini.denx.de>
Message-ID: <20090402052259.GI14366@game.jcrosoft.org>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On 00:12 Thu 02 Apr     , Wolfgang Denk wrote:
> Dear Jean-Christophe PLAGNIOL-VILLARD,
> 
> In message <1238606479-914-1-git-send-email-plagnioj@jcrosoft.com> you wrote:
> > Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
> > ---
> > better description will be wrote for the final patch
> 
> Maybe we should then remove some redundancy in the file names, too?
> Like that:
> 
> > Best Regards,
> > 
> >  Makefile                                     |    4 +++-
> >  drivers/usb/{ => core}/Makefile              |   22 +---------------------
> >  drivers/usb/{ => core}/usb_ehci_core.c       |    2 +-
> >  drivers/usb/{ => core}/usb_ohci.c            |    2 +-
> 
> usb_ehci_core.c => ehci_core.c ?
> usb_ohci.c => ohci.c ?
is this case ehci.c could be better
> 
> >  drivers/usb/{ => core}/usbdcore.c            |    2 +-
> >  drivers/usb/{ => gadget}/Makefile            |   19 +------------------
> >  drivers/usb/{ => gadget}/usbdcore_ep0.c      |    2 +-
> >  drivers/usb/{ => gadget}/usbdcore_mpc8xx.c   |    4 ++--
> >  drivers/usb/{ => gadget}/usbdcore_omap1510.c |    4 ++--
> 
> usbdcore => dcore ?
device or udc as in most of the linux file name
> 
> >  drivers/usb/{ => host}/usb_ehci_fsl.c        |    5 ++---
> >  drivers/usb/{ => host}/usb_ehci_fsl.h        |    0
> >  drivers/usb/{ => host}/usb_ehci_ixp.c        |    4 ++--
> >  drivers/usb/{ => host}/usb_ehci_pci.c        |    4 ++--
> >  drivers/usb/{ => host}/usb_ehci_vct.c        |    5 ++---
> >  {drivers/usb => include}/usb_ehci.h          |    0
> >  {drivers/usb => include}/usb_ehci_core.h     |    0
> >  {drivers/usb => include}/usb_ohci.h          |    0
> 
> usb_[eo]hci => [eo]hci ?
> 
Remy what do you prefer?

Best Regards,
J.