All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jan Kiszka <jan.kiszka@domain.hid>
To: Philippe Gerum <rpm@xenomai.org>
Cc: Xenomai core <Xenomai-core@domain.hid>,
	Wolfgang Grandegger <wg@domain.hid>
Subject: Re: [Xenomai-core] [forge] irqbench removal
Date: Thu, 14 Oct 2010 19:55:06 +0200	[thread overview]
Message-ID: <4CB743FA.20504@domain.hid> (raw)
In-Reply-To: <1287073013.1628.25.camel@domain.hid>

[-- Attachment #1: Type: text/plain, Size: 1650 bytes --]

Am 14.10.2010 18:16, Philippe Gerum wrote:
> On Thu, 2010-10-14 at 18:10 +0200, Jan Kiszka wrote:
>> Am 14.10.2010 17:42, Philippe Gerum wrote:
>>> On Sat, 2010-10-09 at 15:23 +0200, Jan Kiszka wrote:
>>>> Philippe,
>>>>
>>>> irqbench does not inherently depend on a third I-pipe domain. It is a
>>>> useful testcase, the only in our portfolio that targets a peripheral
>>>> device use case. In fact, it was only of the first test cases for Native
>>>> RTDM IIRC.
>>>>
>>>> Please revert the removal and then cut out only the few parts that
>>>> actually instantiate an additional domain (i.e. mode 3.
>>>
>>> So, what do we do with this? Any chance we move to arch-neutral code for
>>> this test?
>>
>> Arch-neutral is impossible due to the inherent hardware dependency. But
>> I'm waiting on some comments by Wolfgang on their work as that's
>> probably the best requirements source for multi-arch support.
> 
> I mean that the bulk of the code could be made arch-neutral, with only
> callouts to solve the arch-dependent/uart issues. Typically, 16550's are
> not uncommon on powerpc, but we obviously don't program them via
> ioports. A second level of indirection could provide the entire chip
> handling, to fit other uarts, maybe?

If there are suitable UARTs around, refactoring the code accordingly and
maybe adding support for one of them as reference would be a good next
step. But first I would like to understand (or recall - I think Wolfgang
once explained it) the motivations for not going this path with the
gpiobench test and learn its requirements to avoid doing refactorings
twice or more.

Jan


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 259 bytes --]

  reply	other threads:[~2010-10-14 17:55 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-10-09 13:23 [Xenomai-core] [forge] irqbench removal Jan Kiszka
2010-10-09 13:29 ` Gilles Chanteperdrix
2010-10-09 13:30   ` Jan Kiszka
2010-10-09 13:35     ` Gilles Chanteperdrix
2010-10-09 14:25       ` Jan Kiszka
2010-10-14 15:42 ` Philippe Gerum
2010-10-14 16:10   ` Jan Kiszka
2010-10-14 16:16     ` Philippe Gerum
2010-10-14 17:55       ` Jan Kiszka [this message]
     [not found]         ` <4CB7482C.9060602@domain.hid>
2010-10-14 19:03           ` Jan Kiszka
     [not found]             ` <4CB75976.6040302@domain.hid>
2010-10-15  6:39               ` Jan Kiszka
2010-10-15 11:07                 ` Wolfgang Grandegger
2010-10-15 12:47                   ` Wolfgang Grandegger
2010-10-15 12:59                     ` Jan Kiszka

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=4CB743FA.20504@domain.hid \
    --to=jan.kiszka@domain.hid \
    --cc=Xenomai-core@domain.hid \
    --cc=rpm@xenomai.org \
    --cc=wg@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.