public inbox for linux-m68k@lists.linux-m68k.org
 help / color / mirror / Atom feed
From: Brad Boyer <flar@allandria.com>
To: Laurent@lvivier.info
Cc: geert@linux-m68k.org, linux-m68k@vger.kernel.org
Subject: Re: [PATCH] Add SWIM floppy support for m68k Macs.
Date: Sat, 1 Nov 2008 11:10:13 -0700	[thread overview]
Message-ID: <20081101181013.GA25819@cynthia.pants.nu> (raw)
In-Reply-To: <12255325912043-git-send-email-Laurent@lvivier.info>

On Sat, Nov 01, 2008 at 10:43:11AM +0100, Laurent@lvivier.info wrote:
> It allows to read data from a floppy, but not to write to, and to eject the
> floppy (useful on our Mac without eject button).
> 
> I have tested it on a Quadra 800, but it is supposed to work on:
> 
> II, IIci, IIsi, IIvx, IIx, IIcx, SE/30,
> PowerBook 100, PowerBook 140, PowerBook 145, PowerBook 160,
> PowerBook 165, PowerBook 165c, PowerBook 170, PowerBook 180,
> PowerBook 180c, PowerBook 190, PowerBook 190cs, PowerBook 500,
> Performa 460, Performa 550, LC II, LC III, LC 520, Color Classic,
> Color Classic II, ClassicII,
> Quadra 700, Quadra 800, Quadra 650, Quadra 605, Quadra 610,
> Centris 610, Quadra 630, Performa 580, LC 475, LC 575,

I just have one comment and one question. The model check looks like
it will end up printing out a warning message on the AV models. You
should add them explicitly to the unsupported list the way you did
for the IIfx and Q900/Q950. Also, I'm not sure this driver would
work on a Mac II. They didn't originally ship with a SWIM chip at
all, but that was available as an upgrade. From the factory they
came with the old IWM chip. It's not clear to me if the driver uses
any of the SWIM features other than the capability to detect and
read high density media which the old drives obviously don't do.

Other than that, it's good to see someone working on this.

	Brad Boyer
	flar@allandria.com


  reply	other threads:[~2008-11-01 18:10 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-11-01  9:43 [PATCH] Add SWIM floppy support for m68k Macs Laurent
2008-11-01 18:10 ` Brad Boyer [this message]
2008-11-02  0:29   ` Finn Thain
2008-11-02  5:23     ` Brad Boyer
2008-11-02 12:02   ` Laurent Vivier
2008-11-04 18:34     ` Riccardo
2008-11-04 20:06       ` Brad Boyer
2008-11-05  0:58         ` Riccardo
2008-11-05  2:28           ` Brad Boyer
2008-11-05 21:43             ` Riccardo
2008-11-06  4:18               ` Brad Boyer
2008-11-05  6:58           ` Finn Thain
2008-11-05 21:45             ` Riccardo
2008-11-05  7:37         ` Joshua Juran
2008-11-05 10:31           ` Finn Thain
2008-11-05 12:20             ` Joshua Juran
2008-11-05 13:10               ` Finn Thain
2008-11-05 14:32                 ` Laurent Vivier
2008-11-06  4:09                   ` Brad Boyer
2008-11-02  6:00 ` Finn Thain
2008-11-02 11:59   ` Laurent Vivier
2008-11-02  8:54 ` Geert Uytterhoeven
2008-11-02 11:21   ` Laurent Vivier
2008-11-02 21:28     ` Brad Boyer
2008-11-03  0:14       ` Finn Thain
2008-11-03 18:58         ` Laurent Vivier
2008-11-09 16:16         ` Geert Uytterhoeven
2008-11-09 21:47           ` Laurent Vivier
2008-11-09 23:05           ` Michael Schmitz
2008-11-10  0:15           ` Finn Thain
2008-11-11  8:52             ` Geert Uytterhoeven
2008-11-11  9:43               ` Finn Thain
2008-11-11 10:21                 ` 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=20081101181013.GA25819@cynthia.pants.nu \
    --to=flar@allandria.com \
    --cc=Laurent@lvivier.info \
    --cc=geert@linux-m68k.org \
    --cc=linux-m68k@vger.kernel.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