Re: [PATCH v4 0/2] hwmon:(ina238)Add support for SQ52206

[Wenliang Yan]

At 2025-01-25 13:01:36, "Guenter Roeck" <linux@xxxxxxxxxxxx> wrote:
>On 1/24/25 19:02, Wenliang Yan wrote:
>> Add support for Silergy i2c power monitor SQ52206 to the ina238
>> driver as those two are similar.
>> 
>> Signed-off-by: Wenliang Yan <wenliang202407@xxxxxxx>
>> ---
>> 
>> Add new chip SQ52206, the datasheet depends on
>> https://us1.silergy.com/cloud/index/uniqid/676b659b4a503
>> The password is fx6NEe.
>> 
>> Changes in v4:
>> - addressed various review comments.
>> - Link to v3: https://lore.kernel.org/linux-hwmon/20250117082017.688212-1-wenliang202407@xxxxxxx/
>> 
>
>Your "change logs" are equivalent to "made changes" and completely pointless.
>As consequence, I'll only handle your patches if and when I have nothing else
>to review since I'll have to figure out myself what you changed and if you
>implemented the feedback. I don't really hve time for that, so please do not
>expect that I'll apply this series anytime soon.
>
>Guenter

Thank you for your reply.

Okay, I understand. I will pay attention to adding detailed modifications for
your review in the future. There are two main changes in the V4 version:

1) add 'Acked-by: Krzysztof Kozlowski' information to PATCH 1.

2) formatting adjustments have been made to ina2xx.c in PATCH2 with
Christophe JAILLET's comments.

>> Changes in v3:
>> - addressed various review comments.
>> - Link to v2: https://lore.kernel.org/linux-hwmon/20250113035023.365697-1-wenliang202407@xxxxxxx/
>> 
>> Changes in v2:
>> - Explain why sq52206 compatibility has been added to ina2xx.yaml.
>> - addressed various review comments
>> - Link to v1: https://lore.kernel.org/linux-hwmon/20241224063559.391061-1-wenliang202407@xxxxxxx/
>> 
>> Wenliang Yan (2):
>>    dt-bindings:Add SQ52206 to ina2xx devicetree bindings
>>    hwmon:(ina238)Add support for SQ52206
>> 
>>   .../devicetree/bindings/hwmon/ti,ina2xx.yaml  |   4 +
>>   Documentation/hwmon/ina238.rst                |  15 ++
>>   drivers/hwmon/ina238.c                        | 209 +++++++++++++++---
>>   3 files changed, 195 insertions(+), 33 deletions(-)
>> 

Thanks,
Wenliang Yan