Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933078AbdCKAYd (ORCPT ); Fri, 10 Mar 2017 19:24:33 -0500 Received: from mail-pf0-f173.google.com ([209.85.192.173]:36123 "EHLO mail-pf0-f173.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755129AbdCKAYY (ORCPT ); Fri, 10 Mar 2017 19:24:24 -0500 From: Kevin Hilman To: Neil Armstrong Cc: carlo@caione.org, linux-amlogic@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org Subject: Re: [PATCH 3/3] ARM64: dts: meson-gx: Prepend GX generic compatible like other nodes Organization: BayLibre References: <1489055948-10656-1-git-send-email-narmstrong@baylibre.com> <1489055948-10656-4-git-send-email-narmstrong@baylibre.com> Date: Fri, 10 Mar 2017 16:24:21 -0800 In-Reply-To: <1489055948-10656-4-git-send-email-narmstrong@baylibre.com> (Neil Armstrong's message of "Thu, 9 Mar 2017 11:39:08 +0100") Message-ID: User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.1 (darwin) MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2822 Lines: 86 Neil Armstrong writes: > Signed-off-by: Neil Armstrong Could use a bit more changelog here (in order to support generic drivers that work for all GX family, blah, blah...) Kevin > --- > arch/arm64/boot/dts/amlogic/meson-gx.dtsi | 16 ++++++++-------- > 1 file changed, 8 insertions(+), 8 deletions(-) > > diff --git a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi > index 06d70bf..a010ab9 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi > +++ b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi > @@ -233,7 +233,7 @@ > }; > > i2c_A: i2c@8500 { > - compatible = "amlogic,meson-gxbb-i2c"; > + compatible = "amlogic,meson-gx-i2c", "amlogic,meson-gxbb-i2c"; > reg = <0x0 0x08500 0x0 0x20>; > interrupts = ; > #address-cells = <1>; > @@ -279,7 +279,7 @@ > }; > > i2c_B: i2c@87c0 { > - compatible = "amlogic,meson-gxbb-i2c"; > + compatible = "amlogic,meson-gx-i2c", "amlogic,meson-gxbb-i2c"; > reg = <0x0 0x087c0 0x0 0x20>; > interrupts = ; > #address-cells = <1>; > @@ -288,7 +288,7 @@ > }; > > i2c_C: i2c@87e0 { > - compatible = "amlogic,meson-gxbb-i2c"; > + compatible = "amlogic,meson-gx-i2c", "amlogic,meson-gxbb-i2c"; > reg = <0x0 0x087e0 0x0 0x20>; > interrupts = ; > #address-cells = <1>; > @@ -297,7 +297,7 @@ > }; > > spifc: spi@8c80 { > - compatible = "amlogic,meson-gxbb-spifc"; > + compatible = "amlogic,meson-gx-spifc", "amlogic,meson-gxbb-spifc"; > reg = <0x0 0x08c80 0x0 0x80>; > #address-cells = <1>; > #size-cells = <0>; > @@ -325,7 +325,7 @@ > }; > > sram: sram@c8000000 { > - compatible = "amlogic,meson-gxbb-sram", "mmio-sram"; > + compatible = "amlogic,meson-gx-sram", "amlogic,meson-gxbb-sram", "mmio-sram"; > reg = <0x0 0xc8000000 0x0 0x14000>; > > #address-cells = <1>; > @@ -333,12 +333,12 @@ > ranges = <0 0x0 0xc8000000 0x14000>; > > cpu_scp_lpri: scp-shmem@0 { > - compatible = "amlogic,meson-gxbb-scp-shmem"; > + compatible = "amlogic,meson-gx-scp-shmem", "amlogic,meson-gxbb-scp-shmem"; > reg = <0x13000 0x400>; > }; > > cpu_scp_hpri: scp-shmem@200 { > - compatible = "amlogic,meson-gxbb-scp-shmem"; > + compatible = "amlogic,meson-gx-scp-shmem", "amlogic,meson-gxbb-scp-shmem"; > reg = <0x13400 0x400>; > }; > }; > @@ -390,7 +390,7 @@ > }; > > ir: ir@580 { > - compatible = "amlogic,meson-gxbb-ir"; > + compatible = "amlogic,meson-gx-ir", "amlogic,meson-gxbb-ir"; > reg = <0x0 0x00580 0x0 0x40>; > interrupts = ; > status = "disabled";