From mboxrd@z Thu Jan 1 00:00:00 1970 From: Wolfgang Denk Date: Mon, 10 Aug 2009 09:44:42 +0200 Subject: [U-Boot] [Bug] IXP425 and e1000 network driver In-Reply-To: <4A7FB519.6040000@gmail.com> References: <20090403211819.5637F83797DC@gemini.denx.de> <49D68311.4090807@gmail.com> <20090805202610.D338C832E416@gemini.denx.de> <4A7FB519.6040000@gmail.com> Message-ID: <20090810074442.59D41833DBD2@gemini.denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Dear Ben, In message <4A7FB519.6040000@gmail.com> you wrote: > > I thought I brought this up already, but maybe not. Won't this break > PowerPC? I'm pretty sure (value) != (cpu_to_le32(value)), isn't it? Right. > Isn't the problem that writel() and readl() aren't byte-swapped on BE ARM? Right. writel() and readl() are not a good choice for architecture independent code. AFAIK ioreadX(), iowriteX() are supposed to be architecture independent I/O accessors, but unfortunately not all architectures implement these yet. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de Computers make excellent and efficient servants, but I have no wish to serve under them. Captain, a starship also runs on loyalty to one man. And nothing can replace it or him. -- Spock, "The Ultimate Computer", stardate 4729.4