On Mon, Feb 02, 2009 at 12:47:05PM +0530, Pakaravoor, Jagadeesh wrote:
> Ben,
> 
>   Did you get a chance to look at this patch?

Is it a necesary bugfix, or should it wait for the next merge window?

> -----Original Message-----
> From: [email protected] 
> [mailto:[email protected]] On Behalf Of Pakaravoor, Jagadeesh
> Sent: Friday, January 23, 2009 2:19 PM
> To: [email protected]
> Cc: [email protected]; Ben Dooks
> Subject: [PATCH] I2C: OMAP: Add missing wakeup events
> 
> From: Jagadeesh Bhaskar Pakaravoor <[email protected]>
> 
> Include wake up events for receiver overflow and transmitter
> underflow in OMAP_I2C_WE_REG configuration. Also fix a small
> typo.
> 
> Signed-off-by: Jagadeesh Bhaskar Pakaravoor <[email protected]>
> ---
> Index: kernel_org/drivers/i2c/busses/i2c-omap.c
> ===================================================================
> --- kernel_org.orig/drivers/i2c/busses/i2c-omap.c     2009-01-23 
> 12:46:58.000000000 +0530
> +++ kernel_org/drivers/i2c/busses/i2c-omap.c  2009-01-23 14:15:52.187150630 
> +0530
> @@ -92,8 +92,10 @@
>  #define OMAP_I2C_STAT_AL     (1 << 0)        /* Arbitration lost int ena */
>  
>  /* I2C WE wakeup enable register */
> -#define OMAP_I2C_WE_XDR_WE   (1 << 14)       /* TX drain wakup */
> +#define OMAP_I2C_WE_XDR_WE   (1 << 14)       /* TX drain wakeup */
>  #define OMAP_I2C_WE_RDR_WE   (1 << 13)       /* RX drain wakeup */
> +#define OMAP_I2C_WE_ROVR_WE  (1 << 11)       /* RX overflow wakeup */
> +#define OMAP_I2C_WE_XUDF_WE  (1 << 10)       /* TX underflow wakeup */
>  #define OMAP_I2C_WE_AAS_WE   (1 << 9)        /* Address as slave wakeup*/
>  #define OMAP_I2C_WE_BF_WE    (1 << 8)        /* Bus free wakeup */
>  #define OMAP_I2C_WE_STC_WE   (1 << 6)        /* Start condition wakeup */
> @@ -104,6 +106,7 @@
>  #define OMAP_I2C_WE_AL_WE    (1 << 0)        /* Arbitration lost wakeup */
>  
>  #define OMAP_I2C_WE_ALL              (OMAP_I2C_WE_XDR_WE | 
> OMAP_I2C_WE_RDR_WE | \
> +                             OMAP_I2C_WE_ROVR_WE | OMAP_I2C_WE_XUDF_WE | \
>                               OMAP_I2C_WE_AAS_WE | OMAP_I2C_WE_BF_WE | \
>                               OMAP_I2C_WE_STC_WE | OMAP_I2C_WE_GC_WE | \
>                               OMAP_I2C_WE_DRDY_WE | OMAP_I2C_WE_ARDY_WE | \
> --
> With Regards,
> Jagadeesh Bhaskar P
>  
> ----------------------------
> Some men see things as they are and say why - I dream things that never were 
> and say why not.
> - George Bernard Shaw
> ----------------------------
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
> the body of a message to [email protected]
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
> the body of a message to [email protected]
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

-- 
Ben ([email protected], http://www.fluff.org/)

  'a smiley only costs 4 bytes'
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to