Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757166Ab2BBUv5 (ORCPT ); Thu, 2 Feb 2012 15:51:57 -0500 Received: from xes-mad.com ([216.165.139.218]:21750 "EHLO xes-mad.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756480Ab2BBUv4 (ORCPT ); Thu, 2 Feb 2012 15:51:56 -0500 X-Greylist: delayed 3250 seconds by postgrey-1.27 at vger.kernel.org; Thu, 02 Feb 2012 15:51:55 EST Message-ID: <1328212615.12622.193.camel@petert> Subject: Re: [PATCH] gpio: New driver for the Intel 82801 (ICH) GPIO pins From: Peter Tyser To: guenter.roeck@ericsson.com Cc: Jean Delvare , Grant Likely , LKML Date: Thu, 02 Feb 2012 13:56:55 -0600 In-Reply-To: <1328204105.2261.140.camel@groeck-laptop> References: <20110419145303.111aead7@endymion.delvare> <20120202023129.GA19535@ericsson.com> <20120202084909.2adf0f50@endymion.delvare> <1328204105.2261.140.camel@groeck-laptop> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.2.2- Content-Transfer-Encoding: 7bit Mime-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2525 Lines: 53 > > > > drivers/gpio/Kconfig | 7 > > > > drivers/gpio/Makefile | 1 > > > > drivers/gpio/i801_gpio.c | 432 ++++++++++++++++++++++++++++++++++++++++++++++ > > > > 3 files changed, 440 insertions(+) > > > > > > Did this or an alternate patch for gpio support on the recent Intel ICHs go anywhere ? > > > > > > I found another patch which is using a different approach, but I don't see anything > > > in the latest kernel. > > > > I don't think any driver made it into the kernel, at least I can't find > > any in 3.3-rc2 nor linux-next. I'm not surprised, as neither driver was > > using the MFD framework while this would really be the right thing to > > do here. I never took the time to convert my driver to MFD and > > apparently Peter did not either. This is still on my to-do list but you > > know how long it is. If anyone is faster than me at getting the code is > > a shape suitable for upstream, I'll be happy to help with review and > > testing. > > > I'd love to take this on, but unfortunately my task list isn't getting > shorter either. Just bad that this doesn't seem to make it in. > > One reason of course may be that at least Peter's version tried to > accomplish too much. Looking through the comments, seems there was a > disagreement on unrelated issues such as if there should be a new > "unknown" gpio direction or not. Maybe it would make more sense to > separate the core patch to add ICH gpio support from the rest of Peter's > proposed changes. I agree with Jean, I believe the main issue it wasn't accepted was the fact that it didn't use the MFD framework. The driver I submitted follows the model of other drivers that share the same PCI device (iTCO, esb2rom, etc), so is functionally OK to use as is, it just doesn't use the recommend MFD framework, which was gating its acceptance. > Peter, do you have any plans to work on this in the near future ? I too have been caught up in other tasks. A co-worker recently created a patch to transition the ICHx gpio driver I wrote and other related drivers (esb2rom, iTCO, etc) to the MFD structure with the intention that it would eventually be merged upstream. The patch is still under review internally, but I'll talk to him about submitting it upstream. Best, Peter -- 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/