From mboxrd@z Thu Jan 1 00:00:00 1970 From: Viresh Kumar Subject: Re: [RFC PATCH v2 4/6] sched/fair: Introduce an energy estimation helper function Date: Fri, 13 Apr 2018 11:57:41 +0530 Message-ID: <20180413062741.GA17616@vireshk-i7> References: <20180406153607.17815-1-dietmar.eggemann@arm.com> <20180406153607.17815-5-dietmar.eggemann@arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20180406153607.17815-5-dietmar.eggemann@arm.com> Sender: linux-kernel-owner@vger.kernel.org To: Dietmar Eggemann Cc: linux-kernel@vger.kernel.org, Peter Zijlstra , Quentin Perret , Thara Gopinath , linux-pm@vger.kernel.org, Morten Rasmussen , Chris Redpath , Patrick Bellasi , Valentin Schneider , "Rafael J . Wysocki" , Greg Kroah-Hartman , Vincent Guittot , Todd Kjos , Joel Fernandes , Juri Lelli , Steve Muckle , Eduardo Valentin List-Id: linux-pm@vger.kernel.org On 06-04-18, 16:36, Dietmar Eggemann wrote: > +static inline struct capacity_state > +*find_cap_state(int cpu, unsigned long util) { return NULL; } I saw this somewhere else as well in this series. I believe the line break should happen after "*" as "struct capacity_state *" should be read together as one entity. -- viresh