lkml.org 
[lkml]   [2009]   [May]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [v0 PATCH 1/1] Use dev_name() for AMD8111/AMD8131 EDAC driver
Hi Harry,

On Mon, 11 May 2009 10:47:45 +0800 Harry Ciao <qingtao.cao@windriver.com> wrote:
>
> Andrew Morton wrote:
> >
> > The drivers break x86-64 allmodconfig:
> >
> > drivers/edac/amd8111_edac.c: In function 'amd8111_lpc_bridge_init':
> > drivers/edac/amd8111_edac.c:260: error: implicit declaration of function '__do_inb'
> > drivers/edac/amd8111_edac.c:274: error: implicit declaration of function '__do_outb'
>
> Ah, this is because __do_inb so far only supported on PPC arch:
>
> cao@cao-desktop:/work/community/linux-2.6/arch$ grep __do_inb * -r
> powerpc/include/asm/io.h:#define __do_inb(port) _rec_inb(port)
> powerpc/include/asm/io.h:#define __do_inb(port)
> readb((PCI_IO_ADDR)_IO_BASE + port);
> cao@cao-desktop:/work/community/linux-2.6/arch$
>
> So far the only machine that hosts AMD8111 chip is Maple, however,
> AMD8111 is of Hypertransport device and not necessarily depends on Maple
> nor PPC, so its Kconfig option is only depended on EDAC && EDAC_MC_EDAC
> && PCI.
>
> I think I could put some words into its Kconfig help section that
> AMD8111 should be selected on Maple now.
>
> What do you think? thanks!

You need to make them depend on CONFIG_PPC (or use a different i/o
interface) since the automated builds will enable them for
all{mod,yes}config builds. Some explanation of the dependency is, of
course, appropriate.

--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2009-05-11 05:59    [W:0.033 / U:33.120 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site