From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sascha Hauer Subject: Re: [PATCH 1/3] mfd: Add basic support for the Congatec CGEB BIOS interface Date: Mon, 20 Feb 2012 19:07:52 +0100 Message-ID: <20120220180752.GV3852@pengutronix.de> References: <1328102793-4313-1-git-send-email-s.hauer@pengutronix.de> <1328102793-4313-2-git-send-email-s.hauer@pengutronix.de> <20120220163927.GY5387@sortiz-mobl> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20120220163927.GY5387@sortiz-mobl> Sender: linux-kernel-owner@vger.kernel.org To: Samuel Ortiz Cc: linux-kernel@vger.kernel.org, Jean Delvare , linux-i2c@vger.kernel.org, linux-watchdog@vger.kernel.org, Ben Dooks , kernel@pengutronix.de List-Id: linux-i2c@vger.kernel.org Hi Samuel, On Mon, Feb 20, 2012 at 05:39:27PM +0100, Samuel Ortiz wrote: > Hi Sascha, > > On Wed, Feb 01, 2012 at 02:26:31PM +0100, Sascha Hauer wrote: > > The Congatec CGEB is a BIOS interface found on some Congatec x86 > > modules. It provides access to on board peripherals like I2C busses > > and watchdogs. This driver contains the basic support for accessing > > the CGEB interface and registers the child devices. > After looking at the code, I'm not entirely sure this one belongs to > drivers/mfd/. Have you thought about putting it under arch/x86/platform/ ? > There are similar examples there, like uv or olpc. I don't mind putting it in arch/x86/platform instead. I'm an ARM guy and on ARM there currently is a rush to move everything looking remotely like a driver out of arch/arm/ to drivers/, so putting it under drivers/mfd/ seemed logical to me. Sascha -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |