From mboxrd@z Thu Jan 1 00:00:00 1970 From: Patrick Bellasi Subject: Re: [PATCH v3 05/14] sched/cpufreq: uclamp: add utilization clamping for FAIR tasks Date: Thu, 16 Aug 2018 14:53:00 +0100 Message-ID: <20180816135300.GC2960@e110439-lin> References: <20180806163946.28380-1-patrick.bellasi@arm.com> <20180806163946.28380-6-patrick.bellasi@arm.com> <012b7e15-888c-a200-c34b-4fac0f8ab8b9@arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <012b7e15-888c-a200-c34b-4fac0f8ab8b9@arm.com> Sender: linux-kernel-owner@vger.kernel.org To: Dietmar Eggemann Cc: linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org, Ingo Molnar , Peter Zijlstra , Tejun Heo , "Rafael J . Wysocki" , Viresh Kumar , Vincent Guittot , Paul Turner , Morten Rasmussen , Juri Lelli , Todd Kjos , Joel Fernandes , Steve Muckle , Suren Baghdasaryan List-Id: linux-pm@vger.kernel.org On 15-Aug 17:30, Dietmar Eggemann wrote: > On 08/06/2018 06:39 PM, Patrick Bellasi wrote: > > [...] > > >+#else /* CONFIG_UCLAMP_TASK */ > >+static inline unsigned int uclamp_value(unsigned int cpu, int clamp_id) > >+{ > >+ return uclamp_none(clamp_id); > >+} > > Looks like that uclamp_value() is not used outside CONFIG_UCLAMP_TASK areas. Yes, you right... I use it on some debug patches I'm tracking on top of the series... but I can defenitivel drop it in v4. Cheers Patrick -- #include Patrick Bellasi