Re: [PATCH v3 02/22] common/idpf: updated IDPF VF device ID

["Burakov, Anatoly" <anatoly.burakov@intel.com>]

On 6/12/2024 5:52 AM, Soumyadeep Hore wrote:
> Update IDPF VF device id to 145C.
> 
> Also added device ID for S-IOV device.
> 
> Signed-off-by: Soumyadeep Hore <soumyadeep.hore@intel.com>
> ---
>   drivers/common/idpf/base/idpf_devids.h | 7 +++++--
>   1 file changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/common/idpf/base/idpf_devids.h b/drivers/common/idpf/base/idpf_devids.h
> index c47762d5b7..0eb2def264 100644
> --- a/drivers/common/idpf/base/idpf_devids.h
> +++ b/drivers/common/idpf/base/idpf_devids.h
> @@ -1,5 +1,5 @@
>   /* SPDX-License-Identifier: BSD-3-Clause
> - * Copyright(c) 2001-2023 Intel Corporation
> + * Copyright(c) 2001-2024 Intel Corporation
>    */
>   
>   #ifndef _IDPF_DEVIDS_H_
> @@ -10,7 +10,10 @@
>   
>   /* Device IDs */
>   #define IDPF_DEV_ID_PF			0x1452
> -#define IDPF_DEV_ID_VF			0x1889
> +#define IDPF_DEV_ID_VF			0x145C
> +#ifdef SIOV_SUPPORT
> +#define IDPF_DEV_ID_VF_SIOV		0x0DD5
> +#endif /* SIOV_SUPPORT */
>   

Is there any reason why we need those ifdefs, and are they expected to 
be enabled/disabled in some way by the build process?

-- 
Thanks,
Anatoly