All of lore.kernel.org
 help / color / mirror / Atom feed
From: York Sun <yorksun@freescale.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 2/2] fdt_support: Don't panic if stdout alias is missing
Date: Fri, 30 Oct 2015 09:12:12 -0700	[thread overview]
Message-ID: <563396DC.3030903@freescale.com> (raw)
In-Reply-To: <1441165688-28917-1-git-send-email-scottwood@freescale.com>



On 09/01/2015 08:48 PM, Scott Wood wrote:
> Currently, using fdt_fixup_stdout() on a device tree that is missing
> the relevant alias results in this:
> 
> WARNING: could not set linux,stdout-path FDT_ERR_NOTFOUND.
> ERROR: /chosen node create failed
>  - must RESET the board to recover.
> 
> FDT creation failed! hanging...### ERROR ### Please RESET the board ###
> 
> There is no reason for this to be a fatal error rather than a warning,
> and removing this allows for a smooth transition on a platform where
> the device tree currently lacks the correct aliases but will have them
> in the future.
> 
> Signed-off-by: Scott Wood <scottwood@freescale.com>
> Cc: Kumar Gala <galak@kernel.crashing.org>
> Cc: Simon Glass <sjg@chromium.org>
> ---
> v2: Only continue booting if the problem was a missing alias, not
> the inability to write to the device tree.
> ---
>  common/fdt_support.c | 11 ++++++++---
>  1 file changed, 8 insertions(+), 3 deletions(-)
> 


Applied to u-boot-fsl-qoriq. Awaiting upstream. Thanks.

York

  parent reply	other threads:[~2015-10-30 16:12 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-09-01  2:05 [U-Boot] [PATCH 1/2] arm/fsl-ls: Add CONFIG_OF_STDOUT_VIA_ALIAS Scott Wood
2015-09-01  2:05 ` [U-Boot] [PATCH 2/2] fdt_support: Don't panic if unable to perform stdout fixup Scott Wood
2015-09-01  2:11 ` Scott Wood
2015-09-01  3:13   ` Simon Glass
2015-09-01  3:16     ` Scott Wood
2015-09-02  2:48       ` Simon Glass
2015-09-02  3:00         ` Scott Wood
2015-09-02  3:10           ` Simon Glass
2015-09-02  3:14             ` Scott Wood
2015-09-02  3:04         ` York Sun
2015-09-02  3:06           ` Scott Wood
2015-09-02  3:48   ` [U-Boot] [PATCH v2 2/2] fdt_support: Don't panic if stdout alias is missing Scott Wood
2015-09-02 14:05     ` Simon Glass
2015-10-30 16:12     ` York Sun [this message]
2015-10-30 16:11 ` [U-Boot] [PATCH 1/2] arm/fsl-ls: Add CONFIG_OF_STDOUT_VIA_ALIAS York Sun

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=563396DC.3030903@freescale.com \
    --to=yorksun@freescale.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.