* DMA_ERROR_CODE (was: Re: linux-next: build failure after merge of the final tree)
@ 2010-03-10 11:03 Geert Uytterhoeven
2010-03-11 0:15 ` FUJITA Tomonori
0 siblings, 1 reply; 3+ messages in thread
From: Geert Uytterhoeven @ 2010-03-10 11:03 UTC (permalink / raw)
To: Stephen Rothwell
Cc: David Woodhouse, linux-next, linux-kernel, Maxim Levitsky,
linux-arch
On Mon, Mar 1, 2010 at 10:40, Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> After merging the final tree, today's linux-next build (powerpc
> allyesconfig) failed like this:
>
> drivers/mtd/nand/r852.c:22: error: 'enable_dma' redeclared as different kind of symbol
> arch/powerpc/include/asm/dma.h:189: note: previous definition of 'enable_dma' was here
> drivers/mtd/nand/r852.c: In function 'r852_engine_enable':
> drivers/mtd/nand/r852.c:545: error: implicit declaration of function 'msleep'
>
> Caused by commit 67e054e919248fa1db93de727fb9ad49eb700642 ("mtd: nand:
> Add driver for Ricoh xD/SmartMedia reader") from the mtd.
>
> I applied the following patch for today.
>
> From: Stephen Rothwell <sfr@canb.auug.org.au>
> Date: Mon, 1 Mar 2010 20:21:06 +1100
> Subject: [PATCH] mtd: nand: fix name space clash
>
> and include delay.h for msleep().
Another issue is that this driver uses DMA_ERROR_CODE, which is currently
defined on powerpc/sparc/x86 only.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: DMA_ERROR_CODE (was: Re: linux-next: build failure after merge of the final tree)
2010-03-10 11:03 DMA_ERROR_CODE (was: Re: linux-next: build failure after merge of the final tree) Geert Uytterhoeven
@ 2010-03-11 0:15 ` FUJITA Tomonori
2010-03-11 17:30 ` David Woodhouse
0 siblings, 1 reply; 3+ messages in thread
From: FUJITA Tomonori @ 2010-03-11 0:15 UTC (permalink / raw)
To: geert; +Cc: sfr, dwmw2, linux-next, linux-kernel, maximlevitsky, linux-arch
On Wed, 10 Mar 2010 12:03:37 +0100
Geert Uytterhoeven <geert@linux-m68k.org> wrote:
> On Mon, Mar 1, 2010 at 10:40, Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> > After merging the final tree, today's linux-next build (powerpc
> > allyesconfig) failed like this:
> >
> > drivers/mtd/nand/r852.c:22: error: 'enable_dma' redeclared as different kind of symbol
> > arch/powerpc/include/asm/dma.h:189: note: previous definition of 'enable_dma' was here
> > drivers/mtd/nand/r852.c: In function 'r852_engine_enable':
> > drivers/mtd/nand/r852.c:545: error: implicit declaration of function 'msleep'
> >
> > Caused by commit 67e054e919248fa1db93de727fb9ad49eb700642 ("mtd: nand:
> > Add driver for Ricoh xD/SmartMedia reader") from the mtd.
> >
> > I applied the following patch for today.
> >
> > From: Stephen Rothwell <sfr@canb.auug.org.au>
> > Date: Mon, 1 Mar 2010 20:21:06 +1100
> > Subject: [PATCH] mtd: nand: fix name space clash
> >
> > and include delay.h for msleep().
>
> Another issue is that this driver uses DMA_ERROR_CODE, which is currently
> defined on powerpc/sparc/x86 only.
Why can't use dma_mapping_error()?
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: DMA_ERROR_CODE (was: Re: linux-next: build failure after merge of the final tree)
2010-03-11 0:15 ` FUJITA Tomonori
@ 2010-03-11 17:30 ` David Woodhouse
0 siblings, 0 replies; 3+ messages in thread
From: David Woodhouse @ 2010-03-11 17:30 UTC (permalink / raw)
To: FUJITA Tomonori
Cc: geert, sfr, linux-next, linux-kernel, maximlevitsky, linux-arch
On Thu, 2010-03-11 at 09:15 +0900, FUJITA Tomonori wrote:
> On Wed, 10 Mar 2010 12:03:37 +0100
> Geert Uytterhoeven <geert@linux-m68k.org> wrote:
>
> > On Mon, Mar 1, 2010 at 10:40, Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> > > After merging the final tree, today's linux-next build (powerpc
> > > allyesconfig) failed like this:
> > >
> > > drivers/mtd/nand/r852.c:22: error: 'enable_dma' redeclared as different kind of symbol
> > > arch/powerpc/include/asm/dma.h:189: note: previous definition of 'enable_dma' was here
> > > drivers/mtd/nand/r852.c: In function 'r852_engine_enable':
> > > drivers/mtd/nand/r852.c:545: error: implicit declaration of function 'msleep'
> > >
> > > Caused by commit 67e054e919248fa1db93de727fb9ad49eb700642 ("mtd: nand:
> > > Add driver for Ricoh xD/SmartMedia reader") from the mtd.
> > >
> > > I applied the following patch for today.
> > >
> > > From: Stephen Rothwell <sfr@canb.auug.org.au>
> > > Date: Mon, 1 Mar 2010 20:21:06 +1100
> > > Subject: [PATCH] mtd: nand: fix name space clash
> > >
> > > and include delay.h for msleep().
> >
> > Another issue is that this driver uses DMA_ERROR_CODE, which is currently
> > defined on powerpc/sparc/x86 only.
>
> Why can't use dma_mapping_error()?
http://git.infradead.org/mtd-2.6.git/commitdiff/0c82d3ce
--
David Woodhouse Open Source Technology Centre
David.Woodhouse@intel.com Intel Corporation
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2010-03-11 17:30 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-03-10 11:03 DMA_ERROR_CODE (was: Re: linux-next: build failure after merge of the final tree) Geert Uytterhoeven
2010-03-11 0:15 ` FUJITA Tomonori
2010-03-11 17:30 ` David Woodhouse
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).