On Thu, Dec 17, 2015 at 09:13:25PM +0000, Trent Piepho wrote: > On Thu, 2015-12-17 at 11:07 +0100, Sascha Hauer wrote: > > > / { > > > + chosen { > > > + environment@0 { > > > + compatible = "barebox,environment"; > > > + device-path = &mmc, "partname:1"; > > > + file-path = "barebox.env"; > > > + }; > > > + }; > > > + > > > > I applied this series as is. We might have to move this to the board dts > > files later as putting this into the SoC dtsi file does not allow the > > boards to put the environment somewhere else. That was not possible > > before this patch either, so this doesn't change much at the moment. > > It's possible to change in the board dts by adding a > "/chosen/environment@0" node to the dts with whatever properties should > be overridden from the dtsi file's values. Like how status = "disabled" > in dtsi is changed to status = "okay" in the board file. I do this for > my board as "partname:1" isn't correct for me. We might also have to delete the file-path property in a board dts. I just learned that this indeed works with /delete-property/ in dtc, but I am not sure if we should really use this, the deep levels of device tree includes are already hard enough to track (Not necessarily for socfpga, but have a look at some i.MX6 SoMs) Sascha -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 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