Re: [PATCH 0/8] drm: fix .get_modes() return values

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi

Am 08.03.24 um 17:03 schrieb Jani Nikula:
The drm_connector_helper_funcs .get_modes() hooks are not supposed to
return negative error codes. Some do, but they're not checked, and could
lead to mysterious errors. Fix this all over the place.

Acked-by: Thomas Zimmermann <tzimmermann@xxxxxxx>

for the series.

Do you plan to make the return type an unsigned int eventually?

Best regards
Thomas


BR,
Jani.

Jani Nikula (8):
   drm/probe-helper: warn about negative .get_modes()
   drm/panel: do not return negative error codes from
     drm_panel_get_modes()
   drm/exynos: do not return negative values from .get_modes()
   drm/bridge: lt8912b: do not return negative values from .get_modes()
   drm/imx/ipuv3: do not return negative values from .get_modes()
   drm/vc4: hdmi: do not return negative values from .get_modes()
   drm/bridge: lt9611uxc: use int for holding number of modes
   drm/exynos: simplify the return value handling in
     exynos_dp_get_modes()

  drivers/gpu/drm/bridge/lontium-lt8912b.c     | 16 +++++++---------
  drivers/gpu/drm/bridge/lontium-lt9611uxc.c   |  2 +-
  drivers/gpu/drm/drm_panel.c                  | 17 +++++++++++------
  drivers/gpu/drm/drm_probe_helper.c           |  7 +++++++
  drivers/gpu/drm/exynos/exynos_dp.c           |  7 +++----
  drivers/gpu/drm/exynos/exynos_drm_vidi.c     |  4 ++--
  drivers/gpu/drm/exynos/exynos_hdmi.c         |  4 ++--
  drivers/gpu/drm/imx/ipuv3/parallel-display.c |  4 ++--
  drivers/gpu/drm/vc4/vc4_hdmi.c               |  2 +-
  include/drm/drm_modeset_helper_vtables.h     |  3 ++-
  10 files changed, 38 insertions(+), 28 deletions(-)


--
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstrasse 146, 90461 Nuernberg, Germany
GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman
HRB 36809 (AG Nuernberg)




[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux