[U-Boot-Users] Re: [U-Boot-CVS] u-boot/drivers omap24xx_i2c.c,NONE,1.1 Makefile,1.29,1.30 ns16550.c,1.6,1.7 serial.c,1.5,1.6
Cory Tusar
ctusar at videon-central.com
Mon Jan 10 17:44:13 CET 2005
- Previous message: [U-Boot-Users] RE: [U-Boot-CVS] u-boot/drivers omap24xx_i2c.c,NONE,1.1 Makefile,1.29,1.30 ns16550.c,1.6,1.7 serial.c,1.5,1.6
- Next message: [U-Boot-Users] Re: [U-Boot-CVS] u-boot/drivers omap24xx_i2c.c,NONE,1.1 Makefile,1.29,1.30 ns16550.c,1.6,1.7 serial.c,1.5,1.6
- Messages sorted by:
[ date ]
[ thread ]
[ subject ]
[ author ]
Woodruff, Richard wrote:
> Sorry,
>
> I ran into the build problem long ago and fixed it locally. I was
> preparing to resubmit a patch with a lot of additions and this one
> fix....as I was thinking Wolfgang had bypassed it.
>
> I believe the real fix I used was in the driver itself not above. Its
> much simpler and I think the correct one.
>
> Regards,
> Richard W.
>
>
>
> The File drivers/omap2420_i2c.c needs to look like this:
> -----------------------------------------------------------
> #include <common.h>
>
> #ifdef CONFIG_DRIVER_OMAP24XX_I2C
>
> #include <asm/arch/i2c.h>
> #include <asm/io.h>
>
> #define inw(a) __raw_readw(a)
> #define outw(a,v) __raw_writew(a,v)
>
>
> -----------------------------------------------------------
>
> previously it WRONGLY looked like:
>
> #include <common.h>
> #include <asm/arch/i2c.h>
> #include <asm/io.h>
>
> #define inw(a) __raw_readw(a)
> #define outw(a,v) __raw_writew(a,v)
>
> #ifdef CONFIG_DRIVER_OMAP24XX_I2C
Agreed. After a bit more testing, I found that there were preprocessor
warnings generated due to symbols being defined multiple times between
your i2c.h and the omap1510.h (notably inb(), outb() and the like). I
came up with pretty much the same solution as you, and am running a
MAKEALL test now just to be sure there are no issues this time.
Assuming not, I'll re-send just the above changes as a patch to resolve
this.
Thanks.
-Cory
--
Cory T. Tusar
Embedded Systems Engineer
Videon Central, Inc.
2171 Sandy Drive
State College, PA 16801
(814) 235-1111 x316
(814) 235-1118 fax
"Debugging is twice as hard as writing the code in the first place.
Therefore, if you write the code as cleverly as possible, you are,
by definition, not smart enough to debug it." --Brian W. Kernighan
- Previous message: [U-Boot-Users] RE: [U-Boot-CVS] u-boot/drivers omap24xx_i2c.c,NONE,1.1 Makefile,1.29,1.30 ns16550.c,1.6,1.7 serial.c,1.5,1.6
- Next message: [U-Boot-Users] Re: [U-Boot-CVS] u-boot/drivers omap24xx_i2c.c,NONE,1.1 Makefile,1.29,1.30 ns16550.c,1.6,1.7 serial.c,1.5,1.6
- Messages sorted by:
[ date ]
[ thread ]
[ subject ]
[ author ]
More information about the U-Boot
mailing list