All of lore.kernel.org
 help / color / mirror / Atom feed
From: Philippe Gerum <rpm@xenomai.org>
To: Richard Cochran <richardcochran@domain.hid>
Cc: adeos-main@gna.org, xenomai@xenomai.org,
	Lennart Sorensen <lsorense@domain.hid>
Subject: Re: [Xenomai-core] [Adeos-main] I-pipe for 2.6.32 PPC
Date: Sun, 17 Jan 2010 23:00:06 +0100	[thread overview]
Message-ID: <1263765606.2428.908.camel@domain.hid> (raw)
In-Reply-To: <20100114075343.GA5179@domain.hid>

On Thu, 2010-01-14 at 08:53 +0100, Richard Cochran wrote: 
> On Tue, Jan 12, 2010 at 05:18:02PM -0500, Lennart Sorensen wrote:
> > Well I have not been able to find the magic invocation that lets me take
> > the DENX tree (which I have had around for a long time just to look at
> > occationally, whenever I was trying to get an ipipe patch to apply),
> > apply the ipipe patch, revert the DENX changes to get back to a release
> > kernel, and generate a diff of the ipipe changes.  It has never worked
> > when I tried.
> 
> Lennart,
> 
> It is not so hard (using git) to remove the Denx patches from the
> ipipe tree. I did this myself for 2.6.30 in about a half an hour. If
> you don't know how to use git, then you would have to consider the
> additional time you need to get to understand it. (For me, it was only
> a year or so ;)
> 
> In the ipipe tree, the Denx commits have been "squashed" together into
> one or two really large commits. So, you can just cherry pick the
> adeos commits into a new branch, with a few minor fixups.
> 
> Philippe,
> 
> I actually agree with Lennart that the Denx stuff is an
> annoyance. When considering my "no-denx" branch that I made, I could
> not see any significant Denx change that adeos builds upon. There were
> a few Denx fixes for one specific board that were close to the adeos
> changes, but these were only a few, and easy to fix. So, I could not
> understand why Denx is a prerequisite for adeos.
> 

As I explained earlier in my reply to Lennart, the DENX tree is not a
pre-requisite for having the pipeline run on each and every hw platform,
but this is still the case for some, because they are not
stable/complete/good enough in mainline yet. The reason to stick with it
stems from this fact.

What has to be reassessed, is the number of platforms Xenomai supports
that still need DENX bits today; if only a few of them remain in this
category, then it's probably sound to start maintaining the pipeline
support for them in a separate tree, rebasing I-pipe mainline over Linux
mainline for ppc as well. I have no issue with that.

> I understand that Denx sponsored the original PowerPC Xenomai port. Is
> the reason that ipipe is based on Denx simply to honor that fact? If
> so, I would not think it a bad reason at all.
> 

No, it's not, because if I had chosen to base the pipeline code for ppc
over mainline at that time, I would have maintained a DENX-based version
for the very reason you mentioned, until it proves useless.

> However, I would still prefer the following ordering for the changes:
> 
> 1. stable linux (2.6.xx.y)
> 2. adeos arch indepedendent
> 3. adeos powerpc
> 4. denx
> 5. adeos for denx (minimal changes, I expect)
> 

What we may be aiming at, if workable, is something like:

ipipe-*-mainline
ipipe-*-amcc
ipipe-*-512x

Maybe one for the PA6T as well, if we want to keep supporting the old A2
board rev. I'm unsure right now, since B0 is fine in mainline already.

A mainline pipeline branch for everything that directly works over
mainline, and platform-specific branches for those that do not. Those
special branches would then disappear as soon as mainline is fine for
the platforms they host as well.

> Richard
> 
> _______________________________________________
> Adeos-main mailing list
> Adeos-main@domain.hid
> https://mail.gna.org/listinfo/adeos-main


-- 
Philippe.







  reply	other threads:[~2010-01-17 22:00 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-12-10  6:56 [Adeos-main] I-pipe for 2.6.32 PPC Bernhard Pfund
2009-12-12 10:13 ` Philippe Gerum
2009-12-12 15:09   ` bernhard
2010-01-08 10:59 ` Richard Cochran
2010-01-08 11:10   ` Philippe Gerum
2010-01-08 15:13     ` Bernhard Pfund
2010-01-12 16:03     ` [Xenomai-core] " Philippe Gerum
     [not found]       ` <20100112172443.GP8605@domain.hid>
2010-01-12 17:49         ` Philippe Gerum
     [not found]           ` <20100112185012.GQ8605@domain.hid>
2010-01-12 19:23             ` Philippe Gerum
     [not found]               ` <20100112221802.GR8605@domain.hid>
2010-01-14  7:53                 ` Richard Cochran
2010-01-17 22:00                   ` Philippe Gerum [this message]
2010-01-17 22:51                     ` Wolfgang Denk
2010-01-20 22:27                       ` Philippe Gerum
     [not found]                     ` <20100118175113.GF24305@domain.hid>
2010-01-20 22:42                       ` Philippe Gerum
2010-01-15 15:03                 ` Philippe Gerum
2010-01-15 15:33                   ` Wolfgang Grandegger
2010-01-15 17:28                     ` Philippe Gerum
2010-01-15 20:08                       ` Wolfgang Grandegger
     [not found]                   ` <20100115161415.GA24305@domain.hid>
2010-01-15 17:22                     ` Philippe Gerum
2010-02-23  7:53                       ` Richard Cochran
2010-02-23 10:07                         ` Philippe Gerum
2010-02-23 10:57                           ` Richard Cochran
2010-02-25 16:48                             ` Philippe Gerum
2010-03-19 10:20                               ` Richard Cochran

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=1263765606.2428.908.camel@domain.hid \
    --to=rpm@xenomai.org \
    --cc=adeos-main@gna.org \
    --cc=lsorense@domain.hid \
    --cc=richardcochran@domain.hid \
    --cc=xenomai@xenomai.org \
    /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.