Commit e56e9ad0 authored by Alvin Lee's avatar Alvin Lee Committed by Alex Deucher
Browse files

drm/amd/display: Fix check for null function ptr 



[Why]
Bug fix for null function ptr (should check for NULL instead of not
NULL)

[How]
Fix if condition

Tested-by: default avatarDaniel Wheeler <daniel.wheeler@amd.com>
Reviewed-by: default avatarSamson Tam <samson.tam@amd.com>
Acked-by: default avatarRodrigo Siqueira <Rodrigo.Siqueira@amd.com>
Signed-off-by: default avatarAlvin Lee <Alvin.Lee2@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent cdbc5838

drivers/gpu/drm/amd/display/dmub/src/dmub_srv.c

+2 −2
Original line number Diff line number Diff line
@@ -852,7 +852,7 @@ bool dmub_srv_should_detect(struct dmub_srv *dmub) 


enum dmub_status dmub_srv_clear_inbox0_ack(struct dmub_srv *dmub)

{

	if (!dmub->hw_init || dmub->hw_funcs.clear_inbox0_ack_register)

	if (!dmub->hw_init || !dmub->hw_funcs.clear_inbox0_ack_register)

		return DMUB_STATUS_INVALID;



	dmub->hw_funcs.clear_inbox0_ack_register(dmub);
@@ -878,7 +878,7 @@ enum dmub_status dmub_srv_wait_for_inbox0_ack(struct dmub_srv *dmub, uint32_t ti 
enum dmub_status dmub_srv_send_inbox0_cmd(struct dmub_srv *dmub,

		union dmub_inbox0_data_register data)

{

	if (!dmub->hw_init || dmub->hw_funcs.send_inbox0_cmd)

	if (!dmub->hw_init || !dmub->hw_funcs.send_inbox0_cmd)

		return DMUB_STATUS_INVALID;



	dmub->hw_funcs.send_inbox0_cmd(dmub, data);