[v3,00/12] drm/udl: More fixes

Message

Takashi Iwai Sept. 8, 2022, 9:51 a.m. UTC

Hi,

this is another respin of patch set for cleaning up and fixes for UDL
driver [*].  It covers the PM problems, regressions in the previous
patch set, fixes for the stalls on some systems, as well as more
hardening.


thanks,

Takashi

[*] v2: https://lore.kernel.org/r/[e-mail]

===

v2->v3:
- More fix on Restore-on-display-mode patch, suggested by Daniel
- Yet more fix for ubs.count check patch, suggested by Thomas
- Another patch for passing rectangle directly, suggested by Thomas
- Put more Acks from Daniel and Thomas

v1->v2: cleanups as suggested by Thomas
- Drop numurbs parameter patch
- Clean up / simplify clipping patch
- Code cleanup and changes for urb management patch
- Put Acks on some given patches

===

Takashi Iwai (10):
  drm/udl: Restore display mode on resume
  Revert "drm/udl: Kill pending URBs at suspend and disconnect"
  drm/udl: Suppress error print for -EPROTO at URB completion
  drm/udl: Increase the default URB list size to 20
  drm/udl: Drop unneeded alignment
  drm/udl: Pass rectangle directly to udl_handle_damage()
  drm/udl: Fix potential URB leaks
  drm/udl: Fix inconsistent urbs.count value during udl_free_urb_list()
  drm/udl: Don't re-initialize stuff at retrying the URB list allocation
  drm/udl: Sync pending URBs at the end of suspend

Thomas Zimmermann (2):
  drm/udl: Add reset_resume
  drm/udl: Enable damage clipping

 drivers/gpu/drm/udl/udl_drv.c      | 19 +++++-
 drivers/gpu/drm/udl/udl_drv.h      | 13 +----
 drivers/gpu/drm/udl/udl_main.c     | 93 +++++++++++++++---------------
 drivers/gpu/drm/udl/udl_modeset.c  | 54 ++++-------------
 drivers/gpu/drm/udl/udl_transfer.c | 45 ++-------------
 5 files changed, 80 insertions(+), 144 deletions(-)

Comments

Thomas Zimmermann Sept. 12, 2022, 7:06 a.m. UTC | #1

Hi,

I've meanwhile merged the patchset, including the one updated patch and 
the missing r-b.

Best regards
Thomas


Am 08.09.22 um 11:51 schrieb Takashi Iwai:
> Hi,
> 
> this is another respin of patch set for cleaning up and fixes for UDL
> driver [*].  It covers the PM problems, regressions in the previous
> patch set, fixes for the stalls on some systems, as well as more
> hardening.
> 
> 
> thanks,
> 
> Takashi
> 
> [*] v2: https://lore.kernel.org/r/[e-mail]
> 
> ===
> 
> v2->v3:
> - More fix on Restore-on-display-mode patch, suggested by Daniel
> - Yet more fix for ubs.count check patch, suggested by Thomas
> - Another patch for passing rectangle directly, suggested by Thomas
> - Put more Acks from Daniel and Thomas
> 
> v1->v2: cleanups as suggested by Thomas
> - Drop numurbs parameter patch
> - Clean up / simplify clipping patch
> - Code cleanup and changes for urb management patch
> - Put Acks on some given patches
> 
> ===
> 
> Takashi Iwai (10):
>    drm/udl: Restore display mode on resume
>    Revert "drm/udl: Kill pending URBs at suspend and disconnect"
>    drm/udl: Suppress error print for -EPROTO at URB completion
>    drm/udl: Increase the default URB list size to 20
>    drm/udl: Drop unneeded alignment
>    drm/udl: Pass rectangle directly to udl_handle_damage()
>    drm/udl: Fix potential URB leaks
>    drm/udl: Fix inconsistent urbs.count value during udl_free_urb_list()
>    drm/udl: Don't re-initialize stuff at retrying the URB list allocation
>    drm/udl: Sync pending URBs at the end of suspend
> 
> Thomas Zimmermann (2):
>    drm/udl: Add reset_resume
>    drm/udl: Enable damage clipping
> 
>   drivers/gpu/drm/udl/udl_drv.c      | 19 +++++-
>   drivers/gpu/drm/udl/udl_drv.h      | 13 +----
>   drivers/gpu/drm/udl/udl_main.c     | 93 +++++++++++++++---------------
>   drivers/gpu/drm/udl/udl_modeset.c  | 54 ++++-------------
>   drivers/gpu/drm/udl/udl_transfer.c | 45 ++-------------
>   5 files changed, 80 insertions(+), 144 deletions(-)
>

Takashi Iwai Sept. 12, 2022, 11:51 a.m. UTC | #2

On Mon, 12 Sep 2022 09:06:47 +0200,
Thomas Zimmermann wrote:
> 
> Hi,
> 
> I've meanwhile merged the patchset, including the one updated patch
> and the missing r-b.

Great, thanks!


Takashi