All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alex Deucher <alexdeucher@gmail.com>
To: Evan Quan <evan.quan@amd.com>
Cc: "amd-gfx list" <amd-gfx@lists.freedesktop.org>,
	"Deucher, Alexander" <Alexander.Deucher@amd.com>,
	"Thomas Zimmermann" <tzimmermann@suse.de>,
	"Christian König" <christian.koenig@amd.com>
Subject: Re: [PATCH] drm/amdgpu: use generic fb helpers instead of setting up AMD own's.
Date: Mon, 13 Sep 2021 10:36:31 -0400	[thread overview]
Message-ID: <CADnq5_P-u4w4zMn-hoxuTortkh1XJegTfs=zLVcX=+q832A8XA@mail.gmail.com> (raw)
In-Reply-To: <20210910032530.725067-1-evan.quan@amd.com>

On Thu, Sep 9, 2021 at 11:25 PM Evan Quan <evan.quan@amd.com> wrote:
>
> With the shadow buffer support from generic framebuffer emulation, it's
> possible now to have runpm kicked when no update for console.
>
> Change-Id: I285472c9100ee6f649d3f3f3548f402b9cd34eaf
> Signed-off-by: Evan Quan <evan.quan@amd.com>
> Acked-by: Christian König <christian.koenig@amd.com>

Reviewed-by: Alex Deucher <alexander.deucher@amd.com>

> --
> v1->v2:
>   - rename amdgpu_align_pitch as amdgpu_gem_align_pitch to align with
>     other APIs from the same file (Alex)
> ---
>  drivers/gpu/drm/amd/amdgpu/Makefile         |   2 +-
>  drivers/gpu/drm/amd/amdgpu/amdgpu_device.c  |  12 +-
>  drivers/gpu/drm/amd/amdgpu/amdgpu_display.c |  11 +-
>  drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c     |  13 +
>  drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c      | 388 --------------------
>  drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c     |  30 +-
>  drivers/gpu/drm/amd/amdgpu/amdgpu_mode.h    |  20 -
>  7 files changed, 50 insertions(+), 426 deletions(-)
>  delete mode 100644 drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/Makefile b/drivers/gpu/drm/amd/amdgpu/Makefile
> index 8d0748184a14..73a2151ee43f 100644
> --- a/drivers/gpu/drm/amd/amdgpu/Makefile
> +++ b/drivers/gpu/drm/amd/amdgpu/Makefile
> @@ -45,7 +45,7 @@ amdgpu-y += amdgpu_device.o amdgpu_kms.o \
>         amdgpu_atombios.o atombios_crtc.o amdgpu_connectors.o \
>         atom.o amdgpu_fence.o amdgpu_ttm.o amdgpu_object.o amdgpu_gart.o \
>         amdgpu_encoders.o amdgpu_display.o amdgpu_i2c.o \
> -       amdgpu_fb.o amdgpu_gem.o amdgpu_ring.o \
> +       amdgpu_gem.o amdgpu_ring.o \
>         amdgpu_cs.o amdgpu_bios.o amdgpu_benchmark.o amdgpu_test.o \
>         atombios_dp.o amdgpu_afmt.o amdgpu_trace_points.o \
>         atombios_encoders.o amdgpu_sa.o atombios_i2c.o \
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> index 682d459e992a..bcc308b7f826 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> @@ -3695,8 +3695,6 @@ int amdgpu_device_init(struct amdgpu_device *adev,
>         /* Get a log2 for easy divisions. */
>         adev->mm_stats.log2_max_MBps = ilog2(max(1u, max_MBps));
>
> -       amdgpu_fbdev_init(adev);
> -
>         r = amdgpu_pm_sysfs_init(adev);
>         if (r) {
>                 adev->pm_sysfs_en = false;
> @@ -3854,8 +3852,6 @@ void amdgpu_device_fini_hw(struct amdgpu_device *adev)
>                 amdgpu_ucode_sysfs_fini(adev);
>         sysfs_remove_files(&adev->dev->kobj, amdgpu_dev_attributes);
>
> -       amdgpu_fbdev_fini(adev);
> -
>         amdgpu_irq_fini_hw(adev);
>
>         amdgpu_device_ip_fini_early(adev);
> @@ -3931,7 +3927,7 @@ int amdgpu_device_suspend(struct drm_device *dev, bool fbcon)
>         drm_kms_helper_poll_disable(dev);
>
>         if (fbcon)
> -               amdgpu_fbdev_set_suspend(adev, 1);
> +               drm_fb_helper_set_suspend_unlocked(adev_to_drm(adev)->fb_helper, true);
>
>         cancel_delayed_work_sync(&adev->delayed_init_work);
>
> @@ -4009,7 +4005,7 @@ int amdgpu_device_resume(struct drm_device *dev, bool fbcon)
>         flush_delayed_work(&adev->delayed_init_work);
>
>         if (fbcon)
> -               amdgpu_fbdev_set_suspend(adev, 0);
> +               drm_fb_helper_set_suspend_unlocked(adev_to_drm(adev)->fb_helper, false);
>
>         drm_kms_helper_poll_enable(dev);
>
> @@ -4638,7 +4634,7 @@ int amdgpu_do_asic_reset(struct list_head *device_list_handle,
>                                 if (r)
>                                         goto out;
>
> -                               amdgpu_fbdev_set_suspend(tmp_adev, 0);
> +                               drm_fb_helper_set_suspend_unlocked(adev_to_drm(tmp_adev)->fb_helper, false);
>
>                                 /*
>                                  * The GPU enters bad state once faulty pages
> @@ -5025,7 +5021,7 @@ int amdgpu_device_gpu_recover(struct amdgpu_device *adev,
>                  */
>                 amdgpu_unregister_gpu_instance(tmp_adev);
>
> -               amdgpu_fbdev_set_suspend(tmp_adev, 1);
> +               drm_fb_helper_set_suspend_unlocked(adev_to_drm(adev)->fb_helper, true);
>
>                 /* disable ras on ALL IPs */
>                 if (!need_emergency_restart &&
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c
> index 7a7316731911..58bfc7f00d76 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c
> @@ -1572,13 +1572,10 @@ int amdgpu_display_suspend_helper(struct amdgpu_device *adev)
>                         continue;
>                 }
>                 robj = gem_to_amdgpu_bo(fb->obj[0]);
> -               /* don't unpin kernel fb objects */
> -               if (!amdgpu_fbdev_robj_is_fb(adev, robj)) {
> -                       r = amdgpu_bo_reserve(robj, true);
> -                       if (r == 0) {
> -                               amdgpu_bo_unpin(robj);
> -                               amdgpu_bo_unreserve(robj);
> -                       }
> +               r = amdgpu_bo_reserve(robj, true);
> +               if (r == 0) {
> +                       amdgpu_bo_unpin(robj);
> +                       amdgpu_bo_unreserve(robj);
>                 }
>         }
>         return 0;
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
> index 987482c26b1a..db21af5e84ed 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
> @@ -1348,6 +1348,19 @@ static int amdgpu_pci_probe(struct pci_dev *pdev,
>                 goto err_pci;
>         }
>
> +       /*
> +        * 1. don't init fbdev on hw without DCE
> +        * 2. don't init fbdev if there are no connectors
> +        */
> +       if (adev->mode_info.mode_config_initialized &&
> +           !list_empty(&adev_to_drm(adev)->mode_config.connector_list)) {
> +               /* select 8 bpp console on low vram cards */
> +               if (adev->gmc.real_vram_size <= (32*1024*1024))
> +                       drm_fbdev_generic_setup(adev_to_drm(adev), 8);
> +               else
> +                       drm_fbdev_generic_setup(adev_to_drm(adev), 32);
> +       }
> +
>         ret = amdgpu_debugfs_init(adev);
>         if (ret)
>                 DRM_ERROR("Creating debugfs files failed (%d).\n", ret);
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c
> deleted file mode 100644
> index cd0acbea75da..000000000000
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c
> +++ /dev/null
> @@ -1,388 +0,0 @@
> -/*
> - * Copyright © 2007 David Airlie
> - *
> - * Permission is hereby granted, free of charge, to any person obtaining a
> - * copy of this software and associated documentation files (the "Software"),
> - * to deal in the Software without restriction, including without limitation
> - * the rights to use, copy, modify, merge, publish, distribute, sublicense,
> - * and/or sell copies of the Software, and to permit persons to whom the
> - * Software is furnished to do so, subject to the following conditions:
> - *
> - * The above copyright notice and this permission notice (including the next
> - * paragraph) shall be included in all copies or substantial portions of the
> - * Software.
> - *
> - * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> - * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> - * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
> - * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> - * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
> - * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
> - * DEALINGS IN THE SOFTWARE.
> - *
> - * Authors:
> - *     David Airlie
> - */
> -
> -#include <linux/module.h>
> -#include <linux/pm_runtime.h>
> -#include <linux/slab.h>
> -#include <linux/vga_switcheroo.h>
> -
> -#include <drm/amdgpu_drm.h>
> -#include <drm/drm_crtc.h>
> -#include <drm/drm_crtc_helper.h>
> -#include <drm/drm_fb_helper.h>
> -#include <drm/drm_fourcc.h>
> -
> -#include "amdgpu.h"
> -#include "cikd.h"
> -#include "amdgpu_gem.h"
> -
> -#include "amdgpu_display.h"
> -
> -/* object hierarchy -
> -   this contains a helper + a amdgpu fb
> -   the helper contains a pointer to amdgpu framebuffer baseclass.
> -*/
> -
> -static int
> -amdgpufb_open(struct fb_info *info, int user)
> -{
> -       struct drm_fb_helper *fb_helper = info->par;
> -       int ret = pm_runtime_get_sync(fb_helper->dev->dev);
> -       if (ret < 0 && ret != -EACCES) {
> -               pm_runtime_mark_last_busy(fb_helper->dev->dev);
> -               pm_runtime_put_autosuspend(fb_helper->dev->dev);
> -               return ret;
> -       }
> -       return 0;
> -}
> -
> -static int
> -amdgpufb_release(struct fb_info *info, int user)
> -{
> -       struct drm_fb_helper *fb_helper = info->par;
> -
> -       pm_runtime_mark_last_busy(fb_helper->dev->dev);
> -       pm_runtime_put_autosuspend(fb_helper->dev->dev);
> -       return 0;
> -}
> -
> -static const struct fb_ops amdgpufb_ops = {
> -       .owner = THIS_MODULE,
> -       DRM_FB_HELPER_DEFAULT_OPS,
> -       .fb_open = amdgpufb_open,
> -       .fb_release = amdgpufb_release,
> -       .fb_fillrect = drm_fb_helper_cfb_fillrect,
> -       .fb_copyarea = drm_fb_helper_cfb_copyarea,
> -       .fb_imageblit = drm_fb_helper_cfb_imageblit,
> -};
> -
> -
> -int amdgpu_align_pitch(struct amdgpu_device *adev, int width, int cpp, bool tiled)
> -{
> -       int aligned = width;
> -       int pitch_mask = 0;
> -
> -       switch (cpp) {
> -       case 1:
> -               pitch_mask = 255;
> -               break;
> -       case 2:
> -               pitch_mask = 127;
> -               break;
> -       case 3:
> -       case 4:
> -               pitch_mask = 63;
> -               break;
> -       }
> -
> -       aligned += pitch_mask;
> -       aligned &= ~pitch_mask;
> -       return aligned * cpp;
> -}
> -
> -static void amdgpufb_destroy_pinned_object(struct drm_gem_object *gobj)
> -{
> -       struct amdgpu_bo *abo = gem_to_amdgpu_bo(gobj);
> -       int ret;
> -
> -       ret = amdgpu_bo_reserve(abo, true);
> -       if (likely(ret == 0)) {
> -               amdgpu_bo_kunmap(abo);
> -               amdgpu_bo_unpin(abo);
> -               amdgpu_bo_unreserve(abo);
> -       }
> -       drm_gem_object_put(gobj);
> -}
> -
> -static int amdgpufb_create_pinned_object(struct amdgpu_fbdev *rfbdev,
> -                                        struct drm_mode_fb_cmd2 *mode_cmd,
> -                                        struct drm_gem_object **gobj_p)
> -{
> -       const struct drm_format_info *info;
> -       struct amdgpu_device *adev = rfbdev->adev;
> -       struct drm_gem_object *gobj = NULL;
> -       struct amdgpu_bo *abo = NULL;
> -       bool fb_tiled = false; /* useful for testing */
> -       u32 tiling_flags = 0, domain;
> -       int ret;
> -       int aligned_size, size;
> -       int height = mode_cmd->height;
> -       u32 cpp;
> -       u64 flags = AMDGPU_GEM_CREATE_CPU_ACCESS_REQUIRED |
> -                              AMDGPU_GEM_CREATE_VRAM_CONTIGUOUS     |
> -                              AMDGPU_GEM_CREATE_VRAM_CLEARED;
> -
> -       info = drm_get_format_info(adev_to_drm(adev), mode_cmd);
> -       cpp = info->cpp[0];
> -
> -       /* need to align pitch with crtc limits */
> -       mode_cmd->pitches[0] = amdgpu_align_pitch(adev, mode_cmd->width, cpp,
> -                                                 fb_tiled);
> -       domain = amdgpu_display_supported_domains(adev, flags);
> -       height = ALIGN(mode_cmd->height, 8);
> -       size = mode_cmd->pitches[0] * height;
> -       aligned_size = ALIGN(size, PAGE_SIZE);
> -       ret = amdgpu_gem_object_create(adev, aligned_size, 0, domain, flags,
> -                                      ttm_bo_type_device, NULL, &gobj);
> -       if (ret) {
> -               pr_err("failed to allocate framebuffer (%d)\n", aligned_size);
> -               return -ENOMEM;
> -       }
> -       abo = gem_to_amdgpu_bo(gobj);
> -
> -       if (fb_tiled)
> -               tiling_flags = AMDGPU_TILING_SET(ARRAY_MODE, GRPH_ARRAY_2D_TILED_THIN1);
> -
> -       ret = amdgpu_bo_reserve(abo, false);
> -       if (unlikely(ret != 0))
> -               goto out_unref;
> -
> -       if (tiling_flags) {
> -               ret = amdgpu_bo_set_tiling_flags(abo,
> -                                                tiling_flags);
> -               if (ret)
> -                       dev_err(adev->dev, "FB failed to set tiling flags\n");
> -       }
> -
> -       ret = amdgpu_bo_pin(abo, domain);
> -       if (ret) {
> -               amdgpu_bo_unreserve(abo);
> -               goto out_unref;
> -       }
> -
> -       ret = amdgpu_ttm_alloc_gart(&abo->tbo);
> -       if (ret) {
> -               amdgpu_bo_unreserve(abo);
> -               dev_err(adev->dev, "%p bind failed\n", abo);
> -               goto out_unref;
> -       }
> -
> -       ret = amdgpu_bo_kmap(abo, NULL);
> -       amdgpu_bo_unreserve(abo);
> -       if (ret) {
> -               goto out_unref;
> -       }
> -
> -       *gobj_p = gobj;
> -       return 0;
> -out_unref:
> -       amdgpufb_destroy_pinned_object(gobj);
> -       *gobj_p = NULL;
> -       return ret;
> -}
> -
> -static int amdgpufb_create(struct drm_fb_helper *helper,
> -                          struct drm_fb_helper_surface_size *sizes)
> -{
> -       struct amdgpu_fbdev *rfbdev = (struct amdgpu_fbdev *)helper;
> -       struct amdgpu_device *adev = rfbdev->adev;
> -       struct fb_info *info;
> -       struct drm_framebuffer *fb = NULL;
> -       struct drm_mode_fb_cmd2 mode_cmd;
> -       struct drm_gem_object *gobj = NULL;
> -       struct amdgpu_bo *abo = NULL;
> -       int ret;
> -
> -       memset(&mode_cmd, 0, sizeof(mode_cmd));
> -       mode_cmd.width = sizes->surface_width;
> -       mode_cmd.height = sizes->surface_height;
> -
> -       if (sizes->surface_bpp == 24)
> -               sizes->surface_bpp = 32;
> -
> -       mode_cmd.pixel_format = drm_mode_legacy_fb_format(sizes->surface_bpp,
> -                                                         sizes->surface_depth);
> -
> -       ret = amdgpufb_create_pinned_object(rfbdev, &mode_cmd, &gobj);
> -       if (ret) {
> -               DRM_ERROR("failed to create fbcon object %d\n", ret);
> -               return ret;
> -       }
> -
> -       abo = gem_to_amdgpu_bo(gobj);
> -
> -       /* okay we have an object now allocate the framebuffer */
> -       info = drm_fb_helper_alloc_fbi(helper);
> -       if (IS_ERR(info)) {
> -               ret = PTR_ERR(info);
> -               goto out;
> -       }
> -
> -       ret = amdgpu_display_gem_fb_init(adev_to_drm(adev), &rfbdev->rfb,
> -                                        &mode_cmd, gobj);
> -       if (ret) {
> -               DRM_ERROR("failed to initialize framebuffer %d\n", ret);
> -               goto out;
> -       }
> -
> -       fb = &rfbdev->rfb.base;
> -
> -       /* setup helper */
> -       rfbdev->helper.fb = fb;
> -
> -       info->fbops = &amdgpufb_ops;
> -
> -       info->fix.smem_start = amdgpu_gmc_vram_cpu_pa(adev, abo);
> -       info->fix.smem_len = amdgpu_bo_size(abo);
> -       info->screen_base = amdgpu_bo_kptr(abo);
> -       info->screen_size = amdgpu_bo_size(abo);
> -
> -       drm_fb_helper_fill_info(info, &rfbdev->helper, sizes);
> -
> -       /* setup aperture base/size for vesafb takeover */
> -       info->apertures->ranges[0].base = adev_to_drm(adev)->mode_config.fb_base;
> -       info->apertures->ranges[0].size = adev->gmc.aper_size;
> -
> -       /* Use default scratch pixmap (info->pixmap.flags = FB_PIXMAP_SYSTEM) */
> -
> -       if (info->screen_base == NULL) {
> -               ret = -ENOSPC;
> -               goto out;
> -       }
> -
> -       DRM_INFO("fb mappable at 0x%lX\n",  info->fix.smem_start);
> -       DRM_INFO("vram apper at 0x%lX\n",  (unsigned long)adev->gmc.aper_base);
> -       DRM_INFO("size %lu\n", (unsigned long)amdgpu_bo_size(abo));
> -       DRM_INFO("fb depth is %d\n", fb->format->depth);
> -       DRM_INFO("   pitch is %d\n", fb->pitches[0]);
> -
> -       vga_switcheroo_client_fb_set(adev->pdev, info);
> -       return 0;
> -
> -out:
> -       if (fb && ret) {
> -               drm_gem_object_put(gobj);
> -               drm_framebuffer_unregister_private(fb);
> -               drm_framebuffer_cleanup(fb);
> -               kfree(fb);
> -       }
> -       return ret;
> -}
> -
> -static int amdgpu_fbdev_destroy(struct drm_device *dev, struct amdgpu_fbdev *rfbdev)
> -{
> -       struct amdgpu_framebuffer *rfb = &rfbdev->rfb;
> -       int i;
> -
> -       drm_fb_helper_unregister_fbi(&rfbdev->helper);
> -
> -       if (rfb->base.obj[0]) {
> -               for (i = 0; i < rfb->base.format->num_planes; i++)
> -                       drm_gem_object_put(rfb->base.obj[0]);
> -               amdgpufb_destroy_pinned_object(rfb->base.obj[0]);
> -               rfb->base.obj[0] = NULL;
> -               drm_framebuffer_unregister_private(&rfb->base);
> -               drm_framebuffer_cleanup(&rfb->base);
> -       }
> -       drm_fb_helper_fini(&rfbdev->helper);
> -
> -       return 0;
> -}
> -
> -static const struct drm_fb_helper_funcs amdgpu_fb_helper_funcs = {
> -       .fb_probe = amdgpufb_create,
> -};
> -
> -int amdgpu_fbdev_init(struct amdgpu_device *adev)
> -{
> -       struct amdgpu_fbdev *rfbdev;
> -       int bpp_sel = 32;
> -       int ret;
> -
> -       /* don't init fbdev on hw without DCE */
> -       if (!adev->mode_info.mode_config_initialized)
> -               return 0;
> -
> -       /* don't init fbdev if there are no connectors */
> -       if (list_empty(&adev_to_drm(adev)->mode_config.connector_list))
> -               return 0;
> -
> -       /* select 8 bpp console on low vram cards */
> -       if (adev->gmc.real_vram_size <= (32*1024*1024))
> -               bpp_sel = 8;
> -
> -       rfbdev = kzalloc(sizeof(struct amdgpu_fbdev), GFP_KERNEL);
> -       if (!rfbdev)
> -               return -ENOMEM;
> -
> -       rfbdev->adev = adev;
> -       adev->mode_info.rfbdev = rfbdev;
> -
> -       drm_fb_helper_prepare(adev_to_drm(adev), &rfbdev->helper,
> -                             &amdgpu_fb_helper_funcs);
> -
> -       ret = drm_fb_helper_init(adev_to_drm(adev), &rfbdev->helper);
> -       if (ret) {
> -               kfree(rfbdev);
> -               return ret;
> -       }
> -
> -       /* disable all the possible outputs/crtcs before entering KMS mode */
> -       if (!amdgpu_device_has_dc_support(adev) && !amdgpu_virtual_display)
> -               drm_helper_disable_unused_functions(adev_to_drm(adev));
> -
> -       drm_fb_helper_initial_config(&rfbdev->helper, bpp_sel);
> -       return 0;
> -}
> -
> -void amdgpu_fbdev_fini(struct amdgpu_device *adev)
> -{
> -       if (!adev->mode_info.rfbdev)
> -               return;
> -
> -       amdgpu_fbdev_destroy(adev_to_drm(adev), adev->mode_info.rfbdev);
> -       kfree(adev->mode_info.rfbdev);
> -       adev->mode_info.rfbdev = NULL;
> -}
> -
> -void amdgpu_fbdev_set_suspend(struct amdgpu_device *adev, int state)
> -{
> -       if (adev->mode_info.rfbdev)
> -               drm_fb_helper_set_suspend_unlocked(&adev->mode_info.rfbdev->helper,
> -                                                  state);
> -}
> -
> -int amdgpu_fbdev_total_size(struct amdgpu_device *adev)
> -{
> -       struct amdgpu_bo *robj;
> -       int size = 0;
> -
> -       if (!adev->mode_info.rfbdev)
> -               return 0;
> -
> -       robj = gem_to_amdgpu_bo(adev->mode_info.rfbdev->rfb.base.obj[0]);
> -       size += amdgpu_bo_size(robj);
> -       return size;
> -}
> -
> -bool amdgpu_fbdev_robj_is_fb(struct amdgpu_device *adev, struct amdgpu_bo *robj)
> -{
> -       if (!adev->mode_info.rfbdev)
> -               return false;
> -       if (robj == gem_to_amdgpu_bo(adev->mode_info.rfbdev->rfb.base.obj[0]))
> -               return true;
> -       return false;
> -}
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c
> index 9e2525b96d04..355db8438522 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c
> @@ -876,6 +876,32 @@ int amdgpu_gem_op_ioctl(struct drm_device *dev, void *data,
>         return r;
>  }
>
> +static int amdgpu_gem_align_pitch(struct amdgpu_device *adev,
> +                                 int width,
> +                                 int cpp,
> +                                 bool tiled)
> +{
> +       int aligned = width;
> +       int pitch_mask = 0;
> +
> +       switch (cpp) {
> +       case 1:
> +               pitch_mask = 255;
> +               break;
> +       case 2:
> +               pitch_mask = 127;
> +               break;
> +       case 3:
> +       case 4:
> +               pitch_mask = 63;
> +               break;
> +       }
> +
> +       aligned += pitch_mask;
> +       aligned &= ~pitch_mask;
> +       return aligned * cpp;
> +}
> +
>  int amdgpu_mode_dumb_create(struct drm_file *file_priv,
>                             struct drm_device *dev,
>                             struct drm_mode_create_dumb *args)
> @@ -896,8 +922,8 @@ int amdgpu_mode_dumb_create(struct drm_file *file_priv,
>         if (adev->mman.buffer_funcs_enabled)
>                 flags |= AMDGPU_GEM_CREATE_VRAM_CLEARED;
>
> -       args->pitch = amdgpu_align_pitch(adev, args->width,
> -                                        DIV_ROUND_UP(args->bpp, 8), 0);
> +       args->pitch = amdgpu_gem_align_pitch(adev, args->width,
> +                                            DIV_ROUND_UP(args->bpp, 8), 0);
>         args->size = (u64)args->pitch * args->height;
>         args->size = ALIGN(args->size, PAGE_SIZE);
>         domain = amdgpu_bo_get_preferred_domain(adev,
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_mode.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_mode.h
> index 89fb372ed49c..6043bf6fd414 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_mode.h
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_mode.h
> @@ -232,8 +232,6 @@ struct amdgpu_i2c_chan {
>         struct mutex mutex;
>  };
>
> -struct amdgpu_fbdev;
> -
>  struct amdgpu_afmt {
>         bool enabled;
>         int offset;
> @@ -309,13 +307,6 @@ struct amdgpu_framebuffer {
>         uint64_t address;
>  };
>
> -struct amdgpu_fbdev {
> -       struct drm_fb_helper helper;
> -       struct amdgpu_framebuffer rfb;
> -       struct list_head fbdev_list;
> -       struct amdgpu_device *adev;
> -};
> -
>  struct amdgpu_mode_info {
>         struct atom_context *atom_context;
>         struct card_info *atom_card_info;
> @@ -341,8 +332,6 @@ struct amdgpu_mode_info {
>         struct edid *bios_hardcoded_edid;
>         int bios_hardcoded_edid_size;
>
> -       /* pointer to fbdev info structure */
> -       struct amdgpu_fbdev *rfbdev;
>         /* firmware flags */
>         u32 firmware_flags;
>         /* pointer to backlight encoder */
> @@ -631,15 +620,6 @@ bool amdgpu_crtc_get_scanout_position(struct drm_crtc *crtc,
>                         int *hpos, ktime_t *stime, ktime_t *etime,
>                         const struct drm_display_mode *mode);
>
> -/* fbdev layer */
> -int amdgpu_fbdev_init(struct amdgpu_device *adev);
> -void amdgpu_fbdev_fini(struct amdgpu_device *adev);
> -void amdgpu_fbdev_set_suspend(struct amdgpu_device *adev, int state);
> -int amdgpu_fbdev_total_size(struct amdgpu_device *adev);
> -bool amdgpu_fbdev_robj_is_fb(struct amdgpu_device *adev, struct amdgpu_bo *robj);
> -
> -int amdgpu_align_pitch(struct amdgpu_device *adev, int width, int bpp, bool tiled);
> -
>  /* amdgpu_display.c */
>  void amdgpu_display_print_display_setup(struct drm_device *dev);
>  int amdgpu_display_modeset_create_props(struct amdgpu_device *adev);
> --
> 2.29.0
>

  reply	other threads:[~2021-09-13 14:36 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-09-10  3:25 [PATCH] drm/amdgpu: use generic fb helpers instead of setting up AMD own's Evan Quan
2021-09-13 14:36 ` Alex Deucher [this message]
2021-09-20  8:39   ` Thomas Zimmermann
2021-09-20  8:41   ` Thomas Zimmermann
2021-09-20  8:44     ` Thomas Zimmermann
2021-09-30 15:28       ` Alex Deucher
  -- strict thread matches above, loose matches on Subject: below --
2021-09-03  6:37 Evan Quan
2021-09-03  8:36 ` Christian König
2021-09-07 16:57 ` Alex Deucher
2021-09-08  1:22   ` Quan, Evan

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CADnq5_P-u4w4zMn-hoxuTortkh1XJegTfs=zLVcX=+q832A8XA@mail.gmail.com' \
    --to=alexdeucher@gmail.com \
    --cc=Alexander.Deucher@amd.com \
    --cc=amd-gfx@lists.freedesktop.org \
    --cc=christian.koenig@amd.com \
    --cc=evan.quan@amd.com \
    --cc=tzimmermann@suse.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.