All of lore.kernel.org
 help / color / mirror / Atom feed
From: Richard Purdie <rpurdie@rpsys.net>
To: openembedded-devel@openembedded.org
Subject: Re: [RFC] Tosa patches for 2.6.24
Date: Mon, 28 Jan 2008 00:37:47 +0000	[thread overview]
Message-ID: <1201480667.4670.16.camel@localhost.localdomain> (raw)
In-Reply-To: <fnj683$oaa$5@ger.gmane.org>

Hi,

On Mon, 2008-01-28 at 00:05 +0000, Dmitry Baryshkov wrote:
> As some of you know, currently I work on improving the Sharp SL-6000 (tosa)
> kernel support. Some parts are going to be merged RSN, some parts are still
> in preparation to submission. A pile of patches can't be submitted because
> of backwards dependencies. Currently my patchset contains 54 patches.
> 
> Now back to support of tosa in OE. I think I have several options:
> 
> 0) linux-rp-2.6.23 contains support for the device. It has problems, but it's
>    mostly working. So leave it just for now, don't touch 2.6.24 and wait for
>    2.6.25.
>    I would prefer not to follow this path as I want to enlarge testing base
>    for patchset and to let tosa-owners use decent kernel features.
> 1) Add all my patches on the top of linux-rp-2.6.24 as the tosa-specific part.
>    This can be hard, as I touch various bits of kernel, this can require
>    major efforts.
> 2) Just start linux-tosa recipe branch, importing few patches from linux-rp,
>    and merge them later.
>    No major pros et contras.

I also prefer 1 or 2. I would like to explain some of the history or
tosa support in linux-rp just so you know why its as it is and how we
could change it.

Currently most of the tosa patches are applied as machine specific. This
was because they contained functionality which broke the other zaurus
machines and the patches were the process of being redesigned so they
didn't do this.

If the patches don't break the other machines, then we're free to apply
them for all machines and they can be applied much earlier in the stack
if they're imminent for upstream.

The advantage of 1 is that the changes get tested on the other zaurus
models and it makes sure there are no adverse affects there. It also has
the advantage you get the benefit of the debugging done on the other
zaurus models, invariably something breaks upstream affecting PXA and
gets fixed in that tree, if you share it, you get that benefit. The
disadvantage as you say is the work can be slightly harder but we can
reorder patches to help with that if appropriate. 

So I have a slight preference for 1 but don't object to 2 if you feel
thats the best way to go.

Regards,

Richard




  reply	other threads:[~2008-01-28  0:37 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-01-28  0:05 [RFC] Tosa patches for 2.6.24 Dmitry Baryshkov
2008-01-28  0:37 ` Richard Purdie [this message]
2008-01-28  7:22   ` Marcin Juszkiewicz
2008-01-28 13:25     ` Dmitry Baryshkov
2008-01-28 13:37       ` Marcin Juszkiewicz
2008-01-30 14:00         ` Dmitry Baryshkov
2008-01-28 23:28       ` Paul Sokolovsky

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=1201480667.4670.16.camel@localhost.localdomain \
    --to=rpurdie@rpsys.net \
    --cc=openembedded-devel@lists.openembedded.org \
    --cc=openembedded-devel@openembedded.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.