2001-07-28 21:21:16

by Steven Walter

[permalink] [raw]
Subject: [PATCH] Port tdfxfb to new-style PCI API

I have created a patch that changes the 3dfx framebuffer driver so that
it uses the new-style PCI api. Additionally, it adds the ability to
pass parameters to the module (previously these were only availible when
built into the kernel) and makes the indention conformant to
Coding-Style.

I've tested it myself as both module and built-in with no problems, but
you can never test too much. I'd like to ask adventuresome users of
this driver to try out my patch, with the hopeful end result of
inclusion into the kernel.

The patch is availible from:
http://www.apex.net/users/trwalter/tdfxfb-patch.gz
Its 22k compressed (large because of style/indention changes), so I was
hesitant to post it to the list.

Many thanks in advance to testers, comments are welcome.
--
-Steven
In a time of universal deceit, telling the truth is a revolutionary act.
-- George Orwell


2001-07-30 01:05:37

by Paul Mundt

[permalink] [raw]
Subject: Re: [PATCH] Port tdfxfb to new-style PCI API

On Sat, Jul 28, 2001 at 04:21:17PM -0500, Steven Walter wrote:
> I have created a patch that changes the 3dfx framebuffer driver so that
> it uses the new-style PCI api. Additionally, it adds the ability to
> pass parameters to the module (previously these were only availible when
> built into the kernel) and makes the indention conformant to
> Coding-Style.
>
> I've tested it myself as both module and built-in with no problems, but
> you can never test too much. I'd like to ask adventuresome users of
> this driver to try out my patch, with the hopeful end result of
> inclusion into the kernel.
>
> The patch is availible from:
> http://www.apex.net/users/trwalter/tdfxfb-patch.gz
> Its 22k compressed (large because of style/indention changes), so I was
> hesitant to post it to the list.
>
Looks good for the most part, but maybe we could do without the excessive
white space changes?

How about something more like the attached patch?

Regards,

--
Paul Mundt <[email protected]>


Attachments:
(No filename) (0.98 kB)
tdfxfb-pci.diff (12.88 kB)
Download all attachments

2001-07-30 01:24:53

by Steven Walter

[permalink] [raw]
Subject: Re: [PATCH] Port tdfxfb to new-style PCI API

On Sun, Jul 29, 2001 at 06:04:49PM -0700, Paul Mundt wrote:
> On Sat, Jul 28, 2001 at 04:21:17PM -0500, Steven Walter wrote:
> > I have created a patch that changes the 3dfx framebuffer driver so that
> > it uses the new-style PCI api. Additionally, it adds the ability to
> > pass parameters to the module (previously these were only availible when
> > built into the kernel) and makes the indention conformant to
> > Coding-Style.
> >
> > I've tested it myself as both module and built-in with no problems, but
> > you can never test too much. I'd like to ask adventuresome users of
> > this driver to try out my patch, with the hopeful end result of
> > inclusion into the kernel.
> >
> > The patch is availible from:
> > http://www.apex.net/users/trwalter/tdfxfb-patch.gz
> > Its 22k compressed (large because of style/indention changes), so I was
> > hesitant to post it to the list.
> >
> Looks good for the most part, but maybe we could do without the excessive
> white space changes?
>
> How about something more like the attached patch?

I'm not married to the indention changes. Your patch is fine with me
in so far as new PCI init goes. One thing my patch did that yours
doesn't is that it added support for module parameters.

I suppose I can submit just that in a seperate, incremental patch.
--
-Steven
In a time of universal deceit, telling the truth is a revolutionary act.
-- George Orwell