]> www.infradead.org Git - nvme.git/commitdiff
drm/amd/display: Use ARRAY_SIZE for array length
authorJiapeng Chong <jiapeng.chong@linux.alibaba.com>
Mon, 24 Jun 2024 03:11:58 +0000 (11:11 +0800)
committerAlex Deucher <alexander.deucher@amd.com>
Thu, 27 Jun 2024 21:10:39 +0000 (17:10 -0400)
Use of macro ARRAY_SIZE to calculate array size minimizes
the redundant code and improves code reusability.

./drivers/gpu/drm/amd/display/modules/hdcp/hdcp_ddc.c:164:45-46: WARNING: Use ARRAY_SIZE.
./drivers/gpu/drm/amd/display/modules/hdcp/hdcp_ddc.c:183:47-48: WARNING: Use ARRAY_SIZE.
./drivers/gpu/drm/amd/display/modules/hdcp/hdcp_ddc.c:237:45-46: WARNING: Use ARRAY_SIZE.
./drivers/gpu/drm/amd/display/modules/hdcp/hdcp_ddc.c:257:47-48: WARNING: Use ARRAY_SIZE.

Reported-by: Abaci Robot <abaci@linux.alibaba.com>
Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=9405
Signed-off-by: Jiapeng Chong <jiapeng.chong@linux.alibaba.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/modules/hdcp/hdcp_ddc.c

index 1b2df97226a3f2b72a3b37e061aeb6f082e2e8f2..7ecf76aea950514e12b4afd8599b147245d31285 100644 (file)
@@ -161,8 +161,7 @@ static enum mod_hdcp_status read(struct mod_hdcp *hdcp,
                return MOD_HDCP_STATUS_DDC_FAILURE;
 
        if (is_dp_hdcp(hdcp)) {
-               int num_dpcd_addrs = sizeof(hdcp_dpcd_addrs) /
-                       sizeof(hdcp_dpcd_addrs[0]);
+               int num_dpcd_addrs = ARRAY_SIZE(hdcp_dpcd_addrs);
                if (msg_id >= num_dpcd_addrs)
                        return MOD_HDCP_STATUS_DDC_FAILURE;
 
@@ -180,8 +179,7 @@ static enum mod_hdcp_status read(struct mod_hdcp *hdcp,
                        data_offset += cur_size;
                }
        } else {
-               int num_i2c_offsets = sizeof(hdcp_i2c_offsets) /
-                       sizeof(hdcp_i2c_offsets[0]);
+               int num_i2c_offsets = ARRAY_SIZE(hdcp_i2c_offsets);
                if (msg_id >= num_i2c_offsets)
                        return MOD_HDCP_STATUS_DDC_FAILURE;
 
@@ -234,8 +232,7 @@ static enum mod_hdcp_status write(struct mod_hdcp *hdcp,
                return MOD_HDCP_STATUS_DDC_FAILURE;
 
        if (is_dp_hdcp(hdcp)) {
-               int num_dpcd_addrs = sizeof(hdcp_dpcd_addrs) /
-                       sizeof(hdcp_dpcd_addrs[0]);
+               int num_dpcd_addrs = ARRAY_SIZE(hdcp_dpcd_addrs);
                if (msg_id >= num_dpcd_addrs)
                        return MOD_HDCP_STATUS_DDC_FAILURE;
 
@@ -254,8 +251,7 @@ static enum mod_hdcp_status write(struct mod_hdcp *hdcp,
                        data_offset += cur_size;
                }
        } else {
-               int num_i2c_offsets = sizeof(hdcp_i2c_offsets) /
-                       sizeof(hdcp_i2c_offsets[0]);
+               int num_i2c_offsets = ARRAY_SIZE(hdcp_i2c_offsets);
                if (msg_id >= num_i2c_offsets)
                        return MOD_HDCP_STATUS_DDC_FAILURE;