Re: [PATCH 08/15] twl4030_charger: distinguish between USB current and 'AC' current

From: Pavel Machek
Date: Mon Mar 02 2015 - 16:05:22 EST


Hi!

> The twl4030 charger has two current sources, 'USB' and 'AC' (which is
> really DC of course...).
>
> If 'AC' is providing current, we should set the current limit
> differently to when it isn't (and so USB is used).
> So split 'cur' into 'usb_cur' and 'ac_cur' and use accordingly.
>
> Now we must review the current setting on any interrupt or USB
> event which might indicate that the charger-source has changed.
>
> Signed-off-by: NeilBrown <neilb@xxxxxxx>

> @@ -103,7 +104,9 @@ struct twl4030_bci {
> /* ichg values in uA. If any are 'large', we set CGAIN to

ichg/_cur values

> * '1' which doubles the range for half the precision.
> */
> - int ichg_eoc, ichg_lo, ichg_hi, cur;
> + int ichg_eoc, ichg_lo, ichg_hi;
> + int usb_cur, ac_cur;

> + /* If VAC exceeds 4.5V (MADC 11) and ac is enabled, set current
> + * for 'ac'

/*
* If AC voltage exceeds 4.5V (MADC 11) and AC is enabled, set current
* for 'ac'.


Acked-by: Pavel Machek <pavel@xxxxxx>

--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--
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/