Skip to content
Snippets Groups Projects
Commit e1aa13df authored by Fugang Duan's avatar Fugang Duan Committed by Leonard Crestez
Browse files

MLK-17290-02 i2c: imx-lpi2c: manage irq resource request/release in runtime pm


Manage irq resource request/release in runtime pm to save irq domain's
power.

Signed-off-by: default avatarFrank Li <Frank.Li@nxp.com>
Signed-off-by: default avatarFugang Duan <fugang.duan@nxp.com>
Tested-by: default avatarGuoniu.Zhou <guoniu.zhou@nxp.com>
Reviewed-by: default avatarFrank Li <Frank.Li@nxp.com>
(Vipul: Fixed merge conflicts)
Signed-off-by: default avatarVipul Kumar <vipul_kumar@mentor.com>
parent 31cc4be3
No related branches found
No related tags found
No related merge requests found
...@@ -101,6 +101,9 @@ enum lpi2c_imx_pincfg { ...@@ -101,6 +101,9 @@ enum lpi2c_imx_pincfg {
struct lpi2c_imx_struct { struct lpi2c_imx_struct {
struct i2c_adapter adapter; struct i2c_adapter adapter;
struct clk *clk; struct clk *clk;
int irq;
struct clk *clk_per;
struct clk *clk_ipg;
void __iomem *base; void __iomem *base;
__u8 *rx_buf; __u8 *rx_buf;
__u8 *tx_buf; __u8 *tx_buf;
...@@ -549,7 +552,7 @@ static int lpi2c_imx_probe(struct platform_device *pdev) ...@@ -549,7 +552,7 @@ static int lpi2c_imx_probe(struct platform_device *pdev)
struct lpi2c_imx_struct *lpi2c_imx; struct lpi2c_imx_struct *lpi2c_imx;
struct resource *res; struct resource *res;
unsigned int temp; unsigned int temp;
int irq, ret; int ret;
lpi2c_imx = devm_kzalloc(&pdev->dev, sizeof(*lpi2c_imx), GFP_KERNEL); lpi2c_imx = devm_kzalloc(&pdev->dev, sizeof(*lpi2c_imx), GFP_KERNEL);
if (!lpi2c_imx) if (!lpi2c_imx)
...@@ -560,10 +563,10 @@ static int lpi2c_imx_probe(struct platform_device *pdev) ...@@ -560,10 +563,10 @@ static int lpi2c_imx_probe(struct platform_device *pdev)
if (IS_ERR(lpi2c_imx->base)) if (IS_ERR(lpi2c_imx->base))
return PTR_ERR(lpi2c_imx->base); return PTR_ERR(lpi2c_imx->base);
irq = platform_get_irq(pdev, 0); lpi2c_imx->irq = platform_get_irq(pdev, 0);
if (irq < 0) { if (lpi2c_imx->irq < 0) {
dev_err(&pdev->dev, "can't get irq number\n"); dev_err(&pdev->dev, "can't get irq number\n");
return irq; return lpi2c_imx->irq;
} }
lpi2c_imx->adapter.owner = THIS_MODULE; lpi2c_imx->adapter.owner = THIS_MODULE;
...@@ -584,14 +587,6 @@ static int lpi2c_imx_probe(struct platform_device *pdev) ...@@ -584,14 +587,6 @@ static int lpi2c_imx_probe(struct platform_device *pdev)
if (ret) if (ret)
lpi2c_imx->bitrate = LPI2C_DEFAULT_RATE; lpi2c_imx->bitrate = LPI2C_DEFAULT_RATE;
ret = devm_request_irq(&pdev->dev, irq, lpi2c_imx_isr,
IRQF_NO_SUSPEND,
pdev->name, lpi2c_imx);
if (ret) {
dev_err(&pdev->dev, "can't claim irq %d\n", irq);
return ret;
}
i2c_set_adapdata(&lpi2c_imx->adapter, lpi2c_imx); i2c_set_adapdata(&lpi2c_imx->adapter, lpi2c_imx);
platform_set_drvdata(pdev, lpi2c_imx); platform_set_drvdata(pdev, lpi2c_imx);
...@@ -638,6 +633,7 @@ static int lpi2c_runtime_suspend(struct device *dev) ...@@ -638,6 +633,7 @@ static int lpi2c_runtime_suspend(struct device *dev)
{ {
struct lpi2c_imx_struct *lpi2c_imx = dev_get_drvdata(dev); struct lpi2c_imx_struct *lpi2c_imx = dev_get_drvdata(dev);
devm_free_irq(dev, lpi2c_imx->irq, lpi2c_imx);
clk_disable_unprepare(lpi2c_imx->clk_ipg); clk_disable_unprepare(lpi2c_imx->clk_ipg);
clk_disable_unprepare(lpi2c_imx->clk_per); clk_disable_unprepare(lpi2c_imx->clk_per);
pinctrl_pm_select_idle_state(dev); pinctrl_pm_select_idle_state(dev);
...@@ -663,6 +659,14 @@ static int lpi2c_runtime_resume(struct device *dev) ...@@ -663,6 +659,14 @@ static int lpi2c_runtime_resume(struct device *dev)
dev_err(dev, "can't enable I2C ipg clock, ret=%d\n", ret); dev_err(dev, "can't enable I2C ipg clock, ret=%d\n", ret);
} }
ret = devm_request_irq(dev, lpi2c_imx->irq, lpi2c_imx_isr,
IRQF_NO_SUSPEND,
dev_name(dev), lpi2c_imx);
if (ret) {
dev_err(dev, "can't claim irq %d\n", lpi2c_imx->irq);
return ret;
}
return ret; return ret;
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment