linux-kernel - Re: sof_es8336 build failure [was: [PATCH 5.17 0000/1126] 5.17.2-rc1 review]

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:   Wed, 6 Apr 2022 09:06:21 -0500
From:   Pierre-Louis Bossart <pierre-louis.bossart@...ux.intel.com>
To:     Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Jiri Slaby <jirislaby@...nel.org>
Cc:     linux-kernel@...r.kernel.org, stable@...r.kernel.org,
        torvalds@...ux-foundation.org, akpm@...ux-foundation.org,
        linux@...ck-us.net, shuah@...nel.org, patches@...nelci.org,
        lkft-triage@...ts.linaro.org, pavel@...x.de, jonathanh@...dia.com,
        f.fainelli@...il.com, sudipm.mukherjee@...il.com,
        slade@...dewatkins.com, Mark Brown <broonie@...nel.org>,
        yung-chuan.liao@...ux.intel.com, peter.ujfalusi@...ux.intel.com,
        Jaroslav Kysela <perex@...ex.cz>, Takashi Iwai <tiwai@...e.com>
Subject: Re: sof_es8336 build failure [was: [PATCH 5.17 0000/1126] 5.17.2-rc1
 review]



On 4/6/22 07:28, Greg Kroah-Hartman wrote:
> On Wed, Apr 06, 2022 at 09:37:59AM +0200, Jiri Slaby wrote:
>> Cc some folks and change subject
>>
>> On 06. 04. 22, 9:29, Jiri Slaby wrote:
>>> On 05. 04. 22, 9:12, Greg Kroah-Hartman wrote:
>>>> This is the start of the stable review cycle for the 5.17.2 release.
>>>> There are 1126 patches in this series, all will be posted as a response
>>>> to this one.  If anyone has any issues with these being applied, please
>>>> let me know.
>>> ...
>>>> Pierre-Louis Bossart <pierre-louis.bossart@...ux.intel.com>
>>>>       ASoC: Intel: sof_es8336: use NHLT information to set dmic and SSP
>>>
>>> Fails to build w/ suse x86_64 default config [1] (and x86_32 too):
>>>   > sound/soc/intel/boards/sof_es8336.c: In function ‘sof_es8336_probe’:
>>>   > sound/soc/intel/boards/sof_es8336.c:482:32: error: ‘struct
>>> snd_soc_acpi_mach_params’ has no member named ‘i2s_link_mask’; did you
>>> mean ‘link_mask’?
>>>   >   482 |         if (!mach->mach_params.i2s_link_mask) {
>>>   >       |                                ^~~~~~~~~~~~~
>>>   >       |                                link_mask
>>>   > sound/soc/intel/boards/sof_es8336.c:494:39: error: ‘struct
>>> snd_soc_acpi_mach_params’ has no member named ‘i2s_link_mask’; did you
>>> mean ‘link_mask’?
>>>   >   494 |                 if (mach->mach_params.i2s_link_mask & BIT(2))
>>>   >       |                                       ^~~~~~~~~~~~~
>>>   >       |                                       link_mask
>>>   > sound/soc/intel/boards/sof_es8336.c:496:44: error: ‘struct
>>> snd_soc_acpi_mach_params’ has no member named ‘i2s_link_mask’; did you
>>> mean ‘link_mask’?
>>>   >   496 |                 else if (mach->mach_params.i2s_link_mask &
>>> BIT(1))
>>>   >       |                                            ^~~~~~~~~~~~~
>>>   >       |                                            link_mask
>>>   > sound/soc/intel/boards/sof_es8336.c:498:45: error: ‘struct
>>> snd_soc_acpi_mach_params’ has no member named ‘i2s_link_mask’; did you
>>> mean ‘link_mask’?
>>>   >   498 |                 else  if (mach->mach_params.i2s_link_mask &
>>> BIT(0))
>>>   >       |                                             ^~~~~~~~~~~~~
>>>   >       |                                             link_mask
>>>
>>> So 679aa83a0fb70dcbf9e97c is missing -- but likely more is needed.
>>>
>>> [1] https://raw.githubusercontent.com/SUSE/kernel-source/stable/config/x86_64/default
>>>
>>>
> 
> Offending commit now dropped, thanks!

It's fine, this was part of a large series for a set of platforms based 
on the ES8336 and ES8326 codec, but it's still WIP with patches from 
Mauro for specific devices, UCM changes needed and the ES8326 still in 
review. This can hardly qualify for inclusion in a stable branch.