drm: convert DT component matching to component_match_add_release()
authorRussell King <rmk+kernel@armlinux.org.uk>
Wed, 19 Oct 2016 10:28:27 +0000 (11:28 +0100)
committerSean Paul <seanpaul@chromium.org>
Tue, 25 Oct 2016 15:52:38 +0000 (11:52 -0400)
Convert DT component matching to use component_match_add_release().

Acked-by: Jyri Sarha <jsarha@ti.com>
Reviewed-by: Jyri Sarha <jsarha@ti.com>
Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
Signed-off-by: Sean Paul <seanpaul@chromium.org>
Link: http://patchwork.freedesktop.org/patch/msgid/E1bwo6l-0005Io-Q1@rmk-PC.armlinux.org.uk
13 files changed:
drivers/gpu/drm/arm/hdlcd_drv.c
drivers/gpu/drm/arm/malidp_drv.c
drivers/gpu/drm/armada/armada_drv.c
drivers/gpu/drm/drm_of.c
drivers/gpu/drm/etnaviv/etnaviv_drv.c
drivers/gpu/drm/hisilicon/kirin/kirin_drm_drv.c
drivers/gpu/drm/mediatek/mtk_drm_drv.c
drivers/gpu/drm/msm/msm_drv.c
drivers/gpu/drm/rockchip/rockchip_drm_drv.c
drivers/gpu/drm/sti/sti_drv.c
drivers/gpu/drm/sun4i/sun4i_drv.c
drivers/gpu/drm/tilcdc/tilcdc_external.c
include/drm/drm_of.h

index fb6a418ce6be611176e26bf5fa699503178e8364..6477d1a65266583cd791adf10909d38cc36bd55c 100644 (file)
@@ -453,7 +453,8 @@ static int hdlcd_probe(struct platform_device *pdev)
                return -EAGAIN;
        }
 
-       component_match_add(&pdev->dev, &match, compare_dev, port);
+       drm_of_component_match_add(&pdev->dev, &match, compare_dev, port);
+       of_node_put(port);
 
        return component_master_add_with_match(&pdev->dev, &hdlcd_master_ops,
                                               match);
index 9280358b8f15219da56f1572c53e53272d27dc59..9f4739452a2509950f68a10d1b5a567f9b931298 100644 (file)
@@ -493,7 +493,9 @@ static int malidp_platform_probe(struct platform_device *pdev)
                return -EAGAIN;
        }
 
-       component_match_add(&pdev->dev, &match, malidp_compare_dev, port);
+       drm_of_component_match_add(&pdev->dev, &match, malidp_compare_dev,
+                                  port);
+       of_node_put(port);
        return component_master_add_with_match(&pdev->dev, &malidp_master_ops,
                                               match);
 }
index 1e0e68f608e460ace90ad6a80077e2b5154b03e2..94e46da9a758f7f4496941f3f51534e782b56107 100644 (file)
@@ -254,7 +254,7 @@ static void armada_add_endpoints(struct device *dev,
                        continue;
                }
 
-               component_match_add(dev, match, compare_of, remote);
+               drm_of_component_match_add(dev, match, compare_of, remote);
                of_node_put(remote);
        }
 }
index bc98bb94264d130611bf73ea06a09b74afa5a3a6..47848ed8ca48854afc4279572b887a1b6b3ae56e 100644 (file)
@@ -6,6 +6,11 @@
 #include <drm/drm_crtc.h>
 #include <drm/drm_of.h>
 
+static void drm_release_of(struct device *dev, void *data)
+{
+       of_node_put(data);
+}
+
 /**
  * drm_crtc_port_mask - find the mask of a registered CRTC by port OF node
  * @dev: DRM device
@@ -63,6 +68,24 @@ uint32_t drm_of_find_possible_crtcs(struct drm_device *dev,
 }
 EXPORT_SYMBOL(drm_of_find_possible_crtcs);
 
+/**
+ * drm_of_component_match_add - Add a component helper OF node match rule
+ * @master: master device
+ * @matchptr: component match pointer
+ * @compare: compare function used for matching component
+ * @node: of_node
+ */
+void drm_of_component_match_add(struct device *master,
+                               struct component_match **matchptr,
+                               int (*compare)(struct device *, void *),
+                               struct device_node *node)
+{
+       of_node_get(node);
+       component_match_add_release(master, matchptr, drm_release_of,
+                                   compare, node);
+}
+EXPORT_SYMBOL_GPL(drm_of_component_match_add);
+
 /**
  * drm_of_component_probe - Generic probe function for a component based master
  * @dev: master device containing the OF node
@@ -101,7 +124,7 @@ int drm_of_component_probe(struct device *dev,
                        continue;
                }
 
-               component_match_add(dev, &match, compare_of, port);
+               drm_of_component_match_add(dev, &match, compare_of, port);
                of_node_put(port);
        }
 
@@ -140,7 +163,8 @@ int drm_of_component_probe(struct device *dev,
                                continue;
                        }
 
-                       component_match_add(dev, &match, compare_of, remote);
+                       drm_of_component_match_add(dev, &match, compare_of,
+                                                  remote);
                        of_node_put(remote);
                }
                of_node_put(port);
index aa687669e22b4b05cbbc98f4af5e72ef62979e81..0dee6acbd880a7175ede3de5ca25e91f9a23d058 100644 (file)
@@ -16,6 +16,7 @@
 
 #include <linux/component.h>
 #include <linux/of_platform.h>
+#include <drm/drm_of.h>
 
 #include "etnaviv_drv.h"
 #include "etnaviv_gpu.h"
@@ -629,8 +630,8 @@ static int etnaviv_pdev_probe(struct platform_device *pdev)
                        if (!core_node)
                                break;
 
-                       component_match_add(&pdev->dev, &match, compare_of,
-                                           core_node);
+                       drm_of_component_match_add(&pdev->dev, &match,
+                                                  compare_of, core_node);
                        of_node_put(core_node);
                }
        } else if (dev->platform_data) {
index 90377a609c986e836f3c90e03ed327b0af10ac27..e88fde18c9462e33996804c6f07b11bf8f5b3ccc 100644 (file)
@@ -24,6 +24,7 @@
 #include <drm/drm_fb_cma_helper.h>
 #include <drm/drm_atomic_helper.h>
 #include <drm/drm_crtc_helper.h>
+#include <drm/drm_of.h>
 
 #include "kirin_drm_drv.h"
 
@@ -260,14 +261,13 @@ static struct device_node *kirin_get_remote_node(struct device_node *np)
                DRM_ERROR("no valid endpoint node\n");
                return ERR_PTR(-ENODEV);
        }
-       of_node_put(endpoint);
 
        remote = of_graph_get_remote_port_parent(endpoint);
+       of_node_put(endpoint);
        if (!remote) {
                DRM_ERROR("no valid remote node\n");
                return ERR_PTR(-ENODEV);
        }
-       of_node_put(remote);
 
        if (!of_device_is_available(remote)) {
                DRM_ERROR("not available for remote node\n");
@@ -294,7 +294,8 @@ static int kirin_drm_platform_probe(struct platform_device *pdev)
        if (IS_ERR(remote))
                return PTR_ERR(remote);
 
-       component_match_add(dev, &match, compare_of, remote);
+       drm_of_component_match_add(dev, &match, compare_of, remote);
+       of_node_put(remote);
 
        return component_master_add_with_match(dev, &kirin_drm_ops, match);
 
index db61aa5f32ef7127eef08d2d15b808ad4e568e94..296f541fbe2f121dd54694b6f5753d8d7ad17a2f 100644 (file)
@@ -18,6 +18,7 @@
 #include <drm/drm_crtc_helper.h>
 #include <drm/drm_gem.h>
 #include <drm/drm_gem_cma_helper.h>
+#include <drm/drm_of.h>
 #include <linux/component.h>
 #include <linux/iommu.h>
 #include <linux/of_address.h>
@@ -416,7 +417,8 @@ static int mtk_drm_probe(struct platform_device *pdev)
                    comp_type == MTK_DPI) {
                        dev_info(dev, "Adding component match for %s\n",
                                 node->full_name);
-                       component_match_add(dev, &match, compare_of, node);
+                       drm_of_component_match_add(dev, &match, compare_of,
+                                                  node);
                } else {
                        struct mtk_ddp_comp *comp;
 
index fb5c0b0a7594adcb0f38858cce0fc87d786f4eaa..84d38eaea585757c151e4ccae4a2c72a952823cf 100644 (file)
@@ -15,6 +15,8 @@
  * this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
+#include <drm/drm_of.h>
+
 #include "msm_drv.h"
 #include "msm_debugfs.h"
 #include "msm_fence.h"
@@ -919,8 +921,8 @@ static int add_components_mdp(struct device *mdp_dev,
                        continue;
                }
 
-               component_match_add(master_dev, matchptr, compare_of, intf);
-
+               drm_of_component_match_add(master_dev, matchptr, compare_of,
+                                          intf);
                of_node_put(intf);
                of_node_put(ep_node);
        }
@@ -962,8 +964,8 @@ static int add_display_components(struct device *dev,
                put_device(mdp_dev);
 
                /* add the MDP component itself */
-               component_match_add(dev, matchptr, compare_of,
-                                   mdp_dev->of_node);
+               drm_of_component_match_add(dev, matchptr, compare_of,
+                                          mdp_dev->of_node);
        } else {
                /* MDP4 */
                mdp_dev = dev;
@@ -996,7 +998,7 @@ static int add_gpu_components(struct device *dev,
        if (!np)
                return 0;
 
-       component_match_add(dev, matchptr, compare_of, np);
+       drm_of_component_match_add(dev, matchptr, compare_of, np);
 
        of_node_put(np);
 
index 8c8cbe837e61b41e900e26c3f3f19143e832c49a..6fe161192bb4958eac8c9cd7161e247fcadeb885 100644 (file)
@@ -20,6 +20,7 @@
 #include <drm/drm_crtc_helper.h>
 #include <drm/drm_fb_helper.h>
 #include <drm/drm_gem_cma_helper.h>
+#include <drm/drm_of.h>
 #include <linux/dma-mapping.h>
 #include <linux/pm_runtime.h>
 #include <linux/module.h>
@@ -388,7 +389,7 @@ static void rockchip_add_endpoints(struct device *dev,
                        continue;
                }
 
-               component_match_add(dev, match, compare_of, remote);
+               drm_of_component_match_add(dev, match, compare_of, remote);
                of_node_put(remote);
        }
 }
@@ -437,7 +438,8 @@ static int rockchip_drm_platform_probe(struct platform_device *pdev)
                }
 
                of_node_put(iommu);
-               component_match_add(dev, &match, compare_of, port->parent);
+               drm_of_component_match_add(dev, &match, compare_of,
+                                          port->parent);
                of_node_put(port);
        }
 
index 7087499969bc18a233ca22e4eb7b34adfaf20c0b..6aead2013b625fab015163a3a0626b15ef9cab9a 100644 (file)
@@ -17,6 +17,7 @@
 #include <drm/drm_crtc_helper.h>
 #include <drm/drm_gem_cma_helper.h>
 #include <drm/drm_fb_cma_helper.h>
+#include <drm/drm_of.h>
 
 #include "sti_crtc.h"
 #include "sti_drv.h"
@@ -424,8 +425,8 @@ static int sti_platform_probe(struct platform_device *pdev)
        child_np = of_get_next_available_child(node, NULL);
 
        while (child_np) {
-               component_match_add(dev, &match, compare_of, child_np);
-               of_node_put(child_np);
+               drm_of_component_match_add(dev, &match, compare_of,
+                                          child_np);
                child_np = of_get_next_available_child(node, child_np);
        }
 
index 0da9862ad8ed928e23a6a1e089551967ad5273e8..b3c4ad605e81ba254f065e6ecade6263d9c924d8 100644 (file)
@@ -18,6 +18,7 @@
 #include <drm/drm_fb_cma_helper.h>
 #include <drm/drm_gem_cma_helper.h>
 #include <drm/drm_fb_helper.h>
+#include <drm/drm_of.h>
 
 #include "sun4i_crtc.h"
 #include "sun4i_drv.h"
@@ -239,7 +240,7 @@ static int sun4i_drv_add_endpoints(struct device *dev,
                /* Add current component */
                DRM_DEBUG_DRIVER("Adding component %s\n",
                                 of_node_full_name(node));
-               component_match_add(dev, match, compare_of, node);
+               drm_of_component_match_add(dev, match, compare_of, node);
                count++;
        }
 
index 68e895021005cc62034110748f8f1bf028a2391d..06a4c584f3cbfc8a45919f1289bd8d0844f82791 100644 (file)
@@ -10,6 +10,7 @@
 
 #include <linux/component.h>
 #include <linux/of_graph.h>
+#include <drm/drm_of.h>
 
 #include "tilcdc_drv.h"
 #include "tilcdc_external.h"
@@ -160,7 +161,8 @@ int tilcdc_get_external_components(struct device *dev,
 
                dev_dbg(dev, "Subdevice node '%s' found\n", node->name);
                if (match)
-                       component_match_add(dev, match, dev_match_of, node);
+                       drm_of_component_match_add(dev, match, dev_match_of,
+                                                  node);
                of_node_put(node);
                count++;
        }
index 3fd87b386ed7ccb65b1c8caa318ab6dfb368b401..d6b4c5587bbead7f5d64a41a78d8d7fbf5d09a45 100644 (file)
@@ -4,6 +4,7 @@
 #include <linux/of_graph.h>
 
 struct component_master_ops;
+struct component_match;
 struct device;
 struct drm_device;
 struct drm_encoder;
@@ -12,6 +13,10 @@ struct device_node;
 #ifdef CONFIG_OF
 extern uint32_t drm_of_find_possible_crtcs(struct drm_device *dev,
                                           struct device_node *port);
+extern void drm_of_component_match_add(struct device *master,
+                                      struct component_match **matchptr,
+                                      int (*compare)(struct device *, void *),
+                                      struct device_node *node);
 extern int drm_of_component_probe(struct device *dev,
                                  int (*compare_of)(struct device *, void *),
                                  const struct component_master_ops *m_ops);
@@ -25,6 +30,13 @@ static inline uint32_t drm_of_find_possible_crtcs(struct drm_device *dev,
        return 0;
 }
 
+static void drm_of_component_match_add(struct device *master,
+                                      struct component_match **matchptr,
+                                      int (*compare)(struct device *, void *),
+                                      struct device_node *node)
+{
+}
+
 static inline int
 drm_of_component_probe(struct device *dev,
                       int (*compare_of)(struct device *, void *),