Re: Compile Error on 2.6.17-mm6

[Andrew Morton <akpm@osdl.org>]

On Sat, 8 Jul 2006 21:37:34 -0700
"Randy.Dunlap" <rdunlap@xenotime.net> wrote:

> On Sat, 8 Jul 2006 20:44:48 -0700 Andrew Morton wrote:
> 
> > On Sat, 8 Jul 2006 20:34:24 -0700
> > "Randy.Dunlap" <rdunlap@xenotime.net> wrote:
> > 
> > > On Sat, 8 Jul 2006 17:43:47 -0700 Andrew Morton wrote:
> > > 
> > > > On Fri, 7 Jul 2006 22:22:16 -0400
> > > > Matt Reuther <mreuther@umich.edu> wrote:
> > > > 
> > > > > Here is the error:
> > > > >   CHK     include/linux/compile.h
> > > > >   UPD     include/linux/compile.h
> > > > >   CC      init/version.o
> > > > >   LD      init/built-in.o
> > > > >   LD      .tmp_vmlinux1
> > > > > arch/i386/kernel/built-in.o(.text+0xe282): In function 
> > > > > `cpu_request_microcode':
> > > > > arch/i386/kernel/microcode.c:544: undefined reference to `request_firmware'
> > > > > arch/i386/kernel/built-in.o(.text+0xe304):arch/i386/kernel/microcode.c:573: 
> > > > > undefined reference to `release_firmware'
> > > > 
> > > > CONFIG_FW_LOADER=m
> > > > CONFIG_MICROCODE=y
> > > > 
> > > > So
> > > > 
> > > > config MICROCODE
> > > > 	tristate "/dev/cpu/microcode - Intel IA32 CPU microcode support"
> > > > 	depends on FW_LOADER
> > > > 
> > > > is not sufficient.  There's a fix for this, but I cannot remember what it
> > > > is.  Help.
> > > 
> > > That 1-line depends patch fixes the problem for me (on x86-64,
> > > but they are the same in this area).
> > > 
> > 
> > What patch is that?
> 
> My -mm6 does not have "depends on FW_LOADER" like you wrote above,
> so I thought that you had added that 1-line as a patch.
> 

Oh, OK, yes, I fixed that post-2.6.17-mm6.

>  arch/x86_64/Kconfig |    1 +

Although I missed x86_64.