Getting your opinion about the best place to put one specific device driver...

[Tony Lindgren]

* Arnd Bergmann <arnd at arndb.de> [130213 03:07]:
> On Wednesday 13 February 2013, Jean-Nicolas GRAUX wrote:
> > Le 02/12/2013 06:41 PM, Arnd Bergmann a écrit :
> > > On Tuesday 12 February 2013, Jean-Nicolas GRAUX wrote:
> > > If a more dynamic user interface is needed, I think it would be good
> > > to have a generic mechanism in the pinctrl subsystem to reroute pins
> > > like these, which can work for all pins in the system (or at least
> > > those that have not been claimed by another device). I don't know if
> > > that interface already exists, but Linus would be the right person
> > > to answer that.
> >
> > I think i have one colleague that recently made a patch for that purpose.
> > But i believe this is still in progress. It consists in improving the 
> > pinctrl
> > debugfs interface to be able to manually change the pin muxing and
> > the gpio configuration for debug/testing purpose.
> 
> Sounds good to me. Peter, Tony, do you think that would be a good enough
> interface for other platforms as well?

Yes debugfs is the way to go for modifying the hwobs settings.
 
> > > If there is a generic way to configure pinmux from user space, you
> > > can essentially move this driver into an application running outside
> > > of the kernel.
> > >
> > The matter is that this hwobs driver do more than just changing
> > the muxing of one pin group in the digital baseband.
> > It also control some specific registers to configure the hwobs IP.
> > More in details, for each of the 18 external IOs, it is possible to
> > select one observer mode among several. We also provide some
> > facility for the user to easily detect the pin to monitor on the
> > output connector.
> 
> Hmm, I wonder if that is something that makes sense to have
> in a generic pinctrl debugfs interface. Maybe it can instead
> be part of the new pinctrl driver for this hw block then, which
> should be able to create additional sysfs files in the pinctrl
> debugfs directories. Linus, does that make sense to you?

Makes sense to me.

And pinctrl does not need to have any knowledge of the names of
the hwobs signals etc. That can all be done in a user space app
modifying the pinctrl registers via debugs.

Regards,

Tony