Commit 18322377 authored by Gavin Shan's avatar Gavin Shan Committed by Rob Herring

drivers/of: Export OF changeset functions

The PowerNV PCI hotplug driver is going to use the OF changeset
to manage the changed device sub-tree. This exports those OF
changeset functions for that.
Signed-off-by: default avatarGavin Shan <gwshan@linux.vnet.ibm.com>
Acked-by: default avatarWolfram Sang <wsa+renesas@sang-engineering.com>
Tested-by: default avatarWolfram Sang <wsa+renesas@sang-engineering.com>
Signed-off-by: default avatarRob Herring <robh@kernel.org>
parent dc17340e
...@@ -646,6 +646,7 @@ void of_changeset_init(struct of_changeset *ocs) ...@@ -646,6 +646,7 @@ void of_changeset_init(struct of_changeset *ocs)
memset(ocs, 0, sizeof(*ocs)); memset(ocs, 0, sizeof(*ocs));
INIT_LIST_HEAD(&ocs->entries); INIT_LIST_HEAD(&ocs->entries);
} }
EXPORT_SYMBOL_GPL(of_changeset_init);
/** /**
* of_changeset_destroy - Destroy a changeset * of_changeset_destroy - Destroy a changeset
...@@ -662,20 +663,9 @@ void of_changeset_destroy(struct of_changeset *ocs) ...@@ -662,20 +663,9 @@ void of_changeset_destroy(struct of_changeset *ocs)
list_for_each_entry_safe_reverse(ce, cen, &ocs->entries, node) list_for_each_entry_safe_reverse(ce, cen, &ocs->entries, node)
__of_changeset_entry_destroy(ce); __of_changeset_entry_destroy(ce);
} }
EXPORT_SYMBOL_GPL(of_changeset_destroy);
/** int __of_changeset_apply(struct of_changeset *ocs)
* of_changeset_apply - Applies a changeset
*
* @ocs: changeset pointer
*
* Applies a changeset to the live tree.
* Any side-effects of live tree state changes are applied here on
* sucess, like creation/destruction of devices and side-effects
* like creation of sysfs properties and directories.
* Returns 0 on success, a negative error value in case of an error.
* On error the partially applied effects are reverted.
*/
int of_changeset_apply(struct of_changeset *ocs)
{ {
struct of_changeset_entry *ce; struct of_changeset_entry *ce;
int ret; int ret;
...@@ -704,17 +694,30 @@ int of_changeset_apply(struct of_changeset *ocs) ...@@ -704,17 +694,30 @@ int of_changeset_apply(struct of_changeset *ocs)
} }
/** /**
* of_changeset_revert - Reverts an applied changeset * of_changeset_apply - Applies a changeset
* *
* @ocs: changeset pointer * @ocs: changeset pointer
* *
* Reverts a changeset returning the state of the tree to what it * Applies a changeset to the live tree.
* was before the application. * Any side-effects of live tree state changes are applied here on
* Any side-effects like creation/destruction of devices and * success, like creation/destruction of devices and side-effects
* removal of sysfs properties and directories are applied. * like creation of sysfs properties and directories.
* Returns 0 on success, a negative error value in case of an error. * Returns 0 on success, a negative error value in case of an error.
* On error the partially applied effects are reverted.
*/ */
int of_changeset_revert(struct of_changeset *ocs) int of_changeset_apply(struct of_changeset *ocs)
{
int ret;
mutex_lock(&of_mutex);
ret = __of_changeset_apply(ocs);
mutex_unlock(&of_mutex);
return ret;
}
EXPORT_SYMBOL_GPL(of_changeset_apply);
int __of_changeset_revert(struct of_changeset *ocs)
{ {
struct of_changeset_entry *ce; struct of_changeset_entry *ce;
int ret; int ret;
...@@ -741,6 +744,29 @@ int of_changeset_revert(struct of_changeset *ocs) ...@@ -741,6 +744,29 @@ int of_changeset_revert(struct of_changeset *ocs)
return 0; return 0;
} }
/**
* of_changeset_revert - Reverts an applied changeset
*
* @ocs: changeset pointer
*
* Reverts a changeset returning the state of the tree to what it
* was before the application.
* Any side-effects like creation/destruction of devices and
* removal of sysfs properties and directories are applied.
* Returns 0 on success, a negative error value in case of an error.
*/
int of_changeset_revert(struct of_changeset *ocs)
{
int ret;
mutex_lock(&of_mutex);
ret = __of_changeset_revert(ocs);
mutex_unlock(&of_mutex);
return ret;
}
EXPORT_SYMBOL_GPL(of_changeset_revert);
/** /**
* of_changeset_action - Perform a changeset action * of_changeset_action - Perform a changeset action
* *
...@@ -779,3 +805,4 @@ int of_changeset_action(struct of_changeset *ocs, unsigned long action, ...@@ -779,3 +805,4 @@ int of_changeset_action(struct of_changeset *ocs, unsigned long action,
list_add_tail(&ce->node, &ocs->entries); list_add_tail(&ce->node, &ocs->entries);
return 0; return 0;
} }
EXPORT_SYMBOL_GPL(of_changeset_action);
...@@ -45,6 +45,8 @@ static inline struct device_node *kobj_to_device_node(struct kobject *kobj) ...@@ -45,6 +45,8 @@ static inline struct device_node *kobj_to_device_node(struct kobject *kobj)
extern int of_property_notify(int action, struct device_node *np, extern int of_property_notify(int action, struct device_node *np,
struct property *prop, struct property *old_prop); struct property *prop, struct property *old_prop);
extern void of_node_release(struct kobject *kobj); extern void of_node_release(struct kobject *kobj);
extern int __of_changeset_apply(struct of_changeset *ocs);
extern int __of_changeset_revert(struct of_changeset *ocs);
#else /* CONFIG_OF_DYNAMIC */ #else /* CONFIG_OF_DYNAMIC */
static inline int of_property_notify(int action, struct device_node *np, static inline int of_property_notify(int action, struct device_node *np,
struct property *prop, struct property *old_prop) struct property *prop, struct property *old_prop)
......
...@@ -379,9 +379,9 @@ int of_overlay_create(struct device_node *tree) ...@@ -379,9 +379,9 @@ int of_overlay_create(struct device_node *tree)
} }
/* apply the changeset */ /* apply the changeset */
err = of_changeset_apply(&ov->cset); err = __of_changeset_apply(&ov->cset);
if (err) { if (err) {
pr_err("%s: of_changeset_apply() failed for tree@%s\n", pr_err("%s: __of_changeset_apply() failed for tree@%s\n",
__func__, tree->full_name); __func__, tree->full_name);
goto err_revert_overlay; goto err_revert_overlay;
} }
...@@ -511,7 +511,7 @@ int of_overlay_destroy(int id) ...@@ -511,7 +511,7 @@ int of_overlay_destroy(int id)
list_del(&ov->node); list_del(&ov->node);
of_changeset_revert(&ov->cset); __of_changeset_revert(&ov->cset);
of_free_overlay_info(ov); of_free_overlay_info(ov);
idr_remove(&ov_idr, id); idr_remove(&ov_idr, id);
of_changeset_destroy(&ov->cset); of_changeset_destroy(&ov->cset);
...@@ -542,7 +542,7 @@ int of_overlay_destroy_all(void) ...@@ -542,7 +542,7 @@ int of_overlay_destroy_all(void)
/* the tail of list is guaranteed to be safe to remove */ /* the tail of list is guaranteed to be safe to remove */
list_for_each_entry_safe_reverse(ov, ovn, &ov_list, node) { list_for_each_entry_safe_reverse(ov, ovn, &ov_list, node) {
list_del(&ov->node); list_del(&ov->node);
of_changeset_revert(&ov->cset); __of_changeset_revert(&ov->cset);
of_free_overlay_info(ov); of_free_overlay_info(ov);
idr_remove(&ov_idr, ov->id); idr_remove(&ov_idr, ov->id);
kfree(ov); kfree(ov);
......
...@@ -530,18 +530,14 @@ static void __init of_unittest_changeset(void) ...@@ -530,18 +530,14 @@ static void __init of_unittest_changeset(void)
unittest(!of_changeset_add_property(&chgset, parent, ppadd), "fail add prop\n"); unittest(!of_changeset_add_property(&chgset, parent, ppadd), "fail add prop\n");
unittest(!of_changeset_update_property(&chgset, parent, ppupdate), "fail update prop\n"); unittest(!of_changeset_update_property(&chgset, parent, ppupdate), "fail update prop\n");
unittest(!of_changeset_remove_property(&chgset, parent, ppremove), "fail remove prop\n"); unittest(!of_changeset_remove_property(&chgset, parent, ppremove), "fail remove prop\n");
mutex_lock(&of_mutex);
unittest(!of_changeset_apply(&chgset), "apply failed\n"); unittest(!of_changeset_apply(&chgset), "apply failed\n");
mutex_unlock(&of_mutex);
/* Make sure node names are constructed correctly */ /* Make sure node names are constructed correctly */
unittest((np = of_find_node_by_path("/testcase-data/changeset/n2/n21")), unittest((np = of_find_node_by_path("/testcase-data/changeset/n2/n21")),
"'%s' not added\n", n21->full_name); "'%s' not added\n", n21->full_name);
of_node_put(np); of_node_put(np);
mutex_lock(&of_mutex);
unittest(!of_changeset_revert(&chgset), "revert failed\n"); unittest(!of_changeset_revert(&chgset), "revert failed\n");
mutex_unlock(&of_mutex);
of_changeset_destroy(&chgset); of_changeset_destroy(&chgset);
#endif #endif
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment