linux-kernel - RE: [PATCH V2 2/2] dt-bindings: power: bq27xxx: add bq78z10

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Wed, 3 Feb 2021 03:49:43 +0000
From:   LI Qingwu <qing-wu.li@...ca-geosystems.com.cn>
To:     Krzysztof Kozlowski <krzk@...nel.org>
CC:     "sre@...nel.org" <sre@...nel.org>,
        "robh+dt@...nel.org" <robh+dt@...nel.org>,
        "dmurphy@...com" <dmurphy@...com>,
        "pali@...nel.org" <pali@...nel.org>,
        "linux-pm@...r.kernel.org" <linux-pm@...r.kernel.org>,
        "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        TERTYCHNYI Grygorii <grygorii.tertychnyi@...ca-geosystems.com>,
        ZHIZHIKIN Andrey <andrey.zhizhikin@...ca-geosystems.com>
Subject: RE: [PATCH V2 2/2] dt-bindings: power: bq27xxx: add bq78z10

Hello Krzysztof Kozlowski,

Thanks for input!

No comma before "and".  	>> I will delete in V3

What does it mean authentication? 	>> The gas gauge able to has a security key to seal and unseal access, to protect the design data, like design energy and capacity.

In the future, please send the bindings as first patch. The checkpatch should point this - did you run it?  >>yes, I run the checkpatch, I will reorder in V3.




Best Regards

**********************************************
Li Qingwu (Terry)
Senior Embedded Software Engineer 
Leica Geosystems(Shanghai)Co.,Limited
(Tel): +86 21 61061036
(FAX): +86 21 61061008
(Mobile): +86 187 0185 9600
E-mail: Qing-wu.Li@...ca-geosystems.com.cn
Http: www.leica-geosystems.com.cn
**********************************************

-----Original Message-----
From: Krzysztof Kozlowski <krzk@...nel.org> 
Sent: Tuesday, February 2, 2021 5:08 PM
To: LI Qingwu <qing-wu.li@...ca-geosystems.com.cn>
Cc: sre@...nel.org; robh+dt@...nel.org; dmurphy@...com; pali@...nel.org; afd@...com; linux-pm@...r.kernel.org; devicetree@...r.kernel.org; linux-kernel@...r.kernel.org; TERTYCHNYI Grygorii <grygorii.tertychnyi@...ca-geosystems.com>; ZHIZHIKIN Andrey <andrey.zhizhikin@...ca-geosystems.com>
Subject: Re: [PATCH V2 2/2] dt-bindings: power: bq27xxx: add bq78z10

This email is not from Hexagon’s Office 365 instance. Please be careful while clicking links, opening attachments, or replying to this email.


On Tue, Feb 02, 2021 at 08:48:58AM +0000, LI Qingwu wrote:
> Add bindings for TI BQ78Z100. An I2C interface gas gauge.
> It provides a fully integrated safety protection, and authentication 
> for 1 to 2-series cell Li-Ion and

No comma before "and".

What does it mean authentication?

In the future, please send the bindings as first patch. The checkpatch should point this - did you run it?

Best regards,
Krzysztof


> Li-Polymer battery packs.
>
> Signed-off-by: LI Qingwu <Qing-wu.Li@...ca-geosystems.com.cn>
> ---
>  Documentation/devicetree/bindings/power/supply/bq27xxx.yaml | 1 +