Re: [PATCH] netmos 9855 fix

2007-12-12 Thread chri 
On Dec 12, 2007 10:16 AM, Andrew Morton <[EMAIL PROTECTED]> wrote:

> What was wrong with it?
>

Hi,

on loading only one serial port was present and it wasn't working.
After looking in the data sheet I realized that the base address was
wrong. For further reference here is lspci and relevant dmesg output:

02:00.0 Communication controller: NetMos Technology PCI 9855 Multi-I/O
Controller (rev 01) (prog-if 02)
Subsystem: LSI Logic / Symbios Logic Unknown device 0022
Flags: medium devsel, IRQ 19
I/O ports at df00 [size=8]
I/O ports at de00 [size=8]
I/O ports at dd00 [size=8]
I/O ports at dc00 [size=8]
I/O ports at db00 [size=8]
I/O ports at da00 [size=16]

parport1: PC-style at 0xdd00 [PCSPP,TRISTATE]
parport2: PC-style at 0xdf00 [PCSPP,TRISTATE,EPP]
:02:00.0: ttyS0 at I/O 0xdb00 (irq = 19) is a 16550A
:02:00.0: ttyS1 at I/O 0xda00 (irq = 19) is a 16550A



-- 
Christian Pellegrin, see http://www.evolware.org/chri/
"Real Programmers don't play tennis, or any other sport which requires
you to change clothes. Mountain climbing is OK, and Real Programmers
wear their climbing boots to work in case a mountain should suddenly
spring up in the middle of the computer room."
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] netmos 9855 fix

2007-12-12 Thread Andrew Morton 
On Wed,  5 Dec 2007 11:27:27 +0100 [EMAIL PROTECTED] wrote:

> Fixes wrong netmos 9855 serial port configuration.
> 
> Generated on  20071205  against v2.6.21
> 
> Signed-off-by: Christian Pellegrin <[EMAIL PROTECTED]>
> ---
>  drivers/parport/parport_serial.c |4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/parport/parport_serial.c 
> b/drivers/parport/parport_serial.c
> index 78c0a26..5b04e78 100644
> --- a/drivers/parport/parport_serial.c
> +++ b/drivers/parport/parport_serial.c
> @@ -77,7 +77,7 @@ static struct parport_pc_pci cards[] __devinitdata = {
>   /* titan_110l */{ 1, { { 3, -1 }, } },
>   /* titan_210l */{ 1, { { 3, -1 }, } },
>   /* netmos_9xx5_combo */ { 1, { { 2, -1 }, }, 
> netmos_parallel_init },
> - /* netmos_9855 */   { 1, { { 0, -1 }, }, 
> netmos_parallel_init },
> + /* netmos_9855 */   { 1, { { 2, -1 }, }, 
> netmos_parallel_init },
>   /* avlab_1s1p */{ 1, { { 1, 2}, } },
>   /* avlab_1s2p */{ 2, { { 1, 2}, { 3, 4 },} },
>   /* avlab_2s1p */{ 1, { { 2, 3}, } },
> @@ -185,7 +185,7 @@ static struct pciserial_board pci_parport_serial_boards[] 
> __devinitdata = {
>   .uart_offset= 8,
>   },
>   [netmos_9855] = {
> - .flags  = FL_BASE2 | FL_BASE_BARS,
> + .flags  = FL_BASE4 | FL_BASE_BARS,
>   .num_ports  = 1,
>   .base_baud  = 115200,
>   .uart_offset= 8,

What was wrong with it?

Martin, does this change affect the hardware upon which you tested
44e58a6a0bd604f46be9d808408a1cd880cc9b19, "parport: NetMos nm9855 fix"?

Thanks.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] netmos 9855 fix

2007-12-12 Thread Andrew Morton 
On Wed,  5 Dec 2007 11:27:27 +0100 [EMAIL PROTECTED] wrote:

 Fixes wrong netmos 9855 serial port configuration.
 
 Generated on  20071205  against v2.6.21
 
 Signed-off-by: Christian Pellegrin [EMAIL PROTECTED]
 ---
  drivers/parport/parport_serial.c |4 ++--
  1 files changed, 2 insertions(+), 2 deletions(-)
 
 diff --git a/drivers/parport/parport_serial.c 
 b/drivers/parport/parport_serial.c
 index 78c0a26..5b04e78 100644
 --- a/drivers/parport/parport_serial.c
 +++ b/drivers/parport/parport_serial.c
 @@ -77,7 +77,7 @@ static struct parport_pc_pci cards[] __devinitdata = {
   /* titan_110l */{ 1, { { 3, -1 }, } },
   /* titan_210l */{ 1, { { 3, -1 }, } },
   /* netmos_9xx5_combo */ { 1, { { 2, -1 }, }, 
 netmos_parallel_init },
 - /* netmos_9855 */   { 1, { { 0, -1 }, }, 
 netmos_parallel_init },
 + /* netmos_9855 */   { 1, { { 2, -1 }, }, 
 netmos_parallel_init },
   /* avlab_1s1p */{ 1, { { 1, 2}, } },
   /* avlab_1s2p */{ 2, { { 1, 2}, { 3, 4 },} },
   /* avlab_2s1p */{ 1, { { 2, 3}, } },
 @@ -185,7 +185,7 @@ static struct pciserial_board pci_parport_serial_boards[] 
 __devinitdata = {
   .uart_offset= 8,
   },
   [netmos_9855] = {
 - .flags  = FL_BASE2 | FL_BASE_BARS,
 + .flags  = FL_BASE4 | FL_BASE_BARS,
   .num_ports  = 1,
   .base_baud  = 115200,
   .uart_offset= 8,

What was wrong with it?

Martin, does this change affect the hardware upon which you tested
44e58a6a0bd604f46be9d808408a1cd880cc9b19, parport: NetMos nm9855 fix?

Thanks.
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] netmos 9855 fix

2007-12-12 Thread chri 
On Dec 12, 2007 10:16 AM, Andrew Morton [EMAIL PROTECTED] wrote:

 What was wrong with it?


Hi,

on loading only one serial port was present and it wasn't working.
After looking in the data sheet I realized that the base address was
wrong. For further reference here is lspci and relevant dmesg output:

02:00.0 Communication controller: NetMos Technology PCI 9855 Multi-I/O
Controller (rev 01) (prog-if 02)
Subsystem: LSI Logic / Symbios Logic Unknown device 0022
Flags: medium devsel, IRQ 19
I/O ports at df00 [size=8]
I/O ports at de00 [size=8]
I/O ports at dd00 [size=8]
I/O ports at dc00 [size=8]
I/O ports at db00 [size=8]
I/O ports at da00 [size=16]

parport1: PC-style at 0xdd00 [PCSPP,TRISTATE]
parport2: PC-style at 0xdf00 [PCSPP,TRISTATE,EPP]
:02:00.0: ttyS0 at I/O 0xdb00 (irq = 19) is a 16550A
:02:00.0: ttyS1 at I/O 0xda00 (irq = 19) is a 16550A



-- 
Christian Pellegrin, see http://www.evolware.org/chri/
Real Programmers don't play tennis, or any other sport which requires
you to change clothes. Mountain climbing is OK, and Real Programmers
wear their climbing boots to work in case a mountain should suddenly
spring up in the middle of the computer room.
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH] netmos 9855 fix

2007-12-05 Thread chripell 
Fixes wrong netmos 9855 serial port configuration.

Generated on  20071205  against v2.6.21

Signed-off-by: Christian Pellegrin <[EMAIL PROTECTED]>
---
 drivers/parport/parport_serial.c |4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/parport/parport_serial.c b/drivers/parport/parport_serial.c
index 78c0a26..5b04e78 100644
--- a/drivers/parport/parport_serial.c
+++ b/drivers/parport/parport_serial.c
@@ -77,7 +77,7 @@ static struct parport_pc_pci cards[] __devinitdata = {
/* titan_110l */{ 1, { { 3, -1 }, } },
/* titan_210l */{ 1, { { 3, -1 }, } },
/* netmos_9xx5_combo */ { 1, { { 2, -1 }, }, 
netmos_parallel_init },
-   /* netmos_9855 */   { 1, { { 0, -1 }, }, 
netmos_parallel_init },
+   /* netmos_9855 */   { 1, { { 2, -1 }, }, 
netmos_parallel_init },
/* avlab_1s1p */{ 1, { { 1, 2}, } },
/* avlab_1s2p */{ 2, { { 1, 2}, { 3, 4 },} },
/* avlab_2s1p */{ 1, { { 2, 3}, } },
@@ -185,7 +185,7 @@ static struct pciserial_board pci_parport_serial_boards[] 
__devinitdata = {
.uart_offset= 8,
},
[netmos_9855] = {
-   .flags  = FL_BASE2 | FL_BASE_BARS,
+   .flags  = FL_BASE4 | FL_BASE_BARS,
.num_ports  = 1,
.base_baud  = 115200,
.uart_offset= 8,
--
1.4.4.4
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH] netmos 9855 fix

2007-12-05 Thread chripell 
Fixes wrong netmos 9855 serial port configuration.

Generated on  20071205  against v2.6.21

Signed-off-by: Christian Pellegrin [EMAIL PROTECTED]
---
 drivers/parport/parport_serial.c |4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/parport/parport_serial.c b/drivers/parport/parport_serial.c
index 78c0a26..5b04e78 100644
--- a/drivers/parport/parport_serial.c
+++ b/drivers/parport/parport_serial.c
@@ -77,7 +77,7 @@ static struct parport_pc_pci cards[] __devinitdata = {
/* titan_110l */{ 1, { { 3, -1 }, } },
/* titan_210l */{ 1, { { 3, -1 }, } },
/* netmos_9xx5_combo */ { 1, { { 2, -1 }, }, 
netmos_parallel_init },
-   /* netmos_9855 */   { 1, { { 0, -1 }, }, 
netmos_parallel_init },
+   /* netmos_9855 */   { 1, { { 2, -1 }, }, 
netmos_parallel_init },
/* avlab_1s1p */{ 1, { { 1, 2}, } },
/* avlab_1s2p */{ 2, { { 1, 2}, { 3, 4 },} },
/* avlab_2s1p */{ 1, { { 2, 3}, } },
@@ -185,7 +185,7 @@ static struct pciserial_board pci_parport_serial_boards[] 
__devinitdata = {
.uart_offset= 8,
},
[netmos_9855] = {
-   .flags  = FL_BASE2 | FL_BASE_BARS,
+   .flags  = FL_BASE4 | FL_BASE_BARS,
.num_ports  = 1,
.base_baud  = 115200,
.uart_offset= 8,
--
1.4.4.4
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/