[lkml]   [2005]   [Sep]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] New: Omnikey CardMan 4040 PCMCIA Driver
    > Below you can find a driver for the Omnikey CardMan 4040 PCMCIA
    > Smartcard Reader.

    Someone correct me if I'm wrong, but wouldn't these #defines be a problem with
    the new HZ flexibility:

    #define READ_WRITE_BUFFER_SIZE 512
    #define POLL_LOOP_COUNT 1000

    /* how often to poll for fifo status change */
    #define POLL_PERIOD (HZ/100)

    In particular, 2.6.13 allows a HZ of 100, which would define POLL_PERIOD to 0.
    Your later calls to mod_timer would be setting cmx_poll_timer to the current
    value of jiffies.

    Also, you've got a typo in the comments:

    * - adhere to linux kenrel coding style and policies

    Forgive me if I'm way off - I'm just now getting my feet wet in kernel
    development. Just making comments based on what I (think) I know at this

    Best Regards,
    Chase Venters
    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-09-03 23:29    [W:0.029 / U:4.784 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site