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

From: Conor Dooley
Date: Tue Oct 08 2024 - 12:17:32 EST


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@xxxxxxx>
>
> ---
>
>
>
>
> 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: signature.asc
Description: PGP signature