linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/3] ARN: OMAP1: ams-delta: Post-GPIO-refresh cleanups
@ 2018-11-07 20:17 Janusz Krzysztofik
  2018-11-07 20:17 ` [PATCH 1/3] ARM: OMAP1: ams-delta: Drop board specific global GPIO numbers Janusz Krzysztofik
                   ` (3 more replies)
  0 siblings, 4 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


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

 board-ams-delta.c |    9 ++++-----
 board-ams-delta.h |   22 ----------------------
 2 files changed, 4 insertions(+), 27 deletions(-)


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [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

end of thread, other threads:[~2018-11-29 20:05 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
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 ` [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
2018-11-29 20:05   ` Tony Lindgren

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).