There are several instances of

    ((sdev->id & 0xFF) >> 8) | (sdev->lun & 0xFF)

or similar.  The left operand of | is always 0.  Did you intend to shift
to the other right?

Reply via email to