On 2025-07-01 16:14, Alex Deucher wrote:
> [Why]
> Common resolutions are added to supported modes to enable compatibility
> scenarios that compositors may use to do things like clone displays. There
> is no guarantee however that the panel will natively support these modes.
> 
> [How]
> If the compositor hasn't enabled scaling but a non-native resolution has
> been picked for an LVDS panel turn the scaler on anyway.  This will ensure
> compatibility.
> 
> Signed-off-by: Alex Deucher <alexander.deuc...@amd.com>

Series is
Reviewed-by: Harry Wentland <harry.wentl...@amd.com>

Harry

> ---
>  drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c 
> b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> index 7a4fca9a61bbe..096b23ad4845d 100644
> --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> @@ -7901,7 +7901,8 @@ static int dm_encoder_helper_atomic_check(struct 
> drm_encoder *encoder,
>       int clock, bpp = 0;
>       bool is_y420 = false;
>  
> -     if (connector->connector_type == DRM_MODE_CONNECTOR_eDP) {
> +     if ((connector->connector_type == DRM_MODE_CONNECTOR_eDP) ||
> +         (connector->connector_type == DRM_MODE_CONNECTOR_LVDS)) {
>               struct amdgpu_encoder *amdgpu_encoder = 
> to_amdgpu_encoder(encoder);
>               struct drm_display_mode *native_mode = 
> &amdgpu_encoder->native_mode;
>               enum drm_mode_status result;

Reply via email to