linux-kernel - Re: [PATCH 1/3 v4] ASoC: add hi6210-i2s DT 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]

Date:   Mon, 10 Apr 2017 19:41:19 +0100
From:   Mark Brown <broonie@...nel.org>
To:     John Stultz <john.stultz@...aro.org>
Cc:     lkml <linux-kernel@...r.kernel.org>,
        Zhangfei Gao <zhangfei.gao@...aro.org>,
        Liam Girdwood <lgirdwood@...il.com>,
        Jaroslav Kysela <perex@...ex.cz>,
        Takashi Iwai <tiwai@...e.com>, Wei Xu <xuwei5@...ilicon.com>,
        Rob Herring <robh+dt@...nel.org>,
        Andy Green <andy@...mcat.com>,
        Dave Long <dave.long@...aro.org>,
        Guodong Xu <guodong.xu@...aro.org>
Subject: Re: [PATCH 1/3 v4] ASoC: add hi6210-i2s DT bindings

On Mon, Apr 10, 2017 at 11:37:00AM -0700, John Stultz wrote:
> On Mon, Apr 10, 2017 at 11:13 AM, Mark Brown <broonie@...nel.org> wrote:
> > On Fri, Mar 31, 2017 at 03:05:59PM -0700, John Stultz wrote:

> >> +- #sound-dai-cells: Should be set to 1 (for multi-dai)

> > Please submit a followup patch documenting what these mean, at least for
> > what's in the code.

> So just to make sure I understand, you want documentation on what the
> sound-dia-cells index maps to? (ie 0 to S2, as that's the only one

*dai*

> supported right now?) Or something else?

Yes, that.   People need to be able to tell what the binding means.


Download attachment "signature.asc" of type "application/pgp-signature" (489 bytes)