Messages in this thread | | | From | "Keller, Jacob E" <> | Subject | RE: [Intel-wired-lan] [PATCH net-next v5 3/4] net: devlink: make the devlink_ops::info_get() callback optional | Date | Tue, 29 Nov 2022 00:24:18 +0000 |
| |
> -----Original Message----- > From: Intel-wired-lan <intel-wired-lan-bounces@osuosl.org> On Behalf Of > Vincent Mailhol > Sent: Monday, November 28, 2022 4:06 PM > To: Jiri Pirko <jiri@nvidia.com>; netdev@vger.kernel.org; Jakub Kicinski > <kuba@kernel.org> > Cc: Andrew Lunn <andrew@lunn.ch>; Shijith Thotton <sthotton@marvell.com>; > Simon Horman <simon.horman@corigine.com>; Kurt Kanzenbach > <kurt@linutronix.de>; Eric Dumazet <edumazet@google.com>; Jerin Jacob > <jerinj@marvell.com>; Subbaraya Sundeep <sbhatta@marvell.com>; Ioana > Ciornei <ioana.ciornei@nxp.com>; drivers@pensando.io; Linu Cherian > <lcherian@marvell.com>; Florian Fainelli <f.fainelli@gmail.com>; Herbert Xu > <herbert@gondor.apana.org.au>; Leon Romanovsky <leon@kernel.org>; linux- > rdma@vger.kernel.org; Shalom Toledo <shalomt@mellanox.com>; Srujana > Challa <schalla@marvell.com>; Minghao Chi <chi.minghao@zte.com.cn>; Hao > Chen <chenhao288@hisilicon.com>; Guangbin Huang > <huangguangbin2@huawei.com>; Shannon Nelson <snelson@pensando.io>; > intel-wired-lan@lists.osuosl.org; Vadim Fedorenko <vadfed@fb.com>; Paolo > Abeni <pabeni@redhat.com>; Yisen Zhuang <yisen.zhuang@huawei.com>; Sunil > Goutham <sgoutham@marvell.com>; Ariel Elior <aelior@marvell.com>; Ido > Schimmel <idosch@nvidia.com>; Richard Cochran <richardcochran@gmail.com>; > Arnaud Ebalard <arno@natisbad.org>; Jiri Pirko <jiri@mellanox.com>; Michael > Chan <michael.chan@broadcom.com>; Vincent Mailhol > <mailhol.vincent@wanadoo.fr>; Petr Machata <petrm@nvidia.com>; Salil Mehta > <salil.mehta@huawei.com>; Dimitris Michailidis <dmichail@fungible.com>; > Manish Chopra <manishc@marvell.com>; Boris Brezillon > <bbrezillon@kernel.org>; oss-drivers@corigine.com; Vadim Pasternak > <vadimp@mellanox.com>; linux-kernel@vger.kernel.org; David S . Miller > <davem@davemloft.net>; Taras Chornyi <tchornyi@marvell.com>; hariprasad > <hkelam@marvell.com>; linux-crypto@vger.kernel.org; Jonathan Lemon > <jonathan.lemon@gmail.com>; Vladimir Oltean <olteanv@gmail.com>; Saeed > Mahameed <saeedm@nvidia.com>; Geetha sowjanya <gakula@marvell.com> > Subject: [Intel-wired-lan] [PATCH net-next v5 3/4] net: devlink: make the > devlink_ops::info_get() callback optional > > Some drivers only reported the driver name in their > devlink_ops::info_get() callback. Now that the core provides this > information, the callback became empty. For such drivers, just > removing the callback would prevent the core from executing > devlink_nl_info_fill() meaning that "devlink dev info" would not > return anything. > > Make the callback function optional by executing > devlink_nl_info_fill() even if devlink_ops::info_get() is NULL. > > N.B.: the drivers with devlink support which previously did not > implement devlink_ops::info_get() will now also be able to report > the driver name. >
Makes sense to me.
Thanks, Jake
| |