Hi, On Tue, Nov 22, 2016 at 05:49:11PM +0100, Sebastian Reichel wrote: > On Tue, Nov 15, 2016 at 10:18:51PM +0900, Milo Kim wrote: > > TPS65217 charger driver handles the charger interrupt through the IRQ or > > polling. Both cases can be requested in single function. > > > > Cc: Enric Balletbo i Serra <[email protected]> > > Signed-off-by: Milo Kim <[email protected]> > > --- > > drivers/power/supply/tps65217_charger.c | 70 > > ++++++++++++++++++--------------- > > 1 file changed, 38 insertions(+), 32 deletions(-) > > I don't see the advantage of this. It's more lines of codes than > before and does not really increase readability.
Ok it makes sense with changes from PATCH 3. Please add a note to the commit description, that this prep-work for introducing more irqs. -- Sebastian
signature.asc
Description: PGP signature
