On Wed, Nov 27, 2019 at 01:35:21PM +0100, Ahmad Fatoum wrote: > Hello Sascha, > > On 11/27/19 1:32 PM, Sascha Hauer wrote: > > barebox contains functions and commands to change and delete properties > > and nodes. We shouldn't store pointers into the live tree anywhere. > > aren't these fixups done just before boot? Yes. > What users change the live device tree? We already had problems with users deleting nodes that were referenced at other places. I'm not saying this is wise to do so, but it happened. It might be a good idea to separate the live tree from the one the user actually sees, but then again things like of_device_enable_path() and of_device_enable_and_register_by_name() wouldn't work anymore. Sascha -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox