netdev
[Top] [All Lists]

Re: [patches] 2.6.0-test9 - r8169 DMA API conversion

To: Francois Romieu <romieu@xxxxxxxxxxxxx>
Subject: Re: [patches] 2.6.0-test9 - r8169 DMA API conversion
From: Brad House <brad@xxxxxxxx>
Date: Wed, 19 Nov 2003 19:59:30 -0500
Cc: Jeff Garzik <jgarzik@xxxxxxxxx>, Brad House <brad_mssw@xxxxxxxxxx>, netdev@xxxxxxxxxxx
In-reply-to: <20031120014550.A22377@xxxxxxxxxxxxxxxxxxxxxxxxxx>
References: <47973.68.105.173.45.1069042089.squirrel@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <3FB9A277.70309@xxxxxxxxx> <20031118135848.A2451@xxxxxxxxxxxxxxxxxxxxxxxxxx> <3FBBA76B.4070606@xxxxxxxxx> <20031120010056.A19444@xxxxxxxxxxxxxxxxxxxxxxxxxx> <3FBC05FB.2020409@xxxxxxxx> <20031120014550.A22377@xxxxxxxxxxxxxxxxxxxxxxxxxx>
Sender: netdev-bounce@xxxxxxxxxxx
User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.4) Gecko/20030921
any reason why it's not against the v1.6 ?
the current driver is missing support for the
8110S chip (or at least it doesn't seem to work at all),
so you're cutting out all the chips on the mobos out there.
Haven't looked to see if it's just a PCI ID or other functionality,
but since the V1.2 is from realtek that's in the official
kernel, it would make sense to use the 1.6 patch....

I suppose I could do it, but it would take me much
more time simply because I'm not familiar with the
changes you're making.

-Brad

Francois Romieu wrote:
Brad House <brad@xxxxxxxx> :

Just because I'm way to lazy to actually look, is this
against the original r8169, or my patch from the v1.6
driver ??


It is against the original r8169. Once finished, it should look like:
- first cut at dma api conversion (today)
- remaining bits of dma api conversion (tomorrow)
- rx_copybreak (tomorrow)
- your changes (week-end)
- big-endian fixes (week-end)

Each part divided in reasonably small patches to ease the test/review process.

--
Ueimor




<Prev in Thread] Current Thread [Next in Thread>