linux-kernel - Re: [alsa-devel] [PATCH 0/3] ASoC: stm32: sai: add support of iec958 controls

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: <2ae6c5b6-872a-9aaa-6ca4-e29adb7c8bc0@st.com>
Date:   Tue, 5 Jun 2018 17:50:57 +0200
From:   Arnaud Pouliquen <arnaud.pouliquen@...com>
To:     Mark Brown <broonie@...nel.org>,
        Olivier MOYSAN <olivier.moysan@...com>
CC:     "mark.rutland@....com" <mark.rutland@....com>,
        "robh@...nel.org" <robh@...nel.org>,
        "alsa-devel@...a-project.org" <alsa-devel@...a-project.org>,
        Alexandre TORGUE <alexandre.torgue@...com>,
        "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "tiwai@...e.com" <tiwai@...e.com>, "jsarha@...com" <jsarha@...com>,
        "lgirdwood@...il.com" <lgirdwood@...il.com>,
        "rmk@....linux.org.uk" <rmk@....linux.org.uk>,
        "mcoquelin.stm32@...il.com" <mcoquelin.stm32@...il.com>,
        Benjamin GAIGNARD <benjamin.gaignard@...com>,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>,
        "kernel@...inux.com" <kernel@...inux.com>
Subject: Re: [alsa-devel] [PATCH 0/3] ASoC: stm32: sai: add support of iec958
 controls

Hi Takashi,

On 04/17/2018 01:17 PM, Mark Brown wrote:
> On Tue, Apr 17, 2018 at 08:29:17AM +0000, Olivier MOYSAN wrote:
> 
>> I guess the blocking patch in this patchset is the patch "add IEC958 
>> channel status control helper". This patch has been reviewed several 
>> times, but did not get a ack so far.
>> If you think these helpers will not be merged, I will reintegrate the 
>> corresponding code in stm driver.
>> Please let me know, if I need to prepare a v2 without helpers, or if we 
>> can go further in the review of iec helpers patch ?
> 
> I don't mind either way but you're right here, I'm waiting for Takashi
> to review the first patch.  I'd probably be OK with it just integrated
> into the driver if we have to go that way though.

Gentlemen reminder for this patch set. We would appreciate to have your
feedback on iec helper.
>From our point of view it could be useful to have a generic management
of the iec controls based on helpers instead of redefining them in DAIs.
Having the same behavior for these controls could be useful to ensure
coherence of the control management used by application (for instance
Gstreamer uses it to determine iec raw mode capability for iec61937 streams)


Thanks,
Arnaud