Commit 2d7c957e authored by Peter Ujfalusi's avatar Peter Ujfalusi Committed by Mark Brown

MFD: twl6040: Remove global pointer for platform_device

There is no need to keep global pointer for the platform
device, since it is only used for dev_* prints, and the
device pointer available within the twl6040 structure.
Signed-off-by: default avatarPeter Ujfalusi <peter.ujfalusi@ti.com>
Acked-by: default avatarSamuel Ortiz <sameo@linux.intel.com>
Acked-by: default avatarLiam Girdwood <lrg@ti.com>
Signed-off-by: default avatarMark Brown <broonie@opensource.wolfsonmicro.com>
parent 76067540
...@@ -34,8 +34,6 @@ ...@@ -34,8 +34,6 @@
#include <linux/mfd/core.h> #include <linux/mfd/core.h>
#include <linux/mfd/twl6040.h> #include <linux/mfd/twl6040.h>
static struct platform_device *twl6040_dev;
int twl6040_reg_read(struct twl6040 *twl6040, unsigned int reg) int twl6040_reg_read(struct twl6040 *twl6040, unsigned int reg)
{ {
int ret; int ret;
...@@ -203,11 +201,11 @@ static irqreturn_t twl6040_naudint_handler(int irq, void *data) ...@@ -203,11 +201,11 @@ static irqreturn_t twl6040_naudint_handler(int irq, void *data)
if (intid & TWL6040_THINT) { if (intid & TWL6040_THINT) {
status = twl6040_reg_read(twl6040, TWL6040_REG_STATUS); status = twl6040_reg_read(twl6040, TWL6040_REG_STATUS);
if (status & TWL6040_TSHUTDET) { if (status & TWL6040_TSHUTDET) {
dev_warn(&twl6040_dev->dev, dev_warn(twl6040->dev,
"Thermal shutdown, powering-off"); "Thermal shutdown, powering-off");
twl6040_power(twl6040, 0); twl6040_power(twl6040, 0);
} else { } else {
dev_warn(&twl6040_dev->dev, dev_warn(twl6040->dev,
"Leaving thermal shutdown, powering-on"); "Leaving thermal shutdown, powering-on");
twl6040_power(twl6040, 1); twl6040_power(twl6040, 1);
} }
...@@ -227,7 +225,7 @@ static int twl6040_power_up_completion(struct twl6040 *twl6040, ...@@ -227,7 +225,7 @@ static int twl6040_power_up_completion(struct twl6040 *twl6040,
if (!time_left) { if (!time_left) {
intid = twl6040_reg_read(twl6040, TWL6040_REG_INTID); intid = twl6040_reg_read(twl6040, TWL6040_REG_INTID);
if (!(intid & TWL6040_READYINT)) { if (!(intid & TWL6040_READYINT)) {
dev_err(&twl6040_dev->dev, dev_err(twl6040->dev,
"timeout waiting for READYINT\n"); "timeout waiting for READYINT\n");
return -ETIMEDOUT; return -ETIMEDOUT;
} }
...@@ -255,7 +253,7 @@ int twl6040_power(struct twl6040 *twl6040, int on) ...@@ -255,7 +253,7 @@ int twl6040_power(struct twl6040 *twl6040, int on)
/* wait for power-up completion */ /* wait for power-up completion */
ret = twl6040_power_up_completion(twl6040, naudint); ret = twl6040_power_up_completion(twl6040, naudint);
if (ret) { if (ret) {
dev_err(&twl6040_dev->dev, dev_err(twl6040->dev,
"automatic power-down failed\n"); "automatic power-down failed\n");
twl6040->power_count = 0; twl6040->power_count = 0;
goto out; goto out;
...@@ -264,7 +262,7 @@ int twl6040_power(struct twl6040 *twl6040, int on) ...@@ -264,7 +262,7 @@ int twl6040_power(struct twl6040 *twl6040, int on)
/* use manual power-up sequence */ /* use manual power-up sequence */
ret = twl6040_power_up(twl6040); ret = twl6040_power_up(twl6040);
if (ret) { if (ret) {
dev_err(&twl6040_dev->dev, dev_err(twl6040->dev,
"manual power-up failed\n"); "manual power-up failed\n");
twl6040->power_count = 0; twl6040->power_count = 0;
goto out; goto out;
...@@ -276,7 +274,7 @@ int twl6040_power(struct twl6040 *twl6040, int on) ...@@ -276,7 +274,7 @@ int twl6040_power(struct twl6040 *twl6040, int on)
} else { } else {
/* already powered-down */ /* already powered-down */
if (!twl6040->power_count) { if (!twl6040->power_count) {
dev_err(&twl6040_dev->dev, dev_err(twl6040->dev,
"device is already powered-off\n"); "device is already powered-off\n");
ret = -EPERM; ret = -EPERM;
goto out; goto out;
...@@ -326,7 +324,7 @@ int twl6040_set_pll(struct twl6040 *twl6040, int pll_id, ...@@ -326,7 +324,7 @@ int twl6040_set_pll(struct twl6040 *twl6040, int pll_id,
lppllctl &= ~TWL6040_LPLLFIN; lppllctl &= ~TWL6040_LPLLFIN;
break; break;
default: default:
dev_err(&twl6040_dev->dev, dev_err(twl6040->dev,
"freq_out %d not supported\n", freq_out); "freq_out %d not supported\n", freq_out);
ret = -EINVAL; ret = -EINVAL;
goto pll_out; goto pll_out;
...@@ -347,7 +345,7 @@ int twl6040_set_pll(struct twl6040 *twl6040, int pll_id, ...@@ -347,7 +345,7 @@ int twl6040_set_pll(struct twl6040 *twl6040, int pll_id,
hppllctl); hppllctl);
break; break;
default: default:
dev_err(&twl6040_dev->dev, dev_err(twl6040->dev,
"freq_in %d not supported\n", freq_in); "freq_in %d not supported\n", freq_in);
ret = -EINVAL; ret = -EINVAL;
goto pll_out; goto pll_out;
...@@ -356,7 +354,7 @@ int twl6040_set_pll(struct twl6040 *twl6040, int pll_id, ...@@ -356,7 +354,7 @@ int twl6040_set_pll(struct twl6040 *twl6040, int pll_id,
case TWL6040_SYSCLK_SEL_HPPLL: case TWL6040_SYSCLK_SEL_HPPLL:
/* high-performance PLL can provide only 19.2 MHz */ /* high-performance PLL can provide only 19.2 MHz */
if (freq_out != 19200000) { if (freq_out != 19200000) {
dev_err(&twl6040_dev->dev, dev_err(twl6040->dev,
"freq_out %d not supported\n", freq_out); "freq_out %d not supported\n", freq_out);
ret = -EINVAL; ret = -EINVAL;
goto pll_out; goto pll_out;
...@@ -389,7 +387,7 @@ int twl6040_set_pll(struct twl6040 *twl6040, int pll_id, ...@@ -389,7 +387,7 @@ int twl6040_set_pll(struct twl6040 *twl6040, int pll_id,
TWL6040_HPLLENA; TWL6040_HPLLENA;
break; break;
default: default:
dev_err(&twl6040_dev->dev, dev_err(twl6040->dev,
"freq_in %d not supported\n", freq_in); "freq_in %d not supported\n", freq_in);
ret = -EINVAL; ret = -EINVAL;
goto pll_out; goto pll_out;
...@@ -406,7 +404,7 @@ int twl6040_set_pll(struct twl6040 *twl6040, int pll_id, ...@@ -406,7 +404,7 @@ int twl6040_set_pll(struct twl6040 *twl6040, int pll_id,
twl6040_reg_write(twl6040, TWL6040_REG_LPPLLCTL, lppllctl); twl6040_reg_write(twl6040, TWL6040_REG_LPPLLCTL, lppllctl);
break; break;
default: default:
dev_err(&twl6040_dev->dev, "unknown pll id %d\n", pll_id); dev_err(twl6040->dev, "unknown pll id %d\n", pll_id);
ret = -EINVAL; ret = -EINVAL;
goto pll_out; goto pll_out;
} }
...@@ -471,7 +469,6 @@ static int __devinit twl6040_probe(struct platform_device *pdev) ...@@ -471,7 +469,6 @@ static int __devinit twl6040_probe(struct platform_device *pdev)
platform_set_drvdata(pdev, twl6040); platform_set_drvdata(pdev, twl6040);
twl6040_dev = pdev;
twl6040->dev = &pdev->dev; twl6040->dev = &pdev->dev;
twl6040->audpwron = pdata->audpwron_gpio; twl6040->audpwron = pdata->audpwron_gpio;
twl6040->irq = pdata->naudint_irq; twl6040->irq = pdata->naudint_irq;
...@@ -566,7 +563,6 @@ static int __devinit twl6040_probe(struct platform_device *pdev) ...@@ -566,7 +563,6 @@ static int __devinit twl6040_probe(struct platform_device *pdev)
gpio1_err: gpio1_err:
platform_set_drvdata(pdev, NULL); platform_set_drvdata(pdev, NULL);
kfree(twl6040); kfree(twl6040);
twl6040_dev = NULL;
return ret; return ret;
} }
...@@ -586,7 +582,6 @@ static int __devexit twl6040_remove(struct platform_device *pdev) ...@@ -586,7 +582,6 @@ static int __devexit twl6040_remove(struct platform_device *pdev)
mfd_remove_devices(&pdev->dev); mfd_remove_devices(&pdev->dev);
platform_set_drvdata(pdev, NULL); platform_set_drvdata(pdev, NULL);
kfree(twl6040); kfree(twl6040);
twl6040_dev = NULL;
return 0; return 0;
} }
......
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