linux-kernel - Re: [PATCH 14/17] ASoC: add simple-graph-card document

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: <87wpgg9d5t.wl%kuninori.morimoto.gx@renesas.com>
Date:   Mon, 7 Nov 2016 03:34:40 +0000
From:   Kuninori Morimoto <kuninori.morimoto.gx@...esas.com>
To:     Mark Brown <broonie@...nel.org>
CC:     Rob Herring <robh+dt@...nel.org>,
        Linux-ALSA <alsa-devel@...a-project.org>,
        Liam Girdwood <lgirdwood@...il.com>,
        Simon <horms@...ge.net.au>,
        Laurent <laurent.pinchart@...asonboard.com>,
        Guennadi <g.liakhovetski@....de>,
        Grant Likely <grant.likely@...aro.org>,
        Frank Rowand <frowand.list@...il.com>,
        Linux-DT <devicetree@...r.kernel.org>,
        Linux-Kernel <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 14/17] ASoC: add simple-graph-card document


Hi Mark

> This document really needs quite a bit of fleshing out but I'm not sure
> that should be a blocker for the series as a whole especially given that
> English is not your native language - we can build out later.  I think

I'm sorry about my English...

> One thing I'm not 100% clear on here is why it has to be a CPU DAI (I'm
> guessing just one of them though the above says ports as an option?)
> that creates the card?  Is there a concrete reason for that or is it
> just being defined as good pracctice?

This explain was not understandable, indeed.

This means, CPU side's port on DT need to have compatible,
and CPU side driver need to call asoc_simple_card_try_to_probe_graph_card()
(= [PATCH 12/17]). asoc_simple_card_try_to_probe_graph_card()
will check this compatible, and probe card driver.
I will fixup and repost v3 after of-graph people's review

		compatible = "asoc-simple-graph-card";