From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751200AbdE2PLW (ORCPT ); Mon, 29 May 2017 11:11:22 -0400 Received: from mail-lf0-f67.google.com ([209.85.215.67]:32998 "EHLO mail-lf0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751138AbdE2PLU (ORCPT ); Mon, 29 May 2017 11:11:20 -0400 Subject: Re: [PATCH v2 1/4] dt-bindings: mediatek: add bindings for MediaTek MT7622 SoC To: sean.wang@mediatek.com, robh+dt@kernel.org, mark.rutland@arm.com, devicetree@vger.kernel.org, linux-mediatek@lists.infradead.org Cc: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org References: <9b4e201f8f9ad11be98f22ccc3022b4e5d76a681.1496061587.git.sean.wang@mediatek.com> From: Matthias Brugger Message-ID: Date: Mon, 29 May 2017 17:11:16 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.1 MIME-Version: 1.0 In-Reply-To: <9b4e201f8f9ad11be98f22ccc3022b4e5d76a681.1496061587.git.sean.wang@mediatek.com> Content-Type: text/plain; charset=utf-8; format=flowed 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 On 29/05/17 14:56, sean.wang@mediatek.com wrote: > From: Sean Wang > > This adds dt-binding documentation for MediaTek MT7622 SoC > which currently only includes basic items such as ARM CPU, > MediaTek SYSIRQ and UART. > > Signed-off-by: Sean Wang > --- > Documentation/devicetree/bindings/arm/mediatek.txt | 4 ++++ > .../devicetree/bindings/interrupt-controller/mediatek,sysirq.txt | 3 ++- > Documentation/devicetree/bindings/serial/mtk-uart.txt | 1 + > 3 files changed, 7 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/arm/mediatek.txt b/Documentation/devicetree/bindings/arm/mediatek.txt > index c860b24..fc68570 100644 > --- a/Documentation/devicetree/bindings/arm/mediatek.txt > +++ b/Documentation/devicetree/bindings/arm/mediatek.txt > @@ -12,6 +12,7 @@ compatible: Must contain one of > "mediatek,mt6592" > "mediatek,mt6755" > "mediatek,mt6795" > + "mediatek,mt7622" > "mediatek,mt7623" > "mediatek,mt8127" > "mediatek,mt8135" > @@ -38,6 +39,9 @@ Supported boards: > - Evaluation board for MT6795(Helio X10): > Required root node properties: > - compatible = "mediatek,mt6795-evb", "mediatek,mt6795"; > +- Reference board variant 1 for MT7622: > + Required root node properties: > + - compatible = "mediatek,mt7622-rfb1", "mediatek,mt7622"; > - Evaluation board for MT7623: > Required root node properties: > - compatible = "mediatek,mt7623-evb", "mediatek,mt7623"; > diff --git a/Documentation/devicetree/bindings/interrupt-controller/mediatek,sysirq.txt b/Documentation/devicetree/bindings/interrupt-controller/mediatek,sysirq.txt > index a89c03b..eda1183 100644 > --- a/Documentation/devicetree/bindings/interrupt-controller/mediatek,sysirq.txt > +++ b/Documentation/devicetree/bindings/interrupt-controller/mediatek,sysirq.txt > @@ -1,4 +1,4 @@ > -+Mediatek 65xx/67xx/81xx sysirq > ++Mediatek MT65xx/MT67xx/MT762x/MT81xx sysirq > > Mediatek SOCs sysirq support controllable irq inverter for each GIC SPI > interrupt. > @@ -15,6 +15,7 @@ Required properties: > "mediatek,mt6582-sysirq" > "mediatek,mt6580-sysirq" > "mediatek,mt6577-sysirq" > + "mediatek,mt7622-sysirq" > "mediatek,mt2701-sysirq" This file is a mess and I posted some cleanups on the mailinglist. No need for you to resubmit. If the DT maintainers are fine with my cleanups, I cann change the line accordingly. Regards, Matthias > - interrupt-controller : Identifies the node as an interrupt controller > - #interrupt-cells : Use the same format as specified by GIC in arm,gic.txt. > diff --git a/Documentation/devicetree/bindings/serial/mtk-uart.txt b/Documentation/devicetree/bindings/serial/mtk-uart.txt > index 0015c72..daa1eef 100644 > --- a/Documentation/devicetree/bindings/serial/mtk-uart.txt > +++ b/Documentation/devicetree/bindings/serial/mtk-uart.txt > @@ -8,6 +8,7 @@ Required properties: > * "mediatek,mt6589-uart" for MT6589 compatible UARTS > * "mediatek,mt6755-uart" for MT6755 compatible UARTS > * "mediatek,mt6795-uart" for MT6795 compatible UARTS > + * "mediatek,mt7622-uart" for MT7622 compatible UARTS > * "mediatek,mt7623-uart" for MT7623 compatible UARTS > * "mediatek,mt8127-uart" for MT8127 compatible UARTS > * "mediatek,mt8135-uart" for MT8135 compatible UARTS >