[PATCH 06/10] fdtgrep: Simplify code to inverting the match

Simon Glass sjg at chromium.org
Sun Dec 17 17:36:19 CET 2023


The code to invert the match in h_include() is a bit convoluted.
Simplify it by using disp->invert only once.

Signed-off-by: Simon Glass <sjg at chromium.org>
---

 tools/fdtgrep.c | 10 ++--------
 1 file changed, 2 insertions(+), 8 deletions(-)

diff --git a/tools/fdtgrep.c b/tools/fdtgrep.c
index a6cdc326709d..b06a1a7a8384 100644
--- a/tools/fdtgrep.c
+++ b/tools/fdtgrep.c
@@ -634,14 +634,8 @@ static int h_include(void *priv, const void *fdt, int offset, int type,
 			inc = 0;
 	}
 
-	switch (inc) {
-	case 1:
-		inc = !disp->invert;
-		break;
-	case 0:
-		inc = disp->invert;
-		break;
-	}
+	if (inc != -1 && disp->invert)
+		inc = !inc;
 	debug("   - returning %d\n", inc);
 
 	return inc;
-- 
2.43.0.472.g3155946c3a-goog



More information about the U-Boot mailing list