linux-pci.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [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	[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&amp;data=04%7C01%7Croy.zang%40nxp.com%7Ced683ff5
> 093443cb9c1608d8ed41150e%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0
> %7C1%7C637520211070640953%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiM
> C4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C2000
> &amp;sdata=IWTcj7Xs0AdVVIf%2BSFzge6is9D8o0h6SSi7MpibIYcY%3D&amp;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).