From: <a.o...@bl...> - 2005-09-11 11:07:25
|
On Sun, Sep 11, 2005 at 12:59:16PM +0200, Albert Herranz wrote: > > That change doesn't touch the accessors bit, so I > > don't see where the > > problem is.. > > There's no problem. That's the problem. Your patch > does actually nothing. > > > The appropriate accessors _do_ exist. None of the > > drivers doing > > I/O actually use them, unfortunately. > > We can't use them yet, as you already mentioned it. in/out_be(16|32)(). ioread/write(16|32)be() are simply wrappers around these. > > I think I clarified it enough in the commit message. > > Nothing has been > > broken, intentionally or unintentionally. Like I > > said, patch exists, and > > until it makes it into mainline, > > include/asm-ppc/io.h remains in CVS. > > Yes, nothing is broken. And nothing changed. > > That's why I wondered, why the patch? Just cvs noise? > Or did I miss something subtle? Noise, yes. But a step towards eventually moving it out of CVS. Arthur |