On Fri, Feb 10, 2023, 3:56 AM Carlo Caione <ccai...@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 <ccai...@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.armstr...@linaro.org> > To: David Airlie <airl...@gmail.com> > To: Daniel Vetter <dan...@ffwll.ch> > To: Kevin Hilman <khil...@baylibre.com> > To: Jerome Brunet <jbru...@baylibre.com> > To: Martin Blumenstingl <martin.blumensti...@googlemail.com> > Cc: dri-devel@lists.freedesktop.org > Cc: linux-amlo...@lists.infradead.org > Cc: linux-arm-ker...@lists.infradead.org > Cc: linux-ker...@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-amlo...@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-amlogic >
