Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1398596 > unrolled thread

Re: [PATCH v4 4/5] arm64: add support for ACPI Low Power Idle(LPI)

Started by"Rafael J. Wysocki" <rjw@rjwysocki.net>
First post2016-05-11 02:10 +0200
Last post2016-05-11 17:10 +0200
Articles 2 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH v4 4/5] arm64: add support for ACPI Low Power Idle(LPI) "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2016-05-11 02:10 +0200
    Re: [PATCH v4 4/5] arm64: add support for ACPI Low Power Idle(LPI) Sudeep Holla <sudeep.holla@arm.com> - 2016-05-11 17:10 +0200

#1398596 — Re: [PATCH v4 4/5] arm64: add support for ACPI Low Power Idle(LPI)

From"Rafael J. Wysocki" <rjw@rjwysocki.net>
Date2016-05-11 02:10 +0200
SubjectRe: [PATCH v4 4/5] arm64: add support for ACPI Low Power Idle(LPI)
Message-ID<rxpSW-52V-5@gated-at.bofh.it>
On Tuesday, April 19, 2016 01:30:12 PM Sudeep Holla wrote:
> This patch adds appropriate callbacks to support ACPI Low Power Idle
> (LPI) on ARM64.
> 
> It also selects ARCH_SUPPORTS_ACPI_PROCESSOR_LPI if ACPI is enabled
> on ARM64.

Can you remind me why we need the ARCH_SUPPORTS_ACPI_PROCESSOR_LPI thing in
addition to the previously defined ARCH_SUPPORTS_ACPI_PROCESSOR_CSTATE?

I thought supporting LPI would not require an arch Kconfig option.

Thanks,
Rafael

[toc] | [next] | [standalone]


#1399232

FromSudeep Holla <sudeep.holla@arm.com>
Date2016-05-11 17:10 +0200
Message-ID<rxDVV-2hX-45@gated-at.bofh.it>
In reply to#1398596

On 11/05/16 01:07, Rafael J. Wysocki wrote:
> On Tuesday, April 19, 2016 01:30:12 PM Sudeep Holla wrote:
>> This patch adds appropriate callbacks to support ACPI Low Power Idle
>> (LPI) on ARM64.
>>
>> It also selects ARCH_SUPPORTS_ACPI_PROCESSOR_LPI if ACPI is enabled
>> on ARM64.
>
> Can you remind me why we need the ARCH_SUPPORTS_ACPI_PROCESSOR_LPI thing in
> addition to the previously defined ARCH_SUPPORTS_ACPI_PROCESSOR_CSTATE?
>

That's stupid of me, I removed it from the driver in v4 but the few
remnants of it stayed by mistake(mainly in this patch alone)

> I thought supporting LPI would not require an arch Kconfig option.
>

That's correct.

-- 
Regards,
Sudeep

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web