On Fri, Sep 30, 2022 at 10:50:06PM +0800, cy_huang wrote:
> From: ChiYuan Huang <cy_huang@richtek.com>
>
> This patch set is to add Richtek RT9471 charger support.
>
> The RT9471/D is a highly-integrated 3A switch mode battery charge management
> and system power path management device for single cell Li-Ion and Li-polymer
> battery. The low impedance power path optimizes switch-mode operation
> efficiency, reduces battery charging time and extends battery life during
> discharging phase.
>
Hi,
This patch series was pending almost 2 month.
For the below question
Does it mean to remove 'sysoff_enable' attribute and no more discussion?
https://lore.kernel.org/lkml/CADiBU39FMkDOrMHXTQF+0JGX2tDn_iVXCb19jM6MopnKmaXOpQ@mail.gmail.com/
v7
https://lore.kernel.org/lkml/1664531079-15915-1-git-send-email-u0084500@gmail.com/
v8
https://lore.kernel.org/lkml/1664549409-4280-1-git-send-email-u0084500@gmail.com/
For v7/v8, it may be my fault.
Because I could not get any reply, that's why I try to find any missing header or code
defect to be fixed. I know the version tags make others confused.
I'm sorry about that.
Still hope to get any feedback for the finalized v9 version.
Thanks.
> Since v8:
> - revert v7 for adding irq wakeup capable, i2c core already did it.
>
> Since v7:
> - Add irq wakeup capable.
> - Add missing header 'module.h' for MODULE_DEVICE_TABLE.
>
> Since v6:
> - Use 'ATTRIBUTE_GROUPS' macro to simplify the attribute coding.
> - Explain more details for sysoff_enable attribute.
>
> Since v5:
> - Remove one line wrapper.
> - Merge header content into source file, remove header file.
> - Remove 'charge_term_enable' sysfs entry and merge it into
> 'charge_term_current' property control.
> - Change MODULE_LICENSE from 'GPL v2' to 'GPL'.
> Following by the below discussion
> https://lore.kernel.org/all/YxAVAt2eWB3NFlrk@google.com/
> - Recover all the change in sysfs-class-power.
> - New a sysfs-class-power-rt9471 file.
> - Remove 'charge_term_enable' sysfs entry, directly integrate it in
> 'charge_term_current' power supply property control.
>
> Since v4:
> - Remove the line for the owner field in driver.
> - Add the documentation for sysfs entries.
>
> Since v3:
> - Move unevaluatedProperties line after $ref for binding patch.
> - Add Reviewed-by tag for binding patch.
>
> Since v2:
> - Remove the properties for interrupt controller things in the binding documentation.
> - Fix dtc error for typo, it's 'regulator-name', not 'regulator-compatible'.
> - Add regulator min/max microamp to allow otg vbus current adjustable in example.
> - Specify the active-level for charge-enable-gpios in binding example.
> - Fix checkpatch error about 'foo * bar' to 'foo *bar' in psy_device_to_chip function.
> - Specify the member name directly for the use of linear range.
>
> ChiYuan Huang (3):
> dt-bindings: power: supply: Add Richtek RT9471 battery charger
> power: supply: rt9471: Add Richtek RT9471 charger driver
> Documentation: power: rt9471: Document exported sysfs entries
>
> Documentation/ABI/testing/sysfs-class-power-rt9471 | 32 +
> .../bindings/power/supply/richtek,rt9471.yaml | 73 ++
> drivers/power/supply/Kconfig | 16 +
> drivers/power/supply/Makefile | 1 +
> drivers/power/supply/rt9471.c | 931 +++++++++++++++++++++
> 5 files changed, 1053 insertions(+)
> create mode 100644 Documentation/ABI/testing/sysfs-class-power-rt9471
> create mode 100644 Documentation/devicetree/bindings/power/supply/richtek,rt9471.yaml
> create mode 100644 drivers/power/supply/rt9471.c
>
> --
> 2.7.4
>