<div dir="ltr"><br><br><div class="gmail_quote">On Fri, Nov 7, 2008 at 5:00 AM, Carl-Daniel Hailfinger <span dir="ltr"><<a href="mailto:c-d.hailfinger.devel.2006@gmx.net">c-d.hailfinger.devel.2006@gmx.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div class="Ih2E3d">On 07.11.2008 03:47, Elia Yehuda wrote:<br>
> Signed-off-by: Elia Yehuda <<a href="mailto:z4ziggy@gmail.com">z4ziggy@gmail.com</a>><br>
><br>
<br>
</div>A small changelog (2 lines or so, more if you like) would be appreciated.</blockquote><div><br><br>ok, i thought the title was informative enough... i'll try harder next time!!! :-)<br><br> <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
<div><div></div><div class="Wj3C7c"><br>
> Index: src/devices/pci_rom.c<br>
> ===================================================================<br>
> --- src/devices/pci_rom.c    (revision 3698)<br>
> +++ src/devices/pci_rom.c    (working copy)<br>
> @@ -67,8 +67,10 @@<br>
>      printk_spew("PCI ROM Image, Vendor %04x, Device %04x,\n",<br>
>              rom_data->vendor, rom_data->device);<br>
>      if (dev->vendor != rom_data->vendor || dev->device != rom_data->device)<br>
> {<br>
> -        printk_err("Device or Vendor ID mismatch Vendor %04x, Device<br>
> %04x\n",<br>
> -               rom_data->vendor, rom_data->device);<br>
> +        printk_err("Device or Vendor ID mismatch Vendor %04x, Device %04x "<br>
> +               "(Vendor %04x, Device %04x)\n",<br>
> +               rom_data->vendor, rom_data->device,<br>
> +               dev->vendor, dev->device);<br>
>          return NULL;<br>
>      }<br>
><br>
><br>
<br>
</div></div>The patch was mangled by gmail. AFAIK that's an unfixable bug with the<br>
gmail web interface. Please attach the patch instead.</blockquote><div><br><br>will do<br><br> <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
<br>
About the contents of the patch: The new message still requires people<br>
to read the source code. How about:<br>
"Device or Vendor ID mismatch: read Vendor %04x, Device %04x, expected<br>
<div class="Ih2E3d">Vendor %04x, Device %04x\n"</div></blockquote><div><br><br>such a long line breaks the 80 column boundary. is that ok?<br><br> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="Ih2E3d"><br>
<br>
</div>Regards,<br>
Carl-Daniel<br>
<font color="#888888"><br>
--<br>
<a href="http://www.hailfinger.org/" target="_blank">http://www.hailfinger.org/</a><br>
<br>
</font></blockquote></div><br></div>