Re: [U-Boot] [PATCH] post: fix up I/O helper usage

2011-08-24 Thread Wolfgang Denk
Dear Mike Frysinger, In message <201108141645.40439.vap...@gentoo.org> you wrote: > > > > Sorry, but I will not accept inl() here. > > > > so not even to fix build failures for most non-ppc arches ? i wont be > > able to post an updated patch for a while. > > so i can post a patch to use io{rea

Re: [U-Boot] [PATCH] post: fix up I/O helper usage

2011-08-14 Thread Mike Frysinger
On Friday, July 29, 2011 14:40:23 Mike Frysinger wrote: > On Thu, Jul 28, 2011 at 07:23, Wolfgang Denk wrote: > > What we should be using (and standardizing for) is probably this (at > > least some of the PTBs said so in the past): > > > > (*) ioreadX(), iowriteX() > > that's fine by me >

Re: [U-Boot] [PATCH] post: fix up I/O helper usage

2011-07-29 Thread Mike Frysinger
On Thu, Jul 28, 2011 at 07:23, Wolfgang Denk wrote: > Mike Frysinger wrote: >> >> -     return in_le32((volatile void *)(_POST_WORD_ADDR)); >> >> +     return inl((volatile void *)(_POST_WORD_ADDR)); >> >>  } >> > >> > Is this supposed to fix any real problem, or just a change according >> > to you

Re: [U-Boot] [PATCH] post: fix up I/O helper usage

2011-07-28 Thread Wolfgang Denk
Dear Mike Frysinger, In message you wrote: > > >> - return in_le32((volatile void *)(_POST_WORD_ADDR)); > >> + return inl((volatile void *)(_POST_WORD_ADDR)); > >> } > > > > Is this supposed to fix any real problem, or just a change according > > to your personal preferences? > > the "

Re: [U-Boot] [PATCH] post: fix up I/O helper usage

2011-07-27 Thread Mike Frysinger
On Tue, Jul 26, 2011 at 07:43, Wolfgang Denk wrote: > Mike Frysinger you wrote: >> The I/O API from Linux defaults to little endian accesses.  In order to >> do big endian accesses, there are a "be" variants.  The "le32" variants >> are arch-specific and not terribly common, so change it to the nor

Re: [U-Boot] [PATCH] post: fix up I/O helper usage

2011-07-26 Thread Wolfgang Denk
Dear Mike Frysinger, In message <1304996478-1536-1-git-send-email-vap...@gentoo.org> you wrote: > The I/O API from Linux defaults to little endian accesses. In order to > do big endian accesses, there are a "be" variants. The "le32" variants > are arch-specific and not terribly common, so change