[lkml]   [2004]   [Mar]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: ANN: cmpci 6.64 released
    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

    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,


    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2005-03-22 14:01    [W:0.022 / U:18.624 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site