From: Srinath Mannam <srinath.mannam@broadcom.com>
To: Bjorn Helgaas <helgaas@kernel.org>
Cc: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>,
Ray Jui <rjui@broadcom.com>,
Scott Branden <sbranden@broadcom.com>,
BCM Kernel Feedback <bcm-kernel-feedback-list@broadcom.com>,
linux-pci@vger.kernel.org, iommu@lists.linux-foundation.org,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 0/3] Add IPROC PCIe new features
Date: Thu, 24 Jan 2019 11:06:44 +0530 [thread overview]
Message-ID: <CABe79T6=2uf1SrpDEm9s3smDkcR0JB-Gq8VwDF4yL8zBL6w0zw@mail.gmail.com> (raw)
In-Reply-To: <20190118144149.GA25249@google.com>
Hi Bjorn,
Thank you for the review. I will address your comments in the next patchset.
Regards,
Srinath.
On Fri, Jan 18, 2019 at 8:11 PM Bjorn Helgaas <helgaas@kernel.org> wrote:
>
> On Fri, Jan 18, 2019 at 09:53:20AM +0530, Srinath Mannam wrote:
> > Add changes related to IPROC PCIe RC IP new features.
> >
> > This patch set is based on Linux-5.0-rc2.
> >
> > Srinath Mannam (3):
> > PCI: iproc: Add feature to set order mode
>
> Since this apparently refers to a PCIe feature, the subject should use the
> exact terminology used in the PCIe spec.
>
> > PCI: iproc: CRS state check in config request
>
> This subject should start with a verb, not "CRS".
>
> > PCI: iproc: Add PCIe 32bit outbound memory configuration
> >
> > drivers/pci/controller/pcie-iproc.c | 172 +++++++++++++++++++++++++++++++++++-
> > drivers/pci/controller/pcie-iproc.h | 16 ++++
> > 2 files changed, 184 insertions(+), 4 deletions(-)
> >
> > --
> > 2.7.4
> >
WARNING: multiple messages have this Message-ID (diff)
From: Srinath Mannam via iommu <iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org>
To: Bjorn Helgaas <helgaas-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
Cc: Scott Branden <sbranden-dY08KVG/lbpWk0Htik3J/w@public.gmane.org>,
Ray Jui <rjui-dY08KVG/lbpWk0Htik3J/w@public.gmane.org>,
Linux Kernel Mailing List
<linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org,
BCM Kernel Feedback
<bcm-kernel-feedback-list-dY08KVG/lbpWk0Htik3J/w@public.gmane.org>,
linux-pci-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH 0/3] Add IPROC PCIe new features
Date: Thu, 24 Jan 2019 11:06:44 +0530 [thread overview]
Message-ID: <CABe79T6=2uf1SrpDEm9s3smDkcR0JB-Gq8VwDF4yL8zBL6w0zw@mail.gmail.com> (raw)
In-Reply-To: <20190118144149.GA25249-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org>
Hi Bjorn,
Thank you for the review. I will address your comments in the next patchset.
Regards,
Srinath.
On Fri, Jan 18, 2019 at 8:11 PM Bjorn Helgaas <helgaas-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> wrote:
>
> On Fri, Jan 18, 2019 at 09:53:20AM +0530, Srinath Mannam wrote:
> > Add changes related to IPROC PCIe RC IP new features.
> >
> > This patch set is based on Linux-5.0-rc2.
> >
> > Srinath Mannam (3):
> > PCI: iproc: Add feature to set order mode
>
> Since this apparently refers to a PCIe feature, the subject should use the
> exact terminology used in the PCIe spec.
>
> > PCI: iproc: CRS state check in config request
>
> This subject should start with a verb, not "CRS".
>
> > PCI: iproc: Add PCIe 32bit outbound memory configuration
> >
> > drivers/pci/controller/pcie-iproc.c | 172 +++++++++++++++++++++++++++++++++++-
> > drivers/pci/controller/pcie-iproc.h | 16 ++++
> > 2 files changed, 184 insertions(+), 4 deletions(-)
> >
> > --
> > 2.7.4
> >
next prev parent reply other threads:[~2019-01-24 5:37 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-01-18 4:23 [PATCH 0/3] Add IPROC PCIe new features Srinath Mannam
2019-01-18 4:23 ` [PATCH 1/3] PCI: iproc: Add feature to set order mode Srinath Mannam
2019-01-18 15:07 ` Bjorn Helgaas
2019-01-18 15:07 ` Bjorn Helgaas
2019-01-24 8:40 ` Srinath Mannam
2019-01-24 8:40 ` Srinath Mannam via iommu
2019-01-24 19:31 ` Bjorn Helgaas
2019-01-24 19:31 ` Bjorn Helgaas
2019-01-25 9:43 ` Srinath Mannam
2019-01-25 15:05 ` Bjorn Helgaas
2019-01-18 4:23 ` [PATCH 2/3] PCI: iproc: CRS state check in config request Srinath Mannam
2019-01-18 15:08 ` Bjorn Helgaas
2019-01-18 15:08 ` Bjorn Helgaas
2019-01-24 8:43 ` Srinath Mannam
2019-01-24 8:43 ` Srinath Mannam via iommu
2019-01-18 4:23 ` [PATCH 3/3] PCI: iproc: Add PCIe 32bit outbound memory configuration Srinath Mannam
2019-01-18 14:41 ` [PATCH 0/3] Add IPROC PCIe new features Bjorn Helgaas
2019-01-18 14:41 ` Bjorn Helgaas
2019-01-24 5:36 ` Srinath Mannam [this message]
2019-01-24 5:36 ` Srinath Mannam via iommu
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CABe79T6=2uf1SrpDEm9s3smDkcR0JB-Gq8VwDF4yL8zBL6w0zw@mail.gmail.com' \
--to=srinath.mannam@broadcom.com \
--cc=bcm-kernel-feedback-list@broadcom.com \
--cc=helgaas@kernel.org \
--cc=iommu@lists.linux-foundation.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=lorenzo.pieralisi@arm.com \
--cc=rjui@broadcom.com \
--cc=sbranden@broadcom.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.