[Linux-kernel-mentees] [PATCH] checkpatch: add --fix option to IS_ENABLED_CONFIG check

Dwaipayan Ray dwaipayanray1 at gmail.com
Thu Dec 10 12:09:41 UTC 2020


> I ran as quick rough estimation: grep -e "\sIS_ENABLED" . -R | grep -v CONFIG
>
> I think:
>
> ./lib/crypto/chacha20poly1305-selftest.c: for (total_len =
> POLY1305_DIGEST_SIZE;
> IS_ENABLED(DEBUG_CHACHA20POLY1305_SLOW_CHUNK_TEST)
>
> could be fixed, is that right?
>

Yes, I think so. There were patches of this kind to correct those uses.
Evidently without CONFIG_ prefix the code could be skipped. But some of
those cases might be intentional as pointed here:

https://www.mail-archive.com/linux-kernel@vger.kernel.org/msg2184352.html

Thank you,
Dwaipayan.


More information about the Linux-kernel-mentees mailing list