Re: [PATCH v7 00/10] Support ROHM BD99954 charger IC

["Vaittinen, Matti" <Matti.Vaittinen@fi.rohmeurope.com>]

On Tue, 2020-03-31 at 14:24 +0200, Ard Biesheuvel wrote:
> Hello Matti,
> 
> On Tue, 31 Mar 2020 at 14:20, Matti Vaittinen
> <matti.vaittinen@fi.rohmeurope.com> wrote:
> > Support ROHM BD99954 Battery Management IC
> > 
> > ROHM BD99954 is a Battery Management IC for 1-4 cell Lithium-Ion
> > secondary battery. BD99954 is intended to be used in space-
> > constraint
> > equipment such as Low profile Notebook PC, Tablets and other
> > applications.
> > 
> 
> Is it *really* necessary to cc this to 50 different people?
> 
> In the future, please trim down the cc list and keep it to the
> mailing
> list, maintainers and reviewers of the affected subsystem.

Sorry Ard (and others). This is the list get_maintainers.pl gave for
the series. Partially the large amount of receivers is caused by the
regulator framework change which impacted bunch of individual drivers.
Please, see the amount changed files from summary below. In addition to
that the linear_range code has been drifting from one place to other
during the review iterations until it settled to /lib. For example Greg
gave me some comments while linear_ranges was still heading to
drivers/base - so I did not drop Greg off from new revisions of the
series even if it was no longer changing drivers/base

It's hard for me to decide who of the fellows are relevant and who are
not. I know email flood is annoying and thus I did not send all of the
patches in the series to everyone. And - personally I do like receiving
whole series if I have been added in one patch. But I admit my mailbox
is not as crowded as it used to be so maybe that is why it is not so
annoying for me to delete mails.

Anyways, I will drop everyone who asks to be dropped from subsequent
patches but other than that - I guess it's better to accidentally
include someone who needs to press delete - than drop someone who
want's to have a say on changes.

> >  .../bindings/power/supply/battery.txt         |    6 +
> >  .../bindings/power/supply/rohm,bd99954.yaml   |  155 +++
> >  drivers/power/supply/Kconfig                  |   34 +-
> >  drivers/power/supply/Makefile                 |    1 +
> >  drivers/power/supply/bd70528-charger.c        |  140 +-
> >  drivers/power/supply/bd99954-charger.c        | 1149
> > +++++++++++++++++
> >  drivers/power/supply/power_supply_core.c      |    8 +
> >  drivers/regulator/88pg86x.c                   |    4 +-
> >  drivers/regulator/88pm800-regulator.c         |    4 +-
> >  drivers/regulator/Kconfig                     |    1 +
> >  drivers/regulator/act8865-regulator.c         |    4 +-
> >  drivers/regulator/act8945a-regulator.c        |    2 +-
> >  drivers/regulator/arizona-ldo1.c              |    2 +-
> >  drivers/regulator/arizona-micsupp.c           |    4 +-
> >  drivers/regulator/as3711-regulator.c          |    6 +-
> >  drivers/regulator/as3722-regulator.c          |    4 +-
> >  drivers/regulator/axp20x-regulator.c          |   16 +-
> >  drivers/regulator/bcm590xx-regulator.c        |    8 +-
> >  drivers/regulator/bd70528-regulator.c         |    8 +-
> >  drivers/regulator/bd71828-regulator.c         |   10 +-
> >  drivers/regulator/bd718x7-regulator.c         |   26 +-
> >  drivers/regulator/da903x.c                    |    2 +-
> >  drivers/regulator/helpers.c                   |  130 +-
> >  drivers/regulator/hi6421-regulator.c          |    4 +-
> >  drivers/regulator/lochnagar-regulator.c       |    4 +-
> >  drivers/regulator/lp873x-regulator.c          |    4 +-
> >  drivers/regulator/lp87565-regulator.c         |    2 +-
> >  drivers/regulator/lp8788-buck.c               |    2 +-
> >  drivers/regulator/max77650-regulator.c        |    2 +-
> >  drivers/regulator/mcp16502.c                  |    4 +-
> >  drivers/regulator/mp8859.c                    |    2 +-
> >  drivers/regulator/mt6323-regulator.c          |    6 +-
> >  drivers/regulator/mt6358-regulator.c          |    8 +-
> >  drivers/regulator/mt6380-regulator.c          |    6 +-
> >  drivers/regulator/mt6397-regulator.c          |    6 +-
> >  drivers/regulator/palmas-regulator.c          |    4 +-
> >  drivers/regulator/qcom-rpmh-regulator.c       |    2 +-
> >  drivers/regulator/qcom_rpm-regulator.c        |   14 +-
> >  drivers/regulator/qcom_smd-regulator.c        |   70 +-
> >  drivers/regulator/rk808-regulator.c           |   10 +-
> >  drivers/regulator/s2mps11.c                   |   14 +-
> >  drivers/regulator/sky81452-regulator.c        |    2 +-
> >  drivers/regulator/stpmic1_regulator.c         |   18 +-
> >  drivers/regulator/tps65086-regulator.c        |   10 +-
> >  drivers/regulator/tps65217-regulator.c        |    4 +-
> >  drivers/regulator/tps65218-regulator.c        |    6 +-
> >  drivers/regulator/tps65912-regulator.c        |    4 +-
> >  drivers/regulator/twl-regulator.c             |    4 +-
> >  drivers/regulator/twl6030-regulator.c         |    2 +-
> >  drivers/regulator/wm831x-dcdc.c               |    2 +-
> >  drivers/regulator/wm831x-ldo.c                |    4 +-
> >  drivers/regulator/wm8350-regulator.c          |    2 +-
> >  drivers/regulator/wm8400-regulator.c          |    2 +-
> >  include/linux/linear_range.h                  |   48 +
> >  include/linux/power/bd99954-charger.h         | 1075
> > +++++++++++++++
> >  include/linux/power_supply.h                  |    4 +
> >  include/linux/regulator/driver.h              |   27 +-
> >  lib/Kconfig                                   |    3 +
> >  lib/Kconfig.debug                             |   11 +
> >  lib/Makefile                                  |    2 +
> >  lib/linear_ranges.c                           |  246 ++++
> >  lib/test_linear_ranges.c                      |  228 ++++
> >  62 files changed, 3236 insertions(+), 356 deletions(-)
> >  create mode 100644
> > Documentation/devicetree/bindings/power/supply/rohm,bd99954.yaml
> >  create mode 100644 drivers/power/supply/bd99954-charger.c
> >  create mode 100644 include/linux/linear_range.h
> >  create mode 100644 include/linux/power/bd99954-charger.h
> >  create mode 100644 lib/linear_ranges.c
> >  create mode 100644 lib/test_linear_ranges.c
> > 
> > 

Best Regards
	Matti

> > 
> > 
> > --
> > Matti Vaittinen, Linux device drivers
> > ROHM Semiconductors, Finland SWDC
> > Kiviharjunlenkki 1E
> > 90220 OULU
> > FINLAND
> > 
> > ~~~ "I don't think so," said Rene Descartes. Just then he vanished
> > ~~~
> > Simon says - in Latin please.
> > ~~~ "non cogito me" dixit Rene Descarte, deinde evanescavit ~~~
> > Thanks to Simon Glass for the translation =]

_______________________________________________
Linux-mediatek mailing list
Linux-mediatek@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-mediatek