Re: [PATCH] drm/mediatek: Set sensible cursor width/height values to fix crash

2024-07-19 Thread AngeloGioacchino Del Regno
Il 19/07/24 10:42, CK Hu (胡俊光) ha scritto: Hi, Angelo: On Thu, 2024-07-18 at 13:23 +0200, AngeloGioacchino Del Regno wrote: Il 18/07/24 13:10, Daniel Stone ha scritto: Hi all, On Thu, 18 Jul 2024 at 11:24, AngeloGioacchino Del Regno wrote: Il 18/07/24 11:27, Fei Shao ha scritto: This match

Re: [PATCH] drm/mediatek: Set sensible cursor width/height values to fix crash

2024-07-19 Thread 胡俊光

Re: [PATCH] drm/mediatek: Set sensible cursor width/height values to fix crash

2024-07-18 Thread AngeloGioacchino Del Regno
Il 18/07/24 13:10, Daniel Stone ha scritto: Hi all, On Thu, 18 Jul 2024 at 11:24, AngeloGioacchino Del Regno wrote: Il 18/07/24 11:27, Fei Shao ha scritto: This matches my preference in [1], so of course I'd like to see it merged... if maintainers are okay with it. Given I've tested the exact

Re: [PATCH] drm/mediatek: Set sensible cursor width/height values to fix crash

2024-07-18 Thread Daniel Stone
Hi all, On Thu, 18 Jul 2024 at 11:24, AngeloGioacchino Del Regno wrote: > Il 18/07/24 11:27, Fei Shao ha scritto: > > This matches my preference in [1], so of course I'd like to see it > > merged... if maintainers are okay with it. > > Given I've tested the exact same change before: > > Reviewed-

Re: [PATCH] drm/mediatek: Set sensible cursor width/height values to fix crash

2024-07-18 Thread AngeloGioacchino Del Regno
Il 18/07/24 11:27, Fei Shao ha scritto: On Thu, Jul 18, 2024 at 4:49 PM Chen-Yu Tsai wrote: (CC-ed Fei Shao) On Thu, Jul 18, 2024 at 4:24 PM AngeloGioacchino Del Regno wrote: Hardware-speaking, there is no feature-reduced cursor specific plane, so this driver reserves the last all Overlay

Re: [PATCH] drm/mediatek: Set sensible cursor width/height values to fix crash

2024-07-18 Thread Fei Shao
On Thu, Jul 18, 2024 at 4:49 PM Chen-Yu Tsai wrote: > > (CC-ed Fei Shao) > > On Thu, Jul 18, 2024 at 4:24 PM AngeloGioacchino Del Regno > wrote: > > > > Hardware-speaking, there is no feature-reduced cursor specific > > plane, so this driver reserves the last all Overlay plane as a > > Cursor pla

Re: [PATCH] drm/mediatek: Set sensible cursor width/height values to fix crash

2024-07-18 Thread 宋孝謙

Re: [PATCH] drm/mediatek: Set sensible cursor width/height values to fix crash

2024-07-18 Thread Chen-Yu Tsai
(CC-ed Fei Shao) On Thu, Jul 18, 2024 at 4:24 PM AngeloGioacchino Del Regno wrote: > > Hardware-speaking, there is no feature-reduced cursor specific > plane, so this driver reserves the last all Overlay plane as a > Cursor plane, but sets the maximum cursor width/height to the > maximum value th

[PATCH] drm/mediatek: Set sensible cursor width/height values to fix crash

2024-07-18 Thread AngeloGioacchino Del Regno
Hardware-speaking, there is no feature-reduced cursor specific plane, so this driver reserves the last all Overlay plane as a Cursor plane, but sets the maximum cursor width/height to the maximum value that the full overlay plane can use. While this could be ok, it raises issues with common usersp