Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757447AbXEJF0O (ORCPT ); Thu, 10 May 2007 01:26:14 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1755570AbXEJFZ5 (ORCPT ); Thu, 10 May 2007 01:25:57 -0400 Received: from smtp1.linux-foundation.org ([65.172.181.25]:45335 "EHLO smtp1.linux-foundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755399AbXEJFZz (ORCPT ); Thu, 10 May 2007 01:25:55 -0400 Date: Wed, 9 May 2007 22:25:48 -0700 From: Andrew Morton To: Jeff Garzik Cc: Kristen Carlson Accardi , linux-ide@vger.kernel.org, linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org, htejun@gmail.com, "Randy.Dunlap" Subject: Re: [patch 1/7] libata: check for AN support Message-Id: <20070509222548.0c878055.akpm@linux-foundation.org> In-Reply-To: <4642AA3E.4090308@garzik.org> References: <20070510072247.063476979@intel.com> <20070509163809.7b0b3fde.kristen.c.accardi@intel.com> <20070509220952.5fa34822.akpm@linux-foundation.org> <4642AA3E.4090308@garzik.org> X-Mailer: Sylpheed 2.4.1 (GTK+ 2.8.17; x86_64-unknown-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 921 Lines: 30 On Thu, 10 May 2007 01:14:38 -0400 Jeff Garzik wrote: > Andrew Morton wrote: > >> + * @dev: Device to which command will be sent > >> + * > >> + * Issue SET FEATURES - SATA FEATURES command to device @dev > >> + * on port @ap. > >> + * > >> + * LOCKING: > >> + * PCI/etc. bus probe sem. > >> + * > >> + * RETURNS: > >> + * 0 on success, AC_ERR_* mask otherwise. > >> + */ > > > > ooh, locking and return value documentation. Often missed, and nice. > > > We set high standards in the libata world ;-) > It seems to be working. This series is perhaps the most idiomatic and generally kernelly-looking code I've seen in ages. Who cares if it works? ;) - 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/