[PATCH] serio: allow registered drivers to get status flag

Parse the status byte information to the registered serio drivers as 
well as the character bytes.

Signed-off-by: David Engraf <redacted>

On Tue, Jan 18, 2011 at 01:57:21PM +0100, David Engraf wrote:

Parse the status byte information to the registered serio drivers as
well as the character bytes.

Signed-off-by: David Engraf <redacted>

Hi David,

The flags argument that serio_interrupt() accepts not the raw protocol
data, but sanitized, protocol-independent SERIO_TIMEOUT, SERIO_PARITY
and SERIO_FRAME bits.

Thanks.

-- 
Dmitry

Am 18.01.2011 20:57, schrieb Dmitry Torokhov:

On Tue, Jan 18, 2011 at 01:57:21PM +0100, David Engraf wrote:

quoted

Parse the status byte information to the registered serio drivers as
well as the character bytes.

Signed-off-by: David Engraf<redacted>

quoted

--- linux-2.6.37/drivers/input/serio/serport.c.orig	2011-01-18 12:33:41.421709232 +0100
+++ linux-2.6.37/drivers/input/serio/serport.c	2011-01-18 13:33:24.521711214 +0100

@@ -116,8 +116,8 @@ static void serport_ldisc_close(struct t

  /*
   * serport_ldisc_receive() is called by the low level tty driver when characters
- * are ready for us. We forward the characters, one by one to the 'interrupt'
- * routine.
+ * are ready for us. We forward the characters and flags, one by one to the
+ * 'interrupt' routine.
   */

  static void serport_ldisc_receive(struct tty_struct *tty, const unsigned char *cp, char *fp, int count)

@@ -132,7 +132,7 @@ static void serport_ldisc_receive(struct
  		goto out;

  	for (i = 0; i<  count; i++)
-		serio_interrupt(serport->serio, cp[i], 0);
+		serio_interrupt(serport->serio, cp[i], fp[i]);

Hi David,

The flags argument that serio_interrupt() accepts not the raw protocol
data, but sanitized, protocol-independent SERIO_TIMEOUT, SERIO_PARITY
and SERIO_FRAME bits.

Thanks.

Hi Dmitry,

I've updated the patch so that the raw protocol is converted to the 
SERIO_ bits. The raw protocol can parse only one flag, so I don't need 
to store multiple SERIO_ bits in the character flag variable.

Thanks
- David


Signed-off-by: David Engraf<redacted>

Am 20.01.2011 10:05, schrieb David Engraf:

Am 18.01.2011 20:57, schrieb Dmitry Torokhov:

quoted

On Tue, Jan 18, 2011 at 01:57:21PM +0100, David Engraf wrote:

quoted

Parse the status byte information to the registered serio drivers as
well as the character bytes.

Signed-off-by: David Engraf<redacted>

quoted

--- linux-2.6.37/drivers/input/serio/serport.c.orig 2011-01-18

12:33:41.421709232 +0100

+++ linux-2.6.37/drivers/input/serio/serport.c 2011-01-18

13:33:24.521711214 +0100

@@ -116,8 +116,8 @@ static void serport_ldisc_close(struct t

/*
* serport_ldisc_receive() is called by the low level tty driver when
characters
- * are ready for us. We forward the characters, one by one to the
'interrupt'
- * routine.
+ * are ready for us. We forward the characters and flags, one by one
to the
+ * 'interrupt' routine.
*/

static void serport_ldisc_receive(struct tty_struct *tty, const
unsigned char *cp, char *fp, int count)

@@ -132,7 +132,7 @@ static void serport_ldisc_receive(struct

goto out;

for (i = 0; i< count; i++)
- serio_interrupt(serport->serio, cp[i], 0);
+ serio_interrupt(serport->serio, cp[i], fp[i]);

Hi David,

The flags argument that serio_interrupt() accepts not the raw protocol
data, but sanitized, protocol-independent SERIO_TIMEOUT, SERIO_PARITY
and SERIO_FRAME bits.

Thanks.

Hi Dmitry,

I've updated the patch so that the raw protocol is converted to the
SERIO_ bits. The raw protocol can parse only one flag, so I don't need
to store multiple SERIO_ bits in the character flag variable.

Thanks
- David


Signed-off-by: David Engraf<redacted>

Hi Dmitry,

is my last patch okay for you? Just want to know if I can give you 
further help on this and if it is planned to integrate my patch.

Thanks
- David

On Wed, Jan 26, 2011 at 10:36:42AM +0100, David Engraf wrote:

Am 20.01.2011 10:05, schrieb David Engraf:

quoted

Am 18.01.2011 20:57, schrieb Dmitry Torokhov:

quoted

On Tue, Jan 18, 2011 at 01:57:21PM +0100, David Engraf wrote:

quoted

Parse the status byte information to the registered serio drivers as
well as the character bytes.

Signed-off-by: David Engraf<redacted>

quoted

--- linux-2.6.37/drivers/input/serio/serport.c.orig 2011-01-18

12:33:41.421709232 +0100

+++ linux-2.6.37/drivers/input/serio/serport.c 2011-01-18

13:33:24.521711214 +0100

@@ -116,8 +116,8 @@ static void serport_ldisc_close(struct t

/*
* serport_ldisc_receive() is called by the low level tty driver when
characters
- * are ready for us. We forward the characters, one by one to the
'interrupt'
- * routine.
+ * are ready for us. We forward the characters and flags, one by one
to the
+ * 'interrupt' routine.
*/

static void serport_ldisc_receive(struct tty_struct *tty, const
unsigned char *cp, char *fp, int count)

@@ -132,7 +132,7 @@ static void serport_ldisc_receive(struct

goto out;

for (i = 0; i< count; i++)
- serio_interrupt(serport->serio, cp[i], 0);
+ serio_interrupt(serport->serio, cp[i], fp[i]);

Hi David,

The flags argument that serio_interrupt() accepts not the raw protocol
data, but sanitized, protocol-independent SERIO_TIMEOUT, SERIO_PARITY
and SERIO_FRAME bits.

Thanks.

Hi Dmitry,

I've updated the patch so that the raw protocol is converted to the
SERIO_ bits. The raw protocol can parse only one flag, so I don't need
to store multiple SERIO_ bits in the character flag variable.

Thanks
- David


Signed-off-by: David Engraf<redacted>

Hi Dmitry,

is my last patch okay for you? Just want to know if I can give you
further help on this and if it is planned to integrate my patch.

Hi David,

The patch should hit mainline real soon now, no action is needed on your
part.

Thanks.

-- 
Dmitry