Subject: Re: [PATCH] Octeontx2-af: Fix missing error code in cgx_probe()

From: Simon Horman <[email-masked]>
Date: Mon, 13 Oct 2025 10:23:26 +0100

On Fri, Oct 10, 2025 at 01:42:39PM -0700, Harshit Mogalapalli wrote:
> When CGX fails mapping to NIX, set the error code to -ENODEV, currently
> err is zero and that is treated as success path.
> 
> Reported-by: Dan Carpenter <[email-masked]>
> Closes: https://lore.kernel.org/all/[email-masked]/
> Fixes: d280233fc866 ("Octeontx2-af: Fix NIX X2P calibration failures")
> Signed-off-by: Harshit Mogalapalli <[email-masked]>
> ---
> This is based on static analysis with smatch and only compile tested.

Thanks,

I agree that Smatch is onto something here.

Reviewed-by: Simon Horman <[email-masked]>