dm: core: Add a flag for power domain control on device removal
authorAnatolij Gustschin <agust@denx.de>
Mon, 17 Feb 2020 11:36:43 +0000 (12:36 +0100)
committerSimon Glass <sjg@chromium.org>
Tue, 3 Mar 2020 02:47:38 +0000 (19:47 -0700)
In various cases a power domain must stay enabled after device
removal when booting OS (i.e. serial debug console or display).
Add a flag to selectively skip switching off a power domain.

Fixes: 52edfed65de9 ("dm: core: device: switch off power domain after device removal")
Signed-off-by: Anatolij Gustschin <agust@denx.de>
Acked-by: Neil Armstrong <narmstrong@baylibre.com>
Tested-by: Guillaume La Roque <glaroque@baylibre.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
drivers/core/device-remove.c
include/dm/device.h

index 444e34b..ff5b28c 100644 (file)
@@ -194,8 +194,9 @@ int device_remove(struct udevice *dev, uint flags)
                }
        }
 
-       if (!(drv->flags & DM_FLAG_DEFAULT_PD_CTRL_OFF) &&
-           (dev != gd->cur_serial_dev))
+       if (!(drv->flags &
+             (DM_FLAG_DEFAULT_PD_CTRL_OFF | DM_FLAG_REMOVE_WITH_PD_ON)) &&
+           dev != gd->cur_serial_dev)
                dev_power_domain_off(dev);
 
        if (flags_remove(flags, drv->flags)) {
index ab806d0..a56164b 100644 (file)
@@ -68,6 +68,12 @@ struct driver_info;
 #define DM_FLAG_PLATDATA_VALID         (1 << 12)
 
 /*
+ * Device is removed without switching off its power domain. This might
+ * be required, i. e. for serial console (debug) output when booting OS.
+ */
+#define DM_FLAG_REMOVE_WITH_PD_ON      (1 << 13)
+
+/*
  * One or multiple of these flags are passed to device_remove() so that
  * a selective device removal as specified by the remove-stage and the
  * driver flags can be done.