Thread (18 messages) 18 messages, 4 authors, 2024-08-20

Re: [PATCH net-next v2 2/9] bnxt_en: add support for retrieving crash dump using ethtool

flat view

From: Jakub Kicinski <kuba@kernel.org>
Date: 2024-08-20 01:41:42

On Mon, 19 Aug 2024 12:12:39 +0200 Przemek Kitszel wrote:
quoted
+	if (dump_type == BNXT_DUMP_CRASH &&
+	    bp->fw_dbg_cap & DBG_QCAPS_RESP_FLAGS_CRASHDUMP_HOST_DDR &&  
I would wrap bitwise-and with parenthesis, even if not mandated by the C
standard (the same in other places with combined expressions).
Personal preference, I guess. I generally lean on the side of limiting
the brackets.

👍️ to other comments, thank for reviewing!
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help