[coreboot] [PATCH] Fix bootswitch of SATA/PATA
Uwe Hermann
uwe at hermann-uwe.de
Sun Oct 10 21:05:12 CEST 2010
On Thu, Oct 07, 2010 at 12:31:57AM +0200, Rudolf Marek wrote:
> Hello,
>
> Following patch fixes the boot_switch_sata_ide logic. It can swap
> primary / secondary IDE channel with SATA (in IDE mode).
>
> The bug was that setup was done in wrong device.
>
> Signed-off-by: Rudolf Marek <r.marek at assembler.cz>
Yep, indeed.
Acked-by: Uwe Hermann <uwe at hermann-uwe.de>
> Index: sb700_ide.c
> ===================================================================
> --- sb700_ide.c (revision 5917)
> +++ sb700_ide.c (working copy)
> @@ -52,10 +53,13 @@
>
> /* set ide as primary, if you want to boot from IDE, you'd better set it
> * in $vendor/$mainboard/devicetree.cb */
> +
> +
> if (conf->boot_switch_sata_ide == 1) {
> - byte = pci_read_config8(dev, 0xAD);
> + struct device *sm_dev = dev_find_slot(0, PCI_DEVFN(0x14, 0));
I personally don't like the "struct device *sm_dev" to be defined here,
please consider putting it at the beginning of the function, not in the
middle (yeah, I know, it works, I just don't like it much).
Uwe.
--
http://hermann-uwe.de | http://sigrok.org
http://randomprojects.org | http://unmaintained-free-software.org
More information about the coreboot
mailing list