On Thu, Sep 30, 2021 at 01:56:33PM +0200, Greg Kurz wrote: > On Thu, 30 Sep 2021 09:10:12 +0200 > Thomas Huth wrote: > > > On 27/08/2021 14.09, Thomas Huth wrote: > > > The dtc submodule is currently pointing to non-release commit. It's nicer > > > if submodules point to release versions instead and since dtc 1.6.1 is > > > available now, let's update to that version. > > > > > > Signed-off-by: Thomas Huth > > > --- > > > > dtc | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git a/dtc b/dtc > > > index 85e5d83984..b6910bec11 160000 > > > --- a/dtc > > > +++ b/dtc > > > @@ -1 +1 @@ > > > -Subproject commit 85e5d839847af54efab170f2b1331b2a6421e647 > > > +Subproject commit b6910bec11614980a21e46fbccc35934b671bd81 > > > > Ping! > > > > David, could you please pick up this patch if you don't mind it? > > > > Thanks, > > Thomas > > > > Hi Thomas ! > > greg@bahia:[main]dtc$ git log --oneline 85e5d839847af54efab170f2b1331b2a6421e647..v1.6.1 > b6910bec1161 Bump version to v1.6.1 > 21d61d18f968 Fix CID 1461557 > 4c2ef8f4d14c checks: Introduce is_multiple_of() > e59ca36fb70e Make handling of cpp line information more tolerant > 0c3fd9b6aceb checks: Drop interrupt_cells_is_cell check > 6b3081abc4ac checks: Add check_is_cell() for all phandle+arg properties > 2dffc192a77f yamltree: Remove marker ordering dependency > 61e513439e40 pylibfdt: Rework "avoid unused variable warning" lines > c8bddd106095 tests: add a positive gpio test case > ad4abfadb687 checks: replace strstr and strrchr with strends > 09c6a6e88718 dtc.h: add strends for suffix matching > 9bb9b8d0b4a0 checks: tigthen up nr-gpios prop exception > b07b62ee3342 libfdt: Add FDT alignment check to fdt_check_header() > a2def5479950 libfdt: Check that the root-node name is empty > 4ca61f84dc21 libfdt: Check that there is only one root node > 34d708249a91 dtc: Remove -O dtbo support > 8e7ff260f755 libfdt: Fix a possible "unchecked return value" warning > 88875268c05c checks: Warn on node-name and property name being the same > 9d2279e7e6ee checks: Change node-name check to match devicetree spec > f527c867a8c6 util: limit gnu_printf format attribute to gcc >= 4.4.0 > 183df9e9c2b9 gitignore: Ignore the swp files > 0db6d09584e1 gitignore: Add cscope files > 307afa1a7be8 Update Jon Loeliger's email > ca16a723fa9d fdtdump: Fix gcc11 warning > 64990a272e8f srcpos: increase MAX_SRCFILE_DEPTH > 163f0469bf2e dtc: Allow overlays to have .dtbo extension > 3b01518e688d Set last_comp_version correctly in new dtb and fix potential version issues in fdt_open_into > f7e5737f26aa tests: Fix overlay_overlay_nosugar test case > 7cd5d5fe43d5 libfdt: Tweak description of assume-aligned load helpers > a7c404099349 libfdt: Internally perform potentially unaligned loads > bab85e48a6f4 meson: increase default timeout for tests > f8b46098824d meson: do not assume python is installed, skip tests > 30a56bce4f0b meson: fix -Wall warning > 5e735860c478 libfdt: Check for 8-byte address alignment in fdt_ro_probe_() > 67849a327927 build-sys: add meson build > 05874d08212d pylibfdt: allow build out of tree > 3bc3a6b9fe0c dtc: Fix signedness comparisons warnings: Wrap (-1) > e1147b159e92 dtc: Fix signedness comparisons warnings: change types > 04cf1fdc0fcf convert-dtsv0: Fix signedness comparisons warning > b30013edb878 libfdt: Fix kernel-doc comments > cbca977ea121 checks: Allow PCI bridge child nodes without an address > 73e0f143b73d libfdt: fdt_strerror(): Fix comparison warning > 6c2be7d85315 libfdt: fdt_get_string(): Fix sequential write comparison warnings > 82525f41d59e libfdt: libfdt_wip: Fix comparison warning > fb1f65f15832 libfdt: fdt_create_with_flags(): Fix comparison warning > f28aa271000b libfdt: fdt_move(): Fix comparison warnings > 3d7c6f44195a libfdt: fdt_add_string_(): Fix comparison warning > 10f682788c30 libfdt: fdt_node_offset_by_phandle(): Fix comparison warning > 07158f4cf2a2 libfdt: overlay: Fix comparison warning > ce9e1f25a7de libfdt: fdt_resize(): Fix comparison warning > faa76fc10bc5 libfdt: fdt_splice_(): Fix comparison warning > 54dca0985316 libfdt: fdt_get_string(): Fix comparison warnings > f8e11e61624e libfdt: fdt_grab_space_(): Fix comparison warning > 0c43d4d7bf5a libfdt: fdt_mem_rsv(): Fix comparison warnings > 442ea3dd1579 libfdt: fdt_offset_ptr(): Fix comparison warnings > ca19c3db2bf6 Makefile: Specify cflags for libyaml > 7bb86f1c0956 libfdt: fix fdt_check_node_offset_ w/ VALID_INPUT > 3d522abc7571 dtc: Include stdlib.h in util.h > 808cdaaf524f dtc: Avoid UB when shifting > 3e3138b4a956 libfdt: fix fdt_check_full buffer overrun > 9d7888cbf19c dtc: Consider one-character strings as strings > 8259d59f59de checks: Improve i2c reg property checking > fdabcf2980a4 checks: Remove warning for I2C_OWN_SLAVE_ADDRESS > 2478b1652c8d libfdt: add extern "C" for C++ > f68bfc2668b2 libfdt: trivial typo fix > 7be250b4d059 libfdt: Correct condition for reordering blocks > 81e0919a3e21 checks: Add interrupt provider test > > I don't see anything obvious that would preclude from updating > to v1.6.1, but I don't think either David or I will post another > PR anytime soon. Maybe better if this goes through someone else's > tree. > > FWIW I could successfully build QEMU using the updated submodule, > so as discussed on IRC : > > Acked-by: Greg Kurz > > David will hopefully nack it in time if he's aware of an issue. No, it should be fine. Acked-by: David Gibson -- David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson