Re: [PATCH] ipwireless: driver for 3G PC Card

2008-02-01 Thread Pavel Machek
Hi! > > > +static irqreturn_t ipwireless_handle_v1_interrupt(int irq, > > > + struct ipw_hardware *hw) > > > +{ > > > + unsigned short irqn; > > > + unsigned short ack; > > > + > > > + irqn = inw(hw->base_port + IOIR); > > > + > > > + /* Check if card is

Re: [PATCH] ipwireless: driver for 3G PC Card

2008-02-01 Thread David Sterba
Hi, > > +/* DCR bits */ > > +#define DCR_RXDONE ((unsigned short) 0x1) > > +#define DCR_TXDONE ((unsigned short) 0x2) > > +#define DCR_RXRESET ((unsigned short) 0x4) > > +#define DCR_TXRESET ((unsigned short) 0x8) > > Are those casts neccessary? No, removed. > > +/* I/O ports

Re: [PATCH] ipwireless: driver for 3G PC Card

2008-02-01 Thread David Sterba
Hi, +/* DCR bits */ +#define DCR_RXDONE ((unsigned short) 0x1) +#define DCR_TXDONE ((unsigned short) 0x2) +#define DCR_RXRESET ((unsigned short) 0x4) +#define DCR_TXRESET ((unsigned short) 0x8) Are those casts neccessary? No, removed. +/* I/O ports and bit

Re: [PATCH] ipwireless: driver for 3G PC Card

2008-02-01 Thread Pavel Machek
Hi! +static irqreturn_t ipwireless_handle_v1_interrupt(int irq, + struct ipw_hardware *hw) +{ + unsigned short irqn; + unsigned short ack; + + irqn = inw(hw-base_port + IOIR); + + /* Check if card is present */ + if (irqn ==

Re: [PATCH] ipwireless: driver for 3G PC Card

2008-01-30 Thread Pavel Machek
On Thu 2008-01-31 10:29:30, Stephen Blackheath [to Foxconn] wrote: > Pavel & all, > > > Pavel Machek wrote: > > >> +/* I/O ports and bit definitions for version 2 of the hardware */ > >> + > >> +struct MEMCCR { > >> + unsigned short PCCOR; /* Configuration Option Register */ > >> +

Re: [PATCH] ipwireless: driver for 3G PC Card

2008-01-30 Thread Stephen Blackheath [to Foxconn]
Pavel & all, Pavel Machek wrote: >> +/* I/O ports and bit definitions for version 2 of the hardware */ >> + >> +struct MEMCCR { >> +unsigned short PCCOR; /* Configuration Option Register */ >> +unsigned short PCCSR; /* Configuration and Status Register */ >> +

Re: [PATCH] ipwireless: driver for 3G PC Card

2008-01-30 Thread Pavel Machek
Hi! > +/* I/O ports and bit definitions for version 1 of the hardware */ > + > +/* IER bits*/ > +#define IER_RXENABLED ((unsigned short) 0x1) > +#define IER_TXENABLED ((unsigned short) 0x2) > + > +/* ISR bits */ > +#define IR_RXINTR ((unsigned short) 0x1) > +#define IR_TXINTR

Re: [PATCH] ipwireless: driver for 3G PC Card

2008-01-30 Thread Pavel Machek
On Mon 2008-01-28 20:53:57, Alexey Dobriyan wrote: > On Mon, Jan 28, 2008 at 06:19:29PM +0100, David Sterba wrote: > > ipwireless: driver for PC Card, 3G internet connection > > > +struct nl_first_paket_header { > > +#if defined(__BIG_ENDIAN) > > + unsigned char packet_rank:2; > > + unsigned

Re: [PATCH] ipwireless: driver for 3G PC Card

2008-01-30 Thread Pavel Machek
On Mon 2008-01-28 20:53:57, Alexey Dobriyan wrote: On Mon, Jan 28, 2008 at 06:19:29PM +0100, David Sterba wrote: ipwireless: driver for PC Card, 3G internet connection +struct nl_first_paket_header { +#if defined(__BIG_ENDIAN) + unsigned char packet_rank:2; + unsigned char

Re: [PATCH] ipwireless: driver for 3G PC Card

2008-01-30 Thread Pavel Machek
Hi! +/* I/O ports and bit definitions for version 1 of the hardware */ + +/* IER bits*/ +#define IER_RXENABLED ((unsigned short) 0x1) +#define IER_TXENABLED ((unsigned short) 0x2) + +/* ISR bits */ +#define IR_RXINTR ((unsigned short) 0x1) +#define IR_TXINTR ((unsigned

Re: [PATCH] ipwireless: driver for 3G PC Card

2008-01-30 Thread Stephen Blackheath [to Foxconn]
Pavel all, Pavel Machek wrote: +/* I/O ports and bit definitions for version 2 of the hardware */ + +struct MEMCCR { +unsigned short PCCOR; /* Configuration Option Register */ +unsigned short PCCSR; /* Configuration and Status Register */ +unsigned short

Re: [PATCH] ipwireless: driver for 3G PC Card

2008-01-30 Thread Pavel Machek
On Thu 2008-01-31 10:29:30, Stephen Blackheath [to Foxconn] wrote: Pavel all, Pavel Machek wrote: +/* I/O ports and bit definitions for version 2 of the hardware */ + +struct MEMCCR { + unsigned short PCCOR; /* Configuration Option Register */ + unsigned short PCCSR;

Re: [PATCH] ipwireless: driver for 3G PC Card

2008-01-29 Thread David Sterba
On Tuesday 29 of January 2008 00:18:03 Jiri Slaby wrote: > On 01/28/2008 07:08 PM, Randy Dunlap wrote: > >> +/* > >> + * @return 1 if something has been received from hw > > > > What's with the '@'? > > ... > > >> + > >> +/*! > > > > and the '!' ? > > > >> + * @return true if the card is

Re: [PATCH] ipwireless: driver for 3G PC Card

2008-01-29 Thread David Sterba
On Monday 28 of January 2008 19:08:54 Randy Dunlap wrote: > > +/* Number of bytes in NL packet header (can not do > > cannot Fixed. > > > + * sizeof(nl_packet_header) since it's a bitfield) */ > > +#define NL_FOLLOWING_PACKET_HEADER_SIZE1 > > + > > +struct nl_first_paket_header { > >

Re: [PATCH] ipwireless: driver for 3G PC Card

2008-01-29 Thread David Sterba
On Tuesday 29 of January 2008 00:18:03 Jiri Slaby wrote: On 01/28/2008 07:08 PM, Randy Dunlap wrote: +/* + * @return 1 if something has been received from hw What's with the '@'? ... + +/*! and the '!' ? + * @return true if the card is physically present. + */ What

Re: [PATCH] ipwireless: driver for 3G PC Card

2008-01-29 Thread David Sterba
On Monday 28 of January 2008 19:08:54 Randy Dunlap wrote: +/* Number of bytes in NL packet header (can not do cannot Fixed. + * sizeof(nl_packet_header) since it's a bitfield) */ +#define NL_FOLLOWING_PACKET_HEADER_SIZE1 + +struct nl_first_paket_header {

Re: [PATCH] ipwireless: driver for 3G PC Card

2008-01-28 Thread Pekka Enberg
Hi, On Jan 29, 2008 1:33 AM, Randy Dunlap <[EMAIL PROTECTED]> wrote: > > > What part of kernel documentation uses doxygen? > > > > So then, what's the problem? > > Why is it there? We have a kernel documentation language. > Please use it or plain text. Yes please. Pekka

Re: [PATCH] ipwireless: driver for 3G PC Card

2008-01-28 Thread Randy Dunlap
On Tue, 29 Jan 2008 00:29:19 +0100 Jiri Slaby wrote: > On 01/29/2008 12:28 AM, Randy Dunlap wrote: > > On Tue, 29 Jan 2008 00:18:03 +0100 Jiri Slaby wrote: > > > >> On 01/28/2008 07:08 PM, Randy Dunlap wrote: > +/* > + * @return 1 if something has been received from hw > >>> What's

Re: [PATCH] ipwireless: driver for 3G PC Card

2008-01-28 Thread Jiri Slaby
On 01/29/2008 12:28 AM, Randy Dunlap wrote: On Tue, 29 Jan 2008 00:18:03 +0100 Jiri Slaby wrote: On 01/28/2008 07:08 PM, Randy Dunlap wrote: +/* + * @return 1 if something has been received from hw What's with the '@'? ... + +/*! and the '!' ? + * @return true if the card is physically

Re: [PATCH] ipwireless: driver for 3G PC Card

2008-01-28 Thread Randy Dunlap
On Tue, 29 Jan 2008 00:18:03 +0100 Jiri Slaby wrote: > On 01/28/2008 07:08 PM, Randy Dunlap wrote: > >> +/* > >> + * @return 1 if something has been received from hw > > > > What's with the '@'? > ... > >> + > >> +/*! > > > > and the '!' ? > > > >> + * @return true if the card is physically

Re: [PATCH] ipwireless: driver for 3G PC Card

2008-01-28 Thread Jiri Slaby
On 01/28/2008 07:08 PM, Randy Dunlap wrote: +/* + * @return 1 if something has been received from hw What's with the '@'? ... + +/*! and the '!' ? + * @return true if the card is physically present. + */ What exactly is wrong with @ and !? It's perfectly doxygen-styled. diff --git

Re: [PATCH] ipwireless: driver for 3G PC Card

2008-01-28 Thread Randy Dunlap
On Mon, 28 Jan 2008 18:19:29 +0100 David Sterba wrote: [resending due to send problems, sorry about any dups] > Hi Linus, > > I'm submitting driver for IPWireless PC Card modem for inclusion to 2.6.25. > > The driver has been in -mm series as ipwireless_cs.git tree for > some time and has

Re: [PATCH] ipwireless: driver for 3G PC Card

2008-01-28 Thread Alexey Dobriyan
On Mon, Jan 28, 2008 at 06:19:29PM +0100, David Sterba wrote: > ipwireless: driver for PC Card, 3G internet connection > +struct nl_first_paket_header { > +#if defined(__BIG_ENDIAN) > + unsigned char packet_rank:2; > + unsigned char address:3; > + unsigned char protocol:3; > +#else >

Re: [PATCH] ipwireless: driver for 3G PC Card

2008-01-28 Thread Alexey Dobriyan
On Mon, Jan 28, 2008 at 06:19:29PM +0100, David Sterba wrote: ipwireless: driver for PC Card, 3G internet connection +struct nl_first_paket_header { +#if defined(__BIG_ENDIAN) + unsigned char packet_rank:2; + unsigned char address:3; + unsigned char protocol:3; +#else +

Re: [PATCH] ipwireless: driver for 3G PC Card

2008-01-28 Thread Jiri Slaby
On 01/28/2008 07:08 PM, Randy Dunlap wrote: +/* + * @return 1 if something has been received from hw What's with the '@'? ... + +/*! and the '!' ? + * @return true if the card is physically present. + */ What exactly is wrong with @ and !? It's perfectly doxygen-styled. diff --git

Re: [PATCH] ipwireless: driver for 3G PC Card

2008-01-28 Thread Randy Dunlap
On Tue, 29 Jan 2008 00:18:03 +0100 Jiri Slaby wrote: On 01/28/2008 07:08 PM, Randy Dunlap wrote: +/* + * @return 1 if something has been received from hw What's with the '@'? ... + +/*! and the '!' ? + * @return true if the card is physically present. + */ What

Re: [PATCH] ipwireless: driver for 3G PC Card

2008-01-28 Thread Jiri Slaby
On 01/29/2008 12:28 AM, Randy Dunlap wrote: On Tue, 29 Jan 2008 00:18:03 +0100 Jiri Slaby wrote: On 01/28/2008 07:08 PM, Randy Dunlap wrote: +/* + * @return 1 if something has been received from hw What's with the '@'? ... + +/*! and the '!' ? + * @return true if the card is physically

Re: [PATCH] ipwireless: driver for 3G PC Card

2008-01-28 Thread Randy Dunlap
On Tue, 29 Jan 2008 00:29:19 +0100 Jiri Slaby wrote: On 01/29/2008 12:28 AM, Randy Dunlap wrote: On Tue, 29 Jan 2008 00:18:03 +0100 Jiri Slaby wrote: On 01/28/2008 07:08 PM, Randy Dunlap wrote: +/* + * @return 1 if something has been received from hw What's with the '@'? ... +

Re: [PATCH] ipwireless: driver for 3G PC Card

2008-01-28 Thread Pekka Enberg
Hi, On Jan 29, 2008 1:33 AM, Randy Dunlap [EMAIL PROTECTED] wrote: What part of kernel documentation uses doxygen? So then, what's the problem? Why is it there? We have a kernel documentation language. Please use it or plain text. Yes please. Pekka -- To

Re: [PATCH] ipwireless: driver for 3G PC Card

2008-01-28 Thread Randy Dunlap
On Mon, 28 Jan 2008 18:19:29 +0100 David Sterba wrote: [resending due to send problems, sorry about any dups] Hi Linus, I'm submitting driver for IPWireless PC Card modem for inclusion to 2.6.25. The driver has been in -mm series as ipwireless_cs.git tree for some time and has passed