Re: [PATCH 1/2] drivers: pwm: core: Add pwmchip_add_inversed

From: Jonathan Richardson
Date: Tue Jan 06 2015 - 16:16:46 EST


I'm fine with the changes. I'll remove patch 2 (polarity) from my
patches and send them out again based on these. Hopefully we can move
forward with this.

On 15-01-04 05:12 PM, Tim Kryger wrote:
> Add a new function to register a PWM chip with channels that have their
> initial polarity as inversed. This benefits drivers of controllers that
> by default operate with inversed polarity by removing the need to modify
> the polarity during initialization.
>
> Signed-off-by: Tim Kryger <tim.kryger@xxxxxxxxx>
> ---
> drivers/pwm/core.c | 36 ++++++++++++++++++++++++++++--------
> include/linux/pwm.h | 6 ++++++
> 2 files changed, 34 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/pwm/core.c b/drivers/pwm/core.c
> index 966497d..f28c4ce 100644
> --- a/drivers/pwm/core.c
> +++ b/drivers/pwm/core.c
> @@ -222,14 +222,8 @@ void *pwm_get_chip_data(struct pwm_device *pwm)
> }
> EXPORT_SYMBOL_GPL(pwm_get_chip_data);
>
> -/**
> - * pwmchip_add() - register a new PWM chip
> - * @chip: the PWM chip to add
> - *
> - * Register a new PWM chip. If chip->base < 0 then a dynamically assigned base
> - * will be used.
> - */
> -int pwmchip_add(struct pwm_chip *chip)
> +static int pwmchip_add_with_polarity(struct pwm_chip *chip,
> + enum pwm_polarity polarity)
> {
> struct pwm_device *pwm;
> unsigned int i;
> @@ -259,6 +253,7 @@ int pwmchip_add(struct pwm_chip *chip)
> pwm->chip = chip;
> pwm->pwm = chip->base + i;
> pwm->hwpwm = i;
> + pwm->polarity = polarity;
>
> radix_tree_insert(&pwm_tree, pwm->pwm, pwm);
> }
> @@ -279,9 +274,34 @@ out:
> mutex_unlock(&pwm_lock);
> return ret;
> }
> +
> +/**
> + * pwmchip_add() - register a new PWM chip
> + * @chip: the PWM chip to add
> + *
> + * Register a new PWM chip. If chip->base < 0 then a dynamically assigned base
> + * will be used. The initial polarity for all channels is normal.
> + */
> +int pwmchip_add(struct pwm_chip *chip)
> +{
> + return pwmchip_add_with_polarity(chip, PWM_POLARITY_NORMAL);
> +}
> EXPORT_SYMBOL_GPL(pwmchip_add);
>
> /**
> + * pwmchip_add_inversed() - register a new PWM chip
> + * @chip: the PWM chip to add
> + *
> + * Register a new PWM chip. If chip->base < 0 then a dynamically assigned base
> + * will be used. The initial polarity for all channels is inversed.
> + */
> +int pwmchip_add_inversed(struct pwm_chip *chip)
> +{
> + return pwmchip_add_with_polarity(chip, PWM_POLARITY_INVERSED);
> +}
> +EXPORT_SYMBOL_GPL(pwmchip_add_inversed);
> +
> +/**
> * pwmchip_remove() - remove a PWM chip
> * @chip: the PWM chip to remove
> *
> diff --git a/include/linux/pwm.h b/include/linux/pwm.h
> index e90628c..358547f 100644
> --- a/include/linux/pwm.h
> +++ b/include/linux/pwm.h
> @@ -183,6 +183,7 @@ int pwm_set_chip_data(struct pwm_device *pwm, void *data);
> void *pwm_get_chip_data(struct pwm_device *pwm);
>
> int pwmchip_add(struct pwm_chip *chip);
> +int pwmchip_add_inversed(struct pwm_chip *chip);
> int pwmchip_remove(struct pwm_chip *chip);
> struct pwm_device *pwm_request_from_chip(struct pwm_chip *chip,
> unsigned int index,
> @@ -217,6 +218,11 @@ static inline int pwmchip_add(struct pwm_chip *chip)
> return -EINVAL;
> }
>
> +static inline int pwmchip_add_inversed(struct pwm_chip *chip)
> +{
> + return -EINVAL;
> +}
> +
> static inline int pwmchip_remove(struct pwm_chip *chip)
> {
> return -EINVAL;
>

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/