diff options
author | Christian König <christian.koenig@amd.com> | 2016-09-12 11:16:16 (GMT) |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2016-09-14 19:10:41 (GMT) |
commit | 5d98d0bcff64c9a76d21da9648881e14ed26fff6 (patch) | |
tree | 162704703fc64cd4c98c87229cd7e6dc0207056b /drivers/gpu/drm/ttm | |
parent | 5756b1558eabdfb087aa9896250c6decd12bf872 (diff) | |
download | linux-5d98d0bcff64c9a76d21da9648881e14ed26fff6.tar.xz |
drm/ttm: use ffs in ttm_mem_type_from_place
A bit pointless to search for the first bit set manually.
Signed-off-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Reviewed-by: Chunming Zhou <david1.zhou@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/ttm')
-rw-r--r-- | drivers/gpu/drm/ttm/ttm_bo.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/drivers/gpu/drm/ttm/ttm_bo.c b/drivers/gpu/drm/ttm/ttm_bo.c index c2a30bd..fc6217d 100644 --- a/drivers/gpu/drm/ttm/ttm_bo.c +++ b/drivers/gpu/drm/ttm/ttm_bo.c @@ -57,14 +57,14 @@ static struct attribute ttm_bo_count = { static inline int ttm_mem_type_from_place(const struct ttm_place *place, uint32_t *mem_type) { - int i; + int pos; - for (i = 0; i <= TTM_PL_PRIV5; i++) - if (place->flags & (1 << i)) { - *mem_type = i; - return 0; - } - return -EINVAL; + pos = ffs(place->flags & TTM_PL_MASK_MEM); + if (unlikely(!pos)) + return -EINVAL; + + *mem_type = pos - 1; + return 0; } static void ttm_mem_type_debug(struct ttm_bo_device *bdev, int mem_type) |