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-----