Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/30913 )
Change subject: mainboard/intel: Update mainboard UART Kconfig ......................................................................
Patch Set 2:
partial fixing of problem and leaving on others to act on accordingly always dangerous. I would recommend better to ask onwer to test and give their +2 before merge any common CL unless you have tested all board.
Well again, the information necessary to even see a potential problem (that people have UART_DEBUG selected manually in their .config) was hidden. So I had to rely on others to act in any case. And nobody reacted for 2 months. Until it was merged. You seem to know who is using what board and whom to ask, so you could have passed my review request along. I don't know who is using what board. And Intel+Google seem to always forget to put that information into our MAINTAINERS file.
It seems to me that there evolved a community of Intel+Google developers that pretty much ignores what is going on around them. I don't know how to fix that.
Could you please stop making flippant comments?
I wouldn't even have started if I wasn't attacked for other peoples faults. Subrata is definitely expecting that I know things that you just can't know if you are neither working for Intel or Google.
Also, it is a wonderful position to be in with perfect hindsight vision, but that's not the reality of new development. Also, I think you are failing to empathize the breadth and inexactness it takes to bring a new chipset online within a specific hardware schedule. The beauty of open source is that we can correct mistakes when an experiment or approach doesn't work out so well. However, ignoring the time component to all of this while taking a holier-than-thou tone is not endearing. Please try to put yourself in others' shoes.
When exactly did I ignore the time component? Did I ever say somebody did a bad job back then? If we can correct mistakes, why do I have to justify the correction three times?