[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <1174631473.18866.24.camel@roc-desktop>
Date: Fri, 23 Mar 2007 14:31:13 +0800
From: "Wu, Bryan" <bryan.wu@...log.com>
To: Paul Mundt <lethal@...ux-sh.org>
Cc: "Wu, Bryan" <bryan.wu@...log.com>, Arnd Bergmann <arnd@...db.de>,
Andrew Morton <akpm@...ux-foundation.org>,
bert hubert <bert.hubert@...herlabs.nl>,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH -mm try#2] Blackfin: architecture update patch
On Fri, 2007-03-23 at 15:12 +0900, Paul Mundt wrote:
> On Fri, Mar 23, 2007 at 02:04:30PM +0800, Wu, Bryan wrote:
> > This is the latest blackfin update patch. Because there are lots of
> > issue fixing in this one, I put all modification in one update patch
> > which is located in:
> > https://blackfin.uclinux.org/gf/download/frsrelease/39/2707/blackfin-arch-2.6.21-rc4-mm1-update.patch
> >
> I hope these will split up logically in the future so it's possible to
> reply to them without having to do manual mangling..
>
>>From now on, I will follow this rule. Thanks
> The patch generally looks fine, this is the only thing that really jumped
> out:
>
> > Index: linux-2.6/include/asm-blackfin/pgtable.h
> > ===================================================================
> > --- linux-2.6.orig/include/asm-blackfin/pgtable.h
> > +++ linux-2.6/include/asm-blackfin/pgtable.h
> > @@ -59,4 +59,12 @@
> > #define VMALLOC_START 0
> > #define VMALLOC_END 0xffffffff
> >
> > +#define __HAVE_ARCH_ENTER_LAZY_CPU_MODE
> > +#define arch_enter_lazy_cpu_mode() do {} while (0)
> > +#define arch_leave_lazy_cpu_mode() do {} while (0)
> > +
> > +#define __HAVE_ARCH_ENTER_LAZY_MMU_MODE
> > +#define arch_enter_lazy_mmu_mode() do {} while (0)
> > +#define arch_leave_lazy_mmu_mode() do {} while (0)
> > +
> > #endif /* _BLACKFIN_PGTABLE_H */
>
> asm-generic/pgtable.h already does this if you don't explicitly define
> __HAVE_ARCH_ENTER_LAZY_{CPU,MMU}_MODE. So please kill this entirely. If
> you forgot to include asm-generic/pgtable.h, that's another matter..
OK, I will check with this. Adding this dummy function is because
compiling will fail.
Best Regards,
-Bryan Wu
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists