[U-Boot-Users] soft_i2c

Wolfgang Denk wd at denx.de
Fri Apr 23 16:46:17 CEST 2004


In message <20040423160543.D8298 at gateway.bln.innominate.local> you wrote:
>
> > Do you really think this is more readable and more general?
> 
> Yes, absoluteley - and if _MPC8260 and _8xx declare their port
>   #define DECLARE_I2C_PORT ........
> in their board-configuration file the soft_i2c.c is board independend
> and other boards can easily use it .....

But soft_i2c.c is board independent as is. It's even processor  inde-
pendent as is.

> In the old version he has to add 6 times the same #ifdef s
> This is ugly copy&paste programming in my eyes.
> 
> Your mileage my vary...

If we'd move the code in the board config files we'd have to copy  it
approx.  100 times - that would be even worse. This code is not board
dependent.


I see what you mean - but your patch is even less portable as it will
break all boards that fail to define DECLARE_I2C_PORT. OK -  you  can
add  a #ifdef, but then I really think that there is no difference in
readability left. Another option is to use a file-global  declaration
of  the  pointers,  but  that  would increas the memopry footprint of
U-boot by 264 bytes - after all, I think the  benefit  of  all  these
changes is epsilon and not worth the effort.

Best regards,

Wolfgang Denk

-- 
Software Engineering:  Embedded and Realtime Systems,  Embedded Linux
Phone: (+49)-8142-4596-87  Fax: (+49)-8142-4596-88  Email: wd at denx.de
What the gods would destroy they first submit to  an  IEEE  standards
committee.




More information about the U-Boot mailing list