[PATCH] drm/meson/meson_venc: Relax the supported mode checks

[Da Xue]

On Fri, Feb 10, 2023, 3:56 AM Carlo Caione <ccaione at baylibre.com> wrote:

> Relax a bit the supported modes list by including also 480x1920. This
> was actually tested on real hardware and it works correctly.
>
> Signed-off-by: Carlo Caione <ccaione at baylibre.com>
> ---
> drm/meson/meson_venc: Add more supported resolutions.
>
> In particular relax the resolution checks to allow more resolutions like
> 480x1920.
>
> To: Neil Armstrong <neil.armstrong at linaro.org>
> To: David Airlie <airlied at gmail.com>
> To: Daniel Vetter <daniel at ffwll.ch>
> To: Kevin Hilman <khilman at baylibre.com>
> To: Jerome Brunet <jbrunet at baylibre.com>
> To: Martin Blumenstingl <martin.blumenstingl at googlemail.com>
> Cc: dri-devel at lists.freedesktop.org
> Cc: linux-amlogic at lists.infradead.org
> Cc: linux-arm-kernel at lists.infradead.org
> Cc: linux-kernel at vger.kernel.org
> ---
>  drivers/gpu/drm/meson/meson_venc.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/meson/meson_venc.c
> b/drivers/gpu/drm/meson/meson_venc.c
> index 3c55ed003359..5b3621589a9f 100644
> --- a/drivers/gpu/drm/meson/meson_venc.c
> +++ b/drivers/gpu/drm/meson/meson_venc.c
> @@ -866,10 +866,10 @@ meson_venc_hdmi_supported_mode(const struct
> drm_display_mode *mode)
>                             DRM_MODE_FLAG_PVSYNC | DRM_MODE_FLAG_NVSYNC))
>                 return MODE_BAD;
>
> -       if (mode->hdisplay < 640 || mode->hdisplay > 1920)
> +       if (mode->hdisplay < 480 || mode->hdisplay > 1920)
>                 return MODE_BAD_HVALUE;
>
> -       if (mode->vdisplay < 480 || mode->vdisplay > 1200)
> +       if (mode->vdisplay < 480 || mode->vdisplay > 1920)
>                 return MODE_BAD_VVALUE;
>
Should these be lowered to 400 instead of 480?


>         return MODE_OK;
>
> ---
> base-commit: 38c1e0c65865426676123cc9a127526fa02bcac6
> change-id: 20230210-relax_dmt_limits-355e2a1c7ccf
>
> Best regards,
> --
> Carlo Caione
>
> _______________________________________________
> linux-amlogic mailing list
> linux-amlogic at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-amlogic
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20230213/c82175ec/attachment-0001.htm>