> > int_to_scsilun(sp->cmd->device->lun, &cmd_pkt->lun); > + host_to_fcp_swap((uint8_t *)&cmd_pkt->lun, sizeof(cmd_pkt->lun)); this looks rather odd to me. first and minor cmd_pkt->lun now gets values in different endianesses asigned, which doesn't help static typechecking, aka getting the qla2xxx driver sparse clean. Second the host_to_fcp_swap function looks more than fishy to me. It's doing a loop of unconditional byteswaps. that can't be right on BE hardware, can it? - : send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
