2023-12-12 08:34:50

by Sean Young

[permalink] [raw]
Subject: [PATCH v8 2/6] pwm: Replace ENOTSUPP with EOPNOTSUPP

ENOTSUPP is not a standard error code and should be avoided.

Signed-off-by: Sean Young <[email protected]>
---
include/linux/pwm.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/include/linux/pwm.h b/include/linux/pwm.h
index b64b8a82415c4..c9cb87b59ac80 100644
--- a/include/linux/pwm.h
+++ b/include/linux/pwm.h
@@ -407,12 +407,12 @@ static inline int pwm_apply_might_sleep(struct pwm_device *pwm,
const struct pwm_state *state)
{
might_sleep();
- return -ENOTSUPP;
+ return -EOPNOTSUPP;
}

static inline int pwm_adjust_config(struct pwm_device *pwm)
{
- return -ENOTSUPP;
+ return -EOPNOTSUPP;
}

static inline int pwm_config(struct pwm_device *pwm, int duty_ns,
--
2.43.0


2023-12-12 11:31:43

by Uwe Kleine-König

[permalink] [raw]
Subject: Re: [PATCH v8 2/6] pwm: Replace ENOTSUPP with EOPNOTSUPP

On Tue, Dec 12, 2023 at 08:34:01AM +0000, Sean Young wrote:
> ENOTSUPP is not a standard error code and should be avoided.

I'd write:

According to Documentation/dev-tools/checkpatch.rst ENOTSUPP is
not recommended and EOPNOTSUPP should be used instead.

to give the sentence a bit of authority.

Other than that I'm fine with the change.

Best regards
Uwe

--
Pengutronix e.K. | Uwe Kleine-K?nig |
Industrial Linux Solutions | https://www.pengutronix.de/ |


Attachments:
(No filename) (533.00 B)
signature.asc (499.00 B)
Download all attachments