Dear Wolfgang Denk, > Dear Reinhard Meyer, > > In message<4d1340bd.2090...@emk-elektronik.de> you wrote: >> >> We should have one function like "show_status(int status)" >> with various statuses defined like: >> LED_STATUS_RESET, _BEFORE_RELOC, _AFTER_RELOC, _SENT_DHCP, _STARTING_KERNEL, >> _CRASHED, ... > > Keep the existing code, and just turn the current set of numbers into > an enum.
What current code? 1. show_boot_progress() is quite close, but has hundreds of values... Not sure a enum would be useful there, the values have gaps; rather an errno.h like implementation makes sense there. 2. the LED current code (in some non-ARM) places has LED types (BOOT, ...) and LED modes (OFF, BLINK, ON). That is totally different and quite useless for seven segment and simple LCDisplays. Best Regards, Reinhard _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
