Attention is currently required from: Jian Tong, Karthik Ramasubramanian, Shelley Chen.
Kun Liu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/81817?usp=email )
Change subject: mb/google/brox: Create lotso variant ......................................................................
Patch Set 6:
(2 comments)
File src/mainboard/google/brox/variants/lotso/gpio.c:
https://review.coreboot.org/c/coreboot/+/81817/comment/e21ca413_83d231ab : PS5, Line 9: const struct pad_config *variant_gpio_table(size_t *num) : { : *num = 0; : return NULL; : } : : const struct pad_config *variant_early_gpio_table(size_t *num) : { : *num = 0; : return NULL; : }
I think that you don't need these override functions if they are just empty?
Done
https://review.coreboot.org/c/coreboot/+/81817/comment/9a46898d_074d381e : PS5, Line 26: const struct pad_config *variant_romstage_gpio_table(size_t *num) : { : *num = 0; : return NULL; : } :
same here?
Done