From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754143AbcIIMNf (ORCPT ); Fri, 9 Sep 2016 08:13:35 -0400 Received: from atlantic540.startdedicated.de ([188.138.9.77]:58623 "EHLO atlantic540.startdedicated.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752385AbcIIMMw (ORCPT ); Fri, 9 Sep 2016 08:12:52 -0400 From: Daniel Wagner To: linux-kernel@vger.kernel.org Cc: Ming Lei , "Luis R . Rodriguez" , Greg Kroah-Hartman , "Srivatsa S . Bhat" , "Rafael J . Wysocki" , Daniel Vetter , Takashi Iwai , Bjorn Andersson , Arend van Spriel , Daniel Wagner Subject: [PATCH v5 3/5] firmware: rename fw_load_from_user_helper() and _request_firmware_load() Date: Fri, 9 Sep 2016 14:12:22 +0200 Message-Id: <1473423144-21734-4-git-send-email-wagi@monom.org> X-Mailer: git-send-email 2.7.4 In-Reply-To: <1473423144-21734-1-git-send-email-wagi@monom.org> References: <1473423144-21734-1-git-send-email-wagi@monom.org> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Daniel Wagner fw_load_from_user_helper() and _request_firmware_load() are used when CONFIG_FW_LOADER_USER_HELPER is enabled. In order to clearly mark which part of the code is depending on UMH we stream line these functions to match with the rest of the code, e.g. fw_umh_done(). Suggested by Luis. Cc: Ming Lei Cc: Luis R. Rodriguez Cc: Greg Kroah-Hartman Signed-off-by: Daniel Wagner --- drivers/base/firmware_class.c | 23 +++++++++++------------ 1 file changed, 11 insertions(+), 12 deletions(-) diff --git a/drivers/base/firmware_class.c b/drivers/base/firmware_class.c index 821babe..5e38c27 100644 --- a/drivers/base/firmware_class.c +++ b/drivers/base/firmware_class.c @@ -979,9 +979,8 @@ fw_create_instance(struct firmware *firmware, const char *fw_name, return fw_priv; } -/* load a firmware via user helper */ -static int _request_firmware_load(struct firmware_priv *fw_priv, - unsigned int opt_flags, long timeout) +static int _request_firmware_umh(struct firmware_priv *fw_priv, + unsigned int opt_flags, long timeout) { int retval = 0; struct device *f_dev = &fw_priv->dev; @@ -1032,9 +1031,9 @@ static int _request_firmware_load(struct firmware_priv *fw_priv, return retval; } -static int fw_load_from_user_helper(struct firmware *firmware, - const char *name, struct device *device, - unsigned int opt_flags, long timeout) +static int fw_get_umh_firmware(struct firmware *firmware, const char *name, + struct device *device, unsigned int opt_flags, + long timeout) { struct firmware_priv *fw_priv; @@ -1043,7 +1042,7 @@ static int fw_load_from_user_helper(struct firmware *firmware, return PTR_ERR(fw_priv); fw_priv->buf = firmware->priv; - return _request_firmware_load(fw_priv, opt_flags, timeout); + return _request_firmware_umh(fw_priv, opt_flags, timeout); } #ifdef CONFIG_PM_SLEEP @@ -1064,9 +1063,9 @@ static void kill_requests_without_uevent(void) #else /* CONFIG_FW_LOADER_USER_HELPER */ static inline int -fw_load_from_user_helper(struct firmware *firmware, const char *name, - struct device *device, unsigned int opt_flags, - long timeout) +fw_get_umh_firmware(struct firmware *firmware, const char *name, + struct device *device, unsigned int opt_flags, + long timeout) { return -ENOENT; } @@ -1242,8 +1241,8 @@ _request_firmware(const struct firmware **firmware_p, const char *name, name, ret); if (opt_flags & FW_OPT_USERHELPER) { dev_warn(device, "Falling back to user helper\n"); - ret = fw_load_from_user_helper(fw, name, device, - opt_flags, timeout); + ret = fw_get_umh_firmware(fw, name, device, + opt_flags, timeout); } } -- 2.7.4