From: Finn Thain <fthain@telegraphics.com.au>
To: Brad Boyer <flar@allandria.com>
Cc: Geert Uytterhoeven <geert@linux-m68k.org>,
Joshua Thompson <funaho@jurai.org>,
linux-m68k <linux-m68k@lists.linux-m68k.org>
Subject: Re: [PATCH 0/4] Mac IOP driver fixes
Date: Tue, 2 Jun 2020 09:32:29 +1000 (AEST) [thread overview]
Message-ID: <alpine.LNX.2.22.394.2006020922570.8@nippy.intranet> (raw)
In-Reply-To: <20200601060915.GA8450@allandria.com>
On Sun, 31 May 2020, Brad Boyer wrote:
>
> The swim_iop driver was never completed. As far as I know, the data
> transfer code was never written. I remember testing the driver, and all
> it could do was detect the drives and do eject. Read and write were not
> working. I looked into it at the time, but it's a little messy and is
> much more complicated than the ADB driver. For ADB, the data is small
> enough that it's packed in the IOP message. That's not possible for
> SWIM, so it has a more complex data handling method. If you look at the
> code of that driver and the messages, it looks like the IOP can do DMA
> but it can't. The SWIM IOP message includes a buffer pointer, but the
> IOP chip isn't connected in a way that it can just access it (and it's
> getting passed a virtual address anyway). Based on some of the other
> information I was given, there's a separate message protocol on channel
> 0 (on each IOP) for bulk data transfer.
>
> At the time, I had a diff to fix some bugs in the detection and handle
> the status update messages. However, it hardly seemed worth submitting
> that as a fix since the driver still wasn't useful without the bulk data
> transfer working. I can pass my diff along if you intend to make an
> attempt at a working swim_iop driver.
>
I'm hoping that the SWIM IOP can be used in bypass mode, so it could be
used with the swim.c driver. I haven't been able to make that work yet.
You may want to send the patch to the mailing list anyway. Or maybe send
it to the linux-mac68k bug tracker on sourceforge.
next prev parent reply other threads:[~2020-06-01 23:32 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-05-30 23:12 [PATCH 0/4] Mac IOP driver fixes Finn Thain
2020-05-30 23:12 ` [PATCH 1/4] m68k/mac: Don't send IOP message until channel is idle Finn Thain
2020-05-30 23:12 ` [PATCH 2/4] m68k/mac: Fix IOP status/control register writes Finn Thain
2020-05-30 23:12 ` [PATCH 3/4] m68k/mac: Don't send uninitialized data in IOP message reply Finn Thain
2020-05-30 23:12 ` [PATCH 4/4] m68k/mac: Improve IOP debug messages Finn Thain
2020-05-31 8:41 ` [PATCH 0/4] Mac IOP driver fixes Geert Uytterhoeven
2020-06-01 0:05 ` Finn Thain
2020-06-01 6:09 ` Brad Boyer
2020-06-01 23:32 ` Finn Thain [this message]
2020-06-02 2:21 ` Brad Boyer
2020-06-02 3:48 ` Finn Thain
2020-06-04 3:19 ` Brad Boyer
2020-06-04 4:49 ` Finn Thain
2020-06-04 7:43 ` Brad Boyer
2020-06-05 3:50 ` Finn Thain
2020-06-05 4:23 ` Finn Thain
2020-06-05 9:11 ` Brad Boyer
2020-06-29 21:39 ` Geert Uytterhoeven
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=alpine.LNX.2.22.394.2006020922570.8@nippy.intranet \
--to=fthain@telegraphics.com.au \
--cc=flar@allandria.com \
--cc=funaho@jurai.org \
--cc=geert@linux-m68k.org \
--cc=linux-m68k@lists.linux-m68k.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