From: Gilles Chanteperdrix <gilles.chanteperdrix@xenomai.org>
To: Richard Cochran <richardcochran@domain.hid>
Cc: adeos-main@gna.org, Xenomai-core@domain.hid
Subject: Re: [Xenomai-core] [Adeos-main] ARM IXP: ipipe regression from 2.6.31 to .33 and .35
Date: Fri, 08 Apr 2011 08:16:17 +0200 [thread overview]
Message-ID: <4D9EA831.3020109@domain.hid> (raw)
In-Reply-To: <20110408060503.GA6174@domain.hid>
Richard Cochran wrote:
> On Thu, Apr 07, 2011 at 10:26:02PM +0200, Gilles Chanteperdrix wrote:
>> On my side, I meant to say, the issue is in the implementation of the
>> I-pipe one-shot timer, the function ipipe_mach_set_dec...
>
> I couldn't find anything wrong there, but I did find this...
>
> In ipipe_tsc_asm.S you have
>
> #ifndef CONFIG_CPU_ENDIAN_BE8
> /* Little endian */
> ldr r0, [r0]
> cmp r2, r0
> adc r1, r3, #0
> #else /* Big endian */
> ldr r1, [r0]
> cmp r3, r1
> adc r0, r2, #0
> #endif /* Big endian */
>
> but my .config has
>
> CONFIG_ARCH_SUPPORTS_BIG_ENDIAN=y
> CONFIG_CPU_BIG_ENDIAN=y
> # CONFIG_CPU_ENDIAN_BE8 is not set
> CONFIG_CPU_ENDIAN_BE32=y
>
> and the machine really is big endian, so that might make a difference.
Yes, you can try changing the #ifdef. But note also that the big endian
code was never actually tested. So, you probably would be better
compiling a little user-space application to test this tsc code.
the Linksys WRT54GS is an ixp 425, right?.
--
Gilles.
next prev parent reply other threads:[~2011-04-08 6:16 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-04-07 17:08 [Xenomai-core] ARM IXP: ipipe regression from 2.6.31 to .33 and .35 Richard Cochran
2011-04-07 18:32 ` [Xenomai-core] [Adeos-main] " Gilles Chanteperdrix
2011-04-07 19:02 ` Richard Cochran
2011-04-07 20:12 ` Richard Cochran
2011-04-07 20:26 ` Gilles Chanteperdrix
2011-04-08 6:05 ` Richard Cochran
2011-04-08 6:12 ` Richard Cochran
2011-04-08 6:18 ` Gilles Chanteperdrix
2011-04-08 6:16 ` Gilles Chanteperdrix [this message]
2011-04-08 6:29 ` Richard Cochran
2011-04-08 6:35 ` Gilles Chanteperdrix
2011-04-08 6:46 ` Richard Cochran
2011-04-08 6:35 ` Gilles Chanteperdrix
2011-04-08 6:51 ` Richard Cochran
2011-04-08 15:37 ` Gilles Chanteperdrix
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4D9EA831.3020109@domain.hid \
--to=gilles.chanteperdrix@xenomai.org \
--cc=Xenomai-core@domain.hid \
--cc=adeos-main@gna.org \
--cc=richardcochran@domain.hid \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.