Re: [PATCH v6 5/6] drm/log: Implement suspend/resume
From: Petr Mladek
Date: Tue Nov 05 2024 - 10:08:58 EST
On Tue 2024-11-05 13:42:25, Jocelyn Falempe wrote:
> Normally the console is already suspended when the graphic driver
> suspend callback is called, but if the parameter no_console_suspend
> is set, it might still be active.
> So call console_stop()/console_start() in the suspend/resume
> callbacks, to make sure it won't try to write to the framebuffer
> while the graphic driver is suspended.
>
> Signed-off-by: Jocelyn Falempe <jfalempe@xxxxxxxxxx>
Just to make it clear that I agree with this approach
for this patchset:
Acked-by: Petr Mladek <pmladek@xxxxxxxx>
Best Regards,
Petr
PS: I am not going to review the rest of the patchset. I believe
that John did a good job.
I actually made a quick look at the 2nd patch and the integration
with the nbcon API looked reasonable. But it was too quick look
so that my ack would not be much valuable there.