Thanks for catching the missing semi-colon, fixed. --Jesper On Fri, 10 Apr 2020 08:59:46 +0800 kbuild test robot wrote: > All error/warnings (new ones prefixed by >>): > > drivers/net/ethernet/intel/ice/ice_txrx.c: In function 'ice_rx_frame_truesize': > >> drivers/net/ethernet/intel/ice/ice_txrx.c:439:2: error: expected ';' before 'return' > 439 | return truesize; > | ^~~~~~ > >> drivers/net/ethernet/intel/ice/ice_txrx.c:440:1: warning: no return statement in function returning non-void [-Wreturn-type] > 440 | } > | ^ > > vim +439 drivers/net/ethernet/intel/ice/ice_txrx.c > > 425 > 426 static unsigned int ice_rx_frame_truesize(struct ice_ring *rx_ring, > 427 unsigned int size) > 428 { > 429 unsigned int truesize; > 430 > 431 #if (PAGE_SIZE < 8192) > 432 truesize = ice_rx_pg_size(rx_ring) / 2; /* Must be power-of-2 */ > 433 #else > 434 truesize = ice_rx_offset(rx_ring) ? > 435 SKB_DATA_ALIGN(ice_rx_offset(rx_ring) + size) + > 436 SKB_DATA_ALIGN(sizeof(struct skb_shared_info)) : > 437 SKB_DATA_ALIGN(size) > 438 #endif > > 439 return truesize; > > 440 } > 441 -- Best regards, Jesper Dangaard Brouer MSc.CS, Principal Kernel Engineer at Red Hat LinkedIn: http://www.linkedin.com/in/brouer