Re: [PATCH v10 00/27] ARM: davinci: convert to common clock framework
On Wed, May 9, 2018 at 12:25 PM, David Lechner 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 find. I don't know what I did wrong, but it's working fine. If you want to add my 'tested-by' go ahead. Tested-by: Adam Ford #da850-evm, ethernet, spi flash, SD, UART > > Changes: > > v10 changes (also see individual patches for details): > - Reworked device tree bindings for DaVinci timer. > - Dropped helper functions to conditionally call devm_* versions of functions > - Fix some typos > - Fix some rebasing issues introduced in v9 > > v9 changes (also see individual patches for details): > - Rebased on linux-davnci/master (f5e3203bb775) > - Dropped drivers/clk patches that landed in v4.17 > - New drivers/clk patches for early boot special case > - New patch for ti,davinci-timer device tree bindings > - Updated mach/davinci patches to register clocks in early boot when needed > > v8 changes (also see individual patches for details): > - Rebased on linux-davinci/master > - Dropped use of __init and __initconst attributes in clk drivers > - Add clkdev lookups for PLL SYSCLKs > - Fix genpd clock reference counting issue > - Fix PSC clock driver loading order issue > - Fix typo in device tree and add more power-domains properties > > v7 changes (also see individual patches for details): > - Rebased on linux-davinci/master (v4.16-rc) > - Convert clock drivers to platform devices > - New patch "ARM: davinci: pass clock as parameter to davinci_timer_init()" > - Fix issues with lcdk and aemif clock lookups and power domains > - Fixed other minor issues brought up in v6 review > > v6 changes (also see individual patches for details): > - All of the device tree bindings are changed > - All of the clock drivers are changed significantly > - Fixed issues brought up during review of v5 > - "ARM: davinci: move davinci_clk_init() to init_time" is removed from this > series and submitted separately > > v5 changes: > - Basically, this is an entirely new series > - Patches are broken up into bite-sized pieces > - Converted PSC clock driver to use regmap > - Restored "force" flag for certain DA850 clocks > - Added device tree bindings > - Moved more of the clock init to drivers/clk > - Fixed frequency scaling (maybe*) > > * I have frequency scaling using cpufreq-dt, so I know the clocks are doing > what they need to do to make this work, but I haven't figured out how to > test davinci-cpufreq driver yet. (Patches to make cpufreq-dt work will be > sent separately after this series has landed.) > > > Dependencies: > > There are still some outstanding fixes to get everything working correctly. > These are all just runtime dependencies and only needed for certain platforms. > > - "drm/tilcdc: Fix setting clock divider for omap-l138"[1] > - "clk: davinci: pll-dm355: fix SYSCLKn parent names"[2] > - "remoteproc/davinci: common clock framework related fixes"[3] > > [1]: https://patchwork.freedesktop.org/patch/210696/ > [2]: https://lkml.org/lkml/2018/5/9/626 > [3]: https://lkml.org/lkml/2018/5/2/201 > > You can find a working branch with everything included (plus a few extras, > like > cpufreq-dt) in the "common-clk-v10" branch of > https://github.com/dlech/ev3dev-kernel.git. > > > Testing/debugging for the uninitiated: > > I only have one device to test with, which is based on da850, so I will > have to rely on others to do some testing here. Since we are dealing with > clocks, if something isn't working, you most likely won't see output on > the serial port. To figure out what is going on, you need to enable... > > CONFIG_DEBUG_LL=y > CONFIG_EARLY_PRINTK=y > > and add "earlyprintk clk_ignore_unused" to the kernel command line options. > You may need to select a different UART for this depending on your board. I > think UART1 is the default in the kernel configuration. > > On da850 devices comment out the lines: > > /* pll1_sysclk2 is not affected by CPU scaling, so use it for async3 > */ > parent = clk_hw_get_parent_by_index(&mux->hw, 1); > if (parent) > clk_set_parent(mux->hw.clk, parent->clk); > else >
Re: [PATCH v10 00/27] ARM: davinci: convert to common clock framework
On Wednesday 16 May 2018 01:17 PM, Bartosz Golaszewski wrote:
> 2018-05-16 0:44 GMT+02:00 Adam Ford :
>> On Tue, May 15, 2018 at 4:25 AM, Bartosz Golaszewski wrote:
>>> 2018-05-14 2:40 GMT+02:00 Adam Ford :
On Wed, May 9, 2018 at 12:25 PM, David Lechner
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 c060
Booting using the fdt blob at 0xc060
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.
adam
>>>
>>> Runs fine on da850-lcdk and dm365-evm. I'll test the da850-evm
>>> tomorrow when I'll have access to it.
>>
>> I set the bootargs to: bootargs=console=ttyS2,115200n8
>> clk_ignore_unused root=/dev/mmcblk0p2 rw rootfstype=ext4 rootwait
>>
>> I enabled DEBUG_LL and EARLY_PRINTK, yet when it loads, I only get:
With DEBUG_LL, you dont get any prints "automatically". You need add
printascii() calls where you need. I use the attached patch which
patches printk() calls with printascii()[1]
>>
>> ## Flattened Device Tree blob at c060
>>Booting using the fdt blob at 0xc060
>>Loading Device Tree to c7e57000, end c7e5ef35 ... OK
>>
>> Starting kernel ...
>>
>> Uncompressing Linux... done, booting the kernel.
>>
>>
>> I am doing this at my home, so I don't have a debugger for the
>> DA850-EVM. I am using a SOM that is an AM1808, but I vaguely remember
>> something about enabling a DSP clock somewhere, but I cannot seem to
>> find the e-mail. I know its counter intuitive that we'd need to
>> enable a clock that runs the DSP since it doesn't exist on the AM1808,
>> but I would have thought the clk_ignore_unused would have worked
>> around that issue.
>>
>> If someone else has a DA850-EVM or suggestions, I'm willing to try
>> them as I have time.
>>
>> adam
>
> Hi Adam,
>
> everything works fine for me both when booting the DTB and in legacy
> mode on da850-evm.
>
> I'm using the following bootargs:
> ip=dhcp console=ttyS2,115200n8 root=/dev/nfs rw nfsroot=,v3
> nfsrootdebug
>
> Regular davinci_all_defconfig on David's common-clk-v11 branch.
Adam, if you still cannot get it to work, one problem could be that the
DT size has increased and its being overwritten. You could look at where
you are loading various binaries. Or append the .dtb to zImage and
switch on APPENDED_DTB config in kernel.
Thanks,
Sekhar
[1]
diff --git a/kernel/printk/printk.c b/kernel/printk/printk.c
index 5b5a708..c3419c0 100644
--- a/kernel/printk/printk.c
+++ b/kernel/printk/printk.c
@@ -1488,6 +1488,7 @@ static size_t cont_print_text(char *text, size_t size)
return textlen;
}
+void printascii(char *);
asmlinkage int vprintk_emit(int facility, int level,
const char *dict, size_t dictlen,
const char *fmt, va_list args)
@@ -1552,6 +1553,7 @@ asmlinkage int vprintk_emit(int facility, int level,
text_len--;
lflags |= LOG_NEWLINE;
}
+ printascii(text);
/* strip kernel syslog prefix and extract log level or control flags */
if (facility == 0) {
Re: [PATCH v10 00/27] ARM: davinci: convert to common clock framework
2018-05-16 0:44 GMT+02:00 Adam Ford : > On Tue, May 15, 2018 at 4:25 AM, Bartosz Golaszewski wrote: >> 2018-05-14 2:40 GMT+02:00 Adam Ford : >>> On Wed, May 9, 2018 at 12:25 PM, David Lechner 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 c060 >>>Booting using the fdt blob at 0xc060 >>>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. >>> >>> adam >>> >> >> Runs fine on da850-lcdk and dm365-evm. I'll test the da850-evm >> tomorrow when I'll have access to it. > > I set the bootargs to: bootargs=console=ttyS2,115200n8 > clk_ignore_unused root=/dev/mmcblk0p2 rw rootfstype=ext4 rootwait > > I enabled DEBUG_LL and EARLY_PRINTK, yet when it loads, I only get: > > ## Flattened Device Tree blob at c060 >Booting using the fdt blob at 0xc060 >Loading Device Tree to c7e57000, end c7e5ef35 ... OK > > Starting kernel ... > > Uncompressing Linux... done, booting the kernel. > > > I am doing this at my home, so I don't have a debugger for the > DA850-EVM. I am using a SOM that is an AM1808, but I vaguely remember > something about enabling a DSP clock somewhere, but I cannot seem to > find the e-mail. I know its counter intuitive that we'd need to > enable a clock that runs the DSP since it doesn't exist on the AM1808, > but I would have thought the clk_ignore_unused would have worked > around that issue. > > If someone else has a DA850-EVM or suggestions, I'm willing to try > them as I have time. > > adam Hi Adam, everything works fine for me both when booting the DTB and in legacy mode on da850-evm. I'm using the following bootargs: ip=dhcp console=ttyS2,115200n8 root=/dev/nfs rw nfsroot=,v3 nfsrootdebug Regular davinci_all_defconfig on David's common-clk-v11 branch. Best regards, Bartosz Golaszewski
Re: [PATCH v10 00/27] ARM: davinci: convert to common clock framework
On 5/15/18 5:44 PM, Adam Ford wrote: On Tue, May 15, 2018 at 4:25 AM, Bartosz Golaszewski wrote: 2018-05-14 2:40 GMT+02:00 Adam Ford : On Wed, May 9, 2018 at 12:25 PM, David Lechner 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 c060 Booting using the fdt blob at 0xc060 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. adam Runs fine on da850-lcdk and dm365-evm. I'll test the da850-evm tomorrow when I'll have access to it. I set the bootargs to: bootargs=console=ttyS2,115200n8 clk_ignore_unused root=/dev/mmcblk0p2 rw rootfstype=ext4 rootwait It looks like you forgot earlyprintk in your bootargs. I enabled DEBUG_LL and EARLY_PRINTK, yet when it loads, I only get: ## Flattened Device Tree blob at c060 Booting using the fdt blob at 0xc060 Loading Device Tree to c7e57000, end c7e5ef35 ... OK Starting kernel ... Uncompressing Linux... done, booting the kernel. I am doing this at my home, so I don't have a debugger for the DA850-EVM. I am using a SOM that is an AM1808, but I vaguely remember something about enabling a DSP clock somewhere, but I cannot seem to find the e-mail. I know its counter intuitive that we'd need to enable a clock that runs the DSP since it doesn't exist on the AM1808, but I would have thought the clk_ignore_unused would have worked around that issue. If someone else has a DA850-EVM or suggestions, I'm willing to try them as I have time. adam Bart
Re: [PATCH v10 00/27] ARM: davinci: convert to common clock framework
On Tue, May 15, 2018 at 4:25 AM, Bartosz Golaszewski wrote: > 2018-05-14 2:40 GMT+02:00 Adam Ford : >> On Wed, May 9, 2018 at 12:25 PM, David Lechner 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 c060 >>Booting using the fdt blob at 0xc060 >>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. >> >> adam >> > > Runs fine on da850-lcdk and dm365-evm. I'll test the da850-evm > tomorrow when I'll have access to it. I set the bootargs to: bootargs=console=ttyS2,115200n8 clk_ignore_unused root=/dev/mmcblk0p2 rw rootfstype=ext4 rootwait I enabled DEBUG_LL and EARLY_PRINTK, yet when it loads, I only get: ## Flattened Device Tree blob at c060 Booting using the fdt blob at 0xc060 Loading Device Tree to c7e57000, end c7e5ef35 ... OK Starting kernel ... Uncompressing Linux... done, booting the kernel. I am doing this at my home, so I don't have a debugger for the DA850-EVM. I am using a SOM that is an AM1808, but I vaguely remember something about enabling a DSP clock somewhere, but I cannot seem to find the e-mail. I know its counter intuitive that we'd need to enable a clock that runs the DSP since it doesn't exist on the AM1808, but I would have thought the clk_ignore_unused would have worked around that issue. If someone else has a DA850-EVM or suggestions, I'm willing to try them as I have time. adam > > Bart
Re: [PATCH v10 00/27] ARM: davinci: convert to common clock framework
2018-05-14 2:40 GMT+02:00 Adam Ford : > On Wed, May 9, 2018 at 12:25 PM, David Lechner 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 c060 >Booting using the fdt blob at 0xc060 >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. > > adam > Runs fine on da850-lcdk and dm365-evm. I'll test the da850-evm tomorrow when I'll have access to it. Bart
Re: [PATCH v10 00/27] ARM: davinci: convert to common clock framework
On 05/13/2018 07:40 PM, Adam Ford wrote: On Wed, May 9, 2018 at 12:25 PM, David Lechner 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 c060 Booting using the fdt blob at 0xc060 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.
Re: [PATCH v10 00/27] ARM: davinci: convert to common clock framework
On Wed, May 9, 2018 at 12:25 PM, David Lechner 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 c060 Booting using the fdt blob at 0xc060 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. adam > > Changes: > > v10 changes (also see individual patches for details): > - Reworked device tree bindings for DaVinci timer. > - Dropped helper functions to conditionally call devm_* versions of functions > - Fix some typos > - Fix some rebasing issues introduced in v9 > > v9 changes (also see individual patches for details): > - Rebased on linux-davnci/master (f5e3203bb775) > - Dropped drivers/clk patches that landed in v4.17 > - New drivers/clk patches for early boot special case > - New patch for ti,davinci-timer device tree bindings > - Updated mach/davinci patches to register clocks in early boot when needed > > v8 changes (also see individual patches for details): > - Rebased on linux-davinci/master > - Dropped use of __init and __initconst attributes in clk drivers > - Add clkdev lookups for PLL SYSCLKs > - Fix genpd clock reference counting issue > - Fix PSC clock driver loading order issue > - Fix typo in device tree and add more power-domains properties > > v7 changes (also see individual patches for details): > - Rebased on linux-davinci/master (v4.16-rc) > - Convert clock drivers to platform devices > - New patch "ARM: davinci: pass clock as parameter to davinci_timer_init()" > - Fix issues with lcdk and aemif clock lookups and power domains > - Fixed other minor issues brought up in v6 review > > v6 changes (also see individual patches for details): > - All of the device tree bindings are changed > - All of the clock drivers are changed significantly > - Fixed issues brought up during review of v5 > - "ARM: davinci: move davinci_clk_init() to init_time" is removed from this > series and submitted separately > > v5 changes: > - Basically, this is an entirely new series > - Patches are broken up into bite-sized pieces > - Converted PSC clock driver to use regmap > - Restored "force" flag for certain DA850 clocks > - Added device tree bindings > - Moved more of the clock init to drivers/clk > - Fixed frequency scaling (maybe*) > > * I have frequency scaling using cpufreq-dt, so I know the clocks are doing > what they need to do to make this work, but I haven't figured out how to > test davinci-cpufreq driver yet. (Patches to make cpufreq-dt work will be > sent separately after this series has landed.) > > > Dependencies: > > There are still some outstanding fixes to get everything working correctly. > These are all just runtime dependencies and only needed for certain platforms. > > - "drm/tilcdc: Fix setting clock divider for omap-l138"[1] > - "clk: davinci: pll-dm355: fix SYSCLKn parent names"[2] > - "remoteproc/davinci: common clock framework related fixes"[3] > > [1]: https://patchwork.freedesktop.org/patch/210696/ > [2]: https://lkml.org/lkml/2018/5/9/626 > [3]: https://lkml.org/lkml/2018/5/2/201 > > You can find a working branch with everything included (plus a few extras, > like > cpufreq-dt) in the "common-clk-v10" branch of > https://github.com/dlech/ev3dev-kernel.git. > > > Testing/debugging for the uninitiated: > > I only have one device to test with, which is based on da850, so I will > have to rely on others to do some testing here. Since we are dealing with > clocks, if something isn't working, you most likely won't see output on > the serial port. To figure out what is going on, you need to enable... > > CONFIG_DEBUG_LL=y > CONFIG_EARLY_PRINTK=y > > and add "earlyprintk
Re: [PATCH v10 00/27] ARM: davinci: convert to common clock framework
On 05/11/2018 10:26 AM, Sekhar Nori wrote: Hi David, On Wednesday 09 May 2018 10:55 PM, David Lechner 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. I have not yet looked at the patches, but I got a bunch of W=1 warnings and some sparse warnings when building your branch. Please take a look at these. Unfortunately the output is mixed between sparse and compiler. The "expression using sizeof(void)" can be ignored as its a known issue with sparse, I believe. I've started a common-clk-v11 branch on my GitHub that fixes most of these. Also submitted "clk: davinci: psc-dm355: fix ASP0/1 clkdev lookups" that fixes a couple more. I've purposely not fixed the davinci_clk_reset_* functions since there is already a patch that will remove those functions in the future. I'll wait a bit longer for DT review before re-sending v11 of this series.
Re: [PATCH v10 00/27] ARM: davinci: convert to common clock framework
Hi David,
On Wednesday 09 May 2018 10:55 PM, David Lechner 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.
I have not yet looked at the patches, but I got a bunch of W=1 warnings
and some sparse warnings when building your branch. Please take a look
at these. Unfortunately the output is mixed between sparse and compiler.
The "expression using sizeof(void)" can be ignored as its a known issue
with sparse, I believe.
Thanks,
Sekhar
drivers/clk/davinci/pll-da830.c:39:5: warning: symbol 'da830_pll_init' was not
declared. Should it be static?
drivers/clk/davinci/pll.c:142:16: warning: expression using sizeof(void)
drivers/clk/davinci/pll.c:142:16: warning: expression using sizeof(void)
drivers/clk/davinci/pll-da850.c:87:5: warning: symbol 'da850_pll0_init' was not
declared. Should it be static?
drivers/clk/davinci/pll-da830.c:39:5: warning: no previous prototype for
‘da830_pll_init’ [-Wmissing-prototypes]
int da830_pll_init(struct device *dev, void __iomem *base, struct regmap
*cfgchip)
^~
drivers/clk/davinci/pll-da850.c:87:5: warning: no previous prototype for
‘da850_pll0_init’ [-Wmissing-prototypes]
int da850_pll0_init(struct device *dev, void __iomem *base, struct regmap
*cfgchip)
^~~
drivers/clk/davinci/pll-dm355.c:30:5: warning: symbol 'dm355_pll1_init' was not
declared. Should it be static?
drivers/clk/davinci/pll-dm365.c:59:5: warning: symbol 'dm365_pll1_init' was not
declared. Should it be static?
drivers/clk/davinci/pll-dm365.c:122:5: warning: symbol 'dm365_pll2_init' was
not declared. Should it be static?
drivers/clk/davinci/da8xx-cfgchip.c:581: warning: Function parameter or member
'dev' not described in 'da8xx_cfgchip_register_usb1_clk48'
drivers/clk/davinci/pll-dm646x.c:32:5: warning: symbol 'dm646x_pll1_init' was
not declared. Should it be static?
drivers/clk/davinci/pll-dm644x.c:30:5: warning: symbol 'dm644x_pll1_init' was
not declared. Should it be static?
drivers/clk/davinci/pll-dm365.c:59:5: warning: no previous prototype for
‘dm365_pll1_init’ [-Wmissing-prototypes]
int dm365_pll1_init(struct device *dev, void __iomem *base, struct regmap
*cfgchip)
^~~
drivers/clk/davinci/pll-dm365.c:122:5: warning: no previous prototype for
‘dm365_pll2_init’ [-Wmissing-prototypes]
int dm365_pll2_init(struct device *dev, void __iomem *base, struct regmap
*cfgchip)
^~~
drivers/clk/davinci/pll-dm355.c:30:5: warning: no previous prototype for
‘dm355_pll1_init’ [-Wmissing-prototypes]
int dm355_pll1_init(struct device *dev, void __iomem *base, struct regmap
*cfgchip)
^~~
drivers/clk/davinci/psc.c:310:5: warning: symbol 'davinci_clk_reset_assert' was
not declared. Should it be static?
drivers/clk/davinci/psc.c:316:5: warning: symbol 'davinci_clk_reset_deassert'
was not declared. Should it be static?
drivers/clk/davinci/psc-dm644x.c:66:5: warning: symbol 'dm644x_psc_init' was
not declared. Should it be static?
drivers/clk/davinci/psc-dm355.c:71:5: warning: symbol 'dm355_psc_init' was not
declared. Should it be static?
drivers/clk/davinci/pll-dm646x.c:32:5: warning: no previous prototype for
‘dm646x_pll1_init’ [-Wmissing-prototypes]
int dm646x_pll1_init(struct device *dev, void __iomem *base, struct regmap
*cfgchip)
^~~~
drivers/clk/davinci/psc-dm365.c:76:5: warning: symbol 'dm365_psc_init' was not
declared. Should it be static?
drivers/clk/davinci/psc-dm646x.c:61:5: warning: symbol 'dm646x_psc_init' was
not declared. Should it be static?
drivers/clk/davinci/pll-dm644x.c:30:5: warning: no previous prototype for
‘dm644x_pll1_init’ [-Wmissing-prototypes]
int dm644x_pll1_init(struct device *dev, void __iomem *base, struct regmap
*cfgchip)
^~~~
drivers/clk/davinci/psc-dm355.c:71:5: warning: no previous prototype for
‘dm355_psc_init’ [-Wmissing-prototypes]
int dm355_psc_init(struct device *dev, void __iomem *base)
^~
In file included from drivers/clk/davinci/psc-dm355.c:15:0:
drivers/clk/davinci/psc-dm355.c:26:14: warning: ‘mcbsp0_clkdev’ defined but not
used [-Wunused-const-variable=]
LPSC_CLKDEV1(mcbsp0_clkdev, NULL, "davinci-mcbsp.0");
^
drivers/clk/davinci/psc.h:31:46: note: in definition of macro ‘LPSC_CLKDEV1’
static const struct davinci_lpsc_clkdev_info n[] __initconst = { \
^
drivers/clk/davinci/psc-dm355.c:21:14: warning: ‘mcbsp1_clkdev’ defined but not
used [-Wunused-const-variable=]
LPSC_CLKDEV1(mcbsp1_clkdev, NULL, "davinci-mcbsp.1");
^
drivers/clk/davinci/psc.h:31:46: note: in definition of macro ‘LPSC_CLKDEV1’
static const struct davinci_lpsc_clkdev_info n[] __initconst = { \
^
drivers/clk/davinci/psc-dm365.c:76:5: warning: no
