On Mon, 12 Mar 2012 08:17:19 -0000, Arnd Bergmann <[email protected]> wrote:
On Monday 12 March 2012, Jason Cooper wrote:
> + dnskw_gpio_register(39, "dnskw:power:sata0", 1);
> + dnskw_gpio_register(40, "dnskw:power:sata1", 1);
> +
> + /* Set NAS to turn back on after a power failure */
> + dnskw_gpio_register(37, "dnskw:power:recover", 1);
> +}
> +
I doubt this board is the only one that wants to be added while the
conversion to devicetree is progressing. It makes more sense to me to
break the above out to a separate file, eg board-dnskw.c. I'll follow
and break out the dreamplug code to board-dreamplug.c. Otherwise, I see
board-dt.c becoming a long mess.
Arnd, sound okay to you?
Either way is fine with me. I personally wouldn't expect many more boards
to come in before the DT conversion has mostly completed, but I guess you
have a better feeling for what other hardware is out there that people
are interested in.
It probably depends on how easy it is to add the gpiochips into
devicetree. Being able to put the LED, buttons and fans in devicetree
would make board-dt.c much more manageable. I started to look into it and
wasn't immediately obvious, but will have a closer look.
Happy to resubmit, breaking it out into a board-dnskw.c though.
--
Jamie Lentin
_______________________________________________
devicetree-discuss mailing list
[email protected]
https://lists.ozlabs.org/listinfo/devicetree-discuss
