From: Stefan Richter <stefanr@s5r6.in-berlin.de>
To: Jonathan Woithe <jwoithe@physics.adelaide.edu.au>
Cc: linux1394-devel@lists.sourceforge.net, olh@suse.de,
akpm@linux-foundation.org, torvalds@linux-foundation.org,
linux-kernel@vger.kernel.org, krh@redhat.com
Subject: Re: [git pull] New firewire stack
Date: Thu, 03 May 2007 10:22:54 +0200 [thread overview]
Message-ID: <46399BDE.4040605@s5r6.in-berlin.de> (raw)
In-Reply-To: <200705030004.l4304l8h012587@turbo.physics.adelaide.edu.au>
Jonathan Woithe wrote:
>> Olaf Hering wrote:
>>> NACK.
>>> Upgrade the current drivers/ieee1394/ with the new code, and keep all
>>> existing module names.
[...]
> However, as a compromise how about renaming the existing stack's modules and
> then reusing the existing names for the new stack? Messy I know, but this
> way both stacks would still be available without recompilation for those who
> needed them and the sbp2-as-root dilemma raised by Olaf would also be
> covered.
I.e. new modules:
ieee1394 (was fw-core)
ohci1394 (was fw-ohci)
sbp2 (was fw-sbp2)
eth1394 (to be done) --- but that's a bad name anyway, it
implements IP over 1394, not Ethernet
old modules, for example:
ieee1394-old
ohci1394-old
sbp2-old
eth1394-old
pcilynx (unless somebody plans to port it, then pcilynx-old)
raw1394 (or rename it too?, it requires ieee1394-old)
video1394 (ditto)
dv1394 (ditto)
Looks... weird.
On the other hand, a 1394 module compilation cycle in order to do the
fallback is not such a huge issue, except that it requires the person to
be able to compile modules. That's probably the main issue.
>> As for putting the new stack in drivers/ieee1394 - I don't know, I think it
>> makes sense to keep the new stack in it's own directory.
>
> My immediate thought it that it would be neater and clearer to have both
> stacks in different directories, but I could live with either.
We could rename some of the existing source files to make the
distinction clearer.
> Oh yes, it would be nice to have working PCILynx support again (although I
> acknowledge it's unlikely to happen). Some of us do have these cards
> installed for sniffing purposes (using nosy) but it would be nice to be able
> to use them with libraw1394 as well. It would for example save me having to
> swap cards depending on what I needed to do (I have insufficient PCI slots
> to have both the PCILynx and OHCI cards installed simultaneously).
But then, what is the actual utility of pcilynx? (I mean the current
driver, not the card or a future driver.) Last time I checked, sbp2 was
broken without OHCI's physical DMA, and AFAIK raw1394's newer iso API
and video1394 and dv1394 don't work with pcilynx either.
Porting pcilynx to the new low-level API would be quite resource
demanding --- seen in relation to which resources we have, what the
existing pcilynx driver's state of affairs is, and how rare the hardware
is. (For those who have the hardware, the stand-alone Nosy is
undoubtedly the killer application, not pcilynx.)
--
Stefan Richter
-=====-=-=== -=-= ---==
http://arcgraph.de/sr/
next prev parent reply other threads:[~2007-05-03 8:23 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <mailman.197656.1178135675.32383.linux1394-devel@lists.sourceforge.net>
2007-05-03 0:04 ` Jonathan Woithe
2007-05-03 8:22 ` Stefan Richter [this message]
2007-05-03 11:48 ` Olaf Hering
2007-05-03 13:30 ` Stefan Richter
2007-05-03 16:34 ` Adrian Bunk
2007-05-03 17:33 ` Kristian Høgsberg
2007-05-04 5:54 ` Bill Fink
2007-05-05 21:17 ` Olaf Hering
2007-05-08 0:28 ` Kristian Høgsberg
2007-05-03 23:07 ` Jonathan Woithe
2007-05-01 20:27 Kristian Høgsberg
2007-05-01 21:34 ` Stefan Richter
2007-05-02 9:00 ` Christoph Hellwig
2007-05-02 12:13 ` Stefan Richter
2007-05-02 20:00 ` Kristian Høgsberg
2007-05-02 12:21 ` Olaf Hering
2007-05-02 12:48 ` Stefan Richter
2007-05-02 13:56 ` Gene Heskett
2007-05-02 18:51 ` Stefan Richter
2007-05-02 15:27 ` Adrian Bunk
2007-05-02 20:03 ` Kristian Høgsberg
2007-05-02 19:53 ` Kristian Høgsberg
2007-05-02 20:03 ` Olaf Hering
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=46399BDE.4040605@s5r6.in-berlin.de \
--to=stefanr@s5r6.in-berlin.de \
--cc=akpm@linux-foundation.org \
--cc=jwoithe@physics.adelaide.edu.au \
--cc=krh@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux1394-devel@lists.sourceforge.net \
--cc=olh@suse.de \
--cc=torvalds@linux-foundation.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®