[PATCH linux dev-6.11 v3 2/2] dt-bindings: Add SY24655 to ina2xx devicetree bindings

[PATCH linux dev-6.11 v3 2/2] dt-bindings: Add SY24655 to ina2xx devicetree bindings

[Wenliang]

From: book <book@100ask.localdomain>

SY24655 is similar to INA226. Its supply voltage and pin definitions
are therefore the same. Compared to INA226, SY24655 has two additional
registers for configuring and calculating average power.

Signed-off-by: book <book@100ask.localdomain>
Acked-by: Conor Dooley <conor.dooley@microchip.com>
---
 Documentation/devicetree/bindings/hwmon/ti,ina2xx.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/hwmon/ti,ina2xx.yaml b/Documentation/devicetree/bindings/hwmon/ti,ina2xx.yaml
index 6ae961732e6b..05a9cb36cd82 100644
--- a/Documentation/devicetree/bindings/hwmon/ti,ina2xx.yaml
+++ b/Documentation/devicetree/bindings/hwmon/ti,ina2xx.yaml
@@ -20,6 +20,7 @@ description: |
 properties:
   compatible:
     enum:
+      - silergy,sy24655
       - ti,ina209
       - ti,ina219
       - ti,ina220
-- 
2.47.0.229.g8f8d6eee53

Re: [PATCH linux dev-6.11 v3 2/2] dt-bindings: Add SY24655 to ina2xx devicetree bindings

[Krzysztof Kozlowski]

On 03/11/2024 17:39, Wenliang wrote:
> From: book <book@100ask.localdomain>
> 
> SY24655 is similar to INA226. Its supply voltage and pin definitions
> are therefore the same. Compared to INA226, SY24655 has two additional
> registers for configuring and calculating average power.
> 
> Signed-off-by: book <book@100ask.localdomain>
> Acked-by: Conor Dooley <conor.dooley@microchip.com>

NAK, this never happened. If you think otherwise: provide proof, please.

Nothing improved in this binding, actually it got even worse with fake
email and probably name as well.

Best regards,
Krzysztof

Re: [PATCH linux dev-6.11 v3 2/2] dt-bindings: Add SY24655 to ina2xx devicetree bindings

[Krzysztof Kozlowski]

On 03/11/2024 17:46, Krzysztof Kozlowski wrote:
> On 03/11/2024 17:39, Wenliang wrote:
>> From: book <book@100ask.localdomain>
>>
>> SY24655 is similar to INA226. Its supply voltage and pin definitions
>> are therefore the same. Compared to INA226, SY24655 has two additional
>> registers for configuring and calculating average power.
>>
>> Signed-off-by: book <book@100ask.localdomain>
>> Acked-by: Conor Dooley <conor.dooley@microchip.com>
> 
> NAK, this never happened. If you think otherwise: provide proof, please.

Hm, now I found previous v3, so ack happened, but patch still has
incorrect author.

Please really carefully read submitting patches document, especially
parts about sending new versions, changelogs, subjects.


> 
> Nothing improved in this binding, actually it got even worse with fake
> email and probably name as well.


Best regards,
Krzysztof

Re: [PATCH linux dev-6.11 v3 2/2] dt-bindings: Add SY24655 to ina2xx devicetree bindings

[Conor Dooley]

[-- Attachment #1: Type: text/plain, Size: 1133 bytes --]

On Sun, Nov 03, 2024 at 05:52:59PM +0100, Krzysztof Kozlowski wrote:
> On 03/11/2024 17:46, Krzysztof Kozlowski wrote:
> > On 03/11/2024 17:39, Wenliang wrote:
> >> From: book <book@100ask.localdomain>
> >>
> >> SY24655 is similar to INA226. Its supply voltage and pin definitions
> >> are therefore the same. Compared to INA226, SY24655 has two additional
> >> registers for configuring and calculating average power.
> >>
> >> Signed-off-by: book <book@100ask.localdomain>
> >> Acked-by: Conor Dooley <conor.dooley@microchip.com>
> > 
> > NAK, this never happened. If you think otherwise: provide proof, please.
> 
> Hm, now I found previous v3, so ack happened, but patch still has
> incorrect author.

I didn't ack it with this weird authorship, so that needs to be fixed
before the ack can be applied.

> 
> Please really carefully read submitting patches document, especially
> parts about sending new versions, changelogs, subjects.
> 
> 
> > 
> > Nothing improved in this binding, actually it got even worse with fake
> > email and probably name as well.
> 
> 
> Best regards,
> Krzysztof
> 

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]