RE: [PATCH v3] drm/ast: Support timings, 1280x720/1280x960/1600x900

Hi Thomas, Thanks for the feedback. Let me consider how to achieve your suggestion. Regards, Jammy Huang > > Hi Jamie > > > Am 25.12.24 um 01:32 schrieb Jammy Huang: > > In this patch, 3 new timings are added into support list. > > > > If you want to have n

[PATCH v3] drm/ast: Support timings, 1280x720/1280x960/1600x900

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. v3: - Fix typo. --- drivers/gpu/drm/ast/ast_dp.c | 9

[PATCH v2] drm/ast: Support timings, 1280x720/1280x960/1600x900

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

[PATCH] drm/ast: Support timings, 1280x720/1280x960/1600x900

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 --- drivers/gpu/drm/ast/ast_dp.c | 9 - drivers/gpu/drm/ast/ast_drv.h

RE: [PATCH] drm/ast: Fix black screen after resume

Hi Thomas, Sorry for the miss, and thanks for your help. Regards, Jammy Huang > -Original Message- > From: Thomas Zimmermann > Sent: Thursday, July 18, 2024 4:02 PM > To: Jammy Huang ; jfale...@redhat.com; > maarten.lankho...@linux.intel.com; mrip...@kernel.org; air

[PATCH] drm/ast: Fix black screen after resume

Suspend will disable pcie device. Thus, resume should do full hw initialization again. Add some APIs to ast_drm_thaw() before ast_post_gpu() to fix the issue. Fixes: 5b71707dd13 ("drm/ast: Enable and unlock device access early during init") Signed-off-by: Jammy Huang --- drivers/g

RE: [PATCH v2] drm/ast: Fix soft lockup

Hi Thomas, Thank you. Regards, Jammy Huang > > Hi, > > I've added a Fixes tag and pushed to patch into drm-misc-fixes. > > Best regards > Thomas > > Am 03.04.24 um 11:02 schrieb Jammy Huang: > > There is a while-loop in ast_dp_set_on_off() that co

[PATCH v2] drm/ast: Fix soft lockup

ned-off-by: Jammy Huang Reviewed-by: Jocelyn Falempe --- drivers/gpu/drm/ast/ast_dp.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/gpu/drm/ast/ast_dp.c b/drivers/gpu/drm/ast/ast_dp.c index ebb6d8ebd44e..1e9259416980 100644 --- a/drivers/gpu/drm/ast/ast_dp.c +++ b/drivers/gpu/drm/as

[PATCH] drm/ast: Fix soft lockup

Avoid infinite-loop in ast_dp_set_on_off(). Signed-off-by: Jammy Huang --- drivers/gpu/drm/ast/ast_dp.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/gpu/drm/ast/ast_dp.c b/drivers/gpu/drm/ast/ast_dp.c index ebb6d8ebd44e..1e9259416980 100644 --- a/drivers/gpu/drm/ast/ast_dp.c

Re: [PATCH v5 2/2] drm/ast: report connection status on Display Port.

Hi Jocelyn, On 2023/7/13 下午 04:17, Jocelyn Falempe wrote: On 13/07/2023 09:00, Jammy Huang wrote: Hi Jocelyn, Thanks for your work. On 2023/7/13 下午 02:40, Jocelyn Falempe wrote: Aspeed always report the display port as "connected", because it doesn't set a .detect_ctx callba

Re: [PATCH v5 2/2] drm/ast: report connection status on Display Port.

Hi Jocelyn, Thanks for your work. On 2023/7/13 下午 02:40, Jocelyn Falempe wrote: Aspeed always report the display port as "connected", because it doesn't set a .detect_ctx callback. Fix this by providing the proper detect callback for astdp and dp501. This also fixes the following regression: S

Re: [PATCH v2 00/14] drm/ast: Refactor the device-detection code

Hi, LGTM. Thanks Reviewed-by: Jammy Huang On 2023/6/21 下午 08:53, Thomas Zimmermann wrote: Ast's code for detecting the device type and features is convoluted. It mixes up several state fields, chip types and sub-models. Rework the driver into something more understandable. Patches 1 fi

[PATCH v3] drm/ast: Fix modeset failed on DisplayPort

If we switch display and update cursor together, it could lead to modeset failed because of concurrent access to IO registers. Add lock protection in DP's edid access to avoid this problem. Reviewed-by: Thomas Zimmermann Signed-off-by: Jammy Huang --- v3 changes: - Update label name

[PATCH v3] drm/ast: Fix long time waiting on s3/s4 resume

d-and-tested-by: Wendy Wang Closes: https://bugzilla.kernel.org/show_bug.cgi?id=217278 Acked-by: Thomas Zimmermann Signed-off-by: Jammy Huang --- v3 changes: - Update comments v2 changes: - Fix build error. --- drivers/gpu/drm/ast/ast_dp.c | 55 +++--- driv

Re: [PATCH v2] drm/ast: Fix long time waiting on s3/s4 resume

Hi Thomas, Thanks for your review. On 2023/5/24 下午 06:41, Thomas Zimmermann wrote: Hi, sorry that this took so long. Am 24.05.23 um 04:34 schrieb Jammy Huang: Hi Thomas, Could you help review this patch? This is an issue leading to kernel panic found by Intel. Wendy has confirmed issue

Re: [PATCH] drm/ast: Fix modeset failed on DisplayPort

Hi Thomas, Thanks, I will modify the patch accordingly. On 2023/5/24 下午 06:47, Thomas Zimmermann wrote: Hi, this patch also fell through the cracks. Apologies. Am 24.05.23 um 04:29 schrieb Jammy Huang: Hi Thomas, Do you have other suggestion for this patch?? The main issue was that

[PATCH v2] drm/ast: Fix modeset failed on DisplayPort

If we switch display and update cursor together, it could lead to modeset failed because of concurrent access to IO registers. Add lock protection in DP's edid access to avoid this problem. Signed-off-by: Jammy Huang --- v2 changes: - Fix build error since new struct ast_device is

Re: [PATCH v2] drm/ast: Fix long time waiting on s3/s4 resume

Hi Thomas, Could you help review this patch? This is an issue leading to kernel panic found by Intel. Wendy has confirmed issue resolved by this patch. On 2023/4/14 下午 03:42, Jammy Huang wrote: In resume, DP's launch function, ast_dp_launch, could wait at most 30 seconds before timeo

Re: [PATCH] drm/ast: Fix modeset failed on DisplayPort

Hi Thomas, Do you have other suggestion for this patch?? Please kindly advise. On 2023/4/25 下午 03:39, Jammy Huang wrote: Hi Thomas, I think DP501 is OK. It doesn't use ioregs in ast_dp501_read_edid(). On 2023/4/25 下午 03:27, Thomas Zimmermann wrote: Hi Am 25.04.23 um 09:03 schrieb

Re: [PATCH] drm/ast: Fix modeset failed on DisplayPort

Hi Thomas, I think DP501 is OK. It doesn't use ioregs in ast_dp501_read_edid(). On 2023/4/25 下午 03:27, Thomas Zimmermann wrote: Hi Am 25.04.23 um 09:03 schrieb Jammy Huang: If we switch display and update cursor together, it could lead to modeset failed because of concurrent access

[PATCH] drm/ast: Fix modeset failed on DisplayPort

If we switch display and update cursor together, it could lead to modeset failed because of concurrent access to IO registers. Add lock protection in DP's edid access to avoid this problem. Signed-off-by: Jammy Huang --- drivers/gpu/drm/ast/ast_mode.c | 11 +++ 1 file change

[PATCH v3] drm/ast: Fix ARM compatibility

ARM architecture only has 'memory', so all devices are accessed by MMIO if possible. Signed-off-by: Jammy Huang Reviewed-by: Thomas Zimmermann --- v2 changes: - Use MMIO after AST2500 which enable MMIO by default. v3 changes: - Correct comments --- drivers/gpu/drm/ast/ast_

Re: [PATCH v2] drm/ast: Fix ARM compatibility

Hi Thomas, Thanks for help. I will update a new patch per your comments On 2023/4/20 下午 04:57, Thomas Zimmermann wrote: Hi Am 20.04.23 um 10:09 schrieb Jammy Huang: ARM architecture only has 'memory', so all devices are accessed by MMIO if possible. Signed-off-by: Jammy Huang

[PATCH v2] drm/ast: Fix ARM compatibility

ARM architecture only has 'memory', so all devices are accessed by MMIO if possible. Signed-off-by: Jammy Huang --- v2 changes: - Use MMIO after AST2500 which enable MMIO by default. --- drivers/gpu/drm/ast/ast_main.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-)

Re: [PATCH] drm/ast: Fix ARM compatibility

Jammy Huang: Hi Thomas, The Intel(x86) CPUs have a separate address space for "IO", but the ARM architecture only has "memory", so all IO devices are accessed as if they were memory. Which means ARM does not support isolated IO. Here is a related discussion on

Re: [PATCH] drm/ast: Fix ARM compatibility

https://community.arm.com/support-forums/f/architectures-and-processors-forum/52046/how-to-read-write-an-i-o-port-in-aarch64 Thus, we want to adapt MMIO only after this patch. On 2023/4/17 下午 07:51, Thomas Zimmermann wrote: Hi Am 07.04.23 um 04:09 schrieb Jammy Huang: Hi Thomas, Could you help re

Re: [PATCH 01/11] drm/ast: Use drm_aperture_remove_conflicting_pci_framebuffers

Hi Thomas, The Intel(x86) CPUs have a separate address space for "IO", but the ARM architecture only has "memory", so all IO devices are accessed as if they were memory. Which means ARM does not support isolated IO. Here is a related discussion on ARM's forum. https://community.arm.com/suppo

[PATCH v2] drm/ast: Fix long time waiting on s3/s4 resume

In resume, DP's launch function, ast_dp_launch, could wait at most 30 seconds before timeout to check if DP is enabled. To avoid this problem, we only check if DP enable or not at driver probe. Link: https://bugzilla.kernel.org/show_bug.cgi?id=217278 Signed-off-by: Jammy Huang --- v2 ch

[PATCH] drm/ast: Fix long time waiting on s3/s4 resume

In resume, DP's launch function, ast_dp_launch, could wait at most 30 seconds before timeout to check if DP is enabled. To avoid this problem, we only check if DP enable or not at driver probe. Link: https://bugzilla.kernel.org/show_bug.cgi?id=217278 Signed-off-by: Jammy Huang --- driver

Re: [PATCH] drm/ast: Fix ARM compatibility

Hi Thomas, Could you help review this patch?? We met some problem on nvidia's ARM platfrom and need this patch to fix it. On 2023/3/2 上午 10:19, Jammy Huang wrote: ARM architecture only has 'memory', so all devices are accessed by MMIO. Signed-off-by: Jammy Huang --- driv

[PATCH] drm/ast: Fix ARM compatibility

ARM architecture only has 'memory', so all devices are accessed by MMIO. Signed-off-by: Jammy Huang --- drivers/gpu/drm/ast/ast_main.c | 17 + 1 file changed, 1 insertion(+), 16 deletions(-) diff --git a/drivers/gpu/drm/ast/ast_main.c b/drivers/gpu/drm/ast/ast_ma

Re: [PATCH v3] drm/ast: Fix start address computation

* v3: move the change to plane_init() and also fix the cursor plane.     (Jammy Huang) Tested on a sr645 affected by this bug. Fixes: f2fa5a99ca81 ("drm/ast: Convert ast to SHMEM") Signed-off-by: Jocelyn Falempe Reviewed-by: Thomas Zimmermann Reviewed-by: Jammy Huang

Re: [PATCH v2] drm/ast: Fix start address computation

Hello, The offset given to ast_set_start_address_crt1() should be offset in vram. It should 0 now as your patch. I think it is better to correct it in ast_primary_plane_init() and ast_cursor_plane_init() as below. --- a/drivers/gpu/drm/ast/ast_mode.c +++ b/drivers/gpu/drm/ast/ast_mode.c @@

Re: [PATCH] drm/ast: Fix no display at WayLand after power-off

Hi Thomas, OK. Thanks for the reminder. On 2022/12/9 下午 05:45, Thomas Zimmermann wrote: Hi Am 09.12.22 um 10:11 schrieb Jammy Huang: With WayLand, there is error log when display waken up from power-off: gnome-shell: Failed to post KMS update: CRTC property (GAMMA_LUT) not found gnome

[PATCH] drm/ast: Fix no display at WayLand after power-off

With WayLand, there is error log when display waken up from power-off: gnome-shell: Failed to post KMS update: CRTC property (GAMMA_LUT) not found gnome-shell: Page flip discarded: CRTC property (GAMMA_LUT) not found To fix the issue, enable GAMMA_LUT property on CRTC. Signed-off-by: Jammy Huang

[PATCH] drm/hisilicon/kirin: Fix typo in kirin_drm_drv.c

'config_max_height' should be used for 'max_height'. Signed-off-by: Jammy Huang --- drivers/gpu/drm/hisilicon/kirin/kirin_drm_drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/hisilicon/kirin/kirin_drm_drv.c b/drivers/gpu

Re: [PATCH] drm/ast: Fix video broken in some resolutions on ast2600

Hi Thomas, On 2022/9/20 下午 04:43, Thomas Zimmermann wrote: Hi Am 16.09.22 um 11:17 schrieb Jammy Huang: Some cases are not handled well for ast2600. Signed-off-by: Jammy Huang ---   drivers/gpu/drm/ast/ast_mode.c | 10 --   1 file changed, 8 insertions(+), 2 deletions(-) diff --git

[PATCH] drm/ast: Fix video broken in some resolutions on ast2600

Some cases are not handled well for ast2600. Signed-off-by: Jammy Huang --- drivers/gpu/drm/ast/ast_mode.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/ast/ast_mode.c b/drivers/gpu/drm/ast/ast_mode.c index d327dcbaf032..be9bbc888ac3 100644 --- a

[PATCH v2] drm/ast: Add resolution support for 1152x864@75

Add 1152x864 into support list. Signed-off-by: Jammy Huang --- v2 changes: - add check in mode_valid --- drivers/gpu/drm/ast/ast_mode.c | 9 + drivers/gpu/drm/ast/ast_tables.h | 7 +++ 2 files changed, 16 insertions(+) diff --git a/drivers/gpu/drm/ast/ast_mode.c b/drivers/gpu

[PATCH] drm/ast: Add resolution support for 1152x864@75

Add 1152x864 into support list. Signed-off-by: Jammy Huang --- drivers/gpu/drm/ast/ast_mode.c | 3 +++ drivers/gpu/drm/ast/ast_tables.h | 7 +++ 2 files changed, 10 insertions(+) diff --git a/drivers/gpu/drm/ast/ast_mode.c b/drivers/gpu/drm/ast/ast_mode.c index 214b10178454..0b2c5fe8dc2c

[PATCH] drm/hisilicon/kirin: Fix typo in kirin_drm_drv.c

'config_max_height' should be used for 'max_height'. Signed-off-by: Jammy Huang --- drivers/gpu/drm/hisilicon/kirin/kirin_drm_drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/hisilicon/kirin/kirin_drm_drv.c b/drivers/gpu