linux-kernel - Re: [PATCH v12 1/3] dt-bindings: mmc: mtk-sd: fix yamllint error

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <ed60e1a69b4b61008b2bc76cfed93f46ac5efa2b.camel@mediatek.com>
Date:   Wed, 30 Mar 2022 10:55:56 +0800
From:   Tinghan Shen <tinghan.shen@...iatek.com>
To:     AngeloGioacchino Del Regno 
        <angelogioacchino.delregno@...labora.com>,
        Chaotian Jing <chaotian.jing@...iatek.com>,
        Ulf Hansson <ulf.hansson@...aro.org>,
        Rob Herring <robh+dt@...nel.org>,
        "Krzysztof Kozlowski" <krzysztof.kozlowski@...onical.com>,
        Matthias Brugger <matthias.bgg@...il.com>,
        Wenbin Mei <wenbin.mei@...iatek.com>
CC:     <linux-mmc@...r.kernel.org>, <devicetree@...r.kernel.org>,
        <linux-kernel@...r.kernel.org>,
        <linux-arm-kernel@...ts.infradead.org>,
        <linux-mediatek@...ts.infradead.org>,
        <Project_Global_Chrome_Upstream_Group@...iatek.com>,
        <ryder.lee@...nel.org>, <wenst@...omium.org>,
        <chunfeng.yun@...iatek.com>
Subject: Re: [PATCH v12 1/3] dt-bindings: mmc: mtk-sd: fix yamllint error

Hi Angelo,

On Tue, 2022-03-29 at 14:47 +0200, AngeloGioacchino Del Regno wrote:
> Il 29/03/22 13:45, Tinghan Shen ha scritto:
> > Documentation/devicetree/bindings/mmc/mtk-sd.yaml
> >    54:81     error    line too long (95 > 80 characters)  (line-length)
> > 
> 
> I can't reproduce this error that you're getting... this commit is not
> necessary, as the .yamllint file in the kernel allows a maximum line-length
> of 110 characters.
> 
> rules:
>    line-length:
>      # 80 chars should be enough, but don't fail if a line is longer
>      max: 110
> 
> 
https://urldefense.com/v3/__https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/Documentation/devicetree/bindings/.yamllint?h=next-20220329__;!!CTRNKA9wMg0ARbw!1L9icT80FdI9EYV81qRGpzbEvV1NFP_pwk8YABV0eIiRf2ppttnPlRfu5aXEUsxODiE$
>  
> 
> Please drop this commit.

Thank you for your feedback.

I figured out why I have this error...
It's because I do the yamllint outside of the bindings folder and not specify the yamllint config
file.

I'll drop this at next version.

Best regards,
Tinghan
> 
> > Signed-off-by: Tinghan Shen <tinghan.shen@...iatek.com>
> > ---
> >   Documentation/devicetree/bindings/mmc/mtk-sd.yaml | 3 ++-
> >   1 file changed, 2 insertions(+), 1 deletion(-)