[coreboot] [patch] add required info to not-matched vendor/device id

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Fri Nov 7 04:00:22 CET 2008


On 07.11.2008 03:47, Elia Yehuda wrote:
> Signed-off-by: Elia Yehuda <z4ziggy at gmail.com>
>   

A small changelog (2 lines or so, more if you like) would be appreciated.

> Index: src/devices/pci_rom.c
> ===================================================================
> --- src/devices/pci_rom.c    (revision 3698)
> +++ src/devices/pci_rom.c    (working copy)
> @@ -67,8 +67,10 @@
>      printk_spew("PCI ROM Image, Vendor %04x, Device %04x,\n",
>              rom_data->vendor, rom_data->device);
>      if (dev->vendor != rom_data->vendor || dev->device != rom_data->device)
> {
> -        printk_err("Device or Vendor ID mismatch Vendor %04x, Device
> %04x\n",
> -               rom_data->vendor, rom_data->device);
> +        printk_err("Device or Vendor ID mismatch Vendor %04x, Device %04x "
> +               "(Vendor %04x, Device %04x)\n",
> +               rom_data->vendor, rom_data->device,
> +               dev->vendor, dev->device);
>          return NULL;
>      }
>
>   

The patch was mangled by gmail. AFAIK that's an unfixable bug with the
gmail web interface. Please attach the patch instead.

About the contents of the patch: The new message still requires people
to read the source code. How about:
"Device or Vendor ID mismatch: read Vendor %04x, Device %04x, expected
Vendor %04x, Device %04x\n"

Regards,
Carl-Daniel

-- 
http://www.hailfinger.org/





More information about the coreboot mailing list