Re: [PATCH v14 00/10] cpuidle driver for QCOM SoCs: 8064, 8074, 8084

From: Lina Iyer <lina.iyer@linaro.org>
To: Kevin Hilman <khilman@kernel.org>
Cc: daniel.lezcano@linaro.org, sboyd@codeaurora.org,
	galak@codeaurora.org, linux-arm-msm@vger.kernel.org,
	linux-pm@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	lorenzo.pieralisi@arm.com, msivasub@codeaurora.org,
	devicetree@vger.kernel.org
Subject: Re: [PATCH v14 00/10] cpuidle driver for QCOM SoCs: 8064, 8074, 8084
Date: Wed, 17 Dec 2014 11:25:09 -0700	[thread overview]
Message-ID: <20141217182509.GA1913@linaro.org> (raw)
In-Reply-To: <7hegryb0te.fsf@deeprootsystems.com>

On Wed, Dec 17 2014 at 11:14 -0700, Kevin Hilman wrote:
>Lina Iyer <lina.iyer@linaro.org> writes:
>
>> Dependent patchsets -
>>         https://lkml.org/lkml/2014/8/4/767
>>         http://www.spinics.net/lists/linux-arm-msm/msg10799.html
>>         http://www.spinics.net/lists/linux-arm-msm/msg10795.html
>>
>> Changes since v13:
>> - Return values for idle states propagated back to the cpuidle driver.
>> - Remove static bool cpuidle_drv_init.
>> - Register cpuidle driver and cpuidle device separately.
>>  - cpuidle device registered only when the SPM for the cpu is probed.
>> - Initialization changes to ensure dynamic cpuidle devices are registered only
>>   after the cpuidle driver is registered.
>> - Removed wmb, replaced with a poll loop to ensure that the SPM registers are
>>   written before executing wfi.
>>   - Added spm_register_write_sync() for write guarantees.
>> - Removed irrelevant return value for spm_set_low_power_mode().
>> - Added comments, updated module description.
>> - Removed Reviewed-by and Acked-by on the spm and cpuidle-qcom patches.
>
>I think this series should also include updates to qcom_defconfig that
>enable QCOM_PM and the cpuidle driver.
>
>Then it will be enabled by default and get more broad testing.

Sure. I will be enabling them in the next revision of the series.

>
>Kevin