From: Jose Abreu <jose.ab...@synopsys.com> Date: Thu, 3 May 2018 13:45:30 +0100
> +static int dwmac5_rxp_update_single_entry(void __iomem *ioaddr, > + struct stmmac_tc_entry *entry, int pos) Please follow the Linux networking coding style for function arguments in function declarations and definitions. Each second and subsequent line of the declaration/definition shall start precisely at the column after the openning parenthesis of the first line. Use the appropriate number of TAB then SPACE characters necessary to achieve this. Otherwise, this patch looks great. Thanks.