linux.git/sound/core/init.c, branch v6.8

ALSA: core: Use dev_name of card_dev as debugfs directory name

2023-09-12T13:30:37+00:00

There is no need to use temporary string for the debugfs directory name as
we can use the device name of the card.

This change will also fixes the following compiler warning/error (W=1):

sound/core/init.c: In function ‘snd_card_init’:
sound/core/init.c:367:28: error: ‘%d’ directive writing between 1 and 10 bytes into a region of size 4 [-Werror=format-overflow=]
  367 |         sprintf(name, "card%d", idx);
      |                            ^~
sound/core/init.c:367:23: note: directive argument in the range [0, 2147483646]
  367 |         sprintf(name, "card%d", idx);
      |                       ^~~~~~~~
sound/core/init.c:367:9: note: ‘sprintf’ output between 6 and 15 bytes into a destination of size 8
  367 |         sprintf(name, "card%d", idx);
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors

The idx is guarantied to be less than SNDRV_CARDS (max 256 or 8) by the
code in snd_card_init(), however the compiler does not see that.

The warnings got brought to light by a recent patch upstream:
commit 6d4ab2e97dcf ("extrawarn: enable format and stringop overflow warnings in W=1")

Suggested-by: Arnd Bergmann 
Suggested-by: Takashi Iwai 
Signed-off-by: Peter Ujfalusi 
Link: https://lore.kernel.org/r/20230912110113.3166-1-peter.ujfalusi@linux.intel.com
Signed-off-by: Takashi Iwai

ALSA: core: Drop snd_device_initialize()

2023-08-17T07:24:33+00:00

Now all users of snd_device_intialize() are gone, let's drop it.

Reviewed-by: Jaroslav Kysela 
Signed-off-by: Curtis Malainey 
Tested-by: Curtis Malainey 
Link: https://lore.kernel.org/r/20230816160252.23396-10-tiwai@suse.de
Signed-off-by: Takashi Iwai

ALSA: core: Introduce snd_device_alloc()

2023-08-17T07:21:40+00:00

Introduce a new helper, snd_device_alloc(), for allocating a struct
device that is bound with the sound class.  It's a replacement of
snd_device_initialize().

Reviewed-by: Jaroslav Kysela 
Signed-off-by: Curtis Malainey 
Tested-by: Curtis Malainey 
Link: https://lore.kernel.org/r/20230816160252.23396-2-tiwai@suse.de
Signed-off-by: Takashi Iwai

sound: make all 'class' structures const

2023-06-21T05:29:10+00:00

Now that the driver core allows for struct class to be in read-only
memory, making all 'class' structures to be declared at build time
placing them into read-only memory, instead of having to be dynamically
allocated at load time.

Cc: Jaroslav Kysela 
Cc: Takashi Iwai 
Cc: Ivan Orlov 
Cc: Greg Kroah-Hartman 
Cc: Geoff Levand 
Cc: Thierry Reding 
Cc: "Uwe Kleine-König" 
Cc: alsa-devel@alsa-project.org
Suggested-by: Greg Kroah-Hartman 
Signed-off-by: Ivan Orlov 
Signed-off-by: Greg Kroah-Hartman 
Link: https://lore.kernel.org/r/20230620175633.641141-2-gregkh@linuxfoundation.org
Signed-off-by: Takashi Iwai

ALSA: core: Make snd_card_free() return void

2023-02-08T12:39:27+00:00

The function returns 0 unconditionally. Make it return void instead and
simplify all callers accordingly.

Signed-off-by: Uwe Kleine-König 
Reviewed-by: Jaroslav Kysela 
Reviewed-by: Takashi Sakamoto 
Acked-by: Geoff Levand 
Acked-by: Thierry Reding 
Link: https://lore.kernel.org/r/20230207191907.467756-4-u.kleine-koenig@pengutronix.de
Signed-off-by: Takashi Iwai

ALSA: core: Make snd_card_free_when_closed() return void

2023-02-08T12:39:12+00:00

All callers from other files ignore the return value of this function.
And it can only ever return a non-zero value if the parameter card is NULL.

This cannot happen in snd_card_free() as card was dereferenced just before
snd_card_free_when_closed() is called. So the error handling can be dropped
there.

Signed-off-by: Uwe Kleine-König 
Reviewed-by: Jaroslav Kysela 
Reviewed-by: Takashi Sakamoto 
Acked-by: Geoff Levand 
Acked-by: Thierry Reding 
Link: https://lore.kernel.org/r/20230207191907.467756-3-u.kleine-koenig@pengutronix.de
Signed-off-by: Takashi Iwai

ALSA: core: Make snd_card_disconnect() return void

2023-02-08T12:38:28+00:00

All callers from other files ignore the return value of this function.
And it can only ever return a non-zero value if the parameter card is NULL.

Move the check for card being NULL into snd_card_free_when_closed() to keep
the previous behaviour. Note this isn't necessary for
snd_card_disconnect_sync() because if card was NULL in there the dereference
of card for dev_err() would oops the kernel. Replace this by an oops
triggered by the dereference of card for spin_lock_irq().

Signed-off-by: Uwe Kleine-König 
Reviewed-by: Jaroslav Kysela 
Reviewed-by: Takashi Sakamoto 
Acked-by: Geoff Levand 
Acked-by: Thierry Reding 
Link: https://lore.kernel.org/r/20230207191907.467756-2-u.kleine-koenig@pengutronix.de
Signed-off-by: Takashi Iwai

ALSA: core: Fix double-free at snd_card_new()

2022-09-19T12:36:06+00:00

During the code change to add the support for devres-managed card
instance, we put an explicit kfree(card) call at the error path in
snd_card_new().  This is needed for the early error path before the
card is initialized with the device, but is rather superfluous and
causes a double-free at the error path after the card instance is
initialized, as the destructor of the card object already contains a
kfree() call.

This patch fixes the double-free situation by removing the superfluous
kfree().  Meanwhile we need to call kfree() explicitly for the early
error path, so it's added there instead.

Fixes: e8ad415b7a55 ("ALSA: core: Add managed card creation")
Reported-by: Rondreis 
Cc: 
Link: https://lore.kernel.org/r/CAB7eexL1zBnB636hwS27d-LdPYZ_R1-5fJS_h=ZbCWYU=UPWJg@mail.gmail.com
Link: https://lore.kernel.org/r/20220919123516.28222-1-tiwai@suse.de
Signed-off-by: Takashi Iwai

ALSA: core: Replace scnprintf() with sysfs_emit()

2022-08-02T14:03:45+00:00

sysfs_emit() is a new helper to simplify the sysfs string output.
Replace the open-code with this new helper.

Link: https://lore.kernel.org/r/20220801165639.26030-5-tiwai@suse.de
Signed-off-by: Takashi Iwai

ALSA: core: Fix missing return value comments for kernel docs

2022-07-13T11:42:38+00:00

Each kernel doc comment expects the definition of the return value in
a proper format.  This patch adds or fixes the missing entries for the
remaining ALSA core API functions.

Link: https://lore.kernel.org/r/20220713104759.4365-8-tiwai@suse.de
Signed-off-by: Takashi Iwai