drm: rcar-du: Clarify error message when encoder initialization fails
authorLaurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Tue, 26 May 2015 11:59:42 +0000 (14:59 +0300)
committerLaurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Tue, 16 Jun 2015 11:25:52 +0000 (14:25 +0300)
A failure to initialize an encoder currently prints an error message in
the kernel log without mentioning which encoder failed to initialize. To
help debugging initialization issues print the encoder DT node name.

This requires moving the error message to the rcar_du_encoders_init_one
function and refactoring it slightly.

Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
drivers/gpu/drm/rcar-du/rcar_du_kms.c

index 4bb5af4bc4740b3b31e73086d6cd52fcb384bf35..ab29c179d854e7d5d420ea92675b09d0f1752d59 100644 (file)
@@ -639,6 +639,11 @@ static int rcar_du_encoders_init_one(struct rcar_du_device *rcdu,
        of_node_put(encoder);
        of_node_put(connector);
 
+       if (ret && ret != -EPROBE_DEFER)
+               dev_warn(rcdu->dev,
+                        "failed to initialize encoder %s (%d), skipping\n",
+                        encoder->full_name, ret);
+
        return ret < 0 ? ret : 1;
 }
 
@@ -688,8 +693,6 @@ static int rcar_du_encoders_init(struct rcar_du_device *rcdu)
                                return ret;
                        }
 
-                       dev_info(rcdu->dev,
-                                "encoder initialization failed, skipping\n");
                        continue;
                }