Re: [PATCH] pinctrl: Add one-register-per-pin type device tree basedpinctrl driver

From: Linus Walleij
Date: Sat Jul 14 2012 - 16:17:19 EST


On Tue, Jul 10, 2012 at 11:11 AM, Tony Lindgren <tony@xxxxxxxxxxx> wrote:

> OK so no comments for a while. Here's the patch updated to leave out
> the comments in the binding example.

I reason like this:

- My fears is that the code gets hopeless to understand the mux, the
only way to understand that aspect of the system will be to read the DTS
and have the data sheet ready at hand.

But:

- Tony knows what he's doing and what is best for OMAP. And this gets
(hopefully) all that OMAP mux code out of arch/arm.

- Surely it will be better to go through this subsystem if we're refactoring
it all again later, and all drivers can be transferred to the abstract
pinctrl API which is a big win in itself, bringing coherency to the
drivers/* at large.

So applied it, so it can be evaluated in real operating environments.

But if I don't see OMAP transferred to use this I'll simply delete it
again. :-)

Yours,
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/