From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp-out-168.synserver.de ([212.40.185.168]:1033 "HELO smtp-out-165.synserver.de" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with SMTP id S1750959Ab1KXH2m (ORCPT ); Thu, 24 Nov 2011 02:28:42 -0500 Message-ID: <4ECDF260.5030907@metafoo.de> Date: Thu, 24 Nov 2011 08:29:36 +0100 From: Lars-Peter Clausen MIME-Version: 1.0 To: Ben Dooks CC: Grant Likely , Greg Kroah-Hartman , Jean Delvare , Jonathan Cameron , Michael Hennerich , linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org, linux-i2c@vger.kernel.org, spi-devel-general@lists.sourceforge.net Subject: Re: [PATCH 3/5] SPI: Add helper macro for spi_driver boilerplate References: <1321434819-23678-1-git-send-email-lars@metafoo.de> <1321434819-23678-4-git-send-email-lars@metafoo.de> <20111124001342.GN19115@trinity.fluff.org> In-Reply-To: <20111124001342.GN19115@trinity.fluff.org> Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-iio-owner@vger.kernel.org List-Id: linux-iio@vger.kernel.org On 11/24/2011 01:13 AM, Ben Dooks wrote: > On Wed, Nov 16, 2011 at 10:12:54AM -0700, Grant Likely wrote: >> On Wed, Nov 16, 2011 at 2:13 AM, Lars-Peter Clausen wrote: >>> This patch introduces the module_spi_driver macro which is a convenience macro >>> for SPI driver modules similar to module_platform_driver. It is intended to be >>> used by drivers which init/exit section does nothing but register/unregister >>> the SPI driver. By using this macro it is possible to eliminate a few lines of >>> boilerplate code per SPI driver. >>> >>> Signed-off-by: Lars-Peter Clausen >> >> Acked-by: Grant Likely > > I'm begining to think we need to make some of these driver and device > bits more generic... there seems to be so much similar but not quite > the same code. > I've been thinking the same. A good start would probably be consolidating the platform/spi/i2c device id handling code. Since those are all name based ids the code for handling them looks rather similar, though there are some minor differences. - Lars