On Fri, 01 Nov 2024 17:10:56 +0100, Christophe JAILLET wrote: > If bnxt_re_add_device() fails, 'en_info' still needs to be freed, as > already done in the .remove() function. > > The commit in Fixes incorrectly removed this call, certainly because it > was expecting the .remove() function was called anyway. But if the probe > fails, the remove function is not called. > > [...] Applied, thanks! [1/2] RDMA/bnxt_re: Fix some error handling paths in bnxt_re_probe() https://git.kernel.org/rdma/rdma/c/cf90a4d1b9ff9e [2/2] RDMA/bnxt_re: Remove some dead code https://git.kernel.org/rdma/rdma/c/aceee63a3aba46 Best regards, -- Leon Romanovsky <leon@xxxxxxxxxx>