Re: [Xenomai-help] Build and run xenomai 2.5.5.2 - linux-2.6.33 on imx27

[Gilles Chanteperdrix <gilles.chanteperdrix@xenomai.org>]

Fabrice Gasnier wrote:
> Dear all,
> 
> I'm currently evaluating xenomai 2.5.5.2 on a i.MX27 platform with 
> adeos-ipipe-2.6.33-arm-1.18-01.patch.
> 
> 
> I've encountred two issues for the time being  (same with 
> adeos-ipipe-2.6.33-arm-1.18-00.patch).
> 
> 1°/ First one happens when building a 2.6.33.3 kernel (after xenomai 
> prepare/configure/compile stage):
> 
> CC      arch/arm/plat-mxc/time.o
> arch/arm/plat-mxc/time.c: In function 'mxc_timer_init':
> arch/arm/plat-mxc/time.c:400: error: 'TIM1_BASE_ADDR' undeclared (first 
> use in this function)
> arch/arm/plat-mxc/time.c:400: error: (Each undeclared identifier is 
> reported only once
> arch/arm/plat-mxc/time.c:400: error: for each function it appears in.)
> make[1]: *** [arch/arm/plat-mxc/time.o] Erreur 1
> make: *** [arch/arm/plat-mxc] Erreur 2
> 
> 
> I first tried 2.6.31 kernel that was compiling (patched with 
> adeos-ipipe-2.6.31-arm-1.16-02.patch).
> Looking for differences lead me to patch my kernel with:
> 
> Index: linux-2.6.33.3/arch/arm/plat-mxc/time.c
> ===================================================================
> --- linux-2.6.33.3.orig/arch/arm/plat-mxc/time.c    2011-02-28 
> 11:44:02.000000000 +0100
> +++ linux-2.6.33.3/arch/arm/plat-mxc/time.c    2011-02-28 
> 11:48:19.000000000 +0100
> @@ -395,12 +395,17 @@
>       tsc_info.freq = clk_get_rate(timer_clk);
>       mxc_min_delay = ((__ipipe_cpu_freq + 500000) / 1000000) ?: 1;
> 
> -    if (cpu_is_mx1() || cpu_is_mx2()) {
> -#if defined(CONFIG_ARCH_MX1) || defined(CONFIG_ARCH_MX2)
> +    if (cpu_is_mx1()) {
> +#ifdef CONFIG_ARCH_MX1
>           tsc_info.u.counter_paddr = (TIM1_BASE_ADDR + MX1_2_TCN);
>           tsc_info.counter_vaddr =
>               (unsigned long)(timer_base + MX1_2_TCN);
>   #endif
> +    } else if (cpu_is_mx2()) {
> +#ifdef CONFIG_ARCH_MX2
> +        tsc_info.u.counter_paddr = (GPT1_BASE_ADDR + MX1_2_TCN);
> +        tsc_info.counter_vaddr = (unsigned long)(timer_base + MX1_2_TCN);
> +#endif
>       } else if (cpu_is_mx3()) {
>   #ifdef CONFIG_ARCH_MX3
>           tsc_info.u.counter_paddr = (GPT1_BASE_ADDR + MX3_TCN);

Ok. I'll take this one.

>   #ifdef CONFIG_IPIPE
>   void ipipe_mach_allow_hwtimer_uaccess(unsigned long aips1, unsigned 
> long aips2);
> +#if defined(CONFIG_ARCH_MX1) || defined(CONFIG_ARCH_MX2)
> +void ipipe_mach_allow_uaccess(unsigned long aipi_base_addr, unsigned 
> long bit_num);
> +#endif
>   #endif

Look the function above, you are adding a function which is already
there, please reuse the same function, instead of duplicating it...

-- 
					    Gilles.