All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rex-BC Chen <rex-bc.chen@mediatek.com>
To: Stephen Boyd <sboyd@kernel.org>,
	Chen-Yu Tsai <wenst@chromium.org>,
	"matthias.bgg@gmail.com" <matthias.bgg@gmail.com>
Cc: "mturquette@baylibre.com" <mturquette@baylibre.com>,
	"robh+dt@kernel.org" <robh+dt@kernel.org>,
	"krzysztof.kozlowski+dt@linaro.org"
	<krzysztof.kozlowski+dt@linaro.org>,
	"p.zabel@pengutronix.de" <p.zabel@pengutronix.de>,
	"angelogioacchino.delregno@collabora.com"
	<angelogioacchino.delregno@collabora.com>,
	"Chun-Jie Chen (陳浚桀)" <Chun-Jie.Chen@mediatek.com>,
	"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 v7 13/19] dt-bindings: reset: mediatek: Add infra_ao reset index for MT8192/MT8195
Date: Mon, 23 May 2022 13:14:06 +0800	[thread overview]
Message-ID: <93464881d214501851eafd55fdb25b69b6bf77e9.camel@mediatek.com> (raw)
In-Reply-To: <20220521042323.BA60AC385A5@smtp.kernel.org>

On Sat, 2022-05-21 at 12:23 +0800, Stephen Boyd wrote:
> Quoting Chen-Yu Tsai (2022-05-19 20:10:35)
> > On Fri, May 20, 2022 at 10:58 AM Rex-BC Chen <
> > rex-bc.chen@mediatek.com> wrote:
> > > 
> > > 
> > > Hello Stephen,
> > > 
> > > this patch will have conflict with Matthias's commit branch for
> > > include/dt-bindings/reset/mt8192-resets.h.
> > > 
> > > It's on linux-next.
> > > 
> > > I have fix it in this version, but I think there will be a merge
> > > conflict if you pick my series in this run.
> > > 
> > > 
https://urldefense.com/v3/__https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/include/dt-bindings/reset/mt8192-resets.h?h=next-20220519&id=19c66219e4d5b813ebbd28621cfe9c450659ded7__;!!CTRNKA9wMg0ARbw!xFfRxOWRAvhQMzoKxqRu8xFsMZLpVZEoK0WZBKeMOoeRGBjKphSL2tox-8zWlRwytjtEImTo1VB1f38wwj8XSlyKIw$
> > >  
> > 
> > The commit in question is in Matthias's v5.18-next/dts64 branch,
> > which also
> > has the v5.18-next-dts64 tag. The PR for this tag was already
> > picked up
> > by the soc maintainers, so I guess we could consider it stable.
> > 
> > Matthias, what do you think? Give an ack for the patch to go
> > through the
> > clk tree with the tag merged in as dependency? There's a compile
> > time
> > dependency between this and the next patch so we can't just split
> > them
> > into different trees.
> > 
> 
> It's pretty late to land something that has cross tree dependencies
> like
> that so I guess we'll just take up this patch series after the merge
> window closes.

Hello Stephen,

Thanks for your big help!
I will send v8 to modify comments from Nícolas.

BRs,
Rex


WARNING: multiple messages have this Message-ID (diff)
From: Rex-BC Chen <rex-bc.chen@mediatek.com>
To: Stephen Boyd <sboyd@kernel.org>,
	Chen-Yu Tsai <wenst@chromium.org>,
	"matthias.bgg@gmail.com" <matthias.bgg@gmail.com>
Cc: "mturquette@baylibre.com" <mturquette@baylibre.com>,
	"robh+dt@kernel.org" <robh+dt@kernel.org>,
	"krzysztof.kozlowski+dt@linaro.org"
	<krzysztof.kozlowski+dt@linaro.org>,
	"p.zabel@pengutronix.de" <p.zabel@pengutronix.de>,
	"angelogioacchino.delregno@collabora.com"
	<angelogioacchino.delregno@collabora.com>,
	"Chun-Jie Chen (陳浚桀)" <Chun-Jie.Chen@mediatek.com>,
	"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 v7 13/19] dt-bindings: reset: mediatek: Add infra_ao reset index for MT8192/MT8195
Date: Mon, 23 May 2022 13:14:06 +0800	[thread overview]
Message-ID: <93464881d214501851eafd55fdb25b69b6bf77e9.camel@mediatek.com> (raw)
In-Reply-To: <20220521042323.BA60AC385A5@smtp.kernel.org>

On Sat, 2022-05-21 at 12:23 +0800, Stephen Boyd wrote:
> Quoting Chen-Yu Tsai (2022-05-19 20:10:35)
> > On Fri, May 20, 2022 at 10:58 AM Rex-BC Chen <
> > rex-bc.chen@mediatek.com> wrote:
> > > 
> > > 
> > > Hello Stephen,
> > > 
> > > this patch will have conflict with Matthias's commit branch for
> > > include/dt-bindings/reset/mt8192-resets.h.
> > > 
> > > It's on linux-next.
> > > 
> > > I have fix it in this version, but I think there will be a merge
> > > conflict if you pick my series in this run.
> > > 
> > > 
https://urldefense.com/v3/__https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/include/dt-bindings/reset/mt8192-resets.h?h=next-20220519&id=19c66219e4d5b813ebbd28621cfe9c450659ded7__;!!CTRNKA9wMg0ARbw!xFfRxOWRAvhQMzoKxqRu8xFsMZLpVZEoK0WZBKeMOoeRGBjKphSL2tox-8zWlRwytjtEImTo1VB1f38wwj8XSlyKIw$
> > >  
> > 
> > The commit in question is in Matthias's v5.18-next/dts64 branch,
> > which also
> > has the v5.18-next-dts64 tag. The PR for this tag was already
> > picked up
> > by the soc maintainers, so I guess we could consider it stable.
> > 
> > Matthias, what do you think? Give an ack for the patch to go
> > through the
> > clk tree with the tag merged in as dependency? There's a compile
> > time
> > dependency between this and the next patch so we can't just split
> > them
> > into different trees.
> > 
> 
> It's pretty late to land something that has cross tree dependencies
> like
> that so I guess we'll just take up this patch series after the merge
> window closes.

Hello Stephen,

Thanks for your big help!
I will send v8 to modify comments from Nícolas.

BRs,
Rex


_______________________________________________
Linux-mediatek mailing list
Linux-mediatek@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-mediatek

WARNING: multiple messages have this Message-ID (diff)
From: Rex-BC Chen <rex-bc.chen@mediatek.com>
To: Stephen Boyd <sboyd@kernel.org>,
	Chen-Yu Tsai <wenst@chromium.org>,
	"matthias.bgg@gmail.com" <matthias.bgg@gmail.com>
Cc: "mturquette@baylibre.com" <mturquette@baylibre.com>,
	"robh+dt@kernel.org" <robh+dt@kernel.org>,
	"krzysztof.kozlowski+dt@linaro.org"
	<krzysztof.kozlowski+dt@linaro.org>,
	"p.zabel@pengutronix.de" <p.zabel@pengutronix.de>,
	"angelogioacchino.delregno@collabora.com"
	<angelogioacchino.delregno@collabora.com>,
	"Chun-Jie Chen (陳浚桀)" <Chun-Jie.Chen@mediatek.com>,
	"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 v7 13/19] dt-bindings: reset: mediatek: Add infra_ao reset index for MT8192/MT8195
Date: Mon, 23 May 2022 13:14:06 +0800	[thread overview]
Message-ID: <93464881d214501851eafd55fdb25b69b6bf77e9.camel@mediatek.com> (raw)
In-Reply-To: <20220521042323.BA60AC385A5@smtp.kernel.org>

On Sat, 2022-05-21 at 12:23 +0800, Stephen Boyd wrote:
> Quoting Chen-Yu Tsai (2022-05-19 20:10:35)
> > On Fri, May 20, 2022 at 10:58 AM Rex-BC Chen <
> > rex-bc.chen@mediatek.com> wrote:
> > > 
> > > 
> > > Hello Stephen,
> > > 
> > > this patch will have conflict with Matthias's commit branch for
> > > include/dt-bindings/reset/mt8192-resets.h.
> > > 
> > > It's on linux-next.
> > > 
> > > I have fix it in this version, but I think there will be a merge
> > > conflict if you pick my series in this run.
> > > 
> > > 
https://urldefense.com/v3/__https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/include/dt-bindings/reset/mt8192-resets.h?h=next-20220519&id=19c66219e4d5b813ebbd28621cfe9c450659ded7__;!!CTRNKA9wMg0ARbw!xFfRxOWRAvhQMzoKxqRu8xFsMZLpVZEoK0WZBKeMOoeRGBjKphSL2tox-8zWlRwytjtEImTo1VB1f38wwj8XSlyKIw$
> > >  
> > 
> > The commit in question is in Matthias's v5.18-next/dts64 branch,
> > which also
> > has the v5.18-next-dts64 tag. The PR for this tag was already
> > picked up
> > by the soc maintainers, so I guess we could consider it stable.
> > 
> > Matthias, what do you think? Give an ack for the patch to go
> > through the
> > clk tree with the tag merged in as dependency? There's a compile
> > time
> > dependency between this and the next patch so we can't just split
> > them
> > into different trees.
> > 
> 
> It's pretty late to land something that has cross tree dependencies
> like
> that so I guess we'll just take up this patch series after the merge
> window closes.

Hello Stephen,

Thanks for your big help!
I will send v8 to modify comments from Nícolas.

BRs,
Rex


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  parent reply	other threads:[~2022-05-23  5:14 UTC|newest]

Thread overview: 105+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-05-19 12:55 [PATCH v7 00/19] Cleanup MediaTek clk reset drivers and support SoCs Rex-BC Chen
2022-05-19 12:55 ` Rex-BC Chen
2022-05-19 12:55 ` Rex-BC Chen
2022-05-19 12:55 ` [PATCH v7 01/19] clk: mediatek: reset: Add reset.h Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-19 12:55 ` [PATCH v7 02/19] clk: mediatek: reset: Fix written reset bit offset Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-19 12:55 ` [PATCH v7 03/19] clk: mediatek: reset: Refine and reorder functions in reset.c Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-19 12:55 ` [PATCH v7 04/19] clk: mediatek: reset: Extract common drivers to update function Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-20 14:55   ` Nícolas F. R. A. Prado
2022-05-20 14:55     ` Nícolas F. R. A. Prado
2022-05-20 14:55     ` Nícolas F. R. A. Prado
2022-05-23  5:08     ` Rex-BC Chen
2022-05-23  5:08       ` Rex-BC Chen
2022-05-23  5:08       ` Rex-BC Chen
2022-05-19 12:55 ` [PATCH v7 05/19] clk: mediatek: reset: Merge and revise reset register function Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-20 15:12   ` Nícolas F. R. A. Prado
2022-05-20 15:12     ` Nícolas F. R. A. Prado
2022-05-20 15:12     ` Nícolas F. R. A. Prado
2022-05-23  5:09     ` Rex-BC Chen
2022-05-23  5:09       ` Rex-BC Chen
2022-05-23  5:09       ` Rex-BC Chen
2022-05-19 12:55 ` [PATCH v7 06/19] clk: mediatek: reset: Revise structure to control reset register Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-19 12:55 ` [PATCH v7 07/19] clk: mediatek: reset: Support nonsequence base offsets of reset registers Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-20 15:18   ` Nícolas F. R. A. Prado
2022-05-20 15:18     ` Nícolas F. R. A. Prado
2022-05-20 15:18     ` Nícolas F. R. A. Prado
2022-05-23  5:10     ` Rex-BC Chen
2022-05-23  5:10       ` Rex-BC Chen
2022-05-23  5:10       ` Rex-BC Chen
2022-05-19 12:55 ` [PATCH v7 08/19] clk: mediatek: reset: Support inuput argument index mode Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-19 12:55 ` [PATCH v7 09/19] clk: mediatek: reset: Change return type for clock reset register function Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-19 12:55 ` [PATCH v7 10/19] clk: mediatek: reset: Add new register reset function with device Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-19 12:55 ` [PATCH v7 11/19] clk: mediatek: reset: Add reset support for simple probe Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-19 12:55 ` [PATCH v7 12/19] dt-bindings: arm: mediatek: Add #reset-cells property for MT8192/MT8195 Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-19 12:55 ` [PATCH v7 13/19] dt-bindings: reset: mediatek: Add infra_ao reset index " Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-20  2:58   ` Rex-BC Chen
2022-05-20  2:58     ` Rex-BC Chen
2022-05-20  2:58     ` Rex-BC Chen
2022-05-20  3:10     ` Chen-Yu Tsai
2022-05-20  3:10       ` Chen-Yu Tsai
2022-05-20  3:10       ` Chen-Yu Tsai
     [not found]       ` <20220521042323.BA60AC385A5@smtp.kernel.org>
2022-05-23  5:14         ` Rex-BC Chen [this message]
2022-05-23  5:14           ` Rex-BC Chen
2022-05-23  5:14           ` Rex-BC Chen
2022-05-19 12:55 ` [PATCH v7 14/19] clk: mediatek: reset: Add infra_ao reset support " Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-19 12:55 ` [PATCH v7 15/19] arm64: dts: mediatek: Add infra #reset-cells property for MT8192 Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-19 12:55 ` [PATCH v7 16/19] arm64: dts: mediatek: Add infra #reset-cells property for MT8195 Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-20 15:30   ` Nícolas F. R. A. Prado
2022-05-20 15:30     ` Nícolas F. R. A. Prado
2022-05-20 15:30     ` Nícolas F. R. A. Prado
2022-05-23  5:11     ` Rex-BC Chen
2022-05-23  5:11       ` Rex-BC Chen
2022-05-23  5:11       ` Rex-BC Chen
2022-05-19 12:55 ` [PATCH v7 17/19] dt-bindings: reset: mediatek: Add infra_ao reset index for MT8186 Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-20 22:32   ` Rob Herring
2022-05-20 22:32     ` Rob Herring
2022-05-20 22:32     ` Rob Herring
2022-05-19 12:55 ` [PATCH v7 18/19] dt-bindings: arm: mediatek: Add #reset-cells property " Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-20 22:33   ` Rob Herring
2022-05-20 22:33     ` Rob Herring
2022-05-20 22:33     ` Rob Herring
2022-05-19 12:55 ` [PATCH v7 19/19] clk: mediatek: reset: Add infra_ao reset support " Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-19 12:55   ` Rex-BC Chen
2022-05-20 15:40 ` [PATCH v7 00/19] Cleanup MediaTek clk reset drivers and support SoCs Nícolas F. R. A. Prado
2022-05-20 15:40   ` Nícolas F. R. A. Prado
2022-05-20 15:40   ` Nícolas F. R. A. Prado
2022-05-23  5:12   ` Rex-BC Chen
2022-05-23  5:12     ` Rex-BC Chen
2022-05-23  5:12     ` Rex-BC Chen

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=93464881d214501851eafd55fdb25b69b6bf77e9.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.