lkml.org 
[lkml]   [2016]   [Mar]   [31]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: [PATCH v5 32/46] pwm: deprecate pwm_config(), pwm_enable() and pwm_disable()
Hi Boris,

On Wed, Mar 30, 2016 at 10:03:55PM +0200, Boris Brezillon wrote:
> Prefix those function as deprecated to encourage all existing users to
> switch to pwm_apply_state().

Why not keep at least some of them as wrappers where we do not need to
chnage several parameters at once? It is much easier to have a driver
do:

error = pwm_enable(pwm);
if (error)
...

rather than declaring the state variable, fectch it, adjust and then
apply.

>
> Signed-off-by: Boris Brezillon <boris.brezillon@free-electrons.com>
> ---
> include/linux/pwm.h | 24 ++++++++++++------------
> 1 file changed, 12 insertions(+), 12 deletions(-)
>
> diff --git a/include/linux/pwm.h b/include/linux/pwm.h
> index 4aad4eb..9bac10f 100644
> --- a/include/linux/pwm.h
> +++ b/include/linux/pwm.h
> @@ -225,8 +225,8 @@ int pwm_apply_state(struct pwm_device *pwm, struct pwm_state *state);
> *
> * Returns: 0 on success or a negative error code on failure.
> */
> -static inline int pwm_config(struct pwm_device *pwm, int duty_ns,
> - int period_ns)
> +static inline int __deprecated pwm_config(struct pwm_device *pwm, int duty_ns,
> + int period_ns)
> {
> struct pwm_state pstate;
>
> @@ -252,8 +252,8 @@ static inline int pwm_config(struct pwm_device *pwm, int duty_ns,
> *
> * Returns: 0 on success or a negative error code on failure.
> */
> -static inline int pwm_set_polarity(struct pwm_device *pwm,
> - enum pwm_polarity polarity)
> +static inline int __deprecated pwm_set_polarity(struct pwm_device *pwm,
> + enum pwm_polarity polarity)
> {
> struct pwm_state pstate;
>
> @@ -284,7 +284,7 @@ static inline int pwm_set_polarity(struct pwm_device *pwm,
> *
> * Returns: 0 on success or a negative error code on failure.
> */
> -static inline int pwm_enable(struct pwm_device *pwm)
> +static inline int __deprecated pwm_enable(struct pwm_device *pwm)
> {
> struct pwm_state pstate;
>
> @@ -303,7 +303,7 @@ static inline int pwm_enable(struct pwm_device *pwm)
> * pwm_disable() - stop a PWM output toggling
> * @pwm: PWM device
> */
> -static inline void pwm_disable(struct pwm_device *pwm)
> +static inline void __deprecated pwm_disable(struct pwm_device *pwm)
> {
> struct pwm_state pstate;
>
> @@ -360,24 +360,24 @@ static inline int pwm_apply_state(struct pwm_device *pwm,
> return -ENOTSUPP;
> }
>
> -static inline int pwm_config(struct pwm_device *pwm, int duty_ns,
> - int period_ns)
> +static inline int __deprecated pwm_config(struct pwm_device *pwm, int duty_ns,
> + int period_ns)
> {
> return -EINVAL;
> }
>
> -static inline int pwm_set_polarity(struct pwm_device *pwm,
> - enum pwm_polarity polarity)
> +static inline int __deprecated pwm_set_polarity(struct pwm_device *pwm,
> + enum pwm_polarity polarity)
> {
> return -ENOTSUPP;
> }
>
> -static inline int pwm_enable(struct pwm_device *pwm)
> +static inline int __deprecated pwm_enable(struct pwm_device *pwm)
> {
> return -EINVAL;
> }
>
> -static inline void pwm_disable(struct pwm_device *pwm)
> +static inline void __deprecated pwm_disable(struct pwm_device *pwm)
> {
> }
>
> --
> 2.5.0
>

--
Dmitry

\
 
 \ /
  Last update: 2016-03-31 20:01    [W:3.372 / U:0.424 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site