[PATCH 2/3] mfd: ab8500: delete all GPIO platform data instances

[PATCH 2/3] mfd: ab8500: delete all GPIO platform data instances

[Linus Walleij]

This deletes all instances where the AB8500 GPIO platform
data is passed around. It is completely unused in the kernel
now, so it does not hurt anyone.

Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
---
Sam, Lee: requesting an ACK to take this through either the
pinctrl tree or the ARM SoC tree if I build up more of these
pdata removal patches for abx500.
---
 include/linux/mfd/abx500/ab8500-gpio.h | 10 ----------
 include/linux/mfd/abx500/ab8500.h      |  2 --
 2 files changed, 12 deletions(-)

diff --git a/include/linux/mfd/abx500/ab8500-gpio.h b/include/linux/mfd/abx500/ab8500-gpio.h
index 172b2f201ae0..ebef068cbb6f 100644
--- a/include/linux/mfd/abx500/ab8500-gpio.h
+++ b/include/linux/mfd/abx500/ab8500-gpio.h
@@ -8,16 +8,6 @@
 #ifndef _AB8500_GPIO_H
 #define _AB8500_GPIO_H
 
-/*
- * Platform data to register a block: only the initial gpio/irq number.
- * Array sizes are large enough to contain all AB8500 and AB9540 GPIO
- * registers.
- */
-
-struct abx500_gpio_platform_data {
-	int gpio_base;
-};
-
 enum abx500_gpio_pull_updown {
 	ABX500_GPIO_PULL_DOWN = 0x0,
 	ABX500_GPIO_PULL_NONE = 0x1,
diff --git a/include/linux/mfd/abx500/ab8500.h b/include/linux/mfd/abx500/ab8500.h
index f4acd898dac9..a86ca1406fb8 100644
--- a/include/linux/mfd/abx500/ab8500.h
+++ b/include/linux/mfd/abx500/ab8500.h
@@ -368,7 +368,6 @@ struct ab8500 {
 };
 
 struct ab8500_regulator_platform_data;
-struct ab8500_gpio_platform_data;
 struct ab8500_codec_platform_data;
 struct ab8500_sysctrl_platform_data;
 
@@ -382,7 +381,6 @@ struct ab8500_platform_data {
 	int irq_base;
 	void (*init) (struct ab8500 *);
 	struct ab8500_regulator_platform_data *regulator;
-	struct abx500_gpio_platform_data *gpio;
 	struct ab8500_codec_platform_data *codec;
 	struct ab8500_sysctrl_platform_data *sysctrl;
 };
-- 
1.8.3.1

Re: [PATCH 2/3] mfd: ab8500: delete all GPIO platform data instances

[Lee Jones]

On Tue, 03 Dec 2013, Linus Walleij wrote:

> This deletes all instances where the AB8500 GPIO platform
> data is passed around. It is completely unused in the kernel
> now, so it does not hurt anyone.
> 
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> ---
> Sam, Lee: requesting an ACK to take this through either the
> pinctrl tree or the ARM SoC tree if I build up more of these
> pdata removal patches for abx500.

If it's completely unused, why does it matter which tree it goes
through?

> ---
>  include/linux/mfd/abx500/ab8500-gpio.h | 10 ----------
>  include/linux/mfd/abx500/ab8500.h      |  2 --
>  2 files changed, 12 deletions(-)

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

Re: [PATCH 2/3] mfd: ab8500: delete all GPIO platform data instances

[Lee Jones]

On Tue, 03 Dec 2013, Lee Jones wrote:

> On Tue, 03 Dec 2013, Linus Walleij wrote:
> 
> > This deletes all instances where the AB8500 GPIO platform
> > data is passed around. It is completely unused in the kernel
> > now, so it does not hurt anyone.
> > 
> > Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> > ---
> > Sam, Lee: requesting an ACK to take this through either the
> > pinctrl tree or the ARM SoC tree if I build up more of these
> > pdata removal patches for abx500.
> 
> If it's completely unused, why does it matter which tree it goes
> through?

Ah, I see, sorry!

Yes, you can have your Ack, but any MFD patches which go through must
conform to the MFD rules, so get that $SUBJECT line capitalised. ;)

After $SUBJECT line conformity:
  Acked-by: Lee Jones <lee.jones@linaro.org>

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog