drm: rcar-du: Convert rcar_du_encoders_init_one() return value to 0/<0
authorLaurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Tue, 26 May 2015 12:07:56 +0000 (15:07 +0300)
committerLaurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Tue, 16 Jun 2015 11:25:53 +0000 (14:25 +0300)
The function returns 1 on success, and either 0 or a negative error code
on failure. As the 0 and negative values don't need to be differentiated
by the caller, convert it to the usual scheme of returning 0 on success
and a negative error code on failure.

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

index ab29c179d854e7d5d420ea92675b09d0f1752d59..fa185a450aeae9408865a19d8508679009c15fee 100644 (file)
@@ -573,7 +573,7 @@ static int rcar_du_encoders_init_one(struct rcar_du_device *rcdu,
        if (!entity) {
                dev_dbg(rcdu->dev, "unconnected endpoint %s, skipping\n",
                        ep->local_node->full_name);
-               return 0;
+               return -ENODEV;
        }
 
        entity_ep_node = of_parse_phandle(ep->local_node, "remote-endpoint", 0);
@@ -596,7 +596,7 @@ static int rcar_du_encoders_init_one(struct rcar_du_device *rcdu,
                                 encoder->full_name);
                        of_node_put(entity_ep_node);
                        of_node_put(encoder);
-                       return 0;
+                       return -ENODEV;
                }
 
                break;
@@ -625,7 +625,7 @@ static int rcar_du_encoders_init_one(struct rcar_du_device *rcdu,
                                 encoder->full_name);
                        of_node_put(encoder);
                        of_node_put(connector);
-                       return 0;
+                       return -EINVAL;
                }
        } else {
                /*
@@ -644,7 +644,7 @@ static int rcar_du_encoders_init_one(struct rcar_du_device *rcdu,
                         "failed to initialize encoder %s (%d), skipping\n",
                         encoder->full_name, ret);
 
-       return ret < 0 ? ret : 1;
+       return ret;
 }
 
 static int rcar_du_encoders_init(struct rcar_du_device *rcdu)
@@ -696,7 +696,7 @@ static int rcar_du_encoders_init(struct rcar_du_device *rcdu)
                        continue;
                }
 
-               num_encoders += ret;
+               num_encoders++;
        }
 
        return num_encoders;