[coreboot-gerrit] Change in ...coreboot[master]: kconfig_lint: Make usage of IS_ENABLED() an error

[Julius Werner (Code Review)]

Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32229 )

Change subject: kconfig_lint: Make usage of IS_ENABLED() an error
......................................................................

Patch Set 1:

(2 comments)

https://review.coreboot.org/#/c/32229/1/util/lint/kconfig_lint 
File util/lint/kconfig_lint:

https://review.coreboot.org/#/c/32229/1/util/lint/kconfig_lint@296 
PS1, Line 296: sub check_is_enabled {
Rename the function too?

https://review.coreboot.org/#/c/32229/1/util/lint/kconfig_lint@320 
PS1, Line 320:         } elsif ( $line =~ /^([^:]+):(\d+):(.+IS_ENABLED.*)/ ) {
I think you might also just kill all the IS_ENABLED() code completely? If you take out the macro itself the compiler will already notice when it's used somewhere.