linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Rex-BC Chen <rex-bc.chen@mediatek.com>
To: Stephen Boyd <sboyd@kernel.org>,
	"krzysztof.kozlowski+dt@linaro.org" 
	<krzysztof.kozlowski+dt@linaro.org>,
	"matthias.bgg@gmail.com" <matthias.bgg@gmail.com>,
	"mturquette@baylibre.com" <mturquette@baylibre.com>,
	"robh+dt@kernel.org" <robh+dt@kernel.org>
Cc: "p.zabel@pengutronix.de" <p.zabel@pengutronix.de>,
	"angelogioacchino.delregno@collabora.com"
	<angelogioacchino.delregno@collabora.com>,
	"Chun-Jie Chen (陳浚桀)" <Chun-Jie.Chen@mediatek.com>,
	"wenst@chromium.org" <wenst@chromium.org>,
	"Runyang Chen (陈润洋)" <Runyang.Chen@mediatek.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"linux-clk@vger.kernel.org" <linux-clk@vger.kernel.org>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>,
	"linux-mediatek@lists.infradead.org"
	<linux-mediatek@lists.infradead.org>,
	Project_Global_Chrome_Upstream_Group
	<Project_Global_Chrome_Upstream_Group@mediatek.com>
Subject: Re: [PATCH v6 00/16] Cleanup MediaTek clk reset drivers and support MT8192/MT8195
Date: Tue, 17 May 2022 18:30:31 +0800	[thread overview]
Message-ID: <b7c080dfcdc3488b5b58badbff8396a8604b5969.camel@mediatek.com> (raw)
In-Reply-To: <20220517072329.D367AC385B8@smtp.kernel.org>

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


      parent reply	other threads:[~2022-05-17 10:30 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-05-03  9:38 [PATCH v6 00/16] Cleanup MediaTek clk reset drivers and support MT8192/MT8195 Rex-BC Chen
2022-05-03  9:38 ` [PATCH v6 01/16] clk: mediatek: reset: Add reset.h Rex-BC Chen
2022-05-03  9:38 ` [PATCH v6 02/16] clk: mediatek: reset: Fix written reset bit offset Rex-BC Chen
2022-05-03  9:38 ` [PATCH v6 03/16] clk: mediatek: reset: Refine and reorder functions in reset.c Rex-BC Chen
2022-05-03  9:38 ` [PATCH v6 04/16] clk: mediatek: reset: Extract common drivers to update function Rex-BC Chen
2022-05-03  9:38 ` [PATCH v6 05/16] clk: mediatek: reset: Merge and revise reset register function Rex-BC Chen
2022-05-03  9:38 ` [PATCH v6 06/16] clk: mediatek: reset: Revise structure to control reset register Rex-BC Chen
2022-05-03  9:38 ` [PATCH v6 07/16] clk: mediatek: reset: Support nonsequence base offsets of reset registers Rex-BC Chen
2022-05-03  9:38 ` [PATCH v6 08/16] clk: mediatek: reset: Support inuput argument index mode Rex-BC Chen
2022-05-03 12:37   ` AngeloGioacchino Del Regno
2022-05-03  9:38 ` [PATCH v6 09/16] clk: mediatek: reset: Change return type for clock reset register function Rex-BC Chen
2022-05-03  9:38 ` [PATCH v6 10/16] clk: mediatek: reset: Add new register reset function with device Rex-BC Chen
2022-05-03  9:38 ` [PATCH v6 11/16] clk: mediatek: reset: Add reset support for simple probe Rex-BC Chen
2022-05-03  9:38 ` [PATCH v6 12/16] dt-bindings: arm: mediatek: Add #reset-cells property for MT8192/MT8195 Rex-BC Chen
2022-05-03 12:37   ` AngeloGioacchino Del Regno
2022-05-03  9:38 ` [PATCH v6 13/16] dt-bindings: reset: mediatek: Add infra_ao reset index " Rex-BC Chen
2022-05-03 12:27   ` Krzysztof Kozlowski
2022-05-03 12:37   ` AngeloGioacchino Del Regno
2022-05-03  9:38 ` [PATCH v6 14/16] clk: mediatek: reset: Add infra_ao reset support " Rex-BC Chen
2022-05-03  9:38 ` [PATCH v6 15/16] arm64: dts: mediatek: Add infra #reset-cells property for MT8192 Rex-BC Chen
2022-06-22 11:05   ` Matthias Brugger
2022-05-03  9:38 ` [PATCH v6 16/16] arm64: dts: mediatek: Add infra #reset-cells property for MT8195 Rex-BC Chen
2022-06-22 11:08   ` Matthias Brugger
2022-06-22 12:06     ` Rex-BC Chen
2022-05-09  5:35 ` [PATCH v6 00/16] Cleanup MediaTek clk reset drivers and support MT8192/MT8195 Rex-BC Chen
     [not found]   ` <20220517072329.D367AC385B8@smtp.kernel.org>
2022-05-17 10:30     ` Rex-BC Chen [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=b7c080dfcdc3488b5b58badbff8396a8604b5969.camel@mediatek.com \
    --to=rex-bc.chen@mediatek.com \
    --cc=Chun-Jie.Chen@mediatek.com \
    --cc=Project_Global_Chrome_Upstream_Group@mediatek.com \
    --cc=Runyang.Chen@mediatek.com \
    --cc=angelogioacchino.delregno@collabora.com \
    --cc=devicetree@vger.kernel.org \
    --cc=krzysztof.kozlowski+dt@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-clk@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mediatek@lists.infradead.org \
    --cc=matthias.bgg@gmail.com \
    --cc=mturquette@baylibre.com \
    --cc=p.zabel@pengutronix.de \
    --cc=robh+dt@kernel.org \
    --cc=sboyd@kernel.org \
    --cc=wenst@chromium.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).