Commit 9b79594a authored by Jinjie Ruan's avatar Jinjie Ruan Committed by Steven Price
Browse files

drm/panthor: Use the BITS_PER_LONG macro 



sizeof(unsigned long) * 8 is the number of bits in an unsigned long
variable, replace it with BITS_PER_LONG macro to make them simpler.

And fix the warning:
	WARNING: Comparisons should place the constant on the right side of the test
	#23: FILE: drivers/gpu/drm/panthor/panthor_mmu.c:2696:
	+       if (BITS_PER_LONG < va_bits) {

Signed-off-by: default avatarJinjie Ruan <ruanjinjie@huawei.com>
Reviewed-by: default avatarSteven Price <steven.price@arm.com>
Signed-off-by: default avatarSteven Price <steven.price@arm.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20240902094404.1943710-1-ruanjinjie@huawei.com
parent 3a8d9761

drivers/gpu/drm/panthor/panthor_mmu.c

+2 −2
Original line number Diff line number Diff line
@@ -2693,9 +2693,9 @@ int panthor_mmu_init(struct panthor_device *ptdev) 
	 * which passes iova as an unsigned long. Patch the mmu_features to reflect this

	 * limitation.

	 */

	if (sizeof(unsigned long) * 8 < va_bits) {

	if (va_bits > BITS_PER_LONG) {

		ptdev->gpu_info.mmu_features &= ~GENMASK(7, 0);

		ptdev->gpu_info.mmu_features |= sizeof(unsigned long) * 8;

		ptdev->gpu_info.mmu_features |= BITS_PER_LONG;

	}



	return drmm_add_action_or_reset(&ptdev->base, panthor_mmu_release_wq, mmu->vm.wq);