Commit 61125591 authored by Linus Walleij's avatar Linus Walleij Committed by Dmitry Torokhov

mfd: tc3589x: enforce device-tree only mode

All systems using the TC3589x multifunction expander uses
devicetree, so don't clutter the place with a lot of
and assume it is there.
Signed-off-by: default avatarLinus Walleij <linus.walleij@linaro.org>
Acked-by: default avatarLee Jones <lee.jones@linaro.org>
Signed-off-by: default avatarDmitry Torokhov <dmitry.torokhov@gmail.com>
parent 8b6f53c2
...@@ -669,6 +669,7 @@ config GPIO_STP_XWAY ...@@ -669,6 +669,7 @@ config GPIO_STP_XWAY
config GPIO_TC3589X config GPIO_TC3589X
bool "TC3589X GPIOs" bool "TC3589X GPIOs"
depends on MFD_TC3589X depends on MFD_TC3589X
depends on OF_GPIO
select GPIOLIB_IRQCHIP select GPIOLIB_IRQCHIP
help help
This enables support for the GPIOs found on the TC3589X This enables support for the GPIOs found on the TC3589X
......
...@@ -260,10 +260,7 @@ static int tc3589x_gpio_probe(struct platform_device *pdev) ...@@ -260,10 +260,7 @@ static int tc3589x_gpio_probe(struct platform_device *pdev)
tc3589x_gpio->chip.ngpio = tc3589x->num_gpio; tc3589x_gpio->chip.ngpio = tc3589x->num_gpio;
tc3589x_gpio->chip.dev = &pdev->dev; tc3589x_gpio->chip.dev = &pdev->dev;
tc3589x_gpio->chip.base = -1; tc3589x_gpio->chip.base = -1;
#ifdef CONFIG_OF_GPIO
tc3589x_gpio->chip.of_node = np; tc3589x_gpio->chip.of_node = np;
#endif
/* Bring the GPIO module out of reset */ /* Bring the GPIO module out of reset */
ret = tc3589x_set_bits(tc3589x, TC3589x_RSTCTRL, ret = tc3589x_set_bits(tc3589x, TC3589x_RSTCTRL,
......
...@@ -318,7 +318,6 @@ static void tc3589x_keypad_close(struct input_dev *input) ...@@ -318,7 +318,6 @@ static void tc3589x_keypad_close(struct input_dev *input)
tc3589x_keypad_disable(keypad); tc3589x_keypad_disable(keypad);
} }
#ifdef CONFIG_OF
static const struct tc3589x_keypad_platform_data * static const struct tc3589x_keypad_platform_data *
tc3589x_keypad_of_probe(struct device *dev) tc3589x_keypad_of_probe(struct device *dev)
{ {
...@@ -368,14 +367,6 @@ tc3589x_keypad_of_probe(struct device *dev) ...@@ -368,14 +367,6 @@ tc3589x_keypad_of_probe(struct device *dev)
return plat; return plat;
} }
#else
static inline const struct tc3589x_keypad_platform_data *
tc3589x_keypad_of_probe(struct device *dev)
{
return ERR_PTR(-ENODEV);
}
#endif
static int tc3589x_keypad_probe(struct platform_device *pdev) static int tc3589x_keypad_probe(struct platform_device *pdev)
{ {
......
...@@ -1210,6 +1210,7 @@ config MFD_TIMBERDALE ...@@ -1210,6 +1210,7 @@ config MFD_TIMBERDALE
config MFD_TC3589X config MFD_TC3589X
bool "Toshiba TC35892 and variants" bool "Toshiba TC35892 and variants"
depends on I2C=y depends on I2C=y
depends on OF
select MFD_CORE select MFD_CORE
help help
Support for the Toshiba TC35892 and variants I/O Expander. Support for the Toshiba TC35892 and variants I/O Expander.
......
...@@ -318,7 +318,6 @@ static int tc3589x_device_init(struct tc3589x *tc3589x) ...@@ -318,7 +318,6 @@ static int tc3589x_device_init(struct tc3589x *tc3589x)
return ret; return ret;
} }
#ifdef CONFIG_OF
static const struct of_device_id tc3589x_match[] = { static const struct of_device_id tc3589x_match[] = {
/* Legacy compatible string */ /* Legacy compatible string */
{ .compatible = "tc3589x", .data = (void *) TC3589X_UNKNOWN }, { .compatible = "tc3589x", .data = (void *) TC3589X_UNKNOWN },
...@@ -359,14 +358,6 @@ tc3589x_of_probe(struct device *dev, enum tc3589x_version *version) ...@@ -359,14 +358,6 @@ tc3589x_of_probe(struct device *dev, enum tc3589x_version *version)
return pdata; return pdata;
} }
#else
static inline struct tc3589x_platform_data *
tc3589x_of_probe(struct device *dev, enum tc3589x_version *version)
{
dev_err(dev, "no device tree support\n");
return ERR_PTR(-ENODEV);
}
#endif
static int tc3589x_probe(struct i2c_client *i2c, static int tc3589x_probe(struct i2c_client *i2c,
const struct i2c_device_id *id) const struct i2c_device_id *id)
......
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