[EXT] Re: [PATCH v2 1/1] PCI: layerscape: Add the endpoint linkup notifier support

Bjorn Helgaas helgaas at kernel.org
Tue May 9 07:31:06 AEST 2023


On Mon, May 08, 2023 at 01:31:26PM +0000, Frank Li wrote:
> > -----Original Message-----
> > From: Manivannan Sadhasivam <manivannan.sadhasivam at linaro.org>
> > Sent: Saturday, May 6, 2023 2:59 AM
> > To: Frank Li <frank.li at nxp.com>
> > Cc: M.H. Lian <minghuan.lian at nxp.com>; Mingkai Hu
> > <mingkai.hu at nxp.com>; Roy Zang <roy.zang at nxp.com>; Lorenzo Pieralisi
> > <lpieralisi at kernel.org>; Rob Herring <robh at kernel.org>; Krzysztof
> > Wilczyński <kw at linux.com>; Bjorn Helgaas <bhelgaas at google.com>; open
> > list:PCI DRIVER FOR FREESCALE LAYERSCAPE <linuxppc-dev at lists.ozlabs.org>;
> > open list:PCI DRIVER FOR FREESCALE LAYERSCAPE <linux-
> > pci at vger.kernel.org>; moderated list:PCI DRIVER FOR FREESCALE
> > LAYERSCAPE <linux-arm-kernel at lists.infradead.org>; open list <linux-
> > kernel at vger.kernel.org>; imx at lists.linux.dev
> > Subject: [EXT] Re: [PATCH v2 1/1] PCI: layerscape: Add the endpoint linkup
> > notifier support

All these quoted headers are redundant clutter since we've already
seen them when Manivannan sent his comments.  It would be nice if your
mailer could be configured to omit them.

> > > +static int ls_pcie_ep_interrupt_init(struct ls_pcie_ep *pcie,
> > > +                                  struct platform_device *pdev)
> > > +{
> > > +     u32 val;
> > > +     int ret;
> > > +
> > > +     pcie->irq = platform_get_irq_byname(pdev, "pme");
> > > +     if (pcie->irq < 0) {
> > > +             dev_err(&pdev->dev, "Can't get 'pme' IRQ\n");
> > 
> > PME
> 
> Here should be dts property `pme`, suppose should match
> platform_get_irq_byname(pdev, "pme");

You can also edit out all the other context and questions if you're
not responding to them.

There were a lot of other comments that were useful but are not
relevant to this reply.

Bjorn


More information about the Linuxppc-dev mailing list