From mboxrd@z Thu Jan 1 00:00:00 1970 From: Geert Uytterhoeven Subject: Re: [PATCH v4 1/2] ARM: keystone: pm: switch to use generic pm domains Date: Fri, 21 Nov 2014 10:04:47 +0100 Message-ID: References: <1415631557-22897-1-git-send-email-grygorii.strashko@ti.com> <1709760.E0jX3Myv0h@wuerfel> <546C7FDD.7030906@ti.com> <2900095.WIocOu7ue2@wuerfel> <546DD87B.3080806@ti.com> <546E0970.5090301@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: <546E0970.5090301-l0cyMroinI0@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Grygorii Strashko Cc: Ulf Hansson , Arnd Bergmann , Kevin Hilman , ssantosh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, "Rafael J. Wysocki" , "linux-pm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Rob Herring , Grant Likely , "linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" , "linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Dmitry Torokhov List-Id: devicetree@vger.kernel.org On Thu, Nov 20, 2014 at 4:32 PM, Grygorii Strashko wrote: >> That's indeed the tricky part. >> >> On shmobile, we just use the "NULL" clock, i.e. the first clock, for historical >> reasons. >> >> Using clock-names (e.g. "fck" and "pm") will conflict with existing bindings >> for devices that already mandate specific clock names. >> >> As the clock being a "PM" clock is a property of the clock provider >> (at last on shmobile), I originally came up with not handling it in DT, >> but advertising it in the clock provider driver: >> >>>> - first Geert Uytterhoeven proposed to use CLK_RUNTIME_PM there >>>> https://lkml.org/lkml/2014/11/6/319 > > As I mentioned previously I don't like it, because in many cases one driver is used for > all/set_of clocks which support gating function. As result, some sort of tables will > need to be created and maintained in code by these drivers per each SoC > (or even per each SoC revision) for identification that clock is "PM clock". That depends. On shmobile, there's a separate hardware module for those clocks, hence all clocks provided by drivers/clk/shmobile/clk-mstp.c are "PM" clocks. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert-Td1EMuHUCqxL1ZNQvxDV9g@public.gmane.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html