From mboxrd@z Thu Jan 1 00:00:00 1970 From: Saravana Kannan Date: Wed, 06 Apr 2016 21:49:34 +0000 Subject: Re: [PATCH 1/7] cpufreq: remove redundant CPUFREQ_INCOMPATIBLE notifier event Message-Id: <5705846E.9050006@codeaurora.org> List-Id: References: <406f55ac8030043f0349b084878c9b8d04f7ad86.1438571116.git.viresh.kumar@linaro.org> <4897937.47p7WB7CoU@vostro.rjw.lan> <20150910003918.GL5266@linux> <57057201.9030109@codeaurora.org> <57057FB1.2050904@codeaurora.org> In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: "Rafael J. Wysocki" Cc: Viresh Kumar , "Rafael J. Wysocki" , Lists linaro-kernel , "linux-pm@vger.kernel.org" , Dmitry Eremin-Solenikov , Fabian Frederick , Jean-Christophe Plagniol-Villard , Jonathan Corbet , Len Brown , open@codeaurora.org, "list@codeaurora.org:ACPI" , "list@codeaurora.org:DOCUMENTATION" , "list@codeaurora.org:FRAMEBUFFER LAYER" , open list , Nicholas Mc Guire , Russell King , Tomi Valkeinen , Wolfram Sang On 04/06/2016 02:45 PM, Rafael J. Wysocki wrote: > On Wed, Apr 6, 2016 at 11:29 PM, Saravana Kannan wrote: >> On 04/06/2016 02:21 PM, Rafael J. Wysocki wrote: >>> >>> On Wed, Apr 6, 2016 at 10:30 PM, Saravana Kannan >>> wrote: >>>> >>>> On 09/09/2015 05:53 PM, Rafael J. Wysocki wrote: >>>>> > > [cut] > >>> >>> Well, nobody was using that event. >>> >> >> True, but that's more of a bug in drivers/thermal/cpu-cooling.c and >> drivers/acpi/processor_thermal.c. We should revert this patch and fix those >> drivers. Does that seem acceptable to you? > > I'd rather see a patch series adding the event back along with some > users. One user at least. > Ok, I'll make those two drivers use them and send it out. It's very clearly a bug in those drivers. -Saravana` -- Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project