Date: Tue, 23 Nov 2004 18:50:22 +0100
From: Jean Delvare <khali@linux-fr.org>
To: Justin Thiessen <jthiessen@penguincomputing.com>
Cc: sensors@Stimpy.netroedge.com, linux-kernel@vger.kernel.org,
       Greg KH <greg@kroah.com>
Subject: Re: adm1026 driver port for kernel 2.6.10-rc2 (patch includes
 driver, patch to Kconfig, and patch to Makefile)
Message-Id: <20041123185022.54f6e069.khali@linux-fr.org>
In-Reply-To: <20041123165236.GA4936@penguincomputing.com>
References: <20041102221745.GB18020@penguincomputing.com>
	<NN38qQl1.1099468908.1237810.khali@gcu.info>
	<20041103164354.GB20465@penguincomputing.com>
	<20041118185612.GA20728@penguincomputing.com>
	<20041123165236.GA4936@penguincomputing.com>
Reply-To: LM Sensors <sensors@stimpy.netroedge.com>,
       LM Sensors <sensors@stimpy.netroedge.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 845
Lines: 22

> Taking into account Arjan's comments, I have fixed the awkward
> locking constructs.  I have also switched from using MODULE_PARM()
> to module_param_array().  As Jean requested (and as I muttonheadedly
> ignored) on the last submission, the diff now includes the patches to 
> 
> .../drivers/i2c/chips/Makefile
> .../drivers/i2c/chips/Kconfig

Great, looks good enough to me. I'd like to see this patch applied to
the kernel. Any further change can be done through incremental patches,
which are so easier to review.

Thanks Justin for the good work :)

-- 
Jean Delvare
http://khali.linux-fr.org/
-
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/