Re: [PATCH v21 07/39] ASoC: Add SOC USB APIs for adding an USB backend

+const char *snd_soc_usb_get_components_tag(bool playback) +{

• if (playback)

• return "usbplaybackoffload: 1";
  

• else

• return "usbcaptureoffload : 1";
  

why are there different spaces and do we need spaces in the first place?

+int snd_soc_usb_add_port(struct snd_soc_usb *usb) +{

• mutex_lock(&ctx_mutex);
• list_add_tail(&usb->list, &usb_ctx_list);
• mutex_unlock(&ctx_mutex);
• return 0;

make the function return void?

+int snd_soc_usb_remove_port(struct snd_soc_usb *usb) +{

• struct snd_soc_usb *ctx, *tmp;
• mutex_lock(&ctx_mutex);
• list_for_each_entry_safe(ctx, tmp, &usb_ctx_list, list) {

• if (ctx == usb) {
  

• 	list_del(&ctx->list);
  

• 	break;
  

• }
  

• }
• mutex_unlock(&ctx_mutex);
• return 0;

make this return void?