To avoid modifying all the drivers call firewall_set_default_config before probe to apply the configuration define in device node (if any). When unbinding the device try to apply configuration named "unbind". Signed-off-by: Benjamin Gaignard <benjamin.gaignard@xxxxxx> --- drivers/base/dd.c | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/drivers/base/dd.c b/drivers/base/dd.c index d811e60610d3..6a2153f6b19b 100644 --- a/drivers/base/dd.c +++ b/drivers/base/dd.c @@ -20,6 +20,7 @@ #include <linux/device.h> #include <linux/delay.h> #include <linux/dma-mapping.h> +#include <linux/firewall.h> #include <linux/init.h> #include <linux/module.h> #include <linux/kthread.h> @@ -521,6 +522,10 @@ static int really_probe(struct device *dev, struct device_driver *drv) re_probe: dev->driver = drv; + ret = firewall_set_default_config(dev); + if (ret) + goto firewall_failed; + /* If using pinctrl, bind pins now before probing */ ret = pinctrl_bind_pins(dev); if (ret) @@ -601,6 +606,8 @@ static int really_probe(struct device *dev, struct device_driver *drv) blocking_notifier_call_chain(&dev->bus->p->bus_notifier, BUS_NOTIFY_DRIVER_NOT_BOUND, dev); pinctrl_bind_failed: + firewall_set_config_by_name(dev, "unbind"); +firewall_failed: device_links_no_driver(dev); devres_release_all(dev); arch_teardown_dma_ops(dev); @@ -1135,6 +1142,8 @@ static void __device_release_driver(struct device *dev, struct device *parent) device_links_driver_cleanup(dev); + firewall_set_config_by_name(dev, "unbind"); + devres_release_all(dev); arch_teardown_dma_ops(dev); dev->driver = NULL; -- 2.15.0
