[Intel-wired-lan] [PATCH net v3] i40e: Fix for displaying message regarding NVM version

[Paul Menzel]

Dear Mateusz,


Am 07.12.21 um 14:25 schrieb Mateusz Palczewski:
> When loading the i40e driver, it prints a message like: 'The driver for the
> device detected a newer version of the NVM image v1.x than expected v1.y.
> Please install the most recent version of the network driver.' This is
> misleading as the driver is working as expected.
> 
> Fix that by removing the second part of message and changing it from
> dev_info to dev_dgb.
> 
> Fixes: 4fb29bddb ("i40e: The driver now prints the API version in error message")
> Signed-off-by: Mateusz Palczewski <mateusz.palczewski at intel.com>
> ---
>   v3: Changed the patch after community review
>   v2: Fixed the fixes tag to be 12 digits long
> ---
>   drivers/net/ethernet/intel/i40e/i40e_main.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c b/drivers/net/ethernet/intel/i40e/i40e_main.c
> index 4664d07..c07746b 100644
> --- a/drivers/net/ethernet/intel/i40e/i40e_main.c
> +++ b/drivers/net/ethernet/intel/i40e/i40e_main.c
> @@ -15469,8 +15469,8 @@ static int i40e_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
>   
>   	if (hw->aq.api_maj_ver == I40E_FW_API_VERSION_MAJOR &&
>   	    hw->aq.api_min_ver > I40E_FW_MINOR_VERSION(hw))
> -		dev_info(&pdev->dev,
> -			 "The driver for the device detected a newer version of the NVM image v%u.%u than expected v%u.%u. Please install the most recent version of the network driver.\n",
> +		dev_dbg(&pdev->dev,
> +			"The driver for the device detected a newer version of the NVM image v%u.%u than expected v%u.%u.\n",
>   			 hw->aq.api_maj_ver,
>   			 hw->aq.api_min_ver,
>   			 I40E_FW_API_VERSION_MAJOR,
> 

It’s a debug message now, so only a few people are going to notice, but 
a note on what to do to get rid of the message, would be still be nice.

> To disable this informational message, update to a newer driver version.


Kind regards,

Paul