<div dir="ltr">Signed-off-by: Elia Yehuda <<a href="mailto:z4ziggy@gmail.com">z4ziggy@gmail.com</a>><br><br>This patch adds information in printk_err() about which vendor/device id mismatches to which expected <br>vendor/device on pci_rom_probe().<br>
 <br>Regards,<br>Elia.<br><br><div class="gmail_quote">On Fri, Nov 7, 2008 at 5:24 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 04:04, Elia Yehuda wrote:<br>
> On Fri, Nov 7, 2008 at 5:00 AM, Carl-Daniel Hailfinger <<br>
> <a href="mailto:c-d.hailfinger.devel.2006@gmx.net">c-d.hailfinger.devel.2006@gmx.net</a>> wrote:<br>
><br>
><br>
>> On 07.11.2008 03:47, Elia Yehuda wrote:<br>
>><br>
>>> Signed-off-by: Elia Yehuda <<a href="mailto:z4ziggy@gmail.com">z4ziggy@gmail.com</a>><br>
>>><br>
>>><br>
>> A small changelog (2 lines or so, more if you like) would be appreciated.<br>
>><br>
><br>
><br>
><br>
> ok, i thought the title was informative enough... i'll try harder next<br>
> time!!! :-)<br>
><br>
<br>
</div>Rough guideline for changelogs is:<br>
- Keep them in the mail body.<br>
- Make sure that someone only reading the changelog (and not the code)<br>
knows what's going on.<br>
<br>
"Not-matched vendor/device id" can be anything. Please mention that this<br>
is about PCI ROMs and that the mismatch occurs between the vendor and<br>
device ID stored in the ROM and the PCI vendor and device ID of the<br>
corresponding PCI device.<br>
<div class="Ih2E3d"><br>
<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 !=<br>
>>><br>
>> rom_data->device)<br>
>><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<br>
>>><br>
>> %04x "<br>
>><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>
>> The patch was mangled by gmail. AFAIK that's an unfixable bug with the<br>
>> gmail web interface. Please attach the patch instead.<br>
>><br>
><br>
> will do<br>
><br>
<br>
</div>Thanks.<br>
<div class="Ih2E3d"><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>
>> Vendor %04x, Device %04x\n"<br>
>><br>
><br>
> such a long line breaks the 80 column boundary. is that ok?<br>
><br>
<br>
</div>Of course you can break the line. My point was more about the message<br>
itself than cosmetic linebreaks.<br>
<div><div></div><div class="Wj3C7c"><br>
Regards.<br>
Carl-Daniel<br>
<br>
--<br>
<a href="http://www.hailfinger.org/" target="_blank">http://www.hailfinger.org/</a><br>
<br>
</div></div></blockquote></div><br></div>