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

[Yidi Lin (Code Review)]

Attention is currently required from: Hung-Te Lin, Xi Chen, Yu-Ping Wu.

Yidi Lin 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:

(2 comments)

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

https://review.coreboot.org/c/coreboot/+/84222/comment/9c7d43d3_c82eb853?usp... :
PS2, Line 59: PAD_##name##_FUNC_##func
> I didn't notice mt8173 and mt8183 are excluded from CB:84221. […]

Yes.

https://review.coreboot.org/c/coreboot/+/84222/comment/429ac45e_e0f1a589?usp... :
PS2, Line 59: }
> I mean, `GPIO_PULL_DOWN` is 0, so the current `PAD_FUNC` macro defined here is equivalent to setting […]

I see your point. I will implement option 3 in next patchset.