Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753823Ab0K2GHg (ORCPT ); Mon, 29 Nov 2010 01:07:36 -0500 Received: from foo.birdnet.se ([213.88.146.6]:54149 "HELO foo.stuge.se" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1751160Ab0K2GHf (ORCPT ); Mon, 29 Nov 2010 01:07:35 -0500 Message-ID: <20101129060732.5130.qmail@stuge.se> Date: Mon, 29 Nov 2010 07:07:32 +0100 From: Peter Stuge To: ath9k-devel@venema.h4ckr.net, linux-kernel@vger.kernel.org Subject: Re: [ath9k-devel] [PATCH wireless-next] ath: Rename ath_print to ath_debug Mail-Followup-To: ath9k-devel@lists.ath9k.org, linux-kernel@vger.kernel.org References: <5febb0e1fba0ec2bb77f6ade8b251ba0edf4614c.1290988277.git.joe@perches.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 598 Lines: 17 Luis R. Rodriguez wrote: > On Sun, Nov 28, 2010 at 3:53 PM, Joe Perches wrote: > > Make the function name match the function purpose. > > ath_debug is a debug only facility. > > ath_print seems too generic a name for a debug only use. > > Nack, I don't see the point. The point is to improve readability. I like the patch. //Peter -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/