[Intel-wired-lan] [RFC PATCH net-next v1 11/11] drivers/net/ethernet: clean up mis-targeted comments
Edward Cree
ecree at solarflare.com
Fri Sep 11 21:55:54 UTC 2020
On 11/09/2020 22:42, Jesse Brandeburg wrote:
> Thanks Ed, I think I might just remove the /** on that function then
> (removing it from kdoc processing)
I dunno, that means
a) kerneldoc won't generate html for this struct
b) new additions to the struct without corresponding kerneldoc won't
generate warnings
both of which are not ideal outcomes.
I realise there's value in having totally warning-clean code, but in
this case I think this one warning, even though it's indicating a
toolchain problem rather than a codebase problem, should better stay
(if only to put pressure on the toolchain to fix it).
Otherwise, when and if the toolchain is fixed, what's the chance we'll
remember to put the /** back?
That's just my opinion, though; I won't block patches that disagree.
-ed
More information about the Intel-wired-lan
mailing list