Re: [PATCH 2/2] HID: ntrig mapping more firmware id bits

[Jiri Kosina]

On Fri, 25 Feb 2011, Rafi Rubin wrote:

> Signed-off-by: Rafi Rubin <rafi@xxxxxxxxxxxxxx>
> ---
> Two new firmwares supply hints to map three more bits of the raw
> firmware id code.
> 
> Confirmation, and perhaps the rest of the mapping from someone with both
> the knowledge and legal rights would be appreciated.  Not that I mind
> the pleasure of the puzzle being filled in slowly over time.

Micki, I guess we are waiting for your Signed-off-by: here ... could you 
please comment on that?

Thanks.

> ---
>  drivers/hid/hid-ntrig.c |    6 +++---
>  1 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/hid/hid-ntrig.c b/drivers/hid/hid-ntrig.c
> index 616f091..3ce4624 100644
> --- a/drivers/hid/hid-ntrig.c
> +++ b/drivers/hid/hid-ntrig.c
> @@ -96,15 +96,15 @@ struct ntrig_data {
>   */
>  static int ntrig_version_string(unsigned char *raw, char *buf)
>  {
> -	__u8 a =  (raw[1] & 0x0e) >> 1;
> +	__u8 a =  (raw[1] & 0x1e) >> 1;
>  	__u8 b =  (raw[0] & 0x3c) >> 2;
>  	__u8 c = ((raw[0] & 0x03) << 3) | ((raw[3] & 0xe0) >> 5);
>  	__u8 d = ((raw[3] & 0x07) << 3) | ((raw[2] & 0xe0) >> 5);
> -	__u8 e =   raw[2] & 0x07;
> +	__u8 e =   raw[2] & 0x1f;
>  
>  	/*
>  	 * As yet unmapped bits:
> -	 * 0b11000000 0b11110001 0b00011000 0b00011000
> +	 * 0b11000000 0b11100001 0b00000000 0b00011000
>  	 */
>  
>  	return sprintf(buf, "%u.%u.%u.%u.%u", a, b, c, d, e);
> -- 
> 1.7.2.3
> 

-- 
Jiri Kosina
SUSE Labs, Novell Inc.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/