On Tue, May 25, 2010 at 08:07:59PM -0700, Ping Cheng wrote:
> Signed-off-by: Ping Cheng <[email protected]>

do you have a 2 sentence blurb for me to add to the commit message that
explains what's special about devices that set the WCM_LCD flag?
I can _guess_, especially when looking at the next patch, but this way it's
still confusing.

thanks for the patch though, merged.

Cheers,
  Peter

>  src/wcmValidateDevice.c |   11 ++++++++---
>  src/xf86WacomDefs.h     |    1 +
>  2 files changed, 9 insertions(+), 3 deletions(-)
> 
> diff --git a/src/wcmValidateDevice.c b/src/wcmValidateDevice.c
> index c46043f..df1375c 100644
> --- a/src/wcmValidateDevice.c
> +++ b/src/wcmValidateDevice.c
> @@ -206,6 +206,11 @@ int wcmDeviceTypeKeys(LocalDevicePtr local)
>                       break;
>  
>               /* tablets support menu strips */
> +             case 0x3F:  /* CintiqV5 */
> +             case 0xC5:  /* CintiqV5 */
> +             case 0xC6:  /* CintiqV5 */
> +                     priv->common->tablet_type |= WCM_LCD;
> +                     /* fall through */
>               case 0xB0:  /* I3 */
>               case 0xB1:  /* I3 */
>               case 0xB2:  /* I3 */
> @@ -213,15 +218,13 @@ int wcmDeviceTypeKeys(LocalDevicePtr local)
>               case 0xB4:  /* I3 */
>               case 0xB5:  /* I3 */
>               case 0xB7:  /* I3 */
> -             case 0x3F:  /* CintiqV5 */
> -             case 0xC5:  /* CintiqV5 */
> -             case 0xC6:  /* CintiqV5 */
>                       priv->common->tablet_type = WCM_STRIP | WCM_ROTATION;
>                       break;
>  
>               case 0xE2: /* TPC with 2FGT */
>               case 0xE3: /* TPC with 2FGT */
>                       priv->common->tablet_type = WCM_TPC;
> +                     priv->common->tablet_type |= WCM_LCD;
>                       /* fall through */
>               case 0xD0:  /* Bamboo with 2FGT */
>               case 0xD1:  /* Bamboo with 2FGT */
> @@ -236,10 +239,12 @@ int wcmDeviceTypeKeys(LocalDevicePtr local)
>                       /* fall through */
>               case 0x90: /* TPC */
>                       priv->common->tablet_type |= WCM_TPC;
> +                     priv->common->tablet_type |= WCM_LCD;
>                       break;
>  
>               case 0x9F:
>                       priv->common->tablet_type = WCM_1FGT;
> +                     priv->common->tablet_type |= WCM_LCD;
>                       break;
>  
>               default:
> diff --git a/src/xf86WacomDefs.h b/src/xf86WacomDefs.h
> index 7399b9e..ddddf26 100644
> --- a/src/xf86WacomDefs.h
> +++ b/src/xf86WacomDefs.h
> @@ -118,6 +118,7 @@ struct _WacomModel
>  #define WCM_RING             0x00000020
>  #define WCM_DUALINPUT                0x00000040
>  #define WCM_ROTATION         0x00000080
> +#define WCM_LCD              0x00000100
>  
>  #define TabletHasFeature(common, feature) (((common)->tablet_id & (feature)) 
> != 0)
>  
> -- 
> 1.6.6.1
> 

------------------------------------------------------------------------------

_______________________________________________
Linuxwacom-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel

Reply via email to