----- Original Message ----- From: "Adrian Hunter" <[EMAIL PROTECTED]> To: "Tony Lindgren" <[EMAIL PROTECTED]> Cc: <linux-omap@vger.kernel.org>; <[EMAIL PROTECTED]> Sent: Wednesday, September 10, 2008 12:58 PM Subject: Re: Bug in twl4030backupbatt_voltage()
> Tony Lindgren wrote: >> * Adrian Hunter <[EMAIL PROTECTED]> [080903 08:08]: >>> In the omap tree, commit f91fd98be045bec9c18e13110f22dafd44cb84b5 >>> fixed a bug with the use of function twl4030_madc_conversion() >>> relating to the local variable 'struct twl4030_madc_request req' >>> not being fully initialised. >>> >>> The same problem appears to be in the function >>> twl4030backupbatt_voltage() which is in drivers/power/twl4030_bci_battery.c >> >> Thanks, I'll initialize it to NULL. Wouldn't it have been easier just >> to write a patch instead of the description above? :) > > Apparently. > > There is also req.do_avg which should be zero, if you assume the programmer's > intent was to have uninitialised members set to zero. I guess, I did not look at all the other members of the "twl4030_madc_request" structure. I might have missed it. The do_avg and func_cb fields can both be set to NULL. Regards, Madhu > > -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html