On Thu, 06 Nov 2014 16:05:13 +0100 , Hans de Goede <hdegoede@xxxxxxxxxx> wrote: > Hi, > > On 11/06/2014 03:18 PM, Tomi Valkeinen wrote: > > On 31/10/14 10:15, Tomi Valkeinen wrote: > >> Hi Hans, > >> > >> On 22/10/14 19:45, Hans de Goede wrote: > >> > >>> -We will add a clocks property to the simplefb devicetree bindings, so > >>> that u-boot setup framebuffers passed to the kernel (for early console > >>> support) can properly list the clocks used, and simplefb can claim them to > >>> avoid them getting turned off, thereby breaking the early console > >> > >> Perhaps this has been discussed earlier, but I started to wonder if > >> managing clocks is enough. > >> > >> What about regulators (to power the LCD), gpios (to keep the LCD > >> enabled) and pinctrl (for video signals)? I guess any those could be > >> reset by the kernel if no driver uses them. I'm okay with extending the binding for each of these dependencies /providing/ it can be argued that the way simplefb uses it is merely to inhibit the kernel from making any changes to the resource, and then releasing the resource when it is done. Each resource type will need to be taken on a case-by-case basis since each has it's own behaviour. I'll want to see that the simplefb driver doesn't need to do anything special per device, and that the resource binding is well established and generic. g. -- To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html