wm8960: fix compile error
Subsystems (3)
PATCH 1/1 wm8960: fix compile error
This patch fixes compile error:sound/soc/codecs/wm8960.c: In function 'wm8960_hw_params':sound/soc/codecs/wm8960.c:511: error: 'codec' undeclared (first use in this function) sound/soc/codecs/wm8960.c:511: error: (Each undeclared identifier is reported only once sound/soc/codecs/wm8960.c:511: error: for each function it appears in.) make[3]: *** [sound/soc/codecs/wm8960.o] Error 1
make[2]: *** [sound/soc/codecs] Error 2
make[1]: *** [sound/soc] Error 2
make: *** [sound] Error 2
---
This patch is against Liam's multi-component branch.
Axel
sound/soc/codecs/wm8960.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
sound/soc/codecs/wm8960.c +1 -1
@@ -508,8 +508,8 @@ static int wm8960_hw_params(struct snd_pcm_substream *substream, struct snd_soc_dai *dai) {struct snd_soc_pcm_runtime *rtd = substream->private_data; -struct wm8960_priv *wm8960 =snd_soc_codec_get_drvdata (codec);struct snd_soc_codec *codec = rtd->codec; +struct wm8960_priv *wm8960 =snd_soc_codec_get_drvdata (codec);u16 iface =snd_soc_read (codec,WM8960_IFACE1 ) &0xfff3 ;int i;
Signature · 1 line
1.5.4.3
Quoted 1 line · Axel Lin
This patch is against Liam's multi-component branch.
but please do try to bear in mind the feedback you've previously been given about formatting your patches so they can be easily applied. You probably also want to look at the CC list for your patches, I'm not sure why you're CCing Tejun here.
hi Mark,
2010/7/20 Mark Brown <[email protected]>:
Quoted 8 lines · Axel Lin
Quoted 2 lines
This patch is against Liam's multi-component branch.
Acked-by Mark Brown <[email protected]>but please do try to bear in mind the feedback you've previously been given about formatting your patches so they can be easily applied. You
I'm a little bit confused about the meaning of "formatting your patches"? Do you mean my patches cannot be applied by git am command? or? ( I tried the git am command with the mail I sent and it looks ok )
Quoted 2 lines
probably also want to look at the CC list for your patches, I'm not sure why you're CCing Tejun here.
I get the CC list by get_maintainer.pl. Looks like the script gives me wrong CC list.
Regards,
Axel
Quoted 8 lines · Axel Lin
2010/7/20 Mark Brown <[email protected]>:
Quoted 2 lines
but please do try to bear in mind the feedback you've previously been given about formatting your patches so they can be easily applied. You
I'm a little bit confused about the meaning of "formatting your patches"? Do you mean my patches cannot be applied by git am command? or? ( I tried the git am command with the mail I sent and it looks ok )
In this case the main thing is the subject line which is not consistent with other subject lines for patches in this area of code. This sort of stuff is particularly important when you're submitting large numbers of very small fixes.
Quoted 5 lines
Quoted 2 lines
probably also want to look at the CC list for your patches, I'm not sure why you're CCing Tejun here.
I get the CC list by get_maintainer.pl. Looks like the script gives me wrong CC list.
Yes, you need to apply a bit of taste to the output - it's inclined to generate false positives. For matches from MAINTAINERS it's good but for things generated from the git logs it can easily miss people or pick up people who've been doing kernel wide cleanup work.