* [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).