On Wed, 07 May 2025 14:18:34 +0100, Colin Ian King wrote: > The use of the ternary operator on rv is redundant, rv is > either the initialized value of 0 or a negative error return > code, so it can never be greater than zero, and hence the > zero assignment in ternary operator is redundant. Just return > rv instead. > > > [...] Applied, thanks! [1/1] RDMA/siw: replace redundant ternary operator with just rv https://git.kernel.org/rdma/rdma/c/8536666a52833d Best regards, -- Leon Romanovsky <leon@xxxxxxxxxx>