All of lore.kernel.org
 help / color / mirror / Atom feed
From: Borislav Petkov <bp@alien8.de>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: linux-edac <linux-edac@vger.kernel.org>,
	lkml <linux-kernel@vger.kernel.org>
Subject: [GIT PULL] A minor amd64_edac fix for 3.13
Date: Sun, 10 Nov 2013 13:26:38 +0100	[thread overview]
Message-ID: <20131110122638.GA17775@pd.tnic> (raw)

Hi Linus,

please pull this one minor thing which has been lying around since
forever... (at least it feels like it - I almost forgot about it :-)).

Thanks.

--
The following changes since commit 272b98c6455f00884f0350f775c5342358ebb73f:

  Linux 3.12-rc1 (2013-09-16 16:17:51 -0400)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/bp/bp.git tags/edac_for_3.13

for you to fetch changes up to c3e511e4c4d9cb962976f63d37816ff8e8efacc5:

  amd64_edac: Remove superfluous condition check. (2013-09-19 09:55:20 +0200)

----------------------------------------------------------------
A minor correctness fix.

----------------------------------------------------------------
Aravind Gopalakrishnan (1):
      amd64_edac: Remove superfluous condition check.

 drivers/edac/amd64_edac.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/edac/amd64_edac.c b/drivers/edac/amd64_edac.c
index 3c9e4e98c651..bbd1913ce27c 100644
--- a/drivers/edac/amd64_edac.c
+++ b/drivers/edac/amd64_edac.c
@@ -1576,7 +1576,7 @@ static int f15_m30h_match_to_this_node(struct amd64_pvt *pvt, unsigned range,
 					     num_dcts_intlv, dct_sel);
 
 	/* Verify we stay within the MAX number of channels allowed */
-	if (channel > 4 || channel < 0)
+	if (channel > 4)
 		return -EINVAL;
 
 	leg_mmio_hole = (u8) (dct_cont_base_reg >> 1 & BIT(0));

-- 
Regards/Gruss,
    Boris.

Sent from a fat crate under my desk. Formatting is fine.
--

             reply	other threads:[~2013-11-10 12:27 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-10 12:26 Borislav Petkov [this message]
2013-11-16  2:02 ` [GIT PULL] A minor amd64_edac fix for 3.13 Linus Torvalds
2013-11-16 10:06   ` Borislav Petkov
     [not found]     ` <CA+55aFxHO9035t=CiNTRX1XK5KSxzh8HfZ3Kaspsm6yfs2O=8Q@mail.gmail.com>
2013-11-16 11:12       ` Borislav Petkov
2013-11-16 18:10         ` Linus Torvalds
2013-11-16 18:23           ` Borislav Petkov
2013-11-16 23:34             ` Frank Becker
     [not found]               ` <CA+55aFxZcNfOEadD7EixNf5YZJVLTJ=2T4TOybWfwTP71KnuRg@mail.gmail.com>
2013-11-17  1:22                 ` Frank Becker
2013-11-17  7:47                   ` Borislav Petkov
2013-11-16 10:29   ` Borislav Petkov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20131110122638.GA17775@pd.tnic \
    --to=bp@alien8.de \
    --cc=linux-edac@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=torvalds@linux-foundation.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.