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

From	Shawn Guo <shawnguo@kernel.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH] ARM: dts: imx6sx-sdb: Remove cpufreq OPP override
Date	2017-05-05 03:20 +0200
Message-ID	<tDA4x-2Wa-1@gated-at.bofh.it> (permalink)
References	(5 earlier) <tDlRT-21n-3@gated-at.bofh.it> <tDomJ-3tZ-7@gated-at.bofh.it> <tDoG5-3Su-3@gated-at.bofh.it> <tDpiN-47A-3@gated-at.bofh.it> <tDqHT-5dq-15@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

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?

Shawn

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Re: [PATCH] ARM: dts: imx6sx-sdb: Remove cpufreq OPP override Fabio Estevam <festevam@gmail.com> - 2017-04-25 19:10 +0200
  Re: [PATCH] ARM: dts: imx6sx-sdb: Remove cpufreq OPP override Fabio Estevam <festevam@gmail.com> - 2017-04-25 19:10 +0200
    Re: [PATCH] ARM: dts: imx6sx-sdb: Remove cpufreq OPP override Fabio Estevam <festevam@gmail.com> - 2017-04-25 19:30 +0200
    Re: [PATCH] ARM: dts: imx6sx-sdb: Remove cpufreq OPP override Marek Vasut <marex@denx.de> - 2017-04-25 19:30 +0200
      Re: [PATCH] ARM: dts: imx6sx-sdb: Remove cpufreq OPP override Shawn Guo <shawnguo@kernel.org> - 2017-05-03 16:00 +0200
        Re: [PATCH] ARM: dts: imx6sx-sdb: Remove cpufreq OPP override Marek Vasut <marex@denx.de> - 2017-05-03 16:30 +0200
          Re: [PATCH] ARM: dts: imx6sx-sdb: Remove cpufreq OPP override Marek Vasut <marex@denx.de> - 2017-05-03 16:40 +0200
            Re: [PATCH] ARM: dts: imx6sx-sdb: Remove cpufreq OPP override Shawn Guo <shawnguo@kernel.org> - 2017-05-03 16:50 +0200
              Re: [PATCH] ARM: dts: imx6sx-sdb: Remove cpufreq OPP override Marek Vasut <marex@denx.de> - 2017-05-03 17:00 +0200
          Re: [PATCH] ARM: dts: imx6sx-sdb: Remove cpufreq OPP override Marek Vasut <marex@denx.de> - 2017-05-03 18:00 +0200
            Re: [PATCH] ARM: dts: imx6sx-sdb: Remove cpufreq OPP override Marek Vasut <marex@denx.de> - 2017-05-03 21:50 +0200
              Re: [PATCH] ARM: dts: imx6sx-sdb: Remove cpufreq OPP override Marek Vasut <marex@denx.de> - 2017-05-04 12:10 +0200
                Re: [PATCH] ARM: dts: imx6sx-sdb: Remove cpufreq OPP override Shawn Guo <shawnguo@kernel.org> - 2017-05-04 14:50 +0200
                Re: [PATCH] ARM: dts: imx6sx-sdb: Remove cpufreq OPP override Marek Vasut <marex@denx.de> - 2017-05-04 15:10 +0200
                Re: [PATCH] ARM: dts: imx6sx-sdb: Remove cpufreq OPP override Shawn Guo <shawnguo@kernel.org> - 2017-05-04 15:50 +0200
                Re: [PATCH] ARM: dts: imx6sx-sdb: Remove cpufreq OPP override Marek Vasut <marex@denx.de> - 2017-05-04 17:20 +0200
                Re: [PATCH] ARM: dts: imx6sx-sdb: Remove cpufreq OPP override Shawn Guo <shawnguo@kernel.org> - 2017-05-05 03:20 +0200

csiph-web