Hi Gerald,
> This mimicks what we already do a few lines above for buf_type.
It's also unneeded:
switch(ioctl & 0x18000000)
There are only 4 possible combinations of ioctl & 0x18000000. Three
are covered in this switch statement, and the fourth is eliminated due
to the outer switch statement. You'd be adding dead code.
--Juan