> -----Original Message----- > From: Jakub Kicinski <[email protected]> > Sent: Tuesday, June 18, 2019 4:24 AM > To: Sudarsana Reddy Kalluru <[email protected]> > Cc: [email protected]; [email protected]; Michal Kalderon > <[email protected]>; Ariel Elior <[email protected]>; Jiri Pirko > <[email protected]> > Subject: [EXT] Re: [PATCH net-next 4/4] qed: Add devlink support for > configuration attributes. > > External Email > > ---------------------------------------------------------------------- > On Mon, 17 Jun 2019 04:45:28 -0700, Sudarsana Reddy Kalluru wrote: > > This patch adds implementation for devlink callbacks for reading/ > > configuring the device attributes. > > > > Signed-off-by: Sudarsana Reddy Kalluru <[email protected]> > > Signed-off-by: Ariel Elior <[email protected]> > > You need to provide documentation for your parameters, plus some of them > look like they should potentially be port params, not device params.
Thanks a lot for your review. Will add the required documentation. In case of Marvell adapter, any of the device/adapter/port parameters can be read/configurable via any PF (ethdev) on the port. Hence adding the commands at device level. Hope this is fine.
