From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lukasz Majewski Subject: Re: [PATCH v4 3/7] cpufreq:acpi:x86: Adjust the acpi-cpufreq.c code to work with common boost solution Date: Wed, 26 Jun 2013 14:17:20 +0200 Message-ID: <20130626141720.24f6bcb0@amdc308.digital.local> References: <1370502472-7249-1-git-send-email-l.majewski@samsung.com> <1371661969-7660-1-git-send-email-l.majewski@samsung.com> <1371661969-7660-4-git-send-email-l.majewski@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: In-reply-to: Sender: cpufreq-owner@vger.kernel.org To: Viresh Kumar Cc: "Rafael J. Wysocky" , "cpufreq@vger.kernel.org" , Linux PM list , Vincent Guittot , Jonghwa Lee , Myungjoo Ham , linux-kernel , Lukasz Majewski , Andre Przywara , Daniel Lezcano , Kukjin Kim , Zhang Rui , Eduardo Valentin List-Id: linux-pm@vger.kernel.org On Wed, 26 Jun 2013 16:33:30 +0530, Viresh Kumar wrote: > On 19 June 2013 22:42, Lukasz Majewski wrote: > > diff --git a/drivers/cpufreq/acpi-cpufreq.c > > b/drivers/cpufreq/acpi-cpufreq.c > > > static unsigned int acpi_pstate_strict; > > -static bool boost_enabled, boost_supported; > > +static bool boost_enabled; > > Instead of this variable use cpufreq_boost_enabled() directly. OK. > > And then this patch may change as well.. > > I couldn't find any more issues with this patch but some Intel guy > must review it. Yes, indeed Intel review is welcome. -- Best regards, Lukasz Majewski Samsung R&D Institute Poland (SRPOL) | Linux Platform Group