From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paul Bolle Subject: Re: [PATCH] cpufreq: s5pv210: remove unused call of pr_err() Date: Thu, 22 May 2014 08:41:34 +0200 Message-ID: <1400740894.21254.8.camel@x220> References: <1400702036.30334.6.camel@x220> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-pm-owner@vger.kernel.org List-ID: Content-Type: text/plain; charset="us-ascii" To: Sachin Kamat Cc: "Rafael J. Wysocki" , Viresh Kumar , "cpufreq@vger.kernel.org" , "linux-pm@vger.kernel.org" , LKML Sachin, On Thu, 2014-05-22 at 08:52 +0530, Sachin Kamat wrote: > The config symbol itself is not defined and hence could be removed. > However I feel, the > error message could be retained after trimming down the arguments a bit. My reasoning is, of course, that never used code can simply be dropped. But you want to print an error here. That's fine with me. I prefer if you'd submit a patch that does that, as I have no idea whatsoever why an error is needed here nor what that error should say. I do hope you don't mind me prodding you if that takes too long, and this pointless check of CONFIG_PM_VERBOSE is still here after, say, another development cycle. Paul Bolle