On Tue, Dec 17, 2013 at 05:08:22PM -0500, Anson Huang wrote:
> if (new_freq > old_freq) {
> + if (regulator_is_enabled(pu_reg)) {
> + ret = regulator_set_voltage_tol(pu_reg, imx6_soc_volt[index], 0);
As I pointed out while reviewing the regulator driver this is racy,
nothing stops the regulator state being changed after you read it and if
you skip setting the voltage then when something comes along and enables
the regulator later it will be enabled with a possibly incorrect
voltage.
Attachment:
signature.asc
Description: Digital signature
