dm: core: Add a new flag to track platform data 41/268141/1
authorSimon Glass <sjg@chromium.org>
Mon, 30 Dec 2019 04:19:21 +0000 (21:19 -0700)
committerJaehoon Chung <jh80.chung@samsung.com>
Thu, 16 Dec 2021 11:38:32 +0000 (20:38 +0900)
We want to avoid allocating platform data twice. This could happen if
device_probe() is called after device_ofdata_to_platdata() for the same
device.

Add a flag to track whether device_ofdata_to_platdata() has been called on
a device. Check the flag to make sure it doesn't happen twice, and clear
the flag when the data is freed.

Change-Id: Idbc90897267890b746858ca01fa5db47e9e6b202
Signed-off-by: Simon Glass <sjg@chromium.org>
drivers/core/device-remove.c
drivers/core/device.c
include/dm/device.h

index 586fade..5540053 100644 (file)
@@ -139,6 +139,7 @@ void device_free(struct udevice *dev)
                        dev->parent_priv = NULL;
                }
        }
+       dev->flags &= ~DM_FLAG_PLATDATA_VALID;
 
        devres_release_probe(dev);
 }
index 86cd213..ace2b1d 100644 (file)
@@ -315,7 +315,7 @@ int device_ofdata_to_platdata(struct udevice *dev)
        if (!dev)
                return -EINVAL;
 
-       if (dev->flags & DM_FLAG_ACTIVATED)
+       if (dev->flags & DM_FLAG_PLATDATA_VALID)
                return 0;
 
        drv = dev->driver;
@@ -363,6 +363,8 @@ int device_ofdata_to_platdata(struct udevice *dev)
                        goto fail;
        }
 
+       dev->flags |= DM_FLAG_PLATDATA_VALID;
+
        return 0;
 fail:
        device_free(dev);
index cd4f477..3deee4e 100644 (file)
@@ -61,6 +61,12 @@ struct driver_info;
  */
 #define DM_FLAG_OS_PREPARE             (1 << 10)
 
+/* DM does not enable/disable the power domains corresponding to this device */
+#define DM_FLAG_DEFAULT_PD_CTRL_OFF    (1 << 11)
+
+/* Driver platdata has been read. Cleared when the device is removed */
+#define DM_FLAG_PLATDATA_VALID         (1 << 12)
+
 /*
  * 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