Commit 9caac886 authored by Andy Shevchenko's avatar Andy Shevchenko Committed by Lee Jones

mfd: intel_quark_i2c_gpio: Switch to use struct device *

There is no need to pass struct pci_dev * to intel_quark_register_i2c_clk() and
intel_quark_unregister_i2c_clk(). Change the parameter to struct device *. As a
result store it in the private struct instead of struct pci_dev *.
Signed-off-by: default avatarAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
parent 7f0c5ae1
...@@ -53,7 +53,7 @@ ...@@ -53,7 +53,7 @@
#define INTEL_QUARK_I2C_CLK_HZ 33000000 #define INTEL_QUARK_I2C_CLK_HZ 33000000
struct intel_quark_mfd { struct intel_quark_mfd {
struct pci_dev *pdev; struct device *dev;
struct clk *i2c_clk; struct clk *i2c_clk;
struct clk_lookup *i2c_clk_lookup; struct clk_lookup *i2c_clk_lookup;
}; };
...@@ -123,12 +123,12 @@ static const struct pci_device_id intel_quark_mfd_ids[] = { ...@@ -123,12 +123,12 @@ static const struct pci_device_id intel_quark_mfd_ids[] = {
}; };
MODULE_DEVICE_TABLE(pci, intel_quark_mfd_ids); MODULE_DEVICE_TABLE(pci, intel_quark_mfd_ids);
static int intel_quark_register_i2c_clk(struct intel_quark_mfd *quark_mfd) static int intel_quark_register_i2c_clk(struct device *dev)
{ {
struct pci_dev *pdev = quark_mfd->pdev; struct intel_quark_mfd *quark_mfd = dev_get_drvdata(dev);
struct clk *i2c_clk; struct clk *i2c_clk;
i2c_clk = clk_register_fixed_rate(&pdev->dev, i2c_clk = clk_register_fixed_rate(dev,
INTEL_QUARK_I2C_CONTROLLER_CLK, NULL, INTEL_QUARK_I2C_CONTROLLER_CLK, NULL,
CLK_IS_ROOT, INTEL_QUARK_I2C_CLK_HZ); CLK_IS_ROOT, INTEL_QUARK_I2C_CLK_HZ);
if (IS_ERR(i2c_clk)) if (IS_ERR(i2c_clk))
...@@ -140,16 +140,16 @@ static int intel_quark_register_i2c_clk(struct intel_quark_mfd *quark_mfd) ...@@ -140,16 +140,16 @@ static int intel_quark_register_i2c_clk(struct intel_quark_mfd *quark_mfd)
if (!quark_mfd->i2c_clk_lookup) { if (!quark_mfd->i2c_clk_lookup) {
clk_unregister(quark_mfd->i2c_clk); clk_unregister(quark_mfd->i2c_clk);
dev_err(&pdev->dev, "Fixed clk register failed\n"); dev_err(dev, "Fixed clk register failed\n");
return -ENOMEM; return -ENOMEM;
} }
return 0; return 0;
} }
static void intel_quark_unregister_i2c_clk(struct pci_dev *pdev) static void intel_quark_unregister_i2c_clk(struct device *dev)
{ {
struct intel_quark_mfd *quark_mfd = dev_get_drvdata(&pdev->dev); struct intel_quark_mfd *quark_mfd = dev_get_drvdata(dev);
if (!quark_mfd->i2c_clk_lookup) if (!quark_mfd->i2c_clk_lookup)
return; return;
...@@ -248,10 +248,10 @@ static int intel_quark_mfd_probe(struct pci_dev *pdev, ...@@ -248,10 +248,10 @@ static int intel_quark_mfd_probe(struct pci_dev *pdev,
if (!quark_mfd) if (!quark_mfd)
return -ENOMEM; return -ENOMEM;
quark_mfd->pdev = pdev; quark_mfd->dev = &pdev->dev;
dev_set_drvdata(&pdev->dev, quark_mfd); dev_set_drvdata(&pdev->dev, quark_mfd);
ret = intel_quark_register_i2c_clk(quark_mfd); ret = intel_quark_register_i2c_clk(&pdev->dev);
if (ret) if (ret)
return ret; return ret;
...@@ -272,13 +272,13 @@ static int intel_quark_mfd_probe(struct pci_dev *pdev, ...@@ -272,13 +272,13 @@ static int intel_quark_mfd_probe(struct pci_dev *pdev,
return 0; return 0;
err_unregister_i2c_clk: err_unregister_i2c_clk:
intel_quark_unregister_i2c_clk(pdev); intel_quark_unregister_i2c_clk(&pdev->dev);
return ret; return ret;
} }
static void intel_quark_mfd_remove(struct pci_dev *pdev) static void intel_quark_mfd_remove(struct pci_dev *pdev)
{ {
intel_quark_unregister_i2c_clk(pdev); intel_quark_unregister_i2c_clk(&pdev->dev);
mfd_remove_devices(&pdev->dev); mfd_remove_devices(&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