From mboxrd@z Thu Jan  1 00:00:00 1970
From: Wolfgang Denk <wd@denx.de>
Date: Thu, 18 May 2006 00:43:59 +0200
Subject: [U-Boot-Users] Re: Request to mailing list U-Boot-Users rejected
In-Reply-To: Your message of "Wed, 17 May 2006 17:55:11 +0200."
             <20060517155511.GQ7487@enneenne.com> 
Message-ID: <E1Fgcug-00049H-Lt@mail.sourceforge.net>
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

In message <20060517155511.GQ7487@enneenne.com> you wrote:
> 
> My idea is:
> 
> * First you should fix the au1x00 support with my patch.

Which increases  the  mess  by  adding  yet  another  set  of  mostly
duplicated files...

> * Then I'm going to move files ?./cpu/mpc5xxx/usb_ohci.h?,
> ?./cpu/mips/au1x00_usb_ohci.h? (which are the same file) and the other
> '*ohci.h' files into ?include/usb_ohci.h?.
> 
> * Then I'm going to move file ?cpu/mips/au1x00_usb_ohci.c? into
> ?common/usb_ohci.c? modifing the functions ?usb_lowlevel_init()? and
> ?usb_lowlevel_stop()? according to the different platforms.

... before you start to clean it up?

> Please, let me know if these steps look sane to you. :)

I would expect that first cleaning up the code, and then adding a new
board to the cleaned up version is simpler and easier to understand?

Best regards,

Wolfgang Denk

-- 
Software Engineering:  Embedded and Realtime Systems,  Embedded Linux
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
There is is no reason for any individual to have a computer in  their
home.      -- Ken Olsen (President of Digital Equipment Corporation),
              Convention of the World Future Society, in Boston, 1977