Re: [PATCH] baycom_ser_fdx: also allow i/o ports = 0x1000 and enhanced failure logging

2007-03-05 Thread Ralf Baechle DL5RB
On Mon, Mar 05, 2007 at 02:45:09PM +0100, Thomas Sailer wrote: The baycom modem really doesn't have anything to do with RS232, it just uses the RS232 port as a PIO port + timer. That's why it needs its own 16550 driver. A few people asked me if it is possible to drive one of those RS232

Re: [PATCH] baycom_ser_fdx: also allow i/o ports = 0x1000 and enhanced failure logging

2007-03-05 Thread Thomas Sailer
On Mon, 2007-03-05 at 18:03 +, Ralf Baechle DL5RB wrote: A few people asked me if it is possible to drive one of those RS232 abusing modems through a USB-to-RS232 converter but I guess the answer is no? No way. For 9k6 you're polling the RX line say every 25us. That's just not feasible

[PATCH] baycom_ser_fdx: also allow i/o ports = 0x1000 and enhanced failure logging

2007-03-04 Thread Folkert van Heusden
Hi, The baycom_ser_fdx driver did not allow i/o ports = 0x1000. Now that there are pci cards (with rs232 ports) which use for example 0xb800 this limit should not exists. Also, for non kernel coders find the cause of problems was challenging so I added extra logging. Signed-off-by: Folkert van

Re: [PATCH] baycom_ser_fdx: also allow i/o ports = 0x1000 and enhanced failure logging

2007-03-04 Thread Alan Cox
The baycom_ser_fdx driver did not allow i/o ports = 0x1000. Now that there are pci cards (with rs232 ports) which use for example 0xb800 this limit should not exists. The IRQ test is also wrong for PCI systems the moment an APIC is present. NR_IRQS is the define to use. Also, for non kernel

Re: [PATCH] baycom_ser_fdx: also allow i/o ports = 0x1000 and enhanced failure logging

2007-03-04 Thread Folkert van Heusden
The baycom_ser_fdx driver did not allow i/o ports = 0x1000. Now that there are pci cards (with rs232 ports) which use for example 0xb800 this limit should not exists. The IRQ test is also wrong for PCI systems the moment an APIC is present. NR_IRQS is the define to use. Ok, changed: