On Thu, Sep 12, 2013 at 03:11:45PM +0800, Sean Cross wrote: > > Even if it should be configurable I'm still unsure whether this should > > be done at pci driver level. > > I'm a bit wary of including clock setup like this in the SoC code, as > different boards might be different. If a board designer wants to run > LVDS2 to the PCIe slot, that's entirely acceptable. It might not > actually ever get done, but it seems like the sort of board-specific > thing that device tree was designed to handle. The devicetree was designed to handle hardware descriptions, not configuration options. Chosing the clock parent is probably in the grey area between hardware description and configuration, so I assume when we find a good reason to add this option to the devicetree then it's acceptable to add it there. But we shouldn't add this option for something that, as you say, 'might not actually ever get done'. Just hardcode the clock selection in SoC code for now. If someone ever finds the good reason to make this configurable then he will speak up and we can discuss this again. Sascha -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html