Thread (98 messages) 98 messages, 13 authors, 2012-01-20

[RFC PATCH v3 3/5] pinctrl: imx: add pinctrl imx driver

From: Shawn Guo <hidden>
Date: 2012-01-10 13:41:15
Also in: linux-devicetree, lkml

On Tue, Jan 10, 2012 at 11:43:28AM +0100, Linus Walleij wrote:
On Tue, Jan 10, 2012 at 9:38 AM, Richard Zhao
[off-list ref] wrote:
quoted
On Mon, Jan 09, 2012 at 02:32:12PM +0800, Shawn Guo wrote:
quoted
As I was educated by Linus.W, we can not drink ocean and we need to
split big chunk of work into pieces and achieve it step by step.
Sure, we can not drink out the ocean :). But if people can not use it
in real case, how do we verify the driver? How do we know the driver
goes in the right way?
If you prefer to do a big upfront design of the entire driver then please
by all means do that.

What we need to know right now is what you need from the pin
controller core to do that. We have a (custom) pinconf API now,
does it fit the bill?
Hmm, I do not think it fits right away, as I do not think it's sensible
to have pinctrl client drivers call pin_config_set() directly.
I have spent some time on generic pin config, it was not much
commented so if it helps you please ACK the patches.
Since the ARM community is the one who is most interested in pinctrl
subsystem so far, I guess copying LAKML will help collect more comments
and various tags.  (For example, I'm interested in pinctrl, but I do
not track LKML as close as I do for LAKML).
Next from discussions with Stpehen I've sort of figured out that
we need a config mapping table of sorts and then a means to activate
entries of that table at runtime. I'm trying to come up with something
for this.
That's something we are waiting for pinconf being usable for imx.

-- 
Regards,
Shawn
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help