linux-kernel - Re: [PATCH 7/8] ASoC: samsung: Add missing #sound-dai-cells property documentation

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]

Message-id: <5d525a1a-b7a7-8b90-97f9-a59c9e361509@samsung.com>
Date:   Wed, 07 Feb 2018 17:37:33 +0100
From:   Sylwester Nawrocki <s.nawrocki@...sung.com>
To:     Krzysztof Kozlowski <krzk@...nel.org>
Cc:     broonie@...nel.org, lgirdwood@...il.com,
        alsa-devel@...a-project.org, robh+dt@...nel.org,
        devicetree@...r.kernel.org, linux-samsung-soc@...r.kernel.org,
        jcsing.lee@...sung.com, sbkim73@...sung.com,
        linux-kernel@...r.kernel.org,
        Bartłomiej Żołnierkiewicz 
        <b.zolnierkie@...sung.com>,
        Marek Szyprowski <m.szyprowski@...sung.com>
Subject: Re: [PATCH 7/8] ASoC: samsung: Add missing #sound-dai-cells
 property documentation

On 02/06/2018 02:02 PM, Krzysztof Kozlowski wrote:
> On Mon, Feb 5, 2018 at 4:44 PM, Sylwester Nawrocki
> <s.nawrocki@...sung.com> wrote:
>> The #sound-dai-cells property might be helpful in selecting primary
>> or secondary CPU DAI and it's already present in i2s nodes for some
>> exynos SoCs so let's add it to the DT binding documentation.
>
> The description above looks like you are documenting existing optional
> property but you added it as a required property.

Actually, it would be better to have it as a required property, but
that could be done with any new device bindings. For the exisitng ones 
I will move this property to the "Optional SoC Specific Properties"
paragraph.

-- 
Regards,
Sylwester