This return statement is indented one tab too far.  Delete a tab.

Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
---
 drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c b/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c
index 9d91a382612d..8b30a3accd31 100644
--- a/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c
+++ b/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c
@@ -2967,7 +2967,7 @@ static inline int i40e_check_vf_permission(struct i40e_vf *vf,
 			dev_err(&pf->pdev->dev,
 				"Cannot add more MAC addresses: VF reached its maximum allowed limit (%d)\n",
 				mac_add_max);
-				return -EPERM;
+			return -EPERM;
 		}
 		if (!vf_trusted) {
 			dev_err(&pf->pdev->dev,
-- 
2.51.0

> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-bounces@osuosl.org> On Behalf
> Of Dan Carpenter
> Sent: Friday, November 21, 2025 2:35 PM
> To: Mohammad Heib <mheib@redhat.com>
> Cc: Nguyen, Anthony L <anthony.l.nguyen@intel.com>; Kitszel,
> Przemyslaw <przemyslaw.kitszel@intel.com>; Andrew Lunn
> <andrew+netdev@lunn.ch>; David S. Miller <davem@davemloft.net>; Eric
> Dumazet <edumazet@google.com>; Jakub Kicinski <kuba@kernel.org>; Paolo
> Abeni <pabeni@redhat.com>; intel-wired-lan@lists.osuosl.org;
> netdev@vger.kernel.org; linux-kernel@vger.kernel.org; kernel-
> janitors@vger.kernel.org
> Subject: [Intel-wired-lan] [PATCH net-next] i40e: delete a stray tab
> 
> This return statement is indented one tab too far.  Delete a tab.
> 
> Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
> ---
>  drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c
> b/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c
> index 9d91a382612d..8b30a3accd31 100644
> --- a/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c
> +++ b/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c
> @@ -2967,7 +2967,7 @@ static inline int
> i40e_check_vf_permission(struct i40e_vf *vf,
>  			dev_err(&pf->pdev->dev,
>  				"Cannot add more MAC addresses: VF reached
> its maximum allowed limit (%d)\n",
>  				mac_add_max);
> -				return -EPERM;
> +			return -EPERM;
>  		}
>  		if (!vf_trusted) {
>  			dev_err(&pf->pdev->dev,
> --
> 2.51.0

Reviewed-by: Aleksandr Loktionov <aleksandr.loktionov@intel.com>

On 11/21/2025 5:35 AM, Dan Carpenter wrote:
> This return statement is indented one tab too far.  Delete a tab.
> 
> Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>

netdev maintainers. This seems straightforward enough, did you want to 
take this directly?

If so,

Reviewed-by: Tony Nguyen <anthony.l.nguyen@intel.com>

> ---
>   drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c b/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c
> index 9d91a382612d..8b30a3accd31 100644
> --- a/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c
> +++ b/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c
> @@ -2967,7 +2967,7 @@ static inline int i40e_check_vf_permission(struct i40e_vf *vf,
>   			dev_err(&pf->pdev->dev,
>   				"Cannot add more MAC addresses: VF reached its maximum allowed limit (%d)\n",
>   				mac_add_max);
> -				return -EPERM;
> +			return -EPERM;
>   		}
>   		if (!vf_trusted) {
>   			dev_err(&pf->pdev->dev,

On Fri, 21 Nov 2025 14:35:07 -0800 Tony Nguyen wrote:
> On 11/21/2025 5:35 AM, Dan Carpenter wrote:
> > This return statement is indented one tab too far.  Delete a tab.
> > 
> > Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>  
> 
> netdev maintainers. This seems straightforward enough, did you want to 
> take this directly?

Will, do!
 
> Reviewed-by: Tony Nguyen <anthony.l.nguyen@intel.com>