Re: [PATCH v10 00/27] ARM: davinci: convert to common clock frameworkâ

[David Lechner]

On 05/13/2018 07:40 PM, Adam Ford wrote:
> On Wed, May 9, 2018 at 12:25 PM, David Lechner <david@xxxxxxxxxxxxxx> wrote:
> > This series converts mach-davinci to use the common clock framework.
> >
> > The series works like this, the first 3 patches fix some issues with the clock
> > drivers that have already been accepted into the mainline kernel.
> >
> > Then, starting with "ARM: davinci: pass clock as parameter to
> > davinci_timer_init()", we get the mach code ready for the switch by adding the
> > code needed for the new clock drivers and adding #ifndef CONFIG_COMMON_CLK
> > around the legacy clocks so that we can switch easily between the old and the
> > new.
> >
> > "ARM: davinci: switch to common clock framework" actually flips the switch
> > to start using the new clock drivers. Then the next 8 patches remove all
> > of the old clock code.
> >
> > The final four patches add device tree clock support to the one SoC that
> > supports it.
> >
> > This series has been tested on TI OMAP-L138 LCDK (both device tree and legacy
> > board file).
>
> I am not sure if I did something wrong, but I attempted to build and I
> wasn't able to boot the da850-evm.dtb your repo common-clk-v11,
> however the legacy board file boot was OK.
>
> make davinci_all_defconfig ARCH=arm
> make zImage modules da850-evm.dtb ARCH=arm CROSS_COMPILE=arm-linux- -j8
>
> 3140416 bytes read in 1464 ms (2 MiB/s)
> 20353 bytes read in 15 ms (1.3 MiB/s)
> ## Flattened Device Tree blob at c0600000
>     Booting using the fdt blob at 0xc0600000
>     Loading Device Tree to c7e57000, end c7e5ef80 ... OK
>
> Starting kernel ...
>
> Uncompressing Linux... done, booting the kernel.
>
> (and hang)
>
> If you have some suggestions, I am try them as I get time.

As with each revision of this series, I have included debugging tips
in the cover letter, e.g. earlyprink and not disabling unused clocks.