* Re: patch submission question
[not found] <200405201543.i4KFhHk08317@DYN318364BLD.beaverton.ibm.com>
@ 2004-05-21 20:45 ` Jeff Garzik
0 siblings, 0 replies; only message in thread
From: Jeff Garzik @ 2004-05-21 20:45 UTC (permalink / raw)
To: Don Fry; +Cc: Netdev
Don Fry wrote:
> Jeff,
>
> I have four one-line patches for the pcnet32 driver. Should I send them
> as one patch, or four one-line patches?
>
> 1) Change PCnet/FAST III 79C976 to PCnet/PRO 79C976.
> 2) In pcnet32_interrupt change (delta >= TX_RING_SIZE) to (... > ...)
> 3) add 'static' to pcnet32_get_regs and pcnet32_get_regs_len.
> (prototype has static, routine does not have static).
> 4) reading bcr34 hangs the system with some variants of the chip in
> pcnet2_get_regs.
Sorry I didn't get back to you before you actually sent the patches.
Life is what it is... Also, I hope you don't mind me adding a CC to
netdev@oss.sgi.com. Even for questions like the above, I _really_
encourage you to CC the mailing list, that way all knowledge (including
my preferences) are shared and archived. This is another part of open
development.
The main point is to separate things out into logical changes. For
practical purposes, this usually means I will often lump several minor
changes into a single patch. It is up to you, the maintainer, to decide
where to draw the line.
In your above example, I would send #2 and #4 as separate patches, and
combine #1 and #3 into a single "trivial changes" patch.
Jeff
^ permalink raw reply [flat|nested] only message in thread