From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752503AbcBUC1K (ORCPT ); Sat, 20 Feb 2016 21:27:10 -0500 Received: from gloria.sntech.de ([95.129.55.99]:53868 "EHLO gloria.sntech.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751400AbcBUC1H (ORCPT ); Sat, 20 Feb 2016 21:27:07 -0500 From: Heiko Stuebner To: Caesar Wang , zhengxing Cc: linux-rockchip@lists.infradead.org, keescook@google.com, linux-kernel@vger.kernel.org, jeffy.chen@rock-chips.com, leozwang@google.com Subject: Re: [PATCH v5 2/8] clk: rockchip: rk3036: fix and add node id for emac clock Date: Sun, 21 Feb 2016 03:26:56 +0100 Message-ID: <2143221.dkVJ5VuQke@phil> User-Agent: KMail/4.14.10 (Linux/4.3.0-1-amd64; KDE/4.14.14; x86_64; ; ) In-Reply-To: <1454384899-7270-1-git-send-email-wxt@rock-chips.com> References: <1454384032-6794-1-git-send-email-wxt@rock-chips.com> <1454384899-7270-1-git-send-email-wxt@rock-chips.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Caesar, Xing, Am Dienstag, 2. Februar 2016, 11:48:19 schrieb Caesar Wang: > From: zhengxing > > In the emac driver, we need to refer HCLK_MAC since there are > only 3PLLs (APLL/GPLL/DPLL) on the rk3036, most clock are under the > GPLL, and it is unable to provide the accurate rate for mac_ref which > need to 50MHz probability, we should let it under the DPLL and are > able to set the freq which integer multiples of 50MHz, so we add these > emac node for reference. > > Signed-off-by: Xing Zheng > Signed-off-by: Caesar Wang [...] > --- a/drivers/clk/rockchip/clk-rk3036.c > +++ b/drivers/clk/rockchip/clk-rk3036.c > @@ -343,8 +343,11 @@ static struct rockchip_clk_branch > rk3036_clk_branches[] __initdata = { RK2928_CLKSEL_CON(16), 0, 2, MFLAGS, > 2, 5, DFLAGS, > RK2928_CLKGATE_CON(10), 5, GFLAGS), > > - COMPOSITE_NOGATE(0, "mac_pll_src", mux_pll_src_3plls_p, 0, > - RK2928_CLKSEL_CON(21), 0, 2, MFLAGS, 9, 5, DFLAGS), > + MUX(SCLK_MACPLL, "mac_pll_pre", mux_pll_src_3plls_p, 0, > + RK2928_CLKSEL_CON(21), 0, 2, MFLAGS), > + DIV(0, "mac_pll_src", "mac_pll_pre", 0, > + RK2928_CLKSEL_CON(21), 9, 5, DFLAGS), > + CLK_SET_RATE_NO_REPARENT should do the trick as well. And the whole hclk + clkid part should be separate patches. I took the liberty of splitting them already in [0] to see if I could get the emac running on my kylin board. Probing emac + phy does suceed, but there is no link-detection. Building your kylin-develop4.4 branch [1] results in the same (aka no transmission). Only with the original uboot + 4.1-based kernel that was already on the device did I manage to get a network connection. Is there some additional setup missing somewhere? Heiko [0] https://github.com/mmind/linux-rockchip/commits/tmp/rk3036-emac The 3 additional patches are not strictly necessary there. [1] https://github.com/rockchip-linux/kernel/tree/kylin-develop4.4