On Thu, 2015-12-03 at 16:29 -0800, Jeff Kirsher wrote: > From: Nick <xerofo...@gmail.com>
I guess this is an OK patch, but the From line isn't correct (missing full name) and the changelog is nigh on unreadable. > This fixes the function fm10k_setup_tc to properly check if the > calls to either the function fm10k_init_queueing_scheme or the > function fm10k_mbx_request_irq fail by returning a error code to > signal that the call to either function has failed. Furthermore > if this arises exit immediately from the function fm10k_setup_tc > by returning the returned error code from the failed function call > to signal to the caller that setting up the tc on the device has > failed and the caller needs to handle this failed setup. Maybe something like: Add error handling in fm10k_setup_tc of fm10k_init_queueing_scheme and fm10k_mbx_request_irq. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html