The subject is too vague and you need a changelog.On Tue, Aug 01, 2017 at 05:11:29PM +0100, Ashish Kalra wrote:> +static int inc_txqhead(struct ks_wlan_private *priv)> +{> + priv->tx_dev.qhead = (priv->tx_dev.qhead + 1) % TX_DEVICE_BUFF_SIZE;> + return 0;Just make these void if no one checks the return.regards,dan carpenter