diff options
author | Paul Menzel <paulepanter@users.sourceforge.net> | 2013-02-23 21:31:23 +0100 |
---|---|---|
committer | Stefan Reinauer <stefan.reinauer@coreboot.org> | 2013-02-25 18:47:00 +0100 |
commit | 528640d141193b4aaeb6b8956d4fbc7381df9eb3 (patch) | |
tree | 98e9dfa7e912b8187600d1347f369a02598d56a3 /src/mainboard/technexion | |
parent | 1fc7416545bde75d9cc401638f28ebfb751b663e (diff) | |
download | coreboot-528640d141193b4aaeb6b8956d4fbc7381df9eb3.tar.xz |
mainboard.c: Name enable_dev function uniformly `mainboard_enable`
To reduce the differences between these file name the enabling
device function in the directory `src/mainboard` uniformly
`mainboard_enable` [1].
Thanks to the awesome help of gnomon and BlastHardcheese in the
IRC channel #sed on <irc.freenode.net>. gnomon came up with the
following command to do the actual work.
$ cd src/mainboard
$ for f in */*/mainboard.c ; \
> do src="$(awk '/\.enable_dev = /{v=$NF; sub(/,$/,"",v); print v}' "$f")" ; \
> [[ -z $src ]] && continue ; \
> printf '%s\n' "g/${src}/s/${src}\([,(]\)/mainboard_enable\1/p" w | ed -s "$f" ; \
> done
`src/mainboard/digitallogic/msm586seg/mainboard.c` and
`src/mainboard/technologic/ts5300/mainboard.c` had to be adapted
manually as no comma was used separating the struct members.
And with the following statement, gnomon is even more likable!
My pleasure entirely. Good luck with coreboot; I'm a big fan of the project.
[1] http://www.coreboot.org/pipermail/coreboot/2013-February/074548.html
Change-Id: Ife9cd0c2d9cc1ed14afc6d40063450553f06a6c6
Signed-off-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-on: http://review.coreboot.org/2493
Tested-by: build bot (Jenkins)
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Diffstat (limited to 'src/mainboard/technexion')
-rw-r--r-- | src/mainboard/technexion/tim5690/mainboard.c | 4 | ||||
-rw-r--r-- | src/mainboard/technexion/tim8690/mainboard.c | 4 |
2 files changed, 4 insertions, 4 deletions
diff --git a/src/mainboard/technexion/tim5690/mainboard.c b/src/mainboard/technexion/tim5690/mainboard.c index 7974ce9c7f..5be58cddb5 100644 --- a/src/mainboard/technexion/tim5690/mainboard.c +++ b/src/mainboard/technexion/tim5690/mainboard.c @@ -222,7 +222,7 @@ static void lcd_panel_id(rs690_vbios_regs *vbios_regs, u8 num_id) * enable the dedicated function in tim5690 board. * This function called early than rs690_enable. *************************************************/ -static void tim5690_enable(device_t dev) +static void mainboard_enable(device_t dev) { u16 gpio_base = IT8712F_SIMPLE_IO_BASE; #if CONFIG_VGA_ROM_RUN @@ -257,5 +257,5 @@ void mainboard_post(u8 value) } struct chip_operations mainboard_ops = { - .enable_dev = tim5690_enable, + .enable_dev = mainboard_enable, }; diff --git a/src/mainboard/technexion/tim8690/mainboard.c b/src/mainboard/technexion/tim8690/mainboard.c index 377cbd783f..69788e9f05 100644 --- a/src/mainboard/technexion/tim8690/mainboard.c +++ b/src/mainboard/technexion/tim8690/mainboard.c @@ -140,7 +140,7 @@ static void set_thermal_config(void) * enable the dedicated function in tim8690 board. * This function called early than rs690_enable. *************************************************/ -static void tim8690_enable(device_t dev) +static void mainboard_enable(device_t dev) { printk(BIOS_INFO, "Mainboard tim8690 Enable. dev=0x%p\n", dev); @@ -149,5 +149,5 @@ static void tim8690_enable(device_t dev) } struct chip_operations mainboard_ops = { - .enable_dev = tim8690_enable, + .enable_dev = mainboard_enable, }; |