* [PATCH -next] pci/controller/dwc: convert comma to semicolon
@ 2020-12-16 13:19 Zheng Yongjun
2021-01-06 19:07 ` Bjorn Helgaas
0 siblings, 1 reply; 8+ messages in thread
From: Zheng Yongjun @ 2020-12-16 13:19 UTC (permalink / raw)
To: minghuan.Lian, mingkai.hu, roy.zang, robh, linuxppc-dev,
linux-pci, linux-arm-kernel, linux-kernel
Cc: Zheng Yongjun
Replace a comma between expression statements by a semicolon.
Signed-off-by: Zheng Yongjun <zhengyongjun3@huawei.com>
---
drivers/pci/controller/dwc/pci-layerscape-ep.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pci/controller/dwc/pci-layerscape-ep.c b/drivers/pci/controller/dwc/pci-layerscape-ep.c
index 84206f265e54..917ba8d254fc 100644
--- a/drivers/pci/controller/dwc/pci-layerscape-ep.c
+++ b/drivers/pci/controller/dwc/pci-layerscape-ep.c
@@ -178,7 +178,7 @@ static int __init ls_pcie_ep_probe(struct platform_device *pdev)
pci->dev = dev;
pci->ops = pcie->drvdata->dw_pcie_ops;
- ls_epc->bar_fixed_64bit = (1 << BAR_2) | (1 << BAR_4),
+ ls_epc->bar_fixed_64bit = (1 << BAR_2) | (1 << BAR_4);
pcie->pci = pci;
pcie->ls_epc = ls_epc;
--
2.22.0
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH -next] pci/controller/dwc: convert comma to semicolon
2020-12-16 13:19 [PATCH -next] pci/controller/dwc: convert comma to semicolon Zheng Yongjun
@ 2021-01-06 19:07 ` Bjorn Helgaas
2021-01-15 11:36 ` Lorenzo Pieralisi
0 siblings, 1 reply; 8+ messages in thread
From: Bjorn Helgaas @ 2021-01-06 19:07 UTC (permalink / raw)
To: Zheng Yongjun
Cc: minghuan.Lian, mingkai.hu, roy.zang, robh, linuxppc-dev,
linux-pci, linux-arm-kernel, linux-kernel
On Wed, Dec 16, 2020 at 09:19:44PM +0800, Zheng Yongjun wrote:
> Replace a comma between expression statements by a semicolon.
Looks like a good fix, but read this about the changelog title:
https://lore.kernel.org/r/20171026223701.GA25649@bhelgaas-glaptop.roam.corp.google.com
> Signed-off-by: Zheng Yongjun <zhengyongjun3@huawei.com>
> ---
> drivers/pci/controller/dwc/pci-layerscape-ep.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/pci/controller/dwc/pci-layerscape-ep.c b/drivers/pci/controller/dwc/pci-layerscape-ep.c
> index 84206f265e54..917ba8d254fc 100644
> --- a/drivers/pci/controller/dwc/pci-layerscape-ep.c
> +++ b/drivers/pci/controller/dwc/pci-layerscape-ep.c
> @@ -178,7 +178,7 @@ static int __init ls_pcie_ep_probe(struct platform_device *pdev)
> pci->dev = dev;
> pci->ops = pcie->drvdata->dw_pcie_ops;
>
> - ls_epc->bar_fixed_64bit = (1 << BAR_2) | (1 << BAR_4),
> + ls_epc->bar_fixed_64bit = (1 << BAR_2) | (1 << BAR_4);
>
> pcie->pci = pci;
> pcie->ls_epc = ls_epc;
> --
> 2.22.0
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH -next] pci/controller/dwc: convert comma to semicolon
2021-01-06 19:07 ` Bjorn Helgaas
@ 2021-01-15 11:36 ` Lorenzo Pieralisi
2021-03-07 18:36 ` Krzysztof Wilczyński
0 siblings, 1 reply; 8+ messages in thread
From: Lorenzo Pieralisi @ 2021-01-15 11:36 UTC (permalink / raw)
To: Bjorn Helgaas
Cc: Zheng Yongjun, minghuan.Lian, mingkai.hu, roy.zang, robh,
linuxppc-dev, linux-pci, linux-arm-kernel, linux-kernel
On Wed, Jan 06, 2021 at 01:07:22PM -0600, Bjorn Helgaas wrote:
> On Wed, Dec 16, 2020 at 09:19:44PM +0800, Zheng Yongjun wrote:
> > Replace a comma between expression statements by a semicolon.
>
> Looks like a good fix, but read this about the changelog title:
>
> https://lore.kernel.org/r/20171026223701.GA25649@bhelgaas-glaptop.roam.corp.google.com
I would request NXP maintainers to take this patch, rewrite it as
Bjorn requested and resend it as fast as possible, this is a very
relevant fix.
Thanks,
Lorenzo
> > Signed-off-by: Zheng Yongjun <zhengyongjun3@huawei.com>
> > ---
> > drivers/pci/controller/dwc/pci-layerscape-ep.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/pci/controller/dwc/pci-layerscape-ep.c b/drivers/pci/controller/dwc/pci-layerscape-ep.c
> > index 84206f265e54..917ba8d254fc 100644
> > --- a/drivers/pci/controller/dwc/pci-layerscape-ep.c
> > +++ b/drivers/pci/controller/dwc/pci-layerscape-ep.c
> > @@ -178,7 +178,7 @@ static int __init ls_pcie_ep_probe(struct platform_device *pdev)
> > pci->dev = dev;
> > pci->ops = pcie->drvdata->dw_pcie_ops;
> >
> > - ls_epc->bar_fixed_64bit = (1 << BAR_2) | (1 << BAR_4),
> > + ls_epc->bar_fixed_64bit = (1 << BAR_2) | (1 << BAR_4);
> >
> > pcie->pci = pci;
> > pcie->ls_epc = ls_epc;
> > --
> > 2.22.0
> >
> >
> > _______________________________________________
> > linux-arm-kernel mailing list
> > linux-arm-kernel@lists.infradead.org
> > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH -next] pci/controller/dwc: convert comma to semicolon
2021-01-15 11:36 ` Lorenzo Pieralisi
@ 2021-03-07 18:36 ` Krzysztof Wilczyński
2021-03-22 12:43 ` Lorenzo Pieralisi
0 siblings, 1 reply; 8+ messages in thread
From: Krzysztof Wilczyński @ 2021-03-07 18:36 UTC (permalink / raw)
To: Lorenzo Pieralisi
Cc: Bjorn Helgaas, Zheng Yongjun, minghuan.Lian, mingkai.hu,
roy.zang, robh, linuxppc-dev, linux-pci, linux-arm-kernel,
linux-kernel
Hi,
[...]
> I would request NXP maintainers to take this patch, rewrite it as
> Bjorn requested and resend it as fast as possible, this is a very
> relevant fix.
[...]
Looking at the state of the pci-layerscape-ep.c file in Linus' tree,
this still hasn't been fixed, and it has been a while.
NXP folks, are you intend to pick this up? Do let us know.
Krzysztof
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH -next] pci/controller/dwc: convert comma to semicolon
2021-03-07 18:36 ` Krzysztof Wilczyński
@ 2021-03-22 12:43 ` Lorenzo Pieralisi
2021-03-22 13:40 ` Roy Zang
0 siblings, 1 reply; 8+ messages in thread
From: Lorenzo Pieralisi @ 2021-03-22 12:43 UTC (permalink / raw)
To: Krzysztof Wilczyński, minghuan.Lian, Mingkai Hu, Roy Zang
Cc: Bjorn Helgaas, Zheng Yongjun, robh, linuxppc-dev, linux-pci,
linux-arm-kernel, linux-kernel
On Sun, Mar 07, 2021 at 07:36:57PM +0100, Krzysztof Wilczyński wrote:
> Hi,
>
> [...]
> > I would request NXP maintainers to take this patch, rewrite it as
> > Bjorn requested and resend it as fast as possible, this is a very
> > relevant fix.
> [...]
>
> Looking at the state of the pci-layerscape-ep.c file in Linus' tree,
> this still hasn't been fixed, and it has been a while.
>
> NXP folks, are you intend to pick this up? Do let us know.
Minghuan, Mingkai, Roy,
either one of you reply and follow up this patch or I will have to
update the MAINTAINERS entry and take action accordingly, you are
not maintaining this driver and I won't maintain your code, sorry.
Lorenzo
> Krzysztof
^ permalink raw reply [flat|nested] 8+ messages in thread
* RE: [PATCH -next] pci/controller/dwc: convert comma to semicolon
2021-03-22 12:43 ` Lorenzo Pieralisi
@ 2021-03-22 13:40 ` Roy Zang
2021-03-22 14:44 ` Lorenzo Pieralisi
0 siblings, 1 reply; 8+ messages in thread
From: Roy Zang @ 2021-03-22 13:40 UTC (permalink / raw)
To: Lorenzo Pieralisi, Krzysztof Wilczyński, M.h. Lian, Mingkai Hu
Cc: Bjorn Helgaas, Zheng Yongjun, robh, linuxppc-dev, linux-pci,
linux-arm-kernel, linux-kernel
Yes. It is maintained.
I will send out a patch.
Thanks.
Roy
-----Original Message-----
From: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
On Sun, Mar 07, 2021 at 07:36:57PM +0100, Krzysztof Wilczyński wrote:
> Hi,
>
> [...]
> > I would request NXP maintainers to take this patch, rewrite it as
> > Bjorn requested and resend it as fast as possible, this is a very
> > relevant fix.
> [...]
>
> Looking at the state of the pci-layerscape-ep.c file in Linus' tree,
> this still hasn't been fixed, and it has been a while.
>
> NXP folks, are you intend to pick this up? Do let us know.
Minghuan, Mingkai, Roy,
either one of you reply and follow up this patch or I will have to update the MAINTAINERS entry and take action accordingly, you are not maintaining this driver and I won't maintain your code, sorry.
Lorenzo
> Krzysztof
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH -next] pci/controller/dwc: convert comma to semicolon
2021-03-22 13:40 ` Roy Zang
@ 2021-03-22 14:44 ` Lorenzo Pieralisi
2021-03-22 15:25 ` Roy Zang
0 siblings, 1 reply; 8+ messages in thread
From: Lorenzo Pieralisi @ 2021-03-22 14:44 UTC (permalink / raw)
To: Roy Zang
Cc: Krzysztof Wilczyński, M.h. Lian, Mingkai Hu, Bjorn Helgaas,
Zheng Yongjun, robh, linuxppc-dev, linux-pci, linux-arm-kernel,
linux-kernel
On Mon, Mar 22, 2021 at 01:40:16PM +0000, Roy Zang wrote:
> Yes. It is maintained.
To be maintained you should review its code please.
> I will send out a patch.
Krzysztof already posted one for you, you just need to ack it:
https://patchwork.kernel.org/project/linux-pci/patch/20210311033745.1547044-1-kw@linux.com
For the future email exchanges: don't top-post please.
Thanks,
Lorenzo
> Thanks.
> Roy
>
> -----Original Message-----
> From: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
>
> On Sun, Mar 07, 2021 at 07:36:57PM +0100, Krzysztof Wilczyński wrote:
> > Hi,
> >
> > [...]
> > > I would request NXP maintainers to take this patch, rewrite it as
> > > Bjorn requested and resend it as fast as possible, this is a very
> > > relevant fix.
> > [...]
> >
> > Looking at the state of the pci-layerscape-ep.c file in Linus' tree,
> > this still hasn't been fixed, and it has been a while.
> >
> > NXP folks, are you intend to pick this up? Do let us know.
>
> Minghuan, Mingkai, Roy,
>
> either one of you reply and follow up this patch or I will have to update the MAINTAINERS entry and take action accordingly, you are not maintaining this driver and I won't maintain your code, sorry.
>
> Lorenzo
>
> > Krzysztof
^ permalink raw reply [flat|nested] 8+ messages in thread
* RE: [PATCH -next] pci/controller/dwc: convert comma to semicolon
2021-03-22 14:44 ` Lorenzo Pieralisi
@ 2021-03-22 15:25 ` Roy Zang
0 siblings, 0 replies; 8+ messages in thread
From: Roy Zang @ 2021-03-22 15:25 UTC (permalink / raw)
To: Lorenzo Pieralisi
Cc: Krzysztof Wilczyński, M.h. Lian, Mingkai Hu, Bjorn Helgaas,
Zheng Yongjun, robh, linuxppc-dev, linux-pci, linux-arm-kernel,
linux-kernel
> From: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
> On Mon, Mar 22, 2021 at 01:40:16PM +0000, Roy Zang wrote:
> > Yes. It is maintained.
>
> To be maintained you should review its code please.
Sure.
>
> > I will send out a patch.
>
> Krzysztof already posted one for you, you just need to ack it:
>
> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatchw
> ork.kernel.org%2Fproject%2Flinux-
> pci%2Fpatch%2F20210311033745.1547044-1-
> kw%40linux.com&data=04%7C01%7Croy.zang%40nxp.com%7Ced683ff5
> 093443cb9c1608d8ed41150e%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0
> %7C1%7C637520211070640953%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiM
> C4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C2000
> &sdata=IWTcj7Xs0AdVVIf%2BSFzge6is9D8o0h6SSi7MpibIYcY%3D&r
> eserved=0
>
> For the future email exchanges: don't top-post please.
Acked.
Roy
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2021-03-22 15:26 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-12-16 13:19 [PATCH -next] pci/controller/dwc: convert comma to semicolon Zheng Yongjun
2021-01-06 19:07 ` Bjorn Helgaas
2021-01-15 11:36 ` Lorenzo Pieralisi
2021-03-07 18:36 ` Krzysztof Wilczyński
2021-03-22 12:43 ` Lorenzo Pieralisi
2021-03-22 13:40 ` Roy Zang
2021-03-22 14:44 ` Lorenzo Pieralisi
2021-03-22 15:25 ` Roy Zang
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).