lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <118caa5c-6782-f4ca-443e-19d483171e01@baylibre.com>
Date:   Fri, 1 Dec 2017 10:19:06 +0100
From:   Neil Armstrong <narmstrong@...libre.com>
To:     Philippe Ombredanne <pombredanne@...b.com>
Cc:     Shawn Guo <shawnguo@...nel.org>,
        Sascha Hauer <kernel@...gutronix.de>,
        Fabio Estevam <fabio.estevam@....com>,
        "moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE" 
        <linux-arm-kernel@...ts.infradead.org>,
        LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v4 4/6] ARM: dts: imx6qdl: Add Variscite DART-MX6 SoM
 support

Hi Philippe,

On 01/12/2017 10:02, Philippe Ombredanne wrote:
> On Fri, Dec 1, 2017 at 9:54 AM, Neil Armstrong <narmstrong@...libre.com> 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 <fabio.estevam@....com>
>>>> Signed-off-by: Neil Armstrong <narmstrong@...libre.com>
>>>
>>> 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 <narmstrong@...libre.com>
>>>> + *
>>>> + * 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

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ