Re: [PATCH] i40e (gcc13): synchronize allocate/free functions return type & values

From: Jiri Slaby
Date: Thu Nov 03 2022 - 08:03:52 EST


On 03. 11. 22, 4:41, Jakub Kicinski wrote:
On Mon, 31 Oct 2022 12:44:56 +0100 Jiri Slaby (SUSE) wrote:
I.e. the type of their return value in the definition is int, while the
declaration spell enum i40e_status. Synchronize the definitions to the
latter.

And make sure proper values are returned. I.e. I40E_SUCCESS and not 0,
I40E_ERR_NO_MEMORY and not -ENOMEM.

Let's go the opposite way, towards using standard errno.

This is propagated several layers up throughout the whole i40e driver. It would be a mass change which I'd rather leave up to the driver maintainers -- I don't even have the HW to test.

thanks,
--
js