On Sat, Sep 08, 2007 at 08:41:18PM +0200, Peter Stuge wrote: > On Sat, Sep 08, 2007 at 07:56:32PM +0200, Uwe Hermann wrote: > > On Sat, Sep 08, 2007 at 04:49:46PM +0200, Alex Beregszaszi wrote: > > > attached patch changes out/in combinations to pci_read/write_byte in > > > sis630 chipset enable. > > > > Why? > > There is abstraction available - so it should be used everywhere.
Ah, yes. I missed the fact that those functions do the same as the replaced code. > > Did you test the patch on hardware? > > On Sat, Sep 08, 2007 at 10:59:39AM -0700, ron minnich wrote: > > I don't think this should be done unless someone can test. > > > I agree with the patch - I want the change, but I also agree it would > be nice to have it tested. > > We could argue that we decided to go through with the patch in order > to improve the code and until someone has a problem with the code not > working there is no real problem. I'm not too eager to find out the hard way. This is low-level enough that I think there _might_ be unintended/unnoticed consequences. Anybody with such a chipset willing to test the patch? Uwe. -- http://www.hermann-uwe.de | http://www.holsham-traders.de http://www.crazy-hacks.org | http://www.unmaintained-free-software.org
signature.asc
Description: Digital signature
-- linuxbios mailing list [email protected] http://www.linuxbios.org/mailman/listinfo/linuxbios
