[coreboot-gerrit] New patch to review for coreboot: 56aa3f1 snow: Tidy up chromeos.c.
Gabe Black (gabeblack@chromium.org)
gerrit at coreboot.org
Tue Apr 16 06:01:20 CEST 2013
Gabe Black (gabeblack at chromium.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/3093
-gerrit
commit 56aa3f1860638a9730886d467746d258ee4670b8
Author: Gabe Black <gabeblack at chromium.org>
Date: Mon Apr 15 18:22:11 2013 -0700
snow: Tidy up chromeos.c.
Got rid of a lot of #defines, some of which were converted to enums and
the rest which were eliminated entirely. Got rid of cruft in
get_developer_mode_switch and started using it for the dev mode GPIO.
Instead of a macro defining how many GPIOs are expected, now the code
actually counts the GPIOs as they're added.
Change-Id: I97b6b9f52a72d1276eb3cf36d7f9dd7b335b4d19
Signed-off-by: Gabe Black <gabeblack at chromium.org>
---
src/mainboard/google/snow/chromeos.c | 101 ++++++++++++++++++-----------------
1 file changed, 51 insertions(+), 50 deletions(-)
diff --git a/src/mainboard/google/snow/chromeos.c b/src/mainboard/google/snow/chromeos.c
index f33f103..ca105e7 100644
--- a/src/mainboard/google/snow/chromeos.c
+++ b/src/mainboard/google/snow/chromeos.c
@@ -17,6 +17,7 @@
* Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
*/
+#include <boot/coreboot_tables.h>
#include <console/console.h>
#include <ec/google/chromeec/ec.h>
#include <ec/google/chromeec/ec_commands.h>
@@ -27,19 +28,16 @@
#include <cpu/samsung/exynos5250/gpio.h>
#include <cpu/samsung/exynos5-common/gpio.h>
-#define ACTIVE_LOW 0
-#define ACTIVE_HIGH 1
-#define WP_GPIO 6
-#define DEVMODE_GPIO 54
-#define RECMODE_GPIO 0
-#define FORCE_RECOVERY_MODE 0
-#define FORCE_DEVELOPER_MODE 0
-#define LID_OPEN 5
-#define POWER_BUTTON 3
-
-#include <boot/coreboot_tables.h>
+enum {
+ ACTIVE_LOW = 0,
+ ACTIVE_HIGH = 1
+};
-#define GPIO_COUNT 6
+enum {
+ WP_GPIO = 6,
+ RECMODE_GPIO = 0,
+ LID_GPIO = 5
+};
static struct exynos5_gpio_part1 *gpio_pt1 =
(struct exynos5_gpio_part1 *)EXYNOS5_GPIO_PART1_BASE;
@@ -48,62 +46,65 @@ static struct exynos5_gpio_part2 *gpio_pt2 =
void fill_lb_gpios(struct lb_gpios *gpios)
{
- gpios->size = sizeof(*gpios) + (GPIO_COUNT * sizeof(struct lb_gpio));
- gpios->count = GPIO_COUNT;
+ int count = 0;
/* Write Protect: active low */
- gpios->gpios[0].port = EXYNOS5_GPD1;
- gpios->gpios[0].polarity = ACTIVE_LOW;
- gpios->gpios[0].value = s5p_gpio_get_value(&gpio_pt1->d1, WP_GPIO);
- strncpy((char *)gpios->gpios[0].name,"write protect",
- GPIO_MAX_NAME_LENGTH);
+ gpios->gpios[count].port = EXYNOS5_GPD1;
+ gpios->gpios[count].polarity = ACTIVE_LOW;
+ gpios->gpios[count].value = s5p_gpio_get_value(&gpio_pt1->d1, WP_GPIO);
+ strncpy((char *)gpios->gpios[count].name, "write protect",
+ GPIO_MAX_NAME_LENGTH);
+ count++;
/* Recovery: active low */
- gpios->gpios[1].port = -1;
- gpios->gpios[1].polarity = ACTIVE_HIGH;
- gpios->gpios[1].value = get_recovery_mode_switch();
- strncpy((char *)gpios->gpios[1].name,"recovery", GPIO_MAX_NAME_LENGTH);
+ gpios->gpios[count].port = -1;
+ gpios->gpios[count].polarity = ACTIVE_HIGH;
+ gpios->gpios[count].value = get_recovery_mode_switch();
+ strncpy((char *)gpios->gpios[count].name, "recovery",
+ GPIO_MAX_NAME_LENGTH);
+ count++;
/* Lid: active high */
- gpios->gpios[2].port = EXYNOS5_GPX3;
- gpios->gpios[2].polarity = ACTIVE_HIGH;
- gpios->gpios[2].value = s5p_gpio_get_value(&gpio_pt2->x3, LID_OPEN);
- strncpy((char *)gpios->gpios[2].name,"lid", GPIO_MAX_NAME_LENGTH);
+ gpios->gpios[count].port = EXYNOS5_GPX3;
+ gpios->gpios[count].polarity = ACTIVE_HIGH;
+ gpios->gpios[count].value = s5p_gpio_get_value(&gpio_pt2->x3, LID_GPIO);
+ strncpy((char *)gpios->gpios[count].name, "lid", GPIO_MAX_NAME_LENGTH);
+ count++;
/* Power: virtual GPIO active low */
- gpios->gpios[3].port = -1;
- gpios->gpios[3].polarity = ACTIVE_LOW;
- gpios->gpios[3].value = 1;
- strncpy((char *)gpios->gpios[3].name,"power", GPIO_MAX_NAME_LENGTH);
+ gpios->gpios[count].port = -1;
+ gpios->gpios[count].polarity = ACTIVE_LOW;
+ gpios->gpios[count].value = 1;
+ strncpy((char *)gpios->gpios[count].name, "power",
+ GPIO_MAX_NAME_LENGTH);
+ count++;
/* Developer: virtual GPIO active high */
- gpios->gpios[4].port = -1;
- gpios->gpios[4].polarity = ACTIVE_HIGH;
- gpios->gpios[4].value = 0;
- strncpy((char *)gpios->gpios[4].name,"developer",
- GPIO_MAX_NAME_LENGTH);
+ gpios->gpios[count].port = -1;
+ gpios->gpios[count].polarity = ACTIVE_HIGH;
+ gpios->gpios[count].value = get_developer_mode_switch();
+ strncpy((char *)gpios->gpios[count].name, "developer",
+ GPIO_MAX_NAME_LENGTH);
+ count++;
/* Was VGA Option ROM loaded? */
- gpios->gpios[5].port = -1; /* Indicate that this is a pseudo GPIO */
- gpios->gpios[5].polarity = ACTIVE_HIGH;
- gpios->gpios[5].value = 0;
- strncpy((char *)gpios->gpios[5].name,"oprom", GPIO_MAX_NAME_LENGTH);
+ gpios->gpios[count].port = -1; /* This is a pseudo GPIO */
+ gpios->gpios[count].polarity = ACTIVE_HIGH;
+ gpios->gpios[count].value = 0;
+ strncpy((char *)gpios->gpios[count].name, "oprom",
+ GPIO_MAX_NAME_LENGTH);
+ count++;
+
+ gpios->size = sizeof(*gpios) + (count * sizeof(struct lb_gpio));
+ gpios->count = count;
- printk(BIOS_ERR, "Added %d GPIOS size %d\n", GPIO_COUNT, gpios->size);
+ printk(BIOS_ERR, "Added %d GPIOS size %d\n", count, gpios->size);
}
int get_developer_mode_switch(void)
{
- int dev_mode = 0;
-
- printk(BIOS_DEBUG,"FORCING DEVELOPER MODE.\n");
-
- dev_mode = 1;
- printk(BIOS_DEBUG,"DEVELOPER MODE FROM GPIO %d: %x\n",DEVMODE_GPIO,
- dev_mode);
-
- return dev_mode;
+ return 0;
}
int get_recovery_mode_switch(void)
More information about the coreboot-gerrit
mailing list