On Mon, Aug 30, 2021 at 06:38:28PM +0300, Sagi Grimberg wrote:

quoted

+	if (req->status == cpu_to_le16(NVME_SC_SUCCESS))

endian conversion a zero enum? looks weird..

I though it makes sense to stay consistent with the rest and don't make
it harder for the reader to figure out why this might not needed here.

And obviously the fear of the static code analyzers...

But,
Reviewed-by: Sagi Grimberg <sagi@grimberg.me>

Thanks!

_______________________________________________
Linux-nvme mailing list
Linux-nvme@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-nvme