[PATCH] cmd: pwm: Remove additional pwm description

Simon Glass sjg at chromium.org
Thu Aug 12 23:46:29 CEST 2021


On Tue, 10 Aug 2021 at 07:50, Michal Simek <michal.simek at xilinx.com> wrote:
>
> The first name is taken from command name that's why shouldn't be listed in
> help. And commands shouldn't be listed with <> which means value but value
> itself is command name.
> Also add description for commands to make it clear what it does.
>
> Before
> pwm pwm <invert> <pwm_dev_num> <channel> <polarity>
> pwm <config> <pwm_dev_num> <channel> <period_ns> <duty_ns>
> ...
>
> After:
> pwm invert <pwm_dev_num> <channel> <polarity> - invert polarity
> pwm config <pwm_dev_num> <channel> <period_ns> <duty_ns> - config PWM
> pwm enable <pwm_dev_num> <channel> - enable PWM output
> pwm disable <pwm_dev_num> <channel> - disable PWM output
>
> Signed-off-by: Michal Simek <michal.simek at xilinx.com>
> ---
>
>  cmd/pwm.c | 7 ++++---
>  1 file changed, 4 insertions(+), 3 deletions(-)
>

Reviewed-by: Simon Glass <sjg at chromium.org>


More information about the U-Boot mailing list