David Gibson

> A possible followup improvement would be to restore phandles from `__fixup__` . I think you do need to implement this. For one thing processing but `__local_fixups__` but not `__fixups__`...

> Do you have an idea for the unaligned phandle problem? Sorry, I'm not sure exactly what problem you mean.

> > > Do you have an idea for the unaligned phandle problem? > > > > > > Sorry, I'm not sure exactly what problem you mean. > >...

The first two patches looked fine and I was going to apply.. but the second one breaks a test case: ``` dtc -I dtb -O dts -o overlay_overlay_decompile.test.dts overlay_overlay.test.dtb: PASS...

> This commit uncovers a problem that we already have since commit [915daad](https://github.com/dgibson/dtc/commit/915daadbb62d68bc49c5a14a360d1c39e0131f97) ("Start with empty **local_fixups** and **fixups** nodes"): > > ``` > $ cat test.dts > /dts-v1/; >...

Sorry, finally looking at this after far too long. The description of the first patch is misleading. Although, yes, this does allow use of `&label` for fdtput and fdtget, it...

> Well, first of all, that would be massively more complicated, as essentially all code paths ("subcommands" if you will) in the fdtget/fdtput utils would need to be taught to...

Huh, this looks like something I tried to fix with [2ad7387](https://github.com/dgibson/dtc/commit/2ad738722b79dbfccd4ddbe7408720bac3dba3d6). I generated that trying to fix problems I encountered adding FreeBSD tests back to Cirrus CI. I guess I've...

Sorry, I'm very confused by this reply. > This fails in Cirrus CI as well looking at the logs from latest commit of this repo I'm not seeing that in...

> 1. https://cirrus-ci.com/task/4962482919309312 --> Run setup --> "Compiler for C supports link arguments -Wl,--version-script=/tmp/cirrus-ci-build/libfdt/version.lds: NO" or am I missing something? That's working as intended. AIUI, the BSD linker doesn't support...