From: Nick Piggin <nickpiggin@yahoo.com.au>
To: Ed Schouten <edschouten@gmail.com>
Cc: linux-kernel@vger.kernel.org
Subject: Re: [Patch] i386: Xbox support
Date: Tue, 28 Sep 2004 17:42:40 +1000 [thread overview]
Message-ID: <415915F0.2000803@yahoo.com.au> (raw)
In-Reply-To: <dc54396f040927214651393131@mail.gmail.com>
Ed Schouten wrote:
> Hello Nick,
>
> On Tue, 28 Sep 2004 10:03:39 +1000, Nick Piggin <nickpiggin@yahoo.com.au> wrote:
>
>>Any real point to merging this? (I honestly don't know, I don't follow the
>>xbox hacking scene).
>
>
> Yes, it does (in my opinion). This small 7 KB patch allows you to run
> a vanilla kernel on the machine (with exception of the video driver).
>
> I also noticed my previous mailclient (Squirrelmail) did some
> linebreaking. Please notice:
>
> + if ((bus == 0) && !PCI_SLOT(devfn) && ((PCI_FUNC(devfn) == 1) ||
> (PCI_FUNC(devfn) == 2)))
>
> should be one line ;-)
>
> Yours sincerely,
Well, I ask because there is probably quite a large number of embedded type
devices devices that you could "just add a small patch for" to get it working.
The added fact that you have to "hack" the hardware (I think?) to even get
it to run Linux makes it probably a bit more questionable (it is great that
we can run on xbox, but maybe not too harmful to keep it as an external patch).
Anyway I've otherwise got no objections ;) if you can convince Andrew and/or
Linus to merge it, then fine.
next prev parent reply other threads:[~2004-09-28 7:59 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-09-27 18:02 [Patch] i386: Xbox support Ed Schouten
2004-09-28 0:03 ` Nick Piggin
2004-09-28 2:35 ` Will Dyson
2004-09-28 4:46 ` Ed Schouten
2004-09-28 7:42 ` Nick Piggin [this message]
2004-09-28 8:46 ` Ed Schouten
2004-09-29 18:21 ` Tonnerre
2004-09-29 22:16 ` Ed Schouten
2004-09-28 9:06 ` Pavel Machek
2004-09-28 9:18 ` Nick Piggin
2004-09-28 10:11 ` Switch back to Real mode and then boot strap Aboo Valappil
2004-09-28 10:21 ` Alex Riesen
2004-09-28 10:46 ` Aboo Valappil
2004-09-28 10:57 ` Alex Riesen
2004-09-28 21:26 ` Aboo Valappil
2004-09-28 21:45 ` [Patch] i386: Xbox support Rahul Karnik
2004-09-28 22:06 ` Ed Schouten
2004-09-28 10:11 ` David Weinehall
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=415915F0.2000803@yahoo.com.au \
--to=nickpiggin@yahoo.com.au \
--cc=edschouten@gmail.com \
--cc=linux-kernel@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.