On Tue, 22 Dec 2020 09:55:22 +, Lee Jones wrote:
> On Tue, 22 Dec 2020, Timon Baetz wrote:
>
> > Add of_compatible ("maxim,max8997-muic") to the mfd_cell to have a
> > of_node set in the extcon driver.
> >
> > Add of_compatible ("maxim,max8997-b
On Sat, 30 Jan 2021 17:29:31 +, Timon Baetz wrote:
> The regulator is used for charging control by max8997_charger driver.
>
> Signed-off-by: Timon Baetz
> ---
> arch/arm/boot/dts/exynos4210-i9100.dts | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/arch
Based on the discussion from [0] add an optional DT property to retrieve
the regulator used for charging control in the max8997_charger driver.
[0] https://lore.kernel.org/lkml/20210118124505.gg4...@sirena.org.uk/
Timon Baetz (3):
regulator: dt-bindings: Document charger-supply for max8997
Add charger-supply optional property to enable charging control.
Signed-off-by: Timon Baetz
---
.../devicetree/bindings/regulator/max8997-regulator.txt | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/regulator/max8997-regulator.txt
b/Documentation
The regulator is used for charging control by max8997_charger driver.
Signed-off-by: Timon Baetz
---
arch/arm/boot/dts/exynos4210-i9100.dts | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm/boot/dts/exynos4210-i9100.dts
b/arch/arm/boot/dts/exynos4210-i9100.dts
index 304a8ee2364c
Get regulator from parent device's node and extcon by name.
Signed-off-by: Timon Baetz
---
drivers/power/supply/max8997_charger.c | 12
1 file changed, 8 insertions(+), 4 deletions(-)
diff --git a/drivers/power/supply/max8997_charger.c
b/drivers/power/supply/max8997_char
On Sun, 31 Jan 2021 18:28:40 +0100, Krzysztof Kozlowski wrote:
> On Sat, Jan 30, 2021 at 05:30:14PM +0000, Timon Baetz wrote:
> > Get regulator from parent device's node and extcon by name.
> >
> > Signed-off-by: Timon Baetz
> > ---
> > drive
muic node is only used for extcon consumers.
charger node is used to specify muic and regulator.
Signed-off-by: Timon Baetz
---
arch/arm/boot/dts/exynos4210-i9100.dts | 10 ++
1 file changed, 10 insertions(+)
diff --git a/arch/arm/boot/dts/exynos4210-i9100.dts
b/arch/arm/boot/dts
: Timon Baetz
---
arch/arm/boot/dts/exynos4210-i9100.dts | 21 ++---
1 file changed, 14 insertions(+), 7 deletions(-)
diff --git a/arch/arm/boot/dts/exynos4210-i9100.dts
b/arch/arm/boot/dts/exynos4210-i9100.dts
index 586d801af0b5..fec6da64f7c1 100644
--- a/arch/arm/boot/dts
Add maxim,max8997-battery and maxim,max8997-muic optional nodes.
Signed-off-by: Timon Baetz
---
.../bindings/regulator/max8997-regulator.txt | 11 +++
1 file changed, 11 insertions(+)
diff --git a/Documentation/devicetree/bindings/regulator/max8997-regulator.txt
b
This allows the MAX8997 charger to set the current limit depending on
the detected extcon charger type.
Signed-off-by: Timon Baetz
---
drivers/extcon/extcon-max8997.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/extcon/extcon-max8997.c b/drivers/extcon/extcon-max8997.c
index
/drivers/power/max8997_charger_u1.c#L169-L170
[1]
https://github.com/krzk/linux-vendor-backup/blob/samsung/galaxy-s2-epic-4g-touch-sph-d710-exynos4210-dump/drivers/power/max8997_charger_u1.c#L390-L391
Signed-off-by: Timon Baetz
---
arch/arm/boot/dts/exynos4210-i9100.dts | 9 -
1 file changed
-vendor-backup/blob/samsung/galaxy-s2-epic-4g-touch-sph-d710-exynos4210-dump/drivers/misc/max8997-muic.c#L1675-L1678
Signed-off-by: Timon Baetz
---
drivers/mfd/max8997.c | 4 +-
drivers/power/supply/max8997_charger.c | 94 ++
2 files changed, 96
On Mon, 21 Dec 2020 15:16:27 +0100, Krzysztof Kozlowski wrote:
> On Mon, Dec 21, 2020 at 09:53:15AM +0000, Timon Baetz wrote:
> > Register for extcon notification and set charging current depending on
> > the detected cable type. Current values are taken from vendor kernel,
> &g
Add of_compatible ("maxim,max8997-muic") to the mfd_cell to have a
of_node set in the extcon driver.
Add of_compatible ("maxim,max8997-battery") to the mfd_cell to configure
the charger driver.
Signed-off-by: Timon Baetz
---
drivers/mfd/max8997.c | 4 ++--
1 file changed
-vendor-backup/blob/samsung/galaxy-s2-epic-4g-touch-sph-d710-exynos4210-dump/drivers/misc/max8997-muic.c#L1675-L1678
Signed-off-by: Timon Baetz
---
drivers/power/supply/max8997_charger.c | 94 ++
1 file changed, 94 insertions(+)
diff --git a/drivers/power/supply
Add maxim,max8997-battery and maxim,max8997-muic optional nodes.
Signed-off-by: Timon Baetz
---
.../bindings/regulator/max8997-regulator.txt | 11 +++
1 file changed, 11 insertions(+)
diff --git a/Documentation/devicetree/bindings/regulator/max8997-regulator.txt
b
This allows the MAX8997 charger to set the current limit depending on
the detected extcon charger type.
Signed-off-by: Timon Baetz
---
drivers/extcon/extcon-max8997.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/extcon/extcon-max8997.c b/drivers/extcon/extcon-max8997.c
index
muic node is only used for extcon consumers.
charger node is used to specify muic and regulator.
Signed-off-by: Timon Baetz
---
arch/arm/boot/dts/exynos4210-i9100.dts | 10 ++
1 file changed, 10 insertions(+)
diff --git a/arch/arm/boot/dts/exynos4210-i9100.dts
b/arch/arm/boot/dts
-touch-sph-d710-exynos4210-dump/drivers/power/max8997_charger_u1.c#L169-L170
[1]
https://github.com/krzk/linux-vendor-backup/blob/samsung/galaxy-s2-epic-4g-touch-sph-d710-exynos4210-dump/drivers/power/max8997_charger_u1.c#L390-L391
Signed-off-by: Timon Baetz
---
arch/arm/boot/dts/exynos4210-i9100
/sec_battery_u1.c#L1525
Signed-off-by: Timon Baetz
---
arch/arm/boot/dts/exynos4210-i9100.dts | 21 ++---
1 file changed, 14 insertions(+), 7 deletions(-)
diff --git a/arch/arm/boot/dts/exynos4210-i9100.dts
b/arch/arm/boot/dts/exynos4210-i9100.dts
index 586d801af0b5..e702adb69670 100644
On Tue, 22 Dec 2020 09:40:04 +0100, Krzysztof Kozlowski wrote:
> On Tue, Dec 22, 2020 at 07:31:40AM +0000, Timon Baetz wrote:
> > Register for extcon notification and set charging current depending on
> > the detected cable type. Current values are taken from vendor kernel,
> &g
This allows the MAX8997 charger to set the current limit depending on
the detected extcon charger type.
Signed-off-by: Timon Baetz
---
drivers/extcon/extcon-max8997.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/extcon/extcon-max8997.c b/drivers/extcon/extcon-max8997.c
index
Add maxim,max8997-battery and maxim,max8997-muic optional nodes.
Signed-off-by: Timon Baetz
---
.../bindings/regulator/max8997-regulator.txt | 12
1 file changed, 12 insertions(+)
diff --git a/Documentation/devicetree/bindings/regulator/max8997-regulator.txt
b
Add of_compatible ("maxim,max8997-muic") to the mfd_cell to have a
of_node set in the extcon driver.
Add of_compatible ("maxim,max8997-battery") to the mfd_cell to configure
the charger driver.
Signed-off-by: Timon Baetz
---
drivers/mfd/max8997.c | 4 ++--
1 file changed
-vendor-backup/blob/samsung/galaxy-s2-epic-4g-touch-sph-d710-exynos4210-dump/drivers/misc/max8997-muic.c#L1675-L1678
Signed-off-by: Timon Baetz
---
drivers/power/supply/max8997_charger.c | 89 ++
1 file changed, 89 insertions(+)
diff --git a/drivers/power/supply
muic node is only used for extcon consumers.
charger node is used to specify muic and regulator.
Signed-off-by: Timon Baetz
---
arch/arm/boot/dts/exynos4210-i9100.dts | 10 ++
1 file changed, 10 insertions(+)
diff --git a/arch/arm/boot/dts/exynos4210-i9100.dts
b/arch/arm/boot/dts
-touch-sph-d710-exynos4210-dump/drivers/power/max8997_charger_u1.c#L169-L170
[1]
https://github.com/krzk/linux-vendor-backup/blob/samsung/galaxy-s2-epic-4g-touch-sph-d710-exynos4210-dump/drivers/power/max8997_charger_u1.c#L390-L391
Signed-off-by: Timon Baetz
---
arch/arm/boot/dts/exynos4210-i9100
/sec_battery_u1.c#L1525
Signed-off-by: Timon Baetz
---
arch/arm/boot/dts/exynos4210-i9100.dts | 21 ++---
1 file changed, 14 insertions(+), 7 deletions(-)
diff --git a/arch/arm/boot/dts/exynos4210-i9100.dts
b/arch/arm/boot/dts/exynos4210-i9100.dts
index 586d801af0b5..e702adb69670 100644
On Wed, 6 Jan 2021 14:59:31 +, Mark Brown wrote:
> On Tue, Jan 05, 2021 at 05:55:29PM +0100, Krzysztof Kozlowski wrote:
> > On Mon, Jan 04, 2021 at 09:24:49PM +, Mark Brown wrote:
>
> > > I'm not sure I follow, sorry? Either the core driver can parse the
> > > bindings enough to know wh
On Fri, 8 Jan 2021 16:16:53 +, Mark Brown wrote:
> On Fri, Jan 08, 2021 at 03:16:48PM +0000, Timon Baetz wrote:
>
> > Muic needs a node to be used with extcon_get_edev_by_phandle().
> > Charger needs a node to reference a regulator.
>
> The pattern is to use the p
muic node is only used for extcon consumers.
charger node is used to point to muic and regulator.
Signed-off-by: Timon Baetz
---
v6: No change.
v5: No change.
v4: No change.
v3: Reorder patch, no change.
v2: Add patch.
arch/arm/boot/dts/exynos4210-i9100.dts | 10 ++
1 file changed, 10
-vendor-backup/blob/samsung/galaxy-s2-epic-4g-touch-sph-d710-exynos4210-dump/drivers/misc/max8997-muic.c#L1675-L1678
Signed-off-by: Timon Baetz
---
v6: dev_info() instead of dev_err().
v5: Use devm_regulator_get_optional(), dev_err() on failure.
dev_err() on extcon_get_edev_by_phandle
Both nodes are disabled as there is no battery and pins are not
connected.
Signed-off-by: Timon Baetz
---
v6: No change.
v5: Add patch.
arch/arm/boot/dts/exynos4210-origen.dts | 10 ++
1 file changed, 10 insertions(+)
diff --git a/arch/arm/boot/dts/exynos4210-origen.dts
b/arch/arm
muic node is only used for extcon consumers.
charger node is used to point to muic.
Note: charging control is not working as we don't have a charger-supply.
Signed-off-by: Timon Baetz
---
v6: No change.
v5: Add patch.
arch/arm/boot/dts/exynos4210-trats.dts | 9 +
1 file chang
-touch-sph-d710-exynos4210-dump/drivers/power/max8997_charger_u1.c#L169-L170
[1]
https://github.com/krzk/linux-vendor-backup/blob/samsung/galaxy-s2-epic-4g-touch-sph-d710-exynos4210-dump/drivers/power/max8997_charger_u1.c#L390-L391
Signed-off-by: Timon Baetz
---
v6: No change.
v5: No change.
v4: No
/sec_battery_u1.c#L1525
Signed-off-by: Timon Baetz
---
v6: No change.
v5: No change.
v4: No change.
v3: Remove label.
Fix commit message.
v2: Add patch.
arch/arm/boot/dts/exynos4210-i9100.dts | 21 ++---
1 file changed, 14 insertions(+), 7 deletions(-)
diff --git a/arch/arm/boot/dts
This allows the MAX8997 charger to set the current limit depending on
the detected extcon charger type.
Signed-off-by: Timon Baetz
---
v6: No change.
v5: No change.
v4: No change.
v3: No change.
v2: Remove empty line.
drivers/extcon/extcon-max8997.c | 4
1 file changed, 4 insertions
Add maxim,max8997-battery and maxim,max8997-muic optional nodes.
Signed-off-by: Timon Baetz
Acked-by: Krzysztof Kozlowski
---
v6: No change.
v5: No change.
v4: Make extcon and charger-supply optional.
v3: Reorder patch, no change.
v2: Add patch.
.../bindings/regulator/max8997-regulator.txt
t; [cannot apply to robh/for-next]
> [If your patch is applied to the wrong git tree, kindly drop us a note.
> And when submitting patch, we suggest to use '--base' as documented in
> https://git-scm.com/docs/git-format-patch]
>
> url:
> https://github.com/0day-ci
On Wed, 23 Dec 2020 15:32:07 +, Lee Jones wrote:
> On Wed, 23 Dec 2020, Timon Baetz wrote:
>
> > Add of_compatible ("maxim,max8997-muic") to the mfd_cell to have a
> > of_node set in the extcon driver.
> >
> > Add of_compatible ("maxim,max8997-b
On Thu, 24 Dec 2020 10:55:59 +0100, Krzysztof Kozlowski wrote:
> On Wed, Dec 23, 2020 at 01:43:05PM +0000, Timon Baetz wrote:
> > Register for extcon notification and set charging current depending on
> > the detected cable type. Current values are taken from vendor kernel,
> &g
On Thu, 24 Dec 2020 15:00:38 +0100, Krzysztof Kozlowski wrote:
> On Thu, Dec 24, 2020 at 02:37:06PM +0100, Krzysztof Kozlowski wrote:
> > On Thu, Dec 24, 2020 at 01:13:02PM +0000, Timon Baetz wrote:
> > > On Thu, 24 Dec 2020 10:55:59 +0100, Krzysztof Kozlowski wrote:
&
This allows the MAX8997 charger to set the current limit depending on
the detected extcon charger type.
Signed-off-by: Timon Baetz
---
drivers/extcon/extcon-max8997.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/extcon/extcon-max8997.c b/drivers/extcon/extcon-max8997.c
index
Add maxim,max8997-battery and maxim,max8997-muic optional nodes.
Signed-off-by: Timon Baetz
Acked-by: Krzysztof Kozlowski
---
.../bindings/regulator/max8997-regulator.txt | 12
1 file changed, 12 insertions(+)
diff --git a/Documentation/devicetree/bindings/regulator
-vendor-backup/blob/samsung/galaxy-s2-epic-4g-touch-sph-d710-exynos4210-dump/drivers/misc/max8997-muic.c#L1675-L1678
Signed-off-by: Timon Baetz
---
drivers/power/supply/max8997_charger.c | 96 ++
1 file changed, 96 insertions(+)
diff --git a/drivers/power/supply
muic node is only used for extcon consumers.
charger node is used to point to muic and regulator.
Signed-off-by: Timon Baetz
---
arch/arm/boot/dts/exynos4210-i9100.dts | 10 ++
1 file changed, 10 insertions(+)
diff --git a/arch/arm/boot/dts/exynos4210-i9100.dts
b/arch/arm/boot/dts
muic node is only used for extcon consumers.
charger node is used to point to muic.
Note: charging control is not working as we don't have a charger-supply.
Signed-off-by: Timon Baetz
---
arch/arm/boot/dts/exynos4210-trats.dts | 9 +
1 file changed, 9 insertions(+)
diff --git a
/sec_battery_u1.c#L1525
Signed-off-by: Timon Baetz
---
arch/arm/boot/dts/exynos4210-i9100.dts | 21 ++---
1 file changed, 14 insertions(+), 7 deletions(-)
diff --git a/arch/arm/boot/dts/exynos4210-i9100.dts
b/arch/arm/boot/dts/exynos4210-i9100.dts
index 586d801af0b5..e702adb69670 100644
-touch-sph-d710-exynos4210-dump/drivers/power/max8997_charger_u1.c#L169-L170
[1]
https://github.com/krzk/linux-vendor-backup/blob/samsung/galaxy-s2-epic-4g-touch-sph-d710-exynos4210-dump/drivers/power/max8997_charger_u1.c#L390-L391
Signed-off-by: Timon Baetz
---
arch/arm/boot/dts/exynos4210-i9100
Both nodes are disabled as there is no battery and pins are not
connected.
Signed-off-by: Timon Baetz
---
arch/arm/boot/dts/exynos4210-origen.dts | 10 ++
1 file changed, 10 insertions(+)
diff --git a/arch/arm/boot/dts/exynos4210-origen.dts
b/arch/arm/boot/dts/exynos4210-origen.dts
Based on the discussion from [0] add an optional DT property to retrieve
the regulator used for charging control in the max8997_charger driver.
[0] https://lore.kernel.org/lkml/20210118124505.gg4...@sirena.org.uk/
Changes in v2:
* drop accepted patches
Timon Baetz (1):
power: supply
Get regulator from parent device's node and extcon by name.
Signed-off-by: Timon Baetz
Reviewed-by: Krzysztof Kozlowski
---
drivers/power/supply/max8997_charger.c | 12
1 file changed, 8 insertions(+), 4 deletions(-)
diff --git a/drivers/power/supply/max8997_charger.c
b/dr
On Sat, 30 Jan 2021 17:30:14 +, Timon Baetz wrote:
> Get regulator from parent device's node and extcon by name.
>
> Signed-off-by: Timon Baetz
> ---
> drivers/power/supply/max8997_charger.c | 12
> 1 file changed, 8 insertions(+), 4 deletions(-)
>
&g
Add node for Samsung S5K5BAF CMOS image sensor and
enable the associated MIPI CSI-2 receiver node.
Signed-off-by: Timon Baetz
---
arch/arm/boot/dts/exynos4210-i9100.dts | 52 ++
1 file changed, 52 insertions(+)
diff --git a/arch/arm/boot/dts/exynos4210-i9100.dts
b/arch
Add node for Samsung S5K5BAF CMOS image sensor and
enable the associated MIPI CSI-2 receiver node.
Signed-off-by: Timon Baetz
---
v2: Rename node from s5k5bafx to image-sensor and use GPIO_ACTIVE_LOW
arch/arm/boot/dts/exynos4210-i9100.dts | 52 ++
1 file changed, 52
On Wed, 27 Jan 2021 07:32:42 +, Timon Baetz wrote:
> The max8997 header is using "max8998" in some identifiers.
> Fix it by replacing 8998 with 8997 in enum and macro.
>
> Signed-off-by: Timon Baetz
> ---
> v2: Fix commit message.
>
> include/linux/mf
ned-off-by: Timon Baetz
---
drivers/mfd/max8997.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/mfd/max8997.c b/drivers/mfd/max8997.c
index 55d3a6f97783..68d8f2b95287 100644
--- a/drivers/mfd/max8997.c
+++ b/drivers/mfd/max8997.c
@@ -29,9 +29,9 @@
static co
Commit 21b01cc879cc ("power: supply: max17042_battery: Add support for
the TTE_NOW prop") added a new property (TIME_TO_EMPTY_NOW) at end of
max17042_battery_props.
Reorder max17042_battery_props to properly ignore CURRENT_NOW and
CURRENT_AVG when current sense is disabled.
Signed-off
On Wed, 2 Dec 2020 23:50:57 +0200, Krzysztof Kozlowski wrote:
> On Wed, Dec 02, 2020 at 09:07:19PM +0000, Timon Baetz wrote:
> > Register for extcon notification and set charging current depending on
> > the detected cable type. Current values are taken from i9100 kernel
> >
Allows the MAX8997 charger to set the current limit depending on
the detected extcon charger type.
Signed-off-by: Timon Baetz
---
drivers/extcon/extcon-max8997.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/drivers/extcon/extcon-max8997.c b/drivers/extcon/extcon-max8997.c
index
Set CHARGER current and CHARGER_CV voltage according to Galaxy S2 kernel
fork.
Signed-off-by: Timon Baetz
---
arch/arm/boot/dts/exynos4210-i9100.dts | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/arch/arm/boot/dts/exynos4210-i9100.dts
b/arch/arm/boot/dts/exynos4210
Register for extcon notification and set charging current depending on
the detected cable type. Current values are taken from i9100 kernel
fork.
Enable and disable the CHARGER regulator based on extcon events and
remove regulator-always-on from the device tree.
Signed-off-by: Timon Baetz
On Mon, 1 Feb 2021 19:03:35 +0100, Krzysztof Kozlowski wrote:
> On Mon, Feb 01, 2021 at 09:26:42AM +0000, Timon Baetz wrote:
> > On Sun, 31 Jan 2021 18:28:40 +0100, Krzysztof Kozlowski wrote:
> > > On Sat, Jan 30, 2021 at 05:30:14PM +, Timon Baetz wrote:
> > >
The max8997 header is using "max8998" in some identifiers.
Fix it by replacing 8998 with 8997 in enum and macro.
Signed-off-by: Timon Baetz
---
v2: Fix commit message.
include/linux/mfd/max8997.h | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/include
On Fri, 15 Jan 2021 13:42:14 +, Mark Brown wrote:
> On Fri, Jan 15, 2021 at 06:19:28AM +0000, Timon Baetz wrote:
> > On Fri, 8 Jan 2021 16:16:53 +, Mark Brown wrote:
> > > On Fri, Jan 08, 2021 at 03:16:48PM +, Timon Baetz wrote:
>
> > > >
Replace occurrences of 8998 with 8997.
Signed-off-by: Timon Baetz
---
include/linux/mfd/max8997.h | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/include/linux/mfd/max8997.h b/include/linux/mfd/max8997.h
index e955e2f0a2cc..6c98edcf4b0b 100644
--- a/include/linux/mfd
On Mon, 25 Jan 2021 08:43:47 +, Lee Jones wrote:
> On Sun, 24 Jan 2021, Timon Baetz wrote:
>
> > Replace occurrences of 8998 with 8997.
>
> Because ...
... it is the max8997 header. Do you want me to extend the commit
message?
Thanks,
Timon
Use pdata field of max8998_dev struct to obtain platform data.
Signed-off-by: Timon Baetz
---
drivers/power/supply/max8997_charger.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/power/supply/max8997_charger.c
b/drivers/power/supply/max8997_charger.c
index
Replace register and unregister function calls with
module_platform_driver().
Signed-off-by: Timon Baetz
---
drivers/power/supply/max8997_charger.c | 13 +
1 file changed, 1 insertion(+), 12 deletions(-)
diff --git a/drivers/power/supply/max8997_charger.c
b/drivers/power/supply
Detect charging and discharging state. Ported from downstream Samsung
Galaxy S2 (i9100) kernel fork.
Signed-off-by: Timon Baetz
---
drivers/power/supply/max8997_charger.c | 52 +-
1 file changed, 35 insertions(+), 17 deletions(-)
diff --git a/drivers/power/supply
71 matches
Mail list logo
