drm/dp: Move DisplayPort AUX bus helpers into dp/
authorThomas Zimmermann <tzimmermann@suse.de>
Fri, 14 Jan 2022 11:45:35 +0000 (12:45 +0100)
committerThomas Zimmermann <tzimmermann@suse.de>
Mon, 17 Jan 2022 10:25:45 +0000 (11:25 +0100)
Move drm_dp_aux_bus.c and its header file into the DP subdirectory
and update all users. No functional changes.

Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
Acked-by: Daniel Vetter <daniel@ffwll.ch>
Link: https://patchwork.freedesktop.org/patch/msgid/20220114114535.29157-6-tzimmermann@suse.de
drivers/gpu/drm/Makefile
drivers/gpu/drm/bridge/parade-ps8640.c
drivers/gpu/drm/bridge/ti-sn65dsi86.c
drivers/gpu/drm/dp/Makefile
drivers/gpu/drm/dp/drm_dp_aux_bus.c [new file with mode: 0644]
drivers/gpu/drm/drm_dp_aux_bus.c [deleted file]
drivers/gpu/drm/panel/panel-edp.c
drivers/gpu/drm/panel/panel-samsung-atna33xc20.c
include/drm/dp/drm_dp_aux_bus.h [new file with mode: 0644]
include/drm/drm_dp_aux_bus.h [deleted file]

index 69be80ef1d312b6299689b418be227caacbb9933..700abeb4945e64ed9ea07512d838d8d69156eb79 100644 (file)
@@ -31,8 +31,6 @@ drm-$(CONFIG_DEBUG_FS) += drm_debugfs.o drm_debugfs_crc.o
 drm-$(CONFIG_DRM_LOAD_EDID_FIRMWARE) += drm_edid_load.o
 drm-$(CONFIG_DRM_PRIVACY_SCREEN) += drm_privacy_screen.o drm_privacy_screen_x86.o
 
-obj-$(CONFIG_DRM_DP_AUX_BUS) += drm_dp_aux_bus.o
-
 obj-$(CONFIG_DRM_NOMODESET) += drm_nomodeset.o
 
 drm_cma_helper-y := drm_gem_cma_helper.o
index 3e0c7436f407cd26bf38684a98f9367080144b62..3f17337ee3892d31325e2fb8ae019fbef0ea81ea 100644 (file)
@@ -14,7 +14,7 @@
 #include <linux/regulator/consumer.h>
 
 #include <drm/drm_bridge.h>
-#include <drm/drm_dp_aux_bus.h>
+#include <drm/dp/drm_dp_aux_bus.h>
 #include <drm/dp/drm_dp_helper.h>
 #include <drm/drm_mipi_dsi.h>
 #include <drm/drm_of.h>
index 65d25ce30ce54594ef020d579178e3e535166335..ba136a188be7d5452997ed08a3c37baa6d6e87b1 100644 (file)
@@ -26,7 +26,7 @@
 #include <drm/drm_atomic.h>
 #include <drm/drm_atomic_helper.h>
 #include <drm/drm_bridge.h>
-#include <drm/drm_dp_aux_bus.h>
+#include <drm/dp/drm_dp_aux_bus.h>
 #include <drm/dp/drm_dp_helper.h>
 #include <drm/drm_mipi_dsi.h>
 #include <drm/drm_of.h>
index 5b892aeff5ab2b3df03feec9ea7b46314411db07..75faffc706b1ad4658a44c8bef36da15d3ec5486 100644 (file)
@@ -1,5 +1,7 @@
 # SPDX-License-Identifier: MIT
 
+obj-$(CONFIG_DRM_DP_AUX_BUS) += drm_dp_aux_bus.o
+
 drm_dp_helper-y := drm_dp.o drm_dp_dual_mode_helper.o drm_dp_helper_mod.o drm_dp_mst_topology.o
 drm_dp_helper-$(CONFIG_DRM_DP_AUX_CHARDEV) += drm_dp_aux_dev.o
 drm_dp_helper-$(CONFIG_DRM_DP_CEC) += drm_dp_cec.o
diff --git a/drivers/gpu/drm/dp/drm_dp_aux_bus.c b/drivers/gpu/drm/dp/drm_dp_aux_bus.c
new file mode 100644 (file)
index 0000000..415afce
--- /dev/null
@@ -0,0 +1,323 @@
+// SPDX-License-Identifier: GPL-2.0
+/*
+ * Copyright 2021 Google Inc.
+ *
+ * The DP AUX bus is used for devices that are connected over a DisplayPort
+ * AUX bus. The devices on the far side of the bus are referred to as
+ * endpoints in this code.
+ *
+ * Commonly there is only one device connected to the DP AUX bus: a panel.
+ * Though historically panels (even DP panels) have been modeled as simple
+ * platform devices, putting them under the DP AUX bus allows the panel driver
+ * to perform transactions on that bus.
+ */
+
+#include <linux/init.h>
+#include <linux/kernel.h>
+#include <linux/module.h>
+#include <linux/of_device.h>
+#include <linux/pm_domain.h>
+#include <linux/pm_runtime.h>
+
+#include <drm/dp/drm_dp_aux_bus.h>
+#include <drm/dp/drm_dp_helper.h>
+
+/**
+ * dp_aux_ep_match() - The match function for the dp_aux_bus.
+ * @dev: The device to match.
+ * @drv: The driver to try to match against.
+ *
+ * At the moment, we just match on device tree.
+ *
+ * Return: True if this driver matches this device; false otherwise.
+ */
+static int dp_aux_ep_match(struct device *dev, struct device_driver *drv)
+{
+       return !!of_match_device(drv->of_match_table, dev);
+}
+
+/**
+ * dp_aux_ep_probe() - The probe function for the dp_aux_bus.
+ * @dev: The device to probe.
+ *
+ * Calls through to the endpoint driver probe.
+ *
+ * Return: 0 if no error or negative error code.
+ */
+static int dp_aux_ep_probe(struct device *dev)
+{
+       struct dp_aux_ep_driver *aux_ep_drv = to_dp_aux_ep_drv(dev->driver);
+       struct dp_aux_ep_device *aux_ep = to_dp_aux_ep_dev(dev);
+       int ret;
+
+       ret = dev_pm_domain_attach(dev, true);
+       if (ret)
+               return dev_err_probe(dev, ret, "Failed to attach to PM Domain\n");
+
+       ret = aux_ep_drv->probe(aux_ep);
+       if (ret)
+               dev_pm_domain_detach(dev, true);
+
+       return ret;
+}
+
+/**
+ * dp_aux_ep_remove() - The remove function for the dp_aux_bus.
+ * @dev: The device to remove.
+ *
+ * Calls through to the endpoint driver remove.
+ *
+ */
+static void dp_aux_ep_remove(struct device *dev)
+{
+       struct dp_aux_ep_driver *aux_ep_drv = to_dp_aux_ep_drv(dev->driver);
+       struct dp_aux_ep_device *aux_ep = to_dp_aux_ep_dev(dev);
+
+       if (aux_ep_drv->remove)
+               aux_ep_drv->remove(aux_ep);
+       dev_pm_domain_detach(dev, true);
+}
+
+/**
+ * dp_aux_ep_shutdown() - The shutdown function for the dp_aux_bus.
+ * @dev: The device to shutdown.
+ *
+ * Calls through to the endpoint driver shutdown.
+ */
+static void dp_aux_ep_shutdown(struct device *dev)
+{
+       struct dp_aux_ep_driver *aux_ep_drv;
+
+       if (!dev->driver)
+               return;
+
+       aux_ep_drv = to_dp_aux_ep_drv(dev->driver);
+       if (aux_ep_drv->shutdown)
+               aux_ep_drv->shutdown(to_dp_aux_ep_dev(dev));
+}
+
+static struct bus_type dp_aux_bus_type = {
+       .name           = "dp-aux",
+       .match          = dp_aux_ep_match,
+       .probe          = dp_aux_ep_probe,
+       .remove         = dp_aux_ep_remove,
+       .shutdown       = dp_aux_ep_shutdown,
+};
+
+static ssize_t modalias_show(struct device *dev,
+                            struct device_attribute *attr, char *buf)
+{
+       return of_device_modalias(dev, buf, PAGE_SIZE);
+}
+static DEVICE_ATTR_RO(modalias);
+
+static struct attribute *dp_aux_ep_dev_attrs[] = {
+       &dev_attr_modalias.attr,
+       NULL,
+};
+ATTRIBUTE_GROUPS(dp_aux_ep_dev);
+
+/**
+ * dp_aux_ep_dev_release() - Free memory for the dp_aux_ep device
+ * @dev: The device to free.
+ *
+ * Return: 0 if no error or negative error code.
+ */
+static void dp_aux_ep_dev_release(struct device *dev)
+{
+       kfree(to_dp_aux_ep_dev(dev));
+}
+
+static struct device_type dp_aux_device_type_type = {
+       .groups         = dp_aux_ep_dev_groups,
+       .uevent         = of_device_uevent_modalias,
+       .release        = dp_aux_ep_dev_release,
+};
+
+/**
+ * of_dp_aux_ep_destroy() - Destroy an DP AUX endpoint device
+ * @dev: The device to destroy.
+ * @data: Not used
+ *
+ * This is just used as a callback by of_dp_aux_depopulate_ep_devices() and
+ * is called for _all_ of the child devices of the device providing the AUX bus.
+ * We'll only act on those that are of type "dp_aux_bus_type".
+ *
+ * This function is effectively an inverse of what's in the loop
+ * in of_dp_aux_populate_ep_devices().
+ *
+ * Return: 0 if no error or negative error code.
+ */
+static int of_dp_aux_ep_destroy(struct device *dev, void *data)
+{
+       struct device_node *np = dev->of_node;
+
+       if (dev->bus != &dp_aux_bus_type)
+               return 0;
+
+       if (!of_node_check_flag(np, OF_POPULATED))
+               return 0;
+
+       of_node_clear_flag(np, OF_POPULATED);
+       of_node_put(np);
+
+       device_unregister(dev);
+
+       return 0;
+}
+
+/**
+ * of_dp_aux_depopulate_ep_devices() - Undo of_dp_aux_populate_ep_devices
+ * @aux: The AUX channel whose devices we want to depopulate
+ *
+ * This will destroy all devices that were created
+ * by of_dp_aux_populate_ep_devices().
+ */
+void of_dp_aux_depopulate_ep_devices(struct drm_dp_aux *aux)
+{
+       device_for_each_child_reverse(aux->dev, NULL, of_dp_aux_ep_destroy);
+}
+EXPORT_SYMBOL_GPL(of_dp_aux_depopulate_ep_devices);
+
+/**
+ * of_dp_aux_populate_ep_devices() - Populate the endpoint devices on the DP AUX
+ * @aux: The AUX channel whose devices we want to populate. It is required that
+ *       drm_dp_aux_init() has already been called for this AUX channel.
+ *
+ * This will populate all the devices under the "aux-bus" node of the device
+ * providing the AUX channel (AKA aux->dev).
+ *
+ * When this function finishes, it is _possible_ (but not guaranteed) that
+ * our sub-devices will have finished probing. It should be noted that if our
+ * sub-devices return -EPROBE_DEFER that we will not return any error codes
+ * ourselves but our sub-devices will _not_ have actually probed successfully
+ * yet. There may be other cases (maybe added in the future?) where sub-devices
+ * won't have been probed yet when this function returns, so it's best not to
+ * rely on that.
+ *
+ * If this function succeeds you should later make sure you call
+ * of_dp_aux_depopulate_ep_devices() to undo it, or just use the devm version
+ * of this function.
+ *
+ * Return: 0 if no error or negative error code.
+ */
+int of_dp_aux_populate_ep_devices(struct drm_dp_aux *aux)
+{
+       struct device_node *bus, *np;
+       struct dp_aux_ep_device *aux_ep;
+       int ret;
+
+       /* drm_dp_aux_init() should have been called already; warn if not */
+       WARN_ON_ONCE(!aux->ddc.algo);
+
+       if (!aux->dev->of_node)
+               return 0;
+
+       bus = of_get_child_by_name(aux->dev->of_node, "aux-bus");
+       if (!bus)
+               return 0;
+
+       for_each_available_child_of_node(bus, np) {
+               if (of_node_test_and_set_flag(np, OF_POPULATED))
+                       continue;
+
+               aux_ep = kzalloc(sizeof(*aux_ep), GFP_KERNEL);
+               if (!aux_ep)
+                       continue;
+               aux_ep->aux = aux;
+
+               aux_ep->dev.parent = aux->dev;
+               aux_ep->dev.bus = &dp_aux_bus_type;
+               aux_ep->dev.type = &dp_aux_device_type_type;
+               aux_ep->dev.of_node = of_node_get(np);
+               dev_set_name(&aux_ep->dev, "aux-%s", dev_name(aux->dev));
+
+               ret = device_register(&aux_ep->dev);
+               if (ret) {
+                       dev_err(aux->dev, "Failed to create AUX EP for %pOF: %d\n", np, ret);
+                       of_node_clear_flag(np, OF_POPULATED);
+                       of_node_put(np);
+
+                       /*
+                        * As per docs of device_register(), call this instead
+                        * of kfree() directly for error cases.
+                        */
+                       put_device(&aux_ep->dev);
+
+                       /*
+                        * Following in the footsteps of of_i2c_register_devices(),
+                        * we won't fail the whole function here--we'll just
+                        * continue registering any other devices we find.
+                        */
+               }
+       }
+
+       of_node_put(bus);
+
+       return 0;
+}
+
+static void of_dp_aux_depopulate_ep_devices_void(void *data)
+{
+       of_dp_aux_depopulate_ep_devices(data);
+}
+
+/**
+ * devm_of_dp_aux_populate_ep_devices() - devm wrapper for of_dp_aux_populate_ep_devices()
+ * @aux: The AUX channel whose devices we want to populate
+ *
+ * Handles freeing w/ devm on the device "aux->dev".
+ *
+ * Return: 0 if no error or negative error code.
+ */
+int devm_of_dp_aux_populate_ep_devices(struct drm_dp_aux *aux)
+{
+       int ret;
+
+       ret = of_dp_aux_populate_ep_devices(aux);
+       if (ret)
+               return ret;
+
+       return devm_add_action_or_reset(aux->dev,
+                                       of_dp_aux_depopulate_ep_devices_void,
+                                       aux);
+}
+EXPORT_SYMBOL_GPL(devm_of_dp_aux_populate_ep_devices);
+
+int __dp_aux_dp_driver_register(struct dp_aux_ep_driver *drv, struct module *owner)
+{
+       drv->driver.owner = owner;
+       drv->driver.bus = &dp_aux_bus_type;
+
+       return driver_register(&drv->driver);
+}
+EXPORT_SYMBOL_GPL(__dp_aux_dp_driver_register);
+
+void dp_aux_dp_driver_unregister(struct dp_aux_ep_driver *drv)
+{
+       driver_unregister(&drv->driver);
+}
+EXPORT_SYMBOL_GPL(dp_aux_dp_driver_unregister);
+
+static int __init dp_aux_bus_init(void)
+{
+       int ret;
+
+       ret = bus_register(&dp_aux_bus_type);
+       if (ret)
+               return ret;
+
+       return 0;
+}
+
+static void __exit dp_aux_bus_exit(void)
+{
+       bus_unregister(&dp_aux_bus_type);
+}
+
+subsys_initcall(dp_aux_bus_init);
+module_exit(dp_aux_bus_exit);
+
+MODULE_AUTHOR("Douglas Anderson <dianders@chromium.org>");
+MODULE_DESCRIPTION("DRM DisplayPort AUX bus");
+MODULE_LICENSE("GPL v2");
diff --git a/drivers/gpu/drm/drm_dp_aux_bus.c b/drivers/gpu/drm/drm_dp_aux_bus.c
deleted file mode 100644 (file)
index 565edf6..0000000
+++ /dev/null
@@ -1,323 +0,0 @@
-// SPDX-License-Identifier: GPL-2.0
-/*
- * Copyright 2021 Google Inc.
- *
- * The DP AUX bus is used for devices that are connected over a DisplayPort
- * AUX bus. The devices on the far side of the bus are referred to as
- * endpoints in this code.
- *
- * Commonly there is only one device connected to the DP AUX bus: a panel.
- * Though historically panels (even DP panels) have been modeled as simple
- * platform devices, putting them under the DP AUX bus allows the panel driver
- * to perform transactions on that bus.
- */
-
-#include <linux/init.h>
-#include <linux/kernel.h>
-#include <linux/module.h>
-#include <linux/of_device.h>
-#include <linux/pm_domain.h>
-#include <linux/pm_runtime.h>
-
-#include <drm/drm_dp_aux_bus.h>
-#include <drm/dp/drm_dp_helper.h>
-
-/**
- * dp_aux_ep_match() - The match function for the dp_aux_bus.
- * @dev: The device to match.
- * @drv: The driver to try to match against.
- *
- * At the moment, we just match on device tree.
- *
- * Return: True if this driver matches this device; false otherwise.
- */
-static int dp_aux_ep_match(struct device *dev, struct device_driver *drv)
-{
-       return !!of_match_device(drv->of_match_table, dev);
-}
-
-/**
- * dp_aux_ep_probe() - The probe function for the dp_aux_bus.
- * @dev: The device to probe.
- *
- * Calls through to the endpoint driver probe.
- *
- * Return: 0 if no error or negative error code.
- */
-static int dp_aux_ep_probe(struct device *dev)
-{
-       struct dp_aux_ep_driver *aux_ep_drv = to_dp_aux_ep_drv(dev->driver);
-       struct dp_aux_ep_device *aux_ep = to_dp_aux_ep_dev(dev);
-       int ret;
-
-       ret = dev_pm_domain_attach(dev, true);
-       if (ret)
-               return dev_err_probe(dev, ret, "Failed to attach to PM Domain\n");
-
-       ret = aux_ep_drv->probe(aux_ep);
-       if (ret)
-               dev_pm_domain_detach(dev, true);
-
-       return ret;
-}
-
-/**
- * dp_aux_ep_remove() - The remove function for the dp_aux_bus.
- * @dev: The device to remove.
- *
- * Calls through to the endpoint driver remove.
- *
- */
-static void dp_aux_ep_remove(struct device *dev)
-{
-       struct dp_aux_ep_driver *aux_ep_drv = to_dp_aux_ep_drv(dev->driver);
-       struct dp_aux_ep_device *aux_ep = to_dp_aux_ep_dev(dev);
-
-       if (aux_ep_drv->remove)
-               aux_ep_drv->remove(aux_ep);
-       dev_pm_domain_detach(dev, true);
-}
-
-/**
- * dp_aux_ep_shutdown() - The shutdown function for the dp_aux_bus.
- * @dev: The device to shutdown.
- *
- * Calls through to the endpoint driver shutdown.
- */
-static void dp_aux_ep_shutdown(struct device *dev)
-{
-       struct dp_aux_ep_driver *aux_ep_drv;
-
-       if (!dev->driver)
-               return;
-
-       aux_ep_drv = to_dp_aux_ep_drv(dev->driver);
-       if (aux_ep_drv->shutdown)
-               aux_ep_drv->shutdown(to_dp_aux_ep_dev(dev));
-}
-
-static struct bus_type dp_aux_bus_type = {
-       .name           = "dp-aux",
-       .match          = dp_aux_ep_match,
-       .probe          = dp_aux_ep_probe,
-       .remove         = dp_aux_ep_remove,
-       .shutdown       = dp_aux_ep_shutdown,
-};
-
-static ssize_t modalias_show(struct device *dev,
-                            struct device_attribute *attr, char *buf)
-{
-       return of_device_modalias(dev, buf, PAGE_SIZE);
-}
-static DEVICE_ATTR_RO(modalias);
-
-static struct attribute *dp_aux_ep_dev_attrs[] = {
-       &dev_attr_modalias.attr,
-       NULL,
-};
-ATTRIBUTE_GROUPS(dp_aux_ep_dev);
-
-/**
- * dp_aux_ep_dev_release() - Free memory for the dp_aux_ep device
- * @dev: The device to free.
- *
- * Return: 0 if no error or negative error code.
- */
-static void dp_aux_ep_dev_release(struct device *dev)
-{
-       kfree(to_dp_aux_ep_dev(dev));
-}
-
-static struct device_type dp_aux_device_type_type = {
-       .groups         = dp_aux_ep_dev_groups,
-       .uevent         = of_device_uevent_modalias,
-       .release        = dp_aux_ep_dev_release,
-};
-
-/**
- * of_dp_aux_ep_destroy() - Destroy an DP AUX endpoint device
- * @dev: The device to destroy.
- * @data: Not used
- *
- * This is just used as a callback by of_dp_aux_depopulate_ep_devices() and
- * is called for _all_ of the child devices of the device providing the AUX bus.
- * We'll only act on those that are of type "dp_aux_bus_type".
- *
- * This function is effectively an inverse of what's in the loop
- * in of_dp_aux_populate_ep_devices().
- *
- * Return: 0 if no error or negative error code.
- */
-static int of_dp_aux_ep_destroy(struct device *dev, void *data)
-{
-       struct device_node *np = dev->of_node;
-
-       if (dev->bus != &dp_aux_bus_type)
-               return 0;
-
-       if (!of_node_check_flag(np, OF_POPULATED))
-               return 0;
-
-       of_node_clear_flag(np, OF_POPULATED);
-       of_node_put(np);
-
-       device_unregister(dev);
-
-       return 0;
-}
-
-/**
- * of_dp_aux_depopulate_ep_devices() - Undo of_dp_aux_populate_ep_devices
- * @aux: The AUX channel whose devices we want to depopulate
- *
- * This will destroy all devices that were created
- * by of_dp_aux_populate_ep_devices().
- */
-void of_dp_aux_depopulate_ep_devices(struct drm_dp_aux *aux)
-{
-       device_for_each_child_reverse(aux->dev, NULL, of_dp_aux_ep_destroy);
-}
-EXPORT_SYMBOL_GPL(of_dp_aux_depopulate_ep_devices);
-
-/**
- * of_dp_aux_populate_ep_devices() - Populate the endpoint devices on the DP AUX
- * @aux: The AUX channel whose devices we want to populate. It is required that
- *       drm_dp_aux_init() has already been called for this AUX channel.
- *
- * This will populate all the devices under the "aux-bus" node of the device
- * providing the AUX channel (AKA aux->dev).
- *
- * When this function finishes, it is _possible_ (but not guaranteed) that
- * our sub-devices will have finished probing. It should be noted that if our
- * sub-devices return -EPROBE_DEFER that we will not return any error codes
- * ourselves but our sub-devices will _not_ have actually probed successfully
- * yet. There may be other cases (maybe added in the future?) where sub-devices
- * won't have been probed yet when this function returns, so it's best not to
- * rely on that.
- *
- * If this function succeeds you should later make sure you call
- * of_dp_aux_depopulate_ep_devices() to undo it, or just use the devm version
- * of this function.
- *
- * Return: 0 if no error or negative error code.
- */
-int of_dp_aux_populate_ep_devices(struct drm_dp_aux *aux)
-{
-       struct device_node *bus, *np;
-       struct dp_aux_ep_device *aux_ep;
-       int ret;
-
-       /* drm_dp_aux_init() should have been called already; warn if not */
-       WARN_ON_ONCE(!aux->ddc.algo);
-
-       if (!aux->dev->of_node)
-               return 0;
-
-       bus = of_get_child_by_name(aux->dev->of_node, "aux-bus");
-       if (!bus)
-               return 0;
-
-       for_each_available_child_of_node(bus, np) {
-               if (of_node_test_and_set_flag(np, OF_POPULATED))
-                       continue;
-
-               aux_ep = kzalloc(sizeof(*aux_ep), GFP_KERNEL);
-               if (!aux_ep)
-                       continue;
-               aux_ep->aux = aux;
-
-               aux_ep->dev.parent = aux->dev;
-               aux_ep->dev.bus = &dp_aux_bus_type;
-               aux_ep->dev.type = &dp_aux_device_type_type;
-               aux_ep->dev.of_node = of_node_get(np);
-               dev_set_name(&aux_ep->dev, "aux-%s", dev_name(aux->dev));
-
-               ret = device_register(&aux_ep->dev);
-               if (ret) {
-                       dev_err(aux->dev, "Failed to create AUX EP for %pOF: %d\n", np, ret);
-                       of_node_clear_flag(np, OF_POPULATED);
-                       of_node_put(np);
-
-                       /*
-                        * As per docs of device_register(), call this instead
-                        * of kfree() directly for error cases.
-                        */
-                       put_device(&aux_ep->dev);
-
-                       /*
-                        * Following in the footsteps of of_i2c_register_devices(),
-                        * we won't fail the whole function here--we'll just
-                        * continue registering any other devices we find.
-                        */
-               }
-       }
-
-       of_node_put(bus);
-
-       return 0;
-}
-
-static void of_dp_aux_depopulate_ep_devices_void(void *data)
-{
-       of_dp_aux_depopulate_ep_devices(data);
-}
-
-/**
- * devm_of_dp_aux_populate_ep_devices() - devm wrapper for of_dp_aux_populate_ep_devices()
- * @aux: The AUX channel whose devices we want to populate
- *
- * Handles freeing w/ devm on the device "aux->dev".
- *
- * Return: 0 if no error or negative error code.
- */
-int devm_of_dp_aux_populate_ep_devices(struct drm_dp_aux *aux)
-{
-       int ret;
-
-       ret = of_dp_aux_populate_ep_devices(aux);
-       if (ret)
-               return ret;
-
-       return devm_add_action_or_reset(aux->dev,
-                                       of_dp_aux_depopulate_ep_devices_void,
-                                       aux);
-}
-EXPORT_SYMBOL_GPL(devm_of_dp_aux_populate_ep_devices);
-
-int __dp_aux_dp_driver_register(struct dp_aux_ep_driver *drv, struct module *owner)
-{
-       drv->driver.owner = owner;
-       drv->driver.bus = &dp_aux_bus_type;
-
-       return driver_register(&drv->driver);
-}
-EXPORT_SYMBOL_GPL(__dp_aux_dp_driver_register);
-
-void dp_aux_dp_driver_unregister(struct dp_aux_ep_driver *drv)
-{
-       driver_unregister(&drv->driver);
-}
-EXPORT_SYMBOL_GPL(dp_aux_dp_driver_unregister);
-
-static int __init dp_aux_bus_init(void)
-{
-       int ret;
-
-       ret = bus_register(&dp_aux_bus_type);
-       if (ret)
-               return ret;
-
-       return 0;
-}
-
-static void __exit dp_aux_bus_exit(void)
-{
-       bus_unregister(&dp_aux_bus_type);
-}
-
-subsys_initcall(dp_aux_bus_init);
-module_exit(dp_aux_bus_exit);
-
-MODULE_AUTHOR("Douglas Anderson <dianders@chromium.org>");
-MODULE_DESCRIPTION("DRM DisplayPort AUX bus");
-MODULE_LICENSE("GPL v2");
index 6a6ca891ee2ea6d247481b00f60de5ff4bd24de3..99ca1bd0091c484b49d72e3f370c71e86a06f94d 100644 (file)
@@ -36,7 +36,7 @@
 
 #include <drm/drm_crtc.h>
 #include <drm/drm_device.h>
-#include <drm/drm_dp_aux_bus.h>
+#include <drm/dp/drm_dp_aux_bus.h>
 #include <drm/dp/drm_dp_helper.h>
 #include <drm/drm_panel.h>
 
index bffeadaaf9a2d98ee634a5df71cbbc9cf6ef9dbc..20666b6217e7cc3ba9d6a5c3f30dd455e16aab0d 100644 (file)
@@ -14,7 +14,7 @@
 #include <linux/pm_runtime.h>
 #include <linux/regulator/consumer.h>
 
-#include <drm/drm_dp_aux_bus.h>
+#include <drm/dp/drm_dp_aux_bus.h>
 #include <drm/dp/drm_dp_helper.h>
 #include <drm/drm_edid.h>
 #include <drm/drm_panel.h>
diff --git a/include/drm/dp/drm_dp_aux_bus.h b/include/drm/dp/drm_dp_aux_bus.h
new file mode 100644 (file)
index 0000000..4f19b20
--- /dev/null
@@ -0,0 +1,57 @@
+/* SPDX-License-Identifier: GPL-2.0 */
+/*
+ * Copyright 2021 Google Inc.
+ *
+ * The DP AUX bus is used for devices that are connected over a DisplayPort
+ * AUX bus. The devices on the far side of the bus are referred to as
+ * endpoints in this code.
+ */
+
+#ifndef _DP_AUX_BUS_H_
+#define _DP_AUX_BUS_H_
+
+#include <linux/device.h>
+#include <linux/mod_devicetable.h>
+
+/**
+ * struct dp_aux_ep_device - Main dev structure for DP AUX endpoints
+ *
+ * This is used to instantiate devices that are connected via a DP AUX
+ * bus. Usually the device is a panel, but conceivable other devices could
+ * be hooked up there.
+ */
+struct dp_aux_ep_device {
+       /** @dev: The normal dev pointer */
+       struct device dev;
+       /** @aux: Pointer to the aux bus */
+       struct drm_dp_aux *aux;
+};
+
+struct dp_aux_ep_driver {
+       int (*probe)(struct dp_aux_ep_device *aux_ep);
+       void (*remove)(struct dp_aux_ep_device *aux_ep);
+       void (*shutdown)(struct dp_aux_ep_device *aux_ep);
+       struct device_driver driver;
+};
+
+static inline struct dp_aux_ep_device *to_dp_aux_ep_dev(struct device *dev)
+{
+       return container_of(dev, struct dp_aux_ep_device, dev);
+}
+
+static inline struct dp_aux_ep_driver *to_dp_aux_ep_drv(struct device_driver *drv)
+{
+       return container_of(drv, struct dp_aux_ep_driver, driver);
+}
+
+int of_dp_aux_populate_ep_devices(struct drm_dp_aux *aux);
+void of_dp_aux_depopulate_ep_devices(struct drm_dp_aux *aux);
+int devm_of_dp_aux_populate_ep_devices(struct drm_dp_aux *aux);
+
+#define dp_aux_dp_driver_register(aux_ep_drv) \
+       __dp_aux_dp_driver_register(aux_ep_drv, THIS_MODULE)
+int __dp_aux_dp_driver_register(struct dp_aux_ep_driver *aux_ep_drv,
+                               struct module *owner);
+void dp_aux_dp_driver_unregister(struct dp_aux_ep_driver *aux_ep_drv);
+
+#endif /* _DP_AUX_BUS_H_ */
diff --git a/include/drm/drm_dp_aux_bus.h b/include/drm/drm_dp_aux_bus.h
deleted file mode 100644 (file)
index 4f19b20..0000000
+++ /dev/null
@@ -1,57 +0,0 @@
-/* SPDX-License-Identifier: GPL-2.0 */
-/*
- * Copyright 2021 Google Inc.
- *
- * The DP AUX bus is used for devices that are connected over a DisplayPort
- * AUX bus. The devices on the far side of the bus are referred to as
- * endpoints in this code.
- */
-
-#ifndef _DP_AUX_BUS_H_
-#define _DP_AUX_BUS_H_
-
-#include <linux/device.h>
-#include <linux/mod_devicetable.h>
-
-/**
- * struct dp_aux_ep_device - Main dev structure for DP AUX endpoints
- *
- * This is used to instantiate devices that are connected via a DP AUX
- * bus. Usually the device is a panel, but conceivable other devices could
- * be hooked up there.
- */
-struct dp_aux_ep_device {
-       /** @dev: The normal dev pointer */
-       struct device dev;
-       /** @aux: Pointer to the aux bus */
-       struct drm_dp_aux *aux;
-};
-
-struct dp_aux_ep_driver {
-       int (*probe)(struct dp_aux_ep_device *aux_ep);
-       void (*remove)(struct dp_aux_ep_device *aux_ep);
-       void (*shutdown)(struct dp_aux_ep_device *aux_ep);
-       struct device_driver driver;
-};
-
-static inline struct dp_aux_ep_device *to_dp_aux_ep_dev(struct device *dev)
-{
-       return container_of(dev, struct dp_aux_ep_device, dev);
-}
-
-static inline struct dp_aux_ep_driver *to_dp_aux_ep_drv(struct device_driver *drv)
-{
-       return container_of(drv, struct dp_aux_ep_driver, driver);
-}
-
-int of_dp_aux_populate_ep_devices(struct drm_dp_aux *aux);
-void of_dp_aux_depopulate_ep_devices(struct drm_dp_aux *aux);
-int devm_of_dp_aux_populate_ep_devices(struct drm_dp_aux *aux);
-
-#define dp_aux_dp_driver_register(aux_ep_drv) \
-       __dp_aux_dp_driver_register(aux_ep_drv, THIS_MODULE)
-int __dp_aux_dp_driver_register(struct dp_aux_ep_driver *aux_ep_drv,
-                               struct module *owner);
-void dp_aux_dp_driver_unregister(struct dp_aux_ep_driver *aux_ep_drv);
-
-#endif /* _DP_AUX_BUS_H_ */