Hello there,

Static analyser cppcheck says:

linux-6.18-rc4/drivers/gpu/drm/amd/display/dc/dpp/dcn30/dcn30_dpp.c:96:45: 
style: Same expression on both sides of '/' because 
'sizeof(dpp_input_csc_matrix)' and 'sizeof(struct dpp_input_csc_matrix)' 
represent the same value. [duplicateExpression]

Source code is

    int arr_size = sizeof(dpp_input_csc_matrix)/sizeof(struct 
dpp_input_csc_matrix);

Isn't this just a fancy way of setting arr_size to 1 ?
Suggest code rework.

Regards

David Binderman

Reply via email to