From: Sudarsana Reddy Kalluru <sudarsana.kall...@cavium.com> Date: Mon, 21 May 2018 03:40:12 -0700
> +struct qed_tlv_parsed_buf { > + /* To be filled with the address to set in Value field */ > + u8 *p_val; Assignments to this pointer are cast hundreds of times, like: > + p_buf->p_val = (u8 *)&p_drv_buf->rx_frames; Over and over again. Just make p_val a void pointer, and then you don't need any of the casts at all.