All of lore.kernel.org
 help / color / mirror / Atom feed
From: Matthias Brugger <matthias.bgg@gmail.com>
To: Roger Lu <roger.lu@mediatek.com>,
	Enric Balletbo Serra <eballetbo@gmail.com>,
	Kevin Hilman <khilman@kernel.org>,
	Rob Herring <robh+dt@kernel.org>,
	Nicolas Boichat <drinkcat@google.com>,
	Stephen Boyd <sboyd@kernel.org>,
	Philipp Zabel <p.zabel@pengutronix.de>
Cc: Fan Chen <fan.chen@mediatek.com>,
	HenryC Chen <HenryC.Chen@mediatek.com>,
	YT Lee <yt.lee@mediatek.com>,
	Xiaoqing Liu <Xiaoqing.Liu@mediatek.com>,
	Charles Yang <Charles.Yang@mediatek.com>,
	Angus Lin <Angus.Lin@mediatek.com>,
	Mark Rutland <mark.rutland@arm.com>, Nishanth Menon <nm@ti.com>,
	devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	linux-mediatek@lists.infradead.org, linux-kernel@vger.kernel.org,
	linux-pm@vger.kernel.org,
	Project_Global_Chrome_Upstream_Group@mediatek.com,
	Guenter Roeck <linux@roeck-us.net>
Subject: Re: [PATCH v20 0/7] soc: mediatek: SVS: introduce MTK SVS engine
Date: Thu, 30 Dec 2021 16:13:10 +0100	[thread overview]
Message-ID: <fff75b65-51ab-cafd-a55f-137c0b7c2dc6@gmail.com> (raw)
In-Reply-To: <20210721070904.15636-1-roger.lu@mediatek.com>

Hi Roger,

On 21/07/2021 09:08, Roger Lu wrote:
> 1. SVS driver uses OPP adjust event in [1] to update OPP table voltage part.
> 2. SVS driver gets thermal/GPU device by node [2][3] and CPU device by get_cpu_device().
> After retrieving subsys device, SVS driver calls device_link_add() to make sure probe/suspend callback priority.
> 3. SVS dts refers to reset controller [4] to help reset SVS HW.
> 
> #mt8183 SVS related patches
> [1] https://patchwork.kernel.org/patch/11193513/
> [2] https://patchwork.kernel.org/project/linux-mediatek/patch/20201013102358.22588-2-michael.kao@mediatek.com/
> [3] https://patchwork.kernel.org/project/linux-mediatek/patch/20200306041345.259332-3-drinkcat@chromium.org/
> 

Comments made in v16 actually also hold for v20, so please take them into account :)

Regards,
Matthias

WARNING: multiple messages have this Message-ID (diff)
From: Matthias Brugger <matthias.bgg@gmail.com>
To: Roger Lu <roger.lu@mediatek.com>,
	Enric Balletbo Serra <eballetbo@gmail.com>,
	Kevin Hilman <khilman@kernel.org>,
	Rob Herring <robh+dt@kernel.org>,
	Nicolas Boichat <drinkcat@google.com>,
	Stephen Boyd <sboyd@kernel.org>,
	Philipp Zabel <p.zabel@pengutronix.de>
Cc: Fan Chen <fan.chen@mediatek.com>,
	HenryC Chen <HenryC.Chen@mediatek.com>,
	 YT Lee <yt.lee@mediatek.com>,
	Xiaoqing Liu <Xiaoqing.Liu@mediatek.com>,
	Charles Yang <Charles.Yang@mediatek.com>,
	Angus Lin <Angus.Lin@mediatek.com>,
	Mark Rutland <mark.rutland@arm.com>, Nishanth Menon <nm@ti.com>,
	devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	linux-mediatek@lists.infradead.org, linux-kernel@vger.kernel.org,
	linux-pm@vger.kernel.org,
	Project_Global_Chrome_Upstream_Group@mediatek.com,
	Guenter Roeck <linux@roeck-us.net>
Subject: Re: [PATCH v20 0/7] soc: mediatek: SVS: introduce MTK SVS engine
Date: Thu, 30 Dec 2021 16:13:10 +0100	[thread overview]
Message-ID: <fff75b65-51ab-cafd-a55f-137c0b7c2dc6@gmail.com> (raw)
In-Reply-To: <20210721070904.15636-1-roger.lu@mediatek.com>

Hi Roger,

On 21/07/2021 09:08, Roger Lu wrote:
> 1. SVS driver uses OPP adjust event in [1] to update OPP table voltage part.
> 2. SVS driver gets thermal/GPU device by node [2][3] and CPU device by get_cpu_device().
> After retrieving subsys device, SVS driver calls device_link_add() to make sure probe/suspend callback priority.
> 3. SVS dts refers to reset controller [4] to help reset SVS HW.
> 
> #mt8183 SVS related patches
> [1] https://patchwork.kernel.org/patch/11193513/
> [2] https://patchwork.kernel.org/project/linux-mediatek/patch/20201013102358.22588-2-michael.kao@mediatek.com/
> [3] https://patchwork.kernel.org/project/linux-mediatek/patch/20200306041345.259332-3-drinkcat@chromium.org/
> 

Comments made in v16 actually also hold for v20, so please take them into account :)

Regards,
Matthias

_______________________________________________
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: Matthias Brugger <matthias.bgg@gmail.com>
To: Roger Lu <roger.lu@mediatek.com>,
	Enric Balletbo Serra <eballetbo@gmail.com>,
	Kevin Hilman <khilman@kernel.org>,
	Rob Herring <robh+dt@kernel.org>,
	Nicolas Boichat <drinkcat@google.com>,
	Stephen Boyd <sboyd@kernel.org>,
	Philipp Zabel <p.zabel@pengutronix.de>
Cc: Fan Chen <fan.chen@mediatek.com>,
	HenryC Chen <HenryC.Chen@mediatek.com>,
	 YT Lee <yt.lee@mediatek.com>,
	Xiaoqing Liu <Xiaoqing.Liu@mediatek.com>,
	Charles Yang <Charles.Yang@mediatek.com>,
	Angus Lin <Angus.Lin@mediatek.com>,
	Mark Rutland <mark.rutland@arm.com>, Nishanth Menon <nm@ti.com>,
	devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	linux-mediatek@lists.infradead.org, linux-kernel@vger.kernel.org,
	linux-pm@vger.kernel.org,
	Project_Global_Chrome_Upstream_Group@mediatek.com,
	Guenter Roeck <linux@roeck-us.net>
Subject: Re: [PATCH v20 0/7] soc: mediatek: SVS: introduce MTK SVS engine
Date: Thu, 30 Dec 2021 16:13:10 +0100	[thread overview]
Message-ID: <fff75b65-51ab-cafd-a55f-137c0b7c2dc6@gmail.com> (raw)
In-Reply-To: <20210721070904.15636-1-roger.lu@mediatek.com>

Hi Roger,

On 21/07/2021 09:08, Roger Lu wrote:
> 1. SVS driver uses OPP adjust event in [1] to update OPP table voltage part.
> 2. SVS driver gets thermal/GPU device by node [2][3] and CPU device by get_cpu_device().
> After retrieving subsys device, SVS driver calls device_link_add() to make sure probe/suspend callback priority.
> 3. SVS dts refers to reset controller [4] to help reset SVS HW.
> 
> #mt8183 SVS related patches
> [1] https://patchwork.kernel.org/patch/11193513/
> [2] https://patchwork.kernel.org/project/linux-mediatek/patch/20201013102358.22588-2-michael.kao@mediatek.com/
> [3] https://patchwork.kernel.org/project/linux-mediatek/patch/20200306041345.259332-3-drinkcat@chromium.org/
> 

Comments made in v16 actually also hold for v20, so please take them into account :)

Regards,
Matthias

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

  parent reply	other threads:[~2021-12-30 15:13 UTC|newest]

Thread overview: 48+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-07-21  7:08 [PATCH v20 0/7] soc: mediatek: SVS: introduce MTK SVS engine Roger Lu
2021-07-21  7:08 ` Roger Lu
2021-07-21  7:08 ` Roger Lu
2021-07-21  7:08 ` [PATCH v20 1/7] dt-bindings: soc: mediatek: add mtk svs dt-bindings Roger Lu
2021-07-21  7:08   ` Roger Lu
2021-07-21  7:08   ` Roger Lu
2021-07-21  7:08 ` [PATCH v20 2/7] arm64: dts: mt8183: add svs device information Roger Lu
2021-07-21  7:08   ` Roger Lu
2021-07-21  7:08   ` Roger Lu
2021-07-21  7:09 ` [PATCH v20 3/7] soc: mediatek: SVS: introduce MTK SVS engine Roger Lu
2021-07-21  7:09   ` Roger Lu
2021-07-21  7:09   ` Roger Lu
2021-09-03 14:26   ` Matthias Brugger
2021-09-03 14:26     ` Matthias Brugger
2021-09-03 14:26     ` Matthias Brugger
2021-09-22 13:53     ` mtk11157
2021-09-22 13:53       ` mtk11157
2021-09-22 13:53       ` mtk11157
2022-01-05  9:58   ` AngeloGioacchino Del Regno
2022-01-05  9:58     ` AngeloGioacchino Del Regno
2022-01-05  9:58     ` AngeloGioacchino Del Regno
2022-01-06  5:50     ` Roger Lu
2022-01-06  5:50       ` Roger Lu
2022-01-06  5:50       ` Roger Lu
2021-07-21  7:09 ` [PATCH v20 4/7] soc: mediatek: SVS: add debug commands Roger Lu
2021-07-21  7:09   ` Roger Lu
2021-07-21  7:09   ` Roger Lu
2021-07-21  7:09 ` [PATCH v20 5/7] dt-bindings: soc: mediatek: add mt8192 svs dt-bindings Roger Lu
2021-07-21  7:09   ` Roger Lu
2021-07-21  7:09   ` Roger Lu
2021-12-30 15:30   ` Matthias Brugger
2021-12-30 15:30     ` Matthias Brugger
2021-12-30 15:30     ` Matthias Brugger
2022-01-03  7:45     ` Roger Lu
2022-01-03  7:45       ` Roger Lu
2022-01-03  7:45       ` Roger Lu
2021-07-21  7:09 ` [PATCH v20 6/7] arm64: dts: mt8192: add svs device information Roger Lu
2021-07-21  7:09   ` Roger Lu
2021-07-21  7:09   ` Roger Lu
2021-07-21  7:09 ` [PATCH v20 7/7] soc: mediatek: SVS: add mt8192 SVS GPU driver Roger Lu
2021-07-21  7:09   ` Roger Lu
2021-07-21  7:09   ` Roger Lu
2021-12-30 15:13 ` Matthias Brugger [this message]
2021-12-30 15:13   ` [PATCH v20 0/7] soc: mediatek: SVS: introduce MTK SVS engine Matthias Brugger
2021-12-30 15:13   ` Matthias Brugger
2022-01-03  7:44   ` Roger Lu
2022-01-03  7:44     ` Roger Lu
2022-01-03  7:44     ` Roger Lu

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=fff75b65-51ab-cafd-a55f-137c0b7c2dc6@gmail.com \
    --to=matthias.bgg@gmail.com \
    --cc=Angus.Lin@mediatek.com \
    --cc=Charles.Yang@mediatek.com \
    --cc=HenryC.Chen@mediatek.com \
    --cc=Project_Global_Chrome_Upstream_Group@mediatek.com \
    --cc=Xiaoqing.Liu@mediatek.com \
    --cc=devicetree@vger.kernel.org \
    --cc=drinkcat@google.com \
    --cc=eballetbo@gmail.com \
    --cc=fan.chen@mediatek.com \
    --cc=khilman@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mediatek@lists.infradead.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=linux@roeck-us.net \
    --cc=mark.rutland@arm.com \
    --cc=nm@ti.com \
    --cc=p.zabel@pengutronix.de \
    --cc=robh+dt@kernel.org \
    --cc=roger.lu@mediatek.com \
    --cc=sboyd@kernel.org \
    --cc=yt.lee@mediatek.com \
    /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.