Re: [PATCH 0/4] Add support for Exynos clock output configuration
Hi Daniel, On 18.06.2014 18:17, Daniel Drake wrote: Hi Tomasz, On Tue, May 20, 2014 at 5:43 PM, Tomasz Figa t.f...@samsung.com wrote: Since the block responsible for handling the pin is PMU, not CMU, a separate driver, that binds to PMU node is required and acquires all input clocks by standard DT clock look-up. This way we don't need any cross-IP block drivers and cross-driver register sharing or nodes for fake devices. To represent the PMU mux/gate clock, generic composite clock is registered. Tested on Odroid U3, with HSIC/USB hub using CLKOUT as reference clock, with some additional patches. Depends on: [PATCHv5 0/4] Enable usbphy and hsotg for exynos4 (No link, sorry, I could not find it in any archive yet...) for Exynos4210/4x12 PMU binding and DT nodes. This isn't working for me. Testing linus master e99cfa2d0634881b8a41d56c48b5956b9a3ba162 plus: ARM: dts: exynos4: add port sub-nodes to exynos usb host modules ARM: dts: exynos4412-odroidx: enable common hardware blocks ARM: dts: exynos4412-odroidx: add support for USB (phy, host, device) ARM: dts: refactor Odroid DTS file and add support for Odroid X2 and U2/U3 Testing on ODROID-U2. I apply the first patch here (clk: samsung: exynos4: Add missing CPU/DMC clock hierarchy) and things continue to work. Now when I add the second patch clk: samsung: exynos4: Add CLKOUT clock hierarchy boot hangs at: [4.753740] s3c-rtc 1007.rtc: setting system clock to 2000-01-01 02:43:30 UTC (946694610) [4.753809] ### dt-test ### No testcase data in device tree; not running tests [4.791155] gps-power-domain: Power-off latency e Any ideas? Thanks for testing. It's most likely an issue with ISP power domain. Certain clock controller registers are located there and there is no proper handling of this in the clock driver yet. You can test things by skipping patch 3 and this should give you a working clkout, just limited to clocks not going through clkout hierarchies of clock domains, e.g. xxti and xusbxti. I'm aware of the problem and will fix it in next version. Best regards, Tomasz -- To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 0/4] Add support for Exynos clock output configuration
Hi Tomasz, On Tue, May 20, 2014 at 5:43 PM, Tomasz Figa t.f...@samsung.com wrote: Since the block responsible for handling the pin is PMU, not CMU, a separate driver, that binds to PMU node is required and acquires all input clocks by standard DT clock look-up. This way we don't need any cross-IP block drivers and cross-driver register sharing or nodes for fake devices. To represent the PMU mux/gate clock, generic composite clock is registered. Tested on Odroid U3, with HSIC/USB hub using CLKOUT as reference clock, with some additional patches. Depends on: [PATCHv5 0/4] Enable usbphy and hsotg for exynos4 (No link, sorry, I could not find it in any archive yet...) for Exynos4210/4x12 PMU binding and DT nodes. This isn't working for me. Testing linus master e99cfa2d0634881b8a41d56c48b5956b9a3ba162 plus: ARM: dts: exynos4: add port sub-nodes to exynos usb host modules ARM: dts: exynos4412-odroidx: enable common hardware blocks ARM: dts: exynos4412-odroidx: add support for USB (phy, host, device) ARM: dts: refactor Odroid DTS file and add support for Odroid X2 and U2/U3 Testing on ODROID-U2. I apply the first patch here (clk: samsung: exynos4: Add missing CPU/DMC clock hierarchy) and things continue to work. Now when I add the second patch clk: samsung: exynos4: Add CLKOUT clock hierarchy boot hangs at: [4.753740] s3c-rtc 1007.rtc: setting system clock to 2000-01-01 02:43:30 UTC (946694610) [4.753809] ### dt-test ### No testcase data in device tree; not running tests [4.791155] gps-power-domain: Power-off latency e Any ideas? Thanks Daniel -- To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH 0/4] Add support for Exynos clock output configuration
On all Exynos SoCs there is a dedicated CLKOUT pin that allows many of internal SoC clocks to be output from the SoC. The hardware structure of CLKOUT related clocks looks as follows: CMU |--- clock0 - | PMU | | | | several |--- clock1 - | mux | muxes | | + |--- CLKOUT dividers| ... | gate| and gates | | | |--- clockN - | | Since the block responsible for handling the pin is PMU, not CMU, a separate driver, that binds to PMU node is required and acquires all input clocks by standard DT clock look-up. This way we don't need any cross-IP block drivers and cross-driver register sharing or nodes for fake devices. To represent the PMU mux/gate clock, generic composite clock is registered. Tested on Odroid U3, with HSIC/USB hub using CLKOUT as reference clock, with some additional patches. Depends on: [PATCHv5 0/4] Enable usbphy and hsotg for exynos4 (No link, sorry, I could not find it in any archive yet...) for Exynos4210/4x12 PMU binding and DT nodes. Changes since RFC v1: (https://lkml.org/lkml/2014/5/15/506) - rebased onto v5 of Enable usbphy and hsotg for exynos4 series and current HEAD of samsung-clk tree, - added handling of suspend/resume in the driver, - added missing CPU clocks on Exynos4, - added CLK_SET_RATE_PARENT to CMU CLKOUT gates on Exynos4, - fixed bit field width on Exynos4, - added CLKOUT CMU registers of Exynos4 to save/restore list, - added CLK_SET_RATE_PARENT and CLK_SET_RATE_NO_REPARENT to clkout clock, - changed the binding to use 1-cell clock specifier to allow extension with further PMU clocks in future, if needed. Tomasz Figa (4): clk: samsung: exynos4: Add missing CPU/DMC clock hierarchy clk: samsung: exynos4: Add CLKOUT clock hierarchy clk: samsung: Add driver to control CLKOUT line on Exynos SoCs ARM: dts: exynos: Update PMU node with CLKOUT related data .../devicetree/bindings/arm/samsung/pmu.txt| 30 arch/arm/boot/dts/exynos4.dtsi | 1 + arch/arm/boot/dts/exynos4210.dtsi | 9 ++ arch/arm/boot/dts/exynos4x12.dtsi | 6 + arch/arm/boot/dts/exynos5250.dtsi | 3 + arch/arm/boot/dts/exynos5420.dtsi | 3 + drivers/clk/samsung/Makefile | 1 + drivers/clk/samsung/clk-exynos-clkout.c| 153 ++ drivers/clk/samsung/clk-exynos4.c | 173 + include/dt-bindings/clock/exynos4.h| 6 + 10 files changed, 385 insertions(+) create mode 100644 drivers/clk/samsung/clk-exynos-clkout.c -- 1.9.3 -- To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html