2015-06-01 18:35:51

by Arnd Bergmann

[permalink] [raw]
Subject: Re: [GIT PULL] qcom SoC changes for 4.2-1

On Friday 29 May 2015 14:42:01 Arnd Bergmann wrote:
> On Thursday 28 May 2015 10:55:39 Kumar Gala wrote:
> > Qualcomm ARM Based SoC Updates for v4.2-1
> >
> > * Added Subsystem Power Manager (SPM) driver
> > * Split out 32-bit specific SCM code
> > * Added HDCP SCM call
> >
> >
>
> Pulled into next/drivers, thanks!
>

I've had to apply this patch on top of our for-next branch to make it
build for randconfig.

Can you check that this makes sense?

Arnd

diff --git a/drivers/soc/qcom/Kconfig b/drivers/soc/qcom/Kconfig
index 5eea374c8fa6..01aa2fd3514d 100644
--- a/drivers/soc/qcom/Kconfig
+++ b/drivers/soc/qcom/Kconfig
@@ -13,6 +13,8 @@ config QCOM_GSBI
config QCOM_PM
bool "Qualcomm Power Management"
depends on ARCH_QCOM && !ARM64
+ select QCOM_SCM
+ select ARM_CPU_SUSPEND
help
QCOM Platform specific power driver to manage cores and L2 low power
modes. It interface with various system drivers to put the cores in


2015-06-01 19:04:40

by Kumar Gala

[permalink] [raw]
Subject: Re: [GIT PULL] qcom SoC changes for 4.2-1


> On Jun 1, 2015, at 1:35 PM, Arnd Bergmann <[email protected]> wrote:
>
> On Friday 29 May 2015 14:42:01 Arnd Bergmann wrote:
>> On Thursday 28 May 2015 10:55:39 Kumar Gala wrote:
>>> Qualcomm ARM Based SoC Updates for v4.2-1
>>>
>>> * Added Subsystem Power Manager (SPM) driver
>>> * Split out 32-bit specific SCM code
>>> * Added HDCP SCM call
>>>
>>>
>>
>> Pulled into next/drivers, thanks!
>>
>
> I've had to apply this patch on top of our for-next branch to make it
> build for randconfig.
>
> Can you check that this makes sense?
>
> Arnd

The QCOM_SCM makes sense, Lina can comment on the ARM_CPU_SUSPEND bit, but seems reasonable to me.

- k

>
> diff --git a/drivers/soc/qcom/Kconfig b/drivers/soc/qcom/Kconfig
> index 5eea374c8fa6..01aa2fd3514d 100644
> --- a/drivers/soc/qcom/Kconfig
> +++ b/drivers/soc/qcom/Kconfig
> @@ -13,6 +13,8 @@ config QCOM_GSBI
> config QCOM_PM
> bool "Qualcomm Power Management"
> depends on ARCH_QCOM && !ARM64
> + select QCOM_SCM
> + select ARM_CPU_SUSPEND
> help
> QCOM Platform specific power driver to manage cores and L2 low power
> modes. It interface with various system drivers to put the cores in
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html

--
Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project

2015-06-01 19:13:54

by Lina Iyer

[permalink] [raw]
Subject: Re: [GIT PULL] qcom SoC changes for 4.2-1

On Mon, Jun 01 2015 at 13:04 -0600, Kumar Gala wrote:
>
>> On Jun 1, 2015, at 1:35 PM, Arnd Bergmann <[email protected]> wrote:
>>
>> On Friday 29 May 2015 14:42:01 Arnd Bergmann wrote:
>>> On Thursday 28 May 2015 10:55:39 Kumar Gala wrote:
>>>> Qualcomm ARM Based SoC Updates for v4.2-1
>>>>
>>>> * Added Subsystem Power Manager (SPM) driver
>>>> * Split out 32-bit specific SCM code
>>>> * Added HDCP SCM call
>>>>
>>>>
>>>
>>> Pulled into next/drivers, thanks!
>>>
>>
>> I've had to apply this patch on top of our for-next branch to make it
>> build for randconfig.
>>
>> Can you check that this makes sense?
>>
>> Arnd
>
>The QCOM_SCM makes sense, Lina can comment on the ARM_CPU_SUSPEND bit, but seems reasonable to me.
>
Make sense to have both the selects below.

I will test it and report.

Thanks,
Lina

>- k
>
>>
>> diff --git a/drivers/soc/qcom/Kconfig b/drivers/soc/qcom/Kconfig
>> index 5eea374c8fa6..01aa2fd3514d 100644
>> --- a/drivers/soc/qcom/Kconfig
>> +++ b/drivers/soc/qcom/Kconfig
>> @@ -13,6 +13,8 @@ config QCOM_GSBI
>> config QCOM_PM
>> bool "Qualcomm Power Management"
>> depends on ARCH_QCOM && !ARM64
>> + select QCOM_SCM
>> + select ARM_CPU_SUSPEND
>> help
>> QCOM Platform specific power driver to manage cores and L2 low power
>> modes. It interface with various system drivers to put the cores in
>>
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
>> the body of a message to [email protected]
>> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
>--
>Qualcomm Innovation Center, Inc.
>The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
>a Linux Foundation Collaborative Project
>

2015-06-04 16:49:20

by Lina Iyer

[permalink] [raw]
Subject: Re: [GIT PULL] qcom SoC changes for 4.2-1

On Mon, Jun 01 2015 at 13:13 -0600, Lina Iyer wrote:
>On Mon, Jun 01 2015 at 13:04 -0600, Kumar Gala wrote:
>>
>>>On Jun 1, 2015, at 1:35 PM, Arnd Bergmann <[email protected]> wrote:
>>>
>>>On Friday 29 May 2015 14:42:01 Arnd Bergmann wrote:
>>>>On Thursday 28 May 2015 10:55:39 Kumar Gala wrote:
>>>>>Qualcomm ARM Based SoC Updates for v4.2-1
>>>>>
>>>>>* Added Subsystem Power Manager (SPM) driver
>>>>>* Split out 32-bit specific SCM code
>>>>>* Added HDCP SCM call
>>>>>
>>>>>
>>>>
>>>>Pulled into next/drivers, thanks!
>>>>
>>>
>>>I've had to apply this patch on top of our for-next branch to make it
>>>build for randconfig.
>>>
>>>Can you check that this makes sense?
>>>
>>> Arnd
>>
>>The QCOM_SCM makes sense, Lina can comment on the ARM_CPU_SUSPEND bit, but seems reasonable to me.
>>
>Make sense to have both the selects below.
>
>I will test it and report.
>
I dont see a problem with these changes.

>Thanks,
>Lina
>
>>- k
>>
>>>
>>>diff --git a/drivers/soc/qcom/Kconfig b/drivers/soc/qcom/Kconfig
>>>index 5eea374c8fa6..01aa2fd3514d 100644
>>>--- a/drivers/soc/qcom/Kconfig
>>>+++ b/drivers/soc/qcom/Kconfig
>>>@@ -13,6 +13,8 @@ config QCOM_GSBI
>>>config QCOM_PM
>>> bool "Qualcomm Power Management"
>>> depends on ARCH_QCOM && !ARM64
>>>+ select QCOM_SCM
>>>+ select ARM_CPU_SUSPEND
>>> help
>>> QCOM Platform specific power driver to manage cores and L2 low power
>>> modes. It interface with various system drivers to put the cores in
>>>
>>>--
>>>To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
>>>the body of a message to [email protected]
>>>More majordomo info at http://vger.kernel.org/majordomo-info.html
>>
>>--
>>Qualcomm Innovation Center, Inc.
>>The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
>>a Linux Foundation Collaborative Project
>>