Re: Re: [PATCH linux dev-6.11 2/2] dt-bindings: modified ina2xx to match SY24655(SQ52205)

[Conor Dooley <conor@kernel.org>]

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

Hey,

On Tue, Oct 08, 2024 at 07:58:51PM +0800, wenliang yan wrote:
> Modified the binding of ina2xx to make it compatible with SY24655. 
> 
> 
> 
> 
> Signed-off-by: Wenliang <wenliang202407@163.com>
> 
> ---
> 
> 
> 
> 
> SY24655 is a fixed gain power monitor from Silergy, with a power supply
> 
> of 2.7-5.5V and communication mode of IIC capable of detecting bus voltage
> 
> and voltage on shunt resistors. Its first 5 registers are identical to
> 
> ina226, and also have alert and limit functions. So, the sy24655 is
> 
> compatible with the ina2xx devices.

This should be above the signoff and --- line. Your patch is pretty
badly malformed, did you use b4 or git send-email to submit it?

Cheers,
Conor.

> 
> 
> 
> 
>  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.17.1
> 
> 

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