From: Paulo Marques <pmarques@grupopie.com>
To: Paul Mundt <lethal@linux-sh.org>,
Paulo Marques <pmarques@grupopie.com>,
Andrew Morton <akpm@linux-foundation.org>,
Michal Januszewski <spock@gentoo.org>,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH -mm] video: uvesafb: Add X86 dependency.
Date: Tue, 11 Sep 2007 13:19:57 +0100 [thread overview]
Message-ID: <46E687ED.9000802@grupopie.com> (raw)
In-Reply-To: <20070911115346.GA23493@linux-sh.org>
Paul Mundt wrote:
> On Tue, Sep 11, 2007 at 12:41:49PM +0100, Paulo Marques wrote:
>>[...]
>> Why do you say that it's x86 specific? Am I missing something?
>>
> The emulator it uses only runs on x86 and x86_64. Thus, it's x86
> specific. The v86d and uvesafb pages seem to be in disagremeent, unless
> by 'non-x86' it's only implying x86_64.
>
> Additionally, it needs the vga I/O routines, as per vgacon. Most
> platforms don't define these.
I just saw the v86d emulator code, and you're right. It mmaps /dev/mem
and assumes the video BIOS is mapped there.
We should try to change that to mmap something like
"/sys/bus/pci/devices/0000:01:00.0/rom" instead so that it can work on
any system with a video card plugged on a PCI bus.
It should also be possible for the emulator to translate in/out
instructions to appropriate memory mapped I/O equivalents, no?
Anyway, I think it is up to Michal to decide if we should remove the
kernel support for other archs, or let it stay a bit more while working
on solving the v86d side of things. So I'll just step aside now....
--
Paulo Marques - www.grupopie.com
"667: The neighbor of the beast."
next prev parent reply other threads:[~2007-09-11 12:20 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-09-11 8:17 [PATCH -mm] video: uvesafb: Add X86 dependency Paul Mundt
2007-09-11 11:41 ` Paulo Marques
2007-09-11 11:53 ` Paul Mundt
2007-09-11 12:19 ` Paulo Marques [this message]
2007-09-11 12:31 ` Paul Mundt
2007-09-11 22:59 ` Michal Januszewski
2007-09-11 23:09 ` [PATCH -mm] uvesafb: Don't access VGA registers directly when running on non-x86 Michal Januszewski
2007-09-12 0:44 ` Paul Mundt
2007-09-12 19:41 ` Michal Januszewski
2007-09-13 1:43 ` Paul Mundt
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=46E687ED.9000802@grupopie.com \
--to=pmarques@grupopie.com \
--cc=akpm@linux-foundation.org \
--cc=lethal@linux-sh.org \
--cc=linux-kernel@vger.kernel.org \
--cc=spock@gentoo.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.