[U-Boot] [PATCH][NEXT] hwmon: do not init sensors on startup
Wolfgang Denk
wd at denx.de
Thu Dec 9 15:26:43 CET 2010
Dear Heiko Schocher,
In message <1291903238-29071-1-git-send-email-hs at denx.de> you wrote:
>
> --- a/common/cmd_dtt.c
> +++ b/common/cmd_dtt.c
> @@ -28,6 +28,8 @@
> #include <dtt.h>
> #include <i2c.h>
>
> +static unsigned long sensors_init_done = 0;
What if there are more then 32 sensors?
> int do_dtt (cmd_tbl_t * cmdtp, int flag, int argc, char * const argv[])
> {
> int i;
> @@ -42,8 +44,16 @@ int do_dtt (cmd_tbl_t * cmdtp, int flag, int argc, char * const argv[])
> * Loop through sensors, read
> * temperature, and output it.
> */
> - for (i = 0; i < sizeof (sensors); i++)
> - printf ("DTT%d: %i C\n", i + 1, dtt_get_temp (sensors[i]));
> + for (i = 0; i < sizeof (sensors); i++) {
> + if ((sensors_init_done & (1 << i)) != (1 << i)) {
> + if (dtt_init_one(sensors[i]) == 0)
> + sensors_init_done |= (1 << i);
> + else
> + printf("DTT%d: Failed init!\n", i);
> + }
> + if ((sensors_init_done & (1 << i)) == (1 << i))
> + printf ("DTT%d: %i C\n", i + 1, dtt_get_temp(sensors[i]));
> + }
This is overly complicated, it seems. Why not:
for (i = 0; i < sizeof(sensors); i++) {
if ((sensors_init_done & (1 << i)) == 0) {
if (dtt_init_one(sensors[i]) != 0) {
printf("DTT%d: init failed\n", i);
continue;
}
sensors_init_done |= (1 << i);
}
printf("DTT%d: %i C\n", i + 1, dtt_get_temp(sensors[i]));
}
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
"We don't have to protect the environment -- the Second Coming is at
hand." - James Watt
More information about the U-Boot
mailing list