libfdt: use CONFIG_IS_ENABLED for OF_LIBFDT

PATCH New Vignesh R 2 participants 2 messages Mar 20, 2017 – Mar 27, 2017 Library code

Share & tools

Vignesh R

author

2017-03-20 04:34 UTC #0 lore raw

PATCH v2 1/1 libfdt: use CONFIG_IS_ENABLED for OF_LIBFDT

Use CONFIG_IS_ENABLED() macro to check whether OF_TRANSLATE is enabled, so that code block is compiled irrespective of SPL or U-Boot build and fdt address translation is used.

Signed-off-by Vignesh R <[email protected]>

---

v2:
* Depend on OF_TRANSLATE instead of OF_LIBFDT so that boards like chromebook_link64_defconfig compile fine.

 lib/fdtdec.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

lib/fdtdec.c +1 -1

index 81f47ef2c7f4..1edfbf2d3921 100644
@@ -112,7 +112,7 @@ fdt_addr_t fdtdec_get_addr_size_fixed(const void *blob, int node,
 		return FDT_ADDR_T_NONE;
 	}
 
-#if !defined(CONFIG_SPL_BUILD) && defined(CONFIG_OF_LIBFDT)
+#if CONFIG_IS_ENABLED(OF_TRANSLATE)
 	if (translate)
 		addr = fdt_translate_address(blob, node, prop_addr);
 	else

Signature · 1 line

2.11.0

Simon Glass

2017-03-27 02:27 UTC #1 lore raw

PATCH v2 1/1 libfdt: use CONFIG_IS_ENABLED for OF_LIBFDT

Quoted 5 lines · Vignesh R

Use CONFIG_IS_ENABLED() macro to check whether OF_TRANSLATE is enabled, so that code block is compiled irrespective of SPL or U-Boot build and fdt address translation is used.
Signed-off-by Vignesh R <[email protected]>

Quoted 8 lines

---
v2:
* Depend on OF_TRANSLATE instead of OF_LIBFDT so that boards like chromebook_link64_defconfig compile fine.
 lib/fdtdec.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Applied to u-boot-dm, thanks!