[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <3d156c45-b55d-4ca4-95d6-0d06e067bbdb@kernel.org>
Date: Tue, 23 Dec 2025 14:32:24 +0100
From: Krzysztof Kozlowski <krzk@...nel.org>
To: amitsd@...gle.com, Sebastian Reichel <sre@...nel.org>,
Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>, André Draszik
<andre.draszik@...aro.org>, Lee Jones <lee@...nel.org>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
Badhri Jagan Sridharan <badhri@...gle.com>,
Heikki Krogerus <heikki.krogerus@...ux.intel.com>,
Peter Griffin <peter.griffin@...aro.org>,
Tudor Ambarus <tudor.ambarus@...aro.org>,
Alim Akhtar <alim.akhtar@...sung.com>
Cc: linux-kernel@...r.kernel.org, linux-pm@...r.kernel.org,
devicetree@...r.kernel.org, linux-usb@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, linux-samsung-soc@...r.kernel.org,
RD Babiera <rdbabiera@...gle.com>, Kyle Tso <kyletso@...gle.com>
Subject: Re: [PATCH v2 1/5] dt-bindings: mfd: maxim,max77759: add charger
child node
On 19/12/2025 09:17, Krzysztof Kozlowski wrote:
> On 18/12/2025 23:49, Amit Sunil Dhamne via B4 Relay wrote:
>> From: Amit Sunil Dhamne <amitsd@...gle.com>
>>
>> The Maxim MAX77759 MFD includes a charger function. Extend the max77759
>> binding to include the charger. Also, update the example to include
>> charger.
>>
>> Signed-off-by: Amit Sunil Dhamne <amitsd@...gle.com>
>> ---
>> .../devicetree/bindings/mfd/maxim,max77759.yaml | 33 ++++++++++++++++++++++
>> 1 file changed, 33 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/mfd/maxim,max77759.yaml b/Documentation/devicetree/bindings/mfd/maxim,max77759.yaml
>> index 525de9ab3c2b..1cffdf2e5776 100644
>> --- a/Documentation/devicetree/bindings/mfd/maxim,max77759.yaml
>> +++ b/Documentation/devicetree/bindings/mfd/maxim,max77759.yaml
>> @@ -37,6 +37,30 @@ properties:
>> nvmem-0:
>> $ref: /schemas/nvmem/maxim,max77759-nvmem.yaml
>>
>> + charger:
>> + type: object
>> + description: This is a dual input switch mode battery charger for portable
>> + applications. It supports wired and wireless charging and can operate in
>> + buck and boost mode.
>> +
>
>
> I do not see any improvements, so same comment: this should be folded
> into the parent.
>
> Please read DTS 101 slides or writing bindings or any other talks...
>
No responses to my emails for a few days, so I assume discussion is done
and I mark it as changes requested in patchwork.
Best regards,
Krzysztof
Powered by blists - more mailing lists