From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755624AbcIIWYw (ORCPT ); Fri, 9 Sep 2016 18:24:52 -0400 Received: from mx2.suse.de ([195.135.220.15]:57488 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755589AbcIIWYt (ORCPT ); Fri, 9 Sep 2016 18:24:49 -0400 Date: Sat, 10 Sep 2016 00:24:46 +0200 From: "Luis R. Rodriguez" To: Daniel Wagner Cc: linux-kernel@vger.kernel.org, Ming Lei , "Luis R . Rodriguez" , Greg Kroah-Hartman , "rafael.j.wysocki" , Daniel Vetter , Takashi Iwai , Bjorn Andersson , Arend van Spriel , Daniel Wagner Subject: Re: [PATCH v5 2/5] firmware: encapsulate firmware loading status Message-ID: <20160909222446.GR3296@wotan.suse.de> References: <1473423144-21734-1-git-send-email-wagi@monom.org> <1473423144-21734-3-git-send-email-wagi@monom.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1473423144-21734-3-git-send-email-wagi@monom.org> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Sep 09, 2016 at 02:12:21PM +0200, Daniel Wagner wrote: > From: Daniel Wagner > > The firmware user helper code tracks the current state of the loading > process via unsigned long status and a complection in struct > firmware_buf. We only need this for the usermode helper as such we can > encapsulate all this data into its own data structure. > > Cc: Ming Lei > Cc: Luis R. Rodriguez > Cc: Greg Kroah-Hartman > Signed-off-by: Daniel Wagner > --- > drivers/base/firmware_class.c | 130 +++++++++++++++++++++++++++++------------- > 1 file changed, 91 insertions(+), 39 deletions(-) > > diff --git a/drivers/base/firmware_class.c b/drivers/base/firmware_class.c > index 8eba1fb..821babe 100644 > --- a/drivers/base/firmware_class.c > +++ b/drivers/base/firmware_class.c > @@ -91,12 +91,6 @@ static inline bool fw_is_builtin_firmware(const struct firmware *fw) > } > #endif > > -enum { > - FW_STATUS_LOADING, > - FW_STATUS_DONE, > - FW_STATUS_ABORT, > -}; > - > static int loading_timeout = 60; /* In seconds */ > > static inline long firmware_loading_timeout(void) > @@ -104,6 +98,77 @@ static inline long firmware_loading_timeout(void) > return loading_timeout > 0 ? loading_timeout * HZ : MAX_JIFFY_OFFSET; > } > > +#ifdef CONFIG_FW_LOADER_USER_HELPER > + > +enum { > + FW_UMH_UNKNOWN, > + FW_UMH_LOADING, > + FW_UMH_DONE, > + FW_UMH_ABORTED, > +}; Note the enum here is anonymous still. That's a bit of a lost opportunity. If we name it we can then just use it in arguments, but that can be done later in your other patch. So say we name it enum fw_umh_status or whatever. Luis