[RFC v3 16/26] bus/pci: make driver-only headers private

Xu, Rosen rosen.xu at intel.com
Fri Jul 29 04:41:35 CEST 2022


Hi,

> -----Original Message-----
> From: David Marchand <david.marchand at redhat.com>
> Sent: Thursday, July 28, 2022 23:27
> To: dev at dpdk.org
> Cc: Chautru, Nicolas <nicolas.chautru at intel.com>; Ray Kinsella
> <mdr at ashroe.eu>; Nithin Dabilpuram <ndabilpuram at marvell.com>; Kiran
> Kumar K <kirankumark at marvell.com>; Sunil Kumar Kori
> <skori at marvell.com>; Satha Rao <skoteshwar at marvell.com>; Matan Azrad
> <matan at nvidia.com>; Viacheslav Ovsiienko <viacheslavo at nvidia.com>;
> Zhang, Roy Fan <roy.fan.zhang at intel.com>; Andrew Rybchenko
> <andrew.rybchenko at oktetlabs.ru>; Ashish Gupta
> <ashish.gupta at marvell.com>; Chandubabu Namburu <chandu at amd.com>;
> Ankur Dwivedi <adwivedi at marvell.com>; Anoob Joseph
> <anoobj at marvell.com>; Tejasree Kondoj <ktejasree at marvell.com>;
> Nagadheeraj Rottela <rnagadheeraj at marvell.com>; Srikanth Jampala
> <jsrikanth at marvell.com>; Jay Zhou <jianjay.zhou at huawei.com>; Radha
> Mohan Chintakuntla <radhac at marvell.com>; Veerasenareddy Burru
> <vburru at marvell.com>; Chengwen Feng <fengchengwen at huawei.com>;
> Richardson, Bruce <bruce.richardson at intel.com>; Laatz, Kevin
> <kevin.laatz at intel.com>; Walsh, Conor <conor.walsh at intel.com>; McDaniel,
> Timothy <timothy.mcdaniel at intel.com>; Jerin Jacob <jerinj at marvell.com>;
> Pavan Nikhilesh <pbhagavatula at marvell.com>; Elena Agostini
> <eagostini at nvidia.com>; Ashwin Sekhar T K <asekhar at marvell.com>;
> Harman Kalra <hkalra at marvell.com>; Shepard Siegel
> <shepard.siegel at atomicrules.com>; Ed Czeck <ed.czeck at atomicrules.com>;
> John Miller <john.miller at atomicrules.com>; Webster, Steven
> <steven.webster at windriver.com>; Peters, Matt
> <matt.peters at windriver.com>; Rasesh Mody <rmody at marvell.com>;
> Shahed Shaikh <shshaikh at marvell.com>; Ajit Khaparde
> <ajit.khaparde at broadcom.com>; Somnath Kotur
> <somnath.kotur at broadcom.com>; Chas Williams <chas3 at att.com>; Min Hu
> (Connor) <humin29 at huawei.com>; Rahul Lakkireddy
> <rahul.lakkireddy at chelsio.com>; Su, Simei <simei.su at intel.com>; Wu,
> Wenjun1 <wenjun1.wu at intel.com>; Marcin Wojtas <mw at semihalf.com>;
> Michal Krawczyk <mk at semihalf.com>; Shai Brandes
> <shaibran at amazon.com>; Evgeny Schemeilin <evgenys at amazon.com>;
> Igor Chauskin <igorch at amazon.com>; Daley, John <johndale at cisco.com>;
> Hyong Youb Kim <hyonkim at cisco.com>; Ziyang Xuan
> <xuanziyang2 at huawei.com>; Xiaoyun Wang
> <cloud.wangxiaoyun at huawei.com>; Guoyang Zhou
> <zhouguoyang at huawei.com>; Dongdong Liu <liudongdong3 at huawei.com>;
> Yisen Zhuang <yisen.zhuang at huawei.com>; Zhang, Yuying
> <yuying.zhang at intel.com>; Xing, Beilei <beilei.xing at intel.com>; Guo,
> Junfeng <junfeng.guo at intel.com>; Andrew Boyer <aboyer at pensando.io>;
> Xu, Rosen <rosen.xu at intel.com>; Yang, Qiming <qiming.yang at intel.com>;
> Stephen Hemminger <sthemmin at microsoft.com>; Long Li
> <longli at microsoft.com>; Jiawen Wu <jiawenwu at trustnetic.com>; Devendra
> Singh Rawat <dsinghrawat at marvell.com>; Maciej Czekaj
> <mczekaj at marvell.com>; Jian Wang <jianwang at trustnetic.com>; Maxime
> Coquelin <maxime.coquelin at redhat.com>; Xia, Chenbo
> <chenbo.xia at intel.com>; Jochen Behrens <jbehrens at vmware.com>; Jakub
> Palider <jpalider at marvell.com>; Tomasz Duszynski
> <tduszynski at marvell.com>; Zhang, Tianfei <tianfei.zhang at intel.com>; Wu,
> Jingjing <jingjing.wu at intel.com>; Ori Kam <orika at nvidia.com>; Wang, Xiao
> W <xiao.w.wang at intel.com>; Vijay Kumar Srivastava <vsrivast at xilinx.com>;
> Thomas Monjalon <thomas at monjalon.net>; Ferruh Yigit
> <ferruh.yigit at xilinx.com>
> Subject: [RFC v3 16/26] bus/pci: make driver-only headers private
> 
> The pci bus interface is for drivers only.
> Mark as internal and move the header in the driver headers list.
> 
> While at it, cleanup the code:
> - fix indentation,
> - remove unneeded reference to bus specific singleton object,
> - remove unneeded list head structure type,
> - reorder the definitions and macro manipulating the bus singleton object,
> - remove inclusion of rte_bus.h and fix the code that relied on implicit
>   inclusion,
> 
> Signed-off-by: David Marchand <david.marchand at redhat.com>
> ---
> Changes since RFC v2:
> - squashed cleanup for this bus code from other patches,
> - updated release notes,
> 
Acked-by: Rosen Xu <rosen.xu at intel.com>


More information about the dev mailing list