On Tue, Jan 20, 2015 at 07:51:31AM +0000, [email protected] wrote: > > > > +&cpu0 { > > > > + operating-points = < > > > > + /* kHz uV */ > > > > + 996000 1250000 > > > > + 792000 1175000 > > > > + 396000 1075000 > > > > + >; > > > > + fsl,soc-operating-points = < > > > > + /* ARM kHz SOC uV */ > > > > + 996000 1175000 > > > > + 792000 1175000 > > > > + 396000 1175000 > > > > + >; > > > > > > I'm wondering why different revision of the board require a different > > > OPP settings. > > > > > > > Anson, do you know the reason? > > As i.MX6SX SDB Rev-B board uses PFuse 200 which has only one power supply for > both > VDDARM_IN and VDDSOC_IN, so it needs VDDARM_IN equal VDDSOC_IN, currently > we set Rev-B board as default in imx6sx.dtsi, which uses same voltage for both > VDDARM_IN and VDDSOC_IN.The Rev-A board uses PFuze 100 which has separate > power > supply for VDDARM_IN and VDDSOC_IN, so it can use different voltage on > VDDARM_IN > and VDDSOC_IN according to datasheet.
Thanks for the clarification, Anson. Peter, OPP is a CPU characteristic defined by data sheet, so we should define OPP according to data sheet in imx6sx.dtsi, and only overwrite the property in Rev-B dts per board/PMIC quirk. Also, please put the rationale why we need OPP quick on Rev-B into commit log. Shawn -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
