Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Mon, 30 May 2022 10:34:15 +0000
From:   Charles Keepax <ckeepax@opensource.cirrus.com>
To:     Takashi Iwai <tiwai@suse.de>
CC:     Vitaly Rodionov <vitalyr@opensource.cirrus.com>,
        Jaroslav Kysela <perex@perex.cz>,
        Takashi Iwai <tiwai@suse.com>, Mark Brown <broonie@kernel.org>,
        <alsa-devel@alsa-project.org>, <patches@opensource.cirrus.com>,
        <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v4 00/17] ALSA: hda: cirrus: Add initial DSP support and
 firmware loading
Message-ID: <20220530103415.GU38351@ediswmail.ad.cirrus.com>
References: <20220525131638.5512-1-vitalyr@opensource.cirrus.com>
 <871qwf0x8t.wl-tiwai@suse.de>
 <20220530090846.GS38351@ediswmail.ad.cirrus.com>
 <87czfvxtsc.wl-tiwai@suse.de>
 <20220530093639.GT38351@ediswmail.ad.cirrus.com>
 <87a6azxr7h.wl-tiwai@suse.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <87a6azxr7h.wl-tiwai@suse.de>
User-Agent: Mutt/1.5.21 (2010-09-15)
Precedence: bulk

On Mon, May 30, 2022 at 12:14:26PM +0200, Takashi Iwai wrote:
> On Mon, 30 May 2022 11:36:39 +0200,
> Charles Keepax wrote:
> > On Mon, May 30, 2022 at 11:18:43AM +0200, Takashi Iwai wrote:
> > > On Mon, 30 May 2022 11:08:46 +0200,
> > > Charles Keepax wrote:
> > > > On Fri, May 27, 2022 at 06:13:38PM +0200, Takashi Iwai wrote:
> > > > > On Wed, 25 May 2022 15:16:21 +0200,
> > > > > Vitaly Rodionov wrote:
> > Yeah that should be what is happening here. Although it looks
> > like this code might be removing all the controls if the firmware
> > is unloaded. I will discuss that with the guys, we normal just
> > disable the controls on the wm_adsp stuff.
> 
> OK, that sounds good.  Basically my concern came up from the code
> snippet doing asynchronous addition/removal via work.  This showed
> some yellow signal, as such a pattern doesn't appear in the normal
> implementation.  If this is (still) really necessary, it has to be
> clarified as an exception.
> 

Hm... ok we will think about that. I think that part will
probably still be necessary. Because there is an ALSA control
that selects the firmware, then it is necesarry to defer creating
the controls to some work, since you are already holding the
lock.

I guess we could look at adding locked versions of the add
control functions as well if that might be preferred?

Thanks,
Charles