The linux-next integration testing tree https://git.rulkc.org/pub/scm/linux/kernel/git/next/linux-next.git/atom?h=akpm 2022-06-28T01:53:08+00:00 2022-06-28T01:53:08+00:00 Stephen Rothwell sfr@canb.auug.org.au 2022-06-28T01:53:08+00:00 urn:sha1:c2e609e496f8195af2b789511f24a00546bb976e 2022-06-28T01:53:08+00:00 Stephen Rothwell sfr@canb.auug.org.au 2022-06-28T01:53:08+00:00 urn:sha1:cd6533b02a357e6acdaf3a64e804f7f909d52aad 2022-06-24T15:21:41+00:00 Amadeusz Sławiński amadeuszx.slawinski@linux.intel.com 2022-06-10T12:44:20+00:00 urn:sha1:12abc4d10d5502e4f3d8f1c6f9e8245747a44708 Commit 81da8a0b7975 ("ASoC: remove codec hw_write/control_data") removed use of hw_write_t in struct snd_soc_codec, but it left type definition. Fully clean it up. Fixes: 81da8a0b7975 ("ASoC: remove codec hw_write/control_data") Signed-off-by: Amadeusz Sławiński <amadeuszx.slawinski@linux.intel.com> Reviewed-by: Cezary Rojewski <cezary.rojewski@intel.com> Link: https://lore.kernel.org/r/20220610124420.4160986-1-amadeuszx.slawinski@linux.intel.com Signed-off-by: Mark Brown <broonie@kernel.org> 2022-06-22T12:39:34+00:00 Uwe Kleine-König u.kleine-koenig@pengutronix.de 2022-06-21T14:58:34+00:00 urn:sha1:1892a991886ace2c3450bec801df2cf4028a803a The function snd_soc_unregister_card() returned 0 unconditionally and most callers don't care to check the return value. Make it return void and adapt the callers that didn't ignore the return value before. This is a preparation for making platform remove callbacks return void. Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> Link: https://lore.kernel.org/r/20220621145834.198519-1-u.kleine-koenig@pengutronix.de Signed-off-by: Mark Brown <broonie@kernel.org> 2022-06-21T12:15:35+00:00 Charles Keepax ckeepax@opensource.cirrus.com 2022-06-21T10:20:41+00:00 urn:sha1:6b183919f7051294dc5fc331bb608d5d7f29f5da Currently macros only exist for SX style (implicit sign bit 2's compliment) volume controls where the volumes for left and right are in separate registers. Some future Cirrus devices will have both volumes in the same register, as such add a new macro to support this. Signed-off-by: Charles Keepax <ckeepax@opensource.cirrus.com> Link: https://lore.kernel.org/r/20220621102041.1713504-4-ckeepax@opensource.cirrus.com Signed-off-by: Mark Brown <broonie@kernel.org> 2022-06-20T07:36:02+00:00 Takashi Iwai tiwai@suse.de 2022-06-17T14:40:48+00:00 urn:sha1:f1d40433352e5d4babd59c0dd50b5f9414073ddb Having a lock in snd_rawmidi_runtime can be a problem especially when a substream is accessed from the outside, as the runtime creation might be racy with the external calls. As a first step for hardening, move the spinlock from snd_rawmidi_runtime to snd_rawmidi_substream. This patch just replaces the lock calls, no real functional change is put yet. Link: https://lore.kernel.org/r/20220617144051.18985-3-tiwai@suse.de Signed-off-by: Takashi Iwai <tiwai@suse.de> 2022-06-20T07:36:01+00:00 Takashi Iwai tiwai@suse.de 2022-06-17T14:40:47+00:00 urn:sha1:cd76175a2b204911a3cddef36b99e56945b6938c __snd_rawmidi_transmit_peek() and __snd_rawmidi_transmit_ack() are never called from the outside. Let's make them local static and unexport them. Link: https://lore.kernel.org/r/20220617144051.18985-2-tiwai@suse.de Signed-off-by: Takashi Iwai <tiwai@suse.de> 2022-06-15T05:45:27+00:00 Takashi Iwai tiwai@suse.de 2022-06-09T12:02:17+00:00 urn:sha1:1b7ec5143c34f167266fa21245d99bacb4db4aa6 The purpose of CONFIG_SND_CTL_VALIDATION is rather to enable the debugging feature for the control API. The validation is only a part of it. Let's rename it to be more explicit and intuitive. While we're at it, let's advertise, give more comment to recommend this feature for development in the kconfig help text. Link: https://lore.kernel.org/r/20220609120219.3937-3-tiwai@suse.de Signed-off-by: Takashi Iwai <tiwai@suse.de> 2022-06-15T05:44:55+00:00 Takashi Iwai tiwai@suse.de 2022-06-10T06:45:37+00:00 urn:sha1:c27e1efb61c545f36c450ef60862df9251d239a4 The control elements are managed in a single linked list and we traverse the whole list for matching each numid or ctl id per every inquiry of a control element. This is OK-ish for a small number of elements but obviously it doesn't scale. Especially the matching with the ctl id takes time because it checks each field of the snd_ctl_id element, e.g. the name string is matched with strcmp(). This patch adds the hash tables with Xarray for improving the lookup speed of a control element. There are two xarray tables added to the card; one for numid and another for ctl id. For the numid, we use the numid as the index, while for the ctl id, we calculate a hash key. The lookup is done via a single xa_load() execution. As long as the given control element is found on the Xarray table, that's fine, we can give back a quick lookup result. The problem is when no entry hits on the table, and for this case, we have a slight optimization. Namely, the driver checks whether we had a collision on Xarray table, and do a fallback search (linear lookup of the full entries) only if a hash key collision happened beforehand. So, in theory, the inquiry for a non-existing element might take still time even with this patch in a worst case, but this must be pretty rare. The feature is enabled via CONFIG_SND_CTL_FAST_LOOKUP, which is turned on as default. For simplicity, the option can be turned off only when CONFIG_EXPERT is set ("You are expert? Then you manage 1000 knobs"). Link: https://lore.kernel.org/r/20211028130027.18764-1-tiwai@suse.de Link: https://lore.kernel.org/r/20220609180504.775-1-tiwai@suse.de Link: https://lore.kernel.org/all/cover.1653813866.git.quic_rbankapu@quicinc.com/ Link: https://lore.kernel.org/r/20220610064537.18660-1-tiwai@suse.de Signed-off-by: Takashi Iwai <tiwai@suse.de> 2022-06-14T10:25:45+00:00 Mark Brown broonie@kernel.org 2022-06-14T10:25:45+00:00 urn:sha1:14cc5849120051caf537f663ec620910503bb719 Required for more changes for the ops.