Commit 49afc040 authored by Nihar Chaithanya's avatar Nihar Chaithanya Committed by Jakub Kicinski
Browse files

octeontx2-pf: mcs: Remove dead code and semi-colon from rsrc_name() 



Every case in the switch-block ends with return statement, and the
default: branch handles the cases where rsrc_type is invalid and
returns "Unknown", this makes the return statement at the end of the
function unreachable and redundant.
The semi-colon is not required after the switch-block's curly braces.

Remove the semi-colon after the switch-block's curly braces and the
return statement at the end of the function.

This issue was reported by Coverity Scan.

Signed-off-by: default avatarNihar Chaithanya <niharchaithanya@gmail.com>
Link: https://patch.msgid.link/20250104171905.13293-1-niharchaithanya@gmail.com


Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent 21a8a77a

drivers/net/ethernet/marvell/octeontx2/nic/cn10k_macsec.c

+1 −3
Original line number Diff line number Diff line
@@ -133,9 +133,7 @@ static const char *rsrc_name(enum mcs_rsrc_type rsrc_type) 
		return "SA";

	default:

		return "Unknown";

	};



	return "Unknown";

	}

}



static int cn10k_mcs_alloc_rsrc(struct otx2_nic *pfvf, enum mcs_direction dir,