linux-kernel - Re: [PATCH 3/5 v7] dt-bindings: input: pwm-beeper: add documentation for volume devcetree bindings

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <24da8988-193c-fe6d-7a7e-32526caeeae4@kernel.org>
Date:   Tue, 24 Jan 2023 17:55:03 +0100
From:   Krzysztof Kozlowski <krzk@...nel.org>
To:     Manuel Traut <manuel.traut@...com>, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 3/5 v7] dt-bindings: input: pwm-beeper: add documentation
 for volume devcetree bindings

On 24/01/2023 17:41, Manuel Traut wrote:
> This patch adds the documentation for the devicetree bindings to set
> the volume levels.

Please use scripts/get_maintainers.pl to get a list of necessary people
and lists to CC.  It might happen, that command when run on an older
kernel, gives you outdated entries.  Therefore please be sure you base
your patches on recent Linux kernel. You did not cc anyone, so that
won't work.

Do not use "This commit/patch".
https://elixir.bootlin.com/linux/v5.17.1/source/Documentation/process/submitting-patches.rst#L95

Subject: drop second/last, redundant "documentation and "devicetree
bindings". The "dt-bindings" prefix is already stating that these are
bindings.

> 
> Signed-off-by: Frieder Schrempf <frieder.schrempf@...tron.de>
> Signed-off-by: Manuel Traut <manuel.traut@...com>
> Acked-by: Rob Herring <robh@...nel.org>
> ---
>  .../devicetree/bindings/input/pwm-beeper.yaml | 19 +++++++++++++++++++
>  1 file changed, 19 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/input/pwm-beeper.yaml b/Documentation/devicetree/bindings/input/pwm-beeper.yaml
> index 24a40f0f77c1..fe1bbc10147b 100644
> --- a/Documentation/devicetree/bindings/input/pwm-beeper.yaml
> +++ b/Documentation/devicetree/bindings/input/pwm-beeper.yaml
> @@ -28,6 +28,23 @@ properties:
>  	beeper-hz:
>  		description: bell frequency in Hz
>  
> +
> +	volume-levels:
> +		description: >
> +			Array of PWM duty cycle values that correspond to
> +			linear volume levels. These need to be in the range of
> +			0 to 500, while 0 means 0% duty cycle (mute) and 500
> +			means 50% duty cycle (max volume).
> +			Please note that the actual volume of most beepers is
> +			highly non-linear, which means that low volume levels
> +			are probably somewhere in the range of 1 to 30 (0.1-3%
> +			duty cycle).

Wrong indentation. This should use property units -percent. If it is not
percent, then missing ref.
https://github.com/devicetree-org/dt-schema/blob/main/dtschema/schemas/property-units.yaml

I doubt this was even tested...

> +
> +	default-volume-level:
> +		description: >
> +			The default volume level (index into the array defined
> +			by the "volume-levels" property).

Same problem.

> +
>  required:

Best regards,
Krzysztof