* [PATCH] ARM: dts: rockchip: Add dp83867 CLK_OUT muxing @ 2018-03-05 12:45 Daniel Schultz 2018-03-05 14:15 ` Heiko Stuebner 2018-03-05 20:25 ` Heiko Stübner 0 siblings, 2 replies; 7+ messages in thread From: Daniel Schultz @ 2018-03-05 12:45 UTC (permalink / raw) To: linux-arm-kernel The CLK_O_SEL default is synchronous to XI input clock, which is 25 MHz. Set CLK_O_SEL to channel A transmit clock so we have 125 MHz on CLK_OUT. Signed-off-by: Daniel Schultz <d.schultz@phytec.de> --- The binding will be added with the next merge of net-next: https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git/commit/?id=9708fb630d19ee51ae3aeb3a533e3010da0e8570 arch/arm/boot/dts/rk3288-phycore-som.dtsi | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/boot/dts/rk3288-phycore-som.dtsi b/arch/arm/boot/dts/rk3288-phycore-som.dtsi index bdd80aa..e60535d 100644 --- a/arch/arm/boot/dts/rk3288-phycore-som.dtsi +++ b/arch/arm/boot/dts/rk3288-phycore-som.dtsi @@ -141,6 +141,7 @@ ti,tx-internal-delay = <DP83867_RGMIIDCTL_2_00_NS>; ti,fifo-depth = <DP83867_PHYCR_FIFO_DEPTH_4_B_NIB>; enet-phy-lane-no-swap; + ti,clk-output-sel = <DP83867_CLK_O_SEL_CHN_A_TCLK>; }; }; }; -- 2.7.4 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH] ARM: dts: rockchip: Add dp83867 CLK_OUT muxing 2018-03-05 12:45 [PATCH] ARM: dts: rockchip: Add dp83867 CLK_OUT muxing Daniel Schultz @ 2018-03-05 14:15 ` Heiko Stuebner 2018-03-05 15:57 ` Daniel Schultz 2018-03-05 20:25 ` Heiko Stübner 1 sibling, 1 reply; 7+ messages in thread From: Heiko Stuebner @ 2018-03-05 14:15 UTC (permalink / raw) To: linux-arm-kernel Hi Daniel, Am Montag, 5. M?rz 2018, 13:45:11 CET schrieb Daniel Schultz: > The CLK_O_SEL default is synchronous to XI input clock, which is 25 MHz. > Set CLK_O_SEL to channel A transmit clock so we have 125 MHz on CLK_OUT. > > Signed-off-by: Daniel Schultz <d.schultz@phytec.de> > --- > > The binding will be added with the next merge of net-next: > https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git/commit/?id=9708fb630d19ee51ae3aeb3a533e3010da0e8570 I did find the commit, but no related change of the dp83867 dt binding document [0], including a review by dt-maintainers. While your property does not look overly complicated, the binding should be updated nontheless. Heiko [0] https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git/log/Documentation/devicetree/bindings/net/ti,dp83867.txt?id=9708fb630d19ee51ae3aeb3a533e3010da0e8570 > arch/arm/boot/dts/rk3288-phycore-som.dtsi | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/arm/boot/dts/rk3288-phycore-som.dtsi b/arch/arm/boot/dts/rk3288-phycore-som.dtsi > index bdd80aa..e60535d 100644 > --- a/arch/arm/boot/dts/rk3288-phycore-som.dtsi > +++ b/arch/arm/boot/dts/rk3288-phycore-som.dtsi > @@ -141,6 +141,7 @@ > ti,tx-internal-delay = <DP83867_RGMIIDCTL_2_00_NS>; > ti,fifo-depth = <DP83867_PHYCR_FIFO_DEPTH_4_B_NIB>; > enet-phy-lane-no-swap; > + ti,clk-output-sel = <DP83867_CLK_O_SEL_CHN_A_TCLK>; > }; > }; > }; > ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] ARM: dts: rockchip: Add dp83867 CLK_OUT muxing 2018-03-05 14:15 ` Heiko Stuebner @ 2018-03-05 15:57 ` Daniel Schultz 2018-03-05 17:04 ` Heiko Stübner 0 siblings, 1 reply; 7+ messages in thread From: Daniel Schultz @ 2018-03-05 15:57 UTC (permalink / raw) To: linux-arm-kernel Hi, On 03/05/2018 03:15 PM, Heiko Stuebner wrote: > Hi Daniel, > > Am Montag, 5. M?rz 2018, 13:45:11 CET schrieb Daniel Schultz: >> The CLK_O_SEL default is synchronous to XI input clock, which is 25 MHz. >> Set CLK_O_SEL to channel A transmit clock so we have 125 MHz on CLK_OUT. >> >> Signed-off-by: Daniel Schultz <d.schultz@phytec.de> >> --- >> >> The binding will be added with the next merge of net-next: >> https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git/commit/?id=9708fb630d19ee51ae3aeb3a533e3010da0e8570 https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git/log/Documentation/devicetree/bindings/net/ti,dp83867.txt If I search in the master branch, I get the patch. Did I searched wrong? Daniel > I did find the commit, but no related change of the dp83867 dt binding > document [0], including a review by dt-maintainers. > > While your property does not look overly complicated, the binding > should be updated nontheless. > > > Heiko > > [0] https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git/log/Documentation/devicetree/bindings/net/ti,dp83867.txt?id=9708fb630d19ee51ae3aeb3a533e3010da0e8570 > >> arch/arm/boot/dts/rk3288-phycore-som.dtsi | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/arch/arm/boot/dts/rk3288-phycore-som.dtsi b/arch/arm/boot/dts/rk3288-phycore-som.dtsi >> index bdd80aa..e60535d 100644 >> --- a/arch/arm/boot/dts/rk3288-phycore-som.dtsi >> +++ b/arch/arm/boot/dts/rk3288-phycore-som.dtsi >> @@ -141,6 +141,7 @@ >> ti,tx-internal-delay = <DP83867_RGMIIDCTL_2_00_NS>; >> ti,fifo-depth = <DP83867_PHYCR_FIFO_DEPTH_4_B_NIB>; >> enet-phy-lane-no-swap; >> + ti,clk-output-sel = <DP83867_CLK_O_SEL_CHN_A_TCLK>; >> }; >> }; >> }; >> > -- Mit freundlichen Gr??en, With best regards, Daniel Schultz - Entwicklung - Tel.: +49 6131 92 21 457 d.schultz at phytec.de www.phytec.de Sie finden uns auch auf: Facebook, LinkedIn, Xing, YouTube PHYTEC Messtechnik GmbH | Robert-Koch-Str. 39 | 55129 Mainz, Germany Gesch?ftsf?hrer: Dipl.-Ing. Michael Mitezki, Dipl.-Ing. Bodo Huber | Handelsregister Mainz HRB 4656 | Finanzamt Mainz-Mitte | St.Nr. 266500608, DE 149059855 This E-Mail may contain confidential or privileged information. If you are not the intended recipient (or have received this E-Mail in error) please notify the sender immediately and destroy this E-Mail. Any unauthorized copying, disclosure or distribution of the material in this E-Mail is strictly forbidden. ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] ARM: dts: rockchip: Add dp83867 CLK_OUT muxing 2018-03-05 15:57 ` Daniel Schultz @ 2018-03-05 17:04 ` Heiko Stübner 0 siblings, 0 replies; 7+ messages in thread From: Heiko Stübner @ 2018-03-05 17:04 UTC (permalink / raw) To: linux-arm-kernel Hi Daniel, Am Montag, 5. M?rz 2018, 16:57:13 CET schrieb Daniel Schultz: > On 03/05/2018 03:15 PM, Heiko Stuebner wrote: > > Am Montag, 5. M?rz 2018, 13:45:11 CET schrieb Daniel Schultz: > >> The CLK_O_SEL default is synchronous to XI input clock, which is 25 MHz. > >> Set CLK_O_SEL to channel A transmit clock so we have 125 MHz on CLK_OUT. > >> > >> Signed-off-by: Daniel Schultz <d.schultz@phytec.de> > >> --- > >> > >> The binding will be added with the next merge of net-next: > >> https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git/commit > >> /?id=9708fb630d19ee51ae3aeb3a533e3010da0e8570 > https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git/log/Docum > entation/devicetree/bindings/net/ti,dp83867.txt > > If I search in the master branch, I get the patch. Did I searched wrong? no ... seems I was just blind :-) Heiko > > I did find the commit, but no related change of the dp83867 dt binding > > document [0], including a review by dt-maintainers. > > > > While your property does not look overly complicated, the binding > > should be updated nontheless. > > > > > > Heiko > > > > [0] > > https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git/log/Do > > cumentation/devicetree/bindings/net/ti,dp83867.txt?id=9708fb630d19ee51ae3a > > eb3a533e3010da0e8570> > >> arch/arm/boot/dts/rk3288-phycore-som.dtsi | 1 + > >> 1 file changed, 1 insertion(+) > >> > >> diff --git a/arch/arm/boot/dts/rk3288-phycore-som.dtsi > >> b/arch/arm/boot/dts/rk3288-phycore-som.dtsi index bdd80aa..e60535d > >> 100644 > >> --- a/arch/arm/boot/dts/rk3288-phycore-som.dtsi > >> +++ b/arch/arm/boot/dts/rk3288-phycore-som.dtsi > >> @@ -141,6 +141,7 @@ > >> > >> ti,tx-internal-delay = <DP83867_RGMIIDCTL_2_00_NS>; > >> ti,fifo-depth = <DP83867_PHYCR_FIFO_DEPTH_4_B_NIB>; > >> enet-phy-lane-no-swap; > >> > >> + ti,clk-output-sel = <DP83867_CLK_O_SEL_CHN_A_TCLK>; > >> > >> }; > >> > >> }; > >> > >> }; ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] ARM: dts: rockchip: Add dp83867 CLK_OUT muxing 2018-03-05 12:45 [PATCH] ARM: dts: rockchip: Add dp83867 CLK_OUT muxing Daniel Schultz 2018-03-05 14:15 ` Heiko Stuebner @ 2018-03-05 20:25 ` Heiko Stübner 2018-03-05 21:08 ` Heiko Stübner 1 sibling, 1 reply; 7+ messages in thread From: Heiko Stübner @ 2018-03-05 20:25 UTC (permalink / raw) To: linux-arm-kernel Am Montag, 5. M?rz 2018, 13:45:11 CET schrieb Daniel Schultz: > The CLK_O_SEL default is synchronous to XI input clock, which is 25 MHz. > Set CLK_O_SEL to channel A transmit clock so we have 125 MHz on CLK_OUT. > > Signed-off-by: Daniel Schultz <d.schultz@phytec.de> applied for 4.17 Thanks Heiko ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] ARM: dts: rockchip: Add dp83867 CLK_OUT muxing 2018-03-05 20:25 ` Heiko Stübner @ 2018-03-05 21:08 ` Heiko Stübner 2018-03-06 9:59 ` Daniel Schultz 0 siblings, 1 reply; 7+ messages in thread From: Heiko Stübner @ 2018-03-05 21:08 UTC (permalink / raw) To: linux-arm-kernel Am Montag, 5. M?rz 2018, 21:25:30 CET schrieb Heiko St?bner: > Am Montag, 5. M?rz 2018, 13:45:11 CET schrieb Daniel Schultz: > > The CLK_O_SEL default is synchronous to XI input clock, which is 25 MHz. > > Set CLK_O_SEL to channel A transmit clock so we have 125 MHz on CLK_OUT. > > > > Signed-off-by: Daniel Schultz <d.schultz@phytec.de> > > applied for 4.17 had to move it to 4.18 though. The change to the dt-binding header goes through the networking tree and will only get merged for 4.17-rc1 . So devicetree changes in a different branch aren't available at this point. One commonly practiced alternative is that you provide a v2 with the actual hex values instead of the constants for 4.17 and a follow-up patch replacing them with the constants that I can apply for 4.18. Heiko ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] ARM: dts: rockchip: Add dp83867 CLK_OUT muxing 2018-03-05 21:08 ` Heiko Stübner @ 2018-03-06 9:59 ` Daniel Schultz 0 siblings, 0 replies; 7+ messages in thread From: Daniel Schultz @ 2018-03-06 9:59 UTC (permalink / raw) To: linux-arm-kernel Hi, On 03/05/2018 10:08 PM, Heiko St?bner wrote: > Am Montag, 5. M?rz 2018, 21:25:30 CET schrieb Heiko St?bner: >> Am Montag, 5. M?rz 2018, 13:45:11 CET schrieb Daniel Schultz: >>> The CLK_O_SEL default is synchronous to XI input clock, which is 25 MHz. >>> Set CLK_O_SEL to channel A transmit clock so we have 125 MHz on CLK_OUT. >>> >>> Signed-off-by: Daniel Schultz <d.schultz@phytec.de> >> applied for 4.17 > had to move it to 4.18 though. > > The change to the dt-binding header goes through the networking > tree and will only get merged for 4.17-rc1 . > > So devicetree changes in a different branch aren't available at this > point. > > One commonly practiced alternative is that you provide a v2 > with the actual hex values instead of the constants for 4.17 > and a follow-up patch replacing them with the constants > that I can apply for 4.18. It's not so urgent because the RK818 PMIC currently does not boot and the patch for this problem is also pending. -- Mit freundlichen Gr??en, With best regards, Daniel Schultz ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2018-03-06 9:59 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2018-03-05 12:45 [PATCH] ARM: dts: rockchip: Add dp83867 CLK_OUT muxing Daniel Schultz 2018-03-05 14:15 ` Heiko Stuebner 2018-03-05 15:57 ` Daniel Schultz 2018-03-05 17:04 ` Heiko Stübner 2018-03-05 20:25 ` Heiko Stübner 2018-03-05 21:08 ` Heiko Stübner 2018-03-06 9:59 ` Daniel Schultz
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).