lkml.org 
[lkml]   [2012]   [Apr]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH RESEND] driver core: always handle dpm_order
Date
On Monday, April 23, 2012, Linus Walleij wrote:
> From: Rabin Vincent <rabin.vincent@stericsson.com>
>
> If !dev->class, device_move() does not respect the dpm_order.
> Fix it to do so.
>
> Cc: Rafael J. Wysocki <rjw@sisk.pl>

Acked-by: Rafael J. Wysocki <rjw@sisk.pl>

> Signed-off-by: Rabin Vincent <rabin.vincent@stericsson.com>
> Reviewed-by: Srinidhi Kasagar <srinidhi.kasagar@stericsson.com>
> [Fixed a small dangling label compile warning]
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> ---
> drivers/base/core.c | 36 ++++++++++++++++++------------------
> 1 file changed, 18 insertions(+), 18 deletions(-)
>
> diff --git a/drivers/base/core.c b/drivers/base/core.c
> index e28ce98..f428321 100644
> --- a/drivers/base/core.c
> +++ b/drivers/base/core.c
> @@ -1753,25 +1753,25 @@ int device_move(struct device *dev, struct device *new_parent,
> set_dev_node(dev, dev_to_node(new_parent));
> }
>
> - if (!dev->class)
> - goto out_put;
> - error = device_move_class_links(dev, old_parent, new_parent);
> - if (error) {
> - /* We ignore errors on cleanup since we're hosed anyway... */
> - device_move_class_links(dev, new_parent, old_parent);
> - if (!kobject_move(&dev->kobj, &old_parent->kobj)) {
> - if (new_parent)
> - klist_remove(&dev->p->knode_parent);
> - dev->parent = old_parent;
> - if (old_parent) {
> - klist_add_tail(&dev->p->knode_parent,
> - &old_parent->p->klist_children);
> - set_dev_node(dev, dev_to_node(old_parent));
> + if (dev->class) {
> + error = device_move_class_links(dev, old_parent, new_parent);
> + if (error) {
> + /* We ignore errors on cleanup since we're hosed anyway... */
> + device_move_class_links(dev, new_parent, old_parent);
> + if (!kobject_move(&dev->kobj, &old_parent->kobj)) {
> + if (new_parent)
> + klist_remove(&dev->p->knode_parent);
> + dev->parent = old_parent;
> + if (old_parent) {
> + klist_add_tail(&dev->p->knode_parent,
> + &old_parent->p->klist_children);
> + set_dev_node(dev, dev_to_node(old_parent));
> + }
> }
> + cleanup_glue_dir(dev, new_parent_kobj);
> + put_device(new_parent);
> + goto out;
> }
> - cleanup_glue_dir(dev, new_parent_kobj);
> - put_device(new_parent);
> - goto out;
> }
> switch (dpm_order) {
> case DPM_ORDER_NONE:
> @@ -1786,7 +1786,7 @@ int device_move(struct device *dev, struct device *new_parent,
> device_pm_move_last(dev);
> break;
> }
> -out_put:
> +
> put_device(old_parent);
> out:
> device_pm_unlock();
>



\
 
 \ /
  Last update: 2012-04-23 13:47    [W:0.111 / U:0.720 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site