Commit d27ad7a8 authored by Linus Walleij's avatar Linus Walleij

gpio: syscon: use gpiochip data pointer

This makes the driver use the data pointer added to the gpio_chip
to store a pointer to the state container instead of relying on
container_of().

Cc: Alexander Shiyan <shc_work@mail.ru>
Signed-off-by: default avatarLinus Walleij <linus.walleij@linaro.org>
parent 5b90b8c2
...@@ -59,14 +59,9 @@ struct syscon_gpio_priv { ...@@ -59,14 +59,9 @@ struct syscon_gpio_priv {
u32 dir_reg_offset; u32 dir_reg_offset;
}; };
static inline struct syscon_gpio_priv *to_syscon_gpio(struct gpio_chip *chip)
{
return container_of(chip, struct syscon_gpio_priv, chip);
}
static int syscon_gpio_get(struct gpio_chip *chip, unsigned offset) static int syscon_gpio_get(struct gpio_chip *chip, unsigned offset)
{ {
struct syscon_gpio_priv *priv = to_syscon_gpio(chip); struct syscon_gpio_priv *priv = gpiochip_get_data(chip);
unsigned int val, offs; unsigned int val, offs;
int ret; int ret;
...@@ -82,7 +77,7 @@ static int syscon_gpio_get(struct gpio_chip *chip, unsigned offset) ...@@ -82,7 +77,7 @@ static int syscon_gpio_get(struct gpio_chip *chip, unsigned offset)
static void syscon_gpio_set(struct gpio_chip *chip, unsigned offset, int val) static void syscon_gpio_set(struct gpio_chip *chip, unsigned offset, int val)
{ {
struct syscon_gpio_priv *priv = to_syscon_gpio(chip); struct syscon_gpio_priv *priv = gpiochip_get_data(chip);
unsigned int offs; unsigned int offs;
offs = priv->dreg_offset + priv->data->dat_bit_offset + offset; offs = priv->dreg_offset + priv->data->dat_bit_offset + offset;
...@@ -95,7 +90,7 @@ static void syscon_gpio_set(struct gpio_chip *chip, unsigned offset, int val) ...@@ -95,7 +90,7 @@ static void syscon_gpio_set(struct gpio_chip *chip, unsigned offset, int val)
static int syscon_gpio_dir_in(struct gpio_chip *chip, unsigned offset) static int syscon_gpio_dir_in(struct gpio_chip *chip, unsigned offset)
{ {
struct syscon_gpio_priv *priv = to_syscon_gpio(chip); struct syscon_gpio_priv *priv = gpiochip_get_data(chip);
if (priv->data->flags & GPIO_SYSCON_FEAT_DIR) { if (priv->data->flags & GPIO_SYSCON_FEAT_DIR) {
unsigned int offs; unsigned int offs;
...@@ -113,7 +108,7 @@ static int syscon_gpio_dir_in(struct gpio_chip *chip, unsigned offset) ...@@ -113,7 +108,7 @@ static int syscon_gpio_dir_in(struct gpio_chip *chip, unsigned offset)
static int syscon_gpio_dir_out(struct gpio_chip *chip, unsigned offset, int val) static int syscon_gpio_dir_out(struct gpio_chip *chip, unsigned offset, int val)
{ {
struct syscon_gpio_priv *priv = to_syscon_gpio(chip); struct syscon_gpio_priv *priv = gpiochip_get_data(chip);
if (priv->data->flags & GPIO_SYSCON_FEAT_DIR) { if (priv->data->flags & GPIO_SYSCON_FEAT_DIR) {
unsigned int offs; unsigned int offs;
...@@ -144,7 +139,7 @@ static const struct syscon_gpio_data clps711x_mctrl_gpio = { ...@@ -144,7 +139,7 @@ static const struct syscon_gpio_data clps711x_mctrl_gpio = {
static void keystone_gpio_set(struct gpio_chip *chip, unsigned offset, int val) static void keystone_gpio_set(struct gpio_chip *chip, unsigned offset, int val)
{ {
struct syscon_gpio_priv *priv = to_syscon_gpio(chip); struct syscon_gpio_priv *priv = gpiochip_get_data(chip);
unsigned int offs; unsigned int offs;
int ret; int ret;
...@@ -243,7 +238,7 @@ static int syscon_gpio_probe(struct platform_device *pdev) ...@@ -243,7 +238,7 @@ static int syscon_gpio_probe(struct platform_device *pdev)
platform_set_drvdata(pdev, priv); platform_set_drvdata(pdev, priv);
return gpiochip_add(&priv->chip); return gpiochip_add_data(&priv->chip, priv);
} }
static int syscon_gpio_remove(struct platform_device *pdev) static int syscon_gpio_remove(struct platform_device *pdev)
......
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