From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753018AbdLNOxD (ORCPT ); Thu, 14 Dec 2017 09:53:03 -0500 Received: from mail-wr0-f193.google.com ([209.85.128.193]:36662 "EHLO mail-wr0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752827AbdLNOw7 (ORCPT ); Thu, 14 Dec 2017 09:52:59 -0500 X-Google-Smtp-Source: ACJfBosMSo80+2BqMrnTtBxJtkHzCp01x2nTE8pKyKy6mKlfjKnsgKjNfaOBh0XZg/U6oQhVXYN8VQ== Message-ID: <1513263176.32163.12.camel@baylibre.com> Subject: Re: [PATCH v2] ARM64: dts: meson-axg: add ethernet mac controller From: Jerome Brunet To: Yixun Lan , devicetree@vger.kernel.org, Kevin Hilman Cc: Neil Armstrong , Giuseppe Cavallaro , Alexandre Torgue , Carlo Caione , linux-amlogic@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, netdev@vger.kernel.org Date: Thu, 14 Dec 2017 15:52:56 +0100 In-Reply-To: <20171214030242.113152-1-yixun.lan@amlogic.com> References: <20171214030242.113152-1-yixun.lan@amlogic.com> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.26.2 (3.26.2-1.fc27) Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 2017-12-14 at 11:02 +0800, Yixun Lan wrote: > Add DT info for the stmmac ethernet MAC which found in > the Amlogic's Meson-AXG SoC, also describe the ethernet > pinctrl & clock information here. > > This is tested in the S400 dev board which use a RTL8211F PHY, > and the pins connect to the 'eth_rgmii_y_pins' group. > > Reviewed-by: Neil Armstrong > Signed-off-by: Yixun Lan I think it would have been better to split this into 2 patches. One adding the controller for axg, the other using it in the s400, but maybe Kevin is OK with it... > > --- > Changes in v2 since [1]: > - rebase to kevin's v4.16/dt64 branch > - add Neil's Reviewed-by > - move clock info to board.dts instead of in soc.dtsi > - drop "meson-axg-dwmac" compatible string, since we didn't use this > we could re-add it later when we really need. > - note: to make ethernet work properly,it depend on clock & pinctrl[2], > to compile the DTS, the patch [3] is required. > the code part will be taken via clock & pinctrl subsystem tree. > > [1] > http://lists.infradead.org/pipermail/linux-amlogic/2017-November/005301.html > > [2] > http://lists.infradead.org/pipermail/linux-amlogic/2017-December/005735.html > http://lists.infradead.org/pipermail/linux-amlogic/2017-December/005694.html > > [3] > http://lists.infradead.org/pipermail/linux-amlogic/2017-December/005738.html > --- > arch/arm64/boot/dts/amlogic/meson-axg-s400.dts | 11 ++++++ > arch/arm64/boot/dts/amlogic/meson-axg.dtsi | 50 ++++++++++++++++++++++++++ > 2 files changed, 61 insertions(+) > > diff --git a/arch/arm64/boot/dts/amlogic/meson-axg-s400.dts b/arch/arm64/boot/dts/amlogic/meson-axg-s400.dts > index 70eca1f8736a..138de3bc7cc8 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-axg-s400.dts > +++ b/arch/arm64/boot/dts/amlogic/meson-axg-s400.dts > @@ -20,3 +20,14 @@ > &uart_AO { > status = "okay"; > }; > + > +ðmac { We try to keep nodes alphabetically ordered. Please put ethmac before uart_A0 thx > > With all the dependencies sorted out, it works Tested-by: Jerome Brunet