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