Commit 24adfaff authored by Bernard Zhao's avatar Bernard Zhao Committed by Alex Deucher
Browse files

drm/amd/display: cleanup the code a bit 



In function dc_sink_destruct, kfree will check pointer, no need
to check again.
This change is to cleanup the code a bit.

Acked-by: default avatarChristian König <christian.koenig@amd.com>
Signed-off-by: default avatarBernard Zhao <bernard@vivo.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 13d20aab

drivers/gpu/drm/amd/display/dc/core/dc_sink.c

+1 −9
Original line number Diff line number Diff line
@@ -33,14 +33,6 @@ 
 * Private functions

 ******************************************************************************/



static void dc_sink_destruct(struct dc_sink *sink)

{

	if (sink->dc_container_id) {

		kfree(sink->dc_container_id);

		sink->dc_container_id = NULL;

	}

}



static bool dc_sink_construct(struct dc_sink *sink, const struct dc_sink_init_data *init_params)

{
@@ -75,7 +67,7 @@ void dc_sink_retain(struct dc_sink *sink) 
static void dc_sink_free(struct kref *kref)

{

	struct dc_sink *sink = container_of(kref, struct dc_sink, refcount);

	dc_sink_destruct(sink);

	kfree(sink->dc_container_id);

	kfree(sink);

}