From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sudeep Holla Subject: Re: [PATCH v7 8/8] PCC: Enable PCC only when needed Date: Tue, 04 Aug 2015 15:53:38 +0100 Message-ID: <55C0D1F2.9050505@arm.com> References: <55AE652B.7030409@arm.com> <1583446.GxfV2z2IW5@vostro.rjw.lan> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-pm-owner@vger.kernel.org To: Ashwin Chaugule Cc: "Rafael J. Wysocki" , Sudeep Holla , "Rafael J. Wysocki" , "jaswinder.singh@linaro.org" , "linux-pm@vger.kernel.org" , "linux-acpi@vger.kernel.org" , "linaro-acpi@lists.linaro.org" , "patches@linaro.org" , "viresh.kumar@linaro.org" List-Id: linux-acpi@vger.kernel.org On 03/08/15 18:35, Ashwin Chaugule wrote: > On 21 July 2015 at 21:28, Rafael J. Wysocki wrote: >> On Tuesday, July 21, 2015 04:28:43 PM Sudeep Holla wrote: >>> Hi Rafael, >>> [...] >>> >>> Agreed and I am absolutely fine with that. But if you look at this >>> patch, it does >>> >>> config PCC >>> bool "Platform Communication Channel Driver" >>> depends on ACPI && ACPI_CPPC_LIB >> >> My bad, I've evidently overlooked that. >> >> If PPC is selected from ACPI_CPPC_LIB, the "depends on" above is >> obviously not needed. > > Thanks Sudeep. > So I'll take this dependency out and default PCC to 'n'. Think that > should work for all cases. Agree? > Yes Regards, Sudeep