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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox