From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: Bug in ks8851.c Date: Fri, 29 Mar 2013 15:43:48 -0400 (EDT) Message-ID: <20130329.154348.1417472944436998499.davem@davemloft.net> References: Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: kda@linux-powerpc.org, gerg@uclinux.org, jiri@resnulli.us, linus971@gmail.com, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, mjr@cs.wisc.edu, netdev@vger.kernel.org, sboyd@codeaurora.org, torvalds@linux-foundation.org To: Max.Nekludov@us.elster.com Return-path: Received: from shards.monkeyblade.net ([149.20.54.216]:56543 "EHLO shards.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754251Ab3C2Tnw (ORCPT ); Fri, 29 Mar 2013 15:43:52 -0400 In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: From: Max.Nekludov@us.elster.com Date: Fri, 29 Mar 2013 19:27:36 +0400 > Denis, > here is patch with commit message: > > Subject: [PATCH] Fix bug in ks8851 driver > > According to the Datasheet (page 52): > 15-12 Reserved > 11-0 RXBC Receive Byte Count > This field indicates the present received frame byte size. > > The code has a bug: > rxh = ks8851_rdreg32(ks, KS_RXFHSR); > rxstat = rxh & 0xffff; > rxlen = rxh >> 16; // BUG!!! 0xFFF mask should be applied > > Signed-off-by: Max Nekludov Applied and queued up for -stable, thanks.