From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark Brown Subject: Re: [PATCH 1/5] [ARM] pxa: lx: add PCON i2c driver Date: Mon, 20 Apr 2009 13:25:19 +0100 Message-ID: <20090420122519.GC8776@sirena.org.uk> References: <20090418141932.GA1236@n2100.arm.linux.org.uk> <1240064452-14691-1-git-send-email-linux@arm.linux.org.uk> <20090418162900.0dc16531@hyperion.delvare> <20090418144234.GB1236@n2100.arm.linux.org.uk> <20090418173406.GA16195@sirena.org.uk> <20090418181030.GC1236@n2100.arm.linux.org.uk> <20090419111114.GA8569@sirena.org.uk> <20090420121805.GE3653@n2100.arm.linux.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20090420121805.GE3653-l+eeeJia6m9vn6HldHNs0ANdhmdF6hFW@public.gmane.org> Sender: linux-i2c-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Russell King - ARM Linux Cc: Jean Delvare , linux-arm-kernel-xIg/pKzrS19vn6HldHNs0ANdhmdF6hFW@public.gmane.org, Ben Dooks , linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-i2c@vger.kernel.org On Mon, Apr 20, 2009 at 01:18:05PM +0100, Russell King - ARM Linux wrote: > On Sun, Apr 19, 2009 at 12:11:14PM +0100, Mark Brown wrote: > > Surely that's just an implementation detail? As I understand your > > description the microcontroller is hiding the control interfaces of the > > other things it controls so the function is pretty similar. > No, it's a question of whether drivers/mfd is right. The "shovel > everything that doesn't fit elsewhere into drivers/mfd" approach > is just creating yet another problem, just like drivers/i2c/chips > is percieved to be. The only difference is that it becomes someone > elses problem. Well, my thinking was that it looks like you've got a bunch of subsystem-specific drivers for this chip with a core driver managing shared resources like I/O with the device. To my mind that's pretty much the use case for MFD. As far as general dumping grounds goes we already have drivers/misc.