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.
_______________________________________________
Xenomai-help mailing list
[email protected]
https://mail.gna.org/listinfo/xenomai-help
