* [PATCH v1] soc: mediatek: mtk-pm-domains: Allow mt8186 ADSP default power on
@ 2022-10-12 7:54 Tinghan Shen
2022-10-13 8:23 ` AngeloGioacchino Del Regno
` (2 more replies)
0 siblings, 3 replies; 7+ messages in thread
From: Tinghan Shen @ 2022-10-12 7:54 UTC (permalink / raw)
To: Matthias Brugger
Cc: allen-kh.cheng, linux-arm-kernel, linux-mediatek, linux-kernel,
Project_Global_Chrome_Upstream_Group, Tinghan Shen
In the use case of configuring the access permissions of the ADSP core,
the mt8186 SoC ADSP power will be switched on in the bootloader because
the permission control registers are located in the ADSP subsys.
Signed-off-by: Tinghan Shen <tinghan.shen@mediatek.com>
---
drivers/soc/mediatek/mt8186-pm-domains.h | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/soc/mediatek/mt8186-pm-domains.h b/drivers/soc/mediatek/mt8186-pm-domains.h
index 108af61854a3..fce86f79c505 100644
--- a/drivers/soc/mediatek/mt8186-pm-domains.h
+++ b/drivers/soc/mediatek/mt8186-pm-domains.h
@@ -304,7 +304,6 @@ static const struct scpsys_domain_data scpsys_domain_data_mt8186[] = {
.ctl_offs = 0x9FC,
.pwr_sta_offs = 0x16C,
.pwr_sta2nd_offs = 0x170,
- .caps = MTK_SCPD_KEEP_DEFAULT_OFF,
},
[MT8186_POWER_DOMAIN_ADSP_INFRA] = {
.name = "adsp_infra",
@@ -312,7 +311,6 @@ static const struct scpsys_domain_data scpsys_domain_data_mt8186[] = {
.ctl_offs = 0x9F8,
.pwr_sta_offs = 0x16C,
.pwr_sta2nd_offs = 0x170,
- .caps = MTK_SCPD_KEEP_DEFAULT_OFF,
},
[MT8186_POWER_DOMAIN_ADSP_TOP] = {
.name = "adsp_top",
@@ -332,7 +330,7 @@ static const struct scpsys_domain_data scpsys_domain_data_mt8186[] = {
MT8186_TOP_AXI_PROT_EN_3_CLR,
MT8186_TOP_AXI_PROT_EN_3_STA),
},
- .caps = MTK_SCPD_SRAM_ISO | MTK_SCPD_KEEP_DEFAULT_OFF | MTK_SCPD_ACTIVE_WAKEUP,
+ .caps = MTK_SCPD_SRAM_ISO | MTK_SCPD_ACTIVE_WAKEUP,
},
};
--
2.18.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 7+ messages in thread* Re: [PATCH v1] soc: mediatek: mtk-pm-domains: Allow mt8186 ADSP default power on
2022-10-12 7:54 [PATCH v1] soc: mediatek: mtk-pm-domains: Allow mt8186 ADSP default power on Tinghan Shen
@ 2022-10-13 8:23 ` AngeloGioacchino Del Regno
2022-11-15 11:56 ` TingHan Shen (沈廷翰)
2022-11-23 10:20 ` Matthias Brugger
2 siblings, 0 replies; 7+ messages in thread
From: AngeloGioacchino Del Regno @ 2022-10-13 8:23 UTC (permalink / raw)
To: Tinghan Shen, Matthias Brugger
Cc: allen-kh.cheng, linux-arm-kernel, linux-mediatek, linux-kernel,
Project_Global_Chrome_Upstream_Group
Il 12/10/22 09:54, Tinghan Shen ha scritto:
> In the use case of configuring the access permissions of the ADSP core,
> the mt8186 SoC ADSP power will be switched on in the bootloader because
> the permission control registers are located in the ADSP subsys.
>
> Signed-off-by: Tinghan Shen <tinghan.shen@mediatek.com>
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v1] soc: mediatek: mtk-pm-domains: Allow mt8186 ADSP default power on
2022-10-12 7:54 [PATCH v1] soc: mediatek: mtk-pm-domains: Allow mt8186 ADSP default power on Tinghan Shen
2022-10-13 8:23 ` AngeloGioacchino Del Regno
@ 2022-11-15 11:56 ` TingHan Shen (沈廷翰)
2022-11-23 10:20 ` Matthias Brugger
2 siblings, 0 replies; 7+ messages in thread
From: TingHan Shen (沈廷翰) @ 2022-11-15 11:56 UTC (permalink / raw)
To: matthias.bgg@gmail.com
Cc: linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, linux-mediatek@lists.infradead.org,
Allen-KH Cheng (程冠勳),
Project_Global_Chrome_Upstream_Group
Hi Matthias,
Gently ping for this patch.
May I know this patch current status? It seems is under your reviewing state?
Best regards,
TingHan
On Wed, 2022-10-12 at 15:54 +0800, Tinghan Shen wrote:
> In the use case of configuring the access permissions of the ADSP core,
> the mt8186 SoC ADSP power will be switched on in the bootloader because
> the permission control registers are located in the ADSP subsys.
>
> Signed-off-by: Tinghan Shen <tinghan.shen@mediatek.com>
> ---
> drivers/soc/mediatek/mt8186-pm-domains.h | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/drivers/soc/mediatek/mt8186-pm-domains.h b/drivers/soc/mediatek/mt8186-pm-domains.h
> index 108af61854a3..fce86f79c505 100644
> --- a/drivers/soc/mediatek/mt8186-pm-domains.h
> +++ b/drivers/soc/mediatek/mt8186-pm-domains.h
> @@ -304,7 +304,6 @@ static const struct scpsys_domain_data scpsys_domain_data_mt8186[] = {
> .ctl_offs = 0x9FC,
> .pwr_sta_offs = 0x16C,
> .pwr_sta2nd_offs = 0x170,
> - .caps = MTK_SCPD_KEEP_DEFAULT_OFF,
> },
> [MT8186_POWER_DOMAIN_ADSP_INFRA] = {
> .name = "adsp_infra",
> @@ -312,7 +311,6 @@ static const struct scpsys_domain_data scpsys_domain_data_mt8186[] = {
> .ctl_offs = 0x9F8,
> .pwr_sta_offs = 0x16C,
> .pwr_sta2nd_offs = 0x170,
> - .caps = MTK_SCPD_KEEP_DEFAULT_OFF,
> },
> [MT8186_POWER_DOMAIN_ADSP_TOP] = {
> .name = "adsp_top",
> @@ -332,7 +330,7 @@ static const struct scpsys_domain_data scpsys_domain_data_mt8186[] = {
> MT8186_TOP_AXI_PROT_EN_3_CLR,
> MT8186_TOP_AXI_PROT_EN_3_STA),
> },
> - .caps = MTK_SCPD_SRAM_ISO | MTK_SCPD_KEEP_DEFAULT_OFF | MTK_SCPD_ACTIVE_WAKEUP,
> + .caps = MTK_SCPD_SRAM_ISO | MTK_SCPD_ACTIVE_WAKEUP,
> },
> };
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH v1] soc: mediatek: mtk-pm-domains: Allow mt8186 ADSP default power on
2022-10-12 7:54 [PATCH v1] soc: mediatek: mtk-pm-domains: Allow mt8186 ADSP default power on Tinghan Shen
2022-10-13 8:23 ` AngeloGioacchino Del Regno
2022-11-15 11:56 ` TingHan Shen (沈廷翰)
@ 2022-11-23 10:20 ` Matthias Brugger
2022-11-23 10:40 ` TingHan Shen (沈廷翰)
2 siblings, 1 reply; 7+ messages in thread
From: Matthias Brugger @ 2022-11-23 10:20 UTC (permalink / raw)
To: Tinghan Shen
Cc: allen-kh.cheng, linux-arm-kernel, linux-mediatek, linux-kernel,
Project_Global_Chrome_Upstream_Group
On 12/10/2022 09:54, Tinghan Shen wrote:
> In the use case of configuring the access permissions of the ADSP core,
> the mt8186 SoC ADSP power will be switched on in the bootloader because
> the permission control registers are located in the ADSP subsys.
>
> Signed-off-by: Tinghan Shen <tinghan.shen@mediatek.com>
Do we need a fixes tag here?
My bet is:
Fixes: 88590cbc1703 ("soc: mediatek: pm-domains: Add support for mt8186")
Or is this not yet a problem but needed to be able to implement access
permission configuration in the ADSP driver?
Regards,
Matthias
> ---
> drivers/soc/mediatek/mt8186-pm-domains.h | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/drivers/soc/mediatek/mt8186-pm-domains.h b/drivers/soc/mediatek/mt8186-pm-domains.h
> index 108af61854a3..fce86f79c505 100644
> --- a/drivers/soc/mediatek/mt8186-pm-domains.h
> +++ b/drivers/soc/mediatek/mt8186-pm-domains.h
> @@ -304,7 +304,6 @@ static const struct scpsys_domain_data scpsys_domain_data_mt8186[] = {
> .ctl_offs = 0x9FC,
> .pwr_sta_offs = 0x16C,
> .pwr_sta2nd_offs = 0x170,
> - .caps = MTK_SCPD_KEEP_DEFAULT_OFF,
> },
> [MT8186_POWER_DOMAIN_ADSP_INFRA] = {
> .name = "adsp_infra",
> @@ -312,7 +311,6 @@ static const struct scpsys_domain_data scpsys_domain_data_mt8186[] = {
> .ctl_offs = 0x9F8,
> .pwr_sta_offs = 0x16C,
> .pwr_sta2nd_offs = 0x170,
> - .caps = MTK_SCPD_KEEP_DEFAULT_OFF,
> },
> [MT8186_POWER_DOMAIN_ADSP_TOP] = {
> .name = "adsp_top",
> @@ -332,7 +330,7 @@ static const struct scpsys_domain_data scpsys_domain_data_mt8186[] = {
> MT8186_TOP_AXI_PROT_EN_3_CLR,
> MT8186_TOP_AXI_PROT_EN_3_STA),
> },
> - .caps = MTK_SCPD_SRAM_ISO | MTK_SCPD_KEEP_DEFAULT_OFF | MTK_SCPD_ACTIVE_WAKEUP,
> + .caps = MTK_SCPD_SRAM_ISO | MTK_SCPD_ACTIVE_WAKEUP,
> },
> };
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH v1] soc: mediatek: mtk-pm-domains: Allow mt8186 ADSP default power on
2022-11-23 10:20 ` Matthias Brugger
@ 2022-11-23 10:40 ` TingHan Shen (沈廷翰)
2022-12-02 11:13 ` TingHan Shen (沈廷翰)
0 siblings, 1 reply; 7+ messages in thread
From: TingHan Shen (沈廷翰) @ 2022-11-23 10:40 UTC (permalink / raw)
To: matthias.bgg@gmail.com
Cc: linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, linux-mediatek@lists.infradead.org,
Allen-KH Cheng (程冠勳),
Project_Global_Chrome_Upstream_Group
On Wed, 2022-11-23 at 11:20 +0100, Matthias Brugger wrote:
>
> On 12/10/2022 09:54, Tinghan Shen wrote:
> > In the use case of configuring the access permissions of the ADSP core,
> > the mt8186 SoC ADSP power will be switched on in the bootloader because
> > the permission control registers are located in the ADSP subsys.
> >
> > Signed-off-by: Tinghan Shen <tinghan.shen@mediatek.com>
>
> Do we need a fixes tag here?
> My bet is:
> Fixes: 88590cbc1703 ("soc: mediatek: pm-domains: Add support for mt8186")
>
> Or is this not yet a problem but needed to be able to implement access
> permission configuration in the ADSP driver?
>
> Regards,
> Matthias
This patch is needed to fix a warning when implementing the ADSP access permission configuration in preloader.
[ 0.334154] mtk-power-controller 10006000.syscon:power-controller: /soc/syscon@10006000/power-controller/power-domain@18/power-domain@19/power-domain@20: A default off power domain has been
ON
[ 0.334246] ------------[ cut here ]------------
[ 0.334252] top_adsp_bus already disabled
[ 0.334274] WARNING: CPU: 5 PID: 113 at drivers/clk/clk.c:969 clk_core_disable+0x90/0xb0
If not for this case, the pm-domain works fine.
Best regards,
TingHan
>
> > ---
> > drivers/soc/mediatek/mt8186-pm-domains.h | 4 +---
> > 1 file changed, 1 insertion(+), 3 deletions(-)
> >
> > diff --git a/drivers/soc/mediatek/mt8186-pm-domains.h b/drivers/soc/mediatek/mt8186-pm-domains.h
> > index 108af61854a3..fce86f79c505 100644
> > --- a/drivers/soc/mediatek/mt8186-pm-domains.h
> > +++ b/drivers/soc/mediatek/mt8186-pm-domains.h
> > @@ -304,7 +304,6 @@ static const struct scpsys_domain_data scpsys_domain_data_mt8186[] = {
> > .ctl_offs = 0x9FC,
> > .pwr_sta_offs = 0x16C,
> > .pwr_sta2nd_offs = 0x170,
> > - .caps = MTK_SCPD_KEEP_DEFAULT_OFF,
> > },
> > [MT8186_POWER_DOMAIN_ADSP_INFRA] = {
> > .name = "adsp_infra",
> > @@ -312,7 +311,6 @@ static const struct scpsys_domain_data scpsys_domain_data_mt8186[] = {
> > .ctl_offs = 0x9F8,
> > .pwr_sta_offs = 0x16C,
> > .pwr_sta2nd_offs = 0x170,
> > - .caps = MTK_SCPD_KEEP_DEFAULT_OFF,
> > },
> > [MT8186_POWER_DOMAIN_ADSP_TOP] = {
> > .name = "adsp_top",
> > @@ -332,7 +330,7 @@ static const struct scpsys_domain_data scpsys_domain_data_mt8186[] = {
> > MT8186_TOP_AXI_PROT_EN_3_CLR,
> > MT8186_TOP_AXI_PROT_EN_3_STA),
> > },
> > - .caps = MTK_SCPD_SRAM_ISO | MTK_SCPD_KEEP_DEFAULT_OFF | MTK_SCPD_ACTIVE_WAKEUP,
> > + .caps = MTK_SCPD_SRAM_ISO | MTK_SCPD_ACTIVE_WAKEUP,
> > },
> > };
> >
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH v1] soc: mediatek: mtk-pm-domains: Allow mt8186 ADSP default power on
2022-11-23 10:40 ` TingHan Shen (沈廷翰)
@ 2022-12-02 11:13 ` TingHan Shen (沈廷翰)
2022-12-16 12:44 ` Matthias Brugger
0 siblings, 1 reply; 7+ messages in thread
From: TingHan Shen (沈廷翰) @ 2022-12-02 11:13 UTC (permalink / raw)
To: matthias.bgg@gmail.com
Cc: linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, linux-mediatek@lists.infradead.org,
Allen-KH Cheng (程冠勳),
Project_Global_Chrome_Upstream_Group
Hi Matthias,
On Wed, 2022-11-23 at 18:40 +0800, TingHan Shen wrote:
> On Wed, 2022-11-23 at 11:20 +0100, Matthias Brugger wrote:
> >
> > On 12/10/2022 09:54, Tinghan Shen wrote:
> > > In the use case of configuring the access permissions of the ADSP core,
> > > the mt8186 SoC ADSP power will be switched on in the bootloader because
> > > the permission control registers are located in the ADSP subsys.
> > >
> > > Signed-off-by: Tinghan Shen <tinghan.shen@mediatek.com>
> >
> > Do we need a fixes tag here?
> > My bet is:
> > Fixes: 88590cbc1703 ("soc: mediatek: pm-domains: Add support for mt8186")
> >
> > Or is this not yet a problem but needed to be able to implement access
> > permission configuration in the ADSP driver?
> >
> > Regards,
> > Matthias
>
> This patch is needed to fix a warning when implementing the ADSP access permission configuration in preloader.
>
> [ 0.334154] mtk-power-controller 10006000.syscon:power-controller: /soc/syscon@10006000/power-controller/power-domain@18/power-domain@19/power-domain@20: A default off power domain has been
> ON
> [ 0.334246] ------------[ cut here ]------------
> [ 0.334252] top_adsp_bus already disabled
> [ 0.334274] WARNING: CPU: 5 PID: 113 at drivers/clk/clk.c:969 clk_core_disable+0x90/0xb0
>
> If not for this case, the pm-domain works fine.
>
>
> Best regards,
> TingHan
gentle ping for this.
from above descriptions, should I add the fixes tag to this patch?
Best regards,
TingHan
>
> >
> > > ---
> > > drivers/soc/mediatek/mt8186-pm-domains.h | 4 +---
> > > 1 file changed, 1 insertion(+), 3 deletions(-)
> > >
> > > diff --git a/drivers/soc/mediatek/mt8186-pm-domains.h b/drivers/soc/mediatek/mt8186-pm-domains.h
> > > index 108af61854a3..fce86f79c505 100644
> > > --- a/drivers/soc/mediatek/mt8186-pm-domains.h
> > > +++ b/drivers/soc/mediatek/mt8186-pm-domains.h
> > > @@ -304,7 +304,6 @@ static const struct scpsys_domain_data scpsys_domain_data_mt8186[] = {
> > > .ctl_offs = 0x9FC,
> > > .pwr_sta_offs = 0x16C,
> > > .pwr_sta2nd_offs = 0x170,
> > > - .caps = MTK_SCPD_KEEP_DEFAULT_OFF,
> > > },
> > > [MT8186_POWER_DOMAIN_ADSP_INFRA] = {
> > > .name = "adsp_infra",
> > > @@ -312,7 +311,6 @@ static const struct scpsys_domain_data scpsys_domain_data_mt8186[] = {
> > > .ctl_offs = 0x9F8,
> > > .pwr_sta_offs = 0x16C,
> > > .pwr_sta2nd_offs = 0x170,
> > > - .caps = MTK_SCPD_KEEP_DEFAULT_OFF,
> > > },
> > > [MT8186_POWER_DOMAIN_ADSP_TOP] = {
> > > .name = "adsp_top",
> > > @@ -332,7 +330,7 @@ static const struct scpsys_domain_data scpsys_domain_data_mt8186[] = {
> > > MT8186_TOP_AXI_PROT_EN_3_CLR,
> > > MT8186_TOP_AXI_PROT_EN_3_STA),
> > > },
> > > - .caps = MTK_SCPD_SRAM_ISO | MTK_SCPD_KEEP_DEFAULT_OFF | MTK_SCPD_ACTIVE_WAKEUP,
> > > + .caps = MTK_SCPD_SRAM_ISO | MTK_SCPD_ACTIVE_WAKEUP,
> > > },
> > > };
> > >
>
>
--
Best regards,
TingHan
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH v1] soc: mediatek: mtk-pm-domains: Allow mt8186 ADSP default power on
2022-12-02 11:13 ` TingHan Shen (沈廷翰)
@ 2022-12-16 12:44 ` Matthias Brugger
0 siblings, 0 replies; 7+ messages in thread
From: Matthias Brugger @ 2022-12-16 12:44 UTC (permalink / raw)
To: TingHan Shen (沈廷翰)
Cc: linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, linux-mediatek@lists.infradead.org,
Allen-KH Cheng (程冠勳),
Project_Global_Chrome_Upstream_Group
On 02/12/2022 12:13, TingHan Shen (沈廷翰) wrote:
> Hi Matthias,
>
> On Wed, 2022-11-23 at 18:40 +0800, TingHan Shen wrote:
>> On Wed, 2022-11-23 at 11:20 +0100, Matthias Brugger wrote:
>> >
>> > On 12/10/2022 09:54, Tinghan Shen wrote:
>> > > In the use case of configuring the access permissions of the ADSP core,
>> > > the mt8186 SoC ADSP power will be switched on in the bootloader because
>> > > the permission control registers are located in the ADSP subsys.
>> > >
>> > > Signed-off-by: Tinghan Shen <tinghan.shen@mediatek.com>
>> >
>> > Do we need a fixes tag here?
>> > My bet is:
>> > Fixes: 88590cbc1703 ("soc: mediatek: pm-domains: Add support for mt8186")
>> >
>> > Or is this not yet a problem but needed to be able to implement access
>> > permission configuration in the ADSP driver?
>> >
>> > Regards,
>> > Matthias
>>
>> This patch is needed to fix a warning when implementing the ADSP access permission configuration in preloader.
>>
>> [ 0.334154] mtk-power-controller 10006000.syscon:power-controller: /soc/syscon@10006000/power-controller/power-domain@18/power-domain@19/power-domain@20: A default off power domain has been
>> ON
>> [ 0.334246] ------------[ cut here ]------------
>> [ 0.334252] top_adsp_bus already disabled
>> [ 0.334274] WARNING: CPU: 5 PID: 113 at drivers/clk/clk.c:969 clk_core_disable+0x90/0xb0
>>
>> If not for this case, the pm-domain works fine.
>>
>>
>> Best regards,
>> TingHan
>
> gentle ping for this.
> from above descriptions, should I add the fixes tag to this patch?
>
No that's not needed in this case. Applied now.
Matthias
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2022-12-16 12:46 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-10-12 7:54 [PATCH v1] soc: mediatek: mtk-pm-domains: Allow mt8186 ADSP default power on Tinghan Shen
2022-10-13 8:23 ` AngeloGioacchino Del Regno
2022-11-15 11:56 ` TingHan Shen (沈廷翰)
2022-11-23 10:20 ` Matthias Brugger
2022-11-23 10:40 ` TingHan Shen (沈廷翰)
2022-12-02 11:13 ` TingHan Shen (沈廷翰)
2022-12-16 12:44 ` Matthias Brugger
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).