Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44313 )
Change subject: src: Drop meaningless `CPU_X86_TSC` symbol ......................................................................
Patch Set 1:
Patch Set 1:
Patch Set 1:
How does squashing the TSC patches look? I think it should be rather easy to review.
The resulting commit would be doing multiple things, and I don't want that. What's the big deal in having a temporary Kconfig symbol? I used many more commits to refactor sandybridge raminit once, and used much more hideous macros instead.
FWIW it's only one thing: now you need UDELAY_TSC + Makefile inclusion of that code and you'd go to just the Kconfig symbol. That's one thing and should result in a rather small patch that mostly removes things.