A few things that I found while looking at the CCID2 code.

Patch #1: Is a revised version of `allocation in atomic context' fix
          first submitted on 
http://www.mail-archive.com/[email protected]/msg01941.html
          I found that there is an implicit assumption on the length of the 
pre-allocated
          list, which means that the allocation is always the same, which leads 
to further
          simplification.

Patch #2: Removes an ugly BUG_ON, spotted by Arnaldo, which directly hangs on 
patch #1.

Patch #3: Removes another BUG_ON, which was found to be redundant.

Patch #4: An entire switch/case block can go - dead code.

Patch #5: Sequence number wraparound issues - updated to use the newer seqno 
functions.

Patch #6: Spotted an instance of the DIV_ROUND_UP() macro and used it.

Patch #7: follows48() is nowhere used in the DCCP code and thus removed.

Patch #8: Removes redundant initialisation of "memset + assignment of zero". 
This was also
          spotted by Arnaldo - thanks again.
-
To unsubscribe from this list: send the line "unsubscribe dccp" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to