On Mon, 18 Feb 2008, Ulf Jordan wrote:

On Mon, 18 Feb 2008, Rudolf Marek wrote:

Btw the build is failing with:

srv/svn/linuxbios-extra/tmp/coreboot-v2-3104/src/devices/pnp_device.c:54: error:
`u8' undeclared (first use in this function)

Any idea why it works for me here?

No, but all the failed builds are for powerpc targets. Include file
differences between the platforms?

I think it is necessary to use uint8_t in the code here, see attached patch.

/ulf
Fix broken build after r3104 for ppc targets due to use of u8 instead of
uint8_t.

Signed-off-by: Ulf Jordan <[EMAIL PROTECTED]>

Index: src/devices/pnp_device.c
===================================================================
--- src/devices/pnp_device.c    (revision 3108)
+++ src/devices/pnp_device.c    (arbetskopia)
@@ -51,7 +51,7 @@
 
 void pnp_set_enable(device_t dev, int enable)
 {
-       u8 tmp, bitpos;
+       uint8_t tmp, bitpos;
 
        tmp = pnp_read_config(dev, 0x30);
        /* handle the virtual devices, which share same LDN register */
@@ -67,7 +67,7 @@
 
 int pnp_read_enable(device_t dev)
 {
-       u8 tmp, bitpos;
+       uint8_t tmp, bitpos;
        tmp = pnp_read_config(dev, 0x30);
        /* handle the virtual devices, which share same LDN register */
        bitpos = (dev->path.u.pnp.device >> 8) & 0x7;
-- 
coreboot mailing list
[email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to