From patchwork Fri Nov 2 15:14:32 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arnd Bergmann X-Patchwork-Id: 150047 Delivered-To: patch@linaro.org Received: by 2002:a2e:299d:0:0:0:0:0 with SMTP id p29-v6csp2198527ljp; Fri, 2 Nov 2018 08:15:02 -0700 (PDT) X-Google-Smtp-Source: AJdET5fbS3w1u8kR45OGIAExxmQ0dxfat/xlr2Rter6Hj75+WIJN4w6Z3CpU6U6VXGlXmhSuTYkr X-Received: by 2002:a62:204a:: with SMTP id g71-v6mr12034528pfg.28.1541171702334; Fri, 02 Nov 2018 08:15:02 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1541171702; cv=none; d=google.com; s=arc-20160816; b=i5Wx25Wdl5IutTyce3mT/oH3znDGa51/6XHPsRN0mXUG8MKWZrMRI/9vfQUdVqnPUu YAF+hpT4DuyMXbnv/933nW3nz3+VCN2a6iLLpvMxAkgnSvui8ajjmGRGpFmoEUAitxqX +rLkP10MD5y/slcNCJp5SYJka4dEaggdFAMwRAKTbGFyTQFMAHt+WkDxamzQpdRDIrub /GukJS3juJblMozrxRX3UmI7mbqGBugbzNk33lGtvNXhg2p6QimPMaGSRbthuQrl5OCv 4idlYLLzGM4zS/mnyffjmIrFDRI739G6txFska1GZ8cydnmuxwpEBojOuUxUdJIZ6uk9 60tA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:message-id:date:subject:cc:to:from; bh=fxUKX7PFCpqWxFQwXpjGO64gxkyU6KXrbujHfBwb13A=; b=OACfZ56Mcin5Ae5H+e4Z4ab6K5/efP9C65SgYQ/YvOPw5YWyu2pjsWwGa5s4EbmDtk GYMHNTb46MuUZR6kVvV8JmEyYo6WsW8GftMXAWeqKGOjlfhkBpQZDsAFNCebpCOPwDdQ vF3Wx4/UcjyIiYng6f2exAqjYOJFeVYioc7WKw96PD9wx/tlLiSQwVbv+cnqrn6CipnS QgvEwddWQB5dxcY0OQX+iEI26EomtM9GL4ZSwkM8f4zYT6fYArMaWWWKG9Y5NLSg2BB5 BlAVJKTaN0RTj8BsuxC0AVObNmXhh4ssQsN9niqNitfIjiOXW/OmGma3Pidi52mEtGVf mkIQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id e184-v6si36109729pfa.206.2018.11.02.08.15.02; Fri, 02 Nov 2018 08:15:02 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728019AbeKCAWY (ORCPT + 32 others); Fri, 2 Nov 2018 20:22:24 -0400 Received: from mout.kundenserver.de ([212.227.17.24]:35513 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727704AbeKCAWY (ORCPT ); Fri, 2 Nov 2018 20:22:24 -0400 Received: from wuerfel.lan ([109.192.41.194]) by mrelayeu.kundenserver.de (mreue109 [212.227.15.145]) with ESMTPA (Nemesis) id 1Mgvj1-1fc7o01if7-00hKi9; Fri, 02 Nov 2018 16:14:45 +0100 Received: from wuerfel.lan ([109.192.41.194]) by mrelayeu.kundenserver.de (mreue109 [212.227.15.145]) with ESMTPA (Nemesis) id 1Mgvj1-1fc7o01if7-00hKi9; Fri, 02 Nov 2018 16:14:45 +0100 From: Arnd Bergmann To: Jiri Kosina Cc: Arnd Bergmann , Benjamin Tissoires , Hans de Goede , Colin Ian King , Peter Hutterer , Mustafa Kuscu , Daniel Drake , linux-input@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH] HID: asus: fix build warning wiht CONFIG_ASUS_WMI disabled Date: Fri, 2 Nov 2018 16:14:32 +0100 Message-Id: <20181102151443.758229-1-arnd@arndb.de> X-Mailer: git-send-email 2.18.0 X-Provags-ID: V03:K1:j95rsxkGv3TlKksXbH+UOcInQSsWSTiVPTyZX23/q7gBCHouhmd cF7Um1sxYxynMGddz0zQly9jgCzio5VDSuYfv263SoJklFAr7sj0eiZJy08OzIkfUR0HK/u TjtTDvWjPf4KVcxrcocMg2wA4OSMbhdEaKaWiXn1kxiBnAsJmXkOUG7H7C7+XNYSkVsorPy VBRj6rfMCQYSqh9oVxubA== X-Spam-Flag: NO X-UI-Out-Filterresults: notjunk:1; V01:K0:qjWYcfgqvn8=:1+/wXhZM8OAUBulgihJ5Ur 8UcWpbSZGXEbXbUJ2sQTATM96Kd7XCzTkBK23Oxo/twhCK295PCoikBhtb6qbPFpeSIXL57Yl VxYr/nC62ne4in0SdpfgBC/FO7lyfxXl5+/NwzO1l/rX5X1BBtVGsXGxsD2oneHi/JgNxwPZ1 NeehLYOifrnRBSYA8+0AcX/7ODxvmxB29xOS6gdQ17eJ6uc7L/BVKnjMPbFtG2Ynx32+gzhsE +uLKuJibOFHwB0J10vDefN6XMqJUOIlM3nSI5iuxNOuehx/yGE5OmrBrRbXaTWw5Ht8IGWmLV QRfHeFtuIGb6X55If9yCGkByg+umNnLfXwbJNCmdAoMi501sAfpqQvDW/DV0RPMBEHXf86j9U ZJ9SaQPrBklEG7Ix2KpH4tpPuqxNRpVSO7IT0v20eeiOay6sdjufBEnGMJWvLpvbV1uaxXSsL ZSgUuEp+eQ4mN3J+S7lp5WfN0KxtVluwhq452tB5pLfSHZruZJqc3xy+mxw/Um6tBy8Thkll5 o59MTRAdwLHVH/T6uSKGG49hgW2/goNn+PMkeZTi2jsSYpgNqF+BgrRaS95ow//fadQZyKwG2 HRsqVLVHeLfJPoICwHcqfvnPnytTYGHAxrUqdxs+qwidvviO7k/QpcAkaNeWzUgcY/+KxH0YQ tXIXn5jcxy6pKILI7c8LuqXmHi0kfSHPAq5W5n4Vx3Q9HeY+XuEcCblsV6WFGhoPlSOGqK6uQ N7x3Wc/JiJylfLEQ8HlcggV3gXz4huiPK6xfKdXbvs44cuI8zele/WvcFkkBDXfqwjZHIBLOp TECxQ/b Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org asus_wmi_evaluate_method() is an empty dummy function when CONFIG_ASUS_WMI is disabled, or not reachable from a built-in device driver. This leads to a theoretical evaluation of an uninitialized variable that the compiler complains about, failing to check that the hardcoded return value makes this an unreachable code path: In file included from include/linux/printk.h:336, from include/linux/kernel.h:14, from include/linux/list.h:9, from include/linux/dmi.h:5, from drivers/hid/hid-asus.c:29: drivers/hid/hid-asus.c: In function 'asus_input_configured': include/linux/dynamic_debug.h:135:3: error: 'value' may be used uninitialized in this function [-Werror=maybe-uninitialized] __dynamic_dev_dbg(&descriptor, dev, fmt, \ ^~~~~~~~~~~~~~~~~ drivers/hid/hid-asus.c:359:6: note: 'value' was declared here u32 value; ^~~~~ With an extra IS_ENABLED() check, the warning goes away. Fixes: 3b692c55e58d ("HID: asus: only support backlight when it's not driven by WMI") Signed-off-by: Arnd Bergmann --- drivers/hid/hid-asus.c | 3 +++ 1 file changed, 3 insertions(+) -- 2.18.0 diff --git a/drivers/hid/hid-asus.c b/drivers/hid/hid-asus.c index dc6d6477e961..a1fa2fc8c9b5 100644 --- a/drivers/hid/hid-asus.c +++ b/drivers/hid/hid-asus.c @@ -359,6 +359,9 @@ static bool asus_kbd_wmi_led_control_present(struct hid_device *hdev) u32 value; int ret; + if (!IS_ENABLED(CONFIG_ASUS_WMI)) + return false; + ret = asus_wmi_evaluate_method(ASUS_WMI_METHODID_DSTS2, ASUS_WMI_DEVID_KBD_BACKLIGHT, 0, &value); hid_dbg(hdev, "WMI backlight check: rc %d value %x", ret, value);