Am Freitag, 12. Februar 2016, 07:13:29 schrieb Michael Turquette: > Quoting Heiko Stuebner (2016-02-12 03:39:43) > > Am Donnerstag, 11. Februar 2016, 18:19:36 schrieb Michael Turquette: > > > Quoting Heiko Stuebner (2016-02-11 05:00:02) > > > > > > > Yakir Yang (1): > > > > clk: rockchip: rk3036: enable the CLK_IGNORE_UNUSED flag for > > > > hclk_vio_bus > > > > > > Thanks for sending the very readable PR. One question below, > > > > > > We'll likely have critical clock and handoff clock support merged in > > > the > > > next few days. I'm starting to audit any new CLK_IGNORE_UNUSED flag > > > usage to see if it should use either of those two flags. Is the above > > > patch a candidate? > > > > the above clock is part of the interconnect. We don't model that yet, > > but I guess that would make it more a candidate for the handoff-type > > instead. > > > > I'd guess for the Rockchip side, everything that is CLK_IGNORE_UNUSED > > should be the handoff type (leaf-clocks used by some component we don't > > control yet). > > > > The clocks we already register as critical most likely are the ones that > > should be critical, as they are central clocks where it causes havok > > when > > they get disabled :-) . > > OK. It sounds like you might go back and clean up these clocks to use > the new infrastructure once it gets merged, that has always be the plan ;-) . > so I'll go ahead and pull this tag now. thanks, that will make life a bit easier when I don't have to rip the branch apart again, but can do the conversion on top.
