[alsa-devel] [PATCH] ASoC: si476x: Remove custom register I/O implementation
broonie at kernel.org
Fri Sep 27 12:07:57 CEST 2013
On Thu, Sep 26, 2013 at 03:48:09PM -0700, Andrey Smirnov wrote:
> On Thu, Sep 26, 2013 at 1:58 PM, Mark Brown <broonie at kernel.org> wrote:
> > In what way might these things conflict? What are the clashes you are
> > concerned about?
> That lock serves the purpose of making sure that
> si476x_core_send_command, calls to which all the rest of the functions
> boil down to, is not executed in parallel to itself. So I am concerned
> that with the lock acquisition removal it would be possible to make
> that condition happen.
OK, so if it's purely about send_command() then why is the locking not
being done in that function? Surely the most obvious and robust place
to protect the function is within the function itself?
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 836 bytes
Desc: Digital signature
More information about the Alsa-devel