All of lore.kernel.org
 help / color / mirror / Atom feed
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.


  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.