From patchwork Mon Oct 29 16:05:09 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Alan Cox X-Patchwork-Id: 1664541 Return-Path: X-Original-To: patchwork-linux-acpi@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork1.kernel.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by patchwork1.kernel.org (Postfix) with ESMTP id EBFB73FCF7 for ; Mon, 29 Oct 2012 16:03:50 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759788Ab2J2QDt (ORCPT ); Mon, 29 Oct 2012 12:03:49 -0400 Received: from lxorguk.ukuu.org.uk ([81.2.110.251]:58202 "EHLO lxorguk.ukuu.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759899Ab2J2QDs (ORCPT ); Mon, 29 Oct 2012 12:03:48 -0400 Received: from localhost.localdomain (earthlight.etchedpixels.co.uk [81.2.110.250]) by lxorguk.ukuu.org.uk (8.14.5/8.14.1) with ESMTP id q9TGZi44003568 for ; Mon, 29 Oct 2012 16:35:49 GMT From: Alan Cox Subject: [PATCH] pnpacpi: incorrect test To: linux-acpi@vger.kernel.org Date: Mon, 29 Oct 2012 16:05:09 +0000 Message-ID: <20121029160455.14886.37788.stgit@localhost.localdomain> User-Agent: StGIT/0.14.3 MIME-Version: 1.0 Sender: linux-acpi-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-acpi@vger.kernel.org From: Alan Cox TEST_ALPHA() is broken and always returns 0 Resolves-bug: https://bugzilla.kernel.org/show_bug.cgi?id=45171 --- drivers/pnp/pnpacpi/core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/drivers/pnp/pnpacpi/core.c b/drivers/pnp/pnpacpi/core.c index 26b5d4b..107b6b6 100644 --- a/drivers/pnp/pnpacpi/core.c +++ b/drivers/pnp/pnpacpi/core.c @@ -58,7 +58,7 @@ static inline int __init is_exclusive_device(struct acpi_device *dev) if (!(('0' <= (c) && (c) <= '9') || ('A' <= (c) && (c) <= 'F'))) \ return 0 #define TEST_ALPHA(c) \ - if (!('@' <= (c) || (c) <= 'Z')) \ + if (!('@' <= (c) && (c) <= 'Z')) \ return 0 static int __init ispnpidacpi(const char *id) {