From patchwork Tue Dec 24 02:54:30 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jammy Huang X-Patchwork-Id: 13919659 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 06776E7718B for ; Tue, 24 Dec 2024 02:54:34 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 62FCB10E111; Tue, 24 Dec 2024 02:54:34 +0000 (UTC) Received: from TWMBX01.aspeed.com (mail.aspeedtech.com [211.20.114.72]) by gabe.freedesktop.org (Postfix) with ESMTPS id 85E5E10E111 for ; Tue, 24 Dec 2024 02:54:33 +0000 (UTC) Received: from TWMBX01.aspeed.com (192.168.0.62) by TWMBX01.aspeed.com (192.168.0.62) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1258.12; Tue, 24 Dec 2024 10:54:30 +0800 Received: from twmbx02.aspeed.com (192.168.10.10) by TWMBX01.aspeed.com (192.168.0.62) with Microsoft SMTP Server id 15.2.1258.12 via Frontend Transport; Tue, 24 Dec 2024 10:54:30 +0800 From: Jammy Huang To: , , , , , , CC: , Subject: [PATCH v2] drm/ast: Support timings, 1280x720/1280x960/1600x900 Date: Tue, 24 Dec 2024 10:54:30 +0800 Message-ID: <20241224025430.3773224-1-jammy_huang@aspeedtech.com> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" In this patch, 3 new timings are added into support list. If you want to have new timings, 1280x720 and 1280x960 on DisplayPort, your dp-fw should be newer than version, 20240502. Signed-off-by: Jammy Huang --- v2: - Fix build errors. --- drivers/gpu/drm/ast/ast_dp.c | 9 ++++++++- drivers/gpu/drm/ast/ast_drv.h | 3 +++ drivers/gpu/drm/ast/ast_mode.c | 14 ++++++++++++++ drivers/gpu/drm/ast/ast_tables.h | 18 ++++++++++++++++++ 4 files changed, 43 insertions(+), 1 deletion(-) base-commit: 4bbf9020becbfd8fc2c3da790855b7042fad455b diff --git a/drivers/gpu/drm/ast/ast_dp.c b/drivers/gpu/drm/ast/ast_dp.c index 0e282b7b167c..0b56f0335871 100644 --- a/drivers/gpu/drm/ast/ast_dp.c +++ b/drivers/gpu/drm/ast/ast_dp.c @@ -255,6 +255,10 @@ static void ast_dp_set_mode(struct drm_crtc *crtc, struct ast_vbios_mode_info *v case 1280: if (crtc->mode.crtc_vdisplay == 800) ModeIdx = (ASTDP_1280x800_60_RB - (u8) ulRefreshRateIndex); + else if (crtc->mode.crtc_vdisplay == 720) + ModeIdx = ASTDP_1280x720_60; + else if (crtc->mode.crtc_vdisplay == 960) + ModeIdx = ASTDP_1280x960_60; else // 1024 ModeIdx = (ASTDP_1280x1024_60 + (u8) ulRefreshRateIndex); break; @@ -267,7 +271,10 @@ static void ast_dp_set_mode(struct drm_crtc *crtc, struct ast_vbios_mode_info *v break; case 1600: if (crtc->mode.crtc_vdisplay == 900) - ModeIdx = (ASTDP_1600x900_60_RB - (u8) ulRefreshRateIndex); + if (ulRefreshRateIndex == 2) + ModeIdx = ASTDP_1600x900_60_DMT; + else + ModeIdx = (ASTDP_1600x900_60_RB - (u8) ulRefreshRateIndex); else //1200 ModeIdx = ASTDP_1600x1200_60; break; diff --git a/drivers/gpu/drm/ast/ast_drv.h b/drivers/gpu/drm/ast/ast_drv.h index 21ce3769bf0d..e7cef334b3ec 100644 --- a/drivers/gpu/drm/ast/ast_drv.h +++ b/drivers/gpu/drm/ast/ast_drv.h @@ -442,6 +442,9 @@ int ast_mode_config_init(struct ast_device *ast); #define ASTDP_1600x900_60_RB 0x1D #define ASTDP_1366x768_60 0x1E #define ASTDP_1152x864_75 0x1F +#define ASTDP_1600x900_60_DMT 0x51 +#define ASTDP_1280x720_60 0x52 +#define ASTDP_1280x960_60 0x53 int ast_mm_init(struct ast_device *ast); diff --git a/drivers/gpu/drm/ast/ast_mode.c b/drivers/gpu/drm/ast/ast_mode.c index 9d5321c81e68..097b462351e2 100644 --- a/drivers/gpu/drm/ast/ast_mode.c +++ b/drivers/gpu/drm/ast/ast_mode.c @@ -147,6 +147,10 @@ static bool ast_get_vbios_mode_info(const struct drm_format_info *format, case 1280: if (mode->crtc_vdisplay == 800) vbios_mode->enh_table = &res_1280x800[refresh_rate_index]; + else if (crtc->mode.crtc_vdisplay == 720) + vbios_mode->enh_table = &res_1280x720[refresh_rate_index]; + else if (crtc->mode.crtc_vdisplay == 960) + vbios_mode->enh_table = &res_1280x960[refresh_rate_index]; else vbios_mode->enh_table = &res_1280x1024[refresh_rate_index]; break; @@ -475,6 +479,12 @@ static void ast_set_dclk_reg(struct ast_device *ast, ast_set_index_reg_mask(ast, AST_IO_VGACRI, 0xbb, 0x0f, (clk_info->param3 & 0xc0) | ((clk_info->param3 & 0x3) << 4)); + + /* Set SEQ; Half dclk for this timing */ + if (vbios_mode->enh_table->flags & HalfDCLK) + ast_set_index_reg_mask(ast, AST_IO_VGASRI, 0x01, 0xff, 0x08); + else + ast_set_index_reg_mask(ast, AST_IO_VGASRI, 0x01, 0xf7, 0x00); } static void ast_set_color_reg(struct ast_device *ast, @@ -1027,8 +1037,12 @@ ast_crtc_helper_mode_valid(struct drm_crtc *crtc, const struct drm_display_mode if (ast->support_wide_screen) { if ((mode->hdisplay == 1680) && (mode->vdisplay == 1050)) return MODE_OK; + if ((mode->hdisplay == 1280) && (mode->vdisplay == 960)) + return MODE_OK; if ((mode->hdisplay == 1280) && (mode->vdisplay == 800)) return MODE_OK; + if ((mode->hdisplay == 1280) && (mode->vdisplay == 720)) + return MODE_OK; if ((mode->hdisplay == 1440) && (mode->vdisplay == 900)) return MODE_OK; if ((mode->hdisplay == 1360) && (mode->vdisplay == 768)) diff --git a/drivers/gpu/drm/ast/ast_tables.h b/drivers/gpu/drm/ast/ast_tables.h index 0378c9bc079b..329d6bac867b 100644 --- a/drivers/gpu/drm/ast/ast_tables.h +++ b/drivers/gpu/drm/ast/ast_tables.h @@ -254,6 +254,13 @@ static const struct ast_vbios_enhtable res_1024x768[] = { (SyncPP | Charx8Dot), 0xFF, 4, 0x31 }, }; +static const struct ast_vbios_enhtable res_1280x960[] = { + {1800, 1280, 96, 112, 1000, 960, 1, 3, VCLK108, /* 60Hz */ + (SyncPP | Charx8Dot), 60, 1, 0x3E }, + {1800, 1280, 96, 112, 1000, 960, 1, 3, VCLK108, /* end */ + (SyncPP | Charx8Dot), 0xFF, 1, 0x3E }, +}; + static const struct ast_vbios_enhtable res_1280x1024[] = { {1688, 1280, 48, 112, 1066, 1024, 1, 3, VCLK108, /* 60Hz */ (SyncPP | Charx8Dot), 60, 1, 0x32 }, @@ -280,6 +287,15 @@ static const struct ast_vbios_enhtable res_1152x864[] = { }; /* 16:9 */ +static const struct ast_vbios_enhtable res_1280x720[] = { + {1650, 1280, 110, 40, 750, 720, 5, 5, VCLK148_5, /* 60Hz */ + (SyncPP | Charx8Dot | LineCompareOff | WideScreenMode | NewModeInfo | + HalfDCLK), 60, 1, 0x3D }, + {1650, 1280, 110, 40, 750, 720, 5, 5, VCLK148_5, /* end */ + (SyncPP | Charx8Dot | LineCompareOff | WideScreenMode | NewModeInfo | + HalfDCLK), 0xFF, 1, 0x3D }, +}; + static const struct ast_vbios_enhtable res_1360x768[] = { {1792, 1360, 64, 112, 795, 768, 3, 6, VCLK85_5, /* 60Hz */ (SyncPP | Charx8Dot | LineCompareOff | WideScreenMode | NewModeInfo), 60, 1, 0x39 }, @@ -294,6 +310,8 @@ static const struct ast_vbios_enhtable res_1600x900[] = { AST2500PreCatchCRT), 60, 1, 0x3A }, {2112, 1600, 88, 168, 934, 900, 3, 5, VCLK118_25, /* 60Hz CVT */ (SyncPN | Charx8Dot | LineCompareOff | WideScreenMode | NewModeInfo), 60, 2, 0x3A }, + {1800, 1600, 24, 80, 1000, 900, 1, 3, VCLK108, /* 60Hz DMT */ + (SyncPP | Charx8Dot | LineCompareOff | WideScreenMode | NewModeInfo), 60, 3, 0x3A }, {2112, 1600, 88, 168, 934, 900, 3, 5, VCLK118_25, /* 60Hz CVT */ (SyncPN | Charx8Dot | LineCompareOff | WideScreenMode | NewModeInfo), 0xFF, 2, 0x3A }, };