Change in coreboot[master]: soc/intel/*: drop UART pad configuration from common code

[Michael Niewöhner (Code Review)]

Michael Niewöhner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48829 )

Change subject: soc/intel/*: drop UART pad configuration from common code
......................................................................

Patch Set 6:

(1 comment)

https://review.coreboot.org/c/coreboot/+/48829/6//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/48829/6//COMMIT_MSG@9 
PS6, Line 9: UART pad configuration should not be done in common code, because that
           : may cause short circuits, when the user sets a wrong UART index.
> > > FIXED_UART_FOR_CONSOLE was just a workaround but does not fix the underlying issue.
> >
> > I was unaware of that, but what is the underlying issue? There won't be any GPIO misconfiguration if the UART number is set correctly.
>
> Nothing wrong per se. It's just inconistent and thereby more error-prone.
> Before we introduced FIXED_UART_FOR_CONSOLE, the defaults were almost
> always wrong btw.

One more addition on this: people still don't get this right; see at least ehl rvp... there were some more that set `0` while they mean `disable`, but `0` is UART0 - but the pads are connected elsewhere