From: Baruch Siach on
Hi Christoph,

On Wed, Jun 30, 2010 at 06:05:38PM +0200, Christoph Egger wrote:
> ARCH_DAVINCI_TNETV107X doesn't exist in Kconfig, therefore removing
> all references for it from the source code.
>
> Signed-off-by: Christoph Egger <siccegge(a)cs.fau.de>
> ---
> arch/arm/mach-davinci/include/mach/cputype.h | 6 ------
> 1 files changed, 0 insertions(+), 6 deletions(-)
>
> diff --git a/arch/arm/mach-davinci/include/mach/cputype.h b/arch/arm/mach-davinci/include/mach/cputype.h
> index cea6b89..6143022 100644
> --- a/arch/arm/mach-davinci/include/mach/cputype.h
> +++ b/arch/arm/mach-davinci/include/mach/cputype.h
> @@ -85,10 +85,4 @@ IS_DAVINCI_CPU(tnetv107x, DAVINCI_CPU_ID_TNETV107X)
> #define cpu_is_davinci_da850() 0
> #endif
>
> -#ifdef CONFIG_ARCH_DAVINCI_TNETV107X
> -#define cpu_is_davinci_tnetv107x() is_davinci_tnetv107x()
> -#else
> -#define cpu_is_davinci_tnetv107x() 0

Why remove the #else part?

baruch

> -#endif
> -
> #endif
> --
> 1.7.0.4

--
~. .~ Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
- baruch(a)tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
From: Kirill A. Shutemov on
On Wed, Jun 30, 2010 at 08:07:31PM +0300, Baruch Siach wrote:
> Hi Christoph,
>
> On Wed, Jun 30, 2010 at 06:05:38PM +0200, Christoph Egger wrote:
> > ARCH_DAVINCI_TNETV107X doesn't exist in Kconfig, therefore removing
> > all references for it from the source code.
> >
> > Signed-off-by: Christoph Egger <siccegge(a)cs.fau.de>
> > ---
> > arch/arm/mach-davinci/include/mach/cputype.h | 6 ------
> > 1 files changed, 0 insertions(+), 6 deletions(-)
> >
> > diff --git a/arch/arm/mach-davinci/include/mach/cputype.h b/arch/arm/mach-davinci/include/mach/cputype.h
> > index cea6b89..6143022 100644
> > --- a/arch/arm/mach-davinci/include/mach/cputype.h
> > +++ b/arch/arm/mach-davinci/include/mach/cputype.h
> > @@ -85,10 +85,4 @@ IS_DAVINCI_CPU(tnetv107x, DAVINCI_CPU_ID_TNETV107X)
> > #define cpu_is_davinci_da850() 0
> > #endif
> >
> > -#ifdef CONFIG_ARCH_DAVINCI_TNETV107X
> > -#define cpu_is_davinci_tnetv107x() is_davinci_tnetv107x()
> > -#else
> > -#define cpu_is_davinci_tnetv107x() 0
>
> Why remove the #else part?

Nobody uses cpu_is_davinci_tnetv107x as well.

Acked-by: Kirill A. Shutemov <kirill(a)shutemov.name>

--
Kirill A. Shutemov
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
From: Kevin Hilman on
Christoph Egger <siccegge(a)cs.fau.de> writes:

> ARCH_DAVINCI_TNETV107X doesn't exist in Kconfig, therefore removing
> all references for it from the source code.
>
> Signed-off-by: Christoph Egger <siccegge(a)cs.fau.de>

Please leave this define in.

The full support for this platform is currently in linux-next and will
be merged for 2.6.36.

Kevin
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/