[alsa-devel] [PATCH] sst: fix compile error
guanqun.lu at intel.com
Thu Apr 7 08:44:20 CEST 2011
On Thu, Apr 07, 2011 at 02:27:18PM +0800, Mark Brown wrote:
> On Thu, Apr 07, 2011 at 10:18:13AM +0800, Lu Guanqun wrote:
> > Add several include files to fix the below compile error.
> > drivers/staging/intel_sst/intelmid.c: In function ‘snd_intelmad_sst_register’:
> > drivers/staging/intel_sst/intelmid.c:612:2: error: ‘sst_drv_ctx’ undeclared (first use in this function)
> > drivers/staging/intel_sst/intelmid.c:612:2: note: each undeclared identifier is reported only once for each function it appears in
> > Signed-off-by: Lu Guanqun <guanqun.lu at intel.com>
> Is this needed in 2.6.39? Greg, I'm happy to take this through ASoC if
> you're OK with that.
This isn't needed for 2.6.39.
This compile error is actually caused by my fault, see below commit.
Author: Lu Guanqun <guanqun.lu at intel.com>
Date: Wed Apr 6 10:20:42 2011 +0800
sst: make register_sst_card more self-contained
register_sst_card is used in ASoC code with field `scard_ops` being NULL.
Without this patch, there will be NULL dereference.
Signed-off-by: Lu Guanqun <guanqun.lu at intel.com>
Signed-off-by: Mark Brown <broonie at opensource.wolfsonmicro.com>
And this commit is only in your branch for-2.6.40. I failed to add
CONFIG_INTELMID=y in my previous test, so this error is hidden.
> > ---
> > drivers/staging/intel_sst/intelmid.c | 3 +++
> > 1 files changed, 3 insertions(+), 0 deletions(-)
> > diff --git a/drivers/staging/intel_sst/intelmid.c b/drivers/staging/intel_sst/intelmid.c
> > index 32ded14..bbda30f 100644
> > --- a/drivers/staging/intel_sst/intelmid.c
> > +++ b/drivers/staging/intel_sst/intelmid.c
> > @@ -37,8 +37,11 @@
> > #include <sound/pcm_params.h>
> > #include <sound/initval.h>
> > #include <linux/gpio.h>
> > +#include <linux/firmware.h>
> > #include "intel_sst.h"
> > #include "intel_sst_ioctl.h"
> > +#include "intel_sst_fw_ipc.h"
> > +#include "intel_sst_common.h"
> > #include "intelmid_snd_control.h"
> > #include "intelmid_adc_control.h"
> > #include "intelmid.h"
More information about the Alsa-devel