Thomas Heijligen has uploaded this change for review.

View Change

internal.c: unify the macro for x86 only code

\#if defined(__i386__) || defined(__x86_64__) is commonly
used for x86 only code across flashrom.
Only platfrom.h and hwaccess.* uses the IS_X86 macro

Change-Id: I94a599431f58666189c8cd601286e9b30c8bf62b
Signed-off-by: Thomas Heijligen <thomas.heijligen@secunet.de>
---
M internal.c
1 file changed, 7 insertions(+), 7 deletions(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/42/57942/1
diff --git a/internal.c b/internal.c
index abd4637..b9d17b4 100644
--- a/internal.c
+++ b/internal.c
@@ -88,7 +88,7 @@
return NULL;
}

-#if IS_X86
+#if defined(__i386__) || defined(__x86_64__)
void probe_superio(void)
{
probe_superio_winbond();
@@ -114,7 +114,7 @@
return 0;
}

-#endif /* IS_X86 */
+#endif

static void internal_chip_writeb(const struct flashctx *flash, uint8_t val,
chipaddr addr)
@@ -177,7 +177,7 @@
int not_a_laptop = 0;
char *board_vendor = NULL;
char *board_model = NULL;
-#if IS_X86
+#if defined(__i386__) || defined(__x86_64__)
const char *cb_vendor = NULL;
const char *cb_model = NULL;
#endif
@@ -269,7 +269,7 @@
goto internal_init_exit;
}

-#if IS_X86
+#if defined(__i386__) || defined(__x86_64__)
if ((cb_parse_table(&cb_vendor, &cb_model) == 0) && (board_vendor != NULL) && (board_model != NULL)) {
if (strcasecmp(board_vendor, cb_vendor) || strcasecmp(board_model, cb_model)) {
msg_pwarn("Warning: The mainboard IDs set by -p internal:mainboard (%s:%s) do not\n"
@@ -298,7 +298,7 @@
* FIXME: Find a replacement for DMI on non-x86.
* FIXME: Enable Super I/O probing once port I/O is possible.
*/
-#endif /* IS_X86 */
+#endif

/* Check laptop whitelist. */
board_handle_before_laptop();
@@ -322,7 +322,7 @@
goto internal_init_exit;
}

-#if IS_X86
+#if defined(__i386__) || defined(__x86_64__)
/* Probe unconditionally for ITE Super I/O chips. This enables LPC->SPI translation on IT87* and
* parallel writes on IT8705F. Also, this handles the manual chip select for Gigabyte's DualBIOS. */
init_superio_ite();
@@ -332,7 +332,7 @@
ret = 1;
goto internal_init_exit;
}
-#endif /* IS_X86 */
+#endif

if (internal_buses_supported & BUS_NONSPI)
register_par_master(&par_master_internal, internal_buses_supported, NULL);

To view, visit change 57942. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: I94a599431f58666189c8cd601286e9b30c8bf62b
Gerrit-Change-Number: 57942
Gerrit-PatchSet: 1
Gerrit-Owner: Thomas Heijligen <src@posteo.de>
Gerrit-MessageType: newchange