[APPLIED] [PATCH v3] OMAP1: AMS_DELTA: add modem support

2009-08-10 Thread Tony Lindgren
This patch has been applied to the linux-omap
by youw fwiendly patch wobot.

Branch in linux-omap: omap1-upstream

Initial commit ID (Likely to change): 92c79d20ca55625ad36ee79ec1356558a3aeb061

PatchWorks
http://patchwork.kernel.org/patch/34220/

Git (Likely to change, and takes a while to get mirrored)
http://git.kernel.org/?p=linux/kernel/git/tmlind/linux-omap-2.6.git;a=commit;h=92c79d20ca55625ad36ee79ec1356558a3aeb061


--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v3] OMAP1: AMS_DELTA: add modem support

2009-08-10 Thread Tony Lindgren
* Jonathan McDowell  [090810 13:01]:
> On Mon, Aug 10, 2009 at 11:12:48AM +0200, Janusz Krzysztofik wrote:
> > Jonathan, Tony,
> > 
> > Any chance for this patch ready for 2.6.32 inclusion? Or maybe v2,
> > that has no external dependencies and can be updated later to v3 when
> > Vikram's changes get accepted upstream?
> 
> Vikram's patch is in linux-next, so there shouldn't be a problem with
> this version of your patch for 2.6.32. I haven't been able to test it
> myself (though I hope to next week), but it looks ok from a brief
> inspection, so:
> 
> Acked-By: Jonathan McDowell 

Looks good to me, will queue it to my omap1-upstream shortly.

Tony

>  
> > Monday 06 July 2009 12:24:27 Janusz Krzysztofik wrote:
> > > This patch adds support for modem device found on Amstrad E3 (Delta) 
> > > board.
> > >
> > > Based on earlier patch by Jonathan McDowell, available at
> > > http://the.earth.li/pub/e3/2.6.19/ams-delta-modem.patch.
> > > Modified after Ladislav Michl's arch/arm/mach-omap1/board-voiceblue.c.
> > >
> > > This version, unlike v2, is dependent on 8250 driver changes getting
> > > accepted upstream:
> > > Refer: http://patchwork.kernel.org/patch/31884/
> > >
> > > Signed-off-by: Janusz Krzysztofik 
> > > ---
> > > Registering the modem platform device from a separate arch_initcall (and
> > > not from init_machine() callback) preserves serial console port at ttyS0.
> > >
> > > OMAP mux pin set up just for case if Amstrad bootloader is ever replaced.
> > >
> > > Warning:
> > > could cause compilation break if 8250 patch is not integrated first
> > >
> > > --- linux-2.6.31-rc1/arch/arm/mach-omap1/board-ams-delta.c.orig   
> > > 2009-07-04
> > > 16:39:37.0 +0200 +++
> > > linux-2.6.31-rc1/arch/arm/mach-omap1/board-ams-delta.c2009-07-05
> > > 13:41:08.0 +0200 @@ -15,8 +15,11 @@
> > >  #include 
> > >  #include 
> > >  #include 
> > > +#include 
> > >  #include 
> > > +#include 
> > >
> > > +#include 
> > >  #include 
> > >  #include 
> > >  #include 
> > > @@ -235,6 +238,41 @@ static void __init ams_delta_init(void)
> > >   platform_add_devices(ams_delta_devices, ARRAY_SIZE(ams_delta_devices));
> > >  }
> > >
> > > +static struct plat_serial8250_port ams_delta_modem_ports[] = {
> > > + {
> > > + .membase= (void *) AMS_DELTA_MODEM_VIRT,
> > > + .mapbase= AMS_DELTA_MODEM_PHYS,
> > > + .irq= -EINVAL, /* changed later */
> > > + .flags  = UPF_BOOT_AUTOCONF,
> > > + .irqflags   = IRQF_TRIGGER_RISING,
> > > + .iotype = UPIO_MEM,
> > > + .regshift   = 1,
> > > + .uartclk= BASE_BAUD * 16,
> > > + },
> > > + { },
> > > +};
> > > +
> > > +static struct platform_device ams_delta_modem_device = {
> > > + .name   = "serial8250",
> > > + .id = PLAT8250_DEV_PLATFORM1,
> > > + .dev= {
> > > + .platform_data = ams_delta_modem_ports,
> > > + },
> > > +};
> > > +
> > > +static int __init ams_delta_modem_init(void)
> > > +{
> > > + omap_cfg_reg(M14_1510_GPIO2);
> > > + ams_delta_modem_ports[0].irq = gpio_to_irq(2);
> > > +
> > > + ams_delta_latch2_write(
> > > + AMS_DELTA_LATCH2_MODEM_NRESET | AMS_DELTA_LATCH2_MODEM_CODEC,
> > > + AMS_DELTA_LATCH2_MODEM_NRESET | AMS_DELTA_LATCH2_MODEM_CODEC);
> > > +
> > > + return platform_device_register(&ams_delta_modem_device);
> > > +}
> > > +arch_initcall(ams_delta_modem_init);
> > > +
> > >  static void __init ams_delta_map_io(void)
> > >  {
> > >   omap1_map_common_io();
> > 
> > 
> 
> J.
> 
> -- 
> Revd. Jonathan McDowell, ULC | I am a passenger.
> 
> ---
> List admin: http://lists.arm.linux.org.uk/mailman/listinfo/linux-arm-kernel
> FAQ:http://www.arm.linux.org.uk/mailinglists/faq.php
> Etiquette:  http://www.arm.linux.org.uk/mailinglists/etiquette.php
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v3] OMAP1: AMS_DELTA: add modem support

2009-08-10 Thread Jonathan McDowell
On Mon, Aug 10, 2009 at 11:12:48AM +0200, Janusz Krzysztofik wrote:
> Jonathan, Tony,
> 
> Any chance for this patch ready for 2.6.32 inclusion? Or maybe v2,
> that has no external dependencies and can be updated later to v3 when
> Vikram's changes get accepted upstream?

Vikram's patch is in linux-next, so there shouldn't be a problem with
this version of your patch for 2.6.32. I haven't been able to test it
myself (though I hope to next week), but it looks ok from a brief
inspection, so:

Acked-By: Jonathan McDowell 
 
> Monday 06 July 2009 12:24:27 Janusz Krzysztofik wrote:
> > This patch adds support for modem device found on Amstrad E3 (Delta) board.
> >
> > Based on earlier patch by Jonathan McDowell, available at
> > http://the.earth.li/pub/e3/2.6.19/ams-delta-modem.patch.
> > Modified after Ladislav Michl's arch/arm/mach-omap1/board-voiceblue.c.
> >
> > This version, unlike v2, is dependent on 8250 driver changes getting
> > accepted upstream:
> > Refer: http://patchwork.kernel.org/patch/31884/
> >
> > Signed-off-by: Janusz Krzysztofik 
> > ---
> > Registering the modem platform device from a separate arch_initcall (and
> > not from init_machine() callback) preserves serial console port at ttyS0.
> >
> > OMAP mux pin set up just for case if Amstrad bootloader is ever replaced.
> >
> > Warning:
> > could cause compilation break if 8250 patch is not integrated first
> >
> > --- linux-2.6.31-rc1/arch/arm/mach-omap1/board-ams-delta.c.orig 
> > 2009-07-04
> > 16:39:37.0 +0200 +++
> > linux-2.6.31-rc1/arch/arm/mach-omap1/board-ams-delta.c  2009-07-05
> > 13:41:08.0 +0200 @@ -15,8 +15,11 @@
> >  #include 
> >  #include 
> >  #include 
> > +#include 
> >  #include 
> > +#include 
> >
> > +#include 
> >  #include 
> >  #include 
> >  #include 
> > @@ -235,6 +238,41 @@ static void __init ams_delta_init(void)
> > platform_add_devices(ams_delta_devices, ARRAY_SIZE(ams_delta_devices));
> >  }
> >
> > +static struct plat_serial8250_port ams_delta_modem_ports[] = {
> > +   {
> > +   .membase= (void *) AMS_DELTA_MODEM_VIRT,
> > +   .mapbase= AMS_DELTA_MODEM_PHYS,
> > +   .irq= -EINVAL, /* changed later */
> > +   .flags  = UPF_BOOT_AUTOCONF,
> > +   .irqflags   = IRQF_TRIGGER_RISING,
> > +   .iotype = UPIO_MEM,
> > +   .regshift   = 1,
> > +   .uartclk= BASE_BAUD * 16,
> > +   },
> > +   { },
> > +};
> > +
> > +static struct platform_device ams_delta_modem_device = {
> > +   .name   = "serial8250",
> > +   .id = PLAT8250_DEV_PLATFORM1,
> > +   .dev= {
> > +   .platform_data = ams_delta_modem_ports,
> > +   },
> > +};
> > +
> > +static int __init ams_delta_modem_init(void)
> > +{
> > +   omap_cfg_reg(M14_1510_GPIO2);
> > +   ams_delta_modem_ports[0].irq = gpio_to_irq(2);
> > +
> > +   ams_delta_latch2_write(
> > +   AMS_DELTA_LATCH2_MODEM_NRESET | AMS_DELTA_LATCH2_MODEM_CODEC,
> > +   AMS_DELTA_LATCH2_MODEM_NRESET | AMS_DELTA_LATCH2_MODEM_CODEC);
> > +
> > +   return platform_device_register(&ams_delta_modem_device);
> > +}
> > +arch_initcall(ams_delta_modem_init);
> > +
> >  static void __init ams_delta_map_io(void)
> >  {
> > omap1_map_common_io();
> 
> 

J.

-- 
Revd. Jonathan McDowell, ULC | I am a passenger.
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v3] OMAP1: AMS_DELTA: add modem support

2009-08-10 Thread Janusz Krzysztofik
Jonathan, Tony,

Any chance for this patch ready for 2.6.32 inclusion? Or maybe v2, that has no 
external dependencies and can be updated later to v3 when Vikram's changes 
get accepted upstream?

Thanks,
Janusz

Monday 06 July 2009 12:24:27 Janusz Krzysztofik wrote:
> This patch adds support for modem device found on Amstrad E3 (Delta) board.
>
> Based on earlier patch by Jonathan McDowell, available at
> http://the.earth.li/pub/e3/2.6.19/ams-delta-modem.patch.
> Modified after Ladislav Michl's arch/arm/mach-omap1/board-voiceblue.c.
>
> This version, unlike v2, is dependent on 8250 driver changes getting
> accepted upstream:
> Refer: http://patchwork.kernel.org/patch/31884/
>
> Signed-off-by: Janusz Krzysztofik 
> ---
> Registering the modem platform device from a separate arch_initcall (and
> not from init_machine() callback) preserves serial console port at ttyS0.
>
> OMAP mux pin set up just for case if Amstrad bootloader is ever replaced.
>
> Warning:
> could cause compilation break if 8250 patch is not integrated first
>
> --- linux-2.6.31-rc1/arch/arm/mach-omap1/board-ams-delta.c.orig   
> 2009-07-04
> 16:39:37.0 +0200 +++
> linux-2.6.31-rc1/arch/arm/mach-omap1/board-ams-delta.c2009-07-05
> 13:41:08.0 +0200 @@ -15,8 +15,11 @@
>  #include 
>  #include 
>  #include 
> +#include 
>  #include 
> +#include 
>
> +#include 
>  #include 
>  #include 
>  #include 
> @@ -235,6 +238,41 @@ static void __init ams_delta_init(void)
>   platform_add_devices(ams_delta_devices, ARRAY_SIZE(ams_delta_devices));
>  }
>
> +static struct plat_serial8250_port ams_delta_modem_ports[] = {
> + {
> + .membase= (void *) AMS_DELTA_MODEM_VIRT,
> + .mapbase= AMS_DELTA_MODEM_PHYS,
> + .irq= -EINVAL, /* changed later */
> + .flags  = UPF_BOOT_AUTOCONF,
> + .irqflags   = IRQF_TRIGGER_RISING,
> + .iotype = UPIO_MEM,
> + .regshift   = 1,
> + .uartclk= BASE_BAUD * 16,
> + },
> + { },
> +};
> +
> +static struct platform_device ams_delta_modem_device = {
> + .name   = "serial8250",
> + .id = PLAT8250_DEV_PLATFORM1,
> + .dev= {
> + .platform_data = ams_delta_modem_ports,
> + },
> +};
> +
> +static int __init ams_delta_modem_init(void)
> +{
> + omap_cfg_reg(M14_1510_GPIO2);
> + ams_delta_modem_ports[0].irq = gpio_to_irq(2);
> +
> + ams_delta_latch2_write(
> + AMS_DELTA_LATCH2_MODEM_NRESET | AMS_DELTA_LATCH2_MODEM_CODEC,
> + AMS_DELTA_LATCH2_MODEM_NRESET | AMS_DELTA_LATCH2_MODEM_CODEC);
> +
> + return platform_device_register(&ams_delta_modem_device);
> +}
> +arch_initcall(ams_delta_modem_init);
> +
>  static void __init ams_delta_map_io(void)
>  {
>   omap1_map_common_io();


--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH v3] OMAP1: AMS_DELTA: add modem support

2009-07-06 Thread Janusz Krzysztofik
This patch adds support for modem device found on Amstrad E3 (Delta) board.

Based on earlier patch by Jonathan McDowell, available at 
http://the.earth.li/pub/e3/2.6.19/ams-delta-modem.patch.
Modified after Ladislav Michl's arch/arm/mach-omap1/board-voiceblue.c.

This version, unlike v2, is dependent on 8250 driver changes getting accepted
upstream:
Refer: http://patchwork.kernel.org/patch/31884/

Signed-off-by: Janusz Krzysztofik 
---
Registering the modem platform device from a separate arch_initcall (and not
from init_machine() callback) preserves serial console port at ttyS0.

OMAP mux pin set up just for case if Amstrad bootloader is ever replaced.

Warning: 
could cause compilation break if 8250 patch is not integrated first

--- linux-2.6.31-rc1/arch/arm/mach-omap1/board-ams-delta.c.orig 2009-07-04 
16:39:37.0 +0200
+++ linux-2.6.31-rc1/arch/arm/mach-omap1/board-ams-delta.c  2009-07-05 
13:41:08.0 +0200
@@ -15,8 +15,11 @@
 #include 
 #include 
 #include 
+#include 
 #include 
+#include 
 
+#include 
 #include 
 #include 
 #include 
@@ -235,6 +238,41 @@ static void __init ams_delta_init(void)
platform_add_devices(ams_delta_devices, ARRAY_SIZE(ams_delta_devices));
 }
 
+static struct plat_serial8250_port ams_delta_modem_ports[] = {
+   {
+   .membase= (void *) AMS_DELTA_MODEM_VIRT,
+   .mapbase= AMS_DELTA_MODEM_PHYS,
+   .irq= -EINVAL, /* changed later */
+   .flags  = UPF_BOOT_AUTOCONF,
+   .irqflags   = IRQF_TRIGGER_RISING,
+   .iotype = UPIO_MEM,
+   .regshift   = 1,
+   .uartclk= BASE_BAUD * 16,
+   },
+   { },
+};
+
+static struct platform_device ams_delta_modem_device = {
+   .name   = "serial8250",
+   .id = PLAT8250_DEV_PLATFORM1,
+   .dev= {
+   .platform_data = ams_delta_modem_ports,
+   },
+};
+
+static int __init ams_delta_modem_init(void)
+{
+   omap_cfg_reg(M14_1510_GPIO2);
+   ams_delta_modem_ports[0].irq = gpio_to_irq(2);
+
+   ams_delta_latch2_write(
+   AMS_DELTA_LATCH2_MODEM_NRESET | AMS_DELTA_LATCH2_MODEM_CODEC,
+   AMS_DELTA_LATCH2_MODEM_NRESET | AMS_DELTA_LATCH2_MODEM_CODEC);
+
+   return platform_device_register(&ams_delta_modem_device);
+}
+arch_initcall(ams_delta_modem_init);
+
 static void __init ams_delta_map_io(void)
 {
omap1_map_common_io();

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html