From: Brad Boyer <flar@allandria.com>
To: Finn Thain <fthain@telegraphics.com.au>
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: Fri, 5 Jun 2020 02:11:23 -0700 [thread overview]
Message-ID: <20200605091123.GA29547@allandria.com> (raw)
In-Reply-To: <alpine.LNX.2.22.394.2006051418500.8@nippy.intranet>
On Fri, Jun 05, 2020 at 02:23:50PM +1000, Finn Thain wrote:
> On Fri, 5 Jun 2020, Finn Thain wrote:
> > I think we need to be able to restart the IOP's internal ADB driver.
> > Otherwise it's impossible to have ADB input resume when the floppy
> > drives go idle for a while or when the swim module exits.
> >
> > The alternative is to revive swip_iop.c, and the downside there is that
> > the block layer API has completely changed, meaning that the driver
> > needs a rewrite on the block layer side, as well as needing more work on
> > the IOP protocol side.
> >
>
> I should also mention that swim.c has some pretty serious limitations: no
> write support and no GCR support. These features could be added much more
> easily to swim_iop.c than swim.c because the IOP supports them internally.
With all of that, it sounds like it would be easier to bring back and
fix the swim_iop driver in spite of the need to update the interface
to the kernel. I don't know when I might be able to get my IIfx or Q950
back out and running, but I'll see if I can take a look at the code
even without having the hardware ready to actually test it.
Ideally we would have the code to actually do a full reset on an IOP
and bootstrap the IOP kernel and each IOP driver. I suspect the images
for them are somewhere in the ROM. That's likely only necessary if we
start doing a boot where the MacOS hasn't already loaded fully.
Brad Boyer
flar@allandria.com
next prev parent reply other threads:[~2020-06-05 9:11 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 3/4] m68k/mac: Don't send uninitialized data in IOP message reply 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 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
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 [this message]
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=20200605091123.GA29547@allandria.com \
--to=flar@allandria.com \
--cc=fthain@telegraphics.com.au \
--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;
as well as URLs for NNTP newsgroup(s).