On 10/08/2012 11:33 PM, Mark Brown wrote: > On Fri, Oct 05, 2012 at 06:54:58PM -0700, Andrey Smirnov wrote: > >> + err = regulator_enable(core->supplies.va); >> + if (err < 0) >> + break; >> + >> + err = regulator_enable(core->supplies.vio2); >> + if (err < 0) >> + goto disable_va; >> + >> + err = regulator_enable(core->supplies.vd); >> + if (err < 0) >> + goto disable_vio2; >> + >> + err = regulator_enable(core->supplies.vio1); >> + if (err < 0) >> + goto disable_vd; > If the sequencing is critical here you should have comments explaining > what the requirement is, otherwise this looks like a prime candidate > for conversion to regulator_bulk_enable() (and similarly for all the > other regulator usage, it appears that all the regulators are worked > with in a bulk fashion). > Unfortunately the datasheet is not very clear on the subject. My suspicion at the time of writing was that since there are multiple power domains that it was possible to power up only the ones required for specific subsystem in use. Unfortunately I didn't and still don't have the hardware to test it on(on the board I use there are no dedicated regulators and the chips are always powered on). I'll convert it to the code that uses regulators in bulk fashion. Andrey Smirnov -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
