Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@patch/u-boot/v2026.07/1001-fdt_fixup_ethernet-add-logs.patch`:
- Around line 130-131: The patch file 1001-fdt_fixup_ethernet-add-logs.patch has
an unintended blank line being added before the SPDX-License-Identifier header
at the start of the file. Remove the blank line that appears before the SPDX
header so that the SPDX-License-Identifier comment becomes the first line of the
file, as required by license-scanning tools.
- Around line 92-103: The variable nodeoff is being redeclared at line 92 which
shadows the existing nodeoff variable declared at function scope (line 70 when
FDT_SEQ_MACADDR_FROM_ENV is defined), causing a redundant fdt_path_offset call.
Remove the variable declaration at line 92 and reuse the existing nodeoff
variable instead, assigning the result of fdt_path_offset directly to it.
Additionally, review the status check logic at lines 98-103: the current
condition strcmp((const char *)status_prop->data, "okay") only allows nodes with
status="okay" or no status property, but the previous behavior would patch any
node that is not "disabled". Determine whether this stricter check is
intentional for backward compatibility or if it should be changed to skip only
"disabled" nodes.

In `@patch/u-boot/v2026.07/cmd-fileenv-read-string-from-file-into-env.patch`:
- Around line 25-29: The CMD_FILEENV Kconfig configuration block is missing a
dependency declaration on CMD_FAT. Since the implementation relies on
do_fat_fsload() which is only available when CMD_FAT is enabled, add a
dependency statement within the config CMD_FILEENV block to declare the required
dependency on CMD_FAT. This will prevent link-time failures when enabling
CMD_FILEENV without FAT support.
- Around line 79-86: The sanitization loop exits immediately when it encounters
a non-printable character because it sets *s = 0x00 (null-terminating the
string) without advancing the pointer, causing the while condition to become
false on the next iteration. This truncates the string at the first
non-printable character rather than removing it. To fix this, either: (1) add a
comment to clarify that truncation at first non-printable is intentional
behavior, or (2) if the intent is to strip non-printables while preserving
subsequent content, use a two-pointer approach where you write printable
characters to a destination pointer while reading from source pointer s,
allowing the loop to continue processing the entire string and compact it by
removing non-printables.

---

Nitpick comments:
In `@patch/u-boot/v2026.07/cmd-fileenv-read-string-from-file-into-env.patch`:
- Line 57: The variable `fs_argv` is currently declared as a static global array
that persists across function calls, but since it does not need to retain its
value between invocations, convert it to a local variable. Move the declaration
`static char *fs_argv[5];` from the global scope into the `do_fileenv()`
function (removing the static keyword), placing it near the beginning of the
function before it is first used. Update all references to `fs_argv` within
`do_fileenv()` to work with the local variable.