i2c: mv64xxx: Add runtime PM support
To save power, gate the clock when the bus is inactive, during system sleep, and during shutdown. On some platforms, specifically Allwinner A13/A20, gating the clock implicitly resets the module as well. Since the module already needs to be reset after some suspend/resume cycles, it is simple enough to reset it during every runtime suspend/resume. Because the bus may be used by wakeup source IRQ threads, it needs to be functional as soon as IRQs are enabled. Thus, its system PM hooks need to run in the noirq phase. Signed-off-by: Samuel Holland <samuel@sholland.org> Tested-by: Ondrej Jirman <megous@megous.com> Acked-by: Gregory CLEMENT <gregory.clement@bootlin.com> Signed-off-by: Wolfram Sang <wsa@kernel.org>
This commit is contained in:
parent
622fa41b49
commit
e5c02cf541
|
@ -18,6 +18,7 @@
|
||||||
#include <linux/mv643xx_i2c.h>
|
#include <linux/mv643xx_i2c.h>
|
||||||
#include <linux/platform_device.h>
|
#include <linux/platform_device.h>
|
||||||
#include <linux/pinctrl/consumer.h>
|
#include <linux/pinctrl/consumer.h>
|
||||||
|
#include <linux/pm_runtime.h>
|
||||||
#include <linux/reset.h>
|
#include <linux/reset.h>
|
||||||
#include <linux/io.h>
|
#include <linux/io.h>
|
||||||
#include <linux/of.h>
|
#include <linux/of.h>
|
||||||
|
@ -717,6 +718,10 @@ mv64xxx_i2c_xfer(struct i2c_adapter *adap, struct i2c_msg msgs[], int num)
|
||||||
struct mv64xxx_i2c_data *drv_data = i2c_get_adapdata(adap);
|
struct mv64xxx_i2c_data *drv_data = i2c_get_adapdata(adap);
|
||||||
int rc, ret = num;
|
int rc, ret = num;
|
||||||
|
|
||||||
|
rc = pm_runtime_resume_and_get(&adap->dev);
|
||||||
|
if (rc)
|
||||||
|
return rc;
|
||||||
|
|
||||||
BUG_ON(drv_data->msgs != NULL);
|
BUG_ON(drv_data->msgs != NULL);
|
||||||
drv_data->msgs = msgs;
|
drv_data->msgs = msgs;
|
||||||
drv_data->num_msgs = num;
|
drv_data->num_msgs = num;
|
||||||
|
@ -732,6 +737,9 @@ mv64xxx_i2c_xfer(struct i2c_adapter *adap, struct i2c_msg msgs[], int num)
|
||||||
drv_data->num_msgs = 0;
|
drv_data->num_msgs = 0;
|
||||||
drv_data->msgs = NULL;
|
drv_data->msgs = NULL;
|
||||||
|
|
||||||
|
pm_runtime_mark_last_busy(&adap->dev);
|
||||||
|
pm_runtime_put_autosuspend(&adap->dev);
|
||||||
|
|
||||||
return ret;
|
return ret;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -828,7 +836,6 @@ mv64xxx_of_config(struct mv64xxx_i2c_data *drv_data,
|
||||||
rc = PTR_ERR(drv_data->rstc);
|
rc = PTR_ERR(drv_data->rstc);
|
||||||
goto out;
|
goto out;
|
||||||
}
|
}
|
||||||
reset_control_deassert(drv_data->rstc);
|
|
||||||
|
|
||||||
/* Its not yet defined how timeouts will be specified in device tree.
|
/* Its not yet defined how timeouts will be specified in device tree.
|
||||||
* So hard code the value to 1 second.
|
* So hard code the value to 1 second.
|
||||||
|
@ -893,6 +900,32 @@ static int mv64xxx_i2c_init_recovery_info(struct mv64xxx_i2c_data *drv_data,
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
static int
|
||||||
|
mv64xxx_i2c_runtime_suspend(struct device *dev)
|
||||||
|
{
|
||||||
|
struct mv64xxx_i2c_data *drv_data = dev_get_drvdata(dev);
|
||||||
|
|
||||||
|
reset_control_assert(drv_data->rstc);
|
||||||
|
clk_disable_unprepare(drv_data->reg_clk);
|
||||||
|
clk_disable_unprepare(drv_data->clk);
|
||||||
|
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
static int
|
||||||
|
mv64xxx_i2c_runtime_resume(struct device *dev)
|
||||||
|
{
|
||||||
|
struct mv64xxx_i2c_data *drv_data = dev_get_drvdata(dev);
|
||||||
|
|
||||||
|
clk_prepare_enable(drv_data->clk);
|
||||||
|
clk_prepare_enable(drv_data->reg_clk);
|
||||||
|
reset_control_reset(drv_data->rstc);
|
||||||
|
|
||||||
|
mv64xxx_i2c_hw_init(drv_data);
|
||||||
|
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
|
|
||||||
static int
|
static int
|
||||||
mv64xxx_i2c_probe(struct platform_device *pd)
|
mv64xxx_i2c_probe(struct platform_device *pd)
|
||||||
{
|
{
|
||||||
|
@ -920,18 +953,22 @@ mv64xxx_i2c_probe(struct platform_device *pd)
|
||||||
|
|
||||||
/* Not all platforms have clocks */
|
/* Not all platforms have clocks */
|
||||||
drv_data->clk = devm_clk_get(&pd->dev, NULL);
|
drv_data->clk = devm_clk_get(&pd->dev, NULL);
|
||||||
if (PTR_ERR(drv_data->clk) == -EPROBE_DEFER)
|
if (IS_ERR(drv_data->clk)) {
|
||||||
return -EPROBE_DEFER;
|
if (PTR_ERR(drv_data->clk) == -EPROBE_DEFER)
|
||||||
if (!IS_ERR(drv_data->clk))
|
return -EPROBE_DEFER;
|
||||||
clk_prepare_enable(drv_data->clk);
|
drv_data->clk = NULL;
|
||||||
|
}
|
||||||
|
|
||||||
drv_data->reg_clk = devm_clk_get(&pd->dev, "reg");
|
drv_data->reg_clk = devm_clk_get(&pd->dev, "reg");
|
||||||
if (PTR_ERR(drv_data->reg_clk) == -EPROBE_DEFER)
|
if (IS_ERR(drv_data->reg_clk)) {
|
||||||
return -EPROBE_DEFER;
|
if (PTR_ERR(drv_data->reg_clk) == -EPROBE_DEFER)
|
||||||
if (!IS_ERR(drv_data->reg_clk))
|
return -EPROBE_DEFER;
|
||||||
clk_prepare_enable(drv_data->reg_clk);
|
drv_data->reg_clk = NULL;
|
||||||
|
}
|
||||||
|
|
||||||
drv_data->irq = platform_get_irq(pd, 0);
|
drv_data->irq = platform_get_irq(pd, 0);
|
||||||
|
if (drv_data->irq < 0)
|
||||||
|
return drv_data->irq;
|
||||||
|
|
||||||
if (pdata) {
|
if (pdata) {
|
||||||
drv_data->freq_m = pdata->freq_m;
|
drv_data->freq_m = pdata->freq_m;
|
||||||
|
@ -942,16 +979,12 @@ mv64xxx_i2c_probe(struct platform_device *pd)
|
||||||
} else if (pd->dev.of_node) {
|
} else if (pd->dev.of_node) {
|
||||||
rc = mv64xxx_of_config(drv_data, &pd->dev);
|
rc = mv64xxx_of_config(drv_data, &pd->dev);
|
||||||
if (rc)
|
if (rc)
|
||||||
goto exit_clk;
|
return rc;
|
||||||
}
|
|
||||||
if (drv_data->irq < 0) {
|
|
||||||
rc = drv_data->irq;
|
|
||||||
goto exit_reset;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
rc = mv64xxx_i2c_init_recovery_info(drv_data, &pd->dev);
|
rc = mv64xxx_i2c_init_recovery_info(drv_data, &pd->dev);
|
||||||
if (rc == -EPROBE_DEFER)
|
if (rc == -EPROBE_DEFER)
|
||||||
goto exit_reset;
|
return rc;
|
||||||
|
|
||||||
drv_data->adapter.dev.parent = &pd->dev;
|
drv_data->adapter.dev.parent = &pd->dev;
|
||||||
drv_data->adapter.algo = &mv64xxx_i2c_algo;
|
drv_data->adapter.algo = &mv64xxx_i2c_algo;
|
||||||
|
@ -962,7 +995,14 @@ mv64xxx_i2c_probe(struct platform_device *pd)
|
||||||
platform_set_drvdata(pd, drv_data);
|
platform_set_drvdata(pd, drv_data);
|
||||||
i2c_set_adapdata(&drv_data->adapter, drv_data);
|
i2c_set_adapdata(&drv_data->adapter, drv_data);
|
||||||
|
|
||||||
mv64xxx_i2c_hw_init(drv_data);
|
pm_runtime_set_autosuspend_delay(&pd->dev, MSEC_PER_SEC);
|
||||||
|
pm_runtime_use_autosuspend(&pd->dev);
|
||||||
|
pm_runtime_enable(&pd->dev);
|
||||||
|
if (!pm_runtime_enabled(&pd->dev)) {
|
||||||
|
rc = mv64xxx_i2c_runtime_resume(&pd->dev);
|
||||||
|
if (rc)
|
||||||
|
goto exit_disable_pm;
|
||||||
|
}
|
||||||
|
|
||||||
rc = request_irq(drv_data->irq, mv64xxx_i2c_intr, 0,
|
rc = request_irq(drv_data->irq, mv64xxx_i2c_intr, 0,
|
||||||
MV64XXX_I2C_CTLR_NAME, drv_data);
|
MV64XXX_I2C_CTLR_NAME, drv_data);
|
||||||
|
@ -970,7 +1010,7 @@ mv64xxx_i2c_probe(struct platform_device *pd)
|
||||||
dev_err(&drv_data->adapter.dev,
|
dev_err(&drv_data->adapter.dev,
|
||||||
"mv64xxx: Can't register intr handler irq%d: %d\n",
|
"mv64xxx: Can't register intr handler irq%d: %d\n",
|
||||||
drv_data->irq, rc);
|
drv_data->irq, rc);
|
||||||
goto exit_reset;
|
goto exit_disable_pm;
|
||||||
} else if ((rc = i2c_add_numbered_adapter(&drv_data->adapter)) != 0) {
|
} else if ((rc = i2c_add_numbered_adapter(&drv_data->adapter)) != 0) {
|
||||||
dev_err(&drv_data->adapter.dev,
|
dev_err(&drv_data->adapter.dev,
|
||||||
"mv64xxx: Can't add i2c adapter, rc: %d\n", -rc);
|
"mv64xxx: Can't add i2c adapter, rc: %d\n", -rc);
|
||||||
|
@ -981,54 +1021,50 @@ mv64xxx_i2c_probe(struct platform_device *pd)
|
||||||
|
|
||||||
exit_free_irq:
|
exit_free_irq:
|
||||||
free_irq(drv_data->irq, drv_data);
|
free_irq(drv_data->irq, drv_data);
|
||||||
exit_reset:
|
exit_disable_pm:
|
||||||
reset_control_assert(drv_data->rstc);
|
pm_runtime_disable(&pd->dev);
|
||||||
exit_clk:
|
if (!pm_runtime_status_suspended(&pd->dev))
|
||||||
clk_disable_unprepare(drv_data->reg_clk);
|
mv64xxx_i2c_runtime_suspend(&pd->dev);
|
||||||
clk_disable_unprepare(drv_data->clk);
|
|
||||||
|
|
||||||
return rc;
|
return rc;
|
||||||
}
|
}
|
||||||
|
|
||||||
static int
|
static int
|
||||||
mv64xxx_i2c_remove(struct platform_device *dev)
|
mv64xxx_i2c_remove(struct platform_device *pd)
|
||||||
{
|
{
|
||||||
struct mv64xxx_i2c_data *drv_data = platform_get_drvdata(dev);
|
struct mv64xxx_i2c_data *drv_data = platform_get_drvdata(pd);
|
||||||
|
|
||||||
i2c_del_adapter(&drv_data->adapter);
|
i2c_del_adapter(&drv_data->adapter);
|
||||||
free_irq(drv_data->irq, drv_data);
|
free_irq(drv_data->irq, drv_data);
|
||||||
reset_control_assert(drv_data->rstc);
|
pm_runtime_disable(&pd->dev);
|
||||||
clk_disable_unprepare(drv_data->reg_clk);
|
if (!pm_runtime_status_suspended(&pd->dev))
|
||||||
clk_disable_unprepare(drv_data->clk);
|
mv64xxx_i2c_runtime_suspend(&pd->dev);
|
||||||
|
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
#ifdef CONFIG_PM
|
static void
|
||||||
static int mv64xxx_i2c_resume(struct device *dev)
|
mv64xxx_i2c_shutdown(struct platform_device *pd)
|
||||||
{
|
{
|
||||||
struct mv64xxx_i2c_data *drv_data = dev_get_drvdata(dev);
|
pm_runtime_disable(&pd->dev);
|
||||||
|
if (!pm_runtime_status_suspended(&pd->dev))
|
||||||
mv64xxx_i2c_hw_init(drv_data);
|
mv64xxx_i2c_runtime_suspend(&pd->dev);
|
||||||
|
|
||||||
return 0;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static const struct dev_pm_ops mv64xxx_i2c_pm = {
|
static const struct dev_pm_ops mv64xxx_i2c_pm_ops = {
|
||||||
.resume = mv64xxx_i2c_resume,
|
SET_RUNTIME_PM_OPS(mv64xxx_i2c_runtime_suspend,
|
||||||
|
mv64xxx_i2c_runtime_resume, NULL)
|
||||||
|
SET_NOIRQ_SYSTEM_SLEEP_PM_OPS(pm_runtime_force_suspend,
|
||||||
|
pm_runtime_force_resume)
|
||||||
};
|
};
|
||||||
|
|
||||||
#define mv64xxx_i2c_pm_ops (&mv64xxx_i2c_pm)
|
|
||||||
#else
|
|
||||||
#define mv64xxx_i2c_pm_ops NULL
|
|
||||||
#endif
|
|
||||||
|
|
||||||
static struct platform_driver mv64xxx_i2c_driver = {
|
static struct platform_driver mv64xxx_i2c_driver = {
|
||||||
.probe = mv64xxx_i2c_probe,
|
.probe = mv64xxx_i2c_probe,
|
||||||
.remove = mv64xxx_i2c_remove,
|
.remove = mv64xxx_i2c_remove,
|
||||||
|
.shutdown = mv64xxx_i2c_shutdown,
|
||||||
.driver = {
|
.driver = {
|
||||||
.name = MV64XXX_I2C_CTLR_NAME,
|
.name = MV64XXX_I2C_CTLR_NAME,
|
||||||
.pm = mv64xxx_i2c_pm_ops,
|
.pm = &mv64xxx_i2c_pm_ops,
|
||||||
.of_match_table = mv64xxx_i2c_of_match_table,
|
.of_match_table = mv64xxx_i2c_of_match_table,
|
||||||
},
|
},
|
||||||
};
|
};
|
||||||
|
|
Loading…
Reference in New Issue