Re: [PATCH 00/17] pylibfdt: Update the U-Boot version with changes from upstream dtc

Tom Rini <[email protected]>
Newsgroups org.u-boot-project.lists.u-boot
Message-ID <20260824181658.GL523975@bill-the-cat>
On Mon, Aug 24, 2026 at 05:12:57PM +0400, Alexey Charkov wrote:

> U-Boot's copy of the libfdt Python bindings in
> scripts/dtc/pylibfdt/libfdt.i_shipped was last synced wholesale in 2019,
> by commit 903fe17aa8c8 ("pylibfdt: Sync up with upstream"), against dtc
> commit 430419c28100. Since then it has only picked up individual fixes,
> so it has drifted a long way behind dtc, which is now at v1.8.1-19.
> 
> Tom asked for an audit of the divergence when reviewing the FdtSw growth
> patch [1]; this is the result. Bring in every missing change from upstream
> as an individual commit, referencing its upstream sibling. 
> 
> This adds: hasprop() and setprop_bool(), get_path(),
> Property.as_stringlist() and the as_*int*_list() accessors,
> address_cells()/size_cells(), add_mem_rsv()/del_mem_rsv(), geometric FdtSw
> buffer growth, a missing Py_INCREF on the Py_None returned by the
> fdt_getprop() typemap, correct get_mem_rsv() results on current Python,
> and the SWIG 4.3+ return-value compatibility shim.
> 
> Not included is dtc commit 5008d1d6a356 ("pylibfdt: Replace removed SWIG
> Python 2 compatibility macros"), which already landed independently as
> commit 527115ef6783 ("pylibfdt: Replace removed SWIG Python 2
> compatibility macros").
> 
> The first two patches are preparatory: they take the parts of two upstream
> commits that earlier partial imports left behind, so the rest apply
> verbatim. The last patch adds test coverage, since none of these bindings
> have an in-tree caller yet and dtc's own tests for them live in
> tests/pylibfdt_tests.py, which U-Boot does not carry.

Thanks for doing this update.

-- 
Tom
signature.asc (application/pgp-signature, 228 B)
-----BEGIN PGP SIGNATURE-----

iHUEABYKAB0WIQTzzqh0PWDgGS+bTHor4qD1Cr/kCgUCaoyKlgAKCRAr4qD1Cr/k
CrA9AQCLGz9HEqhxZ++KhkLsXp1WMPRLAMTMNRvk6oUR7EPtggEApeslaLGDESWg
f1KzWIS34m4S/v+kxBR/zMLzce4fQw8=
=nxHX
-----END PGP SIGNATURE-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.