drm: radeon: Replace drm_fb_get_bpp_depth() with drm_format_plane_cpp()
authorLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Mon, 17 Oct 2016 22:41:18 +0000 (01:41 +0300)
committerArchit Taneja <architt@codeaurora.org>
Tue, 18 Oct 2016 09:53:44 +0000 (15:23 +0530)
The driver needs the number of bytes per pixel, not the bpp and depth
info meant for fbdev compatibility. Use the right API.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Archit Taneja <architt@codeaurora.org>
Link: http://patchwork.freedesktop.org/patch/msgid/1476744081-24485-11-git-send-email-laurent.pinchart@ideasonboard.com
drivers/gpu/drm/radeon/radeon_fb.c
drivers/gpu/drm/radeon/radeon_gem.c

index 0daad446d2c754768f501604ff3bc8fe623731aa..f65f29911dca14595dd4c70c7f5abe0cc3193559 100644 (file)
@@ -89,13 +89,13 @@ static struct fb_ops radeonfb_ops = {
 };
 
 
-int radeon_align_pitch(struct radeon_device *rdev, int width, int bpp, bool tiled)
+int radeon_align_pitch(struct radeon_device *rdev, int width, int cpp, bool tiled)
 {
        int aligned = width;
        int align_large = (ASIC_IS_AVIVO(rdev)) || tiled;
        int pitch_mask = 0;
 
-       switch (bpp / 8) {
+       switch (cpp) {
        case 1:
                pitch_mask = align_large ? 255 : 127;
                break;
@@ -110,7 +110,7 @@ int radeon_align_pitch(struct radeon_device *rdev, int width, int bpp, bool tile
 
        aligned += pitch_mask;
        aligned &= ~pitch_mask;
-       return aligned;
+       return aligned * cpp;
 }
 
 static void radeonfb_destroy_pinned_object(struct drm_gem_object *gobj)
@@ -139,13 +139,13 @@ static int radeonfb_create_pinned_object(struct radeon_fbdev *rfbdev,
        int ret;
        int aligned_size, size;
        int height = mode_cmd->height;
-       u32 bpp, depth;
+       u32 cpp;
 
-       drm_fb_get_bpp_depth(mode_cmd->pixel_format, &depth, &bpp);
+       cpp = drm_format_plane_cpp(mode_cmd->pixel_format, 0);
 
        /* need to align pitch with crtc limits */
-       mode_cmd->pitches[0] = radeon_align_pitch(rdev, mode_cmd->width, bpp,
-                                                 fb_tiled) * ((bpp + 1) / 8);
+       mode_cmd->pitches[0] = radeon_align_pitch(rdev, mode_cmd->width, cpp,
+                                                 fb_tiled);
 
        if (rdev->family >= CHIP_R600)
                height = ALIGN(mode_cmd->height, 8);
@@ -165,11 +165,11 @@ static int radeonfb_create_pinned_object(struct radeon_fbdev *rfbdev,
                tiling_flags = RADEON_TILING_MACRO;
 
 #ifdef __BIG_ENDIAN
-       switch (bpp) {
-       case 32:
+       switch (cpp) {
+       case 4:
                tiling_flags |= RADEON_TILING_SWAP_32BIT;
                break;
-       case 16:
+       case 2:
                tiling_flags |= RADEON_TILING_SWAP_16BIT;
        default:
                break;
index deb9511725c93acf01943df0bf0e0a4904220eaf..0bcffd8a7bd3ceac0de37cd0b44344011963be77 100644 (file)
@@ -745,7 +745,8 @@ int radeon_mode_dumb_create(struct drm_file *file_priv,
        uint32_t handle;
        int r;
 
-       args->pitch = radeon_align_pitch(rdev, args->width, args->bpp, 0) * ((args->bpp + 1) / 8);
+       args->pitch = radeon_align_pitch(rdev, args->width,
+                                        DIV_ROUND_UP(args->bpp, 8), 0);
        args->size = args->pitch * args->height;
        args->size = ALIGN(args->size, PAGE_SIZE);