> -----Original Message-----
> From: Richardson, Bruce <bruce.richard...@intel.com>
> Sent: Wednesday, August 28, 2024 3:55 PM
> To: Zeng, ZhichaoX <zhichaox.z...@intel.com>
> Cc: dev@dpdk.org
> Subject: Re: [PATCH] net/ice: support for more flexible loading of DDP package
>
> On Wed, Aug 28, 2024 at 11:53:35AM +0800, Zhichao Zeng wrote:
> > The "Dynamic Device Personalization" package is loaded at
> > initialization time by the driver, but the specific package file
> > loaded depends upon what package file is found first by searching
> > through a hard-coded list of firmware paths.
> >
> > To enable greater control over the package loading, this commit two
> > ways to support custom DDP packages:
> > 1. Add device option to choose a specific DDP package file to load.
> > For example:
> > -a 80:00.0,ddp_pkg_file=/path/to/ice-version.pkg
> > 2. Read firmware search path from
> > "/sys/module/firmware_class/parameters/path" like the kernel behavior.
> >
> > Signed-off-by: Bruce Richardson <bruce.richard...@intel.com>
> > Signed-off-by: Zhichao Zeng <zhichaox.z...@intel.com>
>
> Hi Zhichao,
>
> since there are two different methods being supported for picking a DDP
> package this patch would be better split into two, one for each method added.
>
> The support for #1 above is already on-list as a standalone patch[1], so you
> really only need to do a new patch for #2 above. However, I'm ok for you to
> take my patch and include it in a 2-patch set for this if you prefer, since
> both
> patches will be related to choosing a DDP file. I'll leave it up to you
> whether v2
> is a single patch for the search path, or a 2-patch set including [1].
>
> Regards,
> /Bruce
>
> [1]
> https://patches.dpdk.org/project/dpdk/patch/20240812152815.1132697-
> 5-bruce.richard...@intel.com/
>
Hi Bruce,
Thanks for your comments, sorry I didn't check the patchwork and didn't notice
that #1 had been submitted, I'll rework the patch for #2 separately, thanks.
Regards
Zhichao
> > ---
> > doc/guides/nics/ice.rst | 12 +++++++
> > drivers/net/ice/ice_ethdev.c | 61
> > ++++++++++++++++++++++++++++++++++++
> > drivers/net/ice/ice_ethdev.h | 2 ++
> > 3 files changed, 75 insertions(+)
> >
<Snip>
