Re: [PATCH v4 1/2] dt-bindings: power: supply: Add STC3117 Fuel Gauge
From: Bhavin Sharma
Date: Thu Nov 28 2024 - 03:41:59 EST
Hi Krzysztof,
Thank you for your review.
> On 27/11/2024 16:19, Bhavin Sharma wrote:
> > +
> > +allOf:
> > + - $ref: power-supply.yaml#
> > +
> > +properties:
> > + compatible:
> > + enum:
> > + - st,stc3117
> > +
> > + reg:
> > + maxItems: 1
>
> I asked you some questions on v2, then on v3 and no responses.
I sincerely apologize for not addressing your questions in versions 2 and 3
of the patch.
Regarding the battery configuration, we need the following information:
- Battery capacity
- Battery impedance
- OCV curve
- Minimum and maximum voltage
- Current sense register
Currently, I have set the battery capacity and impedance directly in the
driver as default values. However, I understand these should ideally be
defined in the device tree source (DTS). I will update the DTS accordingly
to include these parameters.
Additionally, since the OCV curve, minimum/maximum voltage, and sense
register values are fixed, I would like your opinion on whether these should
also be defined in the DTS or if it is acceptable to keep them in the driver itself.
Once again, I apologize for the oversight and thank you for your understanding.
Best regards,
Bhavin
________________________________________
From: Krzysztof Kozlowski <krzk@xxxxxxxxxx>
Sent: Wednesday, November 27, 2024 11:52 PM
To: Bhavin Sharma <bhavin.sharma@xxxxxxxxxxxxxxxxx>; sre@xxxxxxxxxx <sre@xxxxxxxxxx>; krzk+dt@xxxxxxxxxx <krzk+dt@xxxxxxxxxx>; robh@xxxxxxxxxx <robh@xxxxxxxxxx>; conor+dt@xxxxxxxxxx <conor+dt@xxxxxxxxxx>
Cc: Hardevsinh Palaniya <hardevsinh.palaniya@xxxxxxxxxxxxxxxxx>; linux-pm@xxxxxxxxxxxxxxx <linux-pm@xxxxxxxxxxxxxxx>; devicetree@xxxxxxxxxxxxxxx <devicetree@xxxxxxxxxxxxxxx>; linux-kernel@xxxxxxxxxxxxxxx <linux-kernel@xxxxxxxxxxxxxxx>
Subject: Re: [PATCH v4 1/2] dt-bindings: power: supply: Add STC3117 Fuel Gauge
CAUTION: This email originated from outside the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe.
On 27/11/2024 16:19, Bhavin Sharma wrote:
> +
> +allOf:
> + - $ref: power-supply.yaml#
> +
> +properties:
> + compatible:
> + enum:
> + - st,stc3117
> +
> + reg:
> + maxItems: 1
I asked you some questions on v2, then on v3 and no responses.
You implemented some changes but still did not answer my question. I am
not going to ask again, obviously expecting different result on the same
makes little sense.
No ack from me.
Best regards,
Krzysztof