1
1

uboot-lantiq: fix C-style comments in AWK scripts

Modern gawk rejects C-style /* ... */ comments in AWK code, treating
them as regex patterns where '*' has nothing to quantify. Replace all
such comments with AWK-style '#' comments in lantiq_bdi_conf.awk and
lantiq_ram_init_uart.awk.

Also replace the pattern 'if (x) /* comment */ else action' which used
a C comment as a null statement with the equivalent 'if (!x) action'.

Fixes build error:
  awk: error: ? * + or {interval} not preceded by valid subpattern

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Link: https://github.com/openwrt/openwrt/pull/22458
Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>
This commit is contained in:
Hauke Mehrtens 2026-03-17 01:06:24 +01:00
parent 62ca0a987f
commit 9b1f73ec31

View File

@ -251,7 +251,7 @@ Signed-off-by: Daniel Schwierzeck <daniel.schwierzeck@gmail.com>
+} +}
+ +
+/^#define/ { +/^#define/ {
+ /* DC03 contains MC enable bit and must not be set here */ + # DC03 contains MC enable bit and must not be set here
+ if (tolower($2) != "mc_dc03_value") + if (tolower($2) != "mc_dc03_value")
+ printf("WM32 0x%x %s\n", reg_base, tolower($3)) + printf("WM32 0x%x %s\n", reg_base, tolower($3))
+ +
@ -357,7 +357,7 @@ Signed-off-by: Daniel Schwierzeck <daniel.schwierzeck@gmail.com>
+} +}
--- /dev/null --- /dev/null
+++ b/tools/lantiq_ram_init_uart.awk +++ b/tools/lantiq_ram_init_uart.awk
@@ -0,0 +1,117 @@ @@ -0,0 +1,115 @@
+#!/usr/bin/awk -f +#!/usr/bin/awk -f
+# +#
+# Copyright (C) 2011-2012 Luka Perkov <luka@openwrt.org> +# Copyright (C) 2011-2012 Luka Perkov <luka@openwrt.org>
@ -383,45 +383,45 @@ Signed-off-by: Daniel Schwierzeck <daniel.schwierzeck@gmail.com>
+ +
+function mc_danube_prologue() +function mc_danube_prologue()
+{ +{
+ /* Clear access error log registers */ + # Clear access error log registers
+ print "0xbf800010", "0x0" + print "0xbf800010", "0x0"
+ print "0xbf800020", "0x0" + print "0xbf800020", "0x0"
+ +
+ /* Enable DDR and SRAM module in memory controller */ + # Enable DDR and SRAM module in memory controller
+ print "0xbf800060", "0x5" + print "0xbf800060", "0x5"
+ +
+ /* Clear start bit of DDR memory controller */ + # Clear start bit of DDR memory controller
+ print "0xbf801030", "0x0" + print "0xbf801030", "0x0"
+} +}
+ +
+function mc_ar9_prologue() +function mc_ar9_prologue()
+{ +{
+ /* Clear access error log registers */ + # Clear access error log registers
+ print "0xbf800010", "0x0" + print "0xbf800010", "0x0"
+ print "0xbf800020", "0x0" + print "0xbf800020", "0x0"
+ +
+ /* Enable FPI, DDR and SRAM module in memory controller */ + # Enable FPI, DDR and SRAM module in memory controller
+ print "0xbf800060", "0xD" + print "0xbf800060", "0xD"
+ +
+ /* Clear start bit of DDR memory controller */ + # Clear start bit of DDR memory controller
+ print "0xbf801030", "0x0" + print "0xbf801030", "0x0"
+} +}
+ +
+function mc_ddr1_epilogue() +function mc_ddr1_epilogue()
+{ +{
+ /* Set start bit of DDR memory controller */ + # Set start bit of DDR memory controller
+ print "0xbf801030", "0x100" + print "0xbf801030", "0x100"
+} +}
+ +
+function mc_ddr2_prologue() +function mc_ddr2_prologue()
+{ +{
+ /* Put memory controller in inactive mode */ + # Put memory controller in inactive mode
+ print "0xbf401070", "0x0" + print "0xbf401070", "0x0"
+} +}
+ +
+function mc_ddr2_epilogue(mc_ccr07_value) +function mc_ddr2_epilogue(mc_ccr07_value)
+{ +{
+ /* Put memory controller in active mode */ + # Put memory controller in active mode
+ mc_ccr07_value = or(mc_ccr07_value, 0x100) + mc_ccr07_value = or(mc_ccr07_value, 0x100)
+ printf("0xbf401070 0x%x\n", mc_ccr07_value) + printf("0xbf401070 0x%x\n", mc_ccr07_value)
+} +}
@ -452,12 +452,10 @@ Signed-off-by: Daniel Schwierzeck <daniel.schwierzeck@gmail.com>
+} +}
+ +
+/^#define/ { +/^#define/ {
+ /* CCR07 contains MC enable bit and must not be set here */ + # CCR07 contains MC enable bit and must not be set here
+ if (tolower($2) == "mc_ccr07_value") + if (tolower($2) == "mc_ccr07_value")
+ mc_ccr07_value = strtonum($3) + mc_ccr07_value = strtonum($3)
+ if (tolower($2) == "mc_dc03_value") + if (tolower($2) != "mc_dc03_value")
+ /* CCR07 contains MC enable bit and must not be set here */
+ else
+ printf("0x%x %s\n", reg_base, tolower($3)) + printf("0x%x %s\n", reg_base, tolower($3))
+ +
+ reg_base += 0x10 + reg_base += 0x10