All of lore.kernel.org
 help / color / mirror / Atom feed
From: Geert Uytterhoeven <geert+renesas@glider.be>
To: Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
	Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>,
	David Airlie <airlied@gmail.com>, Daniel Vetter <daniel@ffwll.ch>,
	Thomas Zimmermann <tzimmermann@suse.de>,
	Magnus Damm <magnus.damm@gmail.com>
Cc: linux-renesas-soc@vger.kernel.org,
	Geert Uytterhoeven <geert+renesas@glider.be>,
	linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org
Subject: [PATCH 15/39] drm: renesas: shmobile: Improve error handling
Date: Thu, 22 Jun 2023 11:21:27 +0200	[thread overview]
Message-ID: <100c650ab37ae09a142a93afa3734400a6a96757.1687423204.git.geert+renesas@glider.be> (raw)
In-Reply-To: <cover.1687423204.git.geert+renesas@glider.be>

Prepare for DT conversion, where panel probe can be deferred, by
streamlining error propagation and handling:
  - Use dev_err_probe() to avoid printing error messages in case of
    probe deferral,
  - Propagate errors where needed.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 drivers/gpu/drm/renesas/shmobile/shmob_drm_drv.c |  3 ++-
 drivers/gpu/drm/renesas/shmobile/shmob_drm_kms.c | 14 +++++++++++---
 2 files changed, 13 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/renesas/shmobile/shmob_drm_drv.c b/drivers/gpu/drm/renesas/shmobile/shmob_drm_drv.c
index 9aa9800899976a23..50fca18282c5cb5e 100644
--- a/drivers/gpu/drm/renesas/shmobile/shmob_drm_drv.c
+++ b/drivers/gpu/drm/renesas/shmobile/shmob_drm_drv.c
@@ -228,7 +228,8 @@ static int shmob_drm_probe(struct platform_device *pdev)
 
 	ret = shmob_drm_modeset_init(sdev);
 	if (ret < 0) {
-		dev_err(&pdev->dev, "failed to initialize mode setting\n");
+		dev_err_probe(&pdev->dev, ret,
+			      "failed to initialize mode setting\n");
 		goto err_free_drm_dev;
 	}
 
diff --git a/drivers/gpu/drm/renesas/shmobile/shmob_drm_kms.c b/drivers/gpu/drm/renesas/shmobile/shmob_drm_kms.c
index 3051318ddc7999bc..1a62e7f8a8a9e6df 100644
--- a/drivers/gpu/drm/renesas/shmobile/shmob_drm_kms.c
+++ b/drivers/gpu/drm/renesas/shmobile/shmob_drm_kms.c
@@ -157,9 +157,17 @@ int shmob_drm_modeset_init(struct shmob_drm_device *sdev)
 	if (ret)
 		return ret;
 
-	shmob_drm_crtc_create(sdev);
-	shmob_drm_encoder_create(sdev);
-	shmob_drm_connector_create(sdev, &sdev->encoder);
+	ret = shmob_drm_crtc_create(sdev);
+	if (ret < 0)
+		return ret;
+
+	ret = shmob_drm_encoder_create(sdev);
+	if (ret < 0)
+		return ret;
+
+	ret = shmob_drm_connector_create(sdev, &sdev->encoder);
+	if (ret < 0)
+		return ret;
 
 	drm_kms_helper_poll_init(sdev->ddev);
 
-- 
2.34.1


WARNING: multiple messages have this Message-ID (diff)
From: Geert Uytterhoeven <geert+renesas@glider.be>
To: Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
	Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>,
	David Airlie <airlied@gmail.com>, Daniel Vetter <daniel@ffwll.ch>,
	Thomas Zimmermann <tzimmermann@suse.de>,
	Magnus Damm <magnus.damm@gmail.com>
Cc: dri-devel@lists.freedesktop.org,
	linux-renesas-soc@vger.kernel.org, linux-kernel@vger.kernel.org,
	Geert Uytterhoeven <geert+renesas@glider.be>
Subject: [PATCH 15/39] drm: renesas: shmobile: Improve error handling
Date: Thu, 22 Jun 2023 11:21:27 +0200	[thread overview]
Message-ID: <100c650ab37ae09a142a93afa3734400a6a96757.1687423204.git.geert+renesas@glider.be> (raw)
In-Reply-To: <cover.1687423204.git.geert+renesas@glider.be>

Prepare for DT conversion, where panel probe can be deferred, by
streamlining error propagation and handling:
  - Use dev_err_probe() to avoid printing error messages in case of
    probe deferral,
  - Propagate errors where needed.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 drivers/gpu/drm/renesas/shmobile/shmob_drm_drv.c |  3 ++-
 drivers/gpu/drm/renesas/shmobile/shmob_drm_kms.c | 14 +++++++++++---
 2 files changed, 13 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/renesas/shmobile/shmob_drm_drv.c b/drivers/gpu/drm/renesas/shmobile/shmob_drm_drv.c
index 9aa9800899976a23..50fca18282c5cb5e 100644
--- a/drivers/gpu/drm/renesas/shmobile/shmob_drm_drv.c
+++ b/drivers/gpu/drm/renesas/shmobile/shmob_drm_drv.c
@@ -228,7 +228,8 @@ static int shmob_drm_probe(struct platform_device *pdev)
 
 	ret = shmob_drm_modeset_init(sdev);
 	if (ret < 0) {
-		dev_err(&pdev->dev, "failed to initialize mode setting\n");
+		dev_err_probe(&pdev->dev, ret,
+			      "failed to initialize mode setting\n");
 		goto err_free_drm_dev;
 	}
 
diff --git a/drivers/gpu/drm/renesas/shmobile/shmob_drm_kms.c b/drivers/gpu/drm/renesas/shmobile/shmob_drm_kms.c
index 3051318ddc7999bc..1a62e7f8a8a9e6df 100644
--- a/drivers/gpu/drm/renesas/shmobile/shmob_drm_kms.c
+++ b/drivers/gpu/drm/renesas/shmobile/shmob_drm_kms.c
@@ -157,9 +157,17 @@ int shmob_drm_modeset_init(struct shmob_drm_device *sdev)
 	if (ret)
 		return ret;
 
-	shmob_drm_crtc_create(sdev);
-	shmob_drm_encoder_create(sdev);
-	shmob_drm_connector_create(sdev, &sdev->encoder);
+	ret = shmob_drm_crtc_create(sdev);
+	if (ret < 0)
+		return ret;
+
+	ret = shmob_drm_encoder_create(sdev);
+	if (ret < 0)
+		return ret;
+
+	ret = shmob_drm_connector_create(sdev, &sdev->encoder);
+	if (ret < 0)
+		return ret;
 
 	drm_kms_helper_poll_init(sdev->ddev);
 
-- 
2.34.1


  parent reply	other threads:[~2023-06-22  9:23 UTC|newest]

Thread overview: 199+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-06-22  9:21 [PATCH 00/39] drm: renesas: shmobile: Atomic conversion + DT support Geert Uytterhoeven
2023-06-22  9:21 ` Geert Uytterhoeven
2023-06-22  9:21 ` [PATCH 01/39] dt-bindings: display: Add Renesas SH-Mobile LCDC bindings Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-22 14:52   ` Rob Herring
2023-06-22 14:52     ` Rob Herring
2023-07-17 13:57     ` Geert Uytterhoeven
2023-07-17 13:57       ` Geert Uytterhoeven
2023-06-23 14:43   ` Laurent Pinchart
2023-06-23 14:43     ` Laurent Pinchart
2023-06-23 15:19     ` Geert Uytterhoeven
2023-06-23 15:19       ` Geert Uytterhoeven
2023-06-23 15:33       ` Laurent Pinchart
2023-06-23 15:33         ` Laurent Pinchart
2023-06-22  9:21 ` [PATCH 02/39] media: uapi: Add MEDIA_BUS_FMT_RGB666_2X9 variants Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-23 14:56   ` Laurent Pinchart
2023-06-23 14:56     ` Laurent Pinchart
2023-06-22  9:21 ` [PATCH 03/39] drm: renesas: shmobile: Fix overlay plane disable Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-23 14:59   ` Laurent Pinchart
2023-06-23 14:59     ` Laurent Pinchart
2023-06-22  9:21 ` [PATCH 04/39] drm: renesas: shmobile: Fix ARGB32 overlay format typo Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-22 10:11   ` Sergei Shtylyov
2023-06-22 10:11     ` Sergei Shtylyov
2023-06-23 14:58   ` Laurent Pinchart
2023-06-23 14:58     ` Laurent Pinchart
2023-06-22  9:21 ` [PATCH 05/39] drm: renesas: shmobile: Correct encoder/connector types Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-23 15:03   ` Laurent Pinchart
2023-06-23 15:03     ` Laurent Pinchart
2023-06-22  9:21 ` [PATCH 06/39] drm: renesas: shmobile: Add support for Runtime PM Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-23 15:07   ` Laurent Pinchart
2023-06-23 15:07     ` Laurent Pinchart
2023-06-23 15:11     ` Laurent Pinchart
2023-06-23 15:11       ` Laurent Pinchart
2023-06-23 15:22       ` Geert Uytterhoeven
2023-06-23 15:22         ` Geert Uytterhoeven
2023-06-23 15:34         ` Laurent Pinchart
2023-06-23 15:34           ` Laurent Pinchart
2023-06-23 17:41           ` Geert Uytterhoeven
2023-06-23 17:41             ` Geert Uytterhoeven
2023-06-23 17:52             ` Laurent Pinchart
2023-06-23 17:52               ` Laurent Pinchart
2023-06-22  9:21 ` [PATCH 07/39] drm: renesas: shmobile: Restore indentation of shmob_drm_setup_clocks() Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-23 15:04   ` Laurent Pinchart
2023-06-23 15:04     ` Laurent Pinchart
2023-06-22  9:21 ` [PATCH 08/39] drm: renesas: shmobile: Use %p4cc to print fourcc code Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-23 15:04   ` Laurent Pinchart
2023-06-23 15:04     ` Laurent Pinchart
2023-06-22  9:21 ` [PATCH 09/39] drm: renesas: shmobile: Add missing YCbCr formats Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-23 15:13   ` Laurent Pinchart
2023-06-23 15:13     ` Laurent Pinchart
2023-06-22  9:21 ` [PATCH 10/39] drm: renesas: shmobile: Improve shmob_drm_format_info table Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-23 15:30   ` Laurent Pinchart
2023-06-23 15:30     ` Laurent Pinchart
2023-06-22  9:21 ` [PATCH 11/39] drm: renesas: shmobile: Remove backlight support Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-23 15:36   ` Laurent Pinchart
2023-06-23 15:36     ` Laurent Pinchart
2023-06-22  9:21 ` [PATCH 12/39] drm: renesas: shmobile: Don't set display info width and height twice Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-28  1:46   ` [12/39] " Sui Jingfeng
2023-06-22  9:21 ` [PATCH 13/39] drm: renesas: shmobile: Rename input clocks Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-28  1:50   ` [13/39] " Sui Jingfeng
2023-06-22  9:21 ` [PATCH 14/39] drm: renesas: shmobile: Remove support for SYS panels Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-22  9:21 ` Geert Uytterhoeven [this message]
2023-06-22  9:21   ` [PATCH 15/39] drm: renesas: shmobile: Improve error handling Geert Uytterhoeven
2023-06-23 15:41   ` Laurent Pinchart
2023-06-23 15:41     ` Laurent Pinchart
2023-06-22  9:21 ` [PATCH 16/39] drm: renesas: shmobile: Convert to use devm_request_irq() Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-23 15:41   ` Laurent Pinchart
2023-06-23 15:41     ` Laurent Pinchart
2023-06-22  9:21 ` [PATCH 17/39] drm: renesas: shmobile: Use drmm_universal_plane_alloc() Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-23 15:46   ` Laurent Pinchart
2023-06-23 15:46     ` Laurent Pinchart
2023-06-22  9:21 ` [PATCH 18/39] drm: renesas: shmobile: Embed drm_device in shmob_drm_device Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-23 15:48   ` Laurent Pinchart
2023-06-23 15:48     ` Laurent Pinchart
2023-06-22  9:21 ` [PATCH 19/39] drm: renesas: shmobile: Convert container helpers to static inline functions Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-23 15:49   ` Laurent Pinchart
2023-06-23 15:49     ` Laurent Pinchart
2023-06-22  9:21 ` [PATCH 20/39] drm: renesas: shmobile: Replace .dev_private with container_of() Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-23 15:50   ` Laurent Pinchart
2023-06-23 15:50     ` Laurent Pinchart
2023-06-24  9:49   ` [20/39] " Sui Jingfeng
2023-06-22  9:21 ` [PATCH 21/39] drm: renesas: shmobile: Use struct videomode in platform data Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-23 15:53   ` Laurent Pinchart
2023-06-23 15:53     ` Laurent Pinchart
2023-06-22  9:21 ` [PATCH 22/39] drm: renesas: shmobile: Use media bus formats " Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-23 15:54   ` Laurent Pinchart
2023-06-23 15:54     ` Laurent Pinchart
2023-06-22  9:21 ` [PATCH 23/39] drm: renesas: shmobile: Move interface handling to connector setup Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-23 16:39   ` Laurent Pinchart
2023-06-23 16:39     ` Laurent Pinchart
2023-06-23 17:51     ` Geert Uytterhoeven
2023-06-23 17:51       ` Geert Uytterhoeven
2023-06-23 17:53       ` Laurent Pinchart
2023-06-23 17:53         ` Laurent Pinchart
2023-06-22  9:21 ` [PATCH 24/39] drm: renesas: shmobile: Unify plane allocation Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-23 16:50   ` Laurent Pinchart
2023-06-23 16:50     ` Laurent Pinchart
2023-06-23 17:55     ` Geert Uytterhoeven
2023-06-23 17:55       ` Geert Uytterhoeven
2023-06-23 18:50       ` Laurent Pinchart
2023-06-23 18:50         ` Laurent Pinchart
2023-06-25  8:58         ` Geert Uytterhoeven
2023-06-25  8:58           ` Geert Uytterhoeven
2023-06-25 16:56           ` Laurent Pinchart
2023-06-25 16:56             ` Laurent Pinchart
2023-06-22  9:21 ` [PATCH 25/39] drm: renesas: shmobile: Rename shmob_drm_crtc.crtc Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-23 16:51   ` Laurent Pinchart
2023-06-23 16:51     ` Laurent Pinchart
2023-06-27 14:38   ` [25/39] " Sui Jingfeng
2023-06-22  9:21 ` [PATCH 26/39] drm: renesas: shmobile: Rename shmob_drm_connector.connector Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-23 16:51   ` Laurent Pinchart
2023-06-23 16:51     ` Laurent Pinchart
2023-06-27 14:40   ` [26/39] " Sui Jingfeng
2023-06-22  9:21 ` [PATCH 27/39] drm: renesas: shmobile: Rename shmob_drm_plane.plane Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-23 16:52   ` Laurent Pinchart
2023-06-23 16:52     ` Laurent Pinchart
2023-06-27 14:40   ` [27/39] " Sui Jingfeng
2023-06-22  9:21 ` [PATCH 28/39] drm: renesas: shmobile: Use drm_crtc_handle_vblank() Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-23 16:53   ` Laurent Pinchart
2023-06-23 16:53     ` Laurent Pinchart
2023-06-24  9:33   ` [28/39] " Sui Jingfeng
2023-06-25  8:55     ` Geert Uytterhoeven
2023-06-25  8:55       ` Geert Uytterhoeven
2023-06-22  9:21 ` [PATCH 29/39] drm: renesas: shmobile: Move shmob_drm_crtc_finish_page_flip() Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-23 16:53   ` Laurent Pinchart
2023-06-23 16:53     ` Laurent Pinchart
2023-06-22  9:21 ` [PATCH 30/39] drm: renesas: shmobile: Wait for page flip when turning CRTC off Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-23 17:08   ` Laurent Pinchart
2023-06-23 17:08     ` Laurent Pinchart
2023-06-22  9:21 ` [PATCH 31/39] drm: renesas: shmobile: Turn vblank on/off when enabling/disabling CRTC Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-23 17:09   ` Laurent Pinchart
2023-06-23 17:09     ` Laurent Pinchart
2023-06-22  9:21 ` [PATCH 32/39] drm: renesas: shmobile: Shutdown the display on remove Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-23 17:10   ` Laurent Pinchart
2023-06-23 17:10     ` Laurent Pinchart
2023-06-27 14:57   ` [32/39] " Sui Jingfeng
2023-07-05 10:29     ` Geert Uytterhoeven
2023-07-05 10:29       ` Geert Uytterhoeven
2023-07-06  3:51       ` suijingfeng
2023-06-22  9:21 ` [PATCH 33/39] drm: renesas: shmobile: Cleanup encoder Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-23 17:12   ` Laurent Pinchart
2023-06-23 17:12     ` Laurent Pinchart
2023-06-22  9:21 ` [PATCH 34/39] drm: renesas: shmobile: Atomic conversion part 1 Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-22  9:21 ` [PATCH 35/39] drm: renesas: shmobile: Atomic conversion part 2 Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-22  9:21 ` [PATCH 36/39] drm: renesas: shmobile: Use suspend/resume helpers Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-23 17:14   ` Laurent Pinchart
2023-06-23 17:14     ` Laurent Pinchart
2023-06-22  9:21 ` [PATCH 37/39] drm: renesas: shmobile: Remove internal CRTC state tracking Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-23 17:15   ` Laurent Pinchart
2023-06-23 17:15     ` Laurent Pinchart
2023-06-22  9:21 ` [PATCH 38/39] drm: renesas: shmobile: Atomic conversion part 3 Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-23 17:18   ` Laurent Pinchart
2023-06-23 17:18     ` Laurent Pinchart
2023-06-22  9:21 ` [PATCH 39/39] drm: renesas: shmobile: Add DT support Geert Uytterhoeven
2023-06-22  9:21   ` Geert Uytterhoeven
2023-06-23 17:50   ` Laurent Pinchart
2023-06-23 17:50     ` Laurent Pinchart
2023-06-23 17:54     ` Sam Ravnborg
2023-06-23 17:54       ` Sam Ravnborg
2023-06-23 18:09       ` Geert Uytterhoeven
2023-06-23 18:09         ` Geert Uytterhoeven
2023-06-23 18:52         ` Laurent Pinchart
2023-06-23 18:52           ` Laurent Pinchart

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=100c650ab37ae09a142a93afa3734400a6a96757.1687423204.git.geert+renesas@glider.be \
    --to=geert+renesas@glider.be \
    --cc=airlied@gmail.com \
    --cc=daniel@ffwll.ch \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=kieran.bingham+renesas@ideasonboard.com \
    --cc=laurent.pinchart@ideasonboard.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-renesas-soc@vger.kernel.org \
    --cc=magnus.damm@gmail.com \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.