From: Brad Boyer <flar@allandria.com>
To: Laurent Vivier <Laurent@lvivier.info>
Cc: Geert Uytterhoeven <geert@linux-m68k.org>, linux-m68k@vger.kernel.org
Subject: Re: [PATCH] Add SWIM floppy support for m68k Macs.
Date: Sun, 2 Nov 2008 13:28:52 -0800 [thread overview]
Message-ID: <20081102212852.GA8162@cynthia.pants.nu> (raw)
In-Reply-To: <AC634DD2-CE12-4975-BB8E-B564C7DFDC06@lvivier.info>
On Sun, Nov 02, 2008 at 12:21:23PM +0100, Laurent Vivier wrote:
> Le 2 nov. 08 à 09:54, Geert Uytterhoeven a écrit :
> > The preferred way to do this these days is to create a platform
> > device with a
> > struct resource that points to the SWIMBase.
>
> Is this what Finn explains also in his answer ?
> If so, I agree.
Not exactly. What Finn suggested is to match the way we handle
many of the other on-board things like ADB and SCSI. What Geert
suggests is a much more recent way to do things that we generally
haven't done for Mac-only drivers. I think Finn used it for the
new Sonic ethernet driver because we share it with other systems.
The code in macsonic.c also has the extra complexity from having
both NuBus and on-board hardware to support, and NuBus also hasn't
been updated to the new driver model.
Finn's suggestion of using the macintosh_config entry and data
table is the easiest way to get it working, but Geert's method
would move to the real device/driver model that most other
systems are using now. Ideally this would be a macio driver
instead of a platform driver in my opinion, but I haven't finished
getting the macio bus code working on m68k.
Brad Boyer
flar@allandria.com
--
To unsubscribe from this list: send the line "unsubscribe linux-m68k" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2008-11-02 21:29 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
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 [this message]
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=20081102212852.GA8162@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