* [PATCH 1/3] ARM: OMAP1: ams-delta: Drop board specific global GPIO numbers
2018-11-07 20:17 [PATCH 0/3] ARN: OMAP1: ams-delta: Post-GPIO-refresh cleanups Janusz Krzysztofik
@ 2018-11-07 20:17 ` Janusz Krzysztofik
2018-11-07 20:17 ` [PATCH 2/3] ARM: OMAP1: ams-delta: Drop unused symbols from the board header Janusz Krzysztofik
` (2 subsequent siblings)
3 siblings, 0 replies; 6+ messages in thread
From: Janusz Krzysztofik @ 2018-11-07 20:17 UTC (permalink / raw)
To: Tony Lindgren
Cc: Aaro Koskinen, Linus Walleij, linux-arm-kernel, linux-omap,
linux-gpio, linux-kernel, Janusz Krzysztofik
As all users of the board specific GPIO pins have been converted from
legacy integer-based to descriptor-based interface, there is no longer
a need to maintain statically assigned GPIO pin numbers. Drop support
for that.
Signed-off-by: Janusz Krzysztofik <jmkrzyszt@gmail.com>
---
arch/arm/mach-omap1/board-ams-delta.c | 3 ---
arch/arm/mach-omap1/board-ams-delta.h | 16 ----------------
2 files changed, 19 deletions(-)
diff --git a/arch/arm/mach-omap1/board-ams-delta.c b/arch/arm/mach-omap1/board-ams-delta.c
index 19e0c071d675..a6986a83a916 100644
--- a/arch/arm/mach-omap1/board-ams-delta.c
+++ b/arch/arm/mach-omap1/board-ams-delta.c
@@ -167,7 +167,6 @@ static struct omap_usb_config ams_delta_usb_config __initdata = {
.pins[0] = 2,
};
-#define LATCH1_GPIO_BASE 232
#define LATCH1_NGPIO 8
static struct resource latch1_resources[] = {
@@ -183,7 +182,6 @@ static struct resource latch1_resources[] = {
static struct bgpio_pdata latch1_pdata = {
.label = LATCH1_LABEL,
- .base = LATCH1_GPIO_BASE,
.ngpio = LATCH1_NGPIO,
};
@@ -219,7 +217,6 @@ static struct resource latch2_resources[] = {
static struct bgpio_pdata latch2_pdata = {
.label = LATCH2_LABEL,
- .base = AMS_DELTA_LATCH2_GPIO_BASE,
.ngpio = AMS_DELTA_LATCH2_NGPIO,
};
diff --git a/arch/arm/mach-omap1/board-ams-delta.h b/arch/arm/mach-omap1/board-ams-delta.h
index a74a306d7e77..06e4c64a47f8 100644
--- a/arch/arm/mach-omap1/board-ams-delta.h
+++ b/arch/arm/mach-omap1/board-ams-delta.h
@@ -41,22 +41,6 @@
#define AMS_DELTA_GPIO_PIN_CONFIG 11
#define AMS_DELTA_GPIO_PIN_NAND_RB 12
-#define AMS_DELTA_GPIO_PIN_LCD_VBLEN 240
-#define AMS_DELTA_GPIO_PIN_LCD_NDISP 241
-#define AMS_DELTA_GPIO_PIN_NAND_NCE 242
-#define AMS_DELTA_GPIO_PIN_NAND_NRE 243
-#define AMS_DELTA_GPIO_PIN_NAND_NWP 244
-#define AMS_DELTA_GPIO_PIN_NAND_NWE 245
-#define AMS_DELTA_GPIO_PIN_NAND_ALE 246
-#define AMS_DELTA_GPIO_PIN_NAND_CLE 247
-#define AMS_DELTA_GPIO_PIN_KEYBRD_PWR 248
-#define AMS_DELTA_GPIO_PIN_KEYBRD_DATAOUT 249
-#define AMS_DELTA_GPIO_PIN_SCARD_RSTIN 250
-#define AMS_DELTA_GPIO_PIN_SCARD_CMDVCC 251
-#define AMS_DELTA_GPIO_PIN_MODEM_NRESET 252
-#define AMS_DELTA_GPIO_PIN_MODEM_CODEC 253
-
-#define AMS_DELTA_LATCH2_GPIO_BASE AMS_DELTA_GPIO_PIN_LCD_VBLEN
#define AMS_DELTA_LATCH2_NGPIO 16
#endif /* CONFIG_MACH_AMS_DELTA */
--
2.16.4
^ permalink raw reply related [flat|nested] 6+ messages in thread* [PATCH 2/3] ARM: OMAP1: ams-delta: Drop unused symbols from the board header
2018-11-07 20:17 [PATCH 0/3] ARN: OMAP1: ams-delta: Post-GPIO-refresh cleanups Janusz Krzysztofik
2018-11-07 20:17 ` [PATCH 1/3] ARM: OMAP1: ams-delta: Drop board specific global GPIO numbers Janusz Krzysztofik
@ 2018-11-07 20:17 ` Janusz Krzysztofik
2018-11-07 20:17 ` [PATCH 3/3] ARM: OMAP1: ams-delta: Move AMS_DELTA_LATCH2_NGPIO to the board file Janusz Krzysztofik
2018-11-15 10:45 ` [PATCH 0/3] ARN: OMAP1: ams-delta: Post-GPIO-refresh cleanups Linus Walleij
3 siblings, 0 replies; 6+ messages in thread
From: Janusz Krzysztofik @ 2018-11-07 20:17 UTC (permalink / raw)
To: Tony Lindgren
Cc: Aaro Koskinen, Linus Walleij, linux-arm-kernel, linux-omap,
linux-gpio, linux-kernel, Janusz Krzysztofik
Those bitmap symbols defining pins of latch2 register, used with
read()/write() calls before the latch was converted to a GPIO device,
have been obsoleted by integer symbols defined inside the board file.
Signed-off-by: Janusz Krzysztofik <jmkrzyszt@gmail.com>
---
arch/arm/mach-omap1/board-ams-delta.h | 4 ----
1 file changed, 4 deletions(-)
diff --git a/arch/arm/mach-omap1/board-ams-delta.h b/arch/arm/mach-omap1/board-ams-delta.h
index 06e4c64a47f8..2c3aff824cf0 100644
--- a/arch/arm/mach-omap1/board-ams-delta.h
+++ b/arch/arm/mach-omap1/board-ams-delta.h
@@ -28,10 +28,6 @@
#if defined (CONFIG_MACH_AMS_DELTA)
-#define AMD_DELTA_LATCH2_SCARD_RSTIN 0x0400
-#define AMD_DELTA_LATCH2_SCARD_CMDVCC 0x0800
-#define AMS_DELTA_LATCH2_MODEM_CODEC 0x2000
-
#define AMS_DELTA_GPIO_PIN_KEYBRD_DATA 0
#define AMS_DELTA_GPIO_PIN_KEYBRD_CLK 1
#define AMS_DELTA_GPIO_PIN_MODEM_IRQ 2
--
2.16.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 3/3] ARM: OMAP1: ams-delta: Move AMS_DELTA_LATCH2_NGPIO to the board file
2018-11-07 20:17 [PATCH 0/3] ARN: OMAP1: ams-delta: Post-GPIO-refresh cleanups Janusz Krzysztofik
2018-11-07 20:17 ` [PATCH 1/3] ARM: OMAP1: ams-delta: Drop board specific global GPIO numbers Janusz Krzysztofik
2018-11-07 20:17 ` [PATCH 2/3] ARM: OMAP1: ams-delta: Drop unused symbols from the board header Janusz Krzysztofik
@ 2018-11-07 20:17 ` Janusz Krzysztofik
2018-11-15 10:45 ` [PATCH 0/3] ARN: OMAP1: ams-delta: Post-GPIO-refresh cleanups Linus Walleij
3 siblings, 0 replies; 6+ messages in thread
From: Janusz Krzysztofik @ 2018-11-07 20:17 UTC (permalink / raw)
To: Tony Lindgren
Cc: Aaro Koskinen, Linus Walleij, linux-arm-kernel, linux-omap,
linux-gpio, linux-kernel, Janusz Krzysztofik
That symbol is not used outside the board file, there is no need to
keep it in the board header.
Signed-off-by: Janusz Krzysztofik <jmkrzyszt@gmail.com>
---
arch/arm/mach-omap1/board-ams-delta.c | 6 ++++--
arch/arm/mach-omap1/board-ams-delta.h | 2 --
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/arch/arm/mach-omap1/board-ams-delta.c b/arch/arm/mach-omap1/board-ams-delta.c
index a6986a83a916..1947bc63074e 100644
--- a/arch/arm/mach-omap1/board-ams-delta.c
+++ b/arch/arm/mach-omap1/board-ams-delta.c
@@ -204,11 +204,13 @@ static struct platform_device latch1_gpio_device = {
#define LATCH1_PIN_DOCKIT1 6
#define LATCH1_PIN_DOCKIT2 7
+#define LATCH2_NGPIO 16
+
static struct resource latch2_resources[] = {
[0] = {
.name = "dat",
.start = LATCH2_PHYS,
- .end = LATCH2_PHYS + (AMS_DELTA_LATCH2_NGPIO - 1) / 8,
+ .end = LATCH2_PHYS + (LATCH2_NGPIO - 1) / 8,
.flags = IORESOURCE_MEM,
},
};
@@ -217,7 +219,7 @@ static struct resource latch2_resources[] = {
static struct bgpio_pdata latch2_pdata = {
.label = LATCH2_LABEL,
- .ngpio = AMS_DELTA_LATCH2_NGPIO,
+ .ngpio = LATCH2_NGPIO,
};
static struct platform_device latch2_gpio_device = {
diff --git a/arch/arm/mach-omap1/board-ams-delta.h b/arch/arm/mach-omap1/board-ams-delta.h
index 2c3aff824cf0..b5c4a373b905 100644
--- a/arch/arm/mach-omap1/board-ams-delta.h
+++ b/arch/arm/mach-omap1/board-ams-delta.h
@@ -37,8 +37,6 @@
#define AMS_DELTA_GPIO_PIN_CONFIG 11
#define AMS_DELTA_GPIO_PIN_NAND_RB 12
-#define AMS_DELTA_LATCH2_NGPIO 16
-
#endif /* CONFIG_MACH_AMS_DELTA */
#endif /* __ASM_ARCH_OMAP_AMS_DELTA_H */
--
2.16.4
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH 0/3] ARN: OMAP1: ams-delta: Post-GPIO-refresh cleanups
2018-11-07 20:17 [PATCH 0/3] ARN: OMAP1: ams-delta: Post-GPIO-refresh cleanups Janusz Krzysztofik
` (2 preceding siblings ...)
2018-11-07 20:17 ` [PATCH 3/3] ARM: OMAP1: ams-delta: Move AMS_DELTA_LATCH2_NGPIO to the board file Janusz Krzysztofik
@ 2018-11-15 10:45 ` Linus Walleij
2018-11-29 20:05 ` Tony Lindgren
3 siblings, 1 reply; 6+ messages in thread
From: Linus Walleij @ 2018-11-15 10:45 UTC (permalink / raw)
To: Janusz Krzysztofik
Cc: ext Tony Lindgren, Aaro Koskinen, Linux ARM, Linux-OMAP,
open list:GPIO SUBSYSTEM, linux-kernel
On Wed, Nov 7, 2018 at 9:16 PM Janusz Krzysztofik <jmkrzyszt@gmail.com> wrote:
> Janusz Krzysztofik (3):
> ARM: OMAP1: ams-delta: Drop board specific global GPIO numbers
> ARM: OMAP1: ams-delta: Drop unused symbols from the board header
> ARM: OMAP1: ams-delta: Move AMS_DELTA_LATCH2_NGPIO to the board file
This series:
Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH 0/3] ARN: OMAP1: ams-delta: Post-GPIO-refresh cleanups
2018-11-15 10:45 ` [PATCH 0/3] ARN: OMAP1: ams-delta: Post-GPIO-refresh cleanups Linus Walleij
@ 2018-11-29 20:05 ` Tony Lindgren
0 siblings, 0 replies; 6+ messages in thread
From: Tony Lindgren @ 2018-11-29 20:05 UTC (permalink / raw)
To: Linus Walleij
Cc: Janusz Krzysztofik, Aaro Koskinen, Linux ARM, Linux-OMAP,
open list:GPIO SUBSYSTEM, linux-kernel
* Linus Walleij <linus.walleij@linaro.org> [181115 10:46]:
> On Wed, Nov 7, 2018 at 9:16 PM Janusz Krzysztofik <jmkrzyszt@gmail.com> wrote:
>
> > Janusz Krzysztofik (3):
> > ARM: OMAP1: ams-delta: Drop board specific global GPIO numbers
> > ARM: OMAP1: ams-delta: Drop unused symbols from the board header
> > ARM: OMAP1: ams-delta: Move AMS_DELTA_LATCH2_NGPIO to the board file
>
> This series:
> Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
Applying into omap-for-v4.21/omap1 thanks.
Tony
^ permalink raw reply [flat|nested] 6+ messages in thread