| Message ID | 20230803075038.307012-1-david.marchand@redhat.com (mailing list archive) |
|---|---|
| Headers |
Return-Path: <dev-bounces@dpdk.org> X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 92E2C42FC1; Thu, 3 Aug 2023 09:50:50 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 6986C42D0C; Thu, 3 Aug 2023 09:50:50 +0200 (CEST) Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by mails.dpdk.org (Postfix) with ESMTP id EEC184161A for <dev@dpdk.org>; Thu, 3 Aug 2023 09:50:48 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1691049048; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=AVKCVd+tpOg6caCgY/21fd+w8fJD7seutDOMmk090Ao=; b=ETMwiCByPDYmH92g86/TSqgPRG2S3VPvDYqch1HoXDrFIuZ70H76sUvs4CeKqfKTGLswIC tJSgJ/Ll/MlCAcvXUuZDiY2t919UyC9kYh/lnPWIzfgfll5z7ksvXBEBbuO7Q52aNgQUd5 A0cdje0beVa06VqIC7LNfVK+eyO2qFo= Received: from mimecast-mx02.redhat.com (mimecast-mx02.redhat.com [66.187.233.88]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-524-TSIdnb68N8-0Q3TDhacp7w-1; Thu, 03 Aug 2023 03:50:44 -0400 X-MC-Unique: TSIdnb68N8-0Q3TDhacp7w-1 Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.rdu2.redhat.com [10.11.54.8]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id 8CB1B805AF7; Thu, 3 Aug 2023 07:50:43 +0000 (UTC) Received: from dmarchan.redhat.com (unknown [10.45.224.120]) by smtp.corp.redhat.com (Postfix) with ESMTP id 720BFC5796B; Thu, 3 Aug 2023 07:50:42 +0000 (UTC) From: David Marchand <david.marchand@redhat.com> To: dev@dpdk.org Cc: thomas@monjalon.net, ferruh.yigit@amd.com, chenbo.xia@intel.com, nipun.gupta@amd.com Subject: [PATCH 00/14] Cleanup PCI(e) drivers Date: Thu, 3 Aug 2023 09:50:23 +0200 Message-ID: <20230803075038.307012-1-david.marchand@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.1 on 10.11.54.8 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset="US-ASCII"; x-default=true X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions <dev.dpdk.org> List-Unsubscribe: <https://mails.dpdk.org/options/dev>, <mailto:dev-request@dpdk.org?subject=unsubscribe> List-Archive: <http://mails.dpdk.org/archives/dev/> List-Post: <mailto:dev@dpdk.org> List-Help: <mailto:dev-request@dpdk.org?subject=help> List-Subscribe: <https://mails.dpdk.org/listinfo/dev>, <mailto:dev-request@dpdk.org?subject=subscribe> Errors-To: dev-bounces@dpdk.org |
| Series |
Cleanup PCI(e) drivers
|
|
Message
David Marchand
Aug. 3, 2023, 7:50 a.m. UTC
Rather than rely on Linux headers to find some PCI(e) standard constants or reinvent the same PCI capability helper, this series complements the pci library and the pci bus driver. PCI drivers can then use OS agnostic macros and helpers. WARNING: this is only compile tested.
Comments
On Thu, Aug 03, 2023 at 09:50:23AM +0200, David Marchand wrote: > Rather than rely on Linux headers to find some PCI(e) standard constants > or reinvent the same PCI capability helper, this series complements the > pci library and the pci bus driver. > PCI drivers can then use OS agnostic macros and helpers. > > WARNING: this is only compile tested. > > -- > David Marchand > > David Marchand (14): > drivers: remove duplicated PCI master control > bus/pci: add const to some experimental API > bus/pci: find PCI capability > pci: define some capability constants > pci: define some MSIX constants > pci: define some command constants > pci: define some BAR constants > pci: define some PM constants > pci: define some PCIe constants > pci: define some extended capability constants > pci: define some ACS constants > pci: define some PRI constants > pci: define some AER constants > devtools: forbid inclusion of Linux header for PCI > For the rest of the patches I haven't already acked, assume no comments from me. Series-acked-by: Bruce Richardson <bruce.richardson@intel.com>
Recheck-request: iol-sample-apps-testing This is a test email of the retesting framework, please ignore. Testing the behavior of replying to a superseded patch. On Thu, Aug 3, 2023 at 3:50 AM David Marchand <david.marchand@redhat.com> wrote: > Rather than rely on Linux headers to find some PCI(e) standard constants > or reinvent the same PCI capability helper, this series complements the > pci library and the pci bus driver. > PCI drivers can then use OS agnostic macros and helpers. > > WARNING: this is only compile tested. > > -- > David Marchand > > David Marchand (14): > drivers: remove duplicated PCI master control > bus/pci: add const to some experimental API > bus/pci: find PCI capability > pci: define some capability constants > pci: define some MSIX constants > pci: define some command constants > pci: define some BAR constants > pci: define some PM constants > pci: define some PCIe constants > pci: define some extended capability constants > pci: define some ACS constants > pci: define some PRI constants > pci: define some AER constants > devtools: forbid inclusion of Linux header for PCI > > devtools/checkpatches.sh | 8 ++ > drivers/bus/pci/linux/pci_init.h | 18 ---- > drivers/bus/pci/linux/pci_uio.c | 32 +----- > drivers/bus/pci/linux/pci_vfio.c | 142 ++++++------------------- > drivers/bus/pci/pci_common.c | 49 ++++++++- > drivers/bus/pci/rte_bus_pci.h | 35 ++++++- > drivers/bus/pci/version.map | 2 + > drivers/crypto/virtio/virtio_pci.c | 67 ++++-------- > drivers/event/dlb2/pf/dlb2_main.c | 156 ++++++++-------------------- > drivers/net/bnx2x/bnx2x.c | 86 ++++++++------- > drivers/net/bnx2x/bnx2x.h | 46 -------- > drivers/net/cxgbe/base/adapter.h | 31 +----- > drivers/net/gve/gve_ethdev.c | 46 +------- > drivers/net/gve/gve_ethdev.h | 14 +-- > drivers/net/hns3/hns3_ethdev_vf.c | 109 +++---------------- > drivers/net/ngbe/base/ngbe_hw.c | 20 +--- > drivers/net/ngbe/base/ngbe_osdep.h | 3 - > drivers/net/virtio/virtio_pci.c | 131 ++++------------------- > drivers/vdpa/ifc/base/ifcvf_osdep.h | 4 +- > lib/pci/rte_pci.h | 77 +++++++++++++- > 20 files changed, 346 insertions(+), 730 deletions(-) > > -- > 2.41.0 > >
Recheck-request: iol-sample-apps-testing, This is a test email of the retesting framework, please ignore. On Thu, Aug 3, 2023 at 3:50 AM David Marchand <david.marchand@redhat.com> wrote: > Rather than rely on Linux headers to find some PCI(e) standard constants > or reinvent the same PCI capability helper, this series complements the > pci library and the pci bus driver. > PCI drivers can then use OS agnostic macros and helpers. > > WARNING: this is only compile tested. > > -- > David Marchand > > David Marchand (14): > drivers: remove duplicated PCI master control > bus/pci: add const to some experimental API > bus/pci: find PCI capability > pci: define some capability constants > pci: define some MSIX constants > pci: define some command constants > pci: define some BAR constants > pci: define some PM constants > pci: define some PCIe constants > pci: define some extended capability constants > pci: define some ACS constants > pci: define some PRI constants > pci: define some AER constants > devtools: forbid inclusion of Linux header for PCI > > devtools/checkpatches.sh | 8 ++ > drivers/bus/pci/linux/pci_init.h | 18 ---- > drivers/bus/pci/linux/pci_uio.c | 32 +----- > drivers/bus/pci/linux/pci_vfio.c | 142 ++++++------------------- > drivers/bus/pci/pci_common.c | 49 ++++++++- > drivers/bus/pci/rte_bus_pci.h | 35 ++++++- > drivers/bus/pci/version.map | 2 + > drivers/crypto/virtio/virtio_pci.c | 67 ++++-------- > drivers/event/dlb2/pf/dlb2_main.c | 156 ++++++++-------------------- > drivers/net/bnx2x/bnx2x.c | 86 ++++++++------- > drivers/net/bnx2x/bnx2x.h | 46 -------- > drivers/net/cxgbe/base/adapter.h | 31 +----- > drivers/net/gve/gve_ethdev.c | 46 +------- > drivers/net/gve/gve_ethdev.h | 14 +-- > drivers/net/hns3/hns3_ethdev_vf.c | 109 +++---------------- > drivers/net/ngbe/base/ngbe_hw.c | 20 +--- > drivers/net/ngbe/base/ngbe_osdep.h | 3 - > drivers/net/virtio/virtio_pci.c | 131 ++++------------------- > drivers/vdpa/ifc/base/ifcvf_osdep.h | 4 +- > lib/pci/rte_pci.h | 77 +++++++++++++- > 20 files changed, 346 insertions(+), 730 deletions(-) > > -- > 2.41.0 > >
Recheck-request: iol-sample-apps-testing This is a test of the retesting system, please ignore this email. On Thu, Aug 3, 2023 at 3:50 AM David Marchand <david.marchand@redhat.com> wrote: > Rather than rely on Linux headers to find some PCI(e) standard constants > or reinvent the same PCI capability helper, this series complements the > pci library and the pci bus driver. > PCI drivers can then use OS agnostic macros and helpers. > > WARNING: this is only compile tested. > > -- > David Marchand > > David Marchand (14): > drivers: remove duplicated PCI master control > bus/pci: add const to some experimental API > bus/pci: find PCI capability > pci: define some capability constants > pci: define some MSIX constants > pci: define some command constants > pci: define some BAR constants > pci: define some PM constants > pci: define some PCIe constants > pci: define some extended capability constants > pci: define some ACS constants > pci: define some PRI constants > pci: define some AER constants > devtools: forbid inclusion of Linux header for PCI > > devtools/checkpatches.sh | 8 ++ > drivers/bus/pci/linux/pci_init.h | 18 ---- > drivers/bus/pci/linux/pci_uio.c | 32 +----- > drivers/bus/pci/linux/pci_vfio.c | 142 ++++++------------------- > drivers/bus/pci/pci_common.c | 49 ++++++++- > drivers/bus/pci/rte_bus_pci.h | 35 ++++++- > drivers/bus/pci/version.map | 2 + > drivers/crypto/virtio/virtio_pci.c | 67 ++++-------- > drivers/event/dlb2/pf/dlb2_main.c | 156 ++++++++-------------------- > drivers/net/bnx2x/bnx2x.c | 86 ++++++++------- > drivers/net/bnx2x/bnx2x.h | 46 -------- > drivers/net/cxgbe/base/adapter.h | 31 +----- > drivers/net/gve/gve_ethdev.c | 46 +------- > drivers/net/gve/gve_ethdev.h | 14 +-- > drivers/net/hns3/hns3_ethdev_vf.c | 109 +++---------------- > drivers/net/ngbe/base/ngbe_hw.c | 20 +--- > drivers/net/ngbe/base/ngbe_osdep.h | 3 - > drivers/net/virtio/virtio_pci.c | 131 ++++------------------- > drivers/vdpa/ifc/base/ifcvf_osdep.h | 4 +- > lib/pci/rte_pci.h | 77 +++++++++++++- > 20 files changed, 346 insertions(+), 730 deletions(-) > > -- > 2.41.0 > >