Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756422AbZJTD0e (ORCPT ); Mon, 19 Oct 2009 23:26:34 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753138AbZJTD0d (ORCPT ); Mon, 19 Oct 2009 23:26:33 -0400 Received: from mail-px0-f171.google.com ([209.85.216.171]:44347 "EHLO mail-px0-f171.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752574AbZJTD0c (ORCPT ); Mon, 19 Oct 2009 23:26:32 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=date:from:to:cc:subject:message-id:references:mime-version :content-type:content-disposition:in-reply-to:user-agent; b=SD2WLbb8AHgclSZQ8cjVwJeV+e8x0KP0tUm+QKXrIw5FGM6KRt1cBbblPo9+IHGDvk Rz3MHIWCagu6rprmv1+EbzFQqYQjPAqhVrWJlXRTAYpbt8uh93yAOneKfUhwqfDGcJHG JxFtertA0oyNw45bQ0IRCI8+R25Hk2LMEEj/o= Date: Mon, 19 Oct 2009 20:26:33 -0700 From: Dmitry Torokhov To: Alex Chiang Cc: shaohua.li@intel.com, lenb@kernel.org, linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v4 1/6] ACPI: dock: convert sysfs attributes to an attribute_group Message-ID: <20091020032632.GI24370@core.coreip.homeip.net> References: <20091019211132.5412.80438.stgit@bob.kio> <20091019211424.5412.15540.stgit@bob.kio> <20091020013844.GB24370@core.coreip.homeip.net> <20091020025733.GA5814@ldl.fc.hp.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20091020025733.GA5814@ldl.fc.hp.com> User-Agent: Mutt/1.5.19 (2009-01-05) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1004 Lines: 33 On Mon, Oct 19, 2009 at 08:57:33PM -0600, Alex Chiang wrote: > * Dmitry Torokhov : > > On Mon, Oct 19, 2009 at 03:14:24PM -0600, Alex Chiang wrote: > > > As suggested by Dmitry Torokhov, convert the individual sysfs > > > attributes into an attribute group. > > > > > > This change eliminates quite a bit of copy/paste code in the > > > error handling paths. > > > > > > > Looks good to me, thank you for making the changes. > > Thank you for your patience along the way. > > Can I count this as an Acked-by? :) > Yes, you may add Reviewed-by: Dmitry Torokhov I use "Acked-by" for stuff I have authority over which is being merged through someone else's tree for one reason or another. Thanks. -- Dmitry -- 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/