Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933196AbYGQUUe (ORCPT ); Thu, 17 Jul 2008 16:20:34 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S932141AbYGQUTx (ORCPT ); Thu, 17 Jul 2008 16:19:53 -0400 Received: from az33egw01.freescale.net ([192.88.158.102]:33048 "EHLO az33egw01.freescale.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1760584AbYGQUTw (ORCPT ); Thu, 17 Jul 2008 16:19:52 -0400 Date: Thu, 17 Jul 2008 13:18:18 -0700 (PDT) From: Trent Piepho X-X-Sender: xyzzy@t2.domain.actdsltmp To: Grant Likely cc: avorontsov@ru.mvista.com, Richard Purdie , Stephen Rothwell , Kumar Gala , linux-kernel@vger.kernel.org, linuxppc-dev@ozlabs.org Subject: Re: [PATCH v3] leds: implement OpenFirmare GPIO LED driver In-Reply-To: Message-ID: References: <1216133032.5345.73.camel@dax.rpnet.com> <20080715151917.GA30607@polina.dev.rtsoft.ru> <20080717041531.GA27243@secretlab.ca> <20080717140519.GA32617@polina.dev.rtsoft.ru> <20080717141335.GA2219@polina.dev.rtsoft.ru> <20080717150422.GC31932@secretlab.ca> <20080717152006.GA26120@polina.dev.rtsoft.ru> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1343 Lines: 42 On Thu, 17 Jul 2008, Grant Likely wrote: > Alternately, I would also be okay with a scheme where all LED nodes > have a common parent and an of_platform driver would bind against the > parent node; not the individual children. Then the leds-gpio driver > could be refactored to have both platform and of_platform bus > bindings. Basically what I did then in my patch then, refactor leds-gpio so most of it is shared and there is a block of code that does platform binding and another block that does of_platform binding. I didn't change the OF platform binding syntax so as not to complicate the example, but that's easy to do. Something like: leds { compatible = "gpio-led"; gpios = <&mpc8572 6 0 &mpc8572 7 0>; labels = "red", "green"; }; Or like this, which needs a little more code to parse: leds { compatible = "gpio-led"; led@6 { gpios = <&mpc8572 6 0>; label = "red"; }; led@7 { gpios = <&mpc8572 7 0>; label = "green"; }; }; I like the first better. It follows the example from the docs about how devices with multiple gpios should work too. -- 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/