mbox series

[0/6,SRU,K,OEM-5.17] Screen cannot turn on after screen off with Matrox G200eW3 [102b:0536]

Message ID 20221108042634.24176-1-acelan.kao@canonical.com
Headers show
Series Screen cannot turn on after screen off with Matrox G200eW3 [102b:0536] | expand

Message

AceLan Kao Nov. 8, 2022, 4:26 a.m. UTC
From: "Chia-Lin Kao (AceLan)" <acelan.kao@canonical.com>

BugLink: https://launchpad.net/bugs/1995573

[Impact]
Screen cannot turn on after screen is off

[Fix]
Below commit from v6.0-rc1 fixes the issue
c577b2f43e80 drm/mgag200: Enable atomic gamma lut update

And applied some other commits to fix the conflicts
c48a36301634 drm/mgag200: Optimize damage clips
3064debaf55e drm/mgag200: Add FB_DAMAGE_CLIPS support
5913ab941d6e drm/mgag200: Acquire I/O lock while reading EDID
931e3f3a0e99 drm/mgag200: Protect concurrent access to I/O registers with lock
e13f13e039dc drm: Add DRM-managed mutex_init()

[Tests]
Verified on the machine with Matrox G200eW3 [102b:0536] graphics card.

[Where problems could occur]
The commits to solve the conflicts are trivial, and not likely to introduce
regressions. The fix patch change a lot of code and hard to evaluate the
risk, but check the latest linus/master tree and linux-next/master tree,
there is no fixed commit for all these cherry-picked commits, so should be
safe to include them.

Jocelyn Falempe (3):
  drm/mgag200: Optimize damage clips
  drm/mgag200: Add FB_DAMAGE_CLIPS support
  drm/mgag200: Enable atomic gamma lut update

Thomas Zimmermann (3):
  drm: Add DRM-managed mutex_init()
  drm/mgag200: Protect concurrent access to I/O registers with lock
  drm/mgag200: Acquire I/O lock while reading EDID

 drivers/gpu/drm/drm_managed.c          |  27 ++++
 drivers/gpu/drm/mgag200/mgag200_drv.c  |   6 +
 drivers/gpu/drm/mgag200/mgag200_drv.h  |   1 +
 drivers/gpu/drm/mgag200/mgag200_mode.c | 169 ++++++++++++++++++-------
 include/drm/drm_managed.h              |   3 +
 5 files changed, 157 insertions(+), 49 deletions(-)

Comments

Tim Gardner Nov. 8, 2022, 4:24 p.m. UTC | #1
On 11/7/22 9:26 PM, AceLan Kao wrote:
> From: "Chia-Lin Kao (AceLan)" <acelan.kao@canonical.com>
> 
> BugLink: https://launchpad.net/bugs/1995573
> 
> [Impact]
> Screen cannot turn on after screen is off
> 
> [Fix]
> Below commit from v6.0-rc1 fixes the issue
> c577b2f43e80 drm/mgag200: Enable atomic gamma lut update
> 
> And applied some other commits to fix the conflicts
> c48a36301634 drm/mgag200: Optimize damage clips
> 3064debaf55e drm/mgag200: Add FB_DAMAGE_CLIPS support
> 5913ab941d6e drm/mgag200: Acquire I/O lock while reading EDID
> 931e3f3a0e99 drm/mgag200: Protect concurrent access to I/O registers with lock
> e13f13e039dc drm: Add DRM-managed mutex_init()
> 
> [Tests]
> Verified on the machine with Matrox G200eW3 [102b:0536] graphics card.
> 
> [Where problems could occur]
> The commits to solve the conflicts are trivial, and not likely to introduce
> regressions. The fix patch change a lot of code and hard to evaluate the
> risk, but check the latest linus/master tree and linux-next/master tree,
> there is no fixed commit for all these cherry-picked commits, so should be
> safe to include them.
> 
> Jocelyn Falempe (3):
>    drm/mgag200: Optimize damage clips
>    drm/mgag200: Add FB_DAMAGE_CLIPS support
>    drm/mgag200: Enable atomic gamma lut update
> 
> Thomas Zimmermann (3):
>    drm: Add DRM-managed mutex_init()
>    drm/mgag200: Protect concurrent access to I/O registers with lock
>    drm/mgag200: Acquire I/O lock while reading EDID
> 
>   drivers/gpu/drm/drm_managed.c          |  27 ++++
>   drivers/gpu/drm/mgag200/mgag200_drv.c  |   6 +
>   drivers/gpu/drm/mgag200/mgag200_drv.h  |   1 +
>   drivers/gpu/drm/mgag200/mgag200_mode.c | 169 ++++++++++++++++++-------
>   include/drm/drm_managed.h              |   3 +
>   5 files changed, 157 insertions(+), 49 deletions(-)
> 
Acked-by: Tim Gardner <tim.gardner@canonical.com>
Timo Aaltonen Nov. 10, 2022, 4:19 p.m. UTC | #2
AceLan Kao kirjoitti 8.11.2022 klo 6.26:
> From: "Chia-Lin Kao (AceLan)" <acelan.kao@canonical.com>
> 
> BugLink: https://launchpad.net/bugs/1995573
> 
> [Impact]
> Screen cannot turn on after screen is off
> 
> [Fix]
> Below commit from v6.0-rc1 fixes the issue
> c577b2f43e80 drm/mgag200: Enable atomic gamma lut update
> 
> And applied some other commits to fix the conflicts
> c48a36301634 drm/mgag200: Optimize damage clips
> 3064debaf55e drm/mgag200: Add FB_DAMAGE_CLIPS support
> 5913ab941d6e drm/mgag200: Acquire I/O lock while reading EDID
> 931e3f3a0e99 drm/mgag200: Protect concurrent access to I/O registers with lock
> e13f13e039dc drm: Add DRM-managed mutex_init()
> 
> [Tests]
> Verified on the machine with Matrox G200eW3 [102b:0536] graphics card.
> 
> [Where problems could occur]
> The commits to solve the conflicts are trivial, and not likely to introduce
> regressions. The fix patch change a lot of code and hard to evaluate the
> risk, but check the latest linus/master tree and linux-next/master tree,
> there is no fixed commit for all these cherry-picked commits, so should be
> safe to include them.
> 
> Jocelyn Falempe (3):
>    drm/mgag200: Optimize damage clips
>    drm/mgag200: Add FB_DAMAGE_CLIPS support
>    drm/mgag200: Enable atomic gamma lut update
> 
> Thomas Zimmermann (3):
>    drm: Add DRM-managed mutex_init()
>    drm/mgag200: Protect concurrent access to I/O registers with lock
>    drm/mgag200: Acquire I/O lock while reading EDID
> 
>   drivers/gpu/drm/drm_managed.c          |  27 ++++
>   drivers/gpu/drm/mgag200/mgag200_drv.c  |   6 +
>   drivers/gpu/drm/mgag200/mgag200_drv.h  |   1 +
>   drivers/gpu/drm/mgag200/mgag200_mode.c | 169 ++++++++++++++++++-------
>   include/drm/drm_managed.h              |   3 +
>   5 files changed, 157 insertions(+), 49 deletions(-)
> 

applied to oem-5.17, thanks
Kleber Souza Nov. 11, 2022, 3:15 p.m. UTC | #3
On 08.11.22 05:26, AceLan Kao wrote:
> From: "Chia-Lin Kao (AceLan)" <acelan.kao@canonical.com>
> 
> BugLink: https://launchpad.net/bugs/1995573
> 
> [Impact]
> Screen cannot turn on after screen is off
> 
> [Fix]
> Below commit from v6.0-rc1 fixes the issue
> c577b2f43e80 drm/mgag200: Enable atomic gamma lut update
> 
> And applied some other commits to fix the conflicts
> c48a36301634 drm/mgag200: Optimize damage clips
> 3064debaf55e drm/mgag200: Add FB_DAMAGE_CLIPS support
> 5913ab941d6e drm/mgag200: Acquire I/O lock while reading EDID
> 931e3f3a0e99 drm/mgag200: Protect concurrent access to I/O registers with lock
> e13f13e039dc drm: Add DRM-managed mutex_init()
> 
> [Tests]
> Verified on the machine with Matrox G200eW3 [102b:0536] graphics card.
> 
> [Where problems could occur]
> The commits to solve the conflicts are trivial, and not likely to introduce
> regressions. The fix patch change a lot of code and hard to evaluate the
> risk, but check the latest linus/master tree and linux-next/master tree,
> there is no fixed commit for all these cherry-picked commits, so should be
> safe to include them.
> 
> Jocelyn Falempe (3):
>    drm/mgag200: Optimize damage clips
>    drm/mgag200: Add FB_DAMAGE_CLIPS support
>    drm/mgag200: Enable atomic gamma lut update
> 
> Thomas Zimmermann (3):
>    drm: Add DRM-managed mutex_init()
>    drm/mgag200: Protect concurrent access to I/O registers with lock
>    drm/mgag200: Acquire I/O lock while reading EDID
> 
>   drivers/gpu/drm/drm_managed.c          |  27 ++++
>   drivers/gpu/drm/mgag200/mgag200_drv.c  |   6 +
>   drivers/gpu/drm/mgag200/mgag200_drv.h  |   1 +
>   drivers/gpu/drm/mgag200/mgag200_mode.c | 169 ++++++++++++++++++-------
>   include/drm/drm_managed.h              |   3 +
>   5 files changed, 157 insertions(+), 49 deletions(-)
> 

For the Kinetic patchset:

Acked-by: Kleber Sacilotto de Souza <kleber.souza@canonical.com>

Thanks
Kleber Souza Nov. 11, 2022, 3:37 p.m. UTC | #4
On 08.11.22 05:26, AceLan Kao wrote:
> From: "Chia-Lin Kao (AceLan)" <acelan.kao@canonical.com>
> 
> BugLink: https://launchpad.net/bugs/1995573
> 
> [Impact]
> Screen cannot turn on after screen is off
> 
> [Fix]
> Below commit from v6.0-rc1 fixes the issue
> c577b2f43e80 drm/mgag200: Enable atomic gamma lut update
> 
> And applied some other commits to fix the conflicts
> c48a36301634 drm/mgag200: Optimize damage clips
> 3064debaf55e drm/mgag200: Add FB_DAMAGE_CLIPS support
> 5913ab941d6e drm/mgag200: Acquire I/O lock while reading EDID
> 931e3f3a0e99 drm/mgag200: Protect concurrent access to I/O registers with lock
> e13f13e039dc drm: Add DRM-managed mutex_init()
> 
> [Tests]
> Verified on the machine with Matrox G200eW3 [102b:0536] graphics card.
> 
> [Where problems could occur]
> The commits to solve the conflicts are trivial, and not likely to introduce
> regressions. The fix patch change a lot of code and hard to evaluate the
> risk, but check the latest linus/master tree and linux-next/master tree,
> there is no fixed commit for all these cherry-picked commits, so should be
> safe to include them.
> 
> Jocelyn Falempe (3):
>    drm/mgag200: Optimize damage clips
>    drm/mgag200: Add FB_DAMAGE_CLIPS support
>    drm/mgag200: Enable atomic gamma lut update
> 
> Thomas Zimmermann (3):
>    drm: Add DRM-managed mutex_init()
>    drm/mgag200: Protect concurrent access to I/O registers with lock
>    drm/mgag200: Acquire I/O lock while reading EDID
> 
>   drivers/gpu/drm/drm_managed.c          |  27 ++++
>   drivers/gpu/drm/mgag200/mgag200_drv.c  |   6 +
>   drivers/gpu/drm/mgag200/mgag200_drv.h  |   1 +
>   drivers/gpu/drm/mgag200/mgag200_mode.c | 169 ++++++++++++++++++-------
>   include/drm/drm_managed.h              |   3 +
>   5 files changed, 157 insertions(+), 49 deletions(-)
> 

Applied to kinetic:linux.

Thanks,
Kleber