Commit 8c2d34eb authored by Jonathan Gray's avatar Jonathan Gray Committed by Alex Deucher

drm/radeon: use kernel is_power_of_2 rather than local version

Follow the amdgpu change made in
76117507
and replace local radeon function with is_power_of_2().
Reviewed-by: default avatarChristian König <christian.koenig@amd.com>
Signed-off-by: default avatarJonathan Gray <jsg@jsg.id.au>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent de05abe6
...@@ -1085,19 +1085,6 @@ static unsigned int radeon_vga_set_decode(struct pci_dev *pdev, bool state) ...@@ -1085,19 +1085,6 @@ static unsigned int radeon_vga_set_decode(struct pci_dev *pdev, bool state)
return VGA_RSRC_NORMAL_IO | VGA_RSRC_NORMAL_MEM; return VGA_RSRC_NORMAL_IO | VGA_RSRC_NORMAL_MEM;
} }
/**
* radeon_check_pot_argument - check that argument is a power of two
*
* @arg: value to check
*
* Validates that a certain argument is a power of two (all asics).
* Returns true if argument is valid.
*/
static bool radeon_check_pot_argument(int arg)
{
return (arg & (arg - 1)) == 0;
}
/** /**
* radeon_gart_size_auto - Determine a sensible default GART size * radeon_gart_size_auto - Determine a sensible default GART size
* according to ASIC family. * according to ASIC family.
...@@ -1126,7 +1113,7 @@ static int radeon_gart_size_auto(enum radeon_family family) ...@@ -1126,7 +1113,7 @@ static int radeon_gart_size_auto(enum radeon_family family)
static void radeon_check_arguments(struct radeon_device *rdev) static void radeon_check_arguments(struct radeon_device *rdev)
{ {
/* vramlimit must be a power of two */ /* vramlimit must be a power of two */
if (!radeon_check_pot_argument(radeon_vram_limit)) { if (!is_power_of_2(radeon_vram_limit)) {
dev_warn(rdev->dev, "vram limit (%d) must be a power of 2\n", dev_warn(rdev->dev, "vram limit (%d) must be a power of 2\n",
radeon_vram_limit); radeon_vram_limit);
radeon_vram_limit = 0; radeon_vram_limit = 0;
...@@ -1140,7 +1127,7 @@ static void radeon_check_arguments(struct radeon_device *rdev) ...@@ -1140,7 +1127,7 @@ static void radeon_check_arguments(struct radeon_device *rdev)
dev_warn(rdev->dev, "gart size (%d) too small\n", dev_warn(rdev->dev, "gart size (%d) too small\n",
radeon_gart_size); radeon_gart_size);
radeon_gart_size = radeon_gart_size_auto(rdev->family); radeon_gart_size = radeon_gart_size_auto(rdev->family);
} else if (!radeon_check_pot_argument(radeon_gart_size)) { } else if (!is_power_of_2(radeon_gart_size)) {
dev_warn(rdev->dev, "gart size (%d) must be a power of 2\n", dev_warn(rdev->dev, "gart size (%d) must be a power of 2\n",
radeon_gart_size); radeon_gart_size);
radeon_gart_size = radeon_gart_size_auto(rdev->family); radeon_gart_size = radeon_gart_size_auto(rdev->family);
...@@ -1163,7 +1150,7 @@ static void radeon_check_arguments(struct radeon_device *rdev) ...@@ -1163,7 +1150,7 @@ static void radeon_check_arguments(struct radeon_device *rdev)
break; break;
} }
if (!radeon_check_pot_argument(radeon_vm_size)) { if (!is_power_of_2(radeon_vm_size)) {
dev_warn(rdev->dev, "VM size (%d) must be a power of 2\n", dev_warn(rdev->dev, "VM size (%d) must be a power of 2\n",
radeon_vm_size); radeon_vm_size);
radeon_vm_size = 4; radeon_vm_size = 4;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment