Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753412Ab3GVAKT (ORCPT ); Sun, 21 Jul 2013 20:10:19 -0400 Received: from mail-ie0-f172.google.com ([209.85.223.172]:62589 "EHLO mail-ie0-f172.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753139Ab3GVAKQ (ORCPT ); Sun, 21 Jul 2013 20:10:16 -0400 MIME-Version: 1.0 In-Reply-To: <20130704162641.GG27646@sirena.org.uk> References: <1372398218.9243.29.camel@envy.home> <20130704162641.GG27646@sirena.org.uk> From: Grant Likely Date: Mon, 22 Jul 2013 01:09:55 +0100 X-Google-Sender-Auth: XpviQGKdco4xr0rrsIcTDlbyxWs Message-ID: Subject: Re: [PATCH 4/8] minnowboard: Add base platform driver for the MinnowBoard To: Mark Brown Cc: Darren Hart , Linus Walleij , Linux Kernel Mailing List , "H. Peter Anvin" , "peter.p.waskiewicz.jr" , Andy Shevchenko , danders , "vishal.l.verma" , Matthew Garrett , Richard Purdie , platform-driver-x86 Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1300 Lines: 27 On Thu, Jul 4, 2013 at 5:26 PM, Mark Brown wrote: > On Thu, Jun 27, 2013 at 10:43:38PM -0700, Darren Hart wrote: > >> minnow_hwid() just returns an int that the minnowboard platform driver >> read from the GPIO. This seems like a proper abstraction to me. Do you >> object to this one as well? > > We should really have a subsystem for this too - the general idea idea > of identifying boards, fit options and so on by looking at things like > GPIOs or numbers in flash is really common. And yet this is a platform with ACPI. I would expect the ACPI to identify the board, not a custom driver. The newest ACPI spec adds a lot of nice useful things like GPIO and SPI bindings. Talk to Al Stone about the progress his team has made on adding GPIO support to ACPICA. This driver shouldn't be merged into mainline. Keep it as an out-of-tree patch until the proper solution is implemented. That shouldn't be too onerous since we now have available not one, but two mechanisms for describing exactly what you want to do; ACPI or FDT. g. -- 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/