Re: [PATCH] ARM: dts: imx6sx-sdb: Remove cpufreq OPP override

From: Shawn Guo
Date: Thu May 04 2017 - 21:19:43 EST

On Thu, May 04, 2017 at 04:34:14PM +0200, Marek Vasut wrote:
> On 05/04/2017 03:41 PM, Shawn Guo wrote:
> > So I guess you do not understand how the OPP hackery was born and why it
> > shouldn't be there for mainline kernel at all.
> The OPP hackery is there to keep both regulators configured the same
> way, since they share the same input voltage rail IMO.

Yes. But configuring both regulators the same way is only required in
vendor kernel where 'LDO bypass' mode is used. With 'LDO enable' mode
which is the case for upstream kernel, both regulators can be configured
differently even they share the same input rail.

> If you model the
> power distribution correctly, the OPP hackery can be removed.

The OPP hackery can be removed even without reg_arm/reg_soc modeling.
That's why we can do hackery dropping and reg_arm/reg_soc modeling in
separate patches.

@Leonard, if someday we support 'LDO bypass' mode in upstream kernel,
the OPP hackery needs to be back in some way even with reg_arm/reg_soc
modeling in place, right? Or will we have a better way to ensure SW1A
rail can always feed a correct voltage directly to reg_arm&reg_soc?