]> git.karo-electronics.de Git - karo-tx-linux.git/commitdiff
amd64_edac: Remove superfluous condition check.
authorAravind Gopalakrishnan <Aravind.Gopalakrishnan@amd.com>
Wed, 11 Sep 2013 15:55:58 +0000 (10:55 -0500)
committerBorislav Petkov <bp@suse.de>
Thu, 19 Sep 2013 07:55:20 +0000 (09:55 +0200)
The condition check 'channel < 0' is needless since channel is unsigned.

Reported-by: Dave Jones <davej@redhat.com>
Signed-off-by: Aravind Gopalakrishnan <Aravind.Gopalakrishnan@amd.com>
Link: http://lkml.kernel.org/r/20130905144353.GB15894@redhat.com
Signed-off-by: Borislav Petkov <bp@suse.de>
drivers/edac/amd64_edac.c

index 3c9e4e98c65111a069dcbcc5916cd2e061052938..bbd1913ce27c4cdcc2a11dcb9c8c811e7edf8f07 100644 (file)
@@ -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));