Commit b72585f1 authored by Jiapeng Chong's avatar Jiapeng Chong Committed by Alex Deucher
Browse files

drm/amd/display: Use ARRAY_SIZE for array length 



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: default avatarAbaci Robot <abaci@linux.alibaba.com>
Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=9405


Signed-off-by: default avatarJiapeng Chong <jiapeng.chong@linux.alibaba.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 429508c8

drivers/gpu/drm/amd/display/modules/hdcp/hdcp_ddc.c

+4 −8
Original line number Diff line number Diff line
@@ -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;