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@xxxxxxx] > Sent: 2017年7月5日 14:59 > To: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; > linux-kernel@xxxxxxxxxxxxxxx; tglx@xxxxxxxxxxxxx; jason@xxxxxxxxxxxxxx; > marc.zyngier@xxxxxxx; robh+dt@xxxxxxxxxx; mark.rutland@xxxxxxx; > linux@xxxxxxxxxxxxxxx; shawnguo@xxxxxxxxxx; M.h. Lian > <minghuan.lian@xxxxxxx> > Cc: Z.q. Hou <zhiqiang.hou@xxxxxxx> > Subject: [PATCHv4 1/9] irqchip/ls-scfg-msi: fix typo of MSI compatible strings > > From: Minghuan Lian <Minghuan.Lian@xxxxxxx> > > 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@xxxxxxx> > Acked-by: Rob Herring <robh@xxxxxxxxxx> > Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@xxxxxxx> > --- > 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 ?韬{.n?????%??檩??w?{.n????z谵{???塄}?财??j:+v?? ? ???2??璀??摺?囤??z夸z罐?+?????w棹f
