Re: [PATCH v4 1/2] dt-bindings: power: supply: Add STC3117 Fuel Gauge

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Krzysztof,

>>> +  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.
>>
>
> You responded privately - I am not going to do any talks under NDA. I
> also do not provide some sort of personal support service. Keep *ALL*
> discussions public.

My apologies, that was unintentional.

> Explaining what you asked:
>
> Some of these are from monitored-battery. Sense resistor should be
> separate property. But different question is about missing resources,
> like supplies (VCC) and interrupts. Just look at datasheet.

since battery itself serves as the power supply for fuel gauge no additional supplies are required. 

Regarding interrupts yes, datasheet specifies an alarm pin that signals low State of Charge or
Voltage conditions. However, I haven’t incorporated interrupts in the driver to handle this. Should I
list the alarm pin as a resource in the device tree (or relevant configuration) even if it’s not 
implemented in the driver yet?

Best Regards,
Bhavin




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux