From: "Antonino A. Daplas" <adaplas@hotpop.com>
To: linux-fbdev-devel@lists.sourceforge.net,
Richard Purdie <rpurdie@rpsys.net>
Subject: Re: w100 fb driver
Date: Sat, 20 Nov 2004 06:10:08 +0800 [thread overview]
Message-ID: <200411200610.08802.adaplas@hotpop.com> (raw)
In-Reply-To: <021601c4ce58$1111a730$0f01a8c0@max>
On Saturday 20 November 2004 00:51, Richard Purdie wrote:
> I've been working on creating support for the Sharp SL-C7xx series as part
> of a wider effort to create support for the whole Zaurus range of PDAs
> under linux 2.6. The C7xx's contain the ATI w100 video chip which doesn't
> have mainstream kernel support at present.
>
> Sharp did release a driver against 2.4.18 and I've ported this across to
> the 2.6 series. I've also made some fairly large changes to the driver to
> tidy it up and create support for certain video modes Sharp never enabled.
>
> I'd like to discuss whether it'd be possible to include this driver in a
> future kernel? The driver is available at:
> http://www.rpsys.net/openzaurus/w100-r1.patch .
>
> I'm aware the corgi_backlight file included above is out of place and it
> will be moved into the arch/arm/mach-pxa folder. I'm also aware some fb
> changes made in 2.6.10-rc2 need to be applied to this file. Are there any
Not much, but you may need to insert this before initialization:
if (fb_get_options("w100fb", NULL))
return -ENODEV;
> other problems with the driver as it stands that would prevent its
> inclusion, or areas in the driver that it would be advisable to
> change/improve?
Should be okay, just wondering why not use the kernel i2c algo's instead of
using your own.
Can't comment on backlight support as I don't know a thing about that.
Preferrably, comments should be enclosed in /* */, rather than //
>
> If the driver would be acceptable, what method of submission is preferred?
> (the sourceforge pages seem inactive)
Submit patch inline to this list and CC me. Preferrably break the patch
down into several, hopefully standalone patches. Add a changelog, and a
signed-off line.
Referring you to this link on patch submission:
http://www.zip.com.au/~akpm/linux/patches/stuff/tpp.txt
Tony
-------------------------------------------------------
This SF.Net email is sponsored by: InterSystems CACHE
FREE OODBMS DOWNLOAD - A multidimensional database that combines
robust object and relational technologies, making it a perfect match
for Java, C++,COM, XML, ODBC and JDBC. www.intersystems.com/match8
next prev parent reply other threads:[~2004-11-19 22:10 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-11-19 16:51 w100 fb driver Richard Purdie
2004-11-19 22:10 ` Antonino A. Daplas [this message]
2004-11-25 15:19 ` Richard Purdie
2004-11-25 19:55 ` Antonino A. Daplas
2004-11-20 9:45 ` 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=200411200610.08802.adaplas@hotpop.com \
--to=adaplas@hotpop.com \
--cc=linux-fbdev-devel@lists.sourceforge.net \
--cc=rpurdie@rpsys.net \
/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.