bfd1c45e54
Previously whenever an overlay was found to be incompatible with a base device tree blob, the entire base dtb would be skipped in favor of processing the next one. This had the unfortunate effect where overlays would not fully be applied if any incompatibility was found. For example, this is an issue with build device trees specific for one flavor of raspberry pi if the overlay was not compatible _everywhere_. The solution is to forego the `continue` keyword if an overlay is in compatible and instead use a compound conditional statement to skip incompatible overlays but continue trying to apply it to any remaining dtbs.
44 lines
1.4 KiB
Nix
44 lines
1.4 KiB
Nix
{ lib, stdenvNoCC, dtc }:
|
|
|
|
with lib; {
|
|
applyOverlays = (base: overlays': stdenvNoCC.mkDerivation {
|
|
name = "device-tree-overlays";
|
|
nativeBuildInputs = [ dtc ];
|
|
buildCommand = let
|
|
overlays = toList overlays';
|
|
in ''
|
|
mkdir -p $out
|
|
cd "${base}"
|
|
find . -type f -name '*.dtb' -print0 \
|
|
| xargs -0 cp -v --no-preserve=mode --target-directory "$out" --parents
|
|
|
|
for dtb in $(find "$out" -type f -name '*.dtb'); do
|
|
dtbCompat=$(fdtget -t s "$dtb" / compatible 2>/dev/null || true)
|
|
# skip files without `compatible` string
|
|
test -z "$dtbCompat" && continue
|
|
|
|
${flip (concatMapStringsSep "\n") overlays (o: ''
|
|
overlayCompat="$(fdtget -t s "${o.dtboFile}" / compatible)"
|
|
|
|
# skip incompatible and non-matching overlays
|
|
if [[ ! "$dtbCompat" =~ "$overlayCompat" ]]; then
|
|
echo "Skipping overlay ${o.name}: incompatible with $(basename "$dtb")"
|
|
elif ${if (o.filter == null) then "false" else ''
|
|
[[ "''${dtb//${o.filter}/}" == "$dtb" ]]
|
|
''}
|
|
then
|
|
echo "Skipping overlay ${o.name}: filter does not match $(basename "$dtb")"
|
|
else
|
|
echo -n "Applying overlay ${o.name} to $(basename "$dtb")... "
|
|
mv "$dtb"{,.in}
|
|
fdtoverlay -o "$dtb" -i "$dtb.in" "${o.dtboFile}"
|
|
echo "ok"
|
|
rm "$dtb.in"
|
|
fi
|
|
'')}
|
|
|
|
done
|
|
'';
|
|
});
|
|
}
|