| Message ID | 1457600456-4283-1-git-send-email-jonathanh@nvidia.com (mailing list archive) |
|---|---|
| State | Changes Requested, archived |
| Headers | show |
Hi Jon, On Thu, Mar 10, 2016 at 10:00 AM, Jon Hunter <jonathanh@nvidia.com> wrote: > The PM clocks framework requires clients to pass either a con-id or a > valid clk pointer in order to add a clock to a device. Add a new > function of_pm_clk_add_clks() to allows device clocks to be retrieved > from device-tree and populated for a given device. Note that > of_clk_get_from_provider() is not defined if CONFIG_OF and > CONFIG_COMMON_CLK are not selected. Therefore, make of_pm_clk_add_clks() > dependent on these options. > > An optional function pointer may be passed to of_pm_clk_add_clks() that > can be used to filter the clocks that are added for a device when > calling of_pm_clk_add_clks(). > > In order to handle errors encountered when adding clocks from > device-tree, add a function pm_clk_remove_clk() to remove any clocks > (using a pointer to the clk structure) that have been added > successfully before the error occurred. > > Signed-off-by: Jon Hunter <jonathanh@nvidia.com> Tested-by: Geert Uytterhoeven <geert+renesas@glider.be> But more comments below... > --- a/drivers/base/power/clock_ops.c > +++ b/drivers/base/power/clock_ops.c > @@ -137,6 +137,81 @@ int pm_clk_add_clk(struct device *dev, struct clk *clk) > return __pm_clk_add(dev, NULL, clk); > } > > + > +#if defined(CONFIG_OF) && defined(CONFIG_COMMON_CLK) > +/** > + * of_pm_clk_add_clks - Start using device clock(s) for power management. > + * @dev: Device whose clock(s) is going to be used for power management. > + * @of_pm_clk_filter: Optional function for filtering clocks > + * > + * Add a series of clocks described in the 'clocks' device-tree node for > + * a device to the list of clocks used for the power management of @dev. > + * If 'of_pm_clk_filter' is specified, then this filter function will be > + * called for each clock found and the clock will be added to the list > + * of clocks if this function returns true. Return success if clocks are > + * added successfully and return a negative error code if adding a clock > + * fails or there are no clocks that match with the filter function. > + */ > +int of_pm_clk_add_clks(struct device *dev, of_pm_clk_filter fn, void *data) > +{ [...] > + count = of_count_phandle_with_args(dev->of_node, "clocks", > + "#clock-cells"); > + if (count == 0) > + return -ENODEV; [...] + return added ? 0 : -ENODEV; Is it an error condition if no clocks were present in DT, or if no clocks have been accepted by the filter function? If not, the caller has to check for -ENODEV. Now the caller has to call pm_clk_create() first, before it knows if any clocks will be added, it may want to call pm_clk_destroy() later if no clocks have been added. Alternatively, you could return 0 vs. the actual number of clocks added. I hooked up your code in renesas-cpg-mssr.c, and it worked. However, I noticed it added 27 clocks for one device node (rcar_sound), and only then I realized that I only want to add the first clock accepted by the filter, not all of them, as the others are under driver control. I'm not sure this can be handled in the filter function. So I can't use your code (for now)... Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.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 linux-pm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 10/03/16 09:41, Geert Uytterhoeven wrote: > Hi Jon, > > On Thu, Mar 10, 2016 at 10:00 AM, Jon Hunter <jonathanh@nvidia.com> wrote: >> The PM clocks framework requires clients to pass either a con-id or a >> valid clk pointer in order to add a clock to a device. Add a new >> function of_pm_clk_add_clks() to allows device clocks to be retrieved >> from device-tree and populated for a given device. Note that >> of_clk_get_from_provider() is not defined if CONFIG_OF and >> CONFIG_COMMON_CLK are not selected. Therefore, make of_pm_clk_add_clks() >> dependent on these options. >> >> An optional function pointer may be passed to of_pm_clk_add_clks() that >> can be used to filter the clocks that are added for a device when >> calling of_pm_clk_add_clks(). >> >> In order to handle errors encountered when adding clocks from >> device-tree, add a function pm_clk_remove_clk() to remove any clocks >> (using a pointer to the clk structure) that have been added >> successfully before the error occurred. >> >> Signed-off-by: Jon Hunter <jonathanh@nvidia.com> > > Tested-by: Geert Uytterhoeven <geert+renesas@glider.be> > > But more comments below... > >> --- a/drivers/base/power/clock_ops.c >> +++ b/drivers/base/power/clock_ops.c >> @@ -137,6 +137,81 @@ int pm_clk_add_clk(struct device *dev, struct clk *clk) >> return __pm_clk_add(dev, NULL, clk); >> } >> >> + >> +#if defined(CONFIG_OF) && defined(CONFIG_COMMON_CLK) >> +/** >> + * of_pm_clk_add_clks - Start using device clock(s) for power management. >> + * @dev: Device whose clock(s) is going to be used for power management. >> + * @of_pm_clk_filter: Optional function for filtering clocks >> + * >> + * Add a series of clocks described in the 'clocks' device-tree node for >> + * a device to the list of clocks used for the power management of @dev. >> + * If 'of_pm_clk_filter' is specified, then this filter function will be >> + * called for each clock found and the clock will be added to the list >> + * of clocks if this function returns true. Return success if clocks are >> + * added successfully and return a negative error code if adding a clock >> + * fails or there are no clocks that match with the filter function. >> + */ >> +int of_pm_clk_add_clks(struct device *dev, of_pm_clk_filter fn, void *data) >> +{ > > [...] > >> + count = of_count_phandle_with_args(dev->of_node, "clocks", >> + "#clock-cells"); >> + if (count == 0) >> + return -ENODEV; > > [...] > > + return added ? 0 : -ENODEV; > > Is it an error condition if no clocks were present in DT, or if no clocks have > been accepted by the filter function? If not, the caller has to check for > -ENODEV. Now the caller has to call pm_clk_create() first, before it knows if > any clocks will be added, it may want to call pm_clk_destroy() later if no > clocks have been added. It seems odd to me that someone would call this function and either there are no clocks in the DT or they are all filtered out. For example, most drivers are calling of_clk_get() because there are clocks they need to add. > Alternatively, you could return 0 vs. the actual number of clocks added. However, a nice comprise here could be to return the number of clocks added and let the user decide what to do. I like that idea. Not sure I understand what you mean by "0 vs. the actual number of clocks added". Do you just mean the return the number added? > I hooked up your code in renesas-cpg-mssr.c, and it worked. > However, I noticed it added 27 clocks for one device node (rcar_sound), and > only then I realized that I only want to add the first clock accepted by the > filter, not all of them, as the others are under driver control. > I'm not sure this can be handled in the filter function. > So I can't use your code (for now)... I see, that is unfortunate. Are you using the void *data variable in your filter? If not may be you could use this to indicate a clock has been 'found' and don't match any further clocks. If you cannot use it, then I am tempted to drop the filter function for now as this simplifies the code. It can always be added later if someone has a need for it. Cheers Jon -- To unsubscribe from this list: send the line "unsubscribe linux-pm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi Jon, On Thu, Mar 10, 2016 at 1:27 PM, Jon Hunter <jonathanh@nvidia.com> wrote: > On 10/03/16 09:41, Geert Uytterhoeven wrote: >> On Thu, Mar 10, 2016 at 10:00 AM, Jon Hunter <jonathanh@nvidia.com> wrote: >>> The PM clocks framework requires clients to pass either a con-id or a >>> valid clk pointer in order to add a clock to a device. Add a new >>> function of_pm_clk_add_clks() to allows device clocks to be retrieved >>> from device-tree and populated for a given device. Note that >>> of_clk_get_from_provider() is not defined if CONFIG_OF and >>> CONFIG_COMMON_CLK are not selected. Therefore, make of_pm_clk_add_clks() >>> dependent on these options. >>> >>> An optional function pointer may be passed to of_pm_clk_add_clks() that >>> can be used to filter the clocks that are added for a device when >>> calling of_pm_clk_add_clks(). >>> >>> In order to handle errors encountered when adding clocks from >>> device-tree, add a function pm_clk_remove_clk() to remove any clocks >>> (using a pointer to the clk structure) that have been added >>> successfully before the error occurred. >>> >>> Signed-off-by: Jon Hunter <jonathanh@nvidia.com> >> >> Tested-by: Geert Uytterhoeven <geert+renesas@glider.be> >> >> But more comments below... >> >>> --- a/drivers/base/power/clock_ops.c >>> +++ b/drivers/base/power/clock_ops.c >>> @@ -137,6 +137,81 @@ int pm_clk_add_clk(struct device *dev, struct clk *clk) >>> return __pm_clk_add(dev, NULL, clk); >>> } >>> >>> + >>> +#if defined(CONFIG_OF) && defined(CONFIG_COMMON_CLK) >>> +/** >>> + * of_pm_clk_add_clks - Start using device clock(s) for power management. >>> + * @dev: Device whose clock(s) is going to be used for power management. >>> + * @of_pm_clk_filter: Optional function for filtering clocks >>> + * >>> + * Add a series of clocks described in the 'clocks' device-tree node for >>> + * a device to the list of clocks used for the power management of @dev. >>> + * If 'of_pm_clk_filter' is specified, then this filter function will be >>> + * called for each clock found and the clock will be added to the list >>> + * of clocks if this function returns true. Return success if clocks are >>> + * added successfully and return a negative error code if adding a clock >>> + * fails or there are no clocks that match with the filter function. >>> + */ >>> +int of_pm_clk_add_clks(struct device *dev, of_pm_clk_filter fn, void *data) >>> +{ >> >> [...] >> >>> + count = of_count_phandle_with_args(dev->of_node, "clocks", >>> + "#clock-cells"); >>> + if (count == 0) >>> + return -ENODEV; >> >> [...] >> >> + return added ? 0 : -ENODEV; >> >> Is it an error condition if no clocks were present in DT, or if no clocks have >> been accepted by the filter function? If not, the caller has to check for >> -ENODEV. Now the caller has to call pm_clk_create() first, before it knows if >> any clocks will be added, it may want to call pm_clk_destroy() later if no >> clocks have been added. > > It seems odd to me that someone would call this function and either > there are no clocks in the DT or they are all filtered out. For example, > most drivers are calling of_clk_get() because there are clocks they need > to add. of_pm_clk_add_clks() would typically be called from the .attach_dev() callback of the PM Domain, for all devices in the PM Domain. However, there may exist devices without clocks properties, or without gateable clocks. >> Alternatively, you could return 0 vs. the actual number of clocks added. > > However, a nice comprise here could be to return the number of clocks > added and let the user decide what to do. I like that idea. Not sure I > understand what you mean by "0 vs. the actual number of clocks added". > Do you just mean the return the number added? Yes, that what I meant: (current) -ENODEV => (new) 0 (current) 0 => (new) number of clocks added Sorry for the confusion. >> I hooked up your code in renesas-cpg-mssr.c, and it worked. >> However, I noticed it added 27 clocks for one device node (rcar_sound), and >> only then I realized that I only want to add the first clock accepted by the >> filter, not all of them, as the others are under driver control. >> I'm not sure this can be handled in the filter function. >> So I can't use your code (for now)... > > I see, that is unfortunate. Are you using the void *data variable in > your filter? If not may be you could use this to indicate a clock has > been 'found' and don't match any further clocks. Perhaps. I'll think about it... > If you cannot use it, then I am tempted to drop the filter function for > now as this simplifies the code. It can always be added later if someone > has a need for it. That's fine for me. Note that if the need arises for the filter function, you'll have to add a new function and make the old one a wrapper, to avoid having to update all existing users that don't use the filter function. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.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 linux-pm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 10/03/16 12:40, Geert Uytterhoeven wrote: > Hi Jon, > > On Thu, Mar 10, 2016 at 1:27 PM, Jon Hunter <jonathanh@nvidia.com> wrote: >> On 10/03/16 09:41, Geert Uytterhoeven wrote: >>> On Thu, Mar 10, 2016 at 10:00 AM, Jon Hunter <jonathanh@nvidia.com> wrote: >>>> The PM clocks framework requires clients to pass either a con-id or a >>>> valid clk pointer in order to add a clock to a device. Add a new >>>> function of_pm_clk_add_clks() to allows device clocks to be retrieved >>>> from device-tree and populated for a given device. Note that >>>> of_clk_get_from_provider() is not defined if CONFIG_OF and >>>> CONFIG_COMMON_CLK are not selected. Therefore, make of_pm_clk_add_clks() >>>> dependent on these options. >>>> >>>> An optional function pointer may be passed to of_pm_clk_add_clks() that >>>> can be used to filter the clocks that are added for a device when >>>> calling of_pm_clk_add_clks(). >>>> >>>> In order to handle errors encountered when adding clocks from >>>> device-tree, add a function pm_clk_remove_clk() to remove any clocks >>>> (using a pointer to the clk structure) that have been added >>>> successfully before the error occurred. >>>> >>>> Signed-off-by: Jon Hunter <jonathanh@nvidia.com> >>> >>> Tested-by: Geert Uytterhoeven <geert+renesas@glider.be> >>> >>> But more comments below... >>> >>>> --- a/drivers/base/power/clock_ops.c >>>> +++ b/drivers/base/power/clock_ops.c >>>> @@ -137,6 +137,81 @@ int pm_clk_add_clk(struct device *dev, struct clk *clk) >>>> return __pm_clk_add(dev, NULL, clk); >>>> } >>>> >>>> + >>>> +#if defined(CONFIG_OF) && defined(CONFIG_COMMON_CLK) >>>> +/** >>>> + * of_pm_clk_add_clks - Start using device clock(s) for power management. >>>> + * @dev: Device whose clock(s) is going to be used for power management. >>>> + * @of_pm_clk_filter: Optional function for filtering clocks >>>> + * >>>> + * Add a series of clocks described in the 'clocks' device-tree node for >>>> + * a device to the list of clocks used for the power management of @dev. >>>> + * If 'of_pm_clk_filter' is specified, then this filter function will be >>>> + * called for each clock found and the clock will be added to the list >>>> + * of clocks if this function returns true. Return success if clocks are >>>> + * added successfully and return a negative error code if adding a clock >>>> + * fails or there are no clocks that match with the filter function. >>>> + */ >>>> +int of_pm_clk_add_clks(struct device *dev, of_pm_clk_filter fn, void *data) >>>> +{ >>> >>> [...] >>> >>>> + count = of_count_phandle_with_args(dev->of_node, "clocks", >>>> + "#clock-cells"); >>>> + if (count == 0) >>>> + return -ENODEV; >>> >>> [...] >>> >>> + return added ? 0 : -ENODEV; >>> >>> Is it an error condition if no clocks were present in DT, or if no clocks have >>> been accepted by the filter function? If not, the caller has to check for >>> -ENODEV. Now the caller has to call pm_clk_create() first, before it knows if >>> any clocks will be added, it may want to call pm_clk_destroy() later if no >>> clocks have been added. >> >> It seems odd to me that someone would call this function and either >> there are no clocks in the DT or they are all filtered out. For example, >> most drivers are calling of_clk_get() because there are clocks they need >> to add. > > of_pm_clk_add_clks() would typically be called from the .attach_dev() > callback of the PM Domain, for all devices in the PM Domain. However, there may > exist devices without clocks properties, or without gateable clocks. I was planning to use it from within the probe of a driver wishing to use PM_CLK framework in general. Adding them automatically could be nice, but like you have pointed out, some drivers want specific control over certain clocks. May be having a "pm-clocks" node in the DT would be needed for something like this. Or some way of saying it can be controlled by PM_CLK. >>> Alternatively, you could return 0 vs. the actual number of clocks added. >> >> However, a nice comprise here could be to return the number of clocks >> added and let the user decide what to do. I like that idea. Not sure I >> understand what you mean by "0 vs. the actual number of clocks added". >> Do you just mean the return the number added? > > Yes, that what I meant: > (current) -ENODEV => (new) 0 > (current) 0 => (new) number of clocks added > > Sorry for the confusion. That's what I thought you meant and sounds good to me. >>> I hooked up your code in renesas-cpg-mssr.c, and it worked. >>> However, I noticed it added 27 clocks for one device node (rcar_sound), and >>> only then I realized that I only want to add the first clock accepted by the >>> filter, not all of them, as the others are under driver control. >>> I'm not sure this can be handled in the filter function. >>> So I can't use your code (for now)... >> >> I see, that is unfortunate. Are you using the void *data variable in >> your filter? If not may be you could use this to indicate a clock has >> been 'found' and don't match any further clocks. > > Perhaps. I'll think about it... Another option is to add another variable called "max_clocks" which if -1 means there is no user max and defer to the 'count' for the number of clocks to be added. Where 'count' is the number of clocks in the DT blob for the device. Obviously if you set max_clocks = 200 and there are only 2 clocks in the DT blob only 2 would be added. It would also be possible to pass the 'added' number of clocks to the filter and so the filter could decided if it wanted to keep matching or not. However, you could equally keep track of that with the void *data variable yourself and so may be that is redundant. >> If you cannot use it, then I am tempted to drop the filter function for >> now as this simplifies the code. It can always be added later if someone >> has a need for it. > > That's fine for me. > > Note that if the need arises for the filter function, you'll have to add a new > function and make the old one a wrapper, to avoid having to update all > existing users that don't use the filter function. If it is not something that we can sort out now, then I would rather not added it. However, may be a wrapper would not be so bad after all. Even with the above proposal of another variable called 'max_clocks' I would be tempted to split the functions into two, for example, of_pm_clk_add_clks() and of_pm_clk_add_clks_match(), where the later uses the filter and the former does not. Cheers Jon -- To unsubscribe from this list: send the line "unsubscribe linux-pm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/base/power/clock_ops.c b/drivers/base/power/clock_ops.c index 272a52ebafc0..d912944a1a9e 100644 --- a/drivers/base/power/clock_ops.c +++ b/drivers/base/power/clock_ops.c @@ -137,6 +137,81 @@ int pm_clk_add_clk(struct device *dev, struct clk *clk) return __pm_clk_add(dev, NULL, clk); } + +#if defined(CONFIG_OF) && defined(CONFIG_COMMON_CLK) +/** + * of_pm_clk_add_clks - Start using device clock(s) for power management. + * @dev: Device whose clock(s) is going to be used for power management. + * @of_pm_clk_filter: Optional function for filtering clocks + * + * Add a series of clocks described in the 'clocks' device-tree node for + * a device to the list of clocks used for the power management of @dev. + * If 'of_pm_clk_filter' is specified, then this filter function will be + * called for each clock found and the clock will be added to the list + * of clocks if this function returns true. Return success if clocks are + * added successfully and return a negative error code if adding a clock + * fails or there are no clocks that match with the filter function. + */ +int of_pm_clk_add_clks(struct device *dev, of_pm_clk_filter fn, void *data) +{ + struct of_phandle_args clkspec; + struct clk **clks; + unsigned int i, count, added = 0; + int ret; + + if (!dev || !dev->of_node) + return -EINVAL; + + count = of_count_phandle_with_args(dev->of_node, "clocks", + "#clock-cells"); + if (count == 0) + return -ENODEV; + + clks = kcalloc(count, sizeof(*clks), GFP_KERNEL); + if (!clks) + return -ENOMEM; + + for (i = 0; i < count; i++) { + if (of_parse_phandle_with_args(dev->of_node, "clocks", + "#clock-cells", i, &clkspec)) + goto error; + + if (fn && !fn(&clkspec, data)) { + of_node_put(clkspec.np); + continue; + } + + clks[added] = of_clk_get_from_provider(&clkspec); + of_node_put(clkspec.np); + + if (IS_ERR(clks[added])) { + ret = PTR_ERR(clks[added]); + goto error; + } + + ret = pm_clk_add_clk(dev, clks[added]); + if (ret) { + clk_put(clks[added]); + goto error; + } + + added++; + } + + kfree(clks); + + return added ? 0 : -ENODEV; + +error: + while (added--) + pm_clk_remove_clk(dev, clks[added]); + + kfree(clks); + + return ret; +} +#endif /* CONFIG_OF && CONFIG_COMMON_CLK */ + /** * __pm_clk_remove - Destroy PM clock entry. * @ce: PM clock entry to destroy. @@ -198,6 +273,39 @@ void pm_clk_remove(struct device *dev, const char *con_id) } /** + * pm_clk_remove_clk - Stop using a device clock for power management. + * @dev: Device whose clock should not be used for PM any more. + * @clk: Clock pointer + * + * Remove the clock pointed to by @clk from the list of clocks used for + * the power management of @dev. + */ +void pm_clk_remove_clk(struct device *dev, struct clk *clk) +{ + struct pm_subsys_data *psd = dev_to_psd(dev); + struct pm_clock_entry *ce; + + if (!psd || !clk) + return; + + spin_lock_irq(&psd->lock); + + list_for_each_entry(ce, &psd->clock_list, node) { + if (clk == ce->clk) + goto remove; + } + + spin_unlock_irq(&psd->lock); + return; + + remove: + list_del(&ce->node); + spin_unlock_irq(&psd->lock); + + __pm_clk_remove(ce); +} + +/** * pm_clk_init - Initialize a device's list of power management clocks. * @dev: Device to initialize the list of PM clocks for. * diff --git a/include/linux/pm_clock.h b/include/linux/pm_clock.h index 25266c600021..2de3a71bd624 100644 --- a/include/linux/pm_clock.h +++ b/include/linux/pm_clock.h @@ -11,6 +11,7 @@ #include <linux/device.h> #include <linux/notifier.h> +#include <linux/of.h> struct pm_clk_notifier_block { struct notifier_block nb; @@ -43,6 +44,7 @@ extern void pm_clk_destroy(struct device *dev); extern int pm_clk_add(struct device *dev, const char *con_id); extern int pm_clk_add_clk(struct device *dev, struct clk *clk); extern void pm_clk_remove(struct device *dev, const char *con_id); +extern void pm_clk_remove_clk(struct device *dev, struct clk *clk); extern int pm_clk_suspend(struct device *dev); extern int pm_clk_resume(struct device *dev); #else @@ -74,6 +76,9 @@ static inline void pm_clk_remove(struct device *dev, const char *con_id) } #define pm_clk_suspend NULL #define pm_clk_resume NULL +static inline void pm_clk_remove_clk(struct device *dev, struct clk *clk) +{ +} #endif #ifdef CONFIG_HAVE_CLK @@ -86,4 +91,18 @@ static inline void pm_clk_add_notifier(struct bus_type *bus, } #endif +typedef bool (*of_pm_clk_filter)(const struct of_phandle_args *args, + void *data); + +#if defined(CONFIG_PM_CLK) && defined(CONFIG_OF) && defined(CONFIG_COMMON_CLK) +extern int of_pm_clk_add_clks(struct device *dev, of_pm_clk_filter fn, + void *data); +#else +static inline int of_pm_clk_add_clks(struct device *dev, of_pm_clk_filter fn, + void *data) +{ + return -EINVAL; +} +#endif + #endif
The PM clocks framework requires clients to pass either a con-id or a valid clk pointer in order to add a clock to a device. Add a new function of_pm_clk_add_clks() to allows device clocks to be retrieved from device-tree and populated for a given device. Note that of_clk_get_from_provider() is not defined if CONFIG_OF and CONFIG_COMMON_CLK are not selected. Therefore, make of_pm_clk_add_clks() dependent on these options. An optional function pointer may be passed to of_pm_clk_add_clks() that can be used to filter the clocks that are added for a device when calling of_pm_clk_add_clks(). In order to handle errors encountered when adding clocks from device-tree, add a function pm_clk_remove_clk() to remove any clocks (using a pointer to the clk structure) that have been added successfully before the error occurred. Signed-off-by: Jon Hunter <jonathanh@nvidia.com> --- Changes v2-v3: - Constified the of_phandle_args parameter - Added data parameter to filter function Changes v1-v2: - Added support for optional filter function as suggested by Geert U. drivers/base/power/clock_ops.c | 108 +++++++++++++++++++++++++++++++++++++++++ include/linux/pm_clock.h | 19 ++++++++ 2 files changed, 127 insertions(+)