Merge branch 'akpm' (patches from Andrew)
[cascardo/linux.git] / drivers / media / platform / rcar-vin / rcar-core.c
index 4b2007b..098a0b1 100644 (file)
 
 #define notifier_to_vin(n) container_of(n, struct rvin_dev, notifier)
 
-static int rvin_mbus_supported(struct rvin_dev *vin)
+static bool rvin_mbus_supported(struct rvin_graph_entity *entity)
 {
-       struct v4l2_subdev *sd;
+       struct v4l2_subdev *sd = entity->subdev;
        struct v4l2_subdev_mbus_code_enum code = {
                .which = V4L2_SUBDEV_FORMAT_ACTIVE,
        };
 
-       sd = vin_to_source(vin);
-
        code.index = 0;
        while (!v4l2_subdev_call(sd, pad, enum_mbus_code, NULL, &code)) {
                code.index++;
                switch (code.code) {
                case MEDIA_BUS_FMT_YUYV8_1X16:
-               case MEDIA_BUS_FMT_YUYV8_2X8:
-               case MEDIA_BUS_FMT_YUYV10_2X10:
+               case MEDIA_BUS_FMT_UYVY8_2X8:
+               case MEDIA_BUS_FMT_UYVY10_2X10:
                case MEDIA_BUS_FMT_RGB888_1X24:
-                       vin->source.code = code.code;
-                       vin_dbg(vin, "Found supported media bus format: %d\n",
-                               vin->source.code);
+                       entity->code = code.code;
                        return true;
                default:
                        break;
@@ -60,142 +56,168 @@ static int rvin_mbus_supported(struct rvin_dev *vin)
        return false;
 }
 
-static int rvin_graph_notify_complete(struct v4l2_async_notifier *notifier)
+static int rvin_digital_notify_complete(struct v4l2_async_notifier *notifier)
 {
        struct rvin_dev *vin = notifier_to_vin(notifier);
        int ret;
 
+       /* Verify subdevices mbus format */
+       if (!rvin_mbus_supported(&vin->digital)) {
+               vin_err(vin, "Unsupported media bus format for %s\n",
+                       vin->digital.subdev->name);
+               return -EINVAL;
+       }
+
+       vin_dbg(vin, "Found media bus format for %s: %d\n",
+               vin->digital.subdev->name, vin->digital.code);
+
        ret = v4l2_device_register_subdev_nodes(&vin->v4l2_dev);
        if (ret < 0) {
                vin_err(vin, "Failed to register subdev nodes\n");
                return ret;
        }
 
-       if (!rvin_mbus_supported(vin)) {
-               vin_err(vin, "No supported mediabus format found\n");
-               return -EINVAL;
+       return rvin_v4l2_probe(vin);
+}
+
+static void rvin_digital_notify_unbind(struct v4l2_async_notifier *notifier,
+                                      struct v4l2_subdev *subdev,
+                                      struct v4l2_async_subdev *asd)
+{
+       struct rvin_dev *vin = notifier_to_vin(notifier);
+
+       if (vin->digital.subdev == subdev) {
+               vin_dbg(vin, "unbind digital subdev %s\n", subdev->name);
+               rvin_v4l2_remove(vin);
+               vin->digital.subdev = NULL;
+               return;
        }
 
-       return rvin_v4l2_probe(vin);
+       vin_err(vin, "no entity for subdev %s to unbind\n", subdev->name);
 }
 
-static void rvin_graph_notify_unbind(struct v4l2_async_notifier *notifier,
-                                    struct v4l2_subdev *sd,
+static int rvin_digital_notify_bound(struct v4l2_async_notifier *notifier,
+                                    struct v4l2_subdev *subdev,
                                     struct v4l2_async_subdev *asd)
 {
        struct rvin_dev *vin = notifier_to_vin(notifier);
 
-       rvin_v4l2_remove(vin);
+       v4l2_set_subdev_hostdata(subdev, vin);
+
+       if (vin->digital.asd.match.of.node == subdev->dev->of_node) {
+               vin_dbg(vin, "bound digital subdev %s\n", subdev->name);
+               vin->digital.subdev = subdev;
+               return 0;
+       }
+
+       vin_err(vin, "no entity for subdev %s to bind\n", subdev->name);
+       return -EINVAL;
 }
 
-static int rvin_graph_notify_bound(struct v4l2_async_notifier *notifier,
-                                  struct v4l2_subdev *subdev,
-                                  struct v4l2_async_subdev *asd)
+static int rvin_digitial_parse_v4l2(struct rvin_dev *vin,
+                                   struct device_node *ep,
+                                   struct v4l2_mbus_config *mbus_cfg)
 {
-       struct rvin_dev *vin = notifier_to_vin(notifier);
+       struct v4l2_of_endpoint v4l2_ep;
+       int ret;
 
-       vin_dbg(vin, "subdev %s bound\n", subdev->name);
+       ret = v4l2_of_parse_endpoint(ep, &v4l2_ep);
+       if (ret) {
+               vin_err(vin, "Could not parse v4l2 endpoint\n");
+               return -EINVAL;
+       }
 
-       vin->entity.entity = &subdev->entity;
-       vin->entity.subdev = subdev;
+       mbus_cfg->type = v4l2_ep.bus_type;
+
+       switch (mbus_cfg->type) {
+       case V4L2_MBUS_PARALLEL:
+               vin_dbg(vin, "Found PARALLEL media bus\n");
+               mbus_cfg->flags = v4l2_ep.bus.parallel.flags;
+               break;
+       case V4L2_MBUS_BT656:
+               vin_dbg(vin, "Found BT656 media bus\n");
+               mbus_cfg->flags = 0;
+               break;
+       default:
+               vin_err(vin, "Unknown media bus type\n");
+               return -EINVAL;
+       }
 
        return 0;
 }
 
-static int rvin_graph_parse(struct rvin_dev *vin,
-                           struct device_node *node)
+static int rvin_digital_graph_parse(struct rvin_dev *vin)
 {
-       struct device_node *remote;
-       struct device_node *ep = NULL;
-       struct device_node *next;
-       int ret = 0;
-
-       while (1) {
-               next = of_graph_get_next_endpoint(node, ep);
-               if (!next)
-                       break;
-
-               of_node_put(ep);
-               ep = next;
+       struct device_node *ep, *np;
+       int ret;
 
-               remote = of_graph_get_remote_port_parent(ep);
-               if (!remote) {
-                       ret = -EINVAL;
-                       break;
-               }
+       vin->digital.asd.match.of.node = NULL;
+       vin->digital.subdev = NULL;
 
-               /* Skip entities that we have already processed. */
-               if (remote == vin->dev->of_node) {
-                       of_node_put(remote);
-                       continue;
-               }
+       /*
+        * Port 0 id 0 is local digital input, try to get it.
+        * Not all instances can or will have this, that is OK
+        */
+       ep = of_graph_get_endpoint_by_regs(vin->dev->of_node, 0, 0);
+       if (!ep)
+               return 0;
 
-               /* Remote node to connect */
-               if (!vin->entity.node) {
-                       vin->entity.node = remote;
-                       vin->entity.asd.match_type = V4L2_ASYNC_MATCH_OF;
-                       vin->entity.asd.match.of.node = remote;
-                       ret++;
-               }
+       np = of_graph_get_remote_port_parent(ep);
+       if (!np) {
+               vin_err(vin, "No remote parent for digital input\n");
+               of_node_put(ep);
+               return -EINVAL;
        }
+       of_node_put(np);
 
+       ret = rvin_digitial_parse_v4l2(vin, ep, &vin->digital.mbus_cfg);
        of_node_put(ep);
+       if (ret)
+               return ret;
 
-       return ret;
+       vin->digital.asd.match.of.node = np;
+       vin->digital.asd.match_type = V4L2_ASYNC_MATCH_OF;
+
+       return 0;
 }
 
-static int rvin_graph_init(struct rvin_dev *vin)
+static int rvin_digital_graph_init(struct rvin_dev *vin)
 {
        struct v4l2_async_subdev **subdevs = NULL;
        int ret;
 
-       /* Parse the graph to extract a list of subdevice DT nodes. */
-       ret = rvin_graph_parse(vin, vin->dev->of_node);
-       if (ret < 0) {
-               vin_err(vin, "Graph parsing failed\n");
-               goto done;
-       }
-
-       if (!ret) {
-               vin_err(vin, "No subdev found in graph\n");
-               goto done;
-       }
+       ret = rvin_digital_graph_parse(vin);
+       if (ret)
+               return ret;
 
-       if (ret != 1) {
-               vin_err(vin, "More then one subdev found in graph\n");
-               goto done;
+       if (!vin->digital.asd.match.of.node) {
+               vin_dbg(vin, "No digital subdevice found\n");
+               return -ENODEV;
        }
 
        /* Register the subdevices notifier. */
        subdevs = devm_kzalloc(vin->dev, sizeof(*subdevs), GFP_KERNEL);
-       if (subdevs == NULL) {
-               ret = -ENOMEM;
-               goto done;
-       }
+       if (subdevs == NULL)
+               return -ENOMEM;
 
-       subdevs[0] = &vin->entity.asd;
+       subdevs[0] = &vin->digital.asd;
+
+       vin_dbg(vin, "Found digital subdevice %s\n",
+               of_node_full_name(subdevs[0]->match.of.node));
 
-       vin->notifier.subdevs = subdevs;
        vin->notifier.num_subdevs = 1;
-       vin->notifier.bound = rvin_graph_notify_bound;
-       vin->notifier.unbind = rvin_graph_notify_unbind;
-       vin->notifier.complete = rvin_graph_notify_complete;
+       vin->notifier.subdevs = subdevs;
+       vin->notifier.bound = rvin_digital_notify_bound;
+       vin->notifier.unbind = rvin_digital_notify_unbind;
+       vin->notifier.complete = rvin_digital_notify_complete;
 
        ret = v4l2_async_notifier_register(&vin->v4l2_dev, &vin->notifier);
        if (ret < 0) {
                vin_err(vin, "Notifier registration failed\n");
-               goto done;
-       }
-
-       ret = 0;
-
-done:
-       if (ret < 0) {
-               v4l2_async_notifier_unregister(&vin->notifier);
-               of_node_put(vin->entity.node);
+               return ret;
        }
 
-       return ret;
+       return 0;
 }
 
 /* -----------------------------------------------------------------------------
@@ -209,56 +231,14 @@ static const struct of_device_id rvin_of_id_table[] = {
        { .compatible = "renesas,vin-r8a7790", .data = (void *)RCAR_GEN2 },
        { .compatible = "renesas,vin-r8a7779", .data = (void *)RCAR_H1 },
        { .compatible = "renesas,vin-r8a7778", .data = (void *)RCAR_M1 },
+       { .compatible = "renesas,rcar-gen2-vin", .data = (void *)RCAR_GEN2 },
        { },
 };
 MODULE_DEVICE_TABLE(of, rvin_of_id_table);
 
-static int rvin_parse_dt(struct rvin_dev *vin)
-{
-       const struct of_device_id *match;
-       struct v4l2_of_endpoint ep;
-       struct device_node *np;
-       int ret;
-
-       match = of_match_device(of_match_ptr(rvin_of_id_table), vin->dev);
-       if (!match)
-               return -ENODEV;
-
-       vin->chip = (enum chip_id)match->data;
-
-       np = of_graph_get_next_endpoint(vin->dev->of_node, NULL);
-       if (!np) {
-               vin_err(vin, "Could not find endpoint\n");
-               return -EINVAL;
-       }
-
-       ret = v4l2_of_parse_endpoint(np, &ep);
-       if (ret) {
-               vin_err(vin, "Could not parse endpoint\n");
-               return ret;
-       }
-
-       of_node_put(np);
-
-       vin->mbus_cfg.type = ep.bus_type;
-
-       switch (vin->mbus_cfg.type) {
-       case V4L2_MBUS_PARALLEL:
-               vin->mbus_cfg.flags = ep.bus.parallel.flags;
-               break;
-       case V4L2_MBUS_BT656:
-               vin->mbus_cfg.flags = 0;
-               break;
-       default:
-               vin_err(vin, "Unknown media bus type\n");
-               return -EINVAL;
-       }
-
-       return 0;
-}
-
 static int rcar_vin_probe(struct platform_device *pdev)
 {
+       const struct of_device_id *match;
        struct rvin_dev *vin;
        struct resource *mem;
        int irq, ret;
@@ -267,11 +247,12 @@ static int rcar_vin_probe(struct platform_device *pdev)
        if (!vin)
                return -ENOMEM;
 
-       vin->dev = &pdev->dev;
+       match = of_match_device(of_match_ptr(rvin_of_id_table), &pdev->dev);
+       if (!match)
+               return -ENODEV;
 
-       ret = rvin_parse_dt(vin);
-       if (ret)
-               return ret;
+       vin->dev = &pdev->dev;
+       vin->chip = (enum chip_id)match->data;
 
        mem = platform_get_resource(pdev, IORESOURCE_MEM, 0);
        if (mem == NULL)
@@ -282,14 +263,14 @@ static int rcar_vin_probe(struct platform_device *pdev)
                return PTR_ERR(vin->base);
 
        irq = platform_get_irq(pdev, 0);
-       if (irq <= 0)
-               return ret;
+       if (irq < 0)
+               return irq;
 
        ret = rvin_dma_probe(vin, irq);
        if (ret)
                return ret;
 
-       ret = rvin_graph_init(vin);
+       ret = rvin_digital_graph_init(vin);
        if (ret < 0)
                goto error;