dm: core: Allocate platform data when binding a device
When using allocated platform data, allocate it when we bind the device. This makes it possible to fill in this information before the device is probed. This fits with the platform data model (when not using device tree), since platform data exists at bind-time. Signed-off-by: Simon Glass <sjg@chromium.org> Reviewed-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
This commit is contained in:
@@ -88,6 +88,10 @@ int device_unbind(struct udevice *dev)
|
||||
if (ret)
|
||||
return ret;
|
||||
|
||||
if (dev->flags & DM_FLAG_ALLOC_PDATA) {
|
||||
free(dev->platdata);
|
||||
dev->platdata = NULL;
|
||||
}
|
||||
ret = uclass_unbind_device(dev);
|
||||
if (ret)
|
||||
return ret;
|
||||
@@ -111,10 +115,6 @@ void device_free(struct udevice *dev)
|
||||
free(dev->priv);
|
||||
dev->priv = NULL;
|
||||
}
|
||||
if (dev->flags & DM_FLAG_ALLOC_PDATA) {
|
||||
free(dev->platdata);
|
||||
dev->platdata = NULL;
|
||||
}
|
||||
size = dev->uclass->uc_drv->per_device_auto_alloc_size;
|
||||
if (size) {
|
||||
free(dev->uclass_priv);
|
||||
|
||||
@@ -72,8 +72,14 @@ int device_bind(struct udevice *parent, struct driver *drv, const char *name,
|
||||
#else
|
||||
dev->req_seq = -1;
|
||||
#endif
|
||||
if (!dev->platdata && drv->platdata_auto_alloc_size)
|
||||
if (!dev->platdata && drv->platdata_auto_alloc_size) {
|
||||
dev->flags |= DM_FLAG_ALLOC_PDATA;
|
||||
dev->platdata = calloc(1, drv->platdata_auto_alloc_size);
|
||||
if (!dev->platdata) {
|
||||
ret = -ENOMEM;
|
||||
goto fail_alloc1;
|
||||
}
|
||||
}
|
||||
|
||||
/* put dev into parent's successor list */
|
||||
if (parent)
|
||||
@@ -103,6 +109,11 @@ fail_bind:
|
||||
fail_uclass_bind:
|
||||
if (parent)
|
||||
list_del(&dev->sibling_node);
|
||||
if (dev->flags & DM_FLAG_ALLOC_PDATA) {
|
||||
free(dev->platdata);
|
||||
dev->platdata = NULL;
|
||||
}
|
||||
fail_alloc1:
|
||||
free(dev);
|
||||
|
||||
return ret;
|
||||
@@ -139,7 +150,7 @@ int device_probe_child(struct udevice *dev, void *parent_priv)
|
||||
drv = dev->driver;
|
||||
assert(drv);
|
||||
|
||||
/* Allocate private data and platdata if requested */
|
||||
/* Allocate private data if requested */
|
||||
if (drv->priv_auto_alloc_size) {
|
||||
dev->priv = calloc(1, drv->priv_auto_alloc_size);
|
||||
if (!dev->priv) {
|
||||
@@ -148,13 +159,6 @@ int device_probe_child(struct udevice *dev, void *parent_priv)
|
||||
}
|
||||
}
|
||||
/* Allocate private data if requested */
|
||||
if (dev->flags & DM_FLAG_ALLOC_PDATA) {
|
||||
dev->platdata = calloc(1, drv->platdata_auto_alloc_size);
|
||||
if (!dev->platdata) {
|
||||
ret = -ENOMEM;
|
||||
goto fail;
|
||||
}
|
||||
}
|
||||
size = dev->uclass->uc_drv->per_device_auto_alloc_size;
|
||||
if (size) {
|
||||
dev->uclass_priv = calloc(1, size);
|
||||
|
||||
Reference in New Issue
Block a user