Commit d08b8037 authored by Pawel Moll's avatar Pawel Moll

power/reset: vexpress: Fix restart/power off operation

The restart/power off implementation in the vexpress driver
used to obtain the config function when necessary. This was
wrong in two respects:

1. It required memory allocation with disabled interrupts
(it worked, but lockdep - when enabled - reported warnings).

2. Used jiffies-based timeout, while jiffies are not running
at this stage of system shutdown (therefore a config
transaction error - if happened - would have never be reported).

Fixed by pre-allocating the config function per device
and using mdelay for timeout.
Signed-off-by: default avatarPawel Moll <pawel.moll@arm.com>
parent bb6dd575
...@@ -11,7 +11,7 @@ ...@@ -11,7 +11,7 @@
* Copyright (C) 2012 ARM Limited * Copyright (C) 2012 ARM Limited
*/ */
#include <linux/jiffies.h> #include <linux/delay.h>
#include <linux/of.h> #include <linux/of.h>
#include <linux/of_device.h> #include <linux/of_device.h>
#include <linux/platform_device.h> #include <linux/platform_device.h>
...@@ -23,17 +23,12 @@ ...@@ -23,17 +23,12 @@
static void vexpress_reset_do(struct device *dev, const char *what) static void vexpress_reset_do(struct device *dev, const char *what)
{ {
int err = -ENOENT; int err = -ENOENT;
struct vexpress_config_func *func = struct vexpress_config_func *func = dev_get_drvdata(dev);
vexpress_config_func_get_by_dev(dev);
if (func) { if (func) {
unsigned long timeout;
err = vexpress_config_write(func, 0, 0); err = vexpress_config_write(func, 0, 0);
if (!err)
timeout = jiffies + HZ; mdelay(1000);
while (time_before(jiffies, timeout))
cpu_relax();
} }
dev_emerg(dev, "Unable to %s (%d)\n", what, err); dev_emerg(dev, "Unable to %s (%d)\n", what, err);
...@@ -96,12 +91,18 @@ static int vexpress_reset_probe(struct platform_device *pdev) ...@@ -96,12 +91,18 @@ static int vexpress_reset_probe(struct platform_device *pdev)
enum vexpress_reset_func func; enum vexpress_reset_func func;
const struct of_device_id *match = const struct of_device_id *match =
of_match_device(vexpress_reset_of_match, &pdev->dev); of_match_device(vexpress_reset_of_match, &pdev->dev);
struct vexpress_config_func *config_func;
if (match) if (match)
func = (enum vexpress_reset_func)match->data; func = (enum vexpress_reset_func)match->data;
else else
func = pdev->id_entry->driver_data; func = pdev->id_entry->driver_data;
config_func = vexpress_config_func_get_by_dev(&pdev->dev);
if (!config_func)
return -EINVAL;
dev_set_drvdata(&pdev->dev, config_func);
switch (func) { switch (func) {
case FUNC_SHUTDOWN: case FUNC_SHUTDOWN:
vexpress_power_off_device = &pdev->dev; vexpress_power_off_device = &pdev->dev;
......
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