drm/bridge: lt9611uxc: Don't log an error when DSI host can't be found
authorNícolas F. R. A. Prado <nfraprado@collabora.com>
Mon, 15 Apr 2024 21:49:33 +0000 (17:49 -0400)
committerRobert Foss <rfoss@kernel.org>
Tue, 16 Apr 2024 13:18:59 +0000 (15:18 +0200)
Given that failing to find a DSI host causes the driver to defer probe,
make use of dev_err_probe() to log the reason. This makes the defer
probe reason available and avoids alerting userspace about something
that is not necessarily an error.

Fixes: 0cbbd5b1a012 ("drm: bridge: add support for lontium LT9611UXC bridge")
Suggested-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Signed-off-by: Nícolas F. R. A. Prado <nfraprado@collabora.com>
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Signed-off-by: Robert Foss <rfoss@kernel.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20240415-anx7625-defer-log-no-dsi-host-v3-5-619a28148e5c@collabora.com
drivers/gpu/drm/bridge/lontium-lt9611uxc.c

index f4f593ad8f79574604401eeb2bd819e5a3acbc7f..ab702471f3ab1a053aecd1905fd4acbc7a840b3f 100644 (file)
@@ -266,10 +266,8 @@ static struct mipi_dsi_device *lt9611uxc_attach_dsi(struct lt9611uxc *lt9611uxc,
        int ret;
 
        host = of_find_mipi_dsi_host_by_node(dsi_node);
-       if (!host) {
-               dev_err(dev, "failed to find dsi host\n");
-               return ERR_PTR(-EPROBE_DEFER);
-       }
+       if (!host)
+               return ERR_PTR(dev_err_probe(dev, -EPROBE_DEFER, "failed to find dsi host\n"));
 
        dsi = devm_mipi_dsi_device_register_full(dev, host, &info);
        if (IS_ERR(dsi)) {