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

Commit cb635fd5 authored by Hareesh Gundu's avatar Hareesh Gundu
Browse files

msm: kgsl: Do not delete uninitialized preempt timer



Deleting uninitialized timer can result to warning message.
Add a proper check to skip uninitialized preempt timer.

Change-Id: I3c180435314bb7e52d3ffb2c0934fbb883c3a079
Signed-off-by: default avatarHareesh Gundu <hareeshg@codeaurora.org>
parent 6b3ddc57
Loading
Loading
Loading
Loading
+10 −2
Original line number Diff line number Diff line
@@ -562,7 +562,7 @@ static void a5xx_preemption_iommu_close(struct adreno_device *adreno_dev)
}
#endif

void a5xx_preemption_close(struct adreno_device *adreno_dev)
static void _preemption_close(struct adreno_device *adreno_dev)
{
	struct kgsl_device *device = KGSL_DEVICE(adreno_dev);
	struct adreno_preemption *preempt = &adreno_dev->preempt;
@@ -578,6 +578,14 @@ void a5xx_preemption_close(struct adreno_device *adreno_dev)
	}
}

void a5xx_preemption_close(struct adreno_device *adreno_dev)
{
	if (!test_bit(ADRENO_DEVICE_PREEMPTION, &adreno_dev->priv))
		return;

	_preemption_close(adreno_dev);
}

int a5xx_preemption_init(struct adreno_device *adreno_dev)
{
	struct kgsl_device *device = KGSL_DEVICE(adreno_dev);
@@ -618,7 +626,7 @@ int a5xx_preemption_init(struct adreno_device *adreno_dev)

err:
	if (ret)
		a5xx_preemption_close(adreno_dev);
		_preemption_close(adreno_dev);

	return ret;
}
+10 −2
Original line number Diff line number Diff line
@@ -671,7 +671,7 @@ static void a6xx_preemption_iommu_close(struct adreno_device *adreno_dev)
}
#endif

void a6xx_preemption_close(struct adreno_device *adreno_dev)
static void _preemption_close(struct adreno_device *adreno_dev)
{
	struct kgsl_device *device = KGSL_DEVICE(adreno_dev);
	struct adreno_preemption *preempt = &adreno_dev->preempt;
@@ -691,6 +691,14 @@ void a6xx_preemption_close(struct adreno_device *adreno_dev)
	}
}

void a6xx_preemption_close(struct adreno_device *adreno_dev)
{
	if (!test_bit(ADRENO_DEVICE_PREEMPTION, &adreno_dev->priv))
		return;

	_preemption_close(adreno_dev);
}

int a6xx_preemption_init(struct adreno_device *adreno_dev)
{
	struct kgsl_device *device = KGSL_DEVICE(adreno_dev);
@@ -731,7 +739,7 @@ int a6xx_preemption_init(struct adreno_device *adreno_dev)

err:
	if (ret)
		a6xx_preemption_close(adreno_dev);
		_preemption_close(adreno_dev);

	return ret;
}