Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit ff923479 authored by Tom St Denis's avatar Tom St Denis Committed by Alex Deucher
Browse files

drm/amd/amdgpu: make afmt_init cleanup if alloc fails (DCEv8) 



If the allocation fails free memory and return error code.

Signed-off-by: default avatarTom St Denis <tom.stdenis@amd.com>
Reviewed-by: default avatarAlex Deucher <alexander.deucher@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 74c1e842

drivers/gpu/drm/amd/amdgpu/dce_v8_0.c

+12 −2
Original line number Diff line number Diff line
@@ -1913,7 +1913,7 @@ static void dce_v8_0_afmt_enable(struct drm_encoder *encoder, bool enable) 
		  enable ? "En" : "Dis", dig->afmt->offset, amdgpu_encoder->encoder_id);

}



static void dce_v8_0_afmt_init(struct amdgpu_device *adev)

static int dce_v8_0_afmt_init(struct amdgpu_device *adev)

{

	int i;
@@ -1926,9 +1926,17 @@ static void dce_v8_0_afmt_init(struct amdgpu_device *adev) 
		if (adev->mode_info.afmt[i]) {

			adev->mode_info.afmt[i]->offset = dig_offsets[i];

			adev->mode_info.afmt[i]->id = i;

		} else {

			int j;

			for (j = 0; j < i; j++) {

				kfree(adev->mode_info.afmt[j]);

				adev->mode_info.afmt[j] = NULL;

			}

			return -ENOMEM;

		}

	}

	return 0;

}



static void dce_v8_0_afmt_fini(struct amdgpu_device *adev)

{
@@ -2923,7 +2931,9 @@ static int dce_v8_0_sw_init(void *handle) 
		return -EINVAL;



	/* setup afmt */

	dce_v8_0_afmt_init(adev);

	r = dce_v8_0_afmt_init(adev);

	if (r)

		return r;



	r = dce_v8_0_audio_init(adev);

	if (r)