* [PATCH] PM / Domains: replace -ENOTSUPP with -EOPNOTSUPP
@ 2020-11-06 16:49 Lina Iyer
2020-11-09 11:39 ` Ulf Hansson
0 siblings, 1 reply; 4+ messages in thread
From: Lina Iyer @ 2020-11-06 16:49 UTC (permalink / raw)
To: rjw, ulf.hansson; +Cc: linux-pm, linux-arm-msm, Lina Iyer
While submitting a patch to add next_wakeup, checkpatch reported this -
WARNING: ENOTSUPP is not a SUSV4 error code, prefer EOPNOTSUPP
+ return -ENOTSUPP;
Address the above warning in other functions in pm_domain.h.
Signed-off-by: Lina Iyer <ilina@codeaurora.org>
---
include/linux/pm_domain.h | 14 +++++++-------
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/include/linux/pm_domain.h b/include/linux/pm_domain.h
index 49982cd58bfd..e390388e6c17 100644
--- a/include/linux/pm_domain.h
+++ b/include/linux/pm_domain.h
@@ -259,24 +259,24 @@ static inline int pm_genpd_init(struct generic_pm_domain *genpd,
}
static inline int pm_genpd_remove(struct generic_pm_domain *genpd)
{
- return -ENOTSUPP;
+ return -EOPNOTSUPP;
}
static inline int dev_pm_genpd_set_performance_state(struct device *dev,
unsigned int state)
{
- return -ENOTSUPP;
+ return -EOPNOTSUPP;
}
static inline int dev_pm_genpd_add_notifier(struct device *dev,
struct notifier_block *nb)
{
- return -ENOTSUPP;
+ return -EOPNOTSUPP;
}
static inline int dev_pm_genpd_remove_notifier(struct device *dev)
{
- return -ENOTSUPP;
+ return -EOPNOTSUPP;
}
static inline int dev_pm_genpd_set_next_wakeup(struct device *dev, ktime_t next)
@@ -334,13 +334,13 @@ struct device *genpd_dev_pm_attach_by_name(struct device *dev,
static inline int of_genpd_add_provider_simple(struct device_node *np,
struct generic_pm_domain *genpd)
{
- return -ENOTSUPP;
+ return -EOPNOTSUPP;
}
static inline int of_genpd_add_provider_onecell(struct device_node *np,
struct genpd_onecell_data *data)
{
- return -ENOTSUPP;
+ return -EOPNOTSUPP;
}
static inline void of_genpd_del_provider(struct device_node *np) {}
@@ -396,7 +396,7 @@ static inline struct device *genpd_dev_pm_attach_by_name(struct device *dev,
static inline
struct generic_pm_domain *of_genpd_remove_last(struct device_node *np)
{
- return ERR_PTR(-ENOTSUPP);
+ return ERR_PTR(-EOPNOTSUPP);
}
#endif /* CONFIG_PM_GENERIC_DOMAINS_OF */
--
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] PM / Domains: replace -ENOTSUPP with -EOPNOTSUPP
2020-11-06 16:49 [PATCH] PM / Domains: replace -ENOTSUPP with -EOPNOTSUPP Lina Iyer
@ 2020-11-09 11:39 ` Ulf Hansson
2020-11-09 15:24 ` Lina Iyer
0 siblings, 1 reply; 4+ messages in thread
From: Ulf Hansson @ 2020-11-09 11:39 UTC (permalink / raw)
To: Lina Iyer; +Cc: Rafael J. Wysocki, Linux PM, linux-arm-msm
On Fri, 6 Nov 2020 at 17:49, Lina Iyer <ilina@codeaurora.org> wrote:
>
> While submitting a patch to add next_wakeup, checkpatch reported this -
>
> WARNING: ENOTSUPP is not a SUSV4 error code, prefer EOPNOTSUPP
> + return -ENOTSUPP;
>
> Address the above warning in other functions in pm_domain.h.
>
> Signed-off-by: Lina Iyer <ilina@codeaurora.org>
I assume you have looked at callers of these functions too, to make
sure they don't explicitly look at -ENOTSUPP?
Reviewed-by: Ulf Hansson <ulf.hansson@linaro.org>
Kind regards
Uffe
> ---
> include/linux/pm_domain.h | 14 +++++++-------
> 1 file changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/include/linux/pm_domain.h b/include/linux/pm_domain.h
> index 49982cd58bfd..e390388e6c17 100644
> --- a/include/linux/pm_domain.h
> +++ b/include/linux/pm_domain.h
> @@ -259,24 +259,24 @@ static inline int pm_genpd_init(struct generic_pm_domain *genpd,
> }
> static inline int pm_genpd_remove(struct generic_pm_domain *genpd)
> {
> - return -ENOTSUPP;
> + return -EOPNOTSUPP;
> }
>
> static inline int dev_pm_genpd_set_performance_state(struct device *dev,
> unsigned int state)
> {
> - return -ENOTSUPP;
> + return -EOPNOTSUPP;
> }
>
> static inline int dev_pm_genpd_add_notifier(struct device *dev,
> struct notifier_block *nb)
> {
> - return -ENOTSUPP;
> + return -EOPNOTSUPP;
> }
>
> static inline int dev_pm_genpd_remove_notifier(struct device *dev)
> {
> - return -ENOTSUPP;
> + return -EOPNOTSUPP;
> }
>
> static inline int dev_pm_genpd_set_next_wakeup(struct device *dev, ktime_t next)
> @@ -334,13 +334,13 @@ struct device *genpd_dev_pm_attach_by_name(struct device *dev,
> static inline int of_genpd_add_provider_simple(struct device_node *np,
> struct generic_pm_domain *genpd)
> {
> - return -ENOTSUPP;
> + return -EOPNOTSUPP;
> }
>
> static inline int of_genpd_add_provider_onecell(struct device_node *np,
> struct genpd_onecell_data *data)
> {
> - return -ENOTSUPP;
> + return -EOPNOTSUPP;
> }
>
> static inline void of_genpd_del_provider(struct device_node *np) {}
> @@ -396,7 +396,7 @@ static inline struct device *genpd_dev_pm_attach_by_name(struct device *dev,
> static inline
> struct generic_pm_domain *of_genpd_remove_last(struct device_node *np)
> {
> - return ERR_PTR(-ENOTSUPP);
> + return ERR_PTR(-EOPNOTSUPP);
> }
> #endif /* CONFIG_PM_GENERIC_DOMAINS_OF */
>
> --
> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
> a Linux Foundation Collaborative Project
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] PM / Domains: replace -ENOTSUPP with -EOPNOTSUPP
2020-11-09 11:39 ` Ulf Hansson
@ 2020-11-09 15:24 ` Lina Iyer
2020-11-09 15:29 ` Ulf Hansson
0 siblings, 1 reply; 4+ messages in thread
From: Lina Iyer @ 2020-11-09 15:24 UTC (permalink / raw)
To: Ulf Hansson; +Cc: Rafael J. Wysocki, Linux PM, linux-arm-msm
On Mon, Nov 09 2020 at 04:40 -0700, Ulf Hansson wrote:
>On Fri, 6 Nov 2020 at 17:49, Lina Iyer <ilina@codeaurora.org> wrote:
>>
>> While submitting a patch to add next_wakeup, checkpatch reported this -
>>
>> WARNING: ENOTSUPP is not a SUSV4 error code, prefer EOPNOTSUPP
>> + return -ENOTSUPP;
>>
>> Address the above warning in other functions in pm_domain.h.
>>
>> Signed-off-by: Lina Iyer <ilina@codeaurora.org>
>
>I assume you have looked at callers of these functions too, to make
>sure they don't explicitly look at -ENOTSUPP?
>
Yes, the first level callers at least. Most of them seem to return from
probe calls etc. Is there a nice way to make sure the second level
functions and the callers of those are not using -ENOTSUPP return from
these functions.
>Reviewed-by: Ulf Hansson <ulf.hansson@linaro.org>
>
Thanks.
-- Lina
>Kind regards
>Uffe
>
>> ---
>> include/linux/pm_domain.h | 14 +++++++-------
>> 1 file changed, 7 insertions(+), 7 deletions(-)
>>
>> diff --git a/include/linux/pm_domain.h b/include/linux/pm_domain.h
>> index 49982cd58bfd..e390388e6c17 100644
>> --- a/include/linux/pm_domain.h
>> +++ b/include/linux/pm_domain.h
>> @@ -259,24 +259,24 @@ static inline int pm_genpd_init(struct generic_pm_domain *genpd,
>> }
>> static inline int pm_genpd_remove(struct generic_pm_domain *genpd)
>> {
>> - return -ENOTSUPP;
>> + return -EOPNOTSUPP;
>> }
>>
>> static inline int dev_pm_genpd_set_performance_state(struct device *dev,
>> unsigned int state)
>> {
>> - return -ENOTSUPP;
>> + return -EOPNOTSUPP;
>> }
>>
>> static inline int dev_pm_genpd_add_notifier(struct device *dev,
>> struct notifier_block *nb)
>> {
>> - return -ENOTSUPP;
>> + return -EOPNOTSUPP;
>> }
>>
>> static inline int dev_pm_genpd_remove_notifier(struct device *dev)
>> {
>> - return -ENOTSUPP;
>> + return -EOPNOTSUPP;
>> }
>>
>> static inline int dev_pm_genpd_set_next_wakeup(struct device *dev, ktime_t next)
>> @@ -334,13 +334,13 @@ struct device *genpd_dev_pm_attach_by_name(struct device *dev,
>> static inline int of_genpd_add_provider_simple(struct device_node *np,
>> struct generic_pm_domain *genpd)
>> {
>> - return -ENOTSUPP;
>> + return -EOPNOTSUPP;
>> }
>>
>> static inline int of_genpd_add_provider_onecell(struct device_node *np,
>> struct genpd_onecell_data *data)
>> {
>> - return -ENOTSUPP;
>> + return -EOPNOTSUPP;
>> }
>>
>> static inline void of_genpd_del_provider(struct device_node *np) {}
>> @@ -396,7 +396,7 @@ static inline struct device *genpd_dev_pm_attach_by_name(struct device *dev,
>> static inline
>> struct generic_pm_domain *of_genpd_remove_last(struct device_node *np)
>> {
>> - return ERR_PTR(-ENOTSUPP);
>> + return ERR_PTR(-EOPNOTSUPP);
>> }
>> #endif /* CONFIG_PM_GENERIC_DOMAINS_OF */
>>
>> --
>> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
>> a Linux Foundation Collaborative Project
>>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] PM / Domains: replace -ENOTSUPP with -EOPNOTSUPP
2020-11-09 15:24 ` Lina Iyer
@ 2020-11-09 15:29 ` Ulf Hansson
0 siblings, 0 replies; 4+ messages in thread
From: Ulf Hansson @ 2020-11-09 15:29 UTC (permalink / raw)
To: Lina Iyer; +Cc: Rafael J. Wysocki, Linux PM, linux-arm-msm
On Mon, 9 Nov 2020 at 16:24, Lina Iyer <ilina@codeaurora.org> wrote:
>
> On Mon, Nov 09 2020 at 04:40 -0700, Ulf Hansson wrote:
> >On Fri, 6 Nov 2020 at 17:49, Lina Iyer <ilina@codeaurora.org> wrote:
> >>
> >> While submitting a patch to add next_wakeup, checkpatch reported this -
> >>
> >> WARNING: ENOTSUPP is not a SUSV4 error code, prefer EOPNOTSUPP
> >> + return -ENOTSUPP;
> >>
> >> Address the above warning in other functions in pm_domain.h.
> >>
> >> Signed-off-by: Lina Iyer <ilina@codeaurora.org>
> >
> >I assume you have looked at callers of these functions too, to make
> >sure they don't explicitly look at -ENOTSUPP?
> >
> Yes, the first level callers at least. Most of them seem to return from
> probe calls etc. Is there a nice way to make sure the second level
> functions and the callers of those are not using -ENOTSUPP return from
> these functions.
Good question. But I can't think of one (besides code inspection) -
unless you have a running system.
>
> >Reviewed-by: Ulf Hansson <ulf.hansson@linaro.org>
Anyway, let's give this a try and see what happens!
Kind regards
Uffe
--
> >> 1 file changed, 7 insertions(+), 7 deletions(-)
> >>
> >> diff --git a/include/linux/pm_domain.h b/include/linux/pm_domain.h
> >> index 49982cd58bfd..e390388e6c17 100644
> >> --- a/include/linux/pm_domain.h
> >> +++ b/include/linux/pm_domain.h
> >> @@ -259,24 +259,24 @@ static inline int pm_genpd_init(struct generic_pm_domain *genpd,
> >> }
> >> static inline int pm_genpd_remove(struct generic_pm_domain *genpd)
> >> {
> >> - return -ENOTSUPP;
> >> + return -EOPNOTSUPP;
> >> }
> >>
> >> static inline int dev_pm_genpd_set_performance_state(struct device *dev,
> >> unsigned int state)
> >> {
> >> - return -ENOTSUPP;
> >> + return -EOPNOTSUPP;
> >> }
> >>
> >> static inline int dev_pm_genpd_add_notifier(struct device *dev,
> >> struct notifier_block *nb)
> >> {
> >> - return -ENOTSUPP;
> >> + return -EOPNOTSUPP;
> >> }
> >>
> >> static inline int dev_pm_genpd_remove_notifier(struct device *dev)
> >> {
> >> - return -ENOTSUPP;
> >> + return -EOPNOTSUPP;
> >> }
> >>
> >> static inline int dev_pm_genpd_set_next_wakeup(struct device *dev, ktime_t next)
> >> @@ -334,13 +334,13 @@ struct device *genpd_dev_pm_attach_by_name(struct device *dev,
> >> static inline int of_genpd_add_provider_simple(struct device_node *np,
> >> struct generic_pm_domain *genpd)
> >> {
> >> - return -ENOTSUPP;
> >> + return -EOPNOTSUPP;
> >> }
> >>
> >> static inline int of_genpd_add_provider_onecell(struct device_node *np,
> >> struct genpd_onecell_data *data)
> >> {
> >> - return -ENOTSUPP;
> >> + return -EOPNOTSUPP;
> >> }
> >>
> >> static inline void of_genpd_del_provider(struct device_node *np) {}
> >> @@ -396,7 +396,7 @@ static inline struct device *genpd_dev_pm_attach_by_name(struct device *dev,
> >> static inline
> >> struct generic_pm_domain *of_genpd_remove_last(struct device_node *np)
> >> {
> >> - return ERR_PTR(-ENOTSUPP);
> >> + return ERR_PTR(-EOPNOTSUPP);
> >> }
> >> #endif /* CONFIG_PM_GENERIC_DOMAINS_OF */
> >>
> >> --
> >> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
> >> a Linux Foundation Collaborative Project
> >>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-11-09 15:30 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-11-06 16:49 [PATCH] PM / Domains: replace -ENOTSUPP with -EOPNOTSUPP Lina Iyer
2020-11-09 11:39 ` Ulf Hansson
2020-11-09 15:24 ` Lina Iyer
2020-11-09 15:29 ` Ulf Hansson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).