Re: [REGRESSION, bisected] 4.5rc4 sound fsl-soc

["Mika Penttilä" <mika.penttila@nextfour.com>]

Mark,

I can confirm that the patch:

https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/commit/drivers/base/regmap/regcache.c?id=3245d460a1eb55b5c3ca31dde7b5c5ac71546edf


solved the audio codec probing issue for me. Without it there's no sound on imx6 with 4.5-rc4.

Please apply.

Thanks,
Mika

On 20.02.2016 22:45, Fabio Estevam wrote:
> Hi Mika,
>
> Did it work for you?
>
> If so, please ask in the mailing list for Mark Brown to apply that patch.
>
> Thanks
>
> ________________________________________
> From: Mika Penttilä <mika.penttila@nextfour.com>
> Sent: Monday, February 15, 2016 11:00 AM
> To: Fabio Estevam
> Subject: Re: [REGRESSION, bisected] 4.5rc4 sound fsl-soc
>
> On 02/15/2016 01:30 PM, Fabio Estevam wrote:
>> [Sorry for the top post, can't reply properly from this Inbox]
> Hi,
>
> will test that tomorrow morning EET.
>
>
> thanks,
> Mika
>
>
>> Could you please try applying this commit from linux-next?
>> https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/commit/drivers/base/regmap/regcache.c?id=3245d460a1eb55b5c3ca31dde7b5c5ac71546edf
>>
>> Thanks,
>>
>> Fabio Estevam
>>
>> ________________________________________
>> From: Mika Penttilä <mika.penttila@nextfour.com>
>> Sent: Monday, February 15, 2016 9:25 AM
>> To: LKML; mail@maciej.szmigiero.name; Fabio Estevam
>> Subject: [REGRESSION, bisected] 4.5rc4 sound fsl-soc
>>
>> Hi,
>>
>> The following commit :
>>
>> 5c408fee254633a5be69505bc86c6b034f871ab4 is the first bad commit
>> commit 5c408fee254633a5be69505bc86c6b034f871ab4
>> Author: Maciej S. Szmigiero <mail@maciej.szmigiero.name>
>> Date:   Mon Jan 18 20:07:44 2016 +0100
>>
>>     ASoC: fsl_ssi: remove explicit register defaults
>>
>>     There is no guarantee that on fsl_ssi module load
>>     SSI registers will have their power-on-reset values.
>>
>>     In fact, if the driver is reloaded the values in
>>     registers will be whatever they were set to previously.
>>
>>     However, the cache needs to be fully populated at probe
>>     time to avoid non-atomic allocations during register
>>     access.
>>
>>     Special case here is imx21-class SSI, since
>>     according to datasheet it don't have SACC{ST,EN,DIS}
>>     regs.
>>
>>     This fixes hard lockup on fsl_ssi module reload,
>>     at least in AC'97 mode.
>>
>>     Fixes: 05cf237972fe ("ASoC: fsl_ssi: Add driver suspend and resume to support MEGA Fast")
>>     Signed-off-by: Maciej S. Szmigiero <mail@maciej.szmigiero.name>
>>     Tested-by: Fabio Estevam <fabio.estevam@nxp.com>
>>     Signed-off-by: Mark Brown <broonie@kernel.org>
>>
>>
>> causes regmap init failure when loading the sgtl5000 codec on imx6q, and
>> leads to no audio.
>>
>> With the mentioned patch reverted sound works ok.
>>
>> --Mika
>>