From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Lunn Subject: Re: [PATCH] of: support passing console options with stdout-path Date: Tue, 25 Nov 2014 17:21:21 +0100 Message-ID: <20141125162121.GC3443@lunn.ch> References: <1416867838-18652-1-git-send-email-leif.lindholm@linaro.org> <20141124230016.GP20238@lunn.ch> <20141125144439.07212C44343@trevor.secretlab.ca> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20141125144439.07212C44343-WNowdnHR2B42iJbIjFUEsiwD8/FfD2ys@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Grant Likely Cc: Andrew Lunn , Leif Lindholm , mark.rutland-5wv7dgnIgG8@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, ijc-8fiUuRrzOP0dnm+yROfE0A@public.gmane.org, plagnioj-sclMFOaUSTBWk0Htik3J/w@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org List-Id: devicetree@vger.kernel.org > > More worrying, barebox does not support the : either. So there is a > > danger your bootloader suddenly goes silent after a dt blob update. > > Is barebox parsing the property? Or updating it? Or both? I've got no > problem with this patch, but we can also support things like the > 'current-speed' property in the UART to set a default when the stdout > path doesn't have any arguments. Hi Grant I'm no export with barebox. So i just grep'ed the code. It parses the property. Also as with linux, of_find_node_by_path() does not stop at : as you say it should. There is one case of the property being updated, but that is only for the PPC mpc85xx. Barebox does not appear to use 'current-speed' at all. Andrew -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html