Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Tue, 2 Feb 2021 10:08:23 +0100
From:   Krzysztof Kozlowski <krzk@kernel.org>
To:     LI Qingwu <Qing-wu.Li@leica-geosystems.com.cn>
Cc:     sre@kernel.org, robh+dt@kernel.org, dmurphy@ti.com,
        pali@kernel.org, afd@ti.com, linux-pm@vger.kernel.org,
        devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
        grygorii.tertychnyi@leica-geosystems.com,
        andrey.zhizhikin@leica-geosystems.com
Subject: Re: [PATCH V2 2/2] dt-bindings: power: bq27xxx: add bq78z10
Message-ID: <20210202090823.rgah5hxoegsueu54@kozik-lap>
References: <20210202084858.2944-1-Qing-wu.Li@leica-geosystems.com.cn>
 <20210202084858.2944-3-Qing-wu.Li@leica-geosystems.com.cn>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
In-Reply-To: <20210202084858.2944-3-Qing-wu.Li@leica-geosystems.com.cn>
Precedence: bulk

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@leica-geosystems.com.cn>
> ---
>  Documentation/devicetree/bindings/power/supply/bq27xxx.yaml | 1 +