Change in ...flashrom[master]: libflashrom: Add CPP guard to fix big-endian builds

[Nico Huber (Code Review)]

Nico Huber has submitted this change and it was merged. ( https://review.coreboot.org/c/flashrom/+/33306 )

Change subject: libflashrom: Add CPP guard to fix big-endian builds
......................................................................

libflashrom: Add CPP guard to fix big-endian builds

Calm a compiler warning on big-endian builds about the unused static
flashrom_layout_parse_fmap(). The guard is ugly but gets the job done.
We should forbid endian-specific code in the future, I guess.

Change-Id: Id3f4a57e027f88cc469ed50312adddcc8af71a63
Signed-off-by: Nico Huber <[e-mail]>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/33306
Tested-by: build bot (Jenkins) <[e-mail]>
Reviewed-by: Angel Pons <[e-mail]>
---
M libflashrom.c
1 file changed, 2 insertions(+), 0 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Angel Pons: Looks good to me, approved



diff --git a/libflashrom.c b/libflashrom.c
index c5e56ae..fee60ff 100644
--- a/libflashrom.c
+++ b/libflashrom.c
@@ -386,6 +386,7 @@
 #endif
 }
 
+#ifdef __FLASHROM_LITTLE_ENDIAN__
 static int flashrom_layout_parse_fmap(struct flashrom_layout **layout,
 		struct flashctx *const flashctx, const struct fmap *const fmap)
 {
@@ -417,6 +418,7 @@
 	*layout = l;
 	return 0;
 }
+#endif /* __FLASHROM_LITTLE_ENDIAN__ */
 
 /**
  * @brief Read a layout by searching the flash chip for fmap.

-- 
To view, visit https://review.coreboot.org/c/flashrom/+/33306
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: Id3f4a57e027f88cc469ed50312adddcc8af71a63
Gerrit-Change-Number: 33306
Gerrit-PatchSet: 4
Gerrit-Owner: Nico Huber <[e-mail]>
Gerrit-Reviewer: Angel Pons <[e-mail]>
Gerrit-Reviewer: Nico Huber <[e-mail]>
Gerrit-Reviewer: build bot (Jenkins) <[e-mail]>
Gerrit-MessageType: merged