<br><br><div class="gmail_quote">2010/12/26 Michael Karcher <span dir="ltr"><<a href="mailto:Michael.Karcher@fu-berlin.de" target="_blank">Michael.Karcher@fu-berlin.de</a>></span><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

Am Montag, den 06.12.2010, 18:01 +0100 schrieb Idwer Vollering:<br>
<div>> This corrects a djgpp build error, seen with r1232 and later.<br>
><br>
> Signed-off-by: Idwer Vollering <<a href="mailto:vidwer@gmail.com" target="_blank">vidwer@gmail.com</a>><br>
</div>Not acked...<br>
<div><br>
> -int rpci_write_long(struct pci_dev *dev, int reg, uint32_t data)<br>
> +int rpci_write_long(struct pci_dev *dev, int reg, u32 data)<br>
<br>
</div><div>> pcidev.c:210: error: conflicting types for 'rpci_write_long'<br>
> programmer.h:226: error: previous declaration of 'rpci_write_long' was<br>
> here<br>
<br>
</div>I'd rather patch the header file to use uint32_t/uint16_t/uint8_t<br>
instead of u8/u16/u32, as the former types are used throughout flashrom.<br></blockquote><div><br>Patch v2.<br><br>Signed-off-by: Idwer Vollering <<a href="mailto:vidwer@gmail.com" target="_blank">vidwer@gmail.com</a>> <br>
</div><div>
<br>Index: programmer.h<br>===================================================================<br>--- programmer.h        (revision 1247)<br>+++ programmer.h        (working copy)<br>@@ -221,9 +221,9 @@<br> /* rpci_write_* are reversible writes. The original PCI config space register<br>

  * contents will be restored on shutdown.<br>  */<br>-int rpci_write_byte(struct pci_dev *dev, int reg, u8 data);<br>-int rpci_write_word(struct pci_dev *dev, int reg, u16 data);<br>-int rpci_write_long(struct pci_dev *dev, int reg, u32 data);<br>

+int rpci_write_byte(struct pci_dev *dev, int reg, uint8_t data);<br>+int rpci_write_word(struct pci_dev *dev, int reg, uint16_t data);<br>+int rpci_write_long(struct pci_dev *dev, int reg, uint32_t data);<br> #endif<br>
<br>
 /* print.c */<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Is there any reason to not use them on rpci_write_*?<br></blockquote><div><br>No, using the same type in a header as well as in the file that includes that header makes sense.<br>Compile- and runtime tested using make (linux and djgpp) and gmake <span lang="en"><span title="Klik voor alternatieve vertalingen">respectively.</span></span><br>

<br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<br>
Regards,<br>
<font color="#888888">  Michael Karcher<br>
</font></blockquote></div><br>