Re: [PATCH v2 2/2] power: supply: Add Spreadtrum SC2731 charger support

2018-09-16 Thread Baolin Wang
Hi Sebastian, On 16 September 2018 at 18:17, Sebastian Reichel wrote: > Hi Baolin, > > On Mon, Sep 03, 2018 at 04:55:46PM +0800, Baolin Wang wrote: >> This patch adds the SC2731 PMIC switch charger support. >> >> Signed-off-by: Baolin Wang >> --- > > The power-supply sysfs files uses uA instead

Re: [PATCH v2 2/2] power: supply: Add Spreadtrum SC2731 charger support

2018-09-16 Thread Sebastian Reichel
Hi Baolin, On Mon, Sep 03, 2018 at 04:55:46PM +0800, Baolin Wang wrote: > This patch adds the SC2731 PMIC switch charger support. > > Signed-off-by: Baolin Wang > --- The power-supply sysfs files uses uA instead of mA, otherwise the driver and the binding looks fine to me. -- Sebastian > Chan

Re: [PATCH v2 2/2] power: supply: Add Spreadtrum SC2731 charger support

2018-09-03 Thread Krzysztof Kozlowski
On Mon, 3 Sep 2018 at 10:56, Baolin Wang wrote: > > This patch adds the SC2731 PMIC switch charger support. > > Signed-off-by: Baolin Wang > --- > Changes from v1: > - Remove some redundant head files. > - Add one mutex to make sure getting the charger status is correct. > - Fix some coding st

[PATCH v2 2/2] power: supply: Add Spreadtrum SC2731 charger support

2018-09-03 Thread Baolin Wang
This patch adds the SC2731 PMIC switch charger support. Signed-off-by: Baolin Wang --- Changes from v1: - Remove some redundant head files. - Add one mutex to make sure getting the charger status is correct. - Fix some coding style issues. - Use standard DT properties. --- drivers/power/supp