Re: [PATCH v2] ASoC: sgtl5000: Fix powerup/down of micbias

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mi, 2018-01-10 at 14:26 +0000, Fischer, Christian wrote:
> Hello Fabio,
> 
> is it now ok?
> I'm not working with linux-git, so my patch is created by myself.
> 
> Regards,
> Christian
> 
> Signed-off-by: Christian Fischer <fischerc@xxxxxxxxxxxxx>

Hi Christian,

This doesn't look good to me. How are creating/sending the patches?

You should update the commit message to include your Signed-off-by tag
and then resend the entire patch.

thanks,
Daniel.


> 
> --- sound/soc/codecs/sgtl5000.c
> +++ sound/soc/codecs/sgtl5000.c
> @@ -260,6 +260,9 @@ static const struct snd_soc_dapm_widget
>  
>  	SND_SOC_DAPM_PRE("VAG_POWER_PRE", power_vag_event),
>  	SND_SOC_DAPM_POST("VAG_POWER_POST", power_vag_event),
> +
> +	SND_SOC_DAPM_PRE("MIC_BIAS_PRE", mic_bias_event),
> +	SND_SOC_DAPM_POST("MIC_BIAS_POST", mic_bias_event),
>  };
>  
>  /* routes for sgtl5000 */
> 
> -----Ursprüngliche Nachricht-----
> Von: Fabio Estevam [mailto:festevam@xxxxxxxxx] ;
> Gesendet: Mittwoch, 10. Januar 2018 13:42
> An: Fischer, Christian <fischerc@xxxxxxxxxxxxx>
> Cc: alsa-devel@xxxxxxxxxxxxxxxx; Takashi Iwai <tiwai@xxxxxxxx>; Mark Brown <broonie@xxxxxxxxxx>; Liam Girdwood <lgirdwood@xxxxxxxxx>
> Betreff: Re:  [PATCH] ASoC: sgtl5000: Fix powerup/down of micbias
> 
> Hi Christian,
> 
> On Wed, Jan 10, 2018 at 8:07 AM, Fischer, Christian <fischerc@xxxxxxxxxxxxx> wrote:
> > 
> > The bias-bias-voltage should powered up/down automatically.
> > Added missing DAPM_PRE and DAPM_POST widgets to fire up the mic_bias_event function.
> Thanks for submitting the patch.
> 
> You missed your Signed-off-by tag.
> 
> Please resend it.
> 
> Thanks
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel@xxxxxxxxxxxxxxxx
> https://emea01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fmailman.alsa-project.org%2Fmailman%2Flistinfo%2Falsa-devel&data=02%7C01%7Cdaniel.baluta%40nxp.com%7C4a771694af314fa7d73008d558362a37
> %7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C636511912029282885&sdata=Zo42ZJxIxp54YyYLVsFrrRWkDt%2FqByR6uqr8U6pX3iY%3D&reserved=0
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel




[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux