On Sat, Jun 23, 2012 at 06:35:01PM +0200, Marc Dietrich wrote:
> > The regulator configurations were all taken from the AC100 kernel used by
> > the Ubuntu port, specifically:
These generally all look pretty broken...
> > + regulator@0 {
> > + reg = <0>;
> > + regulator-compatible = "sm0";
> > + regulator-name = "+1.2vs_sm0";
> > + regulator-min-microvolt = < 725000>;
> > + regulator-max-microvolt = <1300000>;
Most of these ranges look suspiciously like the maximum possible
variation the regulator has, not what the board actually requires (which
is a depressingly common error, I've no idea why people seem to think
they're supposed to cut'n'paste the physical limits of the regualtor out
of the driver). If something decides to take advantage of the variation
this could be problematic.
> > + regulator@3 {
> > + reg = <3>;
> > + regulator-compatible = "ldo0";
> > + regulator-name = "+3.3vs_ldo0";
> > + regulator-min-microvolt = <1250000>;
> I think the common sense was that this should also be 3.3 V as it is the pex
> clock (which is not used at all on this board). I guess it doesn't matter
> much. So ...
> Acked-By: Marc Dietrich <[email protected]>
> > + regulator-max-microvolt = <3300000>;
This is one example, it looks like the rail needs to be fixed to 3.3V.
--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
