From: "Dale Farnsworth" <dale@farnsworth.org>
To: jcrigby@gmail.com, Linuxppc-embedded <Linuxppc-embedded@ozlabs.org>
Subject: Re: Kernel for MPC Lite 5200 will not compile
Date: 30 Mar 2006 19:53:09 -0000 [thread overview]
Message-ID: <20060330195309.10006.qmail@farnsworth.org> (raw)
In-Reply-To: <4b73d43f0603300829q7d45a029qdb5d8006ccda7275@mail.gmail.com>
John Rigby <jcrigby@gmail.com> wrote:
> Sent this to Wolfgang last night. Accidently omitted Syvain and the list.
>
> The actual bestcomm "microcode" in Sylvain's tree is the same as the latest
> Freescale supported code. The linux driver code that interfaces with it is
> code written by Dale Farnsworth and has Montavista copyright. I don't know
> if it is based on old Freescale (Motorola) code or if it is Dale's creation
> (perhaps Dale can comment).
My bestcomm code is not based on the Freescale code. I did have access to
the Freescale (Motorola) code when I wrote it, but I think you'll find that
there is *very little* similarity between the two code bases.
> I have just spent some time getting UDMA support into the ATA driver using
> the the bestcomm code in Sylvain's tree. In the process I found some bugs
> in the code that may explain some problems people have seen with the FEC
> driver. I'll try to get the patch out tomorrow.
>
> The ATA driver is a port from a version that worked on an older kernel and
> used the supported Freescale Bestcomm code. I did the port because it was
> my understanding that attempts to get the supported Freescale code into a
> public tree had met with resistance in the past. (Second hand info, I'm
> actually fairly new to the 52xx world).
>
> The pluses for the Freescale version is that several devices are supported
> that are not in Sylvain's tree. The version in Sylvain's kernel has a
> generic api plus specific code for FEC. My upcoming patch fixes some bugs,
> adds some missing functionality that I need for ATA and adds the specific
> ATA dma driver code.
>
> The biggest minus for the Freescale version is that it is really opaque.
> Its implementation makes it hard to figure out what the code is actually
> doing.
Yes, this makes it difficult to maintain and is what has kept the
Freescale Bestcomm code from being accepted into the kernel. I would
have preferred not to write an alternate version, but, at the time,
Motorola had a take-or-leave-it stance; they wouldn't consider accepting
patches to clean it up. So then I planned to simply fork their version,
but found the code so convoluted that it was easier to start from scratch.
> If there is a chance of getting the Freescale Bestcomm code into Sylvain's
> tree (and on up the line to Linus) I would welcome that since I would not
> have to port other Bestcomm dependent drivers to the version in Sylvain's
> tree. I would gladly port the current FEC code to the Freescale Bestcomm
> code to avoid the work of porting the other drivers. Also we at Freescale
> would prefer to have one version of the Bestcomm code circulating.
>
> So I guess my question for those with an opinion is what version of Bestcomm
> should be we use for 5200.
It would be better to have one version, and my bestcomm code could be
replaced in Linux by code derived from the current Freescale code. The
challenges are political. Neither Freescale nor the Linux community is
willing to relinquish control of their code to the other. Potentially,
this can be addressed by having a Freescaler, who is responsive to the
Linux community, be a Linux maintainer for the bestcomm code. There are
also licensing concerns though, since Freescale needs to also license
their code under non-GPL terms.
Until these challenges are resolved, I think we're stuck with two versions.
Thanks,
-Dale
prev parent reply other threads:[~2006-03-30 19:53 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-03-28 0:07 Kernel for MPC Lite 5200 will not compile Matthias Fechner
2006-03-28 1:01 ` John Rigby
2006-03-28 9:49 ` Matthias Fechner
2006-03-28 16:15 ` John Rigby
2006-03-28 20:47 ` Sylvain Munaut
2006-03-29 15:32 ` Matthias Fechner
2006-04-01 9:19 ` Matthias Fechner
2006-04-02 17:42 ` Sylvain Munaut
2006-04-02 19:32 ` Matthias Fechner
2006-03-28 22:33 ` Wolfgang Denk
2006-03-29 15:31 ` Matthias Fechner
2006-03-28 22:31 ` Wolfgang Denk
2006-03-29 15:39 ` Matthias Fechner
2006-03-29 20:11 ` Wolfgang Denk
2006-03-29 23:02 ` Sylvain Munaut
2006-03-30 0:34 ` Wolfgang Denk
2006-03-30 6:36 ` Sylvain Munaut
2006-03-30 16:29 ` John Rigby
2006-03-30 16:39 ` John Rigby
2006-03-30 20:05 ` Dale Farnsworth
2006-03-30 21:51 ` John Rigby
2006-03-30 19:53 ` Dale Farnsworth [this message]
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=20060330195309.10006.qmail@farnsworth.org \
--to=dale@farnsworth.org \
--cc=Linuxppc-embedded@ozlabs.org \
--cc=jcrigby@gmail.com \
/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.