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.

Reply via email to