From: "Z.q. Hou" <zhiqiang.hou@nxp.com>
To: "Z.q. Hou" <zhiqiang.hou@nxp.com>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "tglx@linutronix.de" <tglx@linutronix.de>,
        "jason@lakedaemon.net" <jason@lakedaemon.net>,
        "marc.zyngier@arm.com" <marc.zyngier@arm.com>,
        "robh+dt@kernel.org" <robh+dt@kernel.org>,
        "mark.rutland@arm.com" <mark.rutland@arm.com>,
        "linux@armlinux.org.uk" <linux@armlinux.org.uk>,
        "shawnguo@kernel.org" <shawnguo@kernel.org>,
        "M.h. Lian" <minghuan.lian@nxp.com>
Subject: RE: [PATCHv4 1/9] irqchip/ls-scfg-msi: fix typo of MSI compatible
 strings
Thread-Topic: [PATCHv4 1/9] irqchip/ls-scfg-msi: fix typo of MSI compatible
 strings
Thread-Index: AQHS9V54PpyE0QnAkUm5m1guXENQy6KbO0lQ
Date: Tue, 29 Aug 2017 07:04:05 +0000
Message-ID: <AM5PR0402MB277107C571183D2AF0E73EA5849F0@AM5PR0402MB2771.eurprd04.prod.outlook.com>
References: <1499237943-24203-1-git-send-email-Zhiqiang.Hou@nxp.com>
In-Reply-To: <1499237943-24203-1-git-send-email-Zhiqiang.Hou@nxp.com>
Accept-Language: zh-CN, en-US
Content-Language: en-US
spamdiagnosticoutput: 1:99
spamdiagnosticmetadata: NSPM
Content-Type: text/plain; charset="gb2312"
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-originalarrivaltime: 29 Aug 2017 07:04:05.4245
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM5PR0402MB2851
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Transfer-Encoding: 8bit
X-MIME-Autoconverted: from base64 to 8bit by nfs id v7T74GBA028215
Content-Length: 3090
Lines: 82

Hi Thomas, Jason and Marc,

The version 4 patch set didn't get feedback for a long time, can you help to review it, and if there is no problem could you please help to apply it?

> -----Original Message-----
> From: Zhiqiang Hou [mailto:Zhiqiang.Hou@nxp.com]
> Sent: 2017??7??5?? 14:59
> To: linux-arm-kernel@lists.infradead.org; devicetree@vger.kernel.org;
> linux-kernel@vger.kernel.org; tglx@linutronix.de; jason@lakedaemon.net;
> marc.zyngier@arm.com; robh+dt@kernel.org; mark.rutland@arm.com;
> linux@armlinux.org.uk; shawnguo@kernel.org; M.h. Lian
> <minghuan.lian@nxp.com>
> Cc: Z.q. Hou <zhiqiang.hou@nxp.com>
> Subject: [PATCHv4 1/9] irqchip/ls-scfg-msi: fix typo of MSI compatible strings
> 
> From: Minghuan Lian <Minghuan.Lian@nxp.com>
> 
> The patch is to fix typo of the Layerscape SCFG MSI dts compatible strings. "1"
> is replaced by "l".
> 
> Signed-off-by: Minghuan Lian <Minghuan.Lian@nxp.com>
> Acked-by: Rob Herring <robh@kernel.org>
> Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
> ---
> v4-v1:
>  - None
> 
>  .../devicetree/bindings/interrupt-controller/fsl,ls-scfg-msi.txt    | 6 +++---
>  drivers/irqchip/irq-ls-scfg-msi.c                                   | 6
> ++++--
>  2 files changed, 7 insertions(+), 5 deletions(-)
> 
> diff --git
> a/Documentation/devicetree/bindings/interrupt-controller/fsl,ls-scfg-msi.txt
> b/Documentation/devicetree/bindings/interrupt-controller/fsl,ls-scfg-msi.txt
> index 9e38949..2755cd1 100644
> ---
> a/Documentation/devicetree/bindings/interrupt-controller/fsl,ls-scfg-msi.txt
> +++ b/Documentation/devicetree/bindings/interrupt-controller/fsl,ls-scfg
> +++ -msi.txt
> @@ -4,8 +4,8 @@ Required properties:
> 
>  - compatible: should be "fsl,<soc-name>-msi" to identify
>  	      Layerscape PCIe MSI controller block such as:
> -              "fsl,1s1021a-msi"
> -              "fsl,1s1043a-msi"
> +              "fsl,ls1021a-msi"
> +              "fsl,ls1043a-msi"
>  - msi-controller: indicates that this is a PCIe MSI controller node
>  - reg: physical base address of the controller and length of memory mapped.
>  - interrupts: an interrupt to the parent interrupt controller.
> @@ -23,7 +23,7 @@ MSI controller node
>  Examples:
> 
>  	msi1: msi-controller@1571000 {
> -		compatible = "fsl,1s1043a-msi";
> +		compatible = "fsl,ls1043a-msi";
>  		reg = <0x0 0x1571000 0x0 0x8>,
>  		msi-controller;
>  		interrupts = <0 116 0x4>;
> diff --git a/drivers/irqchip/irq-ls-scfg-msi.c b/drivers/irqchip/irq-ls-scfg-msi.c
> index 02cca74c..cef67cc 100644
> --- a/drivers/irqchip/irq-ls-scfg-msi.c
> +++ b/drivers/irqchip/irq-ls-scfg-msi.c
> @@ -219,8 +219,10 @@ static int ls_scfg_msi_remove(struct platform_device
> *pdev)  }
> 
>  static const struct of_device_id ls_scfg_msi_id[] = {
> -	{ .compatible = "fsl,1s1021a-msi", },
> -	{ .compatible = "fsl,1s1043a-msi", },
> +	{ .compatible = "fsl,1s1021a-msi", }, /* a typo */
> +	{ .compatible = "fsl,1s1043a-msi", }, /* a typo */
> +	{ .compatible = "fsl,ls1021a-msi", },
> +	{ .compatible = "fsl,ls1043a-msi", },
>  	{},
>  };
> 
> --
> 2.1.0.27.g96db324

Thanks,
Zhiqiang