Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751421AbdLAJTN (ORCPT ); Fri, 1 Dec 2017 04:19:13 -0500 Received: from mail-wr0-f195.google.com ([209.85.128.195]:33290 "EHLO mail-wr0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750735AbdLAJTJ (ORCPT ); Fri, 1 Dec 2017 04:19:09 -0500 X-Google-Smtp-Source: AGs4zMYsH3c9u3FgQU+1TIMmmr6yOKm48AhxhbYSMOroXy+zYskF21JtLTsWbpPpaIzXGGi/6YbSFQ== Subject: Re: [PATCH v4 4/6] ARM: dts: imx6qdl: Add Variscite DART-MX6 SoM support To: Philippe Ombredanne Cc: Shawn Guo , Sascha Hauer , Fabio Estevam , "moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE" , LKML References: <1511950857-19292-1-git-send-email-narmstrong@baylibre.com> <1511950857-19292-5-git-send-email-narmstrong@baylibre.com> <20171130003912.GB22908@dragon> From: Neil Armstrong Organization: Baylibre Message-ID: <118caa5c-6782-f4ca-443e-19d483171e01@baylibre.com> Date: Fri, 1 Dec 2017 10:19:06 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2687 Lines: 78 Hi Philippe, On 01/12/2017 10:02, Philippe Ombredanne wrote: > On Fri, Dec 1, 2017 at 9:54 AM, Neil Armstrong wrote: >> On 30/11/2017 01:39, Shawn Guo wrote: >>> On Wed, Nov 29, 2017 at 11:20:55AM +0100, Neil Armstrong wrote: >>>> This patch adds support for the Variscite DART-MX6 SoM with : >>>> - i.MX6 Quad or Dual Lite SoC >>>> - 1Gb/2Gb LPDDR2 >>>> - 4-64 GB eMMC >>>> - Camera Interface >>>> - HDMI+CEC interface >>>> - LVDS / DSI / Parallel RGB interfaces >>>> - Ethernet RGMII interface >>>> - On-SoM Wi-Fi/Bluetooth with WiLink wl1835 SDIO Module >>>> - SD/MMC/SDIO interface >>>> - USB Host + USB OTG interface >>>> - I2C interfaces >>>> - SPI interfaces >>>> - PCI-Express 2.0 interface >>>> - on-SoM Audio Codec with HP/Line-In interfaces + DMIC interface >>>> - Digital Audio interface >>>> - S/PDIF interface >>>> >>>> Product website : http://www.variscite.com/products/system-on-module-som/cortex-a9/dart-mx6-cpu-freescale-imx6 >>>> >>>> Support is handled with a SoM-centric dtsi exporting the default interfaces >>>> along the default pinmuxing to be enabled by the board dts file. >>>> >>>> Only board-independent devices like WiFi, eMMC or PMIC are enabled in the dtsi. >>>> >>>> Reviewed-by: Fabio Estevam >>>> Signed-off-by: Neil Armstrong >>> >>> Looks pretty good. Only a couple of minor comments. >>> >>>> --- >>>> arch/arm/boot/dts/imx6qdl-var-dart.dtsi | 504 ++++++++++++++++++++++++++++++++ >>>> 1 file changed, 504 insertions(+) >>>> create mode 100644 arch/arm/boot/dts/imx6qdl-var-dart.dtsi >>>> >>>> diff --git a/arch/arm/boot/dts/imx6qdl-var-dart.dtsi b/arch/arm/boot/dts/imx6qdl-var-dart.dtsi >>>> new file mode 100644 >>>> index 0000000..fd2520b >>>> --- /dev/null >>>> +++ b/arch/arm/boot/dts/imx6qdl-var-dart.dtsi >>>> @@ -0,0 +1,504 @@ >>>> +/* >>>> + * Support for Variscite DART-MX6 Module >>>> + * >>>> + * Copyright 2017 BayLibre, SAS >>>> + * Author: Neil Armstrong >>>> + * >>>> + * SPDX-License-Identifier: (GPL-2.0+ OR MIT) >>>> + */ > > Should this be instead on the first line with a // comment style if > possible, right? I just saw this right now. Damn, I just sent a v5... > >>> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > > Thomas has sent a first doc patch [1] set and is working on an updated > version. Jonathan also wrote a nice background article on the topic > at LWN [2]. > > Linus explained why he wants the // comment style and a first line > location on the list too. > > [1] https://marc.info/?l=linux-kernel&m=151051532322831&w=2 > [2] https://lwn.net/SubscriberLink/739183/262749cbe307ddc7/ > Neil