From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Sreenivasa Honnur" Subject: RE: [PATCH 0/9] drivers/net/s2io.c: Cleanups Date: Mon, 31 Aug 2009 01:21:36 -0400 Message-ID: <78C9135A3D2ECE4B8162EBDCE82CAD7705A1DF76@nekter> References: <20090826.153353.259743590.davem@davemloft.net> <1251326878.13026.52.camel@Joe-Laptop.home> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT Return-path: Received: from mx.neterion.com ([72.1.205.142]:56728 "EHLO owa.neterion.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754380AbZHaFVb convert rfc822-to-8bit (ORCPT ); Mon, 31 Aug 2009 01:21:31 -0400 Content-class: urn:content-classes:message In-Reply-To: <1251326878.13026.52.camel@Joe-Laptop.home> Sender: linux-next-owner@vger.kernel.org List-ID: To: Joe Perches , David Miller Cc: netdev@vger.kernel.org, Anil Murthy , Sivakumar Subramani , Rastapur Santosh , Ramkrishna Vepa , akpm@linux-foundation.org, linux-next@vger.kernel.org Joe, Patches look fine, I reviewed and tested these patches on latest davem-net-next-2.6 sources. Acked-by: Sreenivasa Honnur Regs srini -----Original Message----- From: Joe Perches [mailto:joe@perches.com] Sent: Thursday, August 27, 2009 4:18 AM To: David Miller Cc: netdev@vger.kernel.org; Anil Murthy; Sreenivasa Honnur; Sivakumar Subramani; Rastapur Santosh; Ramkrishna Vepa; akpm@linux-foundation.org; linux-next@vger.kernel.org Subject: Re: [PATCH 0/9] drivers/net/s2io.c: Cleanups On Wed, 2009-08-26 at 15:33 -0700, David Miller wrote: > From: Joe Perches > Date: Mon, 24 Aug 2009 20:29:39 -0700 > > Just a few cleanups, compiled, untested. No hardware. > > s2io.c: Use const for strings > > s2io.c: Shorten code line length by using intermediate pointers > > s2io.c: Use calculated size in kmallocs > > s2io.c: use kzalloc > > s2io.c: Make more conforming to normal kernel style > > s2io.c: convert printks to pr_ > > s2io.c: fix spelling explaination > > s2io.c: Standardize statistics accessors > > s2io.c: Convert skipped nic->config.tx_cfg[i]. to tx_cfg-> > > Since this is a pretty serious set of cleanups, I'd like to let the > S2IO driver folks have some time to look at this and at least have a > chance to ACK/NACK them. No worries. That's fine. It's only cleanups. I haven't heard from them though and they haven't submitted or signed a patch for s2io in over a year. Hey! Neterion people! Is anybody home?