Messages in this thread | | | Subject | Re: [PATCH v4 6/7] ARM: davinci: convert to common clock framework | From | David Lechner <> | Date | Thu, 4 Jan 2018 20:59:19 -0600 |
| |
On 01/04/2018 11:50 AM, David Lechner wrote: > > > On 1/4/18 6:39 AM, Sekhar Nori wrote: >> On Monday 01 January 2018 05:09 AM, David Lechner wrote: >>> This converts all of arch/arm/mach-davinci to the common clock framework. >>> The clock drivers from clock.c and psc.c have been moved to drivers/clk, >>> so these files are removed. >>> >>> There is one subtle change in the clock trees. AUX, BPDIV and OSCDIV >>> clocks now have "ref_clk" as a parent instead of the PLL clock. These >>> clocks are part of the PLL's MMIO block, but they bypass the PLL and >>> therefore it makes more sense to have "ref_clk" as their parent since >>> "ref_clk" is the input clock of the PLL. >>> >>> CONFIG_DAVINCI_RESET_CLOCKS is removed since the common clock frameworks >>> takes care of disabling unused clocks. >>> >>> Known issue: This breaks CPU frequency scaling on da850. >> >> This functionality needs to be restored as part of this series since we >> cannot commit anything with regressions. >> > > Do you have a suggestion on how to accomplish this? I don't have a board for testing, so I don't have a way of knowing if my changes will work or not. > >>> >>> Also, the order of #includes are cleaned up in files while we are touching >>> this code. >>> >>> Signed-off-by: David Lechner <david@lechnology.com> >> >> This is a pretty huge patch again and I hope it can be broken down. >> Ideally one per SoC converted and then the unused code removal. >> > > Will do.
Well, I can do this, but I don't think it will compile or run. We can't have the common clock framework and the legacy davinci clocks enabled at the same time.
| |