From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1765325AbZAQAzY (ORCPT ); Fri, 16 Jan 2009 19:55:24 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754655AbZAQAzG (ORCPT ); Fri, 16 Jan 2009 19:55:06 -0500 Received: from hpsmtp-eml16.KPNXCHANGE.COM ([213.75.38.116]:2726 "EHLO hpsmtp-eml16.kpnxchange.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751549AbZAQAzF (ORCPT ); Fri, 16 Jan 2009 19:55:05 -0500 From: Frans Pop To: Linux Kernel Mailing List Subject: [PATCH] hp-wmi: fix regressions caused by missing if statement Date: Sat, 17 Jan 2009 01:55:01 +0100 User-Agent: KMail/1.9.9 Cc: Larry Finger , Matthew Garrett , linux-acpi@vger.kernel.org, Andrew Morton References: <200901170013.55786.elendil@planet.nl> <200901170131.22541.elendil@planet.nl> In-Reply-To: <200901170131.22541.elendil@planet.nl> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Content-Disposition: inline Message-Id: <200901170155.02757.elendil@planet.nl> X-OriginalArrivalTime: 17 Jan 2009 00:55:03.0580 (UTC) FILETIME=[3B598DC0:01C9783E] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Frans Pop hp-wmi: fix regressions caused by missing if statement Error was introduced in commit fe8e4e039dc3. Signed-off-by: Frans Pop --- > On Saturday 17 January 2009, Frans Pop wrote: > > When I first booted .29-rc2 my HP 2510p notebook hung while loading > > hp-wmi, which I have listed in /etc/modules. Hard poweroff was > > needed. And this obvious patch fixes the regression (tested). Andrew: I've added you as the broken patch also went via you. diff --git a/drivers/platform/x86/hp-wmi.c b/drivers/platform/x86/hp-wmi.c index 7c789f0..6260420 100644 --- a/drivers/platform/x86/hp-wmi.c +++ b/drivers/platform/x86/hp-wmi.c @@ -441,6 +441,7 @@ static int __init hp_wmi_bios_setup(struct platform_device *device) bluetooth_rfkill->toggle_radio = hp_wmi_bluetooth_set; bluetooth_rfkill->user_claim_unsupported = 1; err = rfkill_register(bluetooth_rfkill); + if (err) goto register_bluetooth_error; }