| Message ID | 20250125030300.1230967-1-wenliang202407@163.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | hwmon:(ina238)Add support for SQ52206 | expand |
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@163.com> > --- > > 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@163.com/ > 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 > Changes in v3: > - addressed various review comments. > - Link to v2: https://lore.kernel.org/linux-hwmon/20250113035023.365697-1-wenliang202407@163.com/ > > 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@163.com/ > > 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(-) >
At 2025-01-25 13:01:36, "Guenter Roeck" <linux@roeck-us.net> 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@163.com> >> --- >> >> 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@163.com/ >> > >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@163.com/ >> >> 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@163.com/ >> >> 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
Add support for Silergy i2c power monitor SQ52206 to the ina238 driver as those two are similar. Signed-off-by: Wenliang Yan <wenliang202407@163.com> --- 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@163.com/ Changes in v3: - addressed various review comments. - Link to v2: https://lore.kernel.org/linux-hwmon/20250113035023.365697-1-wenliang202407@163.com/ 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@163.com/ 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(-)