* [PATCH 0/2] drop unused POWER_AVS option for ARM/ARM64 config
@ 2020-11-30 11:27 Andrey Zhizhikin
2020-11-30 11:27 ` [PATCH 1/2] arm64: defconfig: drop unused POWER_AVS option Andrey Zhizhikin
` (2 more replies)
0 siblings, 3 replies; 10+ messages in thread
From: Andrey Zhizhikin @ 2020-11-30 11:27 UTC (permalink / raw)
To: catalin.marinas, will, bjorn.andersson, shawnguo, leoyang.li,
vkoul, geert+renesas, Anson.Huang, michael, krzk, olof,
linux-arm-kernel, linux-kernel
Cc: Andrey Zhizhikin
This series cleans up kernel config files, removing the unused POWER_AVS
option.
It has been removed in commit 785b5bb41b0a ("PM: AVS: Drop the avs
directory and the corresponding Kconfig"), from the tree and
therefore should be removed frorm default config files delivered for ARM
and ARM64 builds.
Andrey Zhizhikin (2):
arm64: defconfig: drop unused POWER_AVS option
ARM: multi_v7_defconfig: drop unused POWER_AVS option
arch/arm/configs/multi_v7_defconfig | 1 -
arch/arm64/configs/defconfig | 1 -
2 files changed, 2 deletions(-)
--
2.17.1
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 1/2] arm64: defconfig: drop unused POWER_AVS option
2020-11-30 11:27 [PATCH 0/2] drop unused POWER_AVS option for ARM/ARM64 config Andrey Zhizhikin
@ 2020-11-30 11:27 ` Andrey Zhizhikin
2020-11-30 12:06 ` Krzysztof Kozlowski
2020-11-30 12:22 ` Nishanth Menon
2020-11-30 11:27 ` [PATCH 2/2] ARM: multi_v7_defconfig: " Andrey Zhizhikin
2020-12-20 15:07 ` [PATCH 0/2] drop unused POWER_AVS option for ARM/ARM64 config ZHIZHIKIN Andrey
2 siblings, 2 replies; 10+ messages in thread
From: Andrey Zhizhikin @ 2020-11-30 11:27 UTC (permalink / raw)
To: catalin.marinas, will, bjorn.andersson, shawnguo, leoyang.li,
vkoul, geert+renesas, Anson.Huang, michael, krzk, olof,
linux-arm-kernel, linux-kernel
Cc: Andrey Zhizhikin, Nishanth Menon, Ulf Hansson
Commit 785b5bb41b0a ("PM: AVS: Drop the avs directory and the
corresponding Kconfig") moved AVS code to SOC-specific folders, and
removed corresponding Kconfig from drivers/power, leaving original
POWER_AVS config option enabled in defconfig file.
Remove the option, which has no references in the tree anymore.
Fixes: 785b5bb41b0a ("PM: AVS: Drop the avs directory and the corresponding Kconfig")
Cc: Nishanth Menon <nm@ti.com>
Cc: Ulf Hansson <ulf.hansson@linaro.org>
Signed-off-by: Andrey Zhizhikin <andrey.zhizhikin@leica-geosystems.com>
---
arch/arm64/configs/defconfig | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
index 5cfe3cf6f2ac..8e3f7ae71de5 100644
--- a/arch/arm64/configs/defconfig
+++ b/arch/arm64/configs/defconfig
@@ -513,7 +513,6 @@ CONFIG_GPIO_PCA953X_IRQ=y
CONFIG_GPIO_BD9571MWV=m
CONFIG_GPIO_MAX77620=y
CONFIG_GPIO_SL28CPLD=m
-CONFIG_POWER_AVS=y
CONFIG_QCOM_CPR=y
CONFIG_ROCKCHIP_IODOMAIN=y
CONFIG_POWER_RESET_MSM=y
--
2.17.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 2/2] ARM: multi_v7_defconfig: drop unused POWER_AVS option
2020-11-30 11:27 [PATCH 0/2] drop unused POWER_AVS option for ARM/ARM64 config Andrey Zhizhikin
2020-11-30 11:27 ` [PATCH 1/2] arm64: defconfig: drop unused POWER_AVS option Andrey Zhizhikin
@ 2020-11-30 11:27 ` Andrey Zhizhikin
2020-11-30 12:22 ` Nishanth Menon
` (2 more replies)
2020-12-20 15:07 ` [PATCH 0/2] drop unused POWER_AVS option for ARM/ARM64 config ZHIZHIKIN Andrey
2 siblings, 3 replies; 10+ messages in thread
From: Andrey Zhizhikin @ 2020-11-30 11:27 UTC (permalink / raw)
To: catalin.marinas, will, bjorn.andersson, shawnguo, leoyang.li,
vkoul, geert+renesas, Anson.Huang, michael, krzk, olof,
linux-arm-kernel, linux-kernel
Cc: Andrey Zhizhikin, Nishanth Menon, Ulf Hansson
Commit 785b5bb41b0a ("PM: AVS: Drop the avs directory and the
corresponding Kconfig") moved AVS code to SOC-specific folders, and
removed corresponding Kconfig from drivers/power, leaving original
POWER_AVS config option enabled in multi_v7_defconfig file.
Remove the option, which has no references in the tree anymore.
Fixes: 785b5bb41b0a ("PM: AVS: Drop the avs directory and the corresponding Kconfig")
Cc: Nishanth Menon <nm@ti.com>
Cc: Ulf Hansson <ulf.hansson@linaro.org>
Signed-off-by: Andrey Zhizhikin <andrey.zhizhikin@leica-geosystems.com>
---
arch/arm/configs/multi_v7_defconfig | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm/configs/multi_v7_defconfig
index a611b0c1e540..91752d918d34 100644
--- a/arch/arm/configs/multi_v7_defconfig
+++ b/arch/arm/configs/multi_v7_defconfig
@@ -466,7 +466,6 @@ CONFIG_GPIO_TPS6586X=y
CONFIG_GPIO_TPS65910=y
CONFIG_GPIO_TWL4030=y
CONFIG_GPIO_MXC=y
-CONFIG_POWER_AVS=y
CONFIG_ROCKCHIP_IODOMAIN=y
CONFIG_POWER_RESET_AS3722=y
CONFIG_POWER_RESET_GPIO=y
--
2.17.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 1/2] arm64: defconfig: drop unused POWER_AVS option
2020-11-30 11:27 ` [PATCH 1/2] arm64: defconfig: drop unused POWER_AVS option Andrey Zhizhikin
@ 2020-11-30 12:06 ` Krzysztof Kozlowski
2020-11-30 12:22 ` Nishanth Menon
1 sibling, 0 replies; 10+ messages in thread
From: Krzysztof Kozlowski @ 2020-11-30 12:06 UTC (permalink / raw)
To: Andrey Zhizhikin
Cc: catalin.marinas, will, bjorn.andersson, shawnguo, leoyang.li,
vkoul, geert+renesas, Anson.Huang, michael, olof,
linux-arm-kernel, linux-kernel, Nishanth Menon, Ulf Hansson
On Mon, Nov 30, 2020 at 11:27:30AM +0000, Andrey Zhizhikin wrote:
> Commit 785b5bb41b0a ("PM: AVS: Drop the avs directory and the
> corresponding Kconfig") moved AVS code to SOC-specific folders, and
> removed corresponding Kconfig from drivers/power, leaving original
> POWER_AVS config option enabled in defconfig file.
>
> Remove the option, which has no references in the tree anymore.
>
> Fixes: 785b5bb41b0a ("PM: AVS: Drop the avs directory and the corresponding Kconfig")
> Cc: Nishanth Menon <nm@ti.com>
> Cc: Ulf Hansson <ulf.hansson@linaro.org>
> Signed-off-by: Andrey Zhizhikin <andrey.zhizhikin@leica-geosystems.com>
> ---
> arch/arm64/configs/defconfig | 1 -
> 1 file changed, 1 deletion(-)
>
Reviewed-by: Krzysztof Kozlowski <krzk@kernel.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 2/2] ARM: multi_v7_defconfig: drop unused POWER_AVS option
2020-11-30 11:27 ` [PATCH 2/2] ARM: multi_v7_defconfig: " Andrey Zhizhikin
@ 2020-11-30 12:22 ` Nishanth Menon
2020-11-30 12:22 ` Krzysztof Kozlowski
2020-11-30 12:56 ` Krzysztof Kozlowski
2 siblings, 0 replies; 10+ messages in thread
From: Nishanth Menon @ 2020-11-30 12:22 UTC (permalink / raw)
To: Andrey Zhizhikin
Cc: catalin.marinas, will, bjorn.andersson, shawnguo, leoyang.li,
vkoul, geert+renesas, Anson.Huang, michael, krzk, olof,
linux-arm-kernel, linux-kernel, Ulf Hansson
On 11:27-20201130, Andrey Zhizhikin wrote:
> Commit 785b5bb41b0a ("PM: AVS: Drop the avs directory and the
> corresponding Kconfig") moved AVS code to SOC-specific folders, and
> removed corresponding Kconfig from drivers/power, leaving original
> POWER_AVS config option enabled in multi_v7_defconfig file.
>
> Remove the option, which has no references in the tree anymore.
>
> Fixes: 785b5bb41b0a ("PM: AVS: Drop the avs directory and the corresponding Kconfig")
> Cc: Nishanth Menon <nm@ti.com>
> Cc: Ulf Hansson <ulf.hansson@linaro.org>
> Signed-off-by: Andrey Zhizhikin <andrey.zhizhikin@leica-geosystems.com>
Reviewed-by: Nishanth Menon <nm@ti.com>
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/2] arm64: defconfig: drop unused POWER_AVS option
2020-11-30 11:27 ` [PATCH 1/2] arm64: defconfig: drop unused POWER_AVS option Andrey Zhizhikin
2020-11-30 12:06 ` Krzysztof Kozlowski
@ 2020-11-30 12:22 ` Nishanth Menon
1 sibling, 0 replies; 10+ messages in thread
From: Nishanth Menon @ 2020-11-30 12:22 UTC (permalink / raw)
To: Andrey Zhizhikin
Cc: catalin.marinas, will, bjorn.andersson, shawnguo, leoyang.li,
vkoul, geert+renesas, Anson.Huang, michael, krzk, olof,
linux-arm-kernel, linux-kernel, Ulf Hansson
On 11:27-20201130, Andrey Zhizhikin wrote:
> Commit 785b5bb41b0a ("PM: AVS: Drop the avs directory and the
> corresponding Kconfig") moved AVS code to SOC-specific folders, and
> removed corresponding Kconfig from drivers/power, leaving original
> POWER_AVS config option enabled in defconfig file.
>
> Remove the option, which has no references in the tree anymore.
>
> Fixes: 785b5bb41b0a ("PM: AVS: Drop the avs directory and the corresponding Kconfig")
> Cc: Nishanth Menon <nm@ti.com>
> Cc: Ulf Hansson <ulf.hansson@linaro.org>
> Signed-off-by: Andrey Zhizhikin <andrey.zhizhikin@leica-geosystems.com>
Reviewed-by: Nishanth Menon <nm@ti.com>
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 2/2] ARM: multi_v7_defconfig: drop unused POWER_AVS option
2020-11-30 11:27 ` [PATCH 2/2] ARM: multi_v7_defconfig: " Andrey Zhizhikin
2020-11-30 12:22 ` Nishanth Menon
@ 2020-11-30 12:22 ` Krzysztof Kozlowski
2020-11-30 12:37 ` ZHIZHIKIN Andrey
2020-11-30 12:56 ` Krzysztof Kozlowski
2 siblings, 1 reply; 10+ messages in thread
From: Krzysztof Kozlowski @ 2020-11-30 12:22 UTC (permalink / raw)
To: Andrey Zhizhikin
Cc: catalin.marinas, will, bjorn.andersson, shawnguo, leoyang.li,
vkoul, geert+renesas, Anson.Huang, michael, olof,
linux-arm-kernel, linux-kernel, Nishanth Menon, Ulf Hansson
On Mon, Nov 30, 2020 at 11:27:31AM +0000, Andrey Zhizhikin wrote:
> Commit 785b5bb41b0a ("PM: AVS: Drop the avs directory and the
> corresponding Kconfig") moved AVS code to SOC-specific folders, and
> removed corresponding Kconfig from drivers/power, leaving original
> POWER_AVS config option enabled in multi_v7_defconfig file.
>
> Remove the option, which has no references in the tree anymore.
>
> Fixes: 785b5bb41b0a ("PM: AVS: Drop the avs directory and the corresponding Kconfig")
> Cc: Nishanth Menon <nm@ti.com>
> Cc: Ulf Hansson <ulf.hansson@linaro.org>
> Signed-off-by: Andrey Zhizhikin <andrey.zhizhikin@leica-geosystems.com>
Thanks for the patch. You should also remove it from the omap2plus config.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 10+ messages in thread
* RE: [PATCH 2/2] ARM: multi_v7_defconfig: drop unused POWER_AVS option
2020-11-30 12:22 ` Krzysztof Kozlowski
@ 2020-11-30 12:37 ` ZHIZHIKIN Andrey
0 siblings, 0 replies; 10+ messages in thread
From: ZHIZHIKIN Andrey @ 2020-11-30 12:37 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: catalin.marinas, will, bjorn.andersson, shawnguo, leoyang.li,
vkoul, geert+renesas, Anson.Huang, michael, olof,
linux-arm-kernel, linux-kernel, Nishanth Menon, Ulf Hansson
Hello Krzysztof,
> -----Original Message-----
> From: Krzysztof Kozlowski <krzk@kernel.org>
> Sent: Monday, November 30, 2020 1:23 PM
> To: ZHIZHIKIN Andrey <andrey.zhizhikin@leica-geosystems.com>
> Cc: catalin.marinas@arm.com; will@kernel.org; bjorn.andersson@linaro.org;
> shawnguo@kernel.org; leoyang.li@nxp.com; vkoul@kernel.org;
> geert+renesas@glider.be; Anson.Huang@nxp.com; michael@walle.cc;
> olof@lixom.net; linux-arm-kernel@lists.infradead.org; linux-
> kernel@vger.kernel.org; Nishanth Menon <nm@ti.com>; Ulf Hansson
> <ulf.hansson@linaro.org>
> Subject: Re: [PATCH 2/2] ARM: multi_v7_defconfig: drop unused
> POWER_AVS option
>
> This email is not from Hexagon’s Office 365 instance. Please be careful while
> clicking links, opening attachments, or replying to this email.
>
>
> On Mon, Nov 30, 2020 at 11:27:31AM +0000, Andrey Zhizhikin wrote:
> > Commit 785b5bb41b0a ("PM: AVS: Drop the avs directory and the
> > corresponding Kconfig") moved AVS code to SOC-specific folders, and
> > removed corresponding Kconfig from drivers/power, leaving original
> > POWER_AVS config option enabled in multi_v7_defconfig file.
> >
> > Remove the option, which has no references in the tree anymore.
> >
> > Fixes: 785b5bb41b0a ("PM: AVS: Drop the avs directory and the
> > corresponding Kconfig")
> > Cc: Nishanth Menon <nm@ti.com>
> > Cc: Ulf Hansson <ulf.hansson@linaro.org>
> > Signed-off-by: Andrey Zhizhikin
> > <andrey.zhizhikin@leica-geosystems.com>
>
> Thanks for the patch. You should also remove it from the omap2plus config.
True, just realized this config option is present in arch/arm/configs/omap2plus_defconfig. Thanks for pointing this out, would submit a separate patch to take care of it now.
>
> Best regards,
> Krzysztof
Regards,
Andrey
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 2/2] ARM: multi_v7_defconfig: drop unused POWER_AVS option
2020-11-30 11:27 ` [PATCH 2/2] ARM: multi_v7_defconfig: " Andrey Zhizhikin
2020-11-30 12:22 ` Nishanth Menon
2020-11-30 12:22 ` Krzysztof Kozlowski
@ 2020-11-30 12:56 ` Krzysztof Kozlowski
2 siblings, 0 replies; 10+ messages in thread
From: Krzysztof Kozlowski @ 2020-11-30 12:56 UTC (permalink / raw)
To: Andrey Zhizhikin
Cc: catalin.marinas, will, bjorn.andersson, shawnguo, leoyang.li,
vkoul, geert+renesas, Anson.Huang, michael, olof,
linux-arm-kernel, linux-kernel, Nishanth Menon, Ulf Hansson
On Mon, Nov 30, 2020 at 11:27:31AM +0000, Andrey Zhizhikin wrote:
> Commit 785b5bb41b0a ("PM: AVS: Drop the avs directory and the
> corresponding Kconfig") moved AVS code to SOC-specific folders, and
> removed corresponding Kconfig from drivers/power, leaving original
> POWER_AVS config option enabled in multi_v7_defconfig file.
>
> Remove the option, which has no references in the tree anymore.
>
> Fixes: 785b5bb41b0a ("PM: AVS: Drop the avs directory and the corresponding Kconfig")
> Cc: Nishanth Menon <nm@ti.com>
> Cc: Ulf Hansson <ulf.hansson@linaro.org>
> Signed-off-by: Andrey Zhizhikin <andrey.zhizhikin@leica-geosystems.com>
> ---
> arch/arm/configs/multi_v7_defconfig | 1 -
> 1 file changed, 1 deletion(-)
>
Reviewed-by: Krzysztof Kozlowski <krzk@kernel.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 10+ messages in thread
* RE: [PATCH 0/2] drop unused POWER_AVS option for ARM/ARM64 config
2020-11-30 11:27 [PATCH 0/2] drop unused POWER_AVS option for ARM/ARM64 config Andrey Zhizhikin
2020-11-30 11:27 ` [PATCH 1/2] arm64: defconfig: drop unused POWER_AVS option Andrey Zhizhikin
2020-11-30 11:27 ` [PATCH 2/2] ARM: multi_v7_defconfig: " Andrey Zhizhikin
@ 2020-12-20 15:07 ` ZHIZHIKIN Andrey
2 siblings, 0 replies; 10+ messages in thread
From: ZHIZHIKIN Andrey @ 2020-12-20 15:07 UTC (permalink / raw)
To: catalin.marinas, will, bjorn.andersson, shawnguo, leoyang.li,
vkoul, geert+renesas, Anson.Huang, michael, krzk, olof,
linux-arm-kernel, linux-kernel, Arnd Bergmann, soc
Hello Arnd,
A while ago, I've submitted this series to clean-up the unused POWER_AVS config option, and
It looks like it did not land in anyone's tree until now.
Similar patch to clean-up the same option in omap2plus_defconfig has been already picked up
by Tony Lindgren and is already present in the linux-next tree, commit 0437141b4e22 ("ARM:
configs: drop unused BACKLIGHT_GENERIC option").
Series has been already reviewed by Krzysztof and Nishanth, but nobody seems to pick it up further.
Since at the time of sending this series, I was not aware that ARM SOC list should also be included
(same thing as for BACKLIGHT_GENERIC series [1]), I wanted to ask you if you can take care of this
series further as it does touch the multi config for arm32 and aarch64.
Or should I re-send the series with collected tags and include ARM SOC list (same as I did for
BACKLIGHT_GENERIC [1])?
> -----Original Message-----
> From: ZHIZHIKIN Andrey <andrey.zhizhikin@leica-geosystems.com>
> Sent: Monday, November 30, 2020 12:27 PM
> To: catalin.marinas@arm.com; will@kernel.org; bjorn.andersson@linaro.org;
> shawnguo@kernel.org; leoyang.li@nxp.com; vkoul@kernel.org;
> geert+renesas@glider.be; Anson.Huang@nxp.com; michael@walle.cc;
> krzk@kernel.org; olof@lixom.net; linux-arm-kernel@lists.infradead.org; linux-
> kernel@vger.kernel.org
> Cc: ZHIZHIKIN Andrey <andrey.zhizhikin@leica-geosystems.com>
> Subject: [PATCH 0/2] drop unused POWER_AVS option for ARM/ARM64 config
>
> This series cleans up kernel config files, removing the unused POWER_AVS
> option.
>
> It has been removed in commit 785b5bb41b0a ("PM: AVS: Drop the avs
> directory and the corresponding Kconfig"), from the tree and
> therefore should be removed frorm default config files delivered for ARM
> and ARM64 builds.
>
> Andrey Zhizhikin (2):
> arm64: defconfig: drop unused POWER_AVS option
> ARM: multi_v7_defconfig: drop unused POWER_AVS option
>
> arch/arm/configs/multi_v7_defconfig | 1 -
> arch/arm64/configs/defconfig | 1 -
> 2 files changed, 2 deletions(-)
>
> --
> 2.17.1
Link: [1]: https://lore.kernel.org/linux-arm-kernel/20201201222922.3183-1-andrey.zhizhikin@leica-geosystems.com/
-- Andrey
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2020-12-20 15:09 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-11-30 11:27 [PATCH 0/2] drop unused POWER_AVS option for ARM/ARM64 config Andrey Zhizhikin
2020-11-30 11:27 ` [PATCH 1/2] arm64: defconfig: drop unused POWER_AVS option Andrey Zhizhikin
2020-11-30 12:06 ` Krzysztof Kozlowski
2020-11-30 12:22 ` Nishanth Menon
2020-11-30 11:27 ` [PATCH 2/2] ARM: multi_v7_defconfig: " Andrey Zhizhikin
2020-11-30 12:22 ` Nishanth Menon
2020-11-30 12:22 ` Krzysztof Kozlowski
2020-11-30 12:37 ` ZHIZHIKIN Andrey
2020-11-30 12:56 ` Krzysztof Kozlowski
2020-12-20 15:07 ` [PATCH 0/2] drop unused POWER_AVS option for ARM/ARM64 config ZHIZHIKIN Andrey
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).