Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/30913 )
Change subject: mainboard/intel: Update mainboard UART Kconfig ......................................................................
Patch Set 2:
Patch Set 2:
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.
It works both ways, and from my observations of your commentary on CLs and IRC you aren't the lone victim in such behavior.
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?
By your commentary and tone beyond this particular CL. You have explicitly noted how you know the best way to do anything an scoff at other solutions ignoring the circumstances while also hurling insults around. As I noted, it's a lot easier to come upon a platform that has fuller documentation and someone has already done a lot of the baking for enabling it. Sure, there are always rough edges, but you seem to suggest that's entirely possible at the time of development which I assert is not.
I don't understand what you mean by the 'justify the correction three times' thing, fwiw.