linux-kernel - Re: [PATCH v2 1/5] arm64: dts: mediatek: Correct uart clock of MT8192

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

Message-ID: <20220111191501.dspzjabuuohinufz@notapiano>
Date:   Tue, 11 Jan 2022 14:15:01 -0500
From:   Nícolas F. R. A. Prado 
        <nfraprado@...labora.com>
To:     "allen-kh.cheng" <allen-kh.cheng@...iatek.com>
Cc:     Matthias Brugger <matthias.bgg@...il.com>,
        Rob Herring <robh+dt@...nel.org>,
        Project_Global_Chrome_Upstream_Group@...iatek.com,
        chun-jie.chen@...iatek.com, devicetree@...r.kernel.org,
        drinkcat@...omium.org, linux-arm-kernel@...ts.infradead.org,
        linux-kernel@...r.kernel.org, linux-mediatek@...ts.infradead.org,
        srv_heupstream@...iatek.com
Subject: Re: [PATCH v2 1/5] arm64: dts: mediatek: Correct uart clock of MT8192

Hi Allen,

thank you for improving the commit message! However, please see my comment
below.

On Thu, Jan 06, 2022 at 11:24:16AM +0800, allen-kh.cheng wrote:
> From: Allen-KH Cheng <Allen-KH.Cheng@...iatek.com>
> 
> When the initial devicetree for mt8192 was added in 48489980e27e ("arm64:
> dts: Add Mediatek SoC MT8192 and evaluation board dts and Makefile"), the
> clock driver for mt8192 was not yet upstream, so the clock property nodes
> were set to the clk26m clock as a placeholder.
> 
> Given that the clock driver has since been added through 710573dee31b ("clk:
> mediatek: Add MT8192 basic clocks support"), as well as its dt-bindings
> through f35f1a23e0e1 ("clk: mediatek: Add dt-bindings of MT8192 clocks") and
> devicetree nodes through 5d2b897bc6f5 ("arm64: dts: mediatek: Add mt8192
> clock controllers"), fix the uart clock property to point to the actual

This sentence is incomplete. It is missing "clock." at the end. Could you please
add it?

Same thing for all other patches in this series.

After fixing that typo in the commit message of all patches, please add my
reviewed-by in all patches:

Reviewed-by: Nícolas F. R. A. Prado <nfraprado@...labora.com>

Thanks,
Nícolas