From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753503AbdLNQpw (ORCPT ); Thu, 14 Dec 2017 11:45:52 -0500 Received: from mail-wr0-f196.google.com ([209.85.128.196]:44615 "EHLO mail-wr0-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753416AbdLNQpu (ORCPT ); Thu, 14 Dec 2017 11:45:50 -0500 X-Google-Smtp-Source: ACJfBov4aMElbWs6/jJzvC5DhAbWUh3KU1+j2xOxZRB08yylURaiH75ZgAIFk4BS9DBmOHRNCXMiww== Message-ID: <1513269947.2261.9.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 17:45:47 +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: > --- > 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 You got this comment regarding the pwm clock setup. the setup of the pwm clocks depends on the use case, so should defined depending on the requirement on the board This is not the case for the ethmac, the clock setup will be same for every board, unless I missed something. the clock bindings should be defined in meson-axg.dtsi, I think > - 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.