From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joe Perches Subject: Re: [patch 08/12] can: c_can: Makethe code readable Date: Tue, 18 Mar 2014 10:37:00 -0700 Message-ID: <1395164220.2812.59.camel@joe-AO722> References: <20140318171007.528610837@linutronix.de> <20140318171127.430027680@linutronix.de> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Cc: LKML , Wolfgang Grandegger , Marc Kleine-Budde , Markus Pargmann , Benedikt Spranger , linux-can@vger.kernel.org, netdev@vger.kernel.org To: Thomas Gleixner Return-path: In-Reply-To: <20140318171127.430027680@linutronix.de> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On Tue, 2014-03-18 at 17:19 +0000, Thomas Gleixner wrote: [patch] Hi The multi-while loop while ((obj = ffs(pend)) && quota > 0) { ... } while ((obj = ffs(pend)) && quota > 0); looks, umm, unusual.