lkml.org 
[lkml]   [2012]   [Sep]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 2/6] ARM: dove: Fix tauros2 device tree init
On Tue, Sep 25, 2012 at 03:22:04AM +0200, Sebastian Hesselbarth wrote:
> On 09/25/2012 02:02 AM, Sebastian Hesselbarth wrote:
> >During the review process of dove DT patches, Tauros2 cache
> >init call was changed and DT support added. This patch fixes
> >the call to Tauros2 init and adds a DT node. Moreover, plat/irq.h
> >include was missing from mach-dove/common.c.
> > ...
> >diff --git a/arch/arm/mach-dove/common.c b/arch/arm/mach-dove/common.c
> >index b37bef1..343a4bc 100644
> >--- a/arch/arm/mach-dove/common.c
> >+++ b/arch/arm/mach-dove/common.c
> >@@ -32,6 +32,7 @@
> > #include<linux/irq.h>
> > #include<plat/time.h>
> > #include<linux/platform_data/usb-ehci-orion.h>
> >+#include<plat/irq.h>
> > #include<plat/common.h>
> > #include<plat/addr-map.h>
> > #include "common.h"
> >@@ -399,7 +400,7 @@ static void __init dove_dt_init(void)
> > (dove_tclk + 499999) / 1000000);
> >
> > #ifdef CONFIG_CACHE_TAUROS2
> >- tauros2_init();
> >+ tauros2_init(0);
> > #endif
> > dove_setup_cpu_mbus();
> >
>
> I thought about the importance of the individual patches and
> except 2/6 all can wait for the next release cycle if too late.
>
> But 2/6 is important because the change in tauros2_init
> breaks build on dove.

Hi Sebastian

Interestingly, kisskb does not show this break:

http://kisskb.ellerman.id.au/kisskb/config/308/

and yesterdays build does not have the parameter to tauros2_init().
Is the cache not enabled in dove_defconfig?

Andrew


\
 
 \ /
  Last update: 2012-09-25 07:41    [W:0.068 / U:1.392 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site