[LinuxBIOS] [PATCH] VIA VT8237R support

Uwe Hermann uwe at hermann-uwe.de
Sat Oct 27 18:35:00 CEST 2007


On Sat, Oct 27, 2007 at 05:21:17PM +0200, Rudolf Marek wrote:
> >> +	printk_info("%s IDE interface %s\n", "Primary",
> >> +		    sb->ide0_enable ? "enabled" : "disabled");
> >> +	printk_info("%s IDE interface %s\n", "Secondary",
> >> +		    sb->ide1_enable ? "enabled" : "disabled");
> > 
> > Looks like size-optimizing? Are you sure this is good? It's a little
> > more difficult to read. (Might just be copypaste and not your idea?)
> 
> copypaste ;)

This code is ok IMO. It's from my 82371EB patch a while a ago I think,
where I used two normal printk_info()s which made the code unnecessarily
bigger, as spotted by Stefan. The above solution fixed that.

Maybe even "enabled" -> "on" and "disabled" -> "off"? Saveѕ some more
bytes and we lose no information whatsoever.


> >> +	/* standard bios sets master bit. */
> >> +	enables = pci_read_config8(dev, PCI_COMMAND);
> >> +	enables |= PCI_COMMAND_MASTER | PCI_COMMAND_IO;
> > 
> > Discussion please? Try to find out why this is a good idea.
> > Is this enabling bus mastering? Why would that be a bad idea?


> > Please try to not use factory BIOS as justification until we've
> > run out of ideas completely.

Full ACK.


> Well the older VIA driver in LB does it too. I think linux will switch bus 
> master when needed. I think we can drop this.

Only if non-Linux payloads are tested/guaranteed to be able to cope with
that, too.


Uwe.
-- 
http://www.hermann-uwe.de  | http://www.holsham-traders.de
http://www.crazy-hacks.org | http://www.unmaintained-free-software.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20071027/9b4642e4/attachment.sig>


More information about the coreboot mailing list