On Tue, Jul 18, 2023 at 04:13:00PM +0200, Takashi Iwai wrote: > The controls_rwsem of snd_card object is rather an internal lock, and > not really meant to be used by others for its data protection. > > This patch addresses it by replacing the controls_rwsem usages with > the own (new) mutex. > > Note that the up_write() and down_write() calls around > gbaudio_remove_component_controls() are simply dropped without > replacement. These temporary up/down were a workaround since > gbaudio_remove_component_controls() itself took the rwsem. Now it was > also gone, we can clean up the workaround, too. > > Cc: Vaibhav Agarwal <vaibhav.sr@xxxxxxxxx> > Cc: Mark Greer <mgreer@xxxxxxxxxxxxxxx> > Cc: Johan Hovold <johan@xxxxxxxxxx> > Cc: Alex Elder <elder@xxxxxxxxxx> > Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> > Cc: greybus-dev@xxxxxxxxxxxxxxxx > Signed-off-by: Takashi Iwai <tiwai@xxxxxxx> > --- > drivers/staging/greybus/audio_codec.c | 18 +++++++----------- > drivers/staging/greybus/audio_codec.h | 1 + > 2 files changed, 8 insertions(+), 11 deletions(-) Acked-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> _______________________________________________ greybus-dev mailing list -- greybus-dev@xxxxxxxxxxxxxxxx To unsubscribe send an email to greybus-dev-leave@xxxxxxxxxxxxxxxx
|