From: Wambui Karuga <wambui.karugax@gmail.com>
To: airlied@linux.ie, daniel@ffwll.ch,
Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
Maxime Ripard <mripard@kernel.org>,
Thomas Zimmermann <tzimmermann@suse.de>
Cc: dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org,
gregkh@linuxfoundation.org
Subject: [PATCH v2 00/17] drm: subsytem-wide debugfs functions refactor
Date: Tue, 10 Mar 2020 16:31:04 +0300 [thread overview]
Message-ID: <20200310133121.27913-1-wambui.karugax@gmail.com> (raw)
This series includes work on various debugfs functions both in drm/core
and across various drivers in the subsystem.
Since commit 987d65d01356 (drm: debugfs: make drm_debugfs_create_files()
never fail), drm_debugfs_create_files() does not fail and only returns
zero. This series therefore removes the left over error handling and
checks for its return value across drm drivers.
As a result of these changes, most drm_debugfs functions are converted
to return void in this series. This also enables the
drm_driver, debugfs_init() hook to be changed to return void.
v2: individual driver patches have been converted to have debugfs
functions return 0 instead of void to prevent breaking individual driver
builds.
The last patch then converts the .debugfs_hook() and its users across
all drivers to return void.
Wambui Karuga (17):
drm/tegra: remove checks for debugfs functions return value
drm/tilcdc: remove check for return value of debugfs functions.
drm/v3d: make v3d_debugfs_init() return 0
drm/vc4: remove check of return value of drm_debugfs functions
drm/arc: make arcgpu_debugfs_init() return 0.
drm/arm: make hdlcd_debugfs_init() return 0
drm/etnaviv: remove check for return value of
drm_debugfs_create_files()
drm/msm: remove checks for return value of drm_debugfs_create_files()
drm/sti: remove use of drm_debugfs functions as return values
drm/vram-helper: make drm_vram_mm_debugfs_init() return 0
drm/nouveau: make nouveau_drm_debugfs_init() return 0
drm/pl111: make pl111_debugfs_init return 0
drm/omap: remove checks for return value of drm_debugfs functions
drm/i915: have *_debugfs_init() functions return void.
drm: make various debugfs_init() functions return 0
drm/debugfs: remove checks for return value of drm_debugfs functions.
drm: convert .debugfs_init() hook to return void.
drivers/gpu/drm/arc/arcpgu_drv.c | 7 +--
drivers/gpu/drm/arm/hdlcd_drv.c | 7 +--
drivers/gpu/drm/arm/malidp_drv.c | 3 +-
drivers/gpu/drm/drm_atomic.c | 8 ++--
drivers/gpu/drm/drm_client.c | 8 ++--
drivers/gpu/drm/drm_crtc_internal.h | 2 +-
drivers/gpu/drm/drm_debugfs.c | 45 +++++--------------
drivers/gpu/drm/drm_framebuffer.c | 8 ++--
drivers/gpu/drm/drm_gem_vram_helper.c | 14 ++----
drivers/gpu/drm/drm_internal.h | 2 +-
drivers/gpu/drm/drm_mipi_dbi.c | 6 +--
drivers/gpu/drm/etnaviv/etnaviv_drv.c | 18 ++------
.../drm/i915/display/intel_display_debugfs.c | 8 ++--
.../drm/i915/display/intel_display_debugfs.h | 4 +-
drivers/gpu/drm/i915/i915_debugfs.c | 8 ++--
drivers/gpu/drm/i915/i915_debugfs.h | 4 +-
drivers/gpu/drm/msm/adreno/a5xx_debugfs.c | 18 +++-----
drivers/gpu/drm/msm/adreno/a5xx_gpu.h | 2 +-
drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c | 14 ++----
drivers/gpu/drm/msm/msm_debugfs.c | 23 +++-------
drivers/gpu/drm/msm/msm_debugfs.h | 2 +-
drivers/gpu/drm/msm/msm_gpu.h | 2 +-
drivers/gpu/drm/nouveau/nouveau_debugfs.c | 26 +++++------
drivers/gpu/drm/nouveau/nouveau_debugfs.h | 8 ++--
drivers/gpu/drm/omapdrm/omap_debugfs.c | 29 +++---------
drivers/gpu/drm/omapdrm/omap_drv.h | 2 +-
drivers/gpu/drm/pl111/pl111_debugfs.c | 8 ++--
drivers/gpu/drm/pl111/pl111_drm.h | 2 +-
drivers/gpu/drm/qxl/qxl_debugfs.c | 21 +++------
drivers/gpu/drm/qxl/qxl_drv.h | 13 +++---
drivers/gpu/drm/qxl/qxl_ttm.c | 6 +--
drivers/gpu/drm/sti/sti_compositor.c | 6 +--
drivers/gpu/drm/sti/sti_compositor.h | 4 +-
drivers/gpu/drm/sti/sti_crtc.c | 2 +-
drivers/gpu/drm/sti/sti_cursor.c | 14 +++---
drivers/gpu/drm/sti/sti_drv.c | 16 ++-----
drivers/gpu/drm/sti/sti_dvo.c | 13 +++---
drivers/gpu/drm/sti/sti_gdp.c | 7 +--
drivers/gpu/drm/sti/sti_hda.c | 13 +++---
drivers/gpu/drm/sti/sti_hdmi.c | 13 +++---
drivers/gpu/drm/sti/sti_hqvdp.c | 12 ++---
drivers/gpu/drm/sti/sti_mixer.c | 10 ++---
drivers/gpu/drm/sti/sti_mixer.h | 2 +-
drivers/gpu/drm/sti/sti_tvout.c | 13 +++---
drivers/gpu/drm/sti/sti_vid.c | 8 ++--
drivers/gpu/drm/sti/sti_vid.h | 2 +-
drivers/gpu/drm/tegra/dc.c | 11 +----
drivers/gpu/drm/tegra/drm.c | 8 ++--
drivers/gpu/drm/tegra/dsi.c | 11 +----
drivers/gpu/drm/tegra/hdmi.c | 11 +----
drivers/gpu/drm/tegra/sor.c | 11 +----
drivers/gpu/drm/tilcdc/tilcdc_drv.c | 17 ++-----
drivers/gpu/drm/v3d/v3d_debugfs.c | 8 ++--
drivers/gpu/drm/v3d/v3d_drv.h | 2 +-
drivers/gpu/drm/vc4/vc4_debugfs.c | 11 ++---
drivers/gpu/drm/vc4/vc4_drv.h | 2 +-
drivers/gpu/drm/virtio/virtgpu_debugfs.c | 3 +-
drivers/gpu/drm/virtio/virtgpu_drv.h | 2 +-
include/drm/drm_client.h | 2 +-
include/drm/drm_debugfs.h | 16 +++----
include/drm/drm_drv.h | 2 +-
include/drm/drm_gem_vram_helper.h | 2 +-
include/drm/drm_mipi_dbi.h | 2 +-
63 files changed, 204 insertions(+), 380 deletions(-)
--
2.25.1
next reply other threads:[~2020-03-10 13:31 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-03-10 13:31 Wambui Karuga [this message]
2020-03-10 13:31 ` [PATCH v2 01/17] drm/tegra: remove checks for debugfs functions return value Wambui Karuga
2020-03-10 13:31 ` [PATCH v2 02/17] drm/tilcdc: remove check for return value of debugfs functions Wambui Karuga
2020-03-10 13:31 ` [PATCH v2 03/17] drm/v3d: make v3d_debugfs_init() return 0 Wambui Karuga
2020-03-10 13:31 ` [PATCH v2 04/17] drm/vc4: remove check of return value of drm_debugfs functions Wambui Karuga
2020-03-10 13:31 ` [PATCH v2 05/17] drm/arc: make arcgpu_debugfs_init() return 0 Wambui Karuga
2020-03-12 7:55 ` Alexey Brodkin
2020-03-10 13:31 ` [PATCH v2 06/17] drm/arm: make hdlcd_debugfs_init() " Wambui Karuga
2020-03-10 15:43 ` Liviu Dudau
2020-03-10 13:31 ` [PATCH v2 07/17] drm/etnaviv: remove check for return value of drm_debugfs_create_files() Wambui Karuga
2020-03-11 16:44 ` Lucas Stach
2020-03-10 13:31 ` [PATCH v2 08/17] drm/msm: remove checks " Wambui Karuga
2020-03-10 13:31 ` [PATCH v2 09/17] drm/sti: remove use of drm_debugfs functions as return values Wambui Karuga
2020-03-10 13:31 ` [PATCH v2 10/17] drm/vram-helper: make drm_vram_mm_debugfs_init() return 0 Wambui Karuga
2020-03-18 15:26 ` Daniel Vetter
2020-03-18 16:02 ` Wambui Karuga
2020-03-18 16:31 ` Daniel Vetter
2020-03-18 16:58 ` Greg KH
2020-03-18 19:10 ` Daniel Vetter
2020-03-19 7:55 ` Greg KH
2020-03-19 10:18 ` Daniel Vetter
2020-03-19 12:27 ` Wambui Karuga
2020-04-07 6:36 ` Thomas Zimmermann
2020-03-10 13:31 ` [PATCH v2 11/17] drm/nouveau: make nouveau_drm_debugfs_init() " Wambui Karuga
2020-03-10 13:31 ` [PATCH v2 12/17] drm/pl111: make pl111_debugfs_init " Wambui Karuga
2020-03-10 13:31 ` [PATCH v2 13/17] drm/omap: remove checks for return value of drm_debugfs functions Wambui Karuga
2020-03-10 13:31 ` [PATCH v2 14/17] drm/i915: have *_debugfs_init() functions return void Wambui Karuga
2020-03-11 8:15 ` Jani Nikula
2020-03-10 13:31 ` [PATCH v2 15/17] drm: make various debugfs_init() functions return 0 Wambui Karuga
2020-03-10 13:31 ` [PATCH v2 16/17] drm/debugfs: remove checks for return value of drm_debugfs functions Wambui Karuga
2020-03-10 13:31 ` [PATCH v2 17/17] drm: convert .debugfs_init() hook to return void Wambui Karuga
2020-03-10 14:33 ` [PATCH v2 00/17] drm: subsytem-wide debugfs functions refactor Greg KH
2020-03-18 16:54 ` Daniel Vetter
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200310133121.27913-1-wambui.karugax@gmail.com \
--to=wambui.karugax@gmail.com \
--cc=airlied@linux.ie \
--cc=daniel@ffwll.ch \
--cc=dri-devel@lists.freedesktop.org \
--cc=gregkh@linuxfoundation.org \
--cc=linux-kernel@vger.kernel.org \
--cc=maarten.lankhorst@linux.intel.com \
--cc=mripard@kernel.org \
--cc=tzimmermann@suse.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®