On Tue, Feb 21, 2012 at 08:40:22AM -0600, Menon, Nishanth wrote: > On Tue, Feb 21, 2012 at 08:04, Tero Kristo <t-kristo@xxxxxx> wrote: > > These are now called vddmin and vddmax, as these fields will be used > > globally for selecting voltage ranges for a pmic channel, and not > > only for voltage processor. > > NAK. I think we need to setup voltage for SoC limits as well. the > programmed voltage to the VP register should be: > VP->vlimito->min = MAX(soc->vdd_min, pmic->vdd_min) > VP->vlimito->max = MIN(soc->vdd_max, pmic->vdd_max) > > else you could be running the SoC beyond design voltage potentially > damaging the device. And if you're doing that kind of thing, you must also check that the resulting min and max are sane. In other words, the minimum is less than the maximum. Sure, it's something that should never happen (because it would be a design error) but if it did happen... -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html