Peter De Schrijver wrote at Wednesday, September 28, 2011 11:12 AM:

On Wed, Sep 28, 2011 at 03:16:42PM +0200, Arnd Bergmann wrote:

quoted

On Wednesday 28 September 2011, Peter De Schrijver wrote:

quoted

--- a/arch/arm/mach-tegra/board-dt.c

...

quoted

quoted

@@ -95,6 +95,8 @@ static void __init tegra_dt_init(void)
                harmony_pinmux_init();
        else if (of_machine_is_compatible("nvidia,seaboard"))
                seaboard_pinmux_init();
+       else if (of_machine_is_compatible("nvidia,ventana"))
+               ventana_pinmux_init();

Are you (Peter, Colin, Olof) confident that this is not getting out
of hand before it's getting better?

We don't planning to add more derived boards besides ventana until we have the
devicetree bindings for the new pinmux API in at which point this code
will become obsolete.

Well, I think there will be one more; we're hoping to start uptreaming Tegra30
support soon, and the reference board for that will be Cardhu. That'll likely
happen before we have the initialize-pinmux-from-device-tree patches complete,
since I'm holding those off until we all think about whether the new pinmux API
core should be driving the initialization rather than individual pinmux drivers.

But, I hope Ventana and Cardhu will be the last before that's in place.

If not, we can certainly switch these if's to a table.

-- 
nvpublic