public inbox for linux-arm-kernel@lists.infradead.org
 help / color / mirror / Atom feed
* [PATCH 2/3] ARM: register character LCD for PB1176
@ 2010-07-06 15:21 Linus Walleij
  2010-07-15 11:56 ` Russell King - ARM Linux
  0 siblings, 1 reply; 3+ messages in thread
From: Linus Walleij @ 2010-07-06 15:21 UTC (permalink / raw)
  To: linux-arm-kernel

This registers the character LCD device for the ARM RealView
PB1176.

Signed-off-by: Linus Walleij <triad@df.lth.se>
---
This one is tested on the PB1176 and works fine.
---
 arch/arm/mach-realview/Kconfig           |    1 +
 arch/arm/mach-realview/realview_pb1176.c |   21 +++++++++++++++++++++
 2 files changed, 22 insertions(+), 0 deletions(-)

diff --git a/arch/arm/mach-realview/Kconfig b/arch/arm/mach-realview/Kconfig
index ee5e392..511d67d 100644
--- a/arch/arm/mach-realview/Kconfig
+++ b/arch/arm/mach-realview/Kconfig
@@ -44,6 +44,7 @@ config MACH_REALVIEW_PB1176
 	bool "Support RealView/PB1176 platform"
 	select CPU_V6
 	select ARM_GIC
+	select ARM_CHAR_LCD
 	help
 	  Include support for the ARM(R) RealView ARM1176 Platform Baseboard.
 
diff --git a/arch/arm/mach-realview/realview_pb1176.c b/arch/arm/mach-realview/realview_pb1176.c
index 9f02350..eb7979c 100644
--- a/arch/arm/mach-realview/realview_pb1176.c
+++ b/arch/arm/mach-realview/realview_pb1176.c
@@ -274,6 +274,26 @@ static struct platform_device pmu_device = {
 	.resource		= &pmu_resource,
 };
 
+static struct resource char_lcd_resources[] = {
+	{
+		.start = REALVIEW_CHAR_LCD_BASE,
+		.end   = (REALVIEW_CHAR_LCD_BASE + SZ_4K - 1),
+		.flags = IORESOURCE_MEM,
+	},
+	{
+		.start	= IRQ_PB1176_CHARLCD,
+		.end	= IRQ_PB1176_CHARLCD,
+		.flags	= IORESOURCE_IRQ,
+	},
+};
+
+static struct platform_device char_lcd_device = {
+	.name		=	"arm-charlcd",
+	.id		=	-1,
+	.num_resources	=	ARRAY_SIZE(char_lcd_resources),
+	.resource	=	char_lcd_resources,
+};
+
 static void __init gic_init_irq(void)
 {
 	/* ARM1176 DevChip GIC, primary */
@@ -336,6 +356,7 @@ static void __init realview_pb1176_init(void)
 	platform_device_register(&realview_i2c_device);
 	realview_usb_register(realview_pb1176_isp1761_resources);
 	platform_device_register(&pmu_device);
+	platform_device_register(&char_lcd_device);
 
 	for (i = 0; i < ARRAY_SIZE(amba_devs); i++) {
 		struct amba_device *d = amba_devs[i];
-- 
1.7.0.1

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

* [PATCH 2/3] ARM: register character LCD for PB1176
  2010-07-06 15:21 [PATCH 2/3] ARM: register character LCD for PB1176 Linus Walleij
@ 2010-07-15 11:56 ` Russell King - ARM Linux
  2010-07-15 13:47   ` Linus Walleij
  0 siblings, 1 reply; 3+ messages in thread
From: Russell King - ARM Linux @ 2010-07-15 11:56 UTC (permalink / raw)
  To: linux-arm-kernel

On Tue, Jul 06, 2010 at 05:21:18PM +0200, Linus Walleij wrote:
> This registers the character LCD device for the ARM RealView
> PB1176.

Shouldn't ARM_CHAR_LCD be a stand-alone option, rather than something
which is always built-in whenever one of these platforms is enabled?

If that link is broken, then these patches shouldn't depend on the first
and could be applied out of order.

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

* [PATCH 2/3] ARM: register character LCD for PB1176
  2010-07-15 11:56 ` Russell King - ARM Linux
@ 2010-07-15 13:47   ` Linus Walleij
  0 siblings, 0 replies; 3+ messages in thread
From: Linus Walleij @ 2010-07-15 13:47 UTC (permalink / raw)
  To: linux-arm-kernel

2010/7/15 Russell King - ARM Linux <linux@arm.linux.org.uk>:

> On Tue, Jul 06, 2010 at 05:21:18PM +0200, Linus Walleij wrote:
>> This registers the character LCD device for the ARM RealView
>> PB1176.
>
> Shouldn't ARM_CHAR_LCD be a stand-alone option, rather than something
> which is always built-in whenever one of these platforms is enabled?

OK I'll break that out when I move it to drivers/misc.

Yours,
Linus Walleij

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

end of thread, other threads:[~2010-07-15 13:47 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-07-06 15:21 [PATCH 2/3] ARM: register character LCD for PB1176 Linus Walleij
2010-07-15 11:56 ` Russell King - ARM Linux
2010-07-15 13:47   ` Linus Walleij

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox