chromeos: Simplify fill_lb_gpios even further

Hardware / Coreboot - Julius Werner [chromium.org] - 4 April 2016 17:50 UTC

A long time ago many Chrome OS boards had pages full of duplicated boilerplate code for the fill_lb_gpios() function, and we spent a lot of time bikeshedding a proper solution that passes a table of lb_gpio structs which can be concisely written with a static struct initializer in http://crosreview.com/234648. Unfortunately we never really finished that patch and in the mean time a different solution using the fill_lb_gpio() helper got standardized onto most boards.

Still, that solution is not quite as clean and concise as the one we had already designed, and it also wasn't applied consistently to all recent boards (causing more boards with bad code to get added afterwards). This patch switches all boards newer than Link to the better solution and also adds some nicer debug output for the GPIOs while I'm there.

If more boards need to be converted from fill_lb_gpio() to this model later (e.g. from a branch), it's quite easy to do with: s/fill_lb_gpio(gpio++,\n\?\s*\([^,]*\),\n\?\s*\([^,]*\),\n\?\s*\([^,]*\),\n\?\s*\([^,]*\));/\t{\1, \2, \4, \3},/

Based on a patch by Furquan Shaikh.

BUG=None BRANCH=None TEST=Booted on Oak. Ran abuild -x.

Change-Id: I449974d1c75c8ed187f5e10935495b2f03725811

c445b4f chromeos: Simplify fill_lb_gpios even further
src/include/boot/coreboot_tables.h | 5 +-
src/lib/coreboot_table.c | 42 +++++++++---
src/mainboard/google/auron/chromeos.c | 39 +++--------
src/mainboard/google/auron_paine/chromeos.c | 26 +++----
src/mainboard/google/bolt/chromeos.c | 26 +++----
src/mainboard/google/chell/chromeos.c | 29 +++-----
src/mainboard/google/cyan/chromeos.c | 25 +++----
src/mainboard/google/falco/chromeos.c | 26 +++----
src/mainboard/google/glados/chromeos.c | 29 +++-----
src/mainboard/google/guado/chromeos.c | 26 +++----
src/mainboard/google/jecht/chromeos.c | 28 +++-----
src/mainboard/google/lars/chromeos.c | 29 +++-----
src/mainboard/google/nyan/chromeos.c | 71 +++----------------
src/mainboard/google/nyan_big/chromeos.c | 71 +++----------------
src/mainboard/google/nyan_blaze/chromeos.c | 71 +++----------------
src/mainboard/google/oak/chromeos.c | 72 +++-----------------
src/mainboard/google/panther/chromeos.c | 26 +++----
src/mainboard/google/peppy/chromeos.c | 26 +++----
src/mainboard/google/rambi/chromeos.c | 26 +++----
src/mainboard/google/rush/chromeos.c | 63 +++--------------
src/mainboard/google/rush_ryu/chromeos.c | 68 ++++---------------
src/mainboard/google/samus/chromeos.c | 26 +++----
src/mainboard/google/slippy/chromeos.c | 26 +++----
src/mainboard/google/smaug/chromeos.c | 66 +++---------------
src/mainboard/google/storm/chromeos.c | 20 +++---
src/mainboard/google/tidus/chromeos.c | 26 +++----
src/mainboard/google/veyron/chromeos.c | 90 ++++---------------------
src/mainboard/google/veyron_brain/chromeos.c | 55 +++------------
src/mainboard/google/veyron_danger/chromeos.c | 56 +++------------
src/mainboard/google/veyron_emile/chromeos.c | 46 +++----------
src/mainboard/google/veyron_mickey/chromeos.c | 46 +++----------
src/mainboard/google/veyron_rialto/chromeos.c | 60 +++--------------
src/mainboard/google/veyron_romy/chromeos.c | 55 +++------------
src/mainboard/intel/kunimitsu/chromeos.c | 29 +++-----
src/mainboard/intel/strago/chromeos.c | 25 +++----
src/mainboard/intel/wtm2/chromeos.c | 23 +++----
36 files changed, 370 insertions(+), 1103 deletions(-)

Upstream: review.coreboot.org


  • Share