Received: by 2002:ac0:a5a7:0:0:0:0:0 with SMTP id m36-v6csp957626imm; Wed, 15 Aug 2018 08:59:08 -0700 (PDT) X-Google-Smtp-Source: AA+uWPxykImoCI/yD58ycTt5KmxVIZNZhqFMryWMQSnKlUST8r8SgLG/Poym7EJuKeY2pwODD4ho X-Received: by 2002:a63:5fc1:: with SMTP id t184-v6mr24898937pgb.183.1534348748737; Wed, 15 Aug 2018 08:59:08 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1534348748; cv=none; d=google.com; s=arc-20160816; b=IWOIOvzH9r8brac6IVyASume3TD2Ngal3GV1EdevgIfIQC6/6K98q/38atFyFyIrx5 SBopJFs5sEpegFAlRwKt9rne4lvdJfPslz6TeOj32M3gYNCB/RIBPZZPUi045ZmcoBcc +lLIr9tb4s48n+jCY7V2inJO8d1opXOIJK00nGG6/RUTuBLF24wBjgiwjOoyWKsW2iLe SCVaxhKQhA2WGL0+WXOJTuGBjkcoT81YUOAo5wWxAtZV+ItEQRW0Ggbgpud/2LZ7x+nK hpHcSmfVtur4Hsr+182VSitnDORtRh3BMiAzdCMqTkJbffR4RRHnIOOehX5ThwlS6w2y 7rWA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:arc-authentication-results; bh=Srpzz7ZQM4wK9mOIW8bqQhqiP2tNXc2JYt0QVRc/QNA=; b=h4Srbly7dhEvILnEBvbe12Zkx4WEq3l+zVuGcKJ6JmfPNlIxpgMBtvnAFa2aJYNPxL qeMlDWze4ztT3te0ESAAE17CsXBrLhTpWFaCywNlhHe8ICgali1UuAzciqP0ZUMly34j Jw/rsM+yul0yqdGLZZ2HiFGzwivilGTCdLkmn5oVJgVKEkft49gGy4SyhtWgmDKllSbF S7jYpHYnaE7SwMOWpVq8vPA5+kdgTVaM0WrPkpvOm728B0BD+baQXvEMu2NNsaH7TtPt kW85Q1yt1Wc1nqd249jgNJsFEdkdXCiXQCL2Jvt8equvG8rO0yReNlbEhvYJ3HP3twnX 7VdA== 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 p18-v6si21100055pgc.414.2018.08.15.08.58.53; Wed, 15 Aug 2018 08:59:08 -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 S1729520AbeHOSum (ORCPT + 99 others); Wed, 15 Aug 2018 14:50:42 -0400 Received: from mail.skyhub.de ([5.9.137.197]:32918 "EHLO mail.skyhub.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729300AbeHOSum (ORCPT ); Wed, 15 Aug 2018 14:50:42 -0400 X-Virus-Scanned: Nedap ESD1 at mail.skyhub.de Received: from mail.skyhub.de ([127.0.0.1]) by localhost (blast.alien8.de [127.0.0.1]) (amavisd-new, port 10026) with ESMTP id wziyr_iWQIcD; Wed, 15 Aug 2018 17:57:42 +0200 (CEST) Received: from nazgul.tnic (95-42-132-194.ip.btc-net.bg [95.42.132.194]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.skyhub.de (SuperMail on ZX Spectrum 128k) with ESMTPSA id 51F991EC0104; Wed, 15 Aug 2018 17:57:40 +0200 (CEST) Date: Wed, 15 Aug 2018 17:57:52 +0200 From: Borislav Petkov To: Takashi Iwai Cc: Tim Small , Mauro Carvalho Chehab , linux-edac@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] EDAC: i82443bxgx: Fix invalid memory type assignment Message-ID: <20180815155752.GC28669@nazgul.tnic> References: <20180810141520.8993-1-tiwai@suse.de> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20180810141520.8993-1-tiwai@suse.de> User-Agent: Mutt/1.6.0 (2016-04-01) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Aug 10, 2018 at 04:15:20PM +0200, Takashi Iwai wrote: > The i82443bxgx_edac driver assigns an invalid negative value when an > unknown DRAM type is detected. Drop the unnecessary '-' that brings > misbehavior. > > Fixes: 5a2c675c8919 ("drivers/edac: new i82443bxgz MC driver") > Cc: > Signed-off-by: Takashi Iwai > --- > drivers/edac/i82443bxgx_edac.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/edac/i82443bxgx_edac.c b/drivers/edac/i82443bxgx_edac.c > index a2ca929e2168..1638f288fa47 100644 > --- a/drivers/edac/i82443bxgx_edac.c > +++ b/drivers/edac/i82443bxgx_edac.c > @@ -273,7 +273,7 @@ static int i82443bxgx_edacmc_probe1(struct pci_dev *pdev, int dev_idx) > break; > default: > edac_dbg(0, "Unknown/reserved DRAM type value in DRAMC register!\n"); > - mtype = -MEM_UNKNOWN; > + mtype = MEM_UNKNOWN; > } > > if ((mtype == MEM_SDR) || (mtype == MEM_RDR)) > -- Is this something you caught by inspection or you really have hw that loads that driver... Wikipedia says 440BX chipset is pentium II and III which, if so, would drop its priority considerably. CC:stable would be not needed either, TBH. Thx. -- Regards/Gruss, Boris. ECO tip #101: Trim your mails when you reply. --