On Thu, May 09, 2019 at 12:22:23AM +0800, Tzung-Bi Shih wrote: > static int max98357a_component_probe(struct snd_soc_component *component) > { > - struct gpio_desc *sdmode; > - > - sdmode = devm_gpiod_get_optional(component->dev, "sdmode", GPIOD_OUT_LOW); > - if (IS_ERR(sdmode)) > - return PTR_ERR(sdmode); > - > - snd_soc_component_set_drvdata(component, sdmode); > - > return 0; > } > This function is now empty so can just be removed entirely. Otherwise this looks good.
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx https://mailman.alsa-project.org/mailman/listinfo/alsa-devel