uboot: import first variable from auxiliary environment

The U-Boot env file is generated by
mkenvimage, which prefixes the variable data with a 4-byte CRC32 by default.
ixpreboot.sh previously imported it with `env import -b` as per uboot docs(-b assume binary format (’0’ separated, “00” terminated).),
which treats the buffer as raw binary and does not strip the CRC.
As a result the first variable's name is parsed a part of CRC.
Later variables import correctly because by then the parser is past the CRC.
Switching to `env import -c` as per uboot docs (-c assume checksum protected environment format.)
makes U-Boot validate and strip the CRC before parsing, so the first variable imports under its real name.

Tested on:
- QEMU

Signed-off-by: Ejub Sabic <ejub1946@outlook.com>

board/common/rootfs/usr/libexec/infix/has-quirk

@@ -1,12 +1,49 @@
-#!/bin/sh
+#!/bin/bash
+
+IFQUIRKSFILE=${IFQUIRKSFILE:-/etc/product/interface-quirks.json}
+
 if [ $# -lt 2 ]; then
         echo "usage: $0 <quirk-name> <ifname>"
         exit 1
 fi
+
 quirk=$1
 ifname=$2
-if [ -f "/etc/product/interface-quirks.json" ]; then
-        echo "$(jq -r --arg iface "$ifname" --arg quirk "$quirk" '.[$iface][$quirk] // "false"' /etc/product/interface-quirks.json)"
-else
-        echo "false"
-fi
+
+[ -f "$IFQUIRKSFILE" ] || { echo false && exit; }
+
+match()
+{
+    jq -e  \
+       --arg quirk "$quirk" --arg pattern "$1" \
+       '.[$pattern][$quirk]' "$IFQUIRKSFILE" >/dev/null || return
+
+    echo true
+    exit 0
+}
+
+ethtoolmatch()
+{
+    local pattern="${1#@ethtool:}"
+
+    grep -qFxvf \
+	 <(ethtool -i "$ifname") \
+	 <(echo -n "$pattern" | awk -v FS="=" -v RS=";" '{ printf("%s: %s\n", $1, $2); }') \
+	&& return
+
+    match "@ethtool:$pattern"
+}
+
+
+for pattern in $(jq -r 'keys[]' "$IFQUIRKSFILE"); do
+    case "$pattern" in
+	@ethtool:*)
+	    ethtoolmatch "$pattern"
+	    ;;
+	"$ifname")
+	    match "$ifname"
+	    ;;
+    esac
+done
+
+echo "false"

board/common/rootfs/usr/libexec/infix/init.d/20-nameif

@@ -34,6 +34,12 @@ ports=$(devlink -j port | jq -r '.port[]
 				 | select(.flavour == "physical")
 				 | .netdev')
 for iface in $ports; do
+	# On systems with multiple switch trees, a port may be _both_
+	# a physical port, registered with devlink, _and_ a DSA
+	# port. In those cases, hold on to our initial "internal"
+	# classification.
+	[ $(ip -j link show dev "$iface" | jq -r '.[0].group') = internal ] && continue
+
 	ip link set "$iface" group port
 done
 

board/common/uboot/scripts/ixpreboot.sh

@@ -33,7 +33,7 @@ for tgt in "${boot_targets}"; do
 	setexpr ixmenu_n ${ixmenu_n} + 1
 
 	if load ${devtype} ${devnum}:${auxpart} ${loadaddr} /uboot.env; then
-	    env import -b ${loadaddr} ${filesize} BOOT_ORDER DEBUG ethact
+	    env import -c ${loadaddr} ${filesize} BOOT_ORDER DEBUG ethact
 	fi
 
 	if test -n "${DEBUG}"; then

src/confd/src/ietf-interfaces.c

@@ -17,18 +17,8 @@
 
 bool iface_has_quirk(const char *ifname, const char *quirkname)
 {
-	struct json_t *iface, *quirk;
-
-	if (!confd.ifquirks)
-		return false;
-
-	iface = json_object_get(confd.ifquirks, ifname);
-	if (!iface)
-		return false;
-
-	quirk = json_object_get(iface, quirkname);
-
-	return quirk ? json_is_true(quirk) : false;
+	return systemf("[ $(/usr/libexec/infix/has-quirk %s %s) = true ]",
+		       quirkname, ifname) == 0;
 }
 
 static bool iface_is_phys(const char *ifname)