From: Li Ming Date: Fri, 16 May 2025 14:32:20 +0000 (+0800) Subject: cxl/feature: Remove redundant code of get supported features X-Git-Tag: v6.16-rc1~60^2~1 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=6eed708a5693709ff0d4dd8512b6934be30d4283;p=thirdparty%2Fkernel%2Flinux.git cxl/feature: Remove redundant code of get supported features In cxlctl_get_supported_features(), there is a code block that handles the case where the requested is equal to 0. But the code following the code block can also handle this situation. So the code block is not needed. Signed-off-by: Li Ming Reviewed-by: Jonathan Cameron Reviewed-by: Dave Jiang Link: https://patch.msgid.link/20250516143220.35302-1-ming.li@zohomail.com Signed-off-by: Dave Jiang --- diff --git a/drivers/cxl/core/features.c b/drivers/cxl/core/features.c index bd512100ef8d5..2167a8d6ff85f 100644 --- a/drivers/cxl/core/features.c +++ b/drivers/cxl/core/features.c @@ -426,14 +426,6 @@ static void *cxlctl_get_supported_features(struct cxl_features_state *cxlfs, rpc_out->size = struct_size(feat_out, ents, requested); feat_out = &rpc_out->get_sup_feats_out; - if (requested == 0) { - feat_out->num_entries = cpu_to_le16(requested); - feat_out->supported_feats = - cpu_to_le16(cxlfs->entries->num_features); - rpc_out->retval = CXL_MBOX_CMD_RC_SUCCESS; - *out_len = out_size; - return no_free_ptr(rpc_out); - } for (i = start, pos = &feat_out->ents[0]; i < cxlfs->entries->num_features; i++, pos++) {