Hi, > this makes me think something is wrong with the data structure. > We should have a be32 there, it seems to me. You mean something like : struct ti_write_data_bytes { u8 bAddrType; u8 bDataType; u8 bDataCounter; __be32 wBaseAddr; u8 bData[0]; } __packed; and, data->wBaseAddr = cpu_to_be32(addr) ? Thanks, Mathieu -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html