Re: [PATCH 2/2] ASoC: wsa881x: mark read_only_wordlength flag

[Srinivas Kandagatla]

On 11/03/2020 18:58, Pierre-Louis Bossart wrote:
> On 3/11/20 6:35 AM, Srinivas Kandagatla wrote:
> > WSA881x works in PDM mode so the wordlength is fixed, which also makes
> > the only field "WordLength" in DPN_BlockCtrl1 register a read-only.
> > Writing to this register will throw up errors with Qualcomm Controller.
>
> it'd be good to clarify the nature of these errors, i.e.
>
> a) is this the Slave device responding with a NAK?
> b) is this the Slave device responding with COMMAND_IGNORED, and those 
> responses not handled by the controller?
> c) something else?


With the current version of Qcom SoundWire controller driver, it can 
only know if the write succeeded or not based on completion interrupt. 
There are no details of the actual error code visible to the driver. But 
the new version of the same controller is expected to return better 
error codes.

thanks,
srini

> Thanks!