From: Daniel Lezcano <daniel.lezcano@linaro.org>
To: "Rafael J. Wysocki" <rjw@rjwysocki.net>,
Krzysztof Kozlowski <krzk@kernel.org>
Cc: linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org,
Nicolas Ferre <nicolas.ferre@microchip.com>,
Alexandre Belloni <alexandre.belloni@bootlin.com>,
Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>,
Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>,
Sudeep Holla <sudeep.holla@arm.com>,
linux-arm-kernel@lists.infradead.org,
linux-samsung-soc@vger.kernel.org
Subject: Re: [PATCH] cpuidle: arm: Enable compile testing for some of drivers
Date: Mon, 13 Jan 2020 12:58:17 +0100 [thread overview]
Message-ID: <a4fa3f89-e792-aeee-b9ea-9af244ace04a@linaro.org> (raw)
In-Reply-To: <112783298.KOQPr5xTch@kreacher>
On 13/01/2020 12:51, Rafael J. Wysocki wrote:
> On Sunday, December 29, 2019 7:09:12 PM CET Krzysztof Kozlowski wrote:
>> Some of cpuidle drivers for ARMv7 can be compile tested on this
>> architecture because they do not depend on mach-specific bits. Enable
>> compile testing for big.LITTLE, Kirkwood, Zynq, AT91, Exynos and mvebu
>> cpuidle drivers.
>>
>> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
>> ---
>> drivers/cpuidle/Kconfig.arm | 12 ++++++------
>> 1 file changed, 6 insertions(+), 6 deletions(-)
>>
>> diff --git a/drivers/cpuidle/Kconfig.arm b/drivers/cpuidle/Kconfig.arm
>> index a224d33dda7f..62272ecfa771 100644
>> --- a/drivers/cpuidle/Kconfig.arm
>> +++ b/drivers/cpuidle/Kconfig.arm
>> @@ -25,7 +25,7 @@ config ARM_PSCI_CPUIDLE
>>
>> config ARM_BIG_LITTLE_CPUIDLE
>> bool "Support for ARM big.LITTLE processors"
>> - depends on ARCH_VEXPRESS_TC2_PM || ARCH_EXYNOS
>> + depends on ARCH_VEXPRESS_TC2_PM || ARCH_EXYNOS || COMPILE_TEST
>> depends on MCPM && !ARM64
>> select ARM_CPU_SUSPEND
>> select CPU_IDLE_MULTIPLE_DRIVERS
>> @@ -51,13 +51,13 @@ config ARM_HIGHBANK_CPUIDLE
>>
>> config ARM_KIRKWOOD_CPUIDLE
>> bool "CPU Idle Driver for Marvell Kirkwood SoCs"
>> - depends on MACH_KIRKWOOD && !ARM64
>> + depends on (MACH_KIRKWOOD || COMPILE_TEST) && !ARM64
>> help
>> This adds the CPU Idle driver for Marvell Kirkwood SoCs.
>>
>> config ARM_ZYNQ_CPUIDLE
>> bool "CPU Idle Driver for Xilinx Zynq processors"
>> - depends on ARCH_ZYNQ && !ARM64
>> + depends on (ARCH_ZYNQ || COMPILE_TEST) && !ARM64
>> help
>> Select this to enable cpuidle on Xilinx Zynq processors.
>>
>> @@ -70,19 +70,19 @@ config ARM_U8500_CPUIDLE
>> config ARM_AT91_CPUIDLE
>> bool "Cpu Idle Driver for the AT91 processors"
>> default y
>> - depends on ARCH_AT91 && !ARM64
>> + depends on (ARCH_AT91 || COMPILE_TEST) && !ARM64
>> help
>> Select this to enable cpuidle for AT91 processors.
>>
>> config ARM_EXYNOS_CPUIDLE
>> bool "Cpu Idle Driver for the Exynos processors"
>> - depends on ARCH_EXYNOS && !ARM64
>> + depends on (ARCH_EXYNOS || COMPILE_TEST) && !ARM64
>> select ARCH_NEEDS_CPU_IDLE_COUPLED if SMP
>> help
>> Select this to enable cpuidle for Exynos processors.
>>
>> config ARM_MVEBU_V7_CPUIDLE
>> bool "CPU Idle Driver for mvebu v7 family processors"
>> - depends on ARCH_MVEBU && !ARM64
>> + depends on (ARCH_MVEBU || COMPILE_TEST) && !ARM64
>> help
>> Select this to enable cpuidle on Armada 370, 38x and XP processors.
>>
>
> Daniel, any concerns regarding this one?
Yes, I have a doubt about this patch. I'll double check before commenting.
--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog
next prev parent reply other threads:[~2020-01-13 11:58 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-12-29 18:09 [PATCH] cpuidle: arm: Enable compile testing for some of drivers Krzysztof Kozlowski
2019-12-29 18:09 ` Krzysztof Kozlowski
2020-01-13 11:51 ` Rafael J. Wysocki
2020-01-13 11:58 ` Daniel Lezcano [this message]
2020-01-13 12:24 ` Daniel Lezcano
2020-01-14 22:51 ` Rafael J. Wysocki
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=a4fa3f89-e792-aeee-b9ea-9af244ace04a@linaro.org \
--to=daniel.lezcano@linaro.org \
--cc=alexandre.belloni@bootlin.com \
--cc=b.zolnierkie@samsung.com \
--cc=krzk@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=lorenzo.pieralisi@arm.com \
--cc=nicolas.ferre@microchip.com \
--cc=rjw@rjwysocki.net \
--cc=sudeep.holla@arm.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 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).