On Fri, Jul 28, 2023 at 10:50:26AM +0200, Geert Uytterhoeven wrote: > Hi all, > > This patch series contains miscellaneous fixes and improvements for > dynamic DT overlays and the related unit tests. > > The first two patches are fixes for a lock-up and a crash. > The remaining patches are smaller fixes, enhancements and cleanups for > the overlay tests, including one new test. > > I ran into the crash when accidentally loading the wrong overlay (using > the out-of-tree DT overlay configfs[1]), and removing it afterwards. > As this case was not yet covered by the unittests, I added a test. > I enhanced the tests to clean up partial state after a failed > overlay apply attempt, which triggered the lock-up. > > Changes compared to v1[2]: > - Correct fixes tag and update description. > - Merge differently, as requested by Rob. > > Thanks for your comments! > > [1] https://elinux.org/R-Car/DT-Overlays > [2] https://lore.kernel.org/r/cover.1689776064.git.geert+renesas@xxxxxxxxx > > Geert Uytterhoeven (13): > of: dynamic: Do not use "%pOF" while holding devtree_lock > of: overlay: Call of_changeset_init() early > of: unittest: Fix overlay type in apply/revert check > of: unittest: Restore indentation in overlay_bad_add_dup_prop test > of: unittest: Improve messages and comments in apply/revert checks > of: unittest: Merge of_unittest_apply{,_revert}_overlay_check() > of: unittest: Cleanup partially-applied overlays > of: unittest: Add separators to of_unittest_overlay_high_level() > of: overlay: unittest: Add test for unresolved symbol > of: unittest-data: Convert remaining overlay DTS files to sugar syntax > of: unittest-data: Fix whitespace - blank lines > of: unittest-data: Fix whitespace - indentation > of: unittest-data: Fix whitespace - angular brackets I've applied patches 2-13. For patch 1, I sent an alternative[1]. Rob [1] https://lore.kernel.org/all/20230728231950.1619073-1-robh@xxxxxxxxxx/