Am 27.07.2017 um 22:43 schrieb Alex Deucher: > Use a lower case b to be consistent with the other wb functions. > > Signed-off-by: Alex Deucher <alexander.deucher at amd.com> I still suggest to get rid of all those variants and always allocate 256bits, but anyway the patch is Reviewed-by: Christian König <christian.koenig at amd.com>. > --- > drivers/gpu/drm/amd/amdgpu/amdgpu.h | 2 +- > drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 2 +- > drivers/gpu/drm/amd/amdgpu/amdgpu_ring.c | 2 +- > 3 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu.h b/drivers/gpu/drm/amd/amdgpu/amdgpu.h > index f364900..5f8d2e8 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu.h > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu.h > @@ -1192,7 +1192,7 @@ struct amdgpu_wb { > int amdgpu_wb_get(struct amdgpu_device *adev, u32 *wb); > void amdgpu_wb_free(struct amdgpu_device *adev, u32 wb); > int amdgpu_wb_get_64bit(struct amdgpu_device *adev, u32 *wb); > -int amdgpu_wb_get_256Bit(struct amdgpu_device *adev, u32 *wb); > +int amdgpu_wb_get_256bit(struct amdgpu_device *adev, u32 *wb); > void amdgpu_wb_free_64bit(struct amdgpu_device *adev, u32 wb); > void amdgpu_wb_free_256bit(struct amdgpu_device *adev, u32 wb); > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > index 06c214f..41f6af3 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > @@ -602,7 +602,7 @@ int amdgpu_wb_get_64bit(struct amdgpu_device *adev, u32 *wb) > } > } > > -int amdgpu_wb_get_256Bit(struct amdgpu_device *adev, u32 *wb) > +int amdgpu_wb_get_256bit(struct amdgpu_device *adev, u32 *wb) > { > int i = 0; > unsigned long offset = bitmap_find_next_zero_area_off(adev->wb.used, > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ring.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ring.c > index 15b7149..3874be8 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ring.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ring.c > @@ -213,7 +213,7 @@ int amdgpu_ring_init(struct amdgpu_device *adev, struct amdgpu_ring *ring, > } > > if (amdgpu_sriov_vf(adev) && ring->funcs->type == AMDGPU_RING_TYPE_GFX) { > - r = amdgpu_wb_get_256Bit(adev, &ring->fence_offs); > + r = amdgpu_wb_get_256bit(adev, &ring->fence_offs); > if (r) { > dev_err(adev->dev, "(%d) ring fence_offs wb alloc failed\n", r); > return r;