On Mon, Aug 06, 2012 at 07:10:00PM +0800, Mitch Bradley wrote: > On 8/6/2012 5:58 PM, Johannes Stezenbach wrote: > > On Mon, Aug 06, 2012 at 08:35:51AM +0200, Linus Walleij wrote: > >> On Mon, Aug 6, 2012 at 4:18 AM, Stephen Warren <swar...@wwwdotorg.org> > >> wrote: > >> > >>> I can't comment on the sysfs-vs-dev interface location, but I don't > >>> think it addresses Johannes' issue; finding out which GPIO IDs are > >>> provided by which devices. > >>> > >>> Perhaps in each device's sysfs node, there should be some information > >>> re: which GPIO range it provides. Right now, perhaps a text file with > >>> the GPIO base it it. > >> > >> Yes that could work ... > > > > The method used by the gpio-mxs.c driver (of_alias_get_id) > > would also work. The question is which method is preferable. > > > > Ideally I would like to use DT attributes to identify my GPIOs > > in the same way as they appear in the schematics. E.g. > > one device may have GPIOs called PORT_A.0 to PORT_A.7, > > another one may be EXT.0 to EXT.15. But a single integer ID > > is good enough since GPIO usage is platform specific anyway. > > However, the mapping must be static and not depend e.g. on > > module load order like now if you use pl061 and some i2c GPIO. > > Software must be able to rely on that e.g. GPIO ID 11 > > always refers to EXT.3. > > There is precedence for a "slot-names" property that correlates specific > hardware entities with physical labels. It has been applied to PCI > plug-in slots and to other devices. See, for example, > http://www.openfirmware.org/1275/proposals/Closed/Accepted/381-it.txt
Sorry about the slow response. After thinking it through I decided that a) adding ad-hoc DT bindings isn't good, and b) doing it properly is above my head atm (I have too many other tasks to take care of). So I decided to use platform data to get stable GPIO numbers and names for now. Actually I think the kernel internal GPIO numbers shouldn't be in the sysfs API, instead userspace should use the names. Probably it's best to not use /sys/class/gpio/export but instead let the board init code export the GPIOs available to userspace with proper names. Not sure yet... Thanks, Johannes -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/