On Mon, Jul 17, 2023 at 12:06:38PM +0800, Matt Johnston wrote:
...
Hi Matt,
> +/* Returns the 48 bit Provisioned Id from an i3c_device_info.pid */
> +static void pid_to_addr(u64 pid, u8 addr[PID_SIZE])
> +{
> + pid = cpu_to_be64(pid);
This assigns a be64 value to a u64 variable,
which is incorrect from a Sparse annotation perspective.
> + memcpy(addr, ((u8 *)&pid) + 2, PID_SIZE);
> +}
...
