Re: [PATCH] regulator: max8997/8966: fix charger cv voltage set bug

2017-05-09 Thread Chanwoo Choi
Hi, On 2017년 05월 08일 14:45, MyungJoo Ham wrote: > > When min charger-CV is <= 4.0V and max charger-CV is >= 4.0V, > we can use 4.00V as CV (register value = 0x1).` > > The original code had a typo that wrote ">=" (max_uV >= 400), > which should've been "<", which is not necessary anyway > as

Re: [PATCH] regulator: max8997/8966: fix charger cv voltage set bug

2017-05-08 Thread Bartlomiej Zolnierkiewicz
On Monday, May 08, 2017 05:45:44 AM MyungJoo Ham wrote: > > When min charger-CV is <= 4.0V and max charger-CV is >= 4.0V, > we can use 4.00V as CV (register value = 0x1).` > > The original code had a typo that wrote ">=" (max_uV >= 400), > which should've been "<", which is not necessary anyw

[PATCH] regulator: max8997/8966: fix charger cv voltage set bug

2017-05-07 Thread MyungJoo Ham
When min charger-CV is <= 4.0V and max charger-CV is >= 4.0V, we can use 4.00V as CV (register value = 0x1).` The original code had a typo that wrote ">=" (max_uV >= 400), which should've been "<", which is not necessary anyway as mentioned by Dan Carpenter. Reported-By: Dan Carpenter Signe