linux-kernel - Re: [PATCH v6 00/16] Cleanup MediaTek clk reset drivers and support MT8192/MT8195

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: <b7c080dfcdc3488b5b58badbff8396a8604b5969.camel@mediatek.com>
Date:   Tue, 17 May 2022 18:30:31 +0800
From:   Rex-BC Chen <rex-bc.chen@...iatek.com>
To:     Stephen Boyd <sboyd@...nel.org>,
        "krzysztof.kozlowski+dt@...aro.org" 
        <krzysztof.kozlowski+dt@...aro.org>,
        "matthias.bgg@...il.com" <matthias.bgg@...il.com>,
        "mturquette@...libre.com" <mturquette@...libre.com>,
        "robh+dt@...nel.org" <robh+dt@...nel.org>
CC:     "p.zabel@...gutronix.de" <p.zabel@...gutronix.de>,
        "angelogioacchino.delregno@...labora.com" 
        <angelogioacchino.delregno@...labora.com>,
        Chun-Jie Chen (陳浚桀) 
        <Chun-Jie.Chen@...iatek.com>,
        "wenst@...omium.org" <wenst@...omium.org>,
        Runyang Chen (陈润洋) 
        <Runyang.Chen@...iatek.com>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "linux-clk@...r.kernel.org" <linux-clk@...r.kernel.org>,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>,
        "linux-mediatek@...ts.infradead.org" 
        <linux-mediatek@...ts.infradead.org>,
        Project_Global_Chrome_Upstream_Group 
        <Project_Global_Chrome_Upstream_Group@...iatek.com>
Subject: Re: [PATCH v6 00/16] Cleanup MediaTek clk reset drivers and support
 MT8192/MT8195

On Tue, 2022-05-17 at 15:23 +0800, Stephen Boyd wrote:
> Quoting Rex-BC Chen (2022-05-08 22:35:55)
> > 
> > The drivers of this series are reviewed.
> > The binding of this series are also acked.
> > Could you spare some time and give us some suggestion?
> 
> Have you considered using the auxiliary bus to split the Mediatek clk
> and reset device up into a clk device and a reset device? The idea
> would
> be to move the reset related code into drivers/reset and have the clk
> code in drivers/clk. It's purely an organizational thing and it can
> certainly be done later but it may be a good idea to do this to
> clearly split out the two different functionalities.

Hello Stephen,

Thanks for your advice.
The purpose of this series is to clean up the drivers and we can apply
new socs for it.

I didn't use the auxiliary bus before. However, I can study to the
implementation of auxiliary bus for this reset drivers in another
series. Is this ok for you?

Thanks!

BRs,
Rex