[PATCH v2] drm/amdgpu:Limit the resolution for virtual_display

[Alex Deucher]

On Thu, Jan 7, 2021 at 8:45 PM Deng, Emily <Emily.Deng at amd.com> wrote:
>
> [AMD Official Use Only - Internal Distribution Only]
>
> Ping ......
>

It's not clear what problem this solves.

Alex


> Best wishes
> Emily Deng
>
>
>
> >-----Original Message-----
> >From: Emily Deng <Emily.Deng at amd.com>
> >Sent: Thursday, January 7, 2021 11:29 AM
> >To: amd-gfx at lists.freedesktop.org
> >Cc: Deng, Emily <Emily.Deng at amd.com>
> >Subject: [PATCH v2] drm/amdgpu:Limit the resolution for virtual_display
> >
> >From: "Emily.Deng" <Emily.Deng at amd.com>
> >
> >Limit the resolution not bigger than 16384, which means
> >dev->mode_info.num_crtc * common_modes[i].w not bigger than 16384.
> >
> >v2:
> >  Refine the code
> >
> >Signed-off-by: Emily.Deng <Emily.Deng at amd.com>
> >---
> > drivers/gpu/drm/amd/amdgpu/dce_virtual.c | 7 +++++--
> > 1 file changed, 5 insertions(+), 2 deletions(-)
> >
> >diff --git a/drivers/gpu/drm/amd/amdgpu/dce_virtual.c
> >b/drivers/gpu/drm/amd/amdgpu/dce_virtual.c
> >index 2b16c8faca34..fd2b3a6dfd60 100644
> >--- a/drivers/gpu/drm/amd/amdgpu/dce_virtual.c
> >+++ b/drivers/gpu/drm/amd/amdgpu/dce_virtual.c
> >@@ -319,6 +319,7 @@ dce_virtual_encoder(struct drm_connector
> >*connector)  static int dce_virtual_get_modes(struct drm_connector
> >*connector)  {
> > struct drm_device *dev = connector->dev;
> >+struct amdgpu_device *adev = dev->dev_private;
> > struct drm_display_mode *mode = NULL;
> > unsigned i;
> > static const struct mode_size {
> >@@ -350,8 +351,10 @@ static int dce_virtual_get_modes(struct
> >drm_connector *connector)
> > };
> >
> > for (i = 0; i < ARRAY_SIZE(common_modes); i++) {
> >-mode = drm_cvt_mode(dev, common_modes[i].w,
> >common_modes[i].h, 60, false, false, false);
> >-drm_mode_probed_add(connector, mode);
> >+if (adev->mode_info.num_crtc * common_modes[i].w <=
> >16384) {
> >+mode = drm_cvt_mode(dev, common_modes[i].w,
> >common_modes[i].h, 60, false, false, false);
> >+drm_mode_probed_add(connector, mode);
> >+}
> > }
> >
> > return 0;
> >--
> >2.25.1
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx