[M] Change in coreboot[main]: soc/mediatek: Remove redundant struct pad_func and PAD_* definitions

[Yu-Ping Wu (Code Review)]

Attention is currently required from: Hung-Te Lin, Xi Chen, Yidi Lin.

Yu-Ping Wu has posted comments on this change by Yidi Lin. ( https://review.coreboot.org/c/coreboot/+/84222?usp=email )

Change subject: soc/mediatek: Remove redundant struct pad_func and PAD_* definitions
......................................................................

Patch Set 2:

(1 comment)

File src/soc/mediatek/common/include/soc/gpio_common.h:

https://review.coreboot.org/c/coreboot/+/84222/comment/72ef82de_2845c555?usp... :
PS2, Line 59: }
> I see your point. I will implement option 3 in next patchset.

Thanks. We can name the 3-arg macro `PAD_FUNC(name, func, select)` (simply because it's shorter) and then drop the `PAD_FUNC_SEL` name. So there will be 3 macros in total: `PAD_FUNC`, `PAD_FUNC_DOWN`, `PAD_FUNC_UP`.