[coreboot] r3104 - trunk/coreboot-v2/src/devices

Ulf Jordan jordan at chalmers.se
Mon Feb 18 23:17:08 CET 2008


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
-------------- next part --------------
Fix broken build after r3104 for ppc targets due to use of u8 instead of
uint8_t.

Signed-off-by: Ulf Jordan <jordan at chalmers.se>

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;


More information about the coreboot mailing list