Re: [PATCH] mfd: fix TWL4030 COR bit polarity for BCI SIH block

[Samuel Ortiz <sameo@linux.intel.com>]

Hi Grazvydas,

On Tue, Sep 28, 2010 at 04:22:19PM +0300, Grazvydas Ignotas wrote:
> The chip TRM documentation contradicts itself about this bit, page 174
> of swcu050e says bit should be 0 for clear-on-read behavior, while
> page 487 says it should be 1. Testing shows it should be 1, so set
> the .set_cor flag accordingly. This is needed for upcoming BCI
> charging driver to function.
Patch applied, thanks.

Cheers,
Samuel.


> Signed-off-by: Grazvydas Ignotas <notasas@gmail.com>
> ---
>  drivers/mfd/twl4030-irq.c |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/mfd/twl4030-irq.c b/drivers/mfd/twl4030-irq.c
> index 097f24d..5b5a559 100644
> --- a/drivers/mfd/twl4030-irq.c
> +++ b/drivers/mfd/twl4030-irq.c
> @@ -144,6 +144,7 @@ static const struct sih sih_modules_twl4030[6] = {
>  		.name		= "bci",
>  		.module		= TWL4030_MODULE_INTERRUPTS,
>  		.control_offset	= TWL4030_INTERRUPTS_BCISIHCTRL,
> +		.set_cor	= true,
>  		.bits		= 12,
>  		.bytes_ixr	= 2,
>  		.edr_offset	= TWL4030_INTERRUPTS_BCIEDR1,
> @@ -408,7 +409,7 @@ static int twl4030_init_sih_modules(unsigned line)
>  		 * set Clear-On-Read (COR) bit.
>  		 *
>  		 * NOTE that sometimes COR polarity is documented as being
> -		 * inverted:  for MADC and BCI, COR=1 means "clear on write".
> +		 * inverted:  for MADC, COR=1 means "clear on write".
>  		 * And for PWR_INT it's not documented...
>  		 */
>  		if (sih->set_cor) {
> -- 
> 1.6.3.3
> 

-- 
Intel Open Source Technology Centre
http://oss.intel.com/