Re: [PATCH 1/1] netdev: broadcom: propagate error code

2016-12-05 Thread David Miller
From: Pan Bian Date: Sat, 3 Dec 2016 17:56:17 +0800 > Function bnxt_hwrm_stat_ctx_alloc() always returns 0, even if the call > to _hwrm_send_message() fails. It may be better to propagate the errors > to the caller of bnxt_hwrm_stat_ctx_alloc(). > > Bugzilla: https://bugzilla.kernel.org/show_bu

Re: [PATCH 1/1] netdev: broadcom: propagate error code

2016-12-03 Thread Michael Chan
On Sat, Dec 3, 2016 at 1:56 AM, Pan Bian wrote: > Function bnxt_hwrm_stat_ctx_alloc() always returns 0, even if the call > to _hwrm_send_message() fails. It may be better to propagate the errors > to the caller of bnxt_hwrm_stat_ctx_alloc(). > > Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?i

[PATCH 1/1] netdev: broadcom: propagate error code

2016-12-03 Thread Pan Bian
Function bnxt_hwrm_stat_ctx_alloc() always returns 0, even if the call to _hwrm_send_message() fails. It may be better to propagate the errors to the caller of bnxt_hwrm_stat_ctx_alloc(). Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=188661 Signed-off-by: Pan Bian --- drivers/net/ethern