Re: ANN: cmpci 6.64 released

[Jeff Garzik <jgarzik@pobox.com>]

C.L. Tien - 田承禮 wrote:
> Hi,
> 
> I made many changes for cmpci.c since last release. Changes are made as follows:
> 
> 1. Fix the S/PDIF out programming bug appeared in 6.16.
> 2. Support 8338 4-channel playback.
> 3. S/PDIF loop can be used after AC3 playback.
> 4. Legacy devices (FM, MPU401 and gameport) are served by other modules.
>    Now the code is thinner.
> 5. Remove parameters setting from kernel configure menu, they can be
>    set easily when loading the module.
> 6. Add spdif_out to output 44.1K/48K 16-bit stereo to S/PDIF out.
> 7. Add hw_copy to duplicate audio of front speakers to surround speakers.
> 8. Change code to minimum patch for kernel 2.6.

The driver looks pretty good, for both 2.4 and 2.6.  And the listed
changes above sound OK.  I see the driver is smaller now.

May I respectfully request that you submit your driver update in the
form of multiple patches, one patch per email?  This is the normal
method of submitting changes to the Linux kernel.  For example, create
and send 8 patches to Andrew Morton for inclusion in the 2.6.x kernel.
Each patch is applied after the last one, in succeeding order.  Typical
email subjects would look like

[PATCH 1/8] cmpci: fix s/pdif out
[PATCH 2/8] cmpci: support 8338 4-channel
[PATCH 3/8] cmpci: s/pdif loop
etc.

Splitting up changes in this manner allows for better verification, and
makes it easier to narrow down bugs to a specific change.  Large "one
big patch" updates often solve many bugs, and then add a few new bugs.

Best regards and thanks,

	Jeff