[PATCH v9 1/2] dt-bindings: trivial: Add tps53685 support

[PATCH v9 1/2] dt-bindings: trivial: Add tps53685 support

[Chiang Brian]

Add device type support for tps53685

Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Signed-off-by: Chiang Brian <chiang.brian@inventec.com>
---
v8 -> v9:
- No code changed, correct the order of Acked-by tag
- Link to v8: https://lore.kernel.org/all/20250602042454.184643-2-chiang.brian@inventec.com/

v3 -> v8:
- No code changed, included Acked-by tag from v2 review
- Patch kept in sync with series version
- Link to v3: https://lore.kernel.org/all/20250515081449.1433772-2-chiang.brian@inventec.com/

v2 -> v3:
- Fix the order of patches
- Link to v2: https://lore.kernel.org/all/20250424132538.2004510-3-chiang.brian@inventec.corp-partner.google.com/

v1 -> v2:
- Correct the subject and commit message
- Link to v1: https://lore.kernel.org/all/20250314032802.3187097-1-chiang.brian@inventec.com/


 Documentation/devicetree/bindings/trivial-devices.yaml | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Documentation/devicetree/bindings/trivial-devices.yaml b/Documentation/devicetree/bindings/trivial-devices.yaml
index 27930708ccd5..a613b9c3f6ea 100644
--- a/Documentation/devicetree/bindings/trivial-devices.yaml
+++ b/Documentation/devicetree/bindings/trivial-devices.yaml
@@ -396,6 +396,8 @@ properties:
           - ti,tps53679
             # TI Dual channel DCAP+ multiphase controller TPS53681
           - ti,tps53681
+            # TI Dual channel DCAP+ multiphase controller TPS53685 with AMD-SVI3
+          - ti,tps53685
             # TI Dual channel DCAP+ multiphase controller TPS53688
           - ti,tps53688
             # TI DC-DC converters on PMBus
-- 
2.25.1

Re: [PATCH v9 1/2] dt-bindings: trivial: Add tps53685 support

[Krzysztof Kozlowski]

On 10/06/2025 12:25, Chiang Brian wrote:
> Add device type support for tps53685
> 
> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> Signed-off-by: Chiang Brian <chiang.brian@inventec.com>
> ---
> v8 -> v9:
> - No code changed, correct the order of Acked-by tag
> - Link to v8: https://lore.kernel.org/all/20250602042454.184643-2-chiang.brian@inventec.com/
Stop sending this to me 6 or more times. Every version you send multiple
times, that's way too much.

Best regards,
Krzysztof

Re: [PATCH v9 1/2] dt-bindings: trivial: Add tps53685 support

[Chiang Brian]

> On 10/06/2025 12:41, Krzysztof Kozlowski wrote:
>
> On 10/06/2025 12:25, Chiang Brian wrote:
> > Add device type support for tps53685
> > 
> > Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> > Signed-off-by: Chiang Brian <chiang.brian@inventec.com>
> > ---
> > v8 -> v9:
> > - No code changed, correct the order of Acked-by tag
> > - Link to v8: https://lore.kernel.org/all/20250602042454.184643-2-chiang.brian@inventec.com/
> Stop sending this to me 6 or more times. Every version you send multiple
> times, that's way too much.

But how do I avoid sending to you even though I need to send this patch 
series?
I apologize for the spamming due to familiar with the workflow.

Best Regards,
Brian Chiang

Re: [PATCH v9 1/2] dt-bindings: trivial: Add tps53685 support

[Rob Herring]

On Tue, Jun 10, 2025 at 12:41:15PM +0200, Krzysztof Kozlowski wrote:
> On 10/06/2025 12:25, Chiang Brian wrote:
> > Add device type support for tps53685
> > 
> > Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> > Signed-off-by: Chiang Brian <chiang.brian@inventec.com>
> > ---
> > v8 -> v9:
> > - No code changed, correct the order of Acked-by tag
> > - Link to v8: https://lore.kernel.org/all/20250602042454.184643-2-chiang.brian@inventec.com/
> Stop sending this to me 6 or more times. Every version you send multiple
> times, that's way too much.

I only have v1, 2, 3, 8 and 9...

Rob

Re: [PATCH v9 1/2] dt-bindings: trivial: Add tps53685 support

[Chiang Brian]

> On Tue, Jun 10, 2025 at 13:25:29 -0500, Rob Herring wrote:
>  
> On Tue, Jun 10, 2025 at 12:41:15PM +0200, Krzysztof Kozlowski wrote:
> > On 10/06/2025 12:25, Chiang Brian wrote:
> > > Add device type support for tps53685
> > > 
> > > Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> > > Signed-off-by: Chiang Brian <chiang.brian@inventec.com>
> > > ---
> > > v8 -> v9:
> > > - No code changed, correct the order of Acked-by tag
> > > - Link to v8: https://lore.kernel.org/all/20250602042454.184643-2-chiang.brian@inventec.com/
> > Stop sending this to me 6 or more times. Every version you send multiple
> > times, that's way too much.
> 
> I only have v1, 2, 3, 8 and 9...

Thank you for clarifying. You're correct - there have only been 5 
dt-bindings patch versions total.

The version numbering jumped because I initially submitted only hwmon 
patches (v1-v4) and was later advised to include dt-bindings patches. 
Starting from v5, I included both patches but didn't align the version 
numbers until v8.

I understand this may have caused some confusion with the numbering 
sequence, and I'll ensure better coordination of patch series from the 
beginning in future submissions.

The following patch history may clarify more clearly.

v9 patch series:
- v9 dt-bindings: https://lore.kernel.org/all/20250610102556.236300-2-chiang.brian@inventec.com/
- v9 hwmon: https://lore.kernel.org/all/20250610102556.236300-3-chiang.brian@inventec.com/

v8 patch series:
- v8 dt-bindings: https://lore.kernel.org/all/20250602042454.184643-2-chiang.brian@inventec.com/
- v8 hwmon: https://lore.kernel.org/all/20250602042454.184643-3-chiang.brian@inventec.com/

v7 patch series:
- v3 dt-bindings: https://lore.kernel.org/all/20250515081449.1433772-2-chiang.brian@inventec.com/
  (Next version number has been modified to v8 to align to the patch series)
- v7 hwmon: https://lore.kernel.org/all/20250515081449.1433772-3-chiang.brian@inventec.com/

v6 patch series:
- v2 dt-bindings: https://lore.kernel.org/all/20250424132538.2004510-3-chiang.brian@inventec.corp-partner.google.com/
- v6 hwmon: https://lore.kernel.org/all/20250424132538.2004510-2-chiang.brian@inventec.corp-partner.google.com/

v5 patch series:
- v1 dt-bindings: https://lore.kernel.org/all/20250314032802.3187097-1-chiang.brian@inventec.com/
  (Start to upload dt-bindings patch after being pointed out)
- v5 hwmon: https://lore.kernel.org/all/20250314033040.3190642-1-chiang.brian@inventec.com/

v4 patch series:
- No dt-bindings
- v4 hwmon: https://lore.kernel.org/all/CAJCfHmW61d2jd_tYpNEqBG_Z58bEnVKAmsvhrEP1zXQoXqrUVw@mail.gmail.com/

v3 patch series:
- No dt-bindings
- v3 hwmon: https://lore.kernel.org/all/CAJCfHmVyaDPh0_ThPjhBP0zMO1oE1AR=4=Zsa0cMPXU3J4v6dw@mail.gmail.com/

v2 patch series:
- No dt-bindings
- v2 hwmon: https://lore.kernel.org/all/CAJCfHmUteFM+nUZWBWvmwFjALg1QUL5r+=syU1HmYTL1ewQWqA@mail.gmail.com/

v1 patch series:
- No dt-bindings
- v1 hwmon: https://lore.kernel.org/all/CAJCfHmVy3O4-nz2_PKF7TcXYr+HqTte1-bdUWLBmV7JOS7He1g@mail.gmail.com/

Best Regards,
Brian Chiang

Re: [PATCH v9 1/2] dt-bindings: trivial: Add tps53685 support

[Krzysztof Kozlowski]

On 10/06/2025 12:41, Chiang Brian wrote:
>> On 10/06/2025 12:41, Krzysztof Kozlowski wrote:
>>
>> On 10/06/2025 12:25, Chiang Brian wrote:
>>> Add device type support for tps53685
>>>
>>> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>>> Signed-off-by: Chiang Brian <chiang.brian@inventec.com>
>>> ---
>>> v8 -> v9:
>>> - No code changed, correct the order of Acked-by tag
>>> - Link to v8: https://lore.kernel.org/all/20250602042454.184643-2-chiang.brian@inventec.com/
>> Stop sending this to me 6 or more times. Every version you send multiple
>> times, that's way too much.
> 
> But how do I avoid sending to you even though I need to send this patch 
> series?
> I apologize for the spamming due to familiar with the workflow.

You sent three times previous version to me ONLY, for testing or
whatever other process. Now you did the same.

How to avoid it? Well, you type things into the keyboard, so type things
which will do not perform above action. E.g. when executing git
send-email it shows you recipients and then ABORT and correct the git
send-email cc-list so it won't add CC based on tags (see manual).

Best regards,
Krzysztof

Re: [PATCH v9 2/2] hwmon: (pmbus/tps53679) Add support for TPS53685

[Brian Chiang]

On Wed, 11 Jun 2025 08:19:46 +0200, Krzysztof Kozlowski wrote:
>On 10/06/2025 12:41, Chiang Brian wrote:
>>> On 10/06/2025 12:41, Krzysztof Kozlowski wrote:
>>>
>>> On 10/06/2025 12:25, Chiang Brian wrote:
>>>> Add device type support for tps53685
>>>>
>>>> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>>>> Signed-off-by: Chiang Brian <chiang.brian@inventec.com>
>>>> ---
>>>> v8 -> v9:
>>>> - No code changed, correct the order of Acked-by tag
>>>> - Link to v8: https://lore.kernel.org/all/20250602042454.184643-2-chiang.brian@inventec.com/
>>> Stop sending this to me 6 or more times. Every version you send multiple
>>> times, that's way too much.
>> 
>> But how do I avoid sending to you even though I need to send this patch 
>> series?
>> I apologize for the spamming due to familiar with the workflow.
>
>You sent three times previous version to me ONLY, for testing or
>whatever other process. Now you did the same.
>
>How to avoid it? Well, you type things into the keyboard, so type things
>which will do not perform above action. E.g. when executing git
>send-email it shows you recipients and then ABORT and correct the git
>send-email cc-list so it won't add CC based on tags (see manual).

Thank you for the explanation. I understand the issue now - git send-email 
was automatically adding you to CC because of the "Acked-by" tag in the 
commit message, in addition to the normal maintainer CC from get_maintainer.pl.

For future patch submissions, I'll use --suppress-cc=misc-by to prevent 
automatic CC generation from commit message tags like "Acked-by", while 
still maintaining proper maintainer CCs through get_maintainer.pl.

I apologize for the duplicate emails.

Best Regards,
Brian Chiang