[XS] Change in coreboot[main]: lib/device_tree.c: Fix wrong check for FDT validity

[Maximilian Brune (Code Review)]

Maximilian Brune has submitted this change. ( https://review.coreboot.org/c/coreboot/+/82773?usp=email )

Change subject: lib/device_tree.c: Fix wrong check for FDT validity
......................................................................

lib/device_tree.c: Fix wrong check for FDT validity

Obviously one should return NULL if a FDT is not valid an not the other
way around.

Signed-off-by: Maximilian Brune 
Change-Id: I77c0e187b841e60965daac17025110181bdd32bc
Reviewed-on: https://review.coreboot.org/c/coreboot/+/82773
Tested-by: build bot (Jenkins) 
Reviewed-by: Elyes Haouas 
---
M src/lib/device_tree.c
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  build bot (Jenkins): Verified
  Elyes Haouas: Looks good to me, approved

diff --git a/src/lib/device_tree.c b/src/lib/device_tree.c
index aff9e15..4f5cc07 100644
--- a/src/lib/device_tree.c
+++ b/src/lib/device_tree.c
@@ -608,7 +608,7 @@
    const struct fdt_header *header = (const struct fdt_header *)blob;
    tree->header = header;

-	if (fdt_is_valid(blob))
+	if (!fdt_is_valid(blob))
    	return NULL;

uint32_t struct_offset = be32toh(header->structure_offset);