* [PATCH v14 01/10] power: supply: add undervoltage health status property
2025-01-08 14:13 [PATCH v14 00/10] Add support for Maxim Integrated MAX77705 PMIC Dzmitry Sankouski
@ 2025-01-08 14:13 ` Dzmitry Sankouski
2025-01-08 14:13 ` [PATCH v14 02/10] dt-bindings: power: supply: max17042: add max77705 support Dzmitry Sankouski
` (9 subsequent siblings)
10 siblings, 0 replies; 23+ messages in thread
From: Dzmitry Sankouski @ 2025-01-08 14:13 UTC (permalink / raw)
To: Sebastian Reichel, Chanwoo Choi, Krzysztof Kozlowski, Lee Jones,
Rob Herring, Conor Dooley, Dmitry Torokhov, Pavel Machek,
Hans de Goede, Marek Szyprowski, Sebastian Krzyszkowiak,
Purism Kernel Team, Krzysztof Kozlowski
Cc: linux-pm, linux-kernel, devicetree, linux-input, linux-leds,
Dzmitry Sankouski
Add POWER_SUPPLY_HEALTH_UNDERVOLTAGE status for power supply
to report under voltage lockout failures.
Signed-off-by: Dzmitry Sankouski <dsankouski@gmail.com>
---
Changes for v5:
- update Documentation/ABI/testing/sysfs-class-power and
drivers/power/supply/power_supply_sysfs.c
---
Documentation/ABI/testing/sysfs-class-power | 2 +-
drivers/power/supply/power_supply_sysfs.c | 1 +
include/linux/power_supply.h | 1 +
3 files changed, 3 insertions(+), 1 deletion(-)
diff --git a/Documentation/ABI/testing/sysfs-class-power b/Documentation/ABI/testing/sysfs-class-power
index cb53dde20a80..2a5c1a09a28f 100644
--- a/Documentation/ABI/testing/sysfs-class-power
+++ b/Documentation/ABI/testing/sysfs-class-power
@@ -453,7 +453,7 @@ Description:
Valid values:
"Unknown", "Good", "Overheat", "Dead",
- "Over voltage", "Unspecified failure", "Cold",
+ "Over voltage", "Under voltage", "Unspecified failure", "Cold",
"Watchdog timer expire", "Safety timer expire",
"Over current", "Calibration required", "Warm",
"Cool", "Hot", "No battery"
diff --git a/drivers/power/supply/power_supply_sysfs.c b/drivers/power/supply/power_supply_sysfs.c
index 9ce5eda24093..edb058c19c9c 100644
--- a/drivers/power/supply/power_supply_sysfs.c
+++ b/drivers/power/supply/power_supply_sysfs.c
@@ -99,6 +99,7 @@ static const char * const POWER_SUPPLY_HEALTH_TEXT[] = {
[POWER_SUPPLY_HEALTH_OVERHEAT] = "Overheat",
[POWER_SUPPLY_HEALTH_DEAD] = "Dead",
[POWER_SUPPLY_HEALTH_OVERVOLTAGE] = "Over voltage",
+ [POWER_SUPPLY_HEALTH_UNDERVOLTAGE] = "Under voltage",
[POWER_SUPPLY_HEALTH_UNSPEC_FAILURE] = "Unspecified failure",
[POWER_SUPPLY_HEALTH_COLD] = "Cold",
[POWER_SUPPLY_HEALTH_WATCHDOG_TIMER_EXPIRE] = "Watchdog timer expire",
diff --git a/include/linux/power_supply.h b/include/linux/power_supply.h
index c3ce9f2b17d4..6ed53b292162 100644
--- a/include/linux/power_supply.h
+++ b/include/linux/power_supply.h
@@ -60,6 +60,7 @@ enum {
POWER_SUPPLY_HEALTH_OVERHEAT,
POWER_SUPPLY_HEALTH_DEAD,
POWER_SUPPLY_HEALTH_OVERVOLTAGE,
+ POWER_SUPPLY_HEALTH_UNDERVOLTAGE,
POWER_SUPPLY_HEALTH_UNSPEC_FAILURE,
POWER_SUPPLY_HEALTH_COLD,
POWER_SUPPLY_HEALTH_WATCHDOG_TIMER_EXPIRE,
--
2.39.5
^ permalink raw reply related [flat|nested] 23+ messages in thread* [PATCH v14 02/10] dt-bindings: power: supply: max17042: add max77705 support
2025-01-08 14:13 [PATCH v14 00/10] Add support for Maxim Integrated MAX77705 PMIC Dzmitry Sankouski
2025-01-08 14:13 ` [PATCH v14 01/10] power: supply: add undervoltage health status property Dzmitry Sankouski
@ 2025-01-08 14:13 ` Dzmitry Sankouski
2025-01-08 14:13 ` [PATCH v14 03/10] dt-bindings: power: supply: add maxim,max77705 charger Dzmitry Sankouski
` (8 subsequent siblings)
10 siblings, 0 replies; 23+ messages in thread
From: Dzmitry Sankouski @ 2025-01-08 14:13 UTC (permalink / raw)
To: Sebastian Reichel, Chanwoo Choi, Krzysztof Kozlowski, Lee Jones,
Rob Herring, Conor Dooley, Dmitry Torokhov, Pavel Machek,
Hans de Goede, Marek Szyprowski, Sebastian Krzyszkowiak,
Purism Kernel Team, Krzysztof Kozlowski
Cc: linux-pm, linux-kernel, devicetree, linux-input, linux-leds,
Dzmitry Sankouski, Krzysztof Kozlowski
Add max77705 fuel gauge support.
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Signed-off-by: Dzmitry Sankouski <dsankouski@gmail.com>
---
Changes in v14:
- add reviewed trailers
Changes in v10:
- keep alphabetical order
---
Documentation/devicetree/bindings/power/supply/maxim,max17042.yaml | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/power/supply/maxim,max17042.yaml b/Documentation/devicetree/bindings/power/supply/maxim,max17042.yaml
index 085e2504d0dc..14242de7fc08 100644
--- a/Documentation/devicetree/bindings/power/supply/maxim,max17042.yaml
+++ b/Documentation/devicetree/bindings/power/supply/maxim,max17042.yaml
@@ -19,6 +19,7 @@ properties:
- maxim,max17047
- maxim,max17050
- maxim,max17055
+ - maxim,max77705-battery
- maxim,max77849-battery
reg:
--
2.39.5
^ permalink raw reply related [flat|nested] 23+ messages in thread* [PATCH v14 03/10] dt-bindings: power: supply: add maxim,max77705 charger
2025-01-08 14:13 [PATCH v14 00/10] Add support for Maxim Integrated MAX77705 PMIC Dzmitry Sankouski
2025-01-08 14:13 ` [PATCH v14 01/10] power: supply: add undervoltage health status property Dzmitry Sankouski
2025-01-08 14:13 ` [PATCH v14 02/10] dt-bindings: power: supply: max17042: add max77705 support Dzmitry Sankouski
@ 2025-01-08 14:13 ` Dzmitry Sankouski
2025-01-08 14:13 ` [PATCH v14 04/10] dt-bindings: mfd: add maxim,max77705 Dzmitry Sankouski
` (7 subsequent siblings)
10 siblings, 0 replies; 23+ messages in thread
From: Dzmitry Sankouski @ 2025-01-08 14:13 UTC (permalink / raw)
To: Sebastian Reichel, Chanwoo Choi, Krzysztof Kozlowski, Lee Jones,
Rob Herring, Conor Dooley, Dmitry Torokhov, Pavel Machek,
Hans de Goede, Marek Szyprowski, Sebastian Krzyszkowiak,
Purism Kernel Team, Krzysztof Kozlowski
Cc: linux-pm, linux-kernel, devicetree, linux-input, linux-leds,
Dzmitry Sankouski, Krzysztof Kozlowski
Add maxim,max77705 charger binding.
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Signed-off-by: Dzmitry Sankouski <dsankouski@gmail.com>
---
Changes in v14:
- fix required children nodes order: reverse reg and compatible
- add reviewed trailers
---
Documentation/devicetree/bindings/power/supply/maxim,max77705.yaml | 50 ++++++++++++++++++++++++++++++++++++++++++++++++++
1 file changed, 50 insertions(+)
diff --git a/Documentation/devicetree/bindings/power/supply/maxim,max77705.yaml b/Documentation/devicetree/bindings/power/supply/maxim,max77705.yaml
new file mode 100644
index 000000000000..bce7fabbd9d3
--- /dev/null
+++ b/Documentation/devicetree/bindings/power/supply/maxim,max77705.yaml
@@ -0,0 +1,50 @@
+# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/power/supply/maxim,max77705.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Maxim MAX777705 charger
+
+maintainers:
+ - Dzmitry Sankouski <dsankouski@gmail.com>
+
+description: |
+ This is a device tree bindings for charger found in Maxim MAX77705 chip.
+
+allOf:
+ - $ref: power-supply.yaml#
+
+properties:
+ compatible:
+ const: maxim,max77705-charger
+
+ interrupts:
+ maxItems: 1
+
+ reg:
+ maxItems: 1
+
+required:
+ - compatible
+ - reg
+ - monitored-battery
+
+unevaluatedProperties: false
+
+examples:
+ - |
+ #include <dt-bindings/interrupt-controller/irq.h>
+
+ i2c {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ charger@69 {
+ compatible = "maxim,max77705-charger";
+ reg = <0x69>;
+ monitored-battery = <&battery>;
+ interrupt-parent = <&pm8998_gpios>;
+ interrupts = <11 IRQ_TYPE_LEVEL_LOW>;
+ };
+ };
--
2.39.5
^ permalink raw reply related [flat|nested] 23+ messages in thread* [PATCH v14 04/10] dt-bindings: mfd: add maxim,max77705
2025-01-08 14:13 [PATCH v14 00/10] Add support for Maxim Integrated MAX77705 PMIC Dzmitry Sankouski
` (2 preceding siblings ...)
2025-01-08 14:13 ` [PATCH v14 03/10] dt-bindings: power: supply: add maxim,max77705 charger Dzmitry Sankouski
@ 2025-01-08 14:13 ` Dzmitry Sankouski
2025-01-08 14:13 ` [PATCH v14 05/10] power: supply: max17042: add max77705 fuel gauge support Dzmitry Sankouski
` (6 subsequent siblings)
10 siblings, 0 replies; 23+ messages in thread
From: Dzmitry Sankouski @ 2025-01-08 14:13 UTC (permalink / raw)
To: Sebastian Reichel, Chanwoo Choi, Krzysztof Kozlowski, Lee Jones,
Rob Herring, Conor Dooley, Dmitry Torokhov, Pavel Machek,
Hans de Goede, Marek Szyprowski, Sebastian Krzyszkowiak,
Purism Kernel Team, Krzysztof Kozlowski
Cc: linux-pm, linux-kernel, devicetree, linux-input, linux-leds,
Dzmitry Sankouski
Add maxim,max77705 binding part, containing leds controller and haptics.
Charger and fuel gauge are separate device, thus not included.
Reviewed-by: Rob Herring (Arm) <robh@kernel.org>
Signed-off-by: Dzmitry Sankouski <dsankouski@gmail.com>
---
Changes in v14:
- add reviewed trailers
- move "patternProperties" after the "properties:" block
Changes in v13:
- drop unused address-cells and size-cells properties
Changes in v12:
- revert: description: mention this is a part
- adjust commit message
Changes in v12:
- move fuelgauge bindings to separate patch because separate device
- move charger bindings to separate patch because separate device
- description: mention this is a part
Changes in v11:
- remove reviewed tags because of major changes
- none of children are supposed to have addresses, all nodes are
unit-less
Changes in v10:
- leds: replace label with color and function properties
- leds: add support for leds-class-multicolor
- move fuelgauge node to patternProperties "^fuel-gauge@[0-9a-f]+$"
to comply with max17042 binding
Changes in v9:
- replace max77705 fuel gauge with max17042
- remove monitored battery because not supported by max17042
Changes in v8:
- fix leds compatible
Changes in v6:
- unevaluatedProperties must be false
- drop excessive sentence from description,
just describe the device
- change leds compatible to maxim,max77705-rgb
Changes in v5:
- formatting changes
- add unevaluatedProperties: false for nodes referencing
common schemas
- remove additionalProperties on nodes with
unevaluatedProperties: false
- add min and max to led index
Changes in v4:
- change dts example intendation from tabs
to spaces
- remove interrupt-names property
- remove obvious reg description
- split long(>80) lines
---
Documentation/devicetree/bindings/mfd/maxim,max77705.yaml | 158 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
MAINTAINERS | 1 +
2 files changed, 159 insertions(+)
diff --git a/Documentation/devicetree/bindings/mfd/maxim,max77705.yaml b/Documentation/devicetree/bindings/mfd/maxim,max77705.yaml
new file mode 100644
index 000000000000..0ec89f0adc64
--- /dev/null
+++ b/Documentation/devicetree/bindings/mfd/maxim,max77705.yaml
@@ -0,0 +1,158 @@
+# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/mfd/maxim,max77705.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Maxim MAX77705 Companion Power Management and USB Type-C interface
+
+maintainers:
+ - Dzmitry Sankouski <dsankouski@gmail.com>
+
+description: |
+ This is a part of device tree bindings for Maxim MAX77705.
+
+ Maxim MAX77705 is a Companion Power Management and Type-C
+ interface IC which includes charger, fuelgauge, LED, haptic motor driver and
+ Type-C management.
+
+properties:
+ compatible:
+ const: maxim,max77705
+
+ reg:
+ maxItems: 1
+
+ interrupts:
+ maxItems: 1
+
+ haptic:
+ type: object
+ additionalProperties: false
+
+ properties:
+ compatible:
+ const: maxim,max77705-haptic
+
+ haptic-supply: true
+
+ pwms:
+ maxItems: 1
+
+ required:
+ - compatible
+ - haptic-supply
+ - pwms
+
+ leds:
+ type: object
+ additionalProperties: false
+ description:
+ Up to 4 LED channels supported.
+
+ properties:
+ compatible:
+ const: maxim,max77705-rgb
+
+ "#address-cells":
+ const: 1
+
+ "#size-cells":
+ const: 0
+
+ multi-led:
+ type: object
+ $ref: /schemas/leds/leds-class-multicolor.yaml#
+ unevaluatedProperties: false
+
+ properties:
+ "#address-cells":
+ const: 1
+
+ "#size-cells":
+ const: 0
+
+ patternProperties:
+ "^led@[0-3]$":
+ type: object
+ $ref: /schemas/leds/common.yaml#
+ unevaluatedProperties: false
+
+ properties:
+ reg:
+ maxItems: 1
+
+ required:
+ - reg
+
+ patternProperties:
+ "^led@[0-3]$":
+ type: object
+ $ref: /schemas/leds/common.yaml#
+ unevaluatedProperties: false
+
+ properties:
+ reg:
+ maxItems: 1
+
+ required:
+ - reg
+
+ required:
+ - compatible
+
+required:
+ - compatible
+
+additionalProperties: false
+
+examples:
+ - |
+ #include <dt-bindings/interrupt-controller/irq.h>
+ #include <dt-bindings/leds/common.h>
+
+ i2c {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ pmic@66 {
+ compatible = "maxim,max77705";
+ reg = <0x66>;
+ interrupt-parent = <&pm8998_gpios>;
+ interrupts = <11 IRQ_TYPE_LEVEL_LOW>;
+ pinctrl-0 = <&chg_int_default>;
+ pinctrl-names = "default";
+
+ leds {
+ compatible = "maxim,max77705-rgb";
+
+ multi-led {
+ color = <LED_COLOR_ID_RGB>;
+ function = LED_FUNCTION_STATUS;
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ led@1 {
+ reg = <1>;
+ color = <LED_COLOR_ID_RED>;
+ };
+
+ led@2 {
+ reg = <2>;
+ color = <LED_COLOR_ID_GREEN>;
+ };
+
+ led@3 {
+ reg = <3>;
+ color = <LED_COLOR_ID_BLUE>;
+ };
+ };
+ };
+
+ haptic {
+ compatible = "maxim,max77705-haptic";
+ haptic-supply = <&vib_regulator>;
+ pwms = <&vib_pwm 0 50000>;
+ };
+ };
+ };
diff --git a/MAINTAINERS b/MAINTAINERS
index f8922b37489d..98a77dece2f1 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -14287,6 +14287,7 @@ B: mailto:linux-samsung-soc@vger.kernel.org
F: Documentation/devicetree/bindings/*/maxim,max14577.yaml
F: Documentation/devicetree/bindings/*/maxim,max77686.yaml
F: Documentation/devicetree/bindings/*/maxim,max77693.yaml
+F: Documentation/devicetree/bindings/*/maxim,max77705*.yaml
F: Documentation/devicetree/bindings/*/maxim,max77843.yaml
F: Documentation/devicetree/bindings/clock/maxim,max77686.txt
F: drivers/*/*max77843.c
--
2.39.5
^ permalink raw reply related [flat|nested] 23+ messages in thread* [PATCH v14 05/10] power: supply: max17042: add max77705 fuel gauge support
2025-01-08 14:13 [PATCH v14 00/10] Add support for Maxim Integrated MAX77705 PMIC Dzmitry Sankouski
` (3 preceding siblings ...)
2025-01-08 14:13 ` [PATCH v14 04/10] dt-bindings: mfd: add maxim,max77705 Dzmitry Sankouski
@ 2025-01-08 14:13 ` Dzmitry Sankouski
2025-01-08 14:13 ` [PATCH v14 06/10] power: supply: max77705: Add charger driver for Maxim 77705 Dzmitry Sankouski
` (5 subsequent siblings)
10 siblings, 0 replies; 23+ messages in thread
From: Dzmitry Sankouski @ 2025-01-08 14:13 UTC (permalink / raw)
To: Sebastian Reichel, Chanwoo Choi, Krzysztof Kozlowski, Lee Jones,
Rob Herring, Conor Dooley, Dmitry Torokhov, Pavel Machek,
Hans de Goede, Marek Szyprowski, Sebastian Krzyszkowiak,
Purism Kernel Team, Krzysztof Kozlowski
Cc: linux-pm, linux-kernel, devicetree, linux-input, linux-leds,
Dzmitry Sankouski
Add max77705 fuel gauge support.
Signed-off-by: Dzmitry Sankouski <dsankouski@gmail.com>
---
Changes in v12:
- add only to platform version IDs
Changes in v11:
- add device type to compatible
Changes in v10:
- keep alphabetical order
---
drivers/power/supply/max17042_battery.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/power/supply/max17042_battery.c b/drivers/power/supply/max17042_battery.c
index 4037843e25bb..655b3f25dbd7 100644
--- a/drivers/power/supply/max17042_battery.c
+++ b/drivers/power/supply/max17042_battery.c
@@ -1234,6 +1234,8 @@ static const struct of_device_id max17042_dt_match[] __used = {
.data = (void *) MAXIM_DEVICE_TYPE_MAX17050 },
{ .compatible = "maxim,max17055",
.data = (void *) MAXIM_DEVICE_TYPE_MAX17055 },
+ { .compatible = "maxim,max77705-battery",
+ .data = (void *) MAXIM_DEVICE_TYPE_MAX17047 },
{ .compatible = "maxim,max77849-battery",
.data = (void *) MAXIM_DEVICE_TYPE_MAX17047 },
{ },
@@ -1256,6 +1258,7 @@ static const struct platform_device_id max17042_platform_id[] = {
{ "max17047", MAXIM_DEVICE_TYPE_MAX17047 },
{ "max17050", MAXIM_DEVICE_TYPE_MAX17050 },
{ "max17055", MAXIM_DEVICE_TYPE_MAX17055 },
+ { "max77705-battery", MAXIM_DEVICE_TYPE_MAX17047 },
{ "max77849-battery", MAXIM_DEVICE_TYPE_MAX17047 },
{ }
};
--
2.39.5
^ permalink raw reply related [flat|nested] 23+ messages in thread* [PATCH v14 06/10] power: supply: max77705: Add charger driver for Maxim 77705
2025-01-08 14:13 [PATCH v14 00/10] Add support for Maxim Integrated MAX77705 PMIC Dzmitry Sankouski
` (4 preceding siblings ...)
2025-01-08 14:13 ` [PATCH v14 05/10] power: supply: max17042: add max77705 fuel gauge support Dzmitry Sankouski
@ 2025-01-08 14:13 ` Dzmitry Sankouski
2025-01-16 0:16 ` Sebastian Reichel
2025-01-08 14:13 ` [PATCH v14 07/10] mfd: simple-mfd-i2c: Add MAX77705 support Dzmitry Sankouski
` (4 subsequent siblings)
10 siblings, 1 reply; 23+ messages in thread
From: Dzmitry Sankouski @ 2025-01-08 14:13 UTC (permalink / raw)
To: Sebastian Reichel, Chanwoo Choi, Krzysztof Kozlowski, Lee Jones,
Rob Herring, Conor Dooley, Dmitry Torokhov, Pavel Machek,
Hans de Goede, Marek Szyprowski, Sebastian Krzyszkowiak,
Purism Kernel Team, Krzysztof Kozlowski
Cc: linux-pm, linux-kernel, devicetree, linux-input, linux-leds,
Dzmitry Sankouski
Add driver for Maxim 77705 switch-mode charger.
It providing power supply class information to userspace.
The driver is configured through DTS (battery and system related
settings).
Signed-off-by: Dzmitry Sankouski <dsankouski@gmail.com>
---
Changes in v12:
- add missing new line
- reorder max77705_charger_enable and max77705_charger_disable action
- convert driver from platform to i2c, adjust commit msg accordingly
Changes in v11:
- remove multiple blank lines
- return dev_err_probe
- fit on one line where possible;
Changes in v10:
- never blank line between call and its error check
- replace remove function with devm_add_action_or_reset
- remove unused inline functions from header
- use dev_err_probe for error handling
Changes in v9:
- move power supply registration before interrupts to prevent
NULL exceptions when handling interrupts
Changes for v8:
- join lines, where fits 100 chars
- change comment style C++ -> C
- remove author from 'based on' file header statement
Changes for v6:
- add i2c init in driver
- replace remove_new back on remove
- handle IS_ERR(i2c_chg)
Changes for v5:
- remove const modifier from max77705_charger_irq_chip
because it's modified with irq_drv_data in probe function
- fix license to GPL 2.0 only, where old vendor code used
GPL 2.0 only
- move power header to power include dir
- use same hardware name in Kconfig and module descriptions
Changes for v4:
- start from scratch
- change word delimiters in filenames to '_'
- use GENMASK in header
- remove debugfs code
- migrate to regmap_add_irq_chip
- fix property getters to follow the same style
---
drivers/power/supply/Kconfig | 6 ++
drivers/power/supply/Makefile | 1 +
drivers/power/supply/max77705_charger.c | 576 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
include/linux/power/max77705_charger.h | 194 +++++++++++++++++++++++++++++++++++++++++++++++
4 files changed, 777 insertions(+)
diff --git a/drivers/power/supply/Kconfig b/drivers/power/supply/Kconfig
index 7b18358f194a..c120d8ed870f 100644
--- a/drivers/power/supply/Kconfig
+++ b/drivers/power/supply/Kconfig
@@ -583,6 +583,12 @@ config CHARGER_MAX77693
help
Say Y to enable support for the Maxim MAX77693 battery charger.
+config CHARGER_MAX77705
+ tristate "Maxim MAX77705 battery charger driver"
+ depends on MFD_MAX77705
+ help
+ Say Y to enable support for the Maxim MAX77705 battery charger.
+
config CHARGER_MAX77976
tristate "Maxim MAX77976 battery charger driver"
depends on I2C
diff --git a/drivers/power/supply/Makefile b/drivers/power/supply/Makefile
index b55cc48a4c86..a4669383d53e 100644
--- a/drivers/power/supply/Makefile
+++ b/drivers/power/supply/Makefile
@@ -80,6 +80,7 @@ obj-$(CONFIG_CHARGER_MAX14577) += max14577_charger.o
obj-$(CONFIG_CHARGER_DETECTOR_MAX14656) += max14656_charger_detector.o
obj-$(CONFIG_CHARGER_MAX77650) += max77650-charger.o
obj-$(CONFIG_CHARGER_MAX77693) += max77693_charger.o
+obj-$(CONFIG_CHARGER_MAX77705) += max77705_charger.o
obj-$(CONFIG_CHARGER_MAX77976) += max77976_charger.o
obj-$(CONFIG_CHARGER_MAX8997) += max8997_charger.o
obj-$(CONFIG_CHARGER_MAX8998) += max8998_charger.o
diff --git a/drivers/power/supply/max77705_charger.c b/drivers/power/supply/max77705_charger.c
new file mode 100644
index 000000000000..563b41420c7c
--- /dev/null
+++ b/drivers/power/supply/max77705_charger.c
@@ -0,0 +1,576 @@
+// SPDX-License-Identifier: GPL-2.0
+/*
+ * Based on max77650-charger.c
+ *
+ * Copyright (C) 2024 Dzmitry Sankouski <dsankouski@gmail.org>
+ *
+ * Battery charger driver for MAXIM 77705 charger/power-supply.
+ */
+
+#include <linux/i2c.h>
+#include <linux/interrupt.h>
+#include <linux/mfd/max77693-common.h>
+#include <linux/mfd/max77705-private.h>
+#include <linux/power/max77705_charger.h>
+#include <linux/module.h>
+#include <linux/platform_device.h>
+#include <linux/power_supply.h>
+#include <linux/regmap.h>
+
+static const char *max77705_charger_model = "max77705";
+static const char *max77705_charger_manufacturer = "Maxim Integrated";
+
+static const struct regmap_config max77705_chg_regmap_config = {
+ .reg_base = MAX77705_CHG_REG_BASE,
+ .reg_bits = 8,
+ .val_bits = 8,
+ .max_register = MAX77705_CHG_REG_SAFEOUT_CTRL,
+};
+
+static enum power_supply_property max77705_charger_props[] = {
+ POWER_SUPPLY_PROP_ONLINE,
+ POWER_SUPPLY_PROP_PRESENT,
+ POWER_SUPPLY_PROP_STATUS,
+ POWER_SUPPLY_PROP_CHARGE_TYPE,
+ POWER_SUPPLY_PROP_HEALTH,
+ POWER_SUPPLY_PROP_VOLTAGE_MAX_DESIGN,
+ POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE,
+ POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT,
+ POWER_SUPPLY_PROP_INPUT_CURRENT_LIMIT,
+};
+
+static int max77705_chgin_irq(void *irq_drv_data)
+{
+ struct max77705_charger_data *charger = irq_drv_data;
+
+ queue_work(charger->wqueue, &charger->chgin_work);
+
+ return 0;
+}
+
+static const struct regmap_irq max77705_charger_irqs[] = {
+ { .mask = MAX77705_BYP_IM, },
+ { .mask = MAX77705_INP_LIMIT_IM, },
+ { .mask = MAX77705_BATP_IM, },
+ { .mask = MAX77705_BAT_IM, },
+ { .mask = MAX77705_CHG_IM, },
+ { .mask = MAX77705_WCIN_IM, },
+ { .mask = MAX77705_CHGIN_IM, },
+ { .mask = MAX77705_AICL_IM, },
+};
+
+static struct regmap_irq_chip max77705_charger_irq_chip = {
+ .name = "max77705-charger",
+ .status_base = MAX77705_CHG_REG_INT,
+ .mask_base = MAX77705_CHG_REG_INT_MASK,
+ .handle_post_irq = max77705_chgin_irq,
+ .num_regs = 1,
+ .irqs = max77705_charger_irqs,
+ .num_irqs = ARRAY_SIZE(max77705_charger_irqs),
+};
+
+static int max77705_charger_enable(struct max77705_charger_data *chg)
+{
+ int rv;
+
+ rv = regmap_update_bits(chg->regmap, MAX77705_CHG_REG_CNFG_09,
+ MAX77705_CHG_EN_MASK, MAX77705_CHG_EN_MASK);
+ if (rv)
+ dev_err(chg->dev, "unable to enable the charger: %d\n", rv);
+
+ return rv;
+}
+
+static void max77705_charger_disable(void *data)
+{
+ struct max77705_charger_data *chg = data;
+ int rv;
+
+ rv = regmap_update_bits(chg->regmap,
+ MAX77705_CHG_REG_CNFG_09,
+ MAX77705_CHG_EN_MASK,
+ MAX77705_CHG_DISABLE);
+ if (rv)
+ dev_err(chg->dev, "unable to disable the charger: %d\n", rv);
+}
+
+static int max77705_get_online(struct regmap *regmap, int *val)
+{
+ unsigned int data;
+ int ret;
+
+ ret = regmap_read(regmap, MAX77705_CHG_REG_INT_OK, &data);
+ if (ret < 0)
+ return ret;
+
+ *val = !!(data & MAX77705_CHGIN_OK);
+
+ return 0;
+}
+
+static int max77705_check_battery(struct max77705_charger_data *charger, int *val)
+{
+ unsigned int reg_data;
+ unsigned int reg_data2;
+ struct regmap *regmap = charger->regmap;
+
+ regmap_read(regmap, MAX77705_CHG_REG_INT_OK, ®_data);
+
+ dev_dbg(charger->dev, "CHG_INT_OK(0x%x)\n", reg_data);
+
+ regmap_read(regmap, MAX77705_CHG_REG_DETAILS_00, ®_data2);
+
+ dev_dbg(charger->dev, "CHG_DETAILS00(0x%x)\n", reg_data2);
+
+ if ((reg_data & MAX77705_BATP_OK) || !(reg_data2 & MAX77705_BATP_DTLS))
+ *val = true;
+ else
+ *val = false;
+
+ return 0;
+}
+
+static int max77705_get_charge_type(struct max77705_charger_data *charger, int *val)
+{
+ struct regmap *regmap = charger->regmap;
+ unsigned int reg_data;
+
+ regmap_read(regmap, MAX77705_CHG_REG_CNFG_09, ®_data);
+ if (!MAX77705_CHARGER_CHG_CHARGING(reg_data)) {
+ *val = POWER_SUPPLY_CHARGE_TYPE_NONE;
+ return 0;
+ }
+
+ regmap_read(regmap, MAX77705_CHG_REG_DETAILS_01, ®_data);
+ reg_data &= MAX77705_CHG_DTLS;
+
+ switch (reg_data) {
+ case 0x0:
+ case MAX77705_CHARGER_CONSTANT_CURRENT:
+ case MAX77705_CHARGER_CONSTANT_VOLTAGE:
+ *val = POWER_SUPPLY_CHARGE_TYPE_FAST;
+ return 0;
+ default:
+ *val = POWER_SUPPLY_CHARGE_TYPE_NONE;
+ return 0;
+ }
+
+ return 0;
+}
+
+static int max77705_get_status(struct max77705_charger_data *charger, int *val)
+{
+ struct regmap *regmap = charger->regmap;
+ unsigned int reg_data;
+
+ regmap_read(regmap, MAX77705_CHG_REG_CNFG_09, ®_data);
+ if (!MAX77705_CHARGER_CHG_CHARGING(reg_data)) {
+ *val = POWER_SUPPLY_CHARGE_TYPE_NONE;
+ return 0;
+ }
+
+ regmap_read(regmap, MAX77705_CHG_REG_DETAILS_01, ®_data);
+ reg_data &= MAX77705_CHG_DTLS;
+
+ switch (reg_data) {
+ case 0x0:
+ case MAX77705_CHARGER_CONSTANT_CURRENT:
+ case MAX77705_CHARGER_CONSTANT_VOLTAGE:
+ *val = POWER_SUPPLY_STATUS_CHARGING;
+ return 0;
+ case MAX77705_CHARGER_END_OF_CHARGE:
+ case MAX77705_CHARGER_DONE:
+ *val = POWER_SUPPLY_STATUS_FULL;
+ return 0;
+ /* those values hard coded as in vendor kernel, because of */
+ /* failure to determine it's actual meaning. */
+ case 0x05:
+ case 0x06:
+ case 0x07:
+ *val = POWER_SUPPLY_STATUS_NOT_CHARGING;
+ return 0;
+ case 0x08:
+ case 0xA:
+ case 0xB:
+ *val = POWER_SUPPLY_STATUS_DISCHARGING;
+ return 0;
+ default:
+ *val = POWER_SUPPLY_STATUS_UNKNOWN;
+ return 0;
+ }
+
+ return 0;
+}
+
+static int max77705_get_vbus_state(struct regmap *regmap, int *value)
+{
+ int ret;
+ unsigned int charge_dtls;
+
+ ret = regmap_read(regmap, MAX77705_CHG_REG_DETAILS_00, &charge_dtls);
+ if (ret)
+ return ret;
+
+ charge_dtls = ((charge_dtls & MAX77705_CHGIN_DTLS) >>
+ MAX77705_CHGIN_DTLS_SHIFT);
+
+ switch (charge_dtls) {
+ case 0x00:
+ *value = POWER_SUPPLY_HEALTH_UNDERVOLTAGE;
+ break;
+ case 0x01:
+ *value = POWER_SUPPLY_HEALTH_UNDERVOLTAGE;
+ break;
+ case 0x02:
+ *value = POWER_SUPPLY_HEALTH_OVERVOLTAGE;
+ break;
+ case 0x03:
+ *value = POWER_SUPPLY_HEALTH_GOOD;
+ break;
+ default:
+ return 0;
+ }
+ return 0;
+}
+
+static int max77705_get_battery_health(struct max77705_charger_data *charger,
+ int *value)
+{
+ struct regmap *regmap = charger->regmap;
+ unsigned int bat_dtls;
+
+ regmap_read(regmap, MAX77705_CHG_REG_DETAILS_01, &bat_dtls);
+ bat_dtls = ((bat_dtls & MAX77705_BAT_DTLS) >> MAX77705_BAT_DTLS_SHIFT);
+
+ switch (bat_dtls) {
+ case MAX77705_BATTERY_NOBAT:
+ dev_dbg(charger->dev, "%s: No battery and the charger is suspended\n",
+ __func__);
+ *value = POWER_SUPPLY_HEALTH_NO_BATTERY;
+ break;
+ case MAX77705_BATTERY_PREQUALIFICATION:
+ dev_dbg(charger->dev, "%s: battery is okay but its voltage is low(~VPQLB)\n",
+ __func__);
+ break;
+ case MAX77705_BATTERY_DEAD:
+ dev_dbg(charger->dev, "%s: battery dead\n", __func__);
+ *value = POWER_SUPPLY_HEALTH_DEAD;
+ break;
+ case MAX77705_BATTERY_GOOD:
+ case MAX77705_BATTERY_LOWVOLTAGE:
+ *value = POWER_SUPPLY_HEALTH_GOOD;
+ break;
+ case MAX77705_BATTERY_OVERVOLTAGE:
+ dev_dbg(charger->dev, "%s: battery ovp\n", __func__);
+ *value = POWER_SUPPLY_HEALTH_OVERVOLTAGE;
+ break;
+ default:
+ dev_dbg(charger->dev, "%s: battery unknown\n", __func__);
+ *value = POWER_SUPPLY_HEALTH_UNSPEC_FAILURE;
+ break;
+ }
+
+ return 0;
+}
+
+static int max77705_get_health(struct max77705_charger_data *charger, int *val)
+{
+ struct regmap *regmap = charger->regmap;
+ int ret, is_online = 0;
+
+ ret = max77705_get_online(regmap, &is_online);
+ if (ret)
+ return ret;
+ if (is_online) {
+ ret = max77705_get_vbus_state(regmap, val);
+ if (ret || (*val != POWER_SUPPLY_HEALTH_GOOD))
+ return ret;
+ }
+ return max77705_get_battery_health(charger, val);
+}
+
+static int max77705_get_input_current(struct max77705_charger_data *charger,
+ int *val)
+{
+ unsigned int reg_data;
+ int get_current = 0;
+ struct regmap *regmap = charger->regmap;
+
+ regmap_read(regmap, MAX77705_CHG_REG_CNFG_09, ®_data);
+
+ reg_data &= MAX77705_CHG_CHGIN_LIM_MASK;
+
+ if (reg_data <= 3)
+ get_current = 100;
+ else if (reg_data >= MAX77705_CHG_CHGIN_LIM_MASK)
+ get_current = MAX77705_CURRENT_CHGIN_MAX;
+ else
+ get_current = (reg_data + 1) * 25;
+
+ *val = get_current;
+
+ return 0;
+}
+
+static int max77705_get_charge_current(struct max77705_charger_data *charger,
+ int *val)
+{
+ unsigned int reg_data;
+ struct regmap *regmap = charger->regmap;
+
+ regmap_read(regmap, MAX77705_CHG_REG_CNFG_02, ®_data);
+ reg_data &= MAX77705_CHG_CC;
+
+ *val = reg_data <= 0x2 ? 100 : reg_data * 50;
+
+ return 0;
+}
+
+static int max77705_set_float_voltage(struct max77705_charger_data *charger,
+ int float_voltage)
+{
+ int float_voltage_mv;
+ unsigned int reg_data = 0;
+ struct regmap *regmap = charger->regmap;
+
+ float_voltage_mv = float_voltage / 1000;
+ reg_data = float_voltage_mv <= 4000 ? 0x0 :
+ float_voltage_mv >= 4500 ? 0x23 :
+ (float_voltage_mv <= 4200) ? (float_voltage_mv - 4000) / 50 :
+ (((float_voltage_mv - 4200) / 10) + 0x04);
+
+ return regmap_update_bits(regmap, MAX77705_CHG_REG_CNFG_04,
+ MAX77705_CHG_CV_PRM_MASK,
+ (reg_data << MAX77705_CHG_CV_PRM_SHIFT));
+}
+
+static int max77705_get_float_voltage(struct max77705_charger_data *charger,
+ int *val)
+{
+ unsigned int reg_data = 0;
+ struct regmap *regmap = charger->regmap;
+
+ regmap_read(regmap, MAX77705_CHG_REG_CNFG_04, ®_data);
+ reg_data &= MAX77705_CHG_PRM_MASK;
+ *val = reg_data <= 0x04 ? reg_data * 50 + 4000 :
+ (reg_data - 4) * 10 + 4200;
+
+ return 0;
+}
+
+static int max77705_chg_get_property(struct power_supply *psy,
+ enum power_supply_property psp,
+ union power_supply_propval *val)
+{
+ struct max77705_charger_data *charger = power_supply_get_drvdata(psy);
+ struct regmap *regmap = charger->regmap;
+
+ switch (psp) {
+ case POWER_SUPPLY_PROP_ONLINE:
+ return max77705_get_online(regmap, &val->intval);
+ case POWER_SUPPLY_PROP_PRESENT:
+ return max77705_check_battery(charger, &val->intval);
+ case POWER_SUPPLY_PROP_STATUS:
+ return max77705_get_status(charger, &val->intval);
+ case POWER_SUPPLY_PROP_CHARGE_TYPE:
+ return max77705_get_charge_type(charger, &val->intval);
+ case POWER_SUPPLY_PROP_HEALTH:
+ return max77705_get_health(charger, &val->intval);
+ case POWER_SUPPLY_PROP_INPUT_CURRENT_LIMIT:
+ return max77705_get_input_current(charger, &val->intval);
+ case POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT:
+ return max77705_get_charge_current(charger, &val->intval);
+ case POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE:
+ return max77705_get_float_voltage(charger, &val->intval);
+ case POWER_SUPPLY_PROP_VOLTAGE_MAX_DESIGN:
+ val->intval = charger->bat_info->voltage_max_design_uv;
+ break;
+ case POWER_SUPPLY_PROP_MODEL_NAME:
+ val->strval = max77705_charger_model;
+ break;
+ case POWER_SUPPLY_PROP_MANUFACTURER:
+ val->strval = max77705_charger_manufacturer;
+ break;
+ default:
+ return -EINVAL;
+ }
+ return 0;
+}
+
+static const struct power_supply_desc max77705_charger_psy_desc = {
+ .name = "max77705-charger",
+ .type = POWER_SUPPLY_TYPE_USB,
+ .properties = max77705_charger_props,
+ .num_properties = ARRAY_SIZE(max77705_charger_props),
+ .get_property = max77705_chg_get_property,
+};
+
+static void max77705_chgin_isr_work(struct work_struct *work)
+{
+ struct max77705_charger_data *charger =
+ container_of(work, struct max77705_charger_data, chgin_work);
+
+ power_supply_changed(charger->psy_chg);
+}
+
+static void max77705_charger_initialize(struct max77705_charger_data *chg)
+{
+ u8 reg_data;
+ struct power_supply_battery_info *info;
+ struct regmap *regmap = chg->regmap;
+
+ if (power_supply_get_battery_info(chg->psy_chg, &info) < 0)
+ return;
+
+ chg->bat_info = info;
+
+ /* unlock charger setting protect */
+ /* slowest LX slope */
+ reg_data = MAX77705_CHGPROT_MASK | MAX77705_SLOWEST_LX_SLOPE;
+ regmap_update_bits(regmap, MAX77705_CHG_REG_CNFG_06, reg_data,
+ reg_data);
+
+ /* fast charge timer disable */
+ /* restart threshold disable */
+ /* pre-qual charge disable */
+ reg_data = (MAX77705_FCHGTIME_DISABLE << MAX77705_FCHGTIME_SHIFT) |
+ (MAX77705_CHG_RSTRT_DISABLE << MAX77705_CHG_RSTRT_SHIFT) |
+ (MAX77705_CHG_PQEN_DISABLE << MAX77705_PQEN_SHIFT);
+ regmap_update_bits(regmap, MAX77705_CHG_REG_CNFG_01,
+ (MAX77705_FCHGTIME_MASK |
+ MAX77705_CHG_RSTRT_MASK |
+ MAX77705_PQEN_MASK),
+ reg_data);
+
+ /* OTG off(UNO on), boost off */
+ regmap_update_bits(regmap, MAX77705_CHG_REG_CNFG_00,
+ MAX77705_OTG_CTRL, 0);
+
+ /* charge current 450mA(default) */
+ /* otg current limit 900mA */
+ regmap_update_bits(regmap, MAX77705_CHG_REG_CNFG_02,
+ MAX77705_OTG_ILIM_MASK,
+ MAX77705_OTG_ILIM_900 << MAX77705_OTG_ILIM_SHIFT);
+
+ /* BAT to SYS OCP 4.80A */
+ regmap_update_bits(regmap, MAX77705_CHG_REG_CNFG_05,
+ MAX77705_REG_B2SOVRC_MASK,
+ MAX77705_B2SOVRC_4_8A << MAX77705_REG_B2SOVRC_SHIFT);
+ /* top off current 150mA */
+ /* top off timer 30min */
+ reg_data = (MAX77705_TO_ITH_150MA << MAX77705_TO_ITH_SHIFT) |
+ (MAX77705_TO_TIME_30M << MAX77705_TO_TIME_SHIFT) |
+ (MAX77705_SYS_TRACK_DISABLE << MAX77705_SYS_TRACK_DIS_SHIFT);
+ regmap_update_bits(regmap, MAX77705_CHG_REG_CNFG_03,
+ (MAX77705_TO_ITH_MASK |
+ MAX77705_TO_TIME_MASK |
+ MAX77705_SYS_TRACK_DIS_MASK), reg_data);
+
+ /* cv voltage 4.2V or 4.35V */
+ /* MINVSYS 3.6V(default) */
+ if (info->voltage_max_design_uv < 0) {
+ dev_warn(chg->dev, "missing battery:voltage-max-design-microvolt\n");
+ max77705_set_float_voltage(chg, 4200000);
+ } else {
+ max77705_set_float_voltage(chg, info->voltage_max_design_uv);
+ }
+
+ regmap_update_bits(regmap, MAX77705_CHG_REG_CNFG_12,
+ MAX77705_VCHGIN_REG_MASK, MAX77705_VCHGIN_4_5);
+ regmap_update_bits(regmap, MAX77705_CHG_REG_CNFG_12,
+ MAX77705_WCIN_REG_MASK, MAX77705_WCIN_4_5);
+
+ /* Watchdog timer */
+ regmap_update_bits(regmap, MAX77705_CHG_REG_CNFG_00,
+ MAX77705_WDTEN_MASK, 0);
+
+ /* Active Discharge Enable */
+ regmap_update_bits(regmap, MAX77705_PMIC_REG_MAINCTRL1, 1, 1);
+
+ /* VBYPSET=5.0V */
+ regmap_update_bits(regmap, MAX77705_CHG_REG_CNFG_11, MAX77705_VBYPSET_MASK, 0);
+
+ /* Switching Frequency : 1.5MHz */
+ regmap_update_bits(regmap, MAX77705_CHG_REG_CNFG_08, MAX77705_REG_FSW_MASK,
+ (MAX77705_CHG_FSW_1_5MHz << MAX77705_REG_FSW_SHIFT));
+
+ /* Auto skip mode */
+ regmap_update_bits(regmap, MAX77705_CHG_REG_CNFG_12, MAX77705_REG_DISKIP_MASK,
+ (MAX77705_AUTO_SKIP << MAX77705_REG_DISKIP_SHIFT));
+}
+
+static int max77705_charger_probe(struct i2c_client *i2c)
+{
+ struct power_supply_config pscfg = {};
+ struct max77705_charger_data *chg;
+ struct device *dev;
+ struct regmap_irq_chip_data *irq_data;
+ int ret;
+
+ dev = &i2c->dev;
+
+ chg = devm_kzalloc(dev, sizeof(*chg), GFP_KERNEL);
+ if (!chg)
+ return -ENOMEM;
+
+ chg->dev = dev;
+ i2c_set_clientdata(i2c, chg);
+
+ chg->regmap = devm_regmap_init_i2c(i2c, &max77705_chg_regmap_config);
+ if (IS_ERR(chg->regmap))
+ return PTR_ERR(chg->regmap);
+
+ ret = regmap_update_bits(chg->regmap,
+ MAX77705_CHG_REG_INT_MASK,
+ MAX77705_CHGIN_IM, 0);
+ if (ret)
+ return ret;
+
+ pscfg.of_node = dev->of_node;
+ pscfg.drv_data = chg;
+
+ chg->psy_chg = devm_power_supply_register(dev, &max77705_charger_psy_desc, &pscfg);
+ if (IS_ERR(chg->psy_chg))
+ return PTR_ERR(chg->psy_chg);
+
+ max77705_charger_irq_chip.irq_drv_data = chg;
+ ret = devm_regmap_add_irq_chip(chg->dev, chg->regmap, i2c->irq,
+ IRQF_ONESHOT | IRQF_SHARED, 0,
+ &max77705_charger_irq_chip,
+ &irq_data);
+ if (ret)
+ return dev_err_probe(dev, ret, "failed to add irq chip\n");
+
+ chg->wqueue = create_singlethread_workqueue(dev_name(dev));
+ if (IS_ERR(chg->wqueue))
+ return dev_err_probe(dev, PTR_ERR(chg->wqueue), "failed to create workqueue\n");
+
+ INIT_WORK(&chg->chgin_work, max77705_chgin_isr_work);
+
+ max77705_charger_initialize(chg);
+
+ ret = max77705_charger_enable(chg);
+ if (ret)
+ return dev_err_probe(dev, ret, "failed to enable charge\n");
+ else
+ return devm_add_action_or_reset(dev, max77705_charger_disable, chg);
+}
+
+static const struct of_device_id max77705_charger_of_match[] = {
+ { .compatible = "maxim,max77705-charger" },
+ { }
+};
+MODULE_DEVICE_TABLE(of, max77705_charger_of_match);
+
+static struct i2c_driver max77705_charger_driver = {
+ .driver = {
+ .name = "max77705-charger",
+ .of_match_table = max77705_charger_of_match,
+ },
+ .probe = max77705_charger_probe,
+};
+module_i2c_driver(max77705_charger_driver);
+
+MODULE_AUTHOR("Dzmitry Sankouski <dsankouski@gmail.com>");
+MODULE_DESCRIPTION("Maxim MAX77705 charger driver");
+MODULE_LICENSE("GPL");
diff --git a/include/linux/power/max77705_charger.h b/include/linux/power/max77705_charger.h
new file mode 100644
index 000000000000..cb5cd03e54e2
--- /dev/null
+++ b/include/linux/power/max77705_charger.h
@@ -0,0 +1,194 @@
+/* SPDX-License-Identifier: GPL-2.0 */
+/*
+ * Maxim MAX77705 definitions.
+ *
+ * Copyright (C) 2015 Samsung Electronics, Inc.
+ * Copyright (C) 2024 Dzmitry Sankouski <dsankouski@gmail.com>
+ */
+
+#ifndef __MAX77705_CHARGER_H
+#define __MAX77705_CHARGER_H __FILE__
+
+/* MAX77705_CHG_REG_CHG_INT */
+#define MAX77705_BYP_I BIT(0)
+#define MAX77705_INP_LIMIT_I BIT(1)
+#define MAX77705_BATP_I BIT(2)
+#define MAX77705_BAT_I BIT(3)
+#define MAX77705_CHG_I BIT(4)
+#define MAX77705_WCIN_I BIT(5)
+#define MAX77705_CHGIN_I BIT(6)
+#define MAX77705_AICL_I BIT(7)
+
+/* MAX77705_CHG_REG_CHG_INT_MASK */
+#define MAX77705_BYP_IM BIT(0)
+#define MAX77705_INP_LIMIT_IM BIT(1)
+#define MAX77705_BATP_IM BIT(2)
+#define MAX77705_BAT_IM BIT(3)
+#define MAX77705_CHG_IM BIT(4)
+#define MAX77705_WCIN_IM BIT(5)
+#define MAX77705_CHGIN_IM BIT(6)
+#define MAX77705_AICL_IM BIT(7)
+
+/* MAX77705_CHG_REG_CHG_INT_OK */
+#define MAX77705_BYP_OK BIT(0)
+#define MAX77705_DISQBAT_OK BIT(1)
+#define MAX77705_BATP_OK BIT(2)
+#define MAX77705_BAT_OK BIT(3)
+#define MAX77705_CHG_OK BIT(4)
+#define MAX77705_WCIN_OK BIT(5)
+#define MAX77705_CHGIN_OK BIT(6)
+#define MAX77705_AICL_OK BIT(7)
+
+/* MAX77705_CHG_REG_DETAILS_00 */
+#define MAX77705_BATP_DTLS BIT(0)
+#define MAX77705_WCIN_DTLS GENMASK(4, 3)
+#define MAX77705_WCIN_DTLS_SHIFT 3
+#define MAX77705_CHGIN_DTLS GENMASK(6, 5)
+#define MAX77705_CHGIN_DTLS_SHIFT 5
+
+/* MAX77705_CHG_REG_DETAILS_01 */
+#define MAX77705_CHG_DTLS GENMASK(3, 0)
+#define MAX77705_CHG_DTLS_SHIFT 0
+#define MAX77705_BAT_DTLS GENMASK(6, 4)
+#define MAX77705_BAT_DTLS_SHIFT 4
+
+/* MAX77705_CHG_REG_DETAILS_02 */
+#define MAX77705_BYP_DTLS GENMASK(3, 0)
+#define MAX77705_BYP_DTLS_SHIFT 0
+
+/* MAX77705_CHG_REG_CNFG_00 */
+#define MAX77705_CHG_SHIFT 0
+#define MAX77705_UNO_SHIFT 1
+#define MAX77705_OTG_SHIFT 1
+#define MAX77705_BUCK_SHIFT 2
+#define MAX77705_BOOST_SHIFT 3
+#define MAX77705_WDTEN_SHIFT 4
+#define MAX77705_MODE_MASK GENMASK(3, 0)
+#define MAX77705_CHG_MASK BIT(MAX77705_CHG_SHIFT)
+#define MAX77705_UNO_MASK BIT(MAX77705_UNO_SHIFT)
+#define MAX77705_OTG_MASK BIT(MAX77705_OTG_SHIFT)
+#define MAX77705_BUCK_MASK BIT(MAX77705_BUCK_SHIFT)
+#define MAX77705_BOOST_MASK BIT(MAX77705_BOOST_SHIFT)
+#define MAX77705_WDTEN_MASK BIT(MAX77705_WDTEN_SHIFT)
+#define MAX77705_UNO_CTRL (MAX77705_UNO_MASK | MAX77705_BOOST_MASK)
+#define MAX77705_OTG_CTRL (MAX77705_OTG_MASK | MAX77705_BOOST_MASK)
+
+/* MAX77705_CHG_REG_CNFG_01 */
+#define MAX77705_FCHGTIME_SHIFT 0
+#define MAX77705_FCHGTIME_MASK GENMASK(2, 0)
+#define MAX77705_CHG_RSTRT_SHIFT 4
+#define MAX77705_CHG_RSTRT_MASK GENMASK(5, 4)
+#define MAX77705_FCHGTIME_DISABLE 0
+#define MAX77705_CHG_RSTRT_DISABLE 0x3
+
+#define MAX77705_PQEN_SHIFT 7
+#define MAX77705_PQEN_MASK BIT(7)
+#define MAX77705_CHG_PQEN_DISABLE 0
+#define MAX77705_CHG_PQEN_ENABLE 1
+
+/* MAX77705_CHG_REG_CNFG_02 */
+#define MAX77705_OTG_ILIM_SHIFT 6
+#define MAX77705_OTG_ILIM_MASK GENMASK(7, 6)
+#define MAX77705_OTG_ILIM_500 0
+#define MAX77705_OTG_ILIM_900 1
+#define MAX77705_OTG_ILIM_1200 2
+#define MAX77705_OTG_ILIM_1500 3
+#define MAX77705_CHG_CC GENMASK(5, 0)
+
+/* MAX77705_CHG_REG_CNFG_03 */
+#define MAX77705_TO_ITH_SHIFT 0
+#define MAX77705_TO_ITH_MASK GENMASK(2, 0)
+#define MAX77705_TO_TIME_SHIFT 3
+#define MAX77705_TO_TIME_MASK GENMASK(5, 3)
+#define MAX77705_SYS_TRACK_DIS_SHIFT 7
+#define MAX77705_SYS_TRACK_DIS_MASK BIT(7)
+#define MAX77705_TO_ITH_150MA 0
+#define MAX77705_TO_TIME_30M 3
+#define MAX77705_SYS_TRACK_ENABLE 0
+#define MAX77705_SYS_TRACK_DISABLE 1
+
+/* MAX77705_CHG_REG_CNFG_04 */
+#define MAX77705_CHG_MINVSYS_SHIFT 6
+#define MAX77705_CHG_MINVSYS_MASK GENMASK(7, 6)
+#define MAX77705_CHG_PRM_SHIFT 0
+#define MAX77705_CHG_PRM_MASK GENMASK(5, 0)
+
+#define MAX77705_CHG_CV_PRM_SHIFT 0
+#define MAX77705_CHG_CV_PRM_MASK GENMASK(5, 0)
+
+/* MAX77705_CHG_REG_CNFG_05 */
+#define MAX77705_REG_B2SOVRC_SHIFT 0
+#define MAX77705_REG_B2SOVRC_MASK GENMASK(3, 0)
+#define MAX77705_B2SOVRC_DISABLE 0
+#define MAX77705_B2SOVRC_4_5A 6
+#define MAX77705_B2SOVRC_4_8A 8
+#define MAX77705_B2SOVRC_5_0A 9
+
+/* MAX77705_CHG_CNFG_06 */
+#define MAX77705_WDTCLR_SHIFT 0
+#define MAX77705_WDTCLR_MASK GENMASK(1, 0)
+#define MAX77705_WDTCLR 1
+#define MAX77705_CHGPROT_MASK GENMASK(3, 2)
+#define MAX77705_CHGPROT_UNLOCKED GENMASK(3, 2)
+#define MAX77705_SLOWEST_LX_SLOPE GENMASK(6, 5)
+
+/* MAX77705_CHG_REG_CNFG_07 */
+#define MAX77705_CHG_FMBST 4
+#define MAX77705_REG_FMBST_SHIFT 2
+#define MAX77705_REG_FMBST_MASK BIT(MAX77705_REG_FMBST_SHIFT)
+#define MAX77705_REG_FGSRC_SHIFT 1
+#define MAX77705_REG_FGSRC_MASK BIT(MAX77705_REG_FGSRC_SHIFT)
+
+/* MAX77705_CHG_REG_CNFG_08 */
+#define MAX77705_REG_FSW_SHIFT 0
+#define MAX77705_REG_FSW_MASK GENMASK(1, 0)
+#define MAX77705_CHG_FSW_3MHz 0
+#define MAX77705_CHG_FSW_2MHz 1
+#define MAX77705_CHG_FSW_1_5MHz 2
+
+/* MAX77705_CHG_REG_CNFG_09 */
+#define MAX77705_CHG_CHGIN_LIM_MASK GENMASK(6, 0)
+#define MAX77705_CHG_EN_MASK BIT(7)
+#define MAX77705_CHG_DISABLE 0
+#define MAX77705_CHARGER_CHG_CHARGING(_reg) \
+ (((_reg) & MAX77705_CHG_EN_MASK) > 1)
+
+
+/* MAX77705_CHG_REG_CNFG_10 */
+#define MAX77705_CHG_WCIN_LIM GENMASK(5, 0)
+
+/* MAX77705_CHG_REG_CNFG_11 */
+#define MAX77705_VBYPSET_SHIFT 0
+#define MAX77705_VBYPSET_MASK GENMASK(6, 0)
+
+/* MAX77705_CHG_REG_CNFG_12 */
+#define MAX77705_CHGINSEL_SHIFT 5
+#define MAX77705_CHGINSEL_MASK BIT(MAX77705_CHGINSEL_SHIFT)
+#define MAX77705_WCINSEL_SHIFT 6
+#define MAX77705_WCINSEL_MASK BIT(MAX77705_WCINSEL_SHIFT)
+#define MAX77705_VCHGIN_REG_MASK GENMASK(4, 3)
+#define MAX77705_WCIN_REG_MASK GENMASK(2, 1)
+#define MAX77705_REG_DISKIP_SHIFT 0
+#define MAX77705_REG_DISKIP_MASK BIT(MAX77705_REG_DISKIP_SHIFT)
+/* REG=4.5V, UVLO=4.7V */
+#define MAX77705_VCHGIN_4_5 0
+/* REG=4.5V, UVLO=4.7V */
+#define MAX77705_WCIN_4_5 0
+#define MAX77705_DISABLE_SKIP 1
+#define MAX77705_AUTO_SKIP 0
+
+/* mA */
+#define MAX77705_CURRENT_STEP 25
+#define MAX77705_CURRENT_WCIN_MAX 1600
+#define MAX77705_CURRENT_CHGIN_MAX 3200
+
+struct max77705_charger_data {
+ struct device *dev;
+ struct regmap *regmap;
+ struct power_supply_battery_info *bat_info;
+ struct workqueue_struct *wqueue;
+ struct work_struct chgin_work;
+ struct power_supply *psy_chg;
+};
+
+#endif /* __MAX77705_CHARGER_H */
--
2.39.5
^ permalink raw reply related [flat|nested] 23+ messages in thread* Re: [PATCH v14 06/10] power: supply: max77705: Add charger driver for Maxim 77705
2025-01-08 14:13 ` [PATCH v14 06/10] power: supply: max77705: Add charger driver for Maxim 77705 Dzmitry Sankouski
@ 2025-01-16 0:16 ` Sebastian Reichel
2025-01-16 12:44 ` Dzmitry Sankouski
0 siblings, 1 reply; 23+ messages in thread
From: Sebastian Reichel @ 2025-01-16 0:16 UTC (permalink / raw)
To: Dzmitry Sankouski
Cc: Chanwoo Choi, Krzysztof Kozlowski, Lee Jones, Rob Herring,
Conor Dooley, Dmitry Torokhov, Pavel Machek, Hans de Goede,
Marek Szyprowski, Sebastian Krzyszkowiak, Purism Kernel Team,
Krzysztof Kozlowski, linux-pm, linux-kernel, devicetree,
linux-input, linux-leds
[-- Attachment #1: Type: text/plain, Size: 22996 bytes --]
Hi,
On Wed, Jan 08, 2025 at 05:13:50PM +0300, Dzmitry Sankouski wrote:
> Add driver for Maxim 77705 switch-mode charger.
> It providing power supply class information to userspace.
>
> The driver is configured through DTS (battery and system related
> settings).
>
> Signed-off-by: Dzmitry Sankouski <dsankouski@gmail.com>
> ---
> Changes in v12:
> - add missing new line
> - reorder max77705_charger_enable and max77705_charger_disable action
> - convert driver from platform to i2c, adjust commit msg accordingly
>
> Changes in v11:
> - remove multiple blank lines
> - return dev_err_probe
> - fit on one line where possible;
>
> Changes in v10:
> - never blank line between call and its error check
> - replace remove function with devm_add_action_or_reset
> - remove unused inline functions from header
> - use dev_err_probe for error handling
>
> Changes in v9:
> - move power supply registration before interrupts to prevent
> NULL exceptions when handling interrupts
>
> Changes for v8:
> - join lines, where fits 100 chars
> - change comment style C++ -> C
> - remove author from 'based on' file header statement
> Changes for v6:
> - add i2c init in driver
> - replace remove_new back on remove
> - handle IS_ERR(i2c_chg)
> Changes for v5:
> - remove const modifier from max77705_charger_irq_chip
> because it's modified with irq_drv_data in probe function
> - fix license to GPL 2.0 only, where old vendor code used
> GPL 2.0 only
> - move power header to power include dir
> - use same hardware name in Kconfig and module descriptions
>
> Changes for v4:
> - start from scratch
> - change word delimiters in filenames to '_'
> - use GENMASK in header
> - remove debugfs code
> - migrate to regmap_add_irq_chip
> - fix property getters to follow the same style
Please add output from tools/testing/selftests/power_supply/test_power_supply_properties.sh
below the fold in the next round. It looks to me, like some properties are
wrongly scaled to mA/mV instead of uA/uV.
> drivers/power/supply/Kconfig | 6 ++
> drivers/power/supply/Makefile | 1 +
> drivers/power/supply/max77705_charger.c | 576 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> include/linux/power/max77705_charger.h | 194 +++++++++++++++++++++++++++++++++++++++++++++++
> 4 files changed, 777 insertions(+)
>
> diff --git a/drivers/power/supply/Kconfig b/drivers/power/supply/Kconfig
> index 7b18358f194a..c120d8ed870f 100644
> --- a/drivers/power/supply/Kconfig
> +++ b/drivers/power/supply/Kconfig
> @@ -583,6 +583,12 @@ config CHARGER_MAX77693
> help
> Say Y to enable support for the Maxim MAX77693 battery charger.
>
> +config CHARGER_MAX77705
> + tristate "Maxim MAX77705 battery charger driver"
> + depends on MFD_MAX77705
> + help
> + Say Y to enable support for the Maxim MAX77705 battery charger.
> +
> config CHARGER_MAX77976
> tristate "Maxim MAX77976 battery charger driver"
> depends on I2C
> diff --git a/drivers/power/supply/Makefile b/drivers/power/supply/Makefile
> index b55cc48a4c86..a4669383d53e 100644
> --- a/drivers/power/supply/Makefile
> +++ b/drivers/power/supply/Makefile
> @@ -80,6 +80,7 @@ obj-$(CONFIG_CHARGER_MAX14577) += max14577_charger.o
> obj-$(CONFIG_CHARGER_DETECTOR_MAX14656) += max14656_charger_detector.o
> obj-$(CONFIG_CHARGER_MAX77650) += max77650-charger.o
> obj-$(CONFIG_CHARGER_MAX77693) += max77693_charger.o
> +obj-$(CONFIG_CHARGER_MAX77705) += max77705_charger.o
> obj-$(CONFIG_CHARGER_MAX77976) += max77976_charger.o
> obj-$(CONFIG_CHARGER_MAX8997) += max8997_charger.o
> obj-$(CONFIG_CHARGER_MAX8998) += max8998_charger.o
> diff --git a/drivers/power/supply/max77705_charger.c b/drivers/power/supply/max77705_charger.c
> new file mode 100644
> index 000000000000..563b41420c7c
> --- /dev/null
> +++ b/drivers/power/supply/max77705_charger.c
> @@ -0,0 +1,576 @@
> +// SPDX-License-Identifier: GPL-2.0
> +/*
> + * Based on max77650-charger.c
> + *
> + * Copyright (C) 2024 Dzmitry Sankouski <dsankouski@gmail.org>
> + *
> + * Battery charger driver for MAXIM 77705 charger/power-supply.
> + */
> +
> +#include <linux/i2c.h>
> +#include <linux/interrupt.h>
> +#include <linux/mfd/max77693-common.h>
> +#include <linux/mfd/max77705-private.h>
> +#include <linux/power/max77705_charger.h>
> +#include <linux/module.h>
> +#include <linux/platform_device.h>
> +#include <linux/power_supply.h>
> +#include <linux/regmap.h>
> +
> +static const char *max77705_charger_model = "max77705";
> +static const char *max77705_charger_manufacturer = "Maxim Integrated";
> +
> +static const struct regmap_config max77705_chg_regmap_config = {
> + .reg_base = MAX77705_CHG_REG_BASE,
> + .reg_bits = 8,
> + .val_bits = 8,
> + .max_register = MAX77705_CHG_REG_SAFEOUT_CTRL,
> +};
> +
> +static enum power_supply_property max77705_charger_props[] = {
> + POWER_SUPPLY_PROP_ONLINE,
> + POWER_SUPPLY_PROP_PRESENT,
> + POWER_SUPPLY_PROP_STATUS,
> + POWER_SUPPLY_PROP_CHARGE_TYPE,
> + POWER_SUPPLY_PROP_HEALTH,
> + POWER_SUPPLY_PROP_VOLTAGE_MAX_DESIGN,
> + POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE,
> + POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT,
> + POWER_SUPPLY_PROP_INPUT_CURRENT_LIMIT,
> +};
> +
> +static int max77705_chgin_irq(void *irq_drv_data)
> +{
> + struct max77705_charger_data *charger = irq_drv_data;
> +
> + queue_work(charger->wqueue, &charger->chgin_work);
> +
> + return 0;
> +}
> +
> +static const struct regmap_irq max77705_charger_irqs[] = {
> + { .mask = MAX77705_BYP_IM, },
> + { .mask = MAX77705_INP_LIMIT_IM, },
> + { .mask = MAX77705_BATP_IM, },
> + { .mask = MAX77705_BAT_IM, },
> + { .mask = MAX77705_CHG_IM, },
> + { .mask = MAX77705_WCIN_IM, },
> + { .mask = MAX77705_CHGIN_IM, },
> + { .mask = MAX77705_AICL_IM, },
> +};
> +
> +static struct regmap_irq_chip max77705_charger_irq_chip = {
> + .name = "max77705-charger",
> + .status_base = MAX77705_CHG_REG_INT,
> + .mask_base = MAX77705_CHG_REG_INT_MASK,
> + .handle_post_irq = max77705_chgin_irq,
> + .num_regs = 1,
> + .irqs = max77705_charger_irqs,
> + .num_irqs = ARRAY_SIZE(max77705_charger_irqs),
> +};
> +
> +static int max77705_charger_enable(struct max77705_charger_data *chg)
> +{
> + int rv;
> +
> + rv = regmap_update_bits(chg->regmap, MAX77705_CHG_REG_CNFG_09,
> + MAX77705_CHG_EN_MASK, MAX77705_CHG_EN_MASK);
> + if (rv)
> + dev_err(chg->dev, "unable to enable the charger: %d\n", rv);
> +
> + return rv;
> +}
> +
> +static void max77705_charger_disable(void *data)
> +{
> + struct max77705_charger_data *chg = data;
> + int rv;
> +
> + rv = regmap_update_bits(chg->regmap,
> + MAX77705_CHG_REG_CNFG_09,
> + MAX77705_CHG_EN_MASK,
> + MAX77705_CHG_DISABLE);
> + if (rv)
> + dev_err(chg->dev, "unable to disable the charger: %d\n", rv);
> +}
> +
> +static int max77705_get_online(struct regmap *regmap, int *val)
> +{
> + unsigned int data;
> + int ret;
> +
> + ret = regmap_read(regmap, MAX77705_CHG_REG_INT_OK, &data);
> + if (ret < 0)
> + return ret;
> +
> + *val = !!(data & MAX77705_CHGIN_OK);
> +
> + return 0;
> +}
> +
> +static int max77705_check_battery(struct max77705_charger_data *charger, int *val)
> +{
> + unsigned int reg_data;
> + unsigned int reg_data2;
> + struct regmap *regmap = charger->regmap;
> +
> + regmap_read(regmap, MAX77705_CHG_REG_INT_OK, ®_data);
> +
> + dev_dbg(charger->dev, "CHG_INT_OK(0x%x)\n", reg_data);
> +
> + regmap_read(regmap, MAX77705_CHG_REG_DETAILS_00, ®_data2);
> +
> + dev_dbg(charger->dev, "CHG_DETAILS00(0x%x)\n", reg_data2);
> +
> + if ((reg_data & MAX77705_BATP_OK) || !(reg_data2 & MAX77705_BATP_DTLS))
> + *val = true;
> + else
> + *val = false;
> +
> + return 0;
> +}
> +
> +static int max77705_get_charge_type(struct max77705_charger_data *charger, int *val)
> +{
> + struct regmap *regmap = charger->regmap;
> + unsigned int reg_data;
> +
> + regmap_read(regmap, MAX77705_CHG_REG_CNFG_09, ®_data);
> + if (!MAX77705_CHARGER_CHG_CHARGING(reg_data)) {
> + *val = POWER_SUPPLY_CHARGE_TYPE_NONE;
> + return 0;
> + }
> +
> + regmap_read(regmap, MAX77705_CHG_REG_DETAILS_01, ®_data);
> + reg_data &= MAX77705_CHG_DTLS;
> +
> + switch (reg_data) {
> + case 0x0:
> + case MAX77705_CHARGER_CONSTANT_CURRENT:
> + case MAX77705_CHARGER_CONSTANT_VOLTAGE:
> + *val = POWER_SUPPLY_CHARGE_TYPE_FAST;
> + return 0;
> + default:
> + *val = POWER_SUPPLY_CHARGE_TYPE_NONE;
> + return 0;
> + }
> +
> + return 0;
> +}
> +
> +static int max77705_get_status(struct max77705_charger_data *charger, int *val)
> +{
> + struct regmap *regmap = charger->regmap;
> + unsigned int reg_data;
> +
> + regmap_read(regmap, MAX77705_CHG_REG_CNFG_09, ®_data);
> + if (!MAX77705_CHARGER_CHG_CHARGING(reg_data)) {
> + *val = POWER_SUPPLY_CHARGE_TYPE_NONE;
> + return 0;
> + }
> +
> + regmap_read(regmap, MAX77705_CHG_REG_DETAILS_01, ®_data);
> + reg_data &= MAX77705_CHG_DTLS;
> +
> + switch (reg_data) {
> + case 0x0:
> + case MAX77705_CHARGER_CONSTANT_CURRENT:
> + case MAX77705_CHARGER_CONSTANT_VOLTAGE:
> + *val = POWER_SUPPLY_STATUS_CHARGING;
> + return 0;
> + case MAX77705_CHARGER_END_OF_CHARGE:
> + case MAX77705_CHARGER_DONE:
> + *val = POWER_SUPPLY_STATUS_FULL;
> + return 0;
> + /* those values hard coded as in vendor kernel, because of */
> + /* failure to determine it's actual meaning. */
> + case 0x05:
> + case 0x06:
> + case 0x07:
> + *val = POWER_SUPPLY_STATUS_NOT_CHARGING;
> + return 0;
> + case 0x08:
> + case 0xA:
> + case 0xB:
> + *val = POWER_SUPPLY_STATUS_DISCHARGING;
> + return 0;
> + default:
> + *val = POWER_SUPPLY_STATUS_UNKNOWN;
> + return 0;
> + }
> +
> + return 0;
> +}
> +
> +static int max77705_get_vbus_state(struct regmap *regmap, int *value)
> +{
> + int ret;
> + unsigned int charge_dtls;
> +
> + ret = regmap_read(regmap, MAX77705_CHG_REG_DETAILS_00, &charge_dtls);
> + if (ret)
> + return ret;
> +
> + charge_dtls = ((charge_dtls & MAX77705_CHGIN_DTLS) >>
> + MAX77705_CHGIN_DTLS_SHIFT);
> +
> + switch (charge_dtls) {
> + case 0x00:
> + *value = POWER_SUPPLY_HEALTH_UNDERVOLTAGE;
> + break;
> + case 0x01:
> + *value = POWER_SUPPLY_HEALTH_UNDERVOLTAGE;
> + break;
> + case 0x02:
> + *value = POWER_SUPPLY_HEALTH_OVERVOLTAGE;
> + break;
> + case 0x03:
> + *value = POWER_SUPPLY_HEALTH_GOOD;
> + break;
> + default:
> + return 0;
> + }
> + return 0;
> +}
> +
> +static int max77705_get_battery_health(struct max77705_charger_data *charger,
> + int *value)
> +{
> + struct regmap *regmap = charger->regmap;
> + unsigned int bat_dtls;
> +
> + regmap_read(regmap, MAX77705_CHG_REG_DETAILS_01, &bat_dtls);
> + bat_dtls = ((bat_dtls & MAX77705_BAT_DTLS) >> MAX77705_BAT_DTLS_SHIFT);
> +
> + switch (bat_dtls) {
> + case MAX77705_BATTERY_NOBAT:
> + dev_dbg(charger->dev, "%s: No battery and the charger is suspended\n",
> + __func__);
> + *value = POWER_SUPPLY_HEALTH_NO_BATTERY;
> + break;
> + case MAX77705_BATTERY_PREQUALIFICATION:
> + dev_dbg(charger->dev, "%s: battery is okay but its voltage is low(~VPQLB)\n",
> + __func__);
> + break;
> + case MAX77705_BATTERY_DEAD:
> + dev_dbg(charger->dev, "%s: battery dead\n", __func__);
> + *value = POWER_SUPPLY_HEALTH_DEAD;
> + break;
> + case MAX77705_BATTERY_GOOD:
> + case MAX77705_BATTERY_LOWVOLTAGE:
> + *value = POWER_SUPPLY_HEALTH_GOOD;
> + break;
> + case MAX77705_BATTERY_OVERVOLTAGE:
> + dev_dbg(charger->dev, "%s: battery ovp\n", __func__);
> + *value = POWER_SUPPLY_HEALTH_OVERVOLTAGE;
> + break;
> + default:
> + dev_dbg(charger->dev, "%s: battery unknown\n", __func__);
> + *value = POWER_SUPPLY_HEALTH_UNSPEC_FAILURE;
> + break;
> + }
> +
> + return 0;
> +}
> +
> +static int max77705_get_health(struct max77705_charger_data *charger, int *val)
> +{
> + struct regmap *regmap = charger->regmap;
> + int ret, is_online = 0;
> +
> + ret = max77705_get_online(regmap, &is_online);
> + if (ret)
> + return ret;
> + if (is_online) {
> + ret = max77705_get_vbus_state(regmap, val);
> + if (ret || (*val != POWER_SUPPLY_HEALTH_GOOD))
> + return ret;
> + }
> + return max77705_get_battery_health(charger, val);
> +}
> +
> +static int max77705_get_input_current(struct max77705_charger_data *charger,
> + int *val)
> +{
> + unsigned int reg_data;
> + int get_current = 0;
> + struct regmap *regmap = charger->regmap;
> +
> + regmap_read(regmap, MAX77705_CHG_REG_CNFG_09, ®_data);
> +
> + reg_data &= MAX77705_CHG_CHGIN_LIM_MASK;
> +
> + if (reg_data <= 3)
> + get_current = 100;
> + else if (reg_data >= MAX77705_CHG_CHGIN_LIM_MASK)
> + get_current = MAX77705_CURRENT_CHGIN_MAX;
> + else
> + get_current = (reg_data + 1) * 25;
> +
> + *val = get_current;
> +
> + return 0;
> +}
> +
> +static int max77705_get_charge_current(struct max77705_charger_data *charger,
> + int *val)
> +{
> + unsigned int reg_data;
> + struct regmap *regmap = charger->regmap;
> +
> + regmap_read(regmap, MAX77705_CHG_REG_CNFG_02, ®_data);
> + reg_data &= MAX77705_CHG_CC;
> +
> + *val = reg_data <= 0x2 ? 100 : reg_data * 50;
> +
> + return 0;
> +}
> +
> +static int max77705_set_float_voltage(struct max77705_charger_data *charger,
> + int float_voltage)
> +{
> + int float_voltage_mv;
> + unsigned int reg_data = 0;
> + struct regmap *regmap = charger->regmap;
> +
> + float_voltage_mv = float_voltage / 1000;
> + reg_data = float_voltage_mv <= 4000 ? 0x0 :
> + float_voltage_mv >= 4500 ? 0x23 :
> + (float_voltage_mv <= 4200) ? (float_voltage_mv - 4000) / 50 :
> + (((float_voltage_mv - 4200) / 10) + 0x04);
> +
> + return regmap_update_bits(regmap, MAX77705_CHG_REG_CNFG_04,
> + MAX77705_CHG_CV_PRM_MASK,
> + (reg_data << MAX77705_CHG_CV_PRM_SHIFT));
> +}
> +
> +static int max77705_get_float_voltage(struct max77705_charger_data *charger,
> + int *val)
> +{
> + unsigned int reg_data = 0;
> + struct regmap *regmap = charger->regmap;
> +
> + regmap_read(regmap, MAX77705_CHG_REG_CNFG_04, ®_data);
> + reg_data &= MAX77705_CHG_PRM_MASK;
> + *val = reg_data <= 0x04 ? reg_data * 50 + 4000 :
> + (reg_data - 4) * 10 + 4200;
> +
> + return 0;
> +}
> +
> +static int max77705_chg_get_property(struct power_supply *psy,
> + enum power_supply_property psp,
> + union power_supply_propval *val)
> +{
> + struct max77705_charger_data *charger = power_supply_get_drvdata(psy);
> + struct regmap *regmap = charger->regmap;
> +
> + switch (psp) {
> + case POWER_SUPPLY_PROP_ONLINE:
> + return max77705_get_online(regmap, &val->intval);
> + case POWER_SUPPLY_PROP_PRESENT:
> + return max77705_check_battery(charger, &val->intval);
> + case POWER_SUPPLY_PROP_STATUS:
> + return max77705_get_status(charger, &val->intval);
> + case POWER_SUPPLY_PROP_CHARGE_TYPE:
> + return max77705_get_charge_type(charger, &val->intval);
> + case POWER_SUPPLY_PROP_HEALTH:
> + return max77705_get_health(charger, &val->intval);
> + case POWER_SUPPLY_PROP_INPUT_CURRENT_LIMIT:
> + return max77705_get_input_current(charger, &val->intval);
> + case POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT:
> + return max77705_get_charge_current(charger, &val->intval);
> + case POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE:
> + return max77705_get_float_voltage(charger, &val->intval);
> + case POWER_SUPPLY_PROP_VOLTAGE_MAX_DESIGN:
> + val->intval = charger->bat_info->voltage_max_design_uv;
> + break;
> + case POWER_SUPPLY_PROP_MODEL_NAME:
> + val->strval = max77705_charger_model;
> + break;
> + case POWER_SUPPLY_PROP_MANUFACTURER:
> + val->strval = max77705_charger_manufacturer;
> + break;
> + default:
> + return -EINVAL;
> + }
> + return 0;
> +}
> +
> +static const struct power_supply_desc max77705_charger_psy_desc = {
> + .name = "max77705-charger",
> + .type = POWER_SUPPLY_TYPE_USB,
> + .properties = max77705_charger_props,
> + .num_properties = ARRAY_SIZE(max77705_charger_props),
> + .get_property = max77705_chg_get_property,
> +};
> +
> +static void max77705_chgin_isr_work(struct work_struct *work)
> +{
> + struct max77705_charger_data *charger =
> + container_of(work, struct max77705_charger_data, chgin_work);
> +
> + power_supply_changed(charger->psy_chg);
> +}
> +
> +static void max77705_charger_initialize(struct max77705_charger_data *chg)
> +{
> + u8 reg_data;
> + struct power_supply_battery_info *info;
> + struct regmap *regmap = chg->regmap;
> +
> + if (power_supply_get_battery_info(chg->psy_chg, &info) < 0)
> + return;
> +
> + chg->bat_info = info;
> +
> + /* unlock charger setting protect */
> + /* slowest LX slope */
> + reg_data = MAX77705_CHGPROT_MASK | MAX77705_SLOWEST_LX_SLOPE;
> + regmap_update_bits(regmap, MAX77705_CHG_REG_CNFG_06, reg_data,
> + reg_data);
> +
> + /* fast charge timer disable */
> + /* restart threshold disable */
> + /* pre-qual charge disable */
> + reg_data = (MAX77705_FCHGTIME_DISABLE << MAX77705_FCHGTIME_SHIFT) |
> + (MAX77705_CHG_RSTRT_DISABLE << MAX77705_CHG_RSTRT_SHIFT) |
> + (MAX77705_CHG_PQEN_DISABLE << MAX77705_PQEN_SHIFT);
> + regmap_update_bits(regmap, MAX77705_CHG_REG_CNFG_01,
> + (MAX77705_FCHGTIME_MASK |
> + MAX77705_CHG_RSTRT_MASK |
> + MAX77705_PQEN_MASK),
> + reg_data);
> +
> + /* OTG off(UNO on), boost off */
> + regmap_update_bits(regmap, MAX77705_CHG_REG_CNFG_00,
> + MAX77705_OTG_CTRL, 0);
> +
> + /* charge current 450mA(default) */
> + /* otg current limit 900mA */
> + regmap_update_bits(regmap, MAX77705_CHG_REG_CNFG_02,
> + MAX77705_OTG_ILIM_MASK,
> + MAX77705_OTG_ILIM_900 << MAX77705_OTG_ILIM_SHIFT);
> +
> + /* BAT to SYS OCP 4.80A */
> + regmap_update_bits(regmap, MAX77705_CHG_REG_CNFG_05,
> + MAX77705_REG_B2SOVRC_MASK,
> + MAX77705_B2SOVRC_4_8A << MAX77705_REG_B2SOVRC_SHIFT);
> + /* top off current 150mA */
> + /* top off timer 30min */
> + reg_data = (MAX77705_TO_ITH_150MA << MAX77705_TO_ITH_SHIFT) |
> + (MAX77705_TO_TIME_30M << MAX77705_TO_TIME_SHIFT) |
> + (MAX77705_SYS_TRACK_DISABLE << MAX77705_SYS_TRACK_DIS_SHIFT);
> + regmap_update_bits(regmap, MAX77705_CHG_REG_CNFG_03,
> + (MAX77705_TO_ITH_MASK |
> + MAX77705_TO_TIME_MASK |
> + MAX77705_SYS_TRACK_DIS_MASK), reg_data);
> +
> + /* cv voltage 4.2V or 4.35V */
> + /* MINVSYS 3.6V(default) */
> + if (info->voltage_max_design_uv < 0) {
> + dev_warn(chg->dev, "missing battery:voltage-max-design-microvolt\n");
> + max77705_set_float_voltage(chg, 4200000);
> + } else {
> + max77705_set_float_voltage(chg, info->voltage_max_design_uv);
> + }
> +
> + regmap_update_bits(regmap, MAX77705_CHG_REG_CNFG_12,
> + MAX77705_VCHGIN_REG_MASK, MAX77705_VCHGIN_4_5);
> + regmap_update_bits(regmap, MAX77705_CHG_REG_CNFG_12,
> + MAX77705_WCIN_REG_MASK, MAX77705_WCIN_4_5);
> +
> + /* Watchdog timer */
> + regmap_update_bits(regmap, MAX77705_CHG_REG_CNFG_00,
> + MAX77705_WDTEN_MASK, 0);
> +
> + /* Active Discharge Enable */
> + regmap_update_bits(regmap, MAX77705_PMIC_REG_MAINCTRL1, 1, 1);
> +
> + /* VBYPSET=5.0V */
> + regmap_update_bits(regmap, MAX77705_CHG_REG_CNFG_11, MAX77705_VBYPSET_MASK, 0);
> +
> + /* Switching Frequency : 1.5MHz */
> + regmap_update_bits(regmap, MAX77705_CHG_REG_CNFG_08, MAX77705_REG_FSW_MASK,
> + (MAX77705_CHG_FSW_1_5MHz << MAX77705_REG_FSW_SHIFT));
> +
> + /* Auto skip mode */
> + regmap_update_bits(regmap, MAX77705_CHG_REG_CNFG_12, MAX77705_REG_DISKIP_MASK,
> + (MAX77705_AUTO_SKIP << MAX77705_REG_DISKIP_SHIFT));
> +}
> +
> +static int max77705_charger_probe(struct i2c_client *i2c)
> +{
> + struct power_supply_config pscfg = {};
> + struct max77705_charger_data *chg;
> + struct device *dev;
> + struct regmap_irq_chip_data *irq_data;
> + int ret;
> +
> + dev = &i2c->dev;
> +
> + chg = devm_kzalloc(dev, sizeof(*chg), GFP_KERNEL);
> + if (!chg)
> + return -ENOMEM;
> +
> + chg->dev = dev;
> + i2c_set_clientdata(i2c, chg);
> +
> + chg->regmap = devm_regmap_init_i2c(i2c, &max77705_chg_regmap_config);
> + if (IS_ERR(chg->regmap))
> + return PTR_ERR(chg->regmap);
> +
> + ret = regmap_update_bits(chg->regmap,
> + MAX77705_CHG_REG_INT_MASK,
> + MAX77705_CHGIN_IM, 0);
> + if (ret)
> + return ret;
> +
> + pscfg.of_node = dev->of_node;
replace with:
psycfg.fwnode = dev_fwnode(dev);
> + pscfg.drv_data = chg;
> +
> + chg->psy_chg = devm_power_supply_register(dev, &max77705_charger_psy_desc, &pscfg);
> + if (IS_ERR(chg->psy_chg))
> + return PTR_ERR(chg->psy_chg);
> +
> + max77705_charger_irq_chip.irq_drv_data = chg;
> + ret = devm_regmap_add_irq_chip(chg->dev, chg->regmap, i2c->irq,
> + IRQF_ONESHOT | IRQF_SHARED, 0,
> + &max77705_charger_irq_chip,
> + &irq_data);
> + if (ret)
> + return dev_err_probe(dev, ret, "failed to add irq chip\n");
> +
> + chg->wqueue = create_singlethread_workqueue(dev_name(dev));
> + if (IS_ERR(chg->wqueue))
> + return dev_err_probe(dev, PTR_ERR(chg->wqueue), "failed to create workqueue\n");
> +
> + INIT_WORK(&chg->chgin_work, max77705_chgin_isr_work);
devm_work_autocancel()
> + max77705_charger_initialize(chg);
> +
> + ret = max77705_charger_enable(chg);
> + if (ret)
> + return dev_err_probe(dev, ret, "failed to enable charge\n");
> + else
You can drop this else and reduce indent of the following line.
> + return devm_add_action_or_reset(dev, max77705_charger_disable, chg);
> +}
> +
> +static const struct of_device_id max77705_charger_of_match[] = {
> + { .compatible = "maxim,max77705-charger" },
> + { }
> +};
> +MODULE_DEVICE_TABLE(of, max77705_charger_of_match);
> +
> +static struct i2c_driver max77705_charger_driver = {
> + .driver = {
> + .name = "max77705-charger",
> + .of_match_table = max77705_charger_of_match,
> + },
> + .probe = max77705_charger_probe,
> +};
> +module_i2c_driver(max77705_charger_driver);
> +
> +MODULE_AUTHOR("Dzmitry Sankouski <dsankouski@gmail.com>");
> +MODULE_DESCRIPTION("Maxim MAX77705 charger driver");
> +MODULE_LICENSE("GPL");
> diff --git a/include/linux/power/max77705_charger.h b/include/linux/power/max77705_charger.h
> new file mode 100644
> index 000000000000..cb5cd03e54e2
> --- /dev/null
> +++ b/include/linux/power/max77705_charger.h
> @@ -0,0 +1,194 @@
> +/* SPDX-License-Identifier: GPL-2.0 */
> +/*
> + * Maxim MAX77705 definitions.
> + *
> + * Copyright (C) 2015 Samsung Electronics, Inc.
> + * Copyright (C) 2024 Dzmitry Sankouski <dsankouski@gmail.com>
> + */
Please put all of it at the head of the driver, like any of the
recent power-supply drivers do.
Greetings,
-- Sebastian
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 23+ messages in thread* Re: [PATCH v14 06/10] power: supply: max77705: Add charger driver for Maxim 77705
2025-01-16 0:16 ` Sebastian Reichel
@ 2025-01-16 12:44 ` Dzmitry Sankouski
0 siblings, 0 replies; 23+ messages in thread
From: Dzmitry Sankouski @ 2025-01-16 12:44 UTC (permalink / raw)
To: Sebastian Reichel
Cc: Chanwoo Choi, Krzysztof Kozlowski, Lee Jones, Rob Herring,
Conor Dooley, Dmitry Torokhov, Pavel Machek, Hans de Goede,
Marek Szyprowski, Sebastian Krzyszkowiak, Purism Kernel Team,
Krzysztof Kozlowski, linux-pm, linux-kernel, devicetree,
linux-input, linux-leds
чт, 16 янв. 2025 г. в 03:17, Sebastian Reichel
<sebastian.reichel@collabora.com>:
>
> Hi,
>
> On Wed, Jan 08, 2025 at 05:13:50PM +0300, Dzmitry Sankouski wrote:
> > Add driver for Maxim 77705 switch-mode charger.
> > It providing power supply class information to userspace.
> >
> > The driver is configured through DTS (battery and system related
> > settings).
> >
> > Signed-off-by: Dzmitry Sankouski <dsankouski@gmail.com>
> > ---
(...)
> > +++ b/include/linux/power/max77705_charger.h
> > @@ -0,0 +1,194 @@
> > +/* SPDX-License-Identifier: GPL-2.0 */
> > +/*
> > + * Maxim MAX77705 definitions.
> > + *
> > + * Copyright (C) 2015 Samsung Electronics, Inc.
> > + * Copyright (C) 2024 Dzmitry Sankouski <dsankouski@gmail.com>
> > + */
>
> Please put all of it at the head of the driver, like any of the
> recent power-supply drivers do.
>
Samsung is unrelated to the driver code, except for that header.
I would leave their copyright sign only in the header, to make that clear.
--
Best regards and thanks for review,
Dzmitry
^ permalink raw reply [flat|nested] 23+ messages in thread
* [PATCH v14 07/10] mfd: simple-mfd-i2c: Add MAX77705 support
2025-01-08 14:13 [PATCH v14 00/10] Add support for Maxim Integrated MAX77705 PMIC Dzmitry Sankouski
` (5 preceding siblings ...)
2025-01-08 14:13 ` [PATCH v14 06/10] power: supply: max77705: Add charger driver for Maxim 77705 Dzmitry Sankouski
@ 2025-01-08 14:13 ` Dzmitry Sankouski
2025-01-09 12:01 ` Lee Jones
2025-01-08 14:13 ` [PATCH v14 08/10] mfd: Add new driver for MAX77705 PMIC Dzmitry Sankouski
` (3 subsequent siblings)
10 siblings, 1 reply; 23+ messages in thread
From: Dzmitry Sankouski @ 2025-01-08 14:13 UTC (permalink / raw)
To: Sebastian Reichel, Chanwoo Choi, Krzysztof Kozlowski, Lee Jones,
Rob Herring, Conor Dooley, Dmitry Torokhov, Pavel Machek,
Hans de Goede, Marek Szyprowski, Sebastian Krzyszkowiak,
Purism Kernel Team, Krzysztof Kozlowski
Cc: linux-pm, linux-kernel, devicetree, linux-input, linux-leds,
Dzmitry Sankouski
Add MAX77705 support - fuel gauge and hwmon devices.
Hwmon provides charger input and system bus measurements.
Signed-off-by: Dzmitry Sankouski <dsankouski@gmail.com>
---
Changes in v13:
- remove compatible from cells
- change mfd compatible to match max77705 fuel gauge node
---
drivers/mfd/simple-mfd-i2c.c | 11 +++++++++++
1 file changed, 11 insertions(+)
diff --git a/drivers/mfd/simple-mfd-i2c.c b/drivers/mfd/simple-mfd-i2c.c
index 6eda79533208..22159913bea0 100644
--- a/drivers/mfd/simple-mfd-i2c.c
+++ b/drivers/mfd/simple-mfd-i2c.c
@@ -83,11 +83,22 @@ static const struct simple_mfd_data maxim_max5970 = {
.mfd_cell_size = ARRAY_SIZE(max5970_cells),
};
+static const struct mfd_cell max77705_sensor_cells[] = {
+ { .name = "max77705-battery" },
+ { .name = "max77705-hwmon", },
+};
+
+static const struct simple_mfd_data maxim_mon_max77705 = {
+ .mfd_cell = max77705_sensor_cells,
+ .mfd_cell_size = ARRAY_SIZE(max77705_sensor_cells),
+};
+
static const struct of_device_id simple_mfd_i2c_of_match[] = {
{ .compatible = "kontron,sl28cpld" },
{ .compatible = "silergy,sy7636a", .data = &silergy_sy7636a},
{ .compatible = "maxim,max5970", .data = &maxim_max5970},
{ .compatible = "maxim,max5978", .data = &maxim_max5970},
+ { .compatible = "maxim,max77705-battery", .data = &maxim_mon_max77705},
{}
};
MODULE_DEVICE_TABLE(of, simple_mfd_i2c_of_match);
--
2.39.5
^ permalink raw reply related [flat|nested] 23+ messages in thread* Re: [PATCH v14 07/10] mfd: simple-mfd-i2c: Add MAX77705 support
2025-01-08 14:13 ` [PATCH v14 07/10] mfd: simple-mfd-i2c: Add MAX77705 support Dzmitry Sankouski
@ 2025-01-09 12:01 ` Lee Jones
2025-01-09 12:03 ` Lee Jones
2025-01-15 18:57 ` Dzmitry Sankouski
0 siblings, 2 replies; 23+ messages in thread
From: Lee Jones @ 2025-01-09 12:01 UTC (permalink / raw)
To: Dzmitry Sankouski
Cc: Sebastian Reichel, Chanwoo Choi, Krzysztof Kozlowski, Rob Herring,
Conor Dooley, Dmitry Torokhov, Pavel Machek, Hans de Goede,
Marek Szyprowski, Sebastian Krzyszkowiak, Purism Kernel Team,
Krzysztof Kozlowski, linux-pm, linux-kernel, devicetree,
linux-input, linux-leds
On Wed, 08 Jan 2025, Dzmitry Sankouski wrote:
> Add MAX77705 support - fuel gauge and hwmon devices.
> Hwmon provides charger input and system bus measurements.
>
> Signed-off-by: Dzmitry Sankouski <dsankouski@gmail.com>
> ---
> Changes in v13:
> - remove compatible from cells
> - change mfd compatible to match max77705 fuel gauge node
> ---
> drivers/mfd/simple-mfd-i2c.c | 11 +++++++++++
> 1 file changed, 11 insertions(+)
>
> diff --git a/drivers/mfd/simple-mfd-i2c.c b/drivers/mfd/simple-mfd-i2c.c
> index 6eda79533208..22159913bea0 100644
> --- a/drivers/mfd/simple-mfd-i2c.c
> +++ b/drivers/mfd/simple-mfd-i2c.c
> @@ -83,11 +83,22 @@ static const struct simple_mfd_data maxim_max5970 = {
> .mfd_cell_size = ARRAY_SIZE(max5970_cells),
> };
>
> +static const struct mfd_cell max77705_sensor_cells[] = {
> + { .name = "max77705-battery" },
> + { .name = "max77705-hwmon", },
> +};
> +
> +static const struct simple_mfd_data maxim_mon_max77705 = {
> + .mfd_cell = max77705_sensor_cells,
> + .mfd_cell_size = ARRAY_SIZE(max77705_sensor_cells),
> +};
> +
> static const struct of_device_id simple_mfd_i2c_of_match[] = {
> { .compatible = "kontron,sl28cpld" },
> { .compatible = "silergy,sy7636a", .data = &silergy_sy7636a},
> { .compatible = "maxim,max5970", .data = &maxim_max5970},
> { .compatible = "maxim,max5978", .data = &maxim_max5970},
> + { .compatible = "maxim,max77705-battery", .data = &maxim_mon_max77705},
Drop the battery part from the MFD (group) name please.
> {}
> };
> MODULE_DEVICE_TABLE(of, simple_mfd_i2c_of_match);
>
> --
> 2.39.5
>
--
Lee Jones [李琼斯]
^ permalink raw reply [flat|nested] 23+ messages in thread* Re: [PATCH v14 07/10] mfd: simple-mfd-i2c: Add MAX77705 support
2025-01-09 12:01 ` Lee Jones
@ 2025-01-09 12:03 ` Lee Jones
2025-01-09 12:53 ` Dzmitry Sankouski
2025-01-15 18:57 ` Dzmitry Sankouski
1 sibling, 1 reply; 23+ messages in thread
From: Lee Jones @ 2025-01-09 12:03 UTC (permalink / raw)
To: Dzmitry Sankouski
Cc: Sebastian Reichel, Chanwoo Choi, Krzysztof Kozlowski, Rob Herring,
Conor Dooley, Dmitry Torokhov, Pavel Machek, Hans de Goede,
Marek Szyprowski, Sebastian Krzyszkowiak, Purism Kernel Team,
Krzysztof Kozlowski, linux-pm, linux-kernel, devicetree,
linux-input, linux-leds
On Thu, 09 Jan 2025, Lee Jones wrote:
> On Wed, 08 Jan 2025, Dzmitry Sankouski wrote:
>
> > Add MAX77705 support - fuel gauge and hwmon devices.
> > Hwmon provides charger input and system bus measurements.
> >
> > Signed-off-by: Dzmitry Sankouski <dsankouski@gmail.com>
> > ---
> > Changes in v13:
> > - remove compatible from cells
> > - change mfd compatible to match max77705 fuel gauge node
> > ---
> > drivers/mfd/simple-mfd-i2c.c | 11 +++++++++++
> > 1 file changed, 11 insertions(+)
> >
> > diff --git a/drivers/mfd/simple-mfd-i2c.c b/drivers/mfd/simple-mfd-i2c.c
> > index 6eda79533208..22159913bea0 100644
> > --- a/drivers/mfd/simple-mfd-i2c.c
> > +++ b/drivers/mfd/simple-mfd-i2c.c
> > @@ -83,11 +83,22 @@ static const struct simple_mfd_data maxim_max5970 = {
> > .mfd_cell_size = ARRAY_SIZE(max5970_cells),
> > };
> >
> > +static const struct mfd_cell max77705_sensor_cells[] = {
> > + { .name = "max77705-battery" },
> > + { .name = "max77705-hwmon", },
> > +};
Why not register these from the proper MFD driver?
> > +static const struct simple_mfd_data maxim_mon_max77705 = {
> > + .mfd_cell = max77705_sensor_cells,
> > + .mfd_cell_size = ARRAY_SIZE(max77705_sensor_cells),
> > +};
> > +
> > static const struct of_device_id simple_mfd_i2c_of_match[] = {
> > { .compatible = "kontron,sl28cpld" },
> > { .compatible = "silergy,sy7636a", .data = &silergy_sy7636a},
> > { .compatible = "maxim,max5970", .data = &maxim_max5970},
> > { .compatible = "maxim,max5978", .data = &maxim_max5970},
> > + { .compatible = "maxim,max77705-battery", .data = &maxim_mon_max77705},
>
> Drop the battery part from the MFD (group) name please.
>
> > {}
> > };
> > MODULE_DEVICE_TABLE(of, simple_mfd_i2c_of_match);
--
Lee Jones [李琼斯]
^ permalink raw reply [flat|nested] 23+ messages in thread* Re: [PATCH v14 07/10] mfd: simple-mfd-i2c: Add MAX77705 support
2025-01-09 12:03 ` Lee Jones
@ 2025-01-09 12:53 ` Dzmitry Sankouski
2025-01-13 13:35 ` Dzmitry Sankouski
0 siblings, 1 reply; 23+ messages in thread
From: Dzmitry Sankouski @ 2025-01-09 12:53 UTC (permalink / raw)
To: Lee Jones
Cc: Sebastian Reichel, Chanwoo Choi, Krzysztof Kozlowski, Rob Herring,
Conor Dooley, Dmitry Torokhov, Pavel Machek, Hans de Goede,
Marek Szyprowski, Sebastian Krzyszkowiak, Purism Kernel Team,
Krzysztof Kozlowski, linux-pm, linux-kernel, devicetree,
linux-input, linux-leds
чт, 9 янв. 2025 г. в 15:03, Lee Jones <lee@kernel.org>:
>
> On Thu, 09 Jan 2025, Lee Jones wrote:
>
> > On Wed, 08 Jan 2025, Dzmitry Sankouski wrote:
> >
> > > Add MAX77705 support - fuel gauge and hwmon devices.
> > > Hwmon provides charger input and system bus measurements.
> > >
> > > Signed-off-by: Dzmitry Sankouski <dsankouski@gmail.com>
> > > ---
> > > Changes in v13:
> > > - remove compatible from cells
> > > - change mfd compatible to match max77705 fuel gauge node
> > > ---
> > > drivers/mfd/simple-mfd-i2c.c | 11 +++++++++++
> > > 1 file changed, 11 insertions(+)
> > >
> > > diff --git a/drivers/mfd/simple-mfd-i2c.c b/drivers/mfd/simple-mfd-i2c.c
> > > index 6eda79533208..22159913bea0 100644
> > > --- a/drivers/mfd/simple-mfd-i2c.c
> > > +++ b/drivers/mfd/simple-mfd-i2c.c
> > > @@ -83,11 +83,22 @@ static const struct simple_mfd_data maxim_max5970 = {
> > > .mfd_cell_size = ARRAY_SIZE(max5970_cells),
> > > };
> > >
> > > +static const struct mfd_cell max77705_sensor_cells[] = {
> > > + { .name = "max77705-battery" },
> > > + { .name = "max77705-hwmon", },
> > > +};
>
> Why not register these from the proper MFD driver?
>
Because the fuel gauge address is different from the max77705 mfd device.
--
Best regards and thanks for review,
Dzmitry
^ permalink raw reply [flat|nested] 23+ messages in thread* Re: [PATCH v14 07/10] mfd: simple-mfd-i2c: Add MAX77705 support
2025-01-09 12:53 ` Dzmitry Sankouski
@ 2025-01-13 13:35 ` Dzmitry Sankouski
0 siblings, 0 replies; 23+ messages in thread
From: Dzmitry Sankouski @ 2025-01-13 13:35 UTC (permalink / raw)
To: Lee Jones
Cc: Sebastian Reichel, Chanwoo Choi, Krzysztof Kozlowski, Rob Herring,
Conor Dooley, Dmitry Torokhov, Pavel Machek, Hans de Goede,
Marek Szyprowski, Sebastian Krzyszkowiak, Purism Kernel Team,
Krzysztof Kozlowski, linux-pm, linux-kernel, devicetree,
linux-input, linux-leds
чт, 9 янв. 2025 г. в 15:53, Dzmitry Sankouski <dsankouski@gmail.com>:
>
> чт, 9 янв. 2025 г. в 15:03, Lee Jones <lee@kernel.org>:
> >
> > On Thu, 09 Jan 2025, Lee Jones wrote:
> >
> > > On Wed, 08 Jan 2025, Dzmitry Sankouski wrote:
> > >
> > > > Add MAX77705 support - fuel gauge and hwmon devices.
> > > > Hwmon provides charger input and system bus measurements.
> > > >
> > > > Signed-off-by: Dzmitry Sankouski <dsankouski@gmail.com>
> > > > ---
> > > > Changes in v13:
> > > > - remove compatible from cells
> > > > - change mfd compatible to match max77705 fuel gauge node
> > > > ---
> > > > drivers/mfd/simple-mfd-i2c.c | 11 +++++++++++
> > > > 1 file changed, 11 insertions(+)
> > > >
> > > > diff --git a/drivers/mfd/simple-mfd-i2c.c b/drivers/mfd/simple-mfd-i2c.c
> > > > index 6eda79533208..22159913bea0 100644
> > > > --- a/drivers/mfd/simple-mfd-i2c.c
> > > > +++ b/drivers/mfd/simple-mfd-i2c.c
> > > > @@ -83,11 +83,22 @@ static const struct simple_mfd_data maxim_max5970 = {
> > > > .mfd_cell_size = ARRAY_SIZE(max5970_cells),
> > > > };
> > > >
> > > > +static const struct mfd_cell max77705_sensor_cells[] = {
> > > > + { .name = "max77705-battery" },
> > > > + { .name = "max77705-hwmon", },
> > > > +};
> >
> > Why not register these from the proper MFD driver?
> >
>
> Because the fuel gauge address is different from the max77705 mfd device.
In more details:
we had a discussion with Krzysztof about fuel gauge device
[1], [2], [3] and agreed that it should be modeled as a separate device,
because it has no common resources with max77705 device, and has separate
address. This means its node are out of MAX77705 mfd node, forming its own
MFD with shared i2c bus.
https://lore.kernel.org/lkml/55f32164-f504-4409-8ce2-6462b833da89@kernel.org/
https://patchwork.kernel.org/project/linux-input/patch/20241202-starqltechn_integration_upstream-v9-3-a1adc3bae2b8@gmail.com/
https://patches.linaro.org/project/linux-leds/patch/20241217-starqltechn_integration_upstream-v12-2-ed840944f948@gmail.com/#951752
--
Best regards and thanks for review,
Dzmitry
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH v14 07/10] mfd: simple-mfd-i2c: Add MAX77705 support
2025-01-09 12:01 ` Lee Jones
2025-01-09 12:03 ` Lee Jones
@ 2025-01-15 18:57 ` Dzmitry Sankouski
2025-01-21 10:15 ` Lee Jones
1 sibling, 1 reply; 23+ messages in thread
From: Dzmitry Sankouski @ 2025-01-15 18:57 UTC (permalink / raw)
To: Lee Jones
Cc: Sebastian Reichel, Chanwoo Choi, Krzysztof Kozlowski, Rob Herring,
Conor Dooley, Dmitry Torokhov, Pavel Machek, Hans de Goede,
Marek Szyprowski, Sebastian Krzyszkowiak, Purism Kernel Team,
Krzysztof Kozlowski, linux-pm, linux-kernel, devicetree,
linux-input, linux-leds
чт, 9 янв. 2025 г. в 15:02, Lee Jones <lee@kernel.org>:
>
> On Wed, 08 Jan 2025, Dzmitry Sankouski wrote:
>
> > Add MAX77705 support - fuel gauge and hwmon devices.
> > Hwmon provides charger input and system bus measurements.
> >
> > Signed-off-by: Dzmitry Sankouski <dsankouski@gmail.com>
(...)
> > static const struct of_device_id simple_mfd_i2c_of_match[] = {
> > { .compatible = "kontron,sl28cpld" },
> > { .compatible = "silergy,sy7636a", .data = &silergy_sy7636a},
> > { .compatible = "maxim,max5970", .data = &maxim_max5970},
> > { .compatible = "maxim,max5978", .data = &maxim_max5970},
> > + { .compatible = "maxim,max77705-battery", .data = &maxim_mon_max77705},
>
> Drop the battery part from the MFD (group) name please.
>
It will then conflict with MAX77705 mfd driver compatible.
--
Best regards and thanks for review,
Dzmitry
^ permalink raw reply [flat|nested] 23+ messages in thread* Re: [PATCH v14 07/10] mfd: simple-mfd-i2c: Add MAX77705 support
2025-01-15 18:57 ` Dzmitry Sankouski
@ 2025-01-21 10:15 ` Lee Jones
2025-01-22 15:19 ` Dzmitry Sankouski
0 siblings, 1 reply; 23+ messages in thread
From: Lee Jones @ 2025-01-21 10:15 UTC (permalink / raw)
To: Dzmitry Sankouski
Cc: Sebastian Reichel, Chanwoo Choi, Krzysztof Kozlowski, Rob Herring,
Conor Dooley, Dmitry Torokhov, Pavel Machek, Hans de Goede,
Marek Szyprowski, Sebastian Krzyszkowiak, Purism Kernel Team,
Krzysztof Kozlowski, linux-pm, linux-kernel, devicetree,
linux-input, linux-leds
On Wed, 15 Jan 2025, Dzmitry Sankouski wrote:
> чт, 9 янв. 2025 г. в 15:02, Lee Jones <lee@kernel.org>:
> >
> > On Wed, 08 Jan 2025, Dzmitry Sankouski wrote:
> >
> > > Add MAX77705 support - fuel gauge and hwmon devices.
> > > Hwmon provides charger input and system bus measurements.
> > >
> > > Signed-off-by: Dzmitry Sankouski <dsankouski@gmail.com>
> (...)
> > > static const struct of_device_id simple_mfd_i2c_of_match[] = {
> > > { .compatible = "kontron,sl28cpld" },
> > > { .compatible = "silergy,sy7636a", .data = &silergy_sy7636a},
> > > { .compatible = "maxim,max5970", .data = &maxim_max5970},
> > > { .compatible = "maxim,max5978", .data = &maxim_max5970},
> > > + { .compatible = "maxim,max77705-battery", .data = &maxim_mon_max77705},
> >
> > Drop the battery part from the MFD (group) name please.
> >
>
> It will then conflict with MAX77705 mfd driver compatible.
Where is that used?
--
Lee Jones [李琼斯]
^ permalink raw reply [flat|nested] 23+ messages in thread* Re: [PATCH v14 07/10] mfd: simple-mfd-i2c: Add MAX77705 support
2025-01-21 10:15 ` Lee Jones
@ 2025-01-22 15:19 ` Dzmitry Sankouski
0 siblings, 0 replies; 23+ messages in thread
From: Dzmitry Sankouski @ 2025-01-22 15:19 UTC (permalink / raw)
To: Lee Jones
Cc: Sebastian Reichel, Chanwoo Choi, Krzysztof Kozlowski, Rob Herring,
Conor Dooley, Dmitry Torokhov, Pavel Machek, Hans de Goede,
Marek Szyprowski, Sebastian Krzyszkowiak, Purism Kernel Team,
Krzysztof Kozlowski, linux-pm, linux-kernel, devicetree,
linux-input, linux-leds
вт, 21 янв. 2025 г. в 13:15, Lee Jones <lee@kernel.org>:
>
> On Wed, 15 Jan 2025, Dzmitry Sankouski wrote:
>
> > чт, 9 янв. 2025 г. в 15:02, Lee Jones <lee@kernel.org>:
> > >
> > > On Wed, 08 Jan 2025, Dzmitry Sankouski wrote:
> > >
> > > > Add MAX77705 support - fuel gauge and hwmon devices.
> > > > Hwmon provides charger input and system bus measurements.
> > > >
> > > > Signed-off-by: Dzmitry Sankouski <dsankouski@gmail.com>
> > (...)
> > > > static const struct of_device_id simple_mfd_i2c_of_match[] = {
> > > > { .compatible = "kontron,sl28cpld" },
> > > > { .compatible = "silergy,sy7636a", .data = &silergy_sy7636a},
> > > > { .compatible = "maxim,max5970", .data = &maxim_max5970},
> > > > { .compatible = "maxim,max5978", .data = &maxim_max5970},
> > > > + { .compatible = "maxim,max77705-battery", .data = &maxim_mon_max77705},
> > >
> > > Drop the battery part from the MFD (group) name please.
> > >
> >
> > It will then conflict with MAX77705 mfd driver compatible.
>
> Where is that used?
In MAX77705 MFD patch:
https://patchwork.kernel.org/project/linux-pm/patch/20250117-starqltechn_integration_upstream-v16-5-11afa877276c@gmail.com/
--
Best regards and thanks for review,
Dzmitry
^ permalink raw reply [flat|nested] 23+ messages in thread
* [PATCH v14 08/10] mfd: Add new driver for MAX77705 PMIC
2025-01-08 14:13 [PATCH v14 00/10] Add support for Maxim Integrated MAX77705 PMIC Dzmitry Sankouski
` (6 preceding siblings ...)
2025-01-08 14:13 ` [PATCH v14 07/10] mfd: simple-mfd-i2c: Add MAX77705 support Dzmitry Sankouski
@ 2025-01-08 14:13 ` Dzmitry Sankouski
2025-01-09 12:21 ` Lee Jones
2025-01-08 14:13 ` [PATCH v14 09/10] input: max77693: add max77705 haptic support Dzmitry Sankouski
` (2 subsequent siblings)
10 siblings, 1 reply; 23+ messages in thread
From: Dzmitry Sankouski @ 2025-01-08 14:13 UTC (permalink / raw)
To: Sebastian Reichel, Chanwoo Choi, Krzysztof Kozlowski, Lee Jones,
Rob Herring, Conor Dooley, Dmitry Torokhov, Pavel Machek,
Hans de Goede, Marek Szyprowski, Sebastian Krzyszkowiak,
Purism Kernel Team, Krzysztof Kozlowski
Cc: linux-pm, linux-kernel, devicetree, linux-input, linux-leds,
Dzmitry Sankouski
Add the core MFD driver for max77705 PMIC. Drivers for sub-devices
will be added in subsequent patches.
Signed-off-by: Dzmitry Sankouski <dsankouski@gmail.com>
---
Changes for v12:
- remove fuelgauge support (it's moved to simple-mfd-i2c, to reflect
the fact it's a separate device with it's own i2c client)
- remove unneeded ending comma
- remove struct dev_pm_ops because defined by DEFINE_SIMPLE_DEV_PM_OPS
- Kconfig: select MFD_SIMPLE_MFD_I2C
Changes for v10:
- never blank line between call and its error check
- remove unnecessary line wrap
- revert wrong changes in max77693-common.h
- move max77705_pm_ops from header to c file
- fail probe, when fuelgauge is not found in sub device list
- remove fuelgauge compatible, because with compatible,
platform matches using compatible, and platform_device
id_entry is empty. With no compatible, platform matches
by device id, and id_entry is populated.
- use dev_err_probe for error handling
Changes for v9:
- use max17042 as fuel gauge chip
- initialize max17042 i2c dummy device in mfd device,
because bus can be used for reading additional values,
not related to fuelgauge, like chip input current, system
bus current
- fix pmic_rev kernel test robot error
Changes for v8:
- fix comment style C++ -> C
- remove unused pmic_ver
Changes for v6:
- add PMIC suffix in Kconfig
- remove filename from file header
- reorder headers alphabetically
- move out fg and chg adresses definitions
- rename led name and compatible
- remove overbracketing
- move charger and fuel gauge i2c initialization
to their drivers
- fix max77705_i2c_driver tabbing
- formatting fixes
Changes for v5:
- license change to 2.0
- use same hardware name in Kconfig and module descriptions
Changes for v4:
- rework driver from scratch
- migrate to regmap_add_irq_chip, remove max77705-irq.c,
rename max77705-core.c to max77705.c
- cleanup headers
- remove debugfs code
- migrate to use max77693_dev structure
- remove max77705.h
---
MAINTAINERS | 2 ++
drivers/mfd/Kconfig | 13 +++++++++
drivers/mfd/Makefile | 2 ++
drivers/mfd/max77705.c | 209 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
include/linux/mfd/max77693-common.h | 4 ++-
include/linux/mfd/max77705-private.h | 178 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
6 files changed, 407 insertions(+), 1 deletion(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 98a77dece2f1..eb5222a833f8 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -14294,6 +14294,7 @@ F: drivers/*/*max77843.c
F: drivers/*/max14577*.c
F: drivers/*/max77686*.c
F: drivers/*/max77693*.c
+F: drivers/*/max77705*.c
F: drivers/clk/clk-max77686.c
F: drivers/extcon/extcon-max14577.c
F: drivers/extcon/extcon-max77693.c
@@ -14301,6 +14302,7 @@ F: drivers/rtc/rtc-max77686.c
F: include/linux/mfd/max14577*.h
F: include/linux/mfd/max77686*.h
F: include/linux/mfd/max77693*.h
+F: include/linux/mfd/max77705*.h
MAXIRADIO FM RADIO RECEIVER DRIVER
M: Hans Verkuil <hverkuil@xs4all.nl>
diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig
index 6b0682af6e32..5064b1b42f76 100644
--- a/drivers/mfd/Kconfig
+++ b/drivers/mfd/Kconfig
@@ -916,6 +916,19 @@ config MFD_MAX77693
additional drivers must be enabled in order to use the functionality
of the device.
+config MFD_MAX77705
+ tristate "Maxim MAX77705 PMIC Support"
+ depends on I2C
+ select MFD_CORE
+ select MFD_SIMPLE_MFD_I2C
+ help
+ Say yes here to add support for Maxim Integrated MAX77705 PMIC.
+ This is a Power Management IC with Charger, safe LDOs, Flash, Haptic
+ and MUIC controls on chip.
+ This driver provides common support for accessing the device;
+ additional drivers must be enabled in order to use the functionality
+ of the device.
+
config MFD_MAX77714
tristate "Maxim Semiconductor MAX77714 PMIC Support"
depends on I2C
diff --git a/drivers/mfd/Makefile b/drivers/mfd/Makefile
index 9220eaf7cf12..c18bb5a57734 100644
--- a/drivers/mfd/Makefile
+++ b/drivers/mfd/Makefile
@@ -168,6 +168,7 @@ obj-$(CONFIG_MFD_MAX77620) += max77620.o
obj-$(CONFIG_MFD_MAX77650) += max77650.o
obj-$(CONFIG_MFD_MAX77686) += max77686.o
obj-$(CONFIG_MFD_MAX77693) += max77693.o
+obj-$(CONFIG_MFD_MAX77705) += max77705.o
obj-$(CONFIG_MFD_MAX77714) += max77714.o
obj-$(CONFIG_MFD_MAX77843) += max77843.o
obj-$(CONFIG_MFD_MAX8907) += max8907.o
@@ -233,6 +234,7 @@ obj-$(CONFIG_MFD_RK8XX_I2C) += rk8xx-i2c.o
obj-$(CONFIG_MFD_RK8XX_SPI) += rk8xx-spi.o
obj-$(CONFIG_MFD_RN5T618) += rn5t618.o
obj-$(CONFIG_MFD_SEC_CORE) += sec-core.o sec-irq.o
+obj-$(CONFIG_MFD_S2DOS05) += s2dos05.o
obj-$(CONFIG_MFD_SYSCON) += syscon.o
obj-$(CONFIG_MFD_LM3533) += lm3533-core.o lm3533-ctrlbank.o
obj-$(CONFIG_MFD_VEXPRESS_SYSREG) += vexpress-sysreg.o
diff --git a/drivers/mfd/max77705.c b/drivers/mfd/max77705.c
new file mode 100644
index 000000000000..18d344620755
--- /dev/null
+++ b/drivers/mfd/max77705.c
@@ -0,0 +1,209 @@
+// SPDX-License-Identifier: GPL-2.0+
+//
+// Maxim MAX77705 PMIC core driver
+//
+// Copyright (C) 2024 Dzmitry Sankouski <dsankouski@gmail.com>
+
+#include <linux/i2c.h>
+#include <linux/interrupt.h>
+#include <linux/mfd/core.h>
+#include <linux/mfd/max77705-private.h>
+#include <linux/mfd/max77693-common.h>
+#include <linux/pm.h>
+#include <linux/power/max17042_battery.h>
+#include <linux/module.h>
+#include <linux/regmap.h>
+#include <linux/of.h>
+
+static struct mfd_cell max77705_devs[] = {
+ {
+ .name = "max77705-rgb",
+ .of_compatible = "maxim,max77705-rgb",
+ },
+ {
+ .name = "max77705-charger",
+ .of_compatible = "maxim,max77705-charger",
+ },
+ {
+ .name = "max77705-haptic",
+ .of_compatible = "maxim,max77705-haptic",
+ },
+};
+
+static const struct regmap_range max77705_readable_ranges[] = {
+ regmap_reg_range(MAX77705_PMIC_REG_PMICID1, MAX77705_PMIC_REG_BSTOUT_MASK),
+ regmap_reg_range(MAX77705_PMIC_REG_INTSRC, MAX77705_PMIC_REG_RESERVED_29),
+ regmap_reg_range(MAX77705_PMIC_REG_BOOSTCONTROL1, MAX77705_PMIC_REG_BOOSTCONTROL1),
+ regmap_reg_range(MAX77705_PMIC_REG_MCONFIG, MAX77705_PMIC_REG_MCONFIG2),
+ regmap_reg_range(MAX77705_PMIC_REG_FORCE_EN_MASK, MAX77705_PMIC_REG_FORCE_EN_MASK),
+ regmap_reg_range(MAX77705_PMIC_REG_BOOSTCONTROL1, MAX77705_PMIC_REG_BOOSTCONTROL1),
+ regmap_reg_range(MAX77705_PMIC_REG_BOOSTCONTROL2, MAX77705_PMIC_REG_BOOSTCONTROL2),
+ regmap_reg_range(MAX77705_PMIC_REG_SW_RESET, MAX77705_PMIC_REG_USBC_RESET),
+};
+
+static const struct regmap_range max77705_writable_ranges[] = {
+ regmap_reg_range(MAX77705_PMIC_REG_MAINCTRL1, MAX77705_PMIC_REG_BSTOUT_MASK),
+ regmap_reg_range(MAX77705_PMIC_REG_INTSRC, MAX77705_PMIC_REG_RESERVED_29),
+ regmap_reg_range(MAX77705_PMIC_REG_BOOSTCONTROL1, MAX77705_PMIC_REG_BOOSTCONTROL1),
+ regmap_reg_range(MAX77705_PMIC_REG_MCONFIG, MAX77705_PMIC_REG_MCONFIG2),
+ regmap_reg_range(MAX77705_PMIC_REG_FORCE_EN_MASK, MAX77705_PMIC_REG_FORCE_EN_MASK),
+ regmap_reg_range(MAX77705_PMIC_REG_BOOSTCONTROL1, MAX77705_PMIC_REG_BOOSTCONTROL1),
+ regmap_reg_range(MAX77705_PMIC_REG_BOOSTCONTROL2, MAX77705_PMIC_REG_BOOSTCONTROL2),
+ regmap_reg_range(MAX77705_PMIC_REG_SW_RESET, MAX77705_PMIC_REG_USBC_RESET),
+
+};
+
+static const struct regmap_access_table max77705_readable_table = {
+ .yes_ranges = max77705_readable_ranges,
+ .n_yes_ranges = ARRAY_SIZE(max77705_readable_ranges),
+};
+
+static const struct regmap_access_table max77705_writable_table = {
+ .yes_ranges = max77705_writable_ranges,
+ .n_yes_ranges = ARRAY_SIZE(max77705_writable_ranges),
+};
+
+static const struct regmap_config max77705_regmap_config = {
+ .reg_bits = 8,
+ .val_bits = 8,
+ .rd_table = &max77705_readable_table,
+ .wr_table = &max77705_writable_table,
+ .max_register = MAX77705_PMIC_REG_USBC_RESET,
+};
+
+static const struct regmap_config max77705_leds_regmap_config = {
+ .reg_base = MAX77705_RGBLED_REG_BASE,
+ .reg_bits = 8,
+ .val_bits = 8,
+ .max_register = MAX77705_LED_REG_END,
+};
+
+static const struct regmap_irq max77705_topsys_irqs[] = {
+ { .mask = MAX77705_SYSTEM_IRQ_BSTEN_INT, },
+ { .mask = MAX77705_SYSTEM_IRQ_SYSUVLO_INT, },
+ { .mask = MAX77705_SYSTEM_IRQ_SYSOVLO_INT, },
+ { .mask = MAX77705_SYSTEM_IRQ_TSHDN_INT, },
+ { .mask = MAX77705_SYSTEM_IRQ_TM_INT, },
+};
+
+static const struct regmap_irq_chip max77705_topsys_irq_chip = {
+ .name = "max77705-topsys",
+ .status_base = MAX77705_PMIC_REG_SYSTEM_INT,
+ .mask_base = MAX77705_PMIC_REG_SYSTEM_INT_MASK,
+ .num_regs = 1,
+ .irqs = max77705_topsys_irqs,
+ .num_irqs = ARRAY_SIZE(max77705_topsys_irqs),
+};
+
+static int max77705_i2c_probe(struct i2c_client *i2c)
+{
+ struct max77693_dev *max77705;
+ struct regmap_irq_chip_data *irq_data;
+ struct irq_domain *domain;
+ int ret;
+ unsigned int pmic_rev_value;
+ enum max77705_hw_rev pmic_rev;
+
+ max77705 = devm_kzalloc(&i2c->dev, sizeof(*max77705), GFP_KERNEL);
+ if (!max77705)
+ return -ENOMEM;
+
+ max77705->i2c = i2c;
+ max77705->dev = &i2c->dev;
+ max77705->irq = i2c->irq;
+ max77705->type = TYPE_MAX77705;
+ i2c_set_clientdata(i2c, max77705);
+
+ max77705->regmap = devm_regmap_init_i2c(i2c, &max77705_regmap_config);
+ if (IS_ERR(max77705->regmap))
+ return PTR_ERR(max77705->regmap);
+
+ if (regmap_read(max77705->regmap, MAX77705_PMIC_REG_PMICREV, &pmic_rev_value) < 0)
+ return -ENODEV;
+
+ pmic_rev = pmic_rev_value & MAX77705_REVISION_MASK;
+ if (pmic_rev != MAX77705_PASS3)
+ return dev_err_probe(max77705->dev, -ENODEV,
+ "Rev.0x%x is not tested\n", pmic_rev);
+
+ max77705->regmap_leds = devm_regmap_init_i2c(i2c, &max77705_leds_regmap_config);
+ if (IS_ERR(max77705->regmap_leds))
+ return dev_err_probe(max77705->dev, PTR_ERR(max77705->regmap_leds),
+ "Failed to register leds regmap\n");
+
+ ret = devm_regmap_add_irq_chip(max77705->dev, max77705->regmap,
+ max77705->irq,
+ IRQF_ONESHOT | IRQF_SHARED, 0,
+ &max77705_topsys_irq_chip,
+ &irq_data);
+ if (ret)
+ return dev_err_probe(max77705->dev, ret, "Failed to add irq chip\n");
+
+ /* Unmask interrupts from all blocks in interrupt source register */
+ ret = regmap_update_bits(max77705->regmap,
+ MAX77705_PMIC_REG_INTSRC_MASK,
+ MAX77705_SRC_IRQ_ALL, (unsigned int)~MAX77705_SRC_IRQ_ALL);
+ if (ret < 0)
+ return dev_err_probe(max77705->dev, ret,
+ "Could not unmask interrupts in INTSRC\n");
+
+ domain = regmap_irq_get_domain(irq_data);
+
+ ret = devm_mfd_add_devices(max77705->dev, PLATFORM_DEVID_NONE,
+ max77705_devs, ARRAY_SIZE(max77705_devs),
+ NULL, 0, domain);
+ if (ret)
+ return dev_err_probe(max77705->dev, ret, "Failed to register child devices\n");
+
+ device_init_wakeup(max77705->dev, true);
+
+ return 0;
+}
+
+static int max77705_suspend(struct device *dev)
+{
+ struct i2c_client *i2c = to_i2c_client(dev);
+ struct max77693_dev *max77705 = i2c_get_clientdata(i2c);
+
+ disable_irq(max77705->irq);
+
+ if (device_may_wakeup(dev))
+ enable_irq_wake(max77705->irq);
+
+ return 0;
+}
+
+static int max77705_resume(struct device *dev)
+{
+ struct i2c_client *i2c = to_i2c_client(dev);
+ struct max77693_dev *max77705 = i2c_get_clientdata(i2c);
+
+ if (device_may_wakeup(dev))
+ disable_irq_wake(max77705->irq);
+
+ enable_irq(max77705->irq);
+
+ return 0;
+}
+DEFINE_SIMPLE_DEV_PM_OPS(max77705_pm_ops, max77705_suspend, max77705_resume);
+
+static const struct of_device_id max77705_i2c_of_match[] = {
+ { .compatible = "maxim,max77705" },
+ { }
+};
+MODULE_DEVICE_TABLE(of, max77705_i2c_of_match);
+
+static struct i2c_driver max77705_i2c_driver = {
+ .driver = {
+ .name = "max77705",
+ .of_match_table = max77705_i2c_of_match,
+ .pm = pm_sleep_ptr(&max77705_pm_ops),
+ .suppress_bind_attrs = true,
+ },
+ .probe = max77705_i2c_probe
+};
+module_i2c_driver(max77705_i2c_driver);
+
+MODULE_DESCRIPTION("Maxim MAX77705 PMIC core driver");
+MODULE_AUTHOR("Dzmitry Sankouski <dsankouski@gmail.com>");
+MODULE_LICENSE("GPL");
diff --git a/include/linux/mfd/max77693-common.h b/include/linux/mfd/max77693-common.h
index a5bce099f1ed..ec2e1b2dceb8 100644
--- a/include/linux/mfd/max77693-common.h
+++ b/include/linux/mfd/max77693-common.h
@@ -1,6 +1,6 @@
/* SPDX-License-Identifier: GPL-2.0+ */
/*
- * Common data shared between Maxim 77693 and 77843 drivers
+ * Common data shared between Maxim 77693, 77705 and 77843 drivers
*
* Copyright (C) 2015 Samsung Electronics
*/
@@ -11,6 +11,7 @@
enum max77693_types {
TYPE_MAX77693_UNKNOWN,
TYPE_MAX77693,
+ TYPE_MAX77705,
TYPE_MAX77843,
TYPE_MAX77693_NUM,
@@ -32,6 +33,7 @@ struct max77693_dev {
struct regmap *regmap_muic;
struct regmap *regmap_haptic; /* Only MAX77693 */
struct regmap *regmap_chg; /* Only MAX77843 */
+ struct regmap *regmap_leds; /* Only MAX77705 */
struct regmap_irq_chip_data *irq_data_led;
struct regmap_irq_chip_data *irq_data_topsys;
diff --git a/include/linux/mfd/max77705-private.h b/include/linux/mfd/max77705-private.h
new file mode 100644
index 000000000000..e4309d3b6f2a
--- /dev/null
+++ b/include/linux/mfd/max77705-private.h
@@ -0,0 +1,178 @@
+/* SPDX-License-Identifier: GPL-2.0 */
+/*
+ * Maxim MAX77705 definitions.
+ *
+ * Copyright (C) 2015 Samsung Electronics, Inc.
+ * Copyright (C) 2024 Dzmitry Sankouski <dsankouski@gmail.com>
+ */
+
+#ifndef __LINUX_MFD_MAX77705_PRIV_H
+#define __LINUX_MFD_MAX77705_PRIV_H
+
+#define MAX77705_SRC_IRQ_CHG BIT(0)
+#define MAX77705_SRC_IRQ_TOP BIT(1)
+#define MAX77705_SRC_IRQ_FG BIT(2)
+#define MAX77705_SRC_IRQ_USBC BIT(3)
+#define MAX77705_SRC_IRQ_ALL (MAX77705_SRC_IRQ_CHG | MAX77705_SRC_IRQ_TOP | \
+ MAX77705_SRC_IRQ_FG | MAX77705_SRC_IRQ_USBC)
+
+/* MAX77705_PMIC_REG_PMICREV register */
+#define MAX77705_VERSION_SHIFT 3
+#define MAX77705_REVISION_MASK GENMASK(2, 0)
+#define MAX77705_VERSION_MASK GENMASK(7, MAX77705_VERSION_SHIFT)
+/* MAX77705_PMIC_REG_MAINCTRL1 register */
+#define MAX77705_MAINCTRL1_BIASEN_SHIFT 7
+#define MAX77705_MAINCTRL1_BIASEN_MASK BIT(MAX77705_MAINCTRL1_BIASEN_SHIFT)
+/* MAX77705_PMIC_REG_MCONFIG2 (haptics) register */
+#define MAX77705_CONFIG2_MEN_SHIFT 6
+#define MAX77705_CONFIG2_MODE_SHIFT 7
+#define MAX77705_CONFIG2_HTYP_SHIFT 5
+/* MAX77705_PMIC_REG_SYSTEM_INT_MASK register */
+#define MAX77705_SYSTEM_IRQ_BSTEN_INT BIT(3)
+#define MAX77705_SYSTEM_IRQ_SYSUVLO_INT BIT(4)
+#define MAX77705_SYSTEM_IRQ_SYSOVLO_INT BIT(5)
+#define MAX77705_SYSTEM_IRQ_TSHDN_INT BIT(6)
+#define MAX77705_SYSTEM_IRQ_TM_INT BIT(7)
+
+enum max77705_hw_rev {
+ MAX77705_PASS1 = 1,
+ MAX77705_PASS2,
+ MAX77705_PASS3
+};
+
+enum max77705_reg {
+ MAX77705_PMIC_REG_PMICID1 = 0x00,
+ MAX77705_PMIC_REG_PMICREV = 0x01,
+ MAX77705_PMIC_REG_MAINCTRL1 = 0x02,
+ MAX77705_PMIC_REG_BSTOUT_MASK = 0x03,
+ MAX77705_PMIC_REG_FORCE_EN_MASK = 0x08,
+ MAX77705_PMIC_REG_MCONFIG = 0x10,
+ MAX77705_PMIC_REG_MCONFIG2 = 0x11,
+ MAX77705_PMIC_REG_INTSRC = 0x22,
+ MAX77705_PMIC_REG_INTSRC_MASK = 0x23,
+ MAX77705_PMIC_REG_SYSTEM_INT = 0x24,
+ MAX77705_PMIC_REG_RESERVED_25 = 0x25,
+ MAX77705_PMIC_REG_SYSTEM_INT_MASK = 0x26,
+ MAX77705_PMIC_REG_RESERVED_27 = 0x27,
+ MAX77705_PMIC_REG_RESERVED_28 = 0x28,
+ MAX77705_PMIC_REG_RESERVED_29 = 0x29,
+ MAX77705_PMIC_REG_BOOSTCONTROL1 = 0x4C,
+ MAX77705_PMIC_REG_BOOSTCONTROL2 = 0x4F,
+ MAX77705_PMIC_REG_SW_RESET = 0x50,
+ MAX77705_PMIC_REG_USBC_RESET = 0x51,
+
+ MAX77705_PMIC_REG_END
+};
+
+enum max77705_chg_reg {
+ MAX77705_CHG_REG_BASE = 0xB0,
+ MAX77705_CHG_REG_INT = 0,
+ MAX77705_CHG_REG_INT_MASK,
+ MAX77705_CHG_REG_INT_OK,
+ MAX77705_CHG_REG_DETAILS_00,
+ MAX77705_CHG_REG_DETAILS_01,
+ MAX77705_CHG_REG_DETAILS_02,
+ MAX77705_CHG_REG_DTLS_03,
+ MAX77705_CHG_REG_CNFG_00,
+ MAX77705_CHG_REG_CNFG_01,
+ MAX77705_CHG_REG_CNFG_02,
+ MAX77705_CHG_REG_CNFG_03,
+ MAX77705_CHG_REG_CNFG_04,
+ MAX77705_CHG_REG_CNFG_05,
+ MAX77705_CHG_REG_CNFG_06,
+ MAX77705_CHG_REG_CNFG_07,
+ MAX77705_CHG_REG_CNFG_08,
+ MAX77705_CHG_REG_CNFG_09,
+ MAX77705_CHG_REG_CNFG_10,
+ MAX77705_CHG_REG_CNFG_11,
+
+ MAX77705_CHG_REG_CNFG_12,
+ MAX77705_CHG_REG_CNFG_13,
+ MAX77705_CHG_REG_CNFG_14,
+ MAX77705_CHG_REG_SAFEOUT_CTRL
+};
+
+enum max77705_fuelgauge_reg {
+ STATUS_REG = 0x00,
+ VALRT_THRESHOLD_REG = 0x01,
+ TALRT_THRESHOLD_REG = 0x02,
+ SALRT_THRESHOLD_REG = 0x03,
+ REMCAP_REP_REG = 0x05,
+ SOCREP_REG = 0x06,
+ TEMPERATURE_REG = 0x08,
+ VCELL_REG = 0x09,
+ TIME_TO_EMPTY_REG = 0x11,
+ FULLSOCTHR_REG = 0x13,
+ CURRENT_REG = 0x0A,
+ AVG_CURRENT_REG = 0x0B,
+ SOCMIX_REG = 0x0D,
+ SOCAV_REG = 0x0E,
+ REMCAP_MIX_REG = 0x0F,
+ FULLCAP_REG = 0x10,
+ RFAST_REG = 0x15,
+ AVR_TEMPERATURE_REG = 0x16,
+ CYCLES_REG = 0x17,
+ DESIGNCAP_REG = 0x18,
+ AVR_VCELL_REG = 0x19,
+ TIME_TO_FULL_REG = 0x20,
+ CONFIG_REG = 0x1D,
+ ICHGTERM_REG = 0x1E,
+ REMCAP_AV_REG = 0x1F,
+ FULLCAP_NOM_REG = 0x23,
+ LEARN_CFG_REG = 0x28,
+ FILTER_CFG_REG = 0x29,
+ MISCCFG_REG = 0x2B,
+ QRTABLE20_REG = 0x32,
+ FULLCAP_REP_REG = 0x35,
+ RCOMP_REG = 0x38,
+ VEMPTY_REG = 0x3A,
+ FSTAT_REG = 0x3D,
+ DISCHARGE_THRESHOLD_REG = 0x40,
+ QRTABLE30_REG = 0x42,
+ ISYS_REG = 0x43,
+ DQACC_REG = 0x45,
+ DPACC_REG = 0x46,
+ AVGISYS_REG = 0x4B,
+ QH_REG = 0x4D,
+ VSYS_REG = 0xB1,
+ TALRTTH2_REG = 0xB2,
+ VBYP_REG = 0xB3,
+ CONFIG2_REG = 0xBB,
+ IIN_REG = 0xD0,
+ OCV_REG = 0xEE,
+ VFOCV_REG = 0xFB,
+ VFSOC_REG = 0xFF,
+
+ MAX77705_FG_END
+};
+
+enum max77705_led_reg {
+ MAX77705_RGBLED_REG_BASE = 0x30,
+ MAX77705_RGBLED_REG_LEDEN = 0,
+ MAX77705_RGBLED_REG_LED0BRT,
+ MAX77705_RGBLED_REG_LED1BRT,
+ MAX77705_RGBLED_REG_LED2BRT,
+ MAX77705_RGBLED_REG_LED3BRT,
+ MAX77705_RGBLED_REG_LEDRMP,
+ MAX77705_RGBLED_REG_LEDBLNK,
+ MAX77705_LED_REG_END
+};
+
+enum max77705_charger_battery_state {
+ MAX77705_BATTERY_NOBAT,
+ MAX77705_BATTERY_PREQUALIFICATION,
+ MAX77705_BATTERY_DEAD,
+ MAX77705_BATTERY_GOOD,
+ MAX77705_BATTERY_LOWVOLTAGE,
+ MAX77705_BATTERY_OVERVOLTAGE,
+ MAX77705_BATTERY_RESERVED
+};
+
+enum max77705_charger_charge_type {
+ MAX77705_CHARGER_CONSTANT_CURRENT = 1,
+ MAX77705_CHARGER_CONSTANT_VOLTAGE,
+ MAX77705_CHARGER_END_OF_CHARGE,
+ MAX77705_CHARGER_DONE
+};
+
+#endif /* __LINUX_MFD_MAX77705_PRIV_H */
--
2.39.5
^ permalink raw reply related [flat|nested] 23+ messages in thread* Re: [PATCH v14 08/10] mfd: Add new driver for MAX77705 PMIC
2025-01-08 14:13 ` [PATCH v14 08/10] mfd: Add new driver for MAX77705 PMIC Dzmitry Sankouski
@ 2025-01-09 12:21 ` Lee Jones
2025-01-13 17:42 ` Dzmitry Sankouski
0 siblings, 1 reply; 23+ messages in thread
From: Lee Jones @ 2025-01-09 12:21 UTC (permalink / raw)
To: Dzmitry Sankouski
Cc: Sebastian Reichel, Chanwoo Choi, Krzysztof Kozlowski, Rob Herring,
Conor Dooley, Dmitry Torokhov, Pavel Machek, Hans de Goede,
Marek Szyprowski, Sebastian Krzyszkowiak, Purism Kernel Team,
Krzysztof Kozlowski, linux-pm, linux-kernel, devicetree,
linux-input, linux-leds
On Wed, 08 Jan 2025, Dzmitry Sankouski wrote:
> Add the core MFD driver for max77705 PMIC. Drivers for sub-devices
> will be added in subsequent patches.
>
> Signed-off-by: Dzmitry Sankouski <dsankouski@gmail.com>
>
> ---
> Changes for v12:
> - remove fuelgauge support (it's moved to simple-mfd-i2c, to reflect
> the fact it's a separate device with it's own i2c client)
> - remove unneeded ending comma
> - remove struct dev_pm_ops because defined by DEFINE_SIMPLE_DEV_PM_OPS
> - Kconfig: select MFD_SIMPLE_MFD_I2C
>
> Changes for v10:
> - never blank line between call and its error check
> - remove unnecessary line wrap
> - revert wrong changes in max77693-common.h
> - move max77705_pm_ops from header to c file
> - fail probe, when fuelgauge is not found in sub device list
> - remove fuelgauge compatible, because with compatible,
> platform matches using compatible, and platform_device
> id_entry is empty. With no compatible, platform matches
> by device id, and id_entry is populated.
> - use dev_err_probe for error handling
>
> Changes for v9:
> - use max17042 as fuel gauge chip
> - initialize max17042 i2c dummy device in mfd device,
> because bus can be used for reading additional values,
> not related to fuelgauge, like chip input current, system
> bus current
> - fix pmic_rev kernel test robot error
>
> Changes for v8:
> - fix comment style C++ -> C
> - remove unused pmic_ver
>
> Changes for v6:
> - add PMIC suffix in Kconfig
> - remove filename from file header
> - reorder headers alphabetically
> - move out fg and chg adresses definitions
> - rename led name and compatible
> - remove overbracketing
> - move charger and fuel gauge i2c initialization
> to their drivers
> - fix max77705_i2c_driver tabbing
> - formatting fixes
> Changes for v5:
> - license change to 2.0
> - use same hardware name in Kconfig and module descriptions
> Changes for v4:
> - rework driver from scratch
> - migrate to regmap_add_irq_chip, remove max77705-irq.c,
> rename max77705-core.c to max77705.c
> - cleanup headers
> - remove debugfs code
> - migrate to use max77693_dev structure
> - remove max77705.h
> ---
> MAINTAINERS | 2 ++
> drivers/mfd/Kconfig | 13 +++++++++
> drivers/mfd/Makefile | 2 ++
> drivers/mfd/max77705.c | 209 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> include/linux/mfd/max77693-common.h | 4 ++-
> include/linux/mfd/max77705-private.h | 178 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> 6 files changed, 407 insertions(+), 1 deletion(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 98a77dece2f1..eb5222a833f8 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -14294,6 +14294,7 @@ F: drivers/*/*max77843.c
> F: drivers/*/max14577*.c
> F: drivers/*/max77686*.c
> F: drivers/*/max77693*.c
> +F: drivers/*/max77705*.c
> F: drivers/clk/clk-max77686.c
> F: drivers/extcon/extcon-max14577.c
> F: drivers/extcon/extcon-max77693.c
> @@ -14301,6 +14302,7 @@ F: drivers/rtc/rtc-max77686.c
> F: include/linux/mfd/max14577*.h
> F: include/linux/mfd/max77686*.h
> F: include/linux/mfd/max77693*.h
> +F: include/linux/mfd/max77705*.h
>
> MAXIRADIO FM RADIO RECEIVER DRIVER
> M: Hans Verkuil <hverkuil@xs4all.nl>
> diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig
> index 6b0682af6e32..5064b1b42f76 100644
> --- a/drivers/mfd/Kconfig
> +++ b/drivers/mfd/Kconfig
> @@ -916,6 +916,19 @@ config MFD_MAX77693
> additional drivers must be enabled in order to use the functionality
> of the device.
>
> +config MFD_MAX77705
> + tristate "Maxim MAX77705 PMIC Support"
> + depends on I2C
> + select MFD_CORE
> + select MFD_SIMPLE_MFD_I2C
> + help
> + Say yes here to add support for Maxim Integrated MAX77705 PMIC.
> + This is a Power Management IC with Charger, safe LDOs, Flash, Haptic
> + and MUIC controls on chip.
> + This driver provides common support for accessing the device;
> + additional drivers must be enabled in order to use the functionality
> + of the device.
> +
> config MFD_MAX77714
> tristate "Maxim Semiconductor MAX77714 PMIC Support"
> depends on I2C
> diff --git a/drivers/mfd/Makefile b/drivers/mfd/Makefile
> index 9220eaf7cf12..c18bb5a57734 100644
> --- a/drivers/mfd/Makefile
> +++ b/drivers/mfd/Makefile
> @@ -168,6 +168,7 @@ obj-$(CONFIG_MFD_MAX77620) += max77620.o
> obj-$(CONFIG_MFD_MAX77650) += max77650.o
> obj-$(CONFIG_MFD_MAX77686) += max77686.o
> obj-$(CONFIG_MFD_MAX77693) += max77693.o
> +obj-$(CONFIG_MFD_MAX77705) += max77705.o
> obj-$(CONFIG_MFD_MAX77714) += max77714.o
> obj-$(CONFIG_MFD_MAX77843) += max77843.o
> obj-$(CONFIG_MFD_MAX8907) += max8907.o
> @@ -233,6 +234,7 @@ obj-$(CONFIG_MFD_RK8XX_I2C) += rk8xx-i2c.o
> obj-$(CONFIG_MFD_RK8XX_SPI) += rk8xx-spi.o
> obj-$(CONFIG_MFD_RN5T618) += rn5t618.o
> obj-$(CONFIG_MFD_SEC_CORE) += sec-core.o sec-irq.o
> +obj-$(CONFIG_MFD_S2DOS05) += s2dos05.o
> obj-$(CONFIG_MFD_SYSCON) += syscon.o
> obj-$(CONFIG_MFD_LM3533) += lm3533-core.o lm3533-ctrlbank.o
> obj-$(CONFIG_MFD_VEXPRESS_SYSREG) += vexpress-sysreg.o
> diff --git a/drivers/mfd/max77705.c b/drivers/mfd/max77705.c
> new file mode 100644
> index 000000000000..18d344620755
> --- /dev/null
> +++ b/drivers/mfd/max77705.c
> @@ -0,0 +1,209 @@
> +// SPDX-License-Identifier: GPL-2.0+
> +//
Only the SPDX should have C++ style comments.
> +// Maxim MAX77705 PMIC core driver
> +//
> +// Copyright (C) 2024 Dzmitry Sankouski <dsankouski@gmail.com>
Update the date when you resubmit.
> +#include <linux/i2c.h>
> +#include <linux/interrupt.h>
> +#include <linux/mfd/core.h>
> +#include <linux/mfd/max77705-private.h>
> +#include <linux/mfd/max77693-common.h>
> +#include <linux/pm.h>
> +#include <linux/power/max17042_battery.h>
> +#include <linux/module.h>
> +#include <linux/regmap.h>
> +#include <linux/of.h>
> +
> +static struct mfd_cell max77705_devs[] = {
> + {
> + .name = "max77705-rgb",
> + .of_compatible = "maxim,max77705-rgb",
> + },
> + {
> + .name = "max77705-charger",
> + .of_compatible = "maxim,max77705-charger",
> + },
> + {
> + .name = "max77705-haptic",
> + .of_compatible = "maxim,max77705-haptic",
> + },
> +};
Use MFD_CELL_OF()
> +static const struct regmap_range max77705_readable_ranges[] = {
> + regmap_reg_range(MAX77705_PMIC_REG_PMICID1, MAX77705_PMIC_REG_BSTOUT_MASK),
> + regmap_reg_range(MAX77705_PMIC_REG_INTSRC, MAX77705_PMIC_REG_RESERVED_29),
> + regmap_reg_range(MAX77705_PMIC_REG_BOOSTCONTROL1, MAX77705_PMIC_REG_BOOSTCONTROL1),
> + regmap_reg_range(MAX77705_PMIC_REG_MCONFIG, MAX77705_PMIC_REG_MCONFIG2),
> + regmap_reg_range(MAX77705_PMIC_REG_FORCE_EN_MASK, MAX77705_PMIC_REG_FORCE_EN_MASK),
> + regmap_reg_range(MAX77705_PMIC_REG_BOOSTCONTROL1, MAX77705_PMIC_REG_BOOSTCONTROL1),
> + regmap_reg_range(MAX77705_PMIC_REG_BOOSTCONTROL2, MAX77705_PMIC_REG_BOOSTCONTROL2),
> + regmap_reg_range(MAX77705_PMIC_REG_SW_RESET, MAX77705_PMIC_REG_USBC_RESET),
> +};
> +
> +static const struct regmap_range max77705_writable_ranges[] = {
> + regmap_reg_range(MAX77705_PMIC_REG_MAINCTRL1, MAX77705_PMIC_REG_BSTOUT_MASK),
> + regmap_reg_range(MAX77705_PMIC_REG_INTSRC, MAX77705_PMIC_REG_RESERVED_29),
> + regmap_reg_range(MAX77705_PMIC_REG_BOOSTCONTROL1, MAX77705_PMIC_REG_BOOSTCONTROL1),
> + regmap_reg_range(MAX77705_PMIC_REG_MCONFIG, MAX77705_PMIC_REG_MCONFIG2),
> + regmap_reg_range(MAX77705_PMIC_REG_FORCE_EN_MASK, MAX77705_PMIC_REG_FORCE_EN_MASK),
> + regmap_reg_range(MAX77705_PMIC_REG_BOOSTCONTROL1, MAX77705_PMIC_REG_BOOSTCONTROL1),
> + regmap_reg_range(MAX77705_PMIC_REG_BOOSTCONTROL2, MAX77705_PMIC_REG_BOOSTCONTROL2),
> + regmap_reg_range(MAX77705_PMIC_REG_SW_RESET, MAX77705_PMIC_REG_USBC_RESET),
> +
Remove this line.
> +};
> +
> +static const struct regmap_access_table max77705_readable_table = {
> + .yes_ranges = max77705_readable_ranges,
> + .n_yes_ranges = ARRAY_SIZE(max77705_readable_ranges),
> +};
> +
> +static const struct regmap_access_table max77705_writable_table = {
> + .yes_ranges = max77705_writable_ranges,
> + .n_yes_ranges = ARRAY_SIZE(max77705_writable_ranges),
> +};
> +
> +static const struct regmap_config max77705_regmap_config = {
> + .reg_bits = 8,
> + .val_bits = 8,
> + .rd_table = &max77705_readable_table,
> + .wr_table = &max77705_writable_table,
> + .max_register = MAX77705_PMIC_REG_USBC_RESET,
> +};
> +
> +static const struct regmap_config max77705_leds_regmap_config = {
> + .reg_base = MAX77705_RGBLED_REG_BASE,
> + .reg_bits = 8,
> + .val_bits = 8,
> + .max_register = MAX77705_LED_REG_END,
> +};
> +
> +static const struct regmap_irq max77705_topsys_irqs[] = {
> + { .mask = MAX77705_SYSTEM_IRQ_BSTEN_INT, },
> + { .mask = MAX77705_SYSTEM_IRQ_SYSUVLO_INT, },
> + { .mask = MAX77705_SYSTEM_IRQ_SYSOVLO_INT, },
> + { .mask = MAX77705_SYSTEM_IRQ_TSHDN_INT, },
> + { .mask = MAX77705_SYSTEM_IRQ_TM_INT, },
Remove the double spaces.
> +};
> +
> +static const struct regmap_irq_chip max77705_topsys_irq_chip = {
> + .name = "max77705-topsys",
> + .status_base = MAX77705_PMIC_REG_SYSTEM_INT,
> + .mask_base = MAX77705_PMIC_REG_SYSTEM_INT_MASK,
> + .num_regs = 1,
> + .irqs = max77705_topsys_irqs,
> + .num_irqs = ARRAY_SIZE(max77705_topsys_irqs),
Remove the superfluous tab.
> +};
> +
> +static int max77705_i2c_probe(struct i2c_client *i2c)
> +{
> + struct max77693_dev *max77705;
> + struct regmap_irq_chip_data *irq_data;
> + struct irq_domain *domain;
> + int ret;
> + unsigned int pmic_rev_value;
> + enum max77705_hw_rev pmic_rev;
Please reorder these. At the very least, put ret at the bottom.
> +
> + max77705 = devm_kzalloc(&i2c->dev, sizeof(*max77705), GFP_KERNEL);
> + if (!max77705)
> + return -ENOMEM;
> +
> + max77705->i2c = i2c;
> + max77705->dev = &i2c->dev;
> + max77705->irq = i2c->irq;
If you're saving i2c, you can derive the other two.
> + max77705->type = TYPE_MAX77705;
> + i2c_set_clientdata(i2c, max77705);
> +
> + max77705->regmap = devm_regmap_init_i2c(i2c, &max77705_regmap_config);
> + if (IS_ERR(max77705->regmap))
> + return PTR_ERR(max77705->regmap);
> +
> + if (regmap_read(max77705->regmap, MAX77705_PMIC_REG_PMICREV, &pmic_rev_value) < 0)
Take the function all out of the if statement please.
> + return -ENODEV;
> +
> + pmic_rev = pmic_rev_value & MAX77705_REVISION_MASK;
> + if (pmic_rev != MAX77705_PASS3)
> + return dev_err_probe(max77705->dev, -ENODEV,
Place dev into a local variable and save yourself lots of chars each time.
> + "Rev.0x%x is not tested\n", pmic_rev);
Use up to 100-chars to neaten this up.
> +
> + max77705->regmap_leds = devm_regmap_init_i2c(i2c, &max77705_leds_regmap_config);
Why not create this in the LEDs driver?
> + if (IS_ERR(max77705->regmap_leds))
> + return dev_err_probe(max77705->dev, PTR_ERR(max77705->regmap_leds),
> + "Failed to register leds regmap\n");
LEDs
> + ret = devm_regmap_add_irq_chip(max77705->dev, max77705->regmap,
> + max77705->irq,
> + IRQF_ONESHOT | IRQF_SHARED, 0,
> + &max77705_topsys_irq_chip,
> + &irq_data);
> + if (ret)
> + return dev_err_probe(max77705->dev, ret, "Failed to add irq chip\n");
IRQ
> +
> + /* Unmask interrupts from all blocks in interrupt source register */
> + ret = regmap_update_bits(max77705->regmap,
> + MAX77705_PMIC_REG_INTSRC_MASK,
> + MAX77705_SRC_IRQ_ALL, (unsigned int)~MAX77705_SRC_IRQ_ALL);
I don't see this much. Are you sure the cast is required?
> + if (ret < 0)
> + return dev_err_probe(max77705->dev, ret,
> + "Could not unmask interrupts in INTSRC\n");
> +
> + domain = regmap_irq_get_domain(irq_data);
> +
> + ret = devm_mfd_add_devices(max77705->dev, PLATFORM_DEVID_NONE,
> + max77705_devs, ARRAY_SIZE(max77705_devs),
> + NULL, 0, domain);
> + if (ret)
> + return dev_err_probe(max77705->dev, ret, "Failed to register child devices\n");
> +
> + device_init_wakeup(max77705->dev, true);
> +
> + return 0;
> +}
> +
> +static int max77705_suspend(struct device *dev)
> +{
> + struct i2c_client *i2c = to_i2c_client(dev);
> + struct max77693_dev *max77705 = i2c_get_clientdata(i2c);
You're given a `struct device` use that instead of converting.
> +
> + disable_irq(max77705->irq);
> +
> + if (device_may_wakeup(dev))
> + enable_irq_wake(max77705->irq);
> +
> + return 0;
> +}
> +
> +static int max77705_resume(struct device *dev)
> +{
> + struct i2c_client *i2c = to_i2c_client(dev);
> + struct max77693_dev *max77705 = i2c_get_clientdata(i2c);
> +
> + if (device_may_wakeup(dev))
> + disable_irq_wake(max77705->irq);
> +
> + enable_irq(max77705->irq);
> +
> + return 0;
> +}
> +DEFINE_SIMPLE_DEV_PM_OPS(max77705_pm_ops, max77705_suspend, max77705_resume);
> +
> +static const struct of_device_id max77705_i2c_of_match[] = {
> + { .compatible = "maxim,max77705" },
> + { }
> +};
> +MODULE_DEVICE_TABLE(of, max77705_i2c_of_match);
> +
> +static struct i2c_driver max77705_i2c_driver = {
> + .driver = {
> + .name = "max77705",
> + .of_match_table = max77705_i2c_of_match,
> + .pm = pm_sleep_ptr(&max77705_pm_ops),
> + .suppress_bind_attrs = true,
> + },
> + .probe = max77705_i2c_probe
> +};
> +module_i2c_driver(max77705_i2c_driver);
> +
> +MODULE_DESCRIPTION("Maxim MAX77705 PMIC core driver");
> +MODULE_AUTHOR("Dzmitry Sankouski <dsankouski@gmail.com>");
> +MODULE_LICENSE("GPL");
> diff --git a/include/linux/mfd/max77693-common.h b/include/linux/mfd/max77693-common.h
> index a5bce099f1ed..ec2e1b2dceb8 100644
> --- a/include/linux/mfd/max77693-common.h
> +++ b/include/linux/mfd/max77693-common.h
> @@ -1,6 +1,6 @@
> /* SPDX-License-Identifier: GPL-2.0+ */
> /*
> - * Common data shared between Maxim 77693 and 77843 drivers
> + * Common data shared between Maxim 77693, 77705 and 77843 drivers
> *
> * Copyright (C) 2015 Samsung Electronics
> */
> @@ -11,6 +11,7 @@
> enum max77693_types {
> TYPE_MAX77693_UNKNOWN,
> TYPE_MAX77693,
> + TYPE_MAX77705,
> TYPE_MAX77843,
>
> TYPE_MAX77693_NUM,
> @@ -32,6 +33,7 @@ struct max77693_dev {
> struct regmap *regmap_muic;
> struct regmap *regmap_haptic; /* Only MAX77693 */
> struct regmap *regmap_chg; /* Only MAX77843 */
> + struct regmap *regmap_leds; /* Only MAX77705 */
>
> struct regmap_irq_chip_data *irq_data_led;
> struct regmap_irq_chip_data *irq_data_topsys;
> diff --git a/include/linux/mfd/max77705-private.h b/include/linux/mfd/max77705-private.h
> new file mode 100644
> index 000000000000..e4309d3b6f2a
> --- /dev/null
> +++ b/include/linux/mfd/max77705-private.h
> @@ -0,0 +1,178 @@
> +/* SPDX-License-Identifier: GPL-2.0 */
> +/*
> + * Maxim MAX77705 definitions.
> + *
> + * Copyright (C) 2015 Samsung Electronics, Inc.
> + * Copyright (C) 2024 Dzmitry Sankouski <dsankouski@gmail.com>
> + */
> +
> +#ifndef __LINUX_MFD_MAX77705_PRIV_H
> +#define __LINUX_MFD_MAX77705_PRIV_H
> +
> +#define MAX77705_SRC_IRQ_CHG BIT(0)
> +#define MAX77705_SRC_IRQ_TOP BIT(1)
> +#define MAX77705_SRC_IRQ_FG BIT(2)
> +#define MAX77705_SRC_IRQ_USBC BIT(3)
> +#define MAX77705_SRC_IRQ_ALL (MAX77705_SRC_IRQ_CHG | MAX77705_SRC_IRQ_TOP | \
> + MAX77705_SRC_IRQ_FG | MAX77705_SRC_IRQ_USBC)
> +
> +/* MAX77705_PMIC_REG_PMICREV register */
> +#define MAX77705_VERSION_SHIFT 3
> +#define MAX77705_REVISION_MASK GENMASK(2, 0)
> +#define MAX77705_VERSION_MASK GENMASK(7, MAX77705_VERSION_SHIFT)
> +/* MAX77705_PMIC_REG_MAINCTRL1 register */
> +#define MAX77705_MAINCTRL1_BIASEN_SHIFT 7
> +#define MAX77705_MAINCTRL1_BIASEN_MASK BIT(MAX77705_MAINCTRL1_BIASEN_SHIFT)
> +/* MAX77705_PMIC_REG_MCONFIG2 (haptics) register */
> +#define MAX77705_CONFIG2_MEN_SHIFT 6
> +#define MAX77705_CONFIG2_MODE_SHIFT 7
> +#define MAX77705_CONFIG2_HTYP_SHIFT 5
> +/* MAX77705_PMIC_REG_SYSTEM_INT_MASK register */
> +#define MAX77705_SYSTEM_IRQ_BSTEN_INT BIT(3)
> +#define MAX77705_SYSTEM_IRQ_SYSUVLO_INT BIT(4)
> +#define MAX77705_SYSTEM_IRQ_SYSOVLO_INT BIT(5)
> +#define MAX77705_SYSTEM_IRQ_TSHDN_INT BIT(6)
> +#define MAX77705_SYSTEM_IRQ_TM_INT BIT(7)
> +
> +enum max77705_hw_rev {
> + MAX77705_PASS1 = 1,
> + MAX77705_PASS2,
> + MAX77705_PASS3
> +};
> +
> +enum max77705_reg {
> + MAX77705_PMIC_REG_PMICID1 = 0x00,
> + MAX77705_PMIC_REG_PMICREV = 0x01,
> + MAX77705_PMIC_REG_MAINCTRL1 = 0x02,
> + MAX77705_PMIC_REG_BSTOUT_MASK = 0x03,
> + MAX77705_PMIC_REG_FORCE_EN_MASK = 0x08,
> + MAX77705_PMIC_REG_MCONFIG = 0x10,
> + MAX77705_PMIC_REG_MCONFIG2 = 0x11,
> + MAX77705_PMIC_REG_INTSRC = 0x22,
> + MAX77705_PMIC_REG_INTSRC_MASK = 0x23,
> + MAX77705_PMIC_REG_SYSTEM_INT = 0x24,
> + MAX77705_PMIC_REG_RESERVED_25 = 0x25,
> + MAX77705_PMIC_REG_SYSTEM_INT_MASK = 0x26,
> + MAX77705_PMIC_REG_RESERVED_27 = 0x27,
> + MAX77705_PMIC_REG_RESERVED_28 = 0x28,
> + MAX77705_PMIC_REG_RESERVED_29 = 0x29,
> + MAX77705_PMIC_REG_BOOSTCONTROL1 = 0x4C,
> + MAX77705_PMIC_REG_BOOSTCONTROL2 = 0x4F,
> + MAX77705_PMIC_REG_SW_RESET = 0x50,
> + MAX77705_PMIC_REG_USBC_RESET = 0x51,
> +
> + MAX77705_PMIC_REG_END
> +};
> +
> +enum max77705_chg_reg {
> + MAX77705_CHG_REG_BASE = 0xB0,
> + MAX77705_CHG_REG_INT = 0,
> + MAX77705_CHG_REG_INT_MASK,
> + MAX77705_CHG_REG_INT_OK,
> + MAX77705_CHG_REG_DETAILS_00,
> + MAX77705_CHG_REG_DETAILS_01,
> + MAX77705_CHG_REG_DETAILS_02,
> + MAX77705_CHG_REG_DTLS_03,
> + MAX77705_CHG_REG_CNFG_00,
> + MAX77705_CHG_REG_CNFG_01,
> + MAX77705_CHG_REG_CNFG_02,
> + MAX77705_CHG_REG_CNFG_03,
> + MAX77705_CHG_REG_CNFG_04,
> + MAX77705_CHG_REG_CNFG_05,
> + MAX77705_CHG_REG_CNFG_06,
> + MAX77705_CHG_REG_CNFG_07,
> + MAX77705_CHG_REG_CNFG_08,
> + MAX77705_CHG_REG_CNFG_09,
> + MAX77705_CHG_REG_CNFG_10,
> + MAX77705_CHG_REG_CNFG_11,
> +
> + MAX77705_CHG_REG_CNFG_12,
> + MAX77705_CHG_REG_CNFG_13,
> + MAX77705_CHG_REG_CNFG_14,
> + MAX77705_CHG_REG_SAFEOUT_CTRL
> +};
> +
> +enum max77705_fuelgauge_reg {
> + STATUS_REG = 0x00,
> + VALRT_THRESHOLD_REG = 0x01,
> + TALRT_THRESHOLD_REG = 0x02,
> + SALRT_THRESHOLD_REG = 0x03,
> + REMCAP_REP_REG = 0x05,
> + SOCREP_REG = 0x06,
> + TEMPERATURE_REG = 0x08,
> + VCELL_REG = 0x09,
> + TIME_TO_EMPTY_REG = 0x11,
> + FULLSOCTHR_REG = 0x13,
> + CURRENT_REG = 0x0A,
> + AVG_CURRENT_REG = 0x0B,
> + SOCMIX_REG = 0x0D,
> + SOCAV_REG = 0x0E,
> + REMCAP_MIX_REG = 0x0F,
> + FULLCAP_REG = 0x10,
> + RFAST_REG = 0x15,
> + AVR_TEMPERATURE_REG = 0x16,
> + CYCLES_REG = 0x17,
> + DESIGNCAP_REG = 0x18,
> + AVR_VCELL_REG = 0x19,
> + TIME_TO_FULL_REG = 0x20,
> + CONFIG_REG = 0x1D,
> + ICHGTERM_REG = 0x1E,
> + REMCAP_AV_REG = 0x1F,
> + FULLCAP_NOM_REG = 0x23,
> + LEARN_CFG_REG = 0x28,
> + FILTER_CFG_REG = 0x29,
> + MISCCFG_REG = 0x2B,
> + QRTABLE20_REG = 0x32,
> + FULLCAP_REP_REG = 0x35,
> + RCOMP_REG = 0x38,
> + VEMPTY_REG = 0x3A,
> + FSTAT_REG = 0x3D,
> + DISCHARGE_THRESHOLD_REG = 0x40,
> + QRTABLE30_REG = 0x42,
> + ISYS_REG = 0x43,
> + DQACC_REG = 0x45,
> + DPACC_REG = 0x46,
> + AVGISYS_REG = 0x4B,
> + QH_REG = 0x4D,
> + VSYS_REG = 0xB1,
> + TALRTTH2_REG = 0xB2,
> + VBYP_REG = 0xB3,
> + CONFIG2_REG = 0xBB,
> + IIN_REG = 0xD0,
> + OCV_REG = 0xEE,
> + VFOCV_REG = 0xFB,
> + VFSOC_REG = 0xFF,
> +
> + MAX77705_FG_END
> +};
> +
> +enum max77705_led_reg {
> + MAX77705_RGBLED_REG_BASE = 0x30,
> + MAX77705_RGBLED_REG_LEDEN = 0,
> + MAX77705_RGBLED_REG_LED0BRT,
> + MAX77705_RGBLED_REG_LED1BRT,
> + MAX77705_RGBLED_REG_LED2BRT,
> + MAX77705_RGBLED_REG_LED3BRT,
> + MAX77705_RGBLED_REG_LEDRMP,
> + MAX77705_RGBLED_REG_LEDBLNK,
> + MAX77705_LED_REG_END
> +};
> +
> +enum max77705_charger_battery_state {
> + MAX77705_BATTERY_NOBAT,
> + MAX77705_BATTERY_PREQUALIFICATION,
> + MAX77705_BATTERY_DEAD,
> + MAX77705_BATTERY_GOOD,
> + MAX77705_BATTERY_LOWVOLTAGE,
> + MAX77705_BATTERY_OVERVOLTAGE,
> + MAX77705_BATTERY_RESERVED
> +};
> +
> +enum max77705_charger_charge_type {
> + MAX77705_CHARGER_CONSTANT_CURRENT = 1,
> + MAX77705_CHARGER_CONSTANT_VOLTAGE,
> + MAX77705_CHARGER_END_OF_CHARGE,
> + MAX77705_CHARGER_DONE
> +};
> +
> +#endif /* __LINUX_MFD_MAX77705_PRIV_H */
>
> --
> 2.39.5
>
--
Lee Jones [李琼斯]
^ permalink raw reply [flat|nested] 23+ messages in thread* Re: [PATCH v14 08/10] mfd: Add new driver for MAX77705 PMIC
2025-01-09 12:21 ` Lee Jones
@ 2025-01-13 17:42 ` Dzmitry Sankouski
0 siblings, 0 replies; 23+ messages in thread
From: Dzmitry Sankouski @ 2025-01-13 17:42 UTC (permalink / raw)
To: Lee Jones
Cc: Sebastian Reichel, Chanwoo Choi, Krzysztof Kozlowski, Rob Herring,
Conor Dooley, Dmitry Torokhov, Pavel Machek, Hans de Goede,
Marek Szyprowski, Sebastian Krzyszkowiak, Purism Kernel Team,
Krzysztof Kozlowski, linux-pm, linux-kernel, devicetree,
linux-input, linux-leds
чт, 9 янв. 2025 г. в 15:21, Lee Jones <lee@kernel.org>:
>
> On Wed, 08 Jan 2025, Dzmitry Sankouski wrote:
>
> > Add the core MFD driver for max77705 PMIC. Drivers for sub-devices
> > will be added in subsequent patches.
> >
> > Signed-off-by: Dzmitry Sankouski <dsankouski@gmail.com>
> >
(...)
> > +
> > + /* Unmask interrupts from all blocks in interrupt source register */
> > + ret = regmap_update_bits(max77705->regmap,
> > + MAX77705_PMIC_REG_INTSRC_MASK,
> > + MAX77705_SRC_IRQ_ALL, (unsigned int)~MAX77705_SRC_IRQ_ALL);
>
> I don't see this much. Are you sure the cast is required?
>
BIT macro creates a 64 bit constant value. When inverted,
it overruns 32 bit value, causing compiler to warn on conversion like
`warning: conversion from ‘long unsigned int’ to ‘unsigned int’`.
--
Best regards and thanks for review,
Dzmitry
^ permalink raw reply [flat|nested] 23+ messages in thread
* [PATCH v14 09/10] input: max77693: add max77705 haptic support
2025-01-08 14:13 [PATCH v14 00/10] Add support for Maxim Integrated MAX77705 PMIC Dzmitry Sankouski
` (7 preceding siblings ...)
2025-01-08 14:13 ` [PATCH v14 08/10] mfd: Add new driver for MAX77705 PMIC Dzmitry Sankouski
@ 2025-01-08 14:13 ` Dzmitry Sankouski
2025-01-08 14:13 ` [PATCH v14 10/10] leds: max77705: Add LEDs support Dzmitry Sankouski
2025-01-15 22:02 ` (subset) [PATCH v14 00/10] Add support for Maxim Integrated MAX77705 PMIC Sebastian Reichel
10 siblings, 0 replies; 23+ messages in thread
From: Dzmitry Sankouski @ 2025-01-08 14:13 UTC (permalink / raw)
To: Sebastian Reichel, Chanwoo Choi, Krzysztof Kozlowski, Lee Jones,
Rob Herring, Conor Dooley, Dmitry Torokhov, Pavel Machek,
Hans de Goede, Marek Szyprowski, Sebastian Krzyszkowiak,
Purism Kernel Team, Krzysztof Kozlowski
Cc: linux-pm, linux-kernel, devicetree, linux-input, linux-leds,
Dzmitry Sankouski
Add support for haptic controller on MAX77705 Multifunction
device.
This driver supports external pwm and LRA (Linear Resonant Actuator) motor.
User can control the haptic device via force feedback framework.
Acked-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Signed-off-by: Dzmitry Sankouski <dsankouski@gmail.com>
---
Changes in v4:
- add max77705 haptic support to max77693 driver
- delete max77705-haptic
---
drivers/input/misc/Kconfig | 4 ++--
drivers/input/misc/Makefile | 1 +
drivers/input/misc/max77693-haptic.c | 15 ++++++++++++++-
3 files changed, 17 insertions(+), 3 deletions(-)
diff --git a/drivers/input/misc/Kconfig b/drivers/input/misc/Kconfig
index 13d135257e06..b9db5abcde08 100644
--- a/drivers/input/misc/Kconfig
+++ b/drivers/input/misc/Kconfig
@@ -241,11 +241,11 @@ config INPUT_MAX77650_ONKEY
config INPUT_MAX77693_HAPTIC
tristate "MAXIM MAX77693/MAX77843 haptic controller support"
- depends on (MFD_MAX77693 || MFD_MAX77843) && PWM
+ depends on (MFD_MAX77693 || MFD_MAX77705 || MFD_MAX77843) && PWM
select INPUT_FF_MEMLESS
help
This option enables support for the haptic controller on
- MAXIM MAX77693 and MAX77843 chips.
+ MAXIM MAX77693, MAX77705 and MAX77843 chips.
To compile this driver as module, choose M here: the
module will be called max77693-haptic.
diff --git a/drivers/input/misc/Makefile b/drivers/input/misc/Makefile
index 6d91804d0a6f..e70164dd73c2 100644
--- a/drivers/input/misc/Makefile
+++ b/drivers/input/misc/Makefile
@@ -53,6 +53,7 @@ obj-$(CONFIG_INPUT_KXTJ9) += kxtj9.o
obj-$(CONFIG_INPUT_M68K_BEEP) += m68kspkr.o
obj-$(CONFIG_INPUT_MAX77650_ONKEY) += max77650-onkey.o
obj-$(CONFIG_INPUT_MAX77693_HAPTIC) += max77693-haptic.o
+obj-$(CONFIG_INPUT_MAX77705_HAPTIC) += max77705-haptic.o
obj-$(CONFIG_INPUT_MAX8925_ONKEY) += max8925_onkey.o
obj-$(CONFIG_INPUT_MAX8997_HAPTIC) += max8997_haptic.o
obj-$(CONFIG_INPUT_MC13783_PWRBUTTON) += mc13783-pwrbutton.o
diff --git a/drivers/input/misc/max77693-haptic.c b/drivers/input/misc/max77693-haptic.c
index 0e646f1b257b..c3b9d33608d7 100644
--- a/drivers/input/misc/max77693-haptic.c
+++ b/drivers/input/misc/max77693-haptic.c
@@ -23,6 +23,7 @@
#include <linux/mfd/max77693.h>
#include <linux/mfd/max77693-common.h>
#include <linux/mfd/max77693-private.h>
+#include <linux/mfd/max77705-private.h>
#include <linux/mfd/max77843-private.h>
#define MAX_MAGNITUDE_SHIFT 16
@@ -115,6 +116,13 @@ static int max77693_haptic_configure(struct max77693_haptic *haptic,
MAX77693_HAPTIC_PWM_DIVISOR_128);
config_reg = MAX77693_HAPTIC_REG_CONFIG2;
break;
+ case TYPE_MAX77705:
+ value = ((haptic->type << MAX77693_CONFIG2_MODE) |
+ (enable << MAX77693_CONFIG2_MEN) |
+ (haptic->mode << MAX77693_CONFIG2_HTYP) |
+ MAX77693_HAPTIC_PWM_DIVISOR_128);
+ config_reg = MAX77705_PMIC_REG_MCONFIG;
+ break;
case TYPE_MAX77843:
value = (haptic->type << MCONFIG_MODE_SHIFT) |
(enable << MCONFIG_MEN_SHIFT) |
@@ -312,6 +320,9 @@ static int max77693_haptic_probe(struct platform_device *pdev)
case TYPE_MAX77693:
haptic->regmap_haptic = max77693->regmap_haptic;
break;
+ case TYPE_MAX77705:
+ haptic->regmap_haptic = max77693->regmap;
+ break;
case TYPE_MAX77843:
haptic->regmap_haptic = max77693->regmap;
break;
@@ -407,6 +418,7 @@ static DEFINE_SIMPLE_DEV_PM_OPS(max77693_haptic_pm_ops,
static const struct platform_device_id max77693_haptic_id[] = {
{ "max77693-haptic", },
+ { "max77705-haptic", },
{ "max77843-haptic", },
{},
};
@@ -414,6 +426,7 @@ MODULE_DEVICE_TABLE(platform, max77693_haptic_id);
static const struct of_device_id of_max77693_haptic_dt_match[] = {
{ .compatible = "maxim,max77693-haptic", },
+ { .compatible = "maxim,max77705-haptic", },
{ .compatible = "maxim,max77843-haptic", },
{ /* sentinel */ },
};
@@ -432,5 +445,5 @@ module_platform_driver(max77693_haptic_driver);
MODULE_AUTHOR("Jaewon Kim <jaewon02.kim@samsung.com>");
MODULE_AUTHOR("Krzysztof Kozlowski <krzk@kernel.org>");
-MODULE_DESCRIPTION("MAXIM 77693/77843 Haptic driver");
+MODULE_DESCRIPTION("MAXIM 77693/77705/77843 Haptic driver");
MODULE_LICENSE("GPL");
--
2.39.5
^ permalink raw reply related [flat|nested] 23+ messages in thread* [PATCH v14 10/10] leds: max77705: Add LEDs support
2025-01-08 14:13 [PATCH v14 00/10] Add support for Maxim Integrated MAX77705 PMIC Dzmitry Sankouski
` (8 preceding siblings ...)
2025-01-08 14:13 ` [PATCH v14 09/10] input: max77693: add max77705 haptic support Dzmitry Sankouski
@ 2025-01-08 14:13 ` Dzmitry Sankouski
2025-01-15 22:02 ` (subset) [PATCH v14 00/10] Add support for Maxim Integrated MAX77705 PMIC Sebastian Reichel
10 siblings, 0 replies; 23+ messages in thread
From: Dzmitry Sankouski @ 2025-01-08 14:13 UTC (permalink / raw)
To: Sebastian Reichel, Chanwoo Choi, Krzysztof Kozlowski, Lee Jones,
Rob Herring, Conor Dooley, Dmitry Torokhov, Pavel Machek,
Hans de Goede, Marek Szyprowski, Sebastian Krzyszkowiak,
Purism Kernel Team, Krzysztof Kozlowski
Cc: linux-pm, linux-kernel, devicetree, linux-input, linux-leds,
Dzmitry Sankouski
This adds basic support for LEDs for the max77705 PMIC.
Signed-off-by: Dzmitry Sankouski <dsankouski@gmail.com>
---
Changes for v11:
- add dependency on led multicolor class
Changes for v10:
- remove multiple line break
- replace en_shift, reg_brightness values with macros
- add support for leds-class-multicolor (inspired from leds-qcom-lpg.c):
- replace max77705_to_led helper with container_of,
because it's no longer covers all cases
- add max77705_add_led, max77705_parse_subled functions
to parse device tree
- split max77705_led_brightness_set into *multi and *single versions
Changes for v8:
- join line where possible to fit in 100 chars
- change comment style C++ -> C
Changes for v6:
- change compatible suffix to 'rgb'
- remove I2C dependency in Kconfig
- remove copyright and author from 'based on' header statement
- replace MFD abbreviation with PMIC
- MAINTAINERS: alphabetic order
- max77705_rgb_blink: replace ternary operators with if..else if sequence
- max77705_rgb_blink: move hardcoded numbers to constants
- max77705_led_brightness_set: move ret to the bottom
- s/map/regmap
- replace device_for_each_child_node with scoped version
- s/rv/ret
Changes for v5:
- use same hardware name in Kconfig and module descriptions
- remove copyrighter owner from module authors
Changes in v4:
- inline BLINK_(ON|OFF) macro
- remove camel case
- drop backwards compatibility(new driver)
- drop module alias
---
MAINTAINERS | 1 +
drivers/leds/Kconfig | 8 +++++
drivers/leds/Makefile | 1 +
drivers/leds/leds-max77705.c | 267 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
include/linux/mfd/max77705-private.h | 17 ++++++++++
5 files changed, 294 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index eb5222a833f8..13fe5f5aea77 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -14290,6 +14290,7 @@ F: Documentation/devicetree/bindings/*/maxim,max77693.yaml
F: Documentation/devicetree/bindings/*/maxim,max77705*.yaml
F: Documentation/devicetree/bindings/*/maxim,max77843.yaml
F: Documentation/devicetree/bindings/clock/maxim,max77686.txt
+F: drivers/leds/leds-max77705.c
F: drivers/*/*max77843.c
F: drivers/*/max14577*.c
F: drivers/*/max77686*.c
diff --git a/drivers/leds/Kconfig b/drivers/leds/Kconfig
index 011216aa45cf..9677414a64c9 100644
--- a/drivers/leds/Kconfig
+++ b/drivers/leds/Kconfig
@@ -766,6 +766,14 @@ config LEDS_MAX77650
help
LEDs driver for MAX77650 family of PMICs from Maxim Integrated.
+config LEDS_MAX77705
+ tristate "LED support for Maxim MAX77705 PMIC"
+ depends on MFD_MAX77705
+ depends on LEDS_CLASS
+ depends on LEDS_CLASS_MULTICOLOR
+ help
+ LED driver for MAX77705 PMIC from Maxim Integrated.
+
config LEDS_MAX8997
tristate "LED support for MAX8997 PMIC"
depends on LEDS_CLASS && MFD_MAX8997
diff --git a/drivers/leds/Makefile b/drivers/leds/Makefile
index 52bf8bf9c4ad..2e316e5e2f24 100644
--- a/drivers/leds/Makefile
+++ b/drivers/leds/Makefile
@@ -60,6 +60,7 @@ obj-$(CONFIG_LEDS_LP8860) += leds-lp8860.o
obj-$(CONFIG_LEDS_LT3593) += leds-lt3593.o
obj-$(CONFIG_LEDS_MAX5970) += leds-max5970.o
obj-$(CONFIG_LEDS_MAX77650) += leds-max77650.o
+obj-$(CONFIG_LEDS_MAX77705) += leds-max77705.o
obj-$(CONFIG_LEDS_MAX8997) += leds-max8997.o
obj-$(CONFIG_LEDS_MC13783) += leds-mc13783.o
obj-$(CONFIG_LEDS_MENF21BMC) += leds-menf21bmc.o
diff --git a/drivers/leds/leds-max77705.c b/drivers/leds/leds-max77705.c
new file mode 100644
index 000000000000..15fa4b3cbcd6
--- /dev/null
+++ b/drivers/leds/leds-max77705.c
@@ -0,0 +1,267 @@
+// SPDX-License-Identifier: GPL-2.0
+/*
+ * Based on leds-max77650 driver
+ *
+ * LED driver for MAXIM 77705 PMIC.
+ * Copyright (C) 2024 Dzmitry Sankouski <dsankouski@gmail.org>
+ */
+
+#include <linux/i2c.h>
+#include <linux/led-class-multicolor.h>
+#include <linux/leds.h>
+#include <linux/mfd/max77705-private.h>
+#include <linux/module.h>
+#include <linux/platform_device.h>
+#include <linux/regmap.h>
+
+#define MAX77705_LED_NUM_LEDS 4
+#define MAX77705_LED_EN_MASK GENMASK(1, 0)
+#define MAX77705_LED_MAX_BRIGHTNESS 0xff
+#define MAX77705_LED_EN_SHIFT(reg) (reg * MAX77705_RGBLED_EN_WIDTH)
+#define MAX77705_LED_REG_BRIGHTNESS(reg) (reg + MAX77705_RGBLED_REG_LED0BRT)
+
+struct max77705_led {
+ struct led_classdev cdev;
+ struct led_classdev_mc mcdev;
+ struct regmap *regmap;
+
+ struct mc_subled *subled_info;
+};
+
+static int max77705_rgb_blink(struct led_classdev *cdev,
+ unsigned long *delay_on,
+ unsigned long *delay_off)
+{
+ struct max77705_led *led = container_of(cdev, struct max77705_led, cdev);
+ int value, on_value, off_value;
+
+ if (*delay_on < MAX77705_RGB_DELAY_100_STEP)
+ on_value = 0;
+ else if (*delay_on < MAX77705_RGB_DELAY_100_STEP_LIM)
+ on_value = *delay_on / MAX77705_RGB_DELAY_100_STEP - 1;
+ else if (*delay_on < MAX77705_RGB_DELAY_250_STEP_LIM)
+ on_value = (*delay_on - MAX77705_RGB_DELAY_100_STEP_LIM) /
+ MAX77705_RGB_DELAY_250_STEP +
+ MAX77705_RGB_DELAY_100_STEP_COUNT;
+ else
+ on_value = 15;
+
+ on_value <<= 4;
+
+ if (*delay_off < 1)
+ off_value = 0;
+ else if (*delay_off < MAX77705_RGB_DELAY_500_STEP)
+ off_value = 1;
+ else if (*delay_off < MAX77705_RGB_DELAY_500_STEP_LIM)
+ off_value = *delay_off / MAX77705_RGB_DELAY_500_STEP;
+ else if (*delay_off < MAX77705_RGB_DELAY_1000_STEP_LIM)
+ off_value = (*delay_off - MAX77705_RGB_DELAY_1000_STEP_LIM) /
+ MAX77705_RGB_DELAY_1000_STEP +
+ MAX77705_RGB_DELAY_500_STEP_COUNT;
+ else if (*delay_off < MAX77705_RGB_DELAY_2000_STEP_LIM)
+ off_value = (*delay_off - MAX77705_RGB_DELAY_2000_STEP_LIM) /
+ MAX77705_RGB_DELAY_2000_STEP +
+ MAX77705_RGB_DELAY_1000_STEP_COUNT;
+ else
+ off_value = 15;
+
+ value = on_value | off_value;
+ return regmap_write(led->regmap, MAX77705_RGBLED_REG_LEDBLNK, value);
+}
+
+static int max77705_led_brightness_set(struct regmap *regmap, struct mc_subled *subled,
+ int num_colors)
+{
+ int ret;
+
+ for (int i = 0; i < num_colors; i++) {
+ unsigned int channel, brightness;
+
+ channel = subled[i].channel;
+ brightness = subled[i].brightness;
+
+ if (brightness == LED_OFF) {
+ /* Flash OFF */
+ ret = regmap_update_bits(regmap,
+ MAX77705_RGBLED_REG_LEDEN,
+ MAX77705_LED_EN_MASK << MAX77705_LED_EN_SHIFT(channel), 0);
+ } else {
+ /* Set current */
+ ret = regmap_write(regmap, MAX77705_LED_REG_BRIGHTNESS(channel),
+ brightness);
+ if (ret < 0)
+ return ret;
+
+ ret = regmap_update_bits(regmap,
+ MAX77705_RGBLED_REG_LEDEN,
+ LED_ON << MAX77705_LED_EN_SHIFT(channel),
+ MAX77705_LED_EN_MASK << MAX77705_LED_EN_SHIFT(channel));
+ }
+ }
+
+ return ret;
+}
+
+static int max77705_led_brightness_set_single(struct led_classdev *cdev,
+ enum led_brightness brightness)
+{
+ struct max77705_led *led = container_of(cdev, struct max77705_led, cdev);
+
+ led->subled_info->brightness = brightness;
+
+ return max77705_led_brightness_set(led->regmap, led->subled_info, 1);
+}
+
+static int max77705_led_brightness_set_multi(struct led_classdev *cdev,
+ enum led_brightness brightness)
+{
+ struct led_classdev_mc *mcdev = lcdev_to_mccdev(cdev);
+ struct max77705_led *led = container_of(mcdev, struct max77705_led, mcdev);
+
+ led_mc_calc_color_components(mcdev, brightness);
+
+ return max77705_led_brightness_set(led->regmap, led->mcdev.subled_info, mcdev->num_colors);
+}
+
+static int max77705_parse_subled(struct device *dev, struct fwnode_handle *np,
+ struct mc_subled *info)
+{
+ u32 color = LED_COLOR_ID_GREEN;
+ u32 reg;
+ int ret;
+
+ ret = fwnode_property_read_u32(np, "reg", ®);
+ if (ret || !reg || reg >= MAX77705_LED_NUM_LEDS)
+ return dev_err_probe(dev, -EINVAL, "invalid \"reg\" of %pOFn\n", np);
+
+ info->channel = reg;
+
+ ret = fwnode_property_read_u32(np, "color", &color);
+ if (ret < 0 && ret != -EINVAL)
+ return dev_err_probe(dev, ret,
+ "failed to parse \"color\" of %pOF\n", np);
+
+ info->color_index = color;
+
+ return 0;
+}
+
+static int max77705_add_led(struct device *dev, struct regmap *regmap, struct fwnode_handle *np)
+{
+ int ret, i = 0;
+ unsigned int color, reg;
+ struct max77705_led *led;
+ struct led_classdev *cdev;
+ struct mc_subled *info;
+ struct fwnode_handle *child;
+ struct led_init_data init_data = {};
+
+ led = devm_kzalloc(dev, sizeof(*led), GFP_KERNEL);
+ if (!led)
+ return -ENOMEM;
+
+ ret = fwnode_property_read_u32(np, "color", &color);
+ if (ret < 0 && ret != -EINVAL)
+ return dev_err_probe(dev, ret,
+ "failed to parse \"color\" of %pOF\n", np);
+
+ led->regmap = regmap;
+ init_data.fwnode = np;
+
+ if (color == LED_COLOR_ID_RGB) {
+ int num_channels = of_get_available_child_count(to_of_node(np));
+
+ ret = fwnode_property_read_u32(np, "reg", ®);
+ if (ret || reg >= MAX77705_LED_NUM_LEDS)
+ ret = -EINVAL;
+
+ info = devm_kcalloc(dev, num_channels, sizeof(*info), GFP_KERNEL);
+ if (!info)
+ return -ENOMEM;
+
+ cdev = &led->mcdev.led_cdev;
+ cdev->max_brightness = MAX77705_LED_MAX_BRIGHTNESS;
+ cdev->brightness_set_blocking = max77705_led_brightness_set_multi;
+ cdev->blink_set = max77705_rgb_blink;
+
+ fwnode_for_each_available_child_node(np, child) {
+ ret = max77705_parse_subled(dev, child, &info[i]);
+ if (ret < 0)
+ return ret;
+
+ info[i].intensity = 0;
+ i++;
+ }
+
+ led->mcdev.subled_info = info;
+ led->mcdev.num_colors = num_channels;
+ led->cdev = *cdev;
+
+ ret = devm_led_classdev_multicolor_register_ext(dev, &led->mcdev, &init_data);
+ if (ret)
+ return ret;
+
+ ret = max77705_led_brightness_set_multi(&led->cdev, LED_OFF);
+ if (ret)
+ return ret;
+ } else {
+ info = devm_kzalloc(dev, sizeof(*info), GFP_KERNEL);
+ if (!info)
+ return -ENOMEM;
+
+ max77705_parse_subled(dev, np, info);
+
+ led->subled_info = info;
+ led->cdev.brightness_set_blocking = max77705_led_brightness_set_single;
+ led->cdev.blink_set = max77705_rgb_blink;
+ led->cdev.max_brightness = MAX77705_LED_MAX_BRIGHTNESS;
+
+ ret = devm_led_classdev_register_ext(dev, &led->cdev, &init_data);
+ if (ret)
+ return ret;
+
+ ret = max77705_led_brightness_set_single(&led->cdev, LED_OFF);
+ if (ret)
+ return ret;
+ }
+
+ return 0;
+}
+
+static int max77705_led_probe(struct platform_device *pdev)
+{
+ struct device *dev = &pdev->dev;
+ struct regmap *regmap;
+ int ret;
+
+ regmap = dev_get_regmap(dev->parent, NULL);
+ if (!regmap)
+ return -ENODEV;
+
+ device_for_each_child_node_scoped(dev, child) {
+ ret = max77705_add_led(dev, regmap, child);
+ if (ret)
+ return ret;
+ }
+
+ return 0;
+}
+
+static const struct of_device_id max77705_led_of_match[] = {
+ { .compatible = "maxim,max77705-rgb" },
+ { }
+};
+MODULE_DEVICE_TABLE(of, max77705_led_of_match);
+
+static struct platform_driver max77705_led_driver = {
+ .driver = {
+ .name = "max77705-led",
+ .of_match_table = max77705_led_of_match,
+ },
+ .probe = max77705_led_probe,
+};
+module_platform_driver(max77705_led_driver);
+
+MODULE_DESCRIPTION("Maxim MAX77705 LED driver");
+MODULE_AUTHOR("Dzmitry Sankouski <dsankouski@gmail.com>");
+MODULE_LICENSE("GPL");
diff --git a/include/linux/mfd/max77705-private.h b/include/linux/mfd/max77705-private.h
index e4309d3b6f2a..41bc9d1188a9 100644
--- a/include/linux/mfd/max77705-private.h
+++ b/include/linux/mfd/max77705-private.h
@@ -33,6 +33,23 @@
#define MAX77705_SYSTEM_IRQ_SYSOVLO_INT BIT(5)
#define MAX77705_SYSTEM_IRQ_TSHDN_INT BIT(6)
#define MAX77705_SYSTEM_IRQ_TM_INT BIT(7)
+/* MAX77705_RGBLED_REG_LEDEN register */
+#define MAX77705_RGBLED_EN_WIDTH 2
+/* MAX77705_RGBLED_REG_LEDBLNK register */
+#define MAX77705_RGB_DELAY_100_STEP_LIM 500
+#define MAX77705_RGB_DELAY_100_STEP_COUNT 4
+#define MAX77705_RGB_DELAY_100_STEP 100
+#define MAX77705_RGB_DELAY_250_STEP_LIM 3250
+#define MAX77705_RGB_DELAY_250_STEP 250
+#define MAX77705_RGB_DELAY_500_STEP 500
+#define MAX77705_RGB_DELAY_500_STEP_COUNT 10
+#define MAX77705_RGB_DELAY_500_STEP_LIM 5000
+#define MAX77705_RGB_DELAY_1000_STEP_LIM 8000
+#define MAX77705_RGB_DELAY_1000_STEP_COUNT 13
+#define MAX77705_RGB_DELAY_1000_STEP 1000
+#define MAX77705_RGB_DELAY_2000_STEP 2000
+#define MAX77705_RGB_DELAY_2000_STEP_COUNT 13
+#define MAX77705_RGB_DELAY_2000_STEP_LIM 12000
enum max77705_hw_rev {
MAX77705_PASS1 = 1,
--
2.39.5
^ permalink raw reply related [flat|nested] 23+ messages in thread* Re: (subset) [PATCH v14 00/10] Add support for Maxim Integrated MAX77705 PMIC
2025-01-08 14:13 [PATCH v14 00/10] Add support for Maxim Integrated MAX77705 PMIC Dzmitry Sankouski
` (9 preceding siblings ...)
2025-01-08 14:13 ` [PATCH v14 10/10] leds: max77705: Add LEDs support Dzmitry Sankouski
@ 2025-01-15 22:02 ` Sebastian Reichel
10 siblings, 0 replies; 23+ messages in thread
From: Sebastian Reichel @ 2025-01-15 22:02 UTC (permalink / raw)
To: Sebastian Reichel, Chanwoo Choi, Krzysztof Kozlowski, Lee Jones,
Rob Herring, Conor Dooley, Dmitry Torokhov, Pavel Machek,
Hans de Goede, Marek Szyprowski, Sebastian Krzyszkowiak,
Purism Kernel Team, Krzysztof Kozlowski, Dzmitry Sankouski
Cc: linux-pm, linux-kernel, devicetree, linux-input, linux-leds,
Krzysztof Kozlowski
On Wed, 08 Jan 2025 17:13:44 +0300, Dzmitry Sankouski wrote:
> The Maxim MAX77705 is a Companion Power Management and Type-C
> interface IC which includes charger, fuelgauge, LED, haptic motor driver and
> Type-C management IC. It's used in Samsung S series smart phones
> starting from S9 model.
>
> Add features:
> - charger
> - fuelgauge
> - haptic
> - led
>
> [...]
Applied, thanks!
[01/10] power: supply: add undervoltage health status property
commit: df998c22321dde3f70cd3cf8c183dfd6bf64c759
[02/10] dt-bindings: power: supply: max17042: add max77705 support
commit: 4519e13aef44269b0f8b6694a7adeb13d7d66b14
[05/10] power: supply: max17042: add max77705 fuel gauge support
commit: 260d7c5e5392ac41c94152005d416172ba0a906d
Best regards,
--
Sebastian Reichel <sebastian.reichel@collabora.com>
^ permalink raw reply [flat|nested] 23+ messages in thread