linux-kernel - Re: [PATCH V2 1/5] ASoC: dt-bindings: Adds properties to "awinic,aw88395"

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] [thread-next>] [day] [month] [year] [list]

Date:   Tue, 12 Sep 2023 11:21:22 -0500
From:   Rob Herring <robh@...nel.org>
To:     wangweidong.a@...nic.com
Cc:     lgirdwood@...il.com, broonie@...nel.org,
        krzysztof.kozlowski+dt@...aro.org, conor+dt@...nel.org,
        perex@...ex.cz, tiwai@...e.com, rf@...nsource.cirrus.com,
        shumingf@...ltek.com, ckeepax@...nsource.cirrus.com,
        herve.codina@...tlin.com, 13916275206@....com,
        ryans.lee@...log.com, linus.walleij@...aro.org,
        sebastian.reichel@...labora.com, fido_max@...ox.ru,
        povik+lin@...ebit.org, arnd@...db.de,
        harshit.m.mogalapalli@...cle.com, liweilei@...nic.com,
        yijiangtao@...nic.com, colin.i.king@...il.com, trix@...hat.com,
        alsa-devel@...a-project.org, devicetree@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH V2 1/5] ASoC: dt-bindings: Adds properties to
 "awinic,aw88395"

On Tue, Sep 12, 2023 at 02:58:48PM +0800, wangweidong.a@...nic.com wrote:
> From: Weidong Wang <wangweidong.a@...nic.com>

The subject is still pretty generic. Ideally, we'd never have the same 
subject twice. I'd do something like this:

ASoC: dt-bindings: awinic,aw88395: Add properties for multiple PA support

> 
> Adds properties to "awinic,aw88395" to make files more complete

Complete in what way? multiple PAs, right. It's not really clear 
to me though how these properties enable support for multiple PAs.

> 
> Signed-off-by: Weidong Wang <wangweidong.a@...nic.com>
> ---
>  .../bindings/sound/awinic,aw88395.yaml        | 19 +++++++++++++++++++
>  1 file changed, 19 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/sound/awinic,aw88395.yaml b/Documentation/devicetree/bindings/sound/awinic,aw88395.yaml
> index 4051c2538caf..4965aa4a5370 100644
> --- a/Documentation/devicetree/bindings/sound/awinic,aw88395.yaml
> +++ b/Documentation/devicetree/bindings/sound/awinic,aw88395.yaml
> @@ -32,11 +32,28 @@ properties:
>    reset-gpios:
>      maxItems: 1
>  
> +  awinic,audio-channel:
> +    description:
> +      It is used to distinguish multiple PA devices, so that different
> +      configurations can be loaded to different PA devices
> +    $ref: /schemas/types.yaml#/definitions/uint32
> +    minimum: 0
> +    maximum: 7
> +
> +  awinic,sync-flag:
> +    description:
> +      Flag bit used to keep the phase synchronized in the case of multiple PA
> +    $ref: /schemas/types.yaml#/definitions/uint32
> +    minimum: 0
> +    maximum: 1
> +
>  required:
>    - compatible
>    - reg
>    - '#sound-dai-cells'
>    - reset-gpios
> +  - awinic,audio-channel
> +  - awinic,sync-flag
>  
>  unevaluatedProperties: false
>  
> @@ -51,5 +68,7 @@ examples:
>              reg = <0x34>;
>              #sound-dai-cells = <0>;
>              reset-gpios = <&gpio 10 GPIO_ACTIVE_LOW>;
> +            awinic,audio-channel = <0>;
> +            awinic,sync-flag = <0>;
>          };
>      };
> -- 
> 2.41.0
>