greybus-dev

greybus-dev@lists.linaro.org

2 participants
549 discussions

Re: [PATCH v3] staging: greybus: audio: Check null pointer

by Alex Elder

On 1/4/22 9:06 AM, Jiasheng Jiang wrote: > As the possible alloc failure of devm_kcalloc(), it could return null > pointer. > Therefore, 'strings' should be checked and return NULL if alloc fails to > prevent the dereference of the NULL pointer. > Also, the caller should also deal with the return value of the > gb_generate_enum_strings() and return -ENOMEM if returns NULL. > Moreover, because the memory allocated with devm_kzalloc() will be > freed automatically when the last reference to the device is dropped, > the 'gbe' in gbaudio_tplg_create_enum_kctl() and > gbaudio_tplg_create_enum_ctl() do not need to free manually. > But the 'control' in gbaudio_tplg_create_widget() and > gbaudio_tplg_process_kcontrols() has a specially error handle to > cleanup. > So it should be better to cleanup 'control' when fails. > > Fixes: e65579e335da ("greybus: audio: topology: Enable enumerated control support") > Signed-off-by: Jiasheng Jiang <jiasheng(a)iscas.ac.cn> This looks good to me. Thanks for fixing things. Reviewed-by: Alex Elder <elder(a)linaro.org> > --- > v3: Same code as v2, but remove the redundant message as requested. > v2: Updated to use common error processing at the end of both > gbaudio_tplg_create_widget() and gbaudio_tplg_process_kcontrols(). > --- > drivers/staging/greybus/audio_topology.c | 15 +++++++++++++++ > 1 file changed, 15 insertions(+) > > diff --git a/drivers/staging/greybus/audio_topology.c b/drivers/staging/greybus/audio_topology.c > index 1fc7727ab7be..6bba735d2e5c 100644 > --- a/drivers/staging/greybus/audio_topology.c > +++ b/drivers/staging/greybus/audio_topology.c > @@ -147,6 +147,9 @@ static const char **gb_generate_enum_strings(struct gbaudio_module_info *gb, > > items = le32_to_cpu(gbenum->items); > strings = devm_kcalloc(gb->dev, items, sizeof(char *), GFP_KERNEL); > + if (!strings) > + return NULL; > + > data = gbenum->names; > > for (i = 0; i < items; i++) { > @@ -655,6 +658,8 @@ static int gbaudio_tplg_create_enum_kctl(struct gbaudio_module_info *gb, > /* since count=1, and reg is dummy */ > gbe->items = le32_to_cpu(gb_enum->items); > gbe->texts = gb_generate_enum_strings(gb, gb_enum); > + if (!gbe->texts) > + return -ENOMEM; > > /* debug enum info */ > dev_dbg(gb->dev, "Max:%d, name_length:%d\n", gbe->items, > @@ -862,6 +867,8 @@ static int gbaudio_tplg_create_enum_ctl(struct gbaudio_module_info *gb, > /* since count=1, and reg is dummy */ > gbe->items = le32_to_cpu(gb_enum->items); > gbe->texts = gb_generate_enum_strings(gb, gb_enum); > + if (!gbe->texts) > + return -ENOMEM; > > /* debug enum info */ > dev_dbg(gb->dev, "Max:%d, name_length:%d\n", gbe->items, > @@ -1034,6 +1041,10 @@ static int gbaudio_tplg_create_widget(struct gbaudio_module_info *module, > csize += le16_to_cpu(gbenum->names_length); > control->texts = (const char * const *) > gb_generate_enum_strings(module, gbenum); > + if (!control->texts) { > + ret = -ENOMEM; > + goto error; > + } > control->items = le32_to_cpu(gbenum->items); > } else { > csize = sizeof(struct gb_audio_control); > @@ -1183,6 +1194,10 @@ static int gbaudio_tplg_process_kcontrols(struct gbaudio_module_info *module, > csize += le16_to_cpu(gbenum->names_length); > control->texts = (const char * const *) > gb_generate_enum_strings(module, gbenum); > + if (!control->texts) { > + ret = -ENOMEM; > + goto error; > + } > control->items = le32_to_cpu(gbenum->items); > } else { > csize = sizeof(struct gb_audio_control); >

3 years, 1 month

Re: [PATCH v2] staging: greybus: audio: Check null pointer

by Alex Elder

On 12/28/21 8:28 PM, Jiasheng Jiang wrote: > On Tue, Dec 28, 2021 at 10:50:22PM +0800, Alex Elder wrote: >> But the two places you're returning are in the middle of a loop (in >> gbaudio_tplg_create_widget() and gbaudio_tplg_process_kcontrols()). >> Each is building up a sort of hierarchical data structure, and as >> you can see, the existing code takes care to de-construct the partially >> built structure in the event an error occurs before it completes. >> >> There is a chance that your simple return would "work", but by >> reading the surrounding code you should recognize that the proper >> way to handle the error is to jump to the cleanup at the end. > > Yes, I agree with your opinion and this time I use "goto error" instead > of directly return in the two callers. > The new patch is as follow. > > As the possible alloc failure of devm_kcalloc(), it could return null > pointer. > Therefore, 'strings' should be checked and return NULL if alloc fails to > prevent the dereference of the NULL pointer. > Also, the caller should also deal with the return value of the > gb_generate_enum_strings() and return -ENOMEM if returns NULL. > Moreover, because the memory allocated with devm_kzalloc() will be > freed automatically when the last reference to the device is dropped, > the 'gbe' in gbaudio_tplg_create_enum_kctl() and > gbaudio_tplg_create_enum_ctl() do not need to free manually. > But the 'control' in gbaudio_tplg_create_widget() and > gbaudio_tplg_process_kcontrols() has a specially error handle to > cleanup. > So it should be better to cleanup 'control' when fails. I haven't really looked at this yet, but in any case I would like you to send version 3 of this patch. The reason is that you should not include the message above in the patch itself. To be clear, I really appreciate your response; please *do* respond to review comments. But do so in e-mail only, and then follow up with a new patch, using the same basic subject (with a new version) and (roughly) the same description. So your patch should have a subject line with v3. It should then contain the original description (not indented with ">>" or anything), updated or improved if appropriate to reflect the current state of the patch. Then, below the "---" line you should include a patch history, with a line or two describing each version. For example, it might look something like: --- v3: Same code as v2, but fixed description as requested. v2: Updated to use common error processing at the end of both gbaudio_tplg_create_widget() and gbaudio_tplg_process_kcontrols(). I will review version 3, and if I don't see anything else wrong with it I'll offer a "Signed-off-by" tag. -Alex > Fixes: e65579e335da ("greybus: audio: topology: Enable enumerated control support") > Signed-off-by: Jiasheng Jiang <jiasheng(a)iscas.ac.cn> > --- > Changelog: > > v1 -> v2 > > *Change 1. Add the cleanup process when alloc fails in two callers and > refine the commit message. > --- > drivers/staging/greybus/audio_topology.c | 15 +++++++++++++++ > 1 file changed, 15 insertions(+) > > diff --git a/drivers/staging/greybus/audio_topology.c b/drivers/staging/greybus/audio_topology.c > index 1fc7727ab7be..6bba735d2e5c 100644 > --- a/drivers/staging/greybus/audio_topology.c > +++ b/drivers/staging/greybus/audio_topology.c > @@ -147,6 +147,9 @@ static const char **gb_generate_enum_strings(struct gbaudio_module_info *gb, > > items = le32_to_cpu(gbenum->items); > strings = devm_kcalloc(gb->dev, items, sizeof(char *), GFP_KERNEL); > + if (!strings) > + return NULL; > + > data = gbenum->names; > > for (i = 0; i < items; i++) { > @@ -655,6 +658,8 @@ static int gbaudio_tplg_create_enum_kctl(struct gbaudio_module_info *gb, > /* since count=1, and reg is dummy */ > gbe->items = le32_to_cpu(gb_enum->items); > gbe->texts = gb_generate_enum_strings(gb, gb_enum); > + if (!gbe->texts) > + return -ENOMEM; > > /* debug enum info */ > dev_dbg(gb->dev, "Max:%d, name_length:%d\n", gbe->items, > @@ -862,6 +867,8 @@ static int gbaudio_tplg_create_enum_ctl(struct gbaudio_module_info *gb, > /* since count=1, and reg is dummy */ > gbe->items = le32_to_cpu(gb_enum->items); > gbe->texts = gb_generate_enum_strings(gb, gb_enum); > + if (!gbe->texts) > + return -ENOMEM; > > /* debug enum info */ > dev_dbg(gb->dev, "Max:%d, name_length:%d\n", gbe->items, > @@ -1034,6 +1041,10 @@ static int gbaudio_tplg_create_widget(struct gbaudio_module_info *module, > csize += le16_to_cpu(gbenum->names_length); > control->texts = (const char * const *) > gb_generate_enum_strings(module, gbenum); > + if (!control->texts) { > + ret = -ENOMEM; > + goto error; > + } > control->items = le32_to_cpu(gbenum->items); > } else { > csize = sizeof(struct gb_audio_control); > @@ -1183,6 +1194,10 @@ static int gbaudio_tplg_process_kcontrols(struct gbaudio_module_info *module, > csize += le16_to_cpu(gbenum->names_length); > control->texts = (const char * const *) > gb_generate_enum_strings(module, gbenum); > + if (!control->texts) { > + ret = -ENOMEM; > + goto error; > + } > control->items = le32_to_cpu(gbenum->items); > } else { > csize = sizeof(struct gb_audio_control); >

3 years, 1 month

[PATCH] staging: greybus: auto_manager: use default_groups in kobj_type

by Greg Kroah-Hartman

There are currently 2 ways to create a set of sysfs files for a kobj_type, through the default_attrs field, and the default_groups field. Move the greybus audio code to use default_groups field which has been the preferred way since aa30f47cf666 ("kobject: Add support for default attribute groups to kobj_type") so that we can soon get rid of the obsolete default_attrs field. Cc: Vaibhav Agarwal <vaibhav.sr(a)gmail.com> Cc: Mark Greer <mgreer(a)animalcreek.com> Cc: Johan Hovold <johan(a)kernel.org> Cc: Alex Elder <elder(a)kernel.org> Cc: greybus-dev(a)lists.linaro.org Cc: linux-staging(a)lists.linux.dev Signed-off-by: Greg Kroah-Hartman <gregkh(a)linuxfoundation.org> --- drivers/staging/greybus/audio_manager_module.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/greybus/audio_manager_module.c b/drivers/staging/greybus/audio_manager_module.c index 525cf8f8394f..0a0f0a394c84 100644 --- a/drivers/staging/greybus/audio_manager_module.c +++ b/drivers/staging/greybus/audio_manager_module.c @@ -142,11 +142,12 @@ static struct attribute *gb_audio_module_default_attrs[] = { &gb_audio_module_op_devices_attribute.attr, NULL, /* need to NULL terminate the list of attributes */ }; +ATTRIBUTE_GROUPS(gb_audio_module_default); static struct kobj_type gb_audio_module_type = { .sysfs_ops = &gb_audio_module_sysfs_ops, .release = gb_audio_module_release, - .default_attrs = gb_audio_module_default_attrs, + .default_groups = gb_audio_module_default_groups, }; static void send_add_uevent(struct gb_audio_manager_module *module) -- 2.34.1

3 years, 1 month

Re: [greybus-dev] [PATCH] staging: greybus: audio: Check null pointer

by Alex Elder

On 12/24/21 3:03 AM, Jiasheng Jiang wrote: > As the possible alloc failure of devm_kcalloc, it could return null > pointer. > To prevent the dereference of the null pointer, it should be checked. I think this is a good change, but I would like you to improve the description, and fix some different bugs introduced by your change. What you are specifically doing is checking for a null return from devm_kcalloc() in gb_generate_enum_strings(), and are returning the NULL pointer if that occurs. That means you need to update all the callers of gb_generate_enum_strings() to also handle a possible null return value. The fix does a good thing, and your description is correct about what you are fixing. But it should supply more complete context for the change. More below. > Fixes: e65579e335da ("greybus: audio: topology: Enable enumerated control support") > Signed-off-by: Jiasheng Jiang <jiasheng(a)iscas.ac.cn> > --- > drivers/staging/greybus/audio_topology.c | 18 ++++++++++++++++++ > 1 file changed, 18 insertions(+) > > diff --git a/drivers/staging/greybus/audio_topology.c b/drivers/staging/greybus/audio_topology.c > index 1fc7727ab7be..e9f47a1f0d28 100644 > --- a/drivers/staging/greybus/audio_topology.c > +++ b/drivers/staging/greybus/audio_topology.c > @@ -146,7 +146,11 @@ static const char **gb_generate_enum_strings(struct gbaudio_module_info *gb, > __u8 *data; > > items = le32_to_cpu(gbenum->items); > + > strings = devm_kcalloc(gb->dev, items, sizeof(char *), GFP_KERNEL); > + if (!strings) > + return NULL; > + > data = gbenum->names; > > for (i = 0; i < items; i++) { > @@ -654,7 +658,10 @@ static int gbaudio_tplg_create_enum_kctl(struct gbaudio_module_info *gb, > > /* since count=1, and reg is dummy */ > gbe->items = le32_to_cpu(gb_enum->items); > + > gbe->texts = gb_generate_enum_strings(gb, gb_enum); > + if (!gbe->texts) > + return -ENOMEM; > > /* debug enum info */ > dev_dbg(gb->dev, "Max:%d, name_length:%d\n", gbe->items, > @@ -861,7 +868,10 @@ static int gbaudio_tplg_create_enum_ctl(struct gbaudio_module_info *gb, > > /* since count=1, and reg is dummy */ > gbe->items = le32_to_cpu(gb_enum->items); > + > gbe->texts = gb_generate_enum_strings(gb, gb_enum); > + if (!gbe->texts) > + return -ENOMEM; > > /* debug enum info */ > dev_dbg(gb->dev, "Max:%d, name_length:%d\n", gbe->items, > @@ -1032,8 +1042,12 @@ static int gbaudio_tplg_create_widget(struct gbaudio_module_info *module, > csize += offsetof(struct gb_audio_ctl_elem_info, value); > csize += offsetof(struct gb_audio_enumerated, names); > csize += le16_to_cpu(gbenum->names_length); > + > control->texts = (const char * const *) > gb_generate_enum_strings(module, gbenum); > + if (!control->texts) > + return -ENOMEM; > + You can't simply return here. If you look a bit above this, where the call to allocate a control structure is done, you see that a NULL return there jumps to the "error" label, so any already allocated and initialized control widgets get cleaned up before returning. > control->items = le32_to_cpu(gbenum->items); > } else { > csize = sizeof(struct gb_audio_control); > @@ -1181,8 +1195,12 @@ static int gbaudio_tplg_process_kcontrols(struct gbaudio_module_info *module, > csize += offsetof(struct gb_audio_ctl_elem_info, value); > csize += offsetof(struct gb_audio_enumerated, names); > csize += le16_to_cpu(gbenum->names_length); > + > control->texts = (const char * const *) > gb_generate_enum_strings(module, gbenum); > + if (!control->texts) > + return -ENOMEM; > + You have basically the same issue here. You can't just return, you must do some cleanup too. -Alex > control->items = le32_to_cpu(gbenum->items); > } else { > csize = sizeof(struct gb_audio_control); >

3 years, 1 month

Re: [greybus-dev] Addition of config USB_HSIC_USB3613 soon?

by Greg Kroah-Hartman

On Wed, Dec 22, 2021 at 10:54:41AM +0100, Lukas Bulwahn wrote: > Dear Vaibhav, dear Johan, dear Alex, dear Greg, > > I have seen that the greybus arche driver has been under heavy > development in 2016 and 2017 with some further clean-up in 2019. > > However, so far, the config GREYBUS_ARCHE for this driver still > depends on the out-of-tree config USB_HSIC_USB3613, with a proper > exception made for compile testing (with COMPILE_TEST). > > Will this USB_HSIC_USB3613 config and driver still be added in the > mainline kernel in the near future, so that the config dependencies > are consistent in mainline? Do you have this hardware? If so, we can add the driver, but given that I did not think the chip ever actually shipped, it didn't make much sense. > Or, are the further out-of-tree additions still maintained for the > current kernel and will stay out of tree? Is this arche driver not > needed anymore and can be dropped? Do you want to drop it as it is causing problems for you? It's a good example driver for those wanting to create a greybus host controller driver so it's nice to have in the tree, unless you have a different one that should be merged instead? thanks, greg k-h

3 years, 2 months

Re: [greybus-dev] [PATCH] Header line: Coding style fix

by Mark Greer

On Fri, Dec 17, 2021 at 11:34:08AM -0300, Jorge Eduardo Fermino Oliveia Silva wrote: [Note: I am traveling for the next week so I won't be very responsive.] Hi Jorge. Before we get to the platch please remember that you should send all Greybus patches to greybus-dev(a)lists.linaro.org and linux-kernel(a)vger.kernel.org. I will add them in now and leave all of the context so other can see what you sent. > Solve CHECK: Lines should not end with a '(' > > Reported-by: Jorge Eduardo Fermino Oliveia Silva <jorgeubermensch(a)gmail.com> > Signed-off-by: Jorge Eduardo Fermino Oliveia Silva <jorgeubermensch(a)gmail.com> > --- > drivers/staging/greybus/audio_manager_private.h | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/staging/greybus/audio_manager_private.h b/drivers/staging/greybus/audio_manager_private.h > index 2b3a766c7de7..a17f09a19014 100644 > --- a/drivers/staging/greybus/audio_manager_private.h > +++ b/drivers/staging/greybus/audio_manager_private.h > @@ -12,10 +12,10 @@ > > #include "audio_manager.h" > > -int gb_audio_manager_module_create( > - struct gb_audio_manager_module **module, > - struct kset *manager_kset, > - int id, struct gb_audio_manager_module_descriptor *desc); > +int gb_audio_manager_module_create(struct gb_audio_manager_module **module, > + struct kset *manager_kset, > + int id, > + struct gb_audio_manager_module_descriptor *desc); > > /* module destroyed via kobject_put */ The part you're removing has all of the parameters at the same indentation level and what you adding has them at two different indentation levels so I'm not sure this is a step forward. Since the kernel coding style doesn't address this specific case, AFAICS, I would leave it as is despite the complaint. If others disagree then go ahead as I really don't care much either way. Mark --

3 years, 2 months

Re: [greybus-dev] [PATCH] greybus: es2: fix typo in a comment

by Alex Elder

On 12/11/21 9:16 PM, Jason Wang wrote: > The double `for' in the comment in line 81 is repeated. Remove one > of them from the comment. This looks fine. But it's so trivial... Are you aware of *any* other similar trivial problems in comments that could be fixed together with this? If so, I would prefer that. If you've looked, and there are no others: Reviewed-by: Alex Elder <elder(a)linaro.org> > Signed-off-by: Jason Wang <wangborong(a)cdjrlc.com> > --- > drivers/greybus/es2.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/greybus/es2.c b/drivers/greybus/es2.c > index 15661c7f3633..e89cca015095 100644 > --- a/drivers/greybus/es2.c > +++ b/drivers/greybus/es2.c > @@ -78,7 +78,7 @@ struct es2_cport_in { > * @hd: pointer to our gb_host_device structure > * > * @cport_in: endpoint, urbs and buffer for cport in messages > - * @cport_out_endpoint: endpoint for for cport out messages > + * @cport_out_endpoint: endpoint for cport out messages > * @cport_out_urb: array of urbs for the CPort out messages > * @cport_out_urb_busy: array of flags to see if the @cport_out_urb is busy or > * not. >

3 years, 2 months

[PATCH v2] staging: greybus: fix stack size warning with UBSAN

by Nathan Chancellor

From: Arnd Bergmann <arnd(a)arndb.de> clang warns about excessive stack usage in this driver when UBSAN is enabled: drivers/staging/greybus/audio_topology.c:977:12: error: stack frame size of 1836 bytes in function 'gbaudio_tplg_create_widget' [-Werror,-Wframe-larger-than=] Rework this code to no longer use compound literals for initializing the structure in each case, but instead keep the common bits in a preallocated constant array and copy them as needed. Link: https://github.com/ClangBuiltLinux/linux/issues/1535 Link: https://lore.kernel.org/r/20210103223541.2790855-1-arnd@kernel.org/ Signed-off-by: Arnd Bergmann <arnd(a)arndb.de> [nathan: Address review comments from v1] Signed-off-by: Nathan Chancellor <nathan(a)kernel.org> --- v1 -> v2: https://lore.kernel.org/r/20210103223541.2790855-1-arnd@kernel.org/ * Use NULL for name field in SND_DOC_DAPM_* in gbaudio_widgets (Alex). * Do not eliminate *dw assignment within the switch cases, as invalid enum values in-between valid enum values (such as snd_soc_dapm_demux) would not be handled properly by the "if value is greater than the array size" check (Alex). This addresses a few other comments by Alex and Dan because w->type is not checked against the array's size. Arnd, if you disagree with this approach, please let me know so that we can get this fixed in a way that everyone is happy with. drivers/staging/greybus/audio_topology.c | 92 ++++++++++++------------ 1 file changed, 45 insertions(+), 47 deletions(-) diff --git a/drivers/staging/greybus/audio_topology.c b/drivers/staging/greybus/audio_topology.c index 1e613d42d823..7f7d558b76d0 100644 --- a/drivers/staging/greybus/audio_topology.c +++ b/drivers/staging/greybus/audio_topology.c @@ -974,6 +974,44 @@ static int gbaudio_widget_event(struct snd_soc_dapm_widget *w, return ret; } +static const struct snd_soc_dapm_widget gbaudio_widgets[] = { + [snd_soc_dapm_spk] = SND_SOC_DAPM_SPK(NULL, gbcodec_event_spk), + [snd_soc_dapm_hp] = SND_SOC_DAPM_HP(NULL, gbcodec_event_hp), + [snd_soc_dapm_mic] = SND_SOC_DAPM_MIC(NULL, gbcodec_event_int_mic), + [snd_soc_dapm_output] = SND_SOC_DAPM_OUTPUT(NULL), + [snd_soc_dapm_input] = SND_SOC_DAPM_INPUT(NULL), + [snd_soc_dapm_switch] = SND_SOC_DAPM_SWITCH_E(NULL, SND_SOC_NOPM, + 0, 0, NULL, + gbaudio_widget_event, + SND_SOC_DAPM_PRE_PMU | + SND_SOC_DAPM_POST_PMD), + [snd_soc_dapm_pga] = SND_SOC_DAPM_PGA_E(NULL, SND_SOC_NOPM, + 0, 0, NULL, 0, + gbaudio_widget_event, + SND_SOC_DAPM_PRE_PMU | + SND_SOC_DAPM_POST_PMD), + [snd_soc_dapm_mixer] = SND_SOC_DAPM_MIXER_E(NULL, SND_SOC_NOPM, + 0, 0, NULL, 0, + gbaudio_widget_event, + SND_SOC_DAPM_PRE_PMU | + SND_SOC_DAPM_POST_PMD), + [snd_soc_dapm_mux] = SND_SOC_DAPM_MUX_E(NULL, SND_SOC_NOPM, + 0, 0, NULL, + gbaudio_widget_event, + SND_SOC_DAPM_PRE_PMU | + SND_SOC_DAPM_POST_PMD), + [snd_soc_dapm_aif_in] = SND_SOC_DAPM_AIF_IN_E(NULL, NULL, 0, + SND_SOC_NOPM, 0, 0, + gbaudio_widget_event, + SND_SOC_DAPM_PRE_PMU | + SND_SOC_DAPM_POST_PMD), + [snd_soc_dapm_aif_out] = SND_SOC_DAPM_AIF_OUT_E(NULL, NULL, 0, + SND_SOC_NOPM, 0, 0, + gbaudio_widget_event, + SND_SOC_DAPM_PRE_PMU | + SND_SOC_DAPM_POST_PMD), +}; + static int gbaudio_tplg_create_widget(struct gbaudio_module_info *module, struct snd_soc_dapm_widget *dw, struct gb_audio_widget *w, int *w_size) @@ -1052,77 +1090,37 @@ static int gbaudio_tplg_create_widget(struct gbaudio_module_info *module, switch (w->type) { case snd_soc_dapm_spk: - *dw = (struct snd_soc_dapm_widget) - SND_SOC_DAPM_SPK(w->name, gbcodec_event_spk); + *dw = gbaudio_widgets[w->type]; module->op_devices |= GBAUDIO_DEVICE_OUT_SPEAKER; break; case snd_soc_dapm_hp: - *dw = (struct snd_soc_dapm_widget) - SND_SOC_DAPM_HP(w->name, gbcodec_event_hp); + *dw = gbaudio_widgets[w->type]; module->op_devices |= (GBAUDIO_DEVICE_OUT_WIRED_HEADSET | GBAUDIO_DEVICE_OUT_WIRED_HEADPHONE); module->ip_devices |= GBAUDIO_DEVICE_IN_WIRED_HEADSET; break; case snd_soc_dapm_mic: - *dw = (struct snd_soc_dapm_widget) - SND_SOC_DAPM_MIC(w->name, gbcodec_event_int_mic); + *dw = gbaudio_widgets[w->type]; module->ip_devices |= GBAUDIO_DEVICE_IN_BUILTIN_MIC; break; case snd_soc_dapm_output: - *dw = (struct snd_soc_dapm_widget)SND_SOC_DAPM_OUTPUT(w->name); - break; case snd_soc_dapm_input: - *dw = (struct snd_soc_dapm_widget)SND_SOC_DAPM_INPUT(w->name); - break; case snd_soc_dapm_switch: - *dw = (struct snd_soc_dapm_widget) - SND_SOC_DAPM_SWITCH_E(w->name, SND_SOC_NOPM, 0, 0, - widget_kctls, - gbaudio_widget_event, - SND_SOC_DAPM_PRE_PMU | - SND_SOC_DAPM_POST_PMD); - break; case snd_soc_dapm_pga: - *dw = (struct snd_soc_dapm_widget) - SND_SOC_DAPM_PGA_E(w->name, SND_SOC_NOPM, 0, 0, NULL, 0, - gbaudio_widget_event, - SND_SOC_DAPM_PRE_PMU | - SND_SOC_DAPM_POST_PMD); - break; case snd_soc_dapm_mixer: - *dw = (struct snd_soc_dapm_widget) - SND_SOC_DAPM_MIXER_E(w->name, SND_SOC_NOPM, 0, 0, NULL, - 0, gbaudio_widget_event, - SND_SOC_DAPM_PRE_PMU | - SND_SOC_DAPM_POST_PMD); - break; case snd_soc_dapm_mux: - *dw = (struct snd_soc_dapm_widget) - SND_SOC_DAPM_MUX_E(w->name, SND_SOC_NOPM, 0, 0, - widget_kctls, gbaudio_widget_event, - SND_SOC_DAPM_PRE_PMU | - SND_SOC_DAPM_POST_PMD); + *dw = gbaudio_widgets[w->type]; break; case snd_soc_dapm_aif_in: - *dw = (struct snd_soc_dapm_widget) - SND_SOC_DAPM_AIF_IN_E(w->name, w->sname, 0, - SND_SOC_NOPM, - 0, 0, gbaudio_widget_event, - SND_SOC_DAPM_PRE_PMU | - SND_SOC_DAPM_POST_PMD); - break; case snd_soc_dapm_aif_out: - *dw = (struct snd_soc_dapm_widget) - SND_SOC_DAPM_AIF_OUT_E(w->name, w->sname, 0, - SND_SOC_NOPM, - 0, 0, gbaudio_widget_event, - SND_SOC_DAPM_PRE_PMU | - SND_SOC_DAPM_POST_PMD); + *dw = gbaudio_widgets[w->type]; + dw->sname = w->sname; break; default: ret = -EINVAL; goto error; } + dw->name = w->name; dev_dbg(module->dev, "%s: widget of type %d created\n", dw->name, dw->id); base-commit: 42eb8fdac2fc5d62392dcfcf0253753e821a97b0 -- 2.34.1

3 years, 2 months

[PATCH] staging: greybus: Add missing rwsem around snd_ctl_remove() calls

by Takashi Iwai

snd_ctl_remove() has to be called with card->controls_rwsem held (when called after the card instantiation). This patch adds the missing rwsem calls around it. Fixes: 510e340efe0c ("staging: greybus: audio: Add helper APIs for dynamic audio modules") Signed-off-by: Takashi Iwai <tiwai(a)suse.de> --- drivers/staging/greybus/audio_helper.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/staging/greybus/audio_helper.c b/drivers/staging/greybus/audio_helper.c index 1ed4772d2771..843760675876 100644 --- a/drivers/staging/greybus/audio_helper.c +++ b/drivers/staging/greybus/audio_helper.c @@ -192,7 +192,11 @@ int gbaudio_remove_component_controls(struct snd_soc_component *component, unsigned int num_controls) { struct snd_card *card = component->card->snd_card; + int err; - return gbaudio_remove_controls(card, component->dev, controls, - num_controls, component->name_prefix); + down_write(&card->controls_rwsem); + err = gbaudio_remove_controls(card, component->dev, controls, + num_controls, component->name_prefix); + up_write(&card->controls_rwsem); + return err; } -- 2.31.1

3 years, 3 months

[PATCH] Staging: greybus: audio_codec: changed a void function to return 0

by Senthu Sivarajah

From: Senthuran Sivarajah <ssivaraj(a)stud.fra-uas.de> Fixed a coding style issue. Signed-off-by: Senthuran Sivarajah <ssivaraj(a)stud.fra-uas.de> --- drivers/staging/greybus/audio_codec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/greybus/audio_codec.c b/drivers/staging/greybus/audio_codec.c index b589cf6b1d03..9f99862791dc 100644 --- a/drivers/staging/greybus/audio_codec.c +++ b/drivers/staging/greybus/audio_codec.c @@ -1028,7 +1028,7 @@ static int gbcodec_probe(struct snd_soc_component *comp) static void gbcodec_remove(struct snd_soc_component *comp) { /* Empty function for now */ - return; + return 0; } static int gbcodec_write(struct snd_soc_component *comp, unsigned int reg, -- 2.27.0

3 years, 3 months

← Newer
1
...
20
21
22
23
24
25
26
...
55
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

greybus-dev