Thread (3 messages) flat view 3 messages, 2 authors, 2018-09-12

Re: [PATCH v2] ethernet: hnae: drop adhoc assert() macros

From: David Miller <davem@davemloft.net>
Date: 2018-09-12 06:07:20
Also in: lkml

From: Igor Stoppa <redacted>
Date: Sat,  8 Sep 2018 18:01:42 +0300
Replace assert() with a less misleading test_condition() using WARN()
Drop one check which had bitrotted and didn't compile anymore.

Signed-off-by: Igor Stoppa <redacted>
I'm still kind of not happy about this.

Make the driver use kernel interfaces like WARN_ON_ONCE()
etc. directly instead of defining alias CPP macros private to the
driver.

If it needs to be conditional upon DEBUG, we have pr_debug() and
the likes as well.

Thank you.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help