From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paolo Bonzini Subject: Re: [kvm-unit-tests PATCH v2 0/2] lib: devicetree: add stdout-path Date: Wed, 29 Jul 2015 15:55:20 +0200 Message-ID: <55B8DB48.90404@redhat.com> References: <1438155550-6676-1-git-send-email-drjones@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit To: Andrew Jones , kvm@vger.kernel.org Return-path: Received: from mx1.redhat.com ([209.132.183.28]:35826 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753557AbbG2NzY (ORCPT ); Wed, 29 Jul 2015 09:55:24 -0400 Received: from int-mx11.intmail.prod.int.phx2.redhat.com (int-mx11.intmail.prod.int.phx2.redhat.com [10.5.11.24]) by mx1.redhat.com (Postfix) with ESMTPS id 0BC563500F2 for ; Wed, 29 Jul 2015 13:55:24 +0000 (UTC) In-Reply-To: <1438155550-6676-1-git-send-email-drjones@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: On 29/07/2015 09:39, Andrew Jones wrote: > Easy addition to the devicetree support that may come in handy > for powerpc (and it may not - I have a ppc series ready to post > that doesn't actually bother with stdout-path yet, but whatever...) > > For v2 I decided it's better to return -FDT_ERR_BADPATH from > dt_get_default_console_node, if there's a bad path. Also, shortened > an overly long name in lib/arm/io.c > > Andrew Jones (2): > devicetree: add /chosen/stdout-path support > arm/arm64: uart0_init: check /chosen/stdout-path > > lib/arm/io.c | 36 +++++++++++++++++++++++++++--------- > lib/devicetree.c | 19 +++++++++++++++++++ > lib/devicetree.h | 9 +++++++++ > 3 files changed, 55 insertions(+), 9 deletions(-) > Looks good. I haven't yet tested it, but I plan on applying it as soon as I do. Paolo