From patchwork Mon Jul 21 12:18:16 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ian Campbell X-Patchwork-Id: 33952 Return-Path: X-Original-To: linaro@patches.linaro.org Delivered-To: linaro@patches.linaro.org Received: from mail-vc0-f200.google.com (mail-vc0-f200.google.com [209.85.220.200]) by ip-10-151-82-157.ec2.internal (Postfix) with ESMTPS id AB50C20492 for ; Mon, 21 Jul 2014 12:20:25 +0000 (UTC) Received: by mail-vc0-f200.google.com with SMTP id hq11sf17684474vcb.3 for ; Mon, 21 Jul 2014 05:20:25 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:delivered-to:message-id:from:to:date:in-reply-to :references:organization:mime-version:cc:subject:precedence:list-id :list-unsubscribe:list-post:list-help:list-subscribe:sender :errors-to:x-original-sender:x-original-authentication-results :mailing-list:list-archive:content-type:content-transfer-encoding; bh=sxIZkAQATfRNaH/MwdiUCNeQfi1vyxM3JhQsmu0lNcg=; b=Eck0xEiFG08oGZhAUdEsXYNZL0+bGKqRUc7bd7ndp6OCgShH82QBHD9tHYlhh+z6+3 PQ8DCPg0kKgZWr5gGubv/+vB/59kJjFRCqlOpTGRork+TOunt8TdAkfJrlY9xQhLVzkG XYURmjfqfoGs/w6EQ6KF9SiUxiDFRZhsxy2+qIqfIDW3yhQ2z9TxkVYLRKHn4sLz4ycT bIEkzxDLaJoxHrprRPVVZ2/VmhAK7J2zkvsvxWOwfYSw503txdeTsdPvJS4+yDdwQvoF lSZdevc72Xcyyw37NjctOth4CDqovl/u6b/DiTyTJht9yWKOr2wthymsCmHMjfiLRccp wpvQ== X-Gm-Message-State: ALoCoQm6pA5ETbmQZsMHK1UyKgX8Rd89lRYmK7ffb/hI6lw32DySPbKXvfR8lQbCo1wFct/FRacf X-Received: by 10.236.137.13 with SMTP id x13mr11059262yhi.32.1405945225495; Mon, 21 Jul 2014 05:20:25 -0700 (PDT) X-BeenThere: patchwork-forward@linaro.org Received: by 10.140.87.37 with SMTP id q34ls1995499qgd.5.gmail; Mon, 21 Jul 2014 05:20:25 -0700 (PDT) X-Received: by 10.221.38.129 with SMTP id ti1mr17456509vcb.9.1405945225293; Mon, 21 Jul 2014 05:20:25 -0700 (PDT) Received: from mail-vc0-f179.google.com (mail-vc0-f179.google.com [209.85.220.179]) by mx.google.com with ESMTPS id ft1si11172069vec.94.2014.07.21.05.20.25 for (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Mon, 21 Jul 2014 05:20:25 -0700 (PDT) Received-SPF: pass (google.com: domain of patch+caf_=patchwork-forward=linaro.org@linaro.org designates 209.85.220.179 as permitted sender) client-ip=209.85.220.179; Received: by mail-vc0-f179.google.com with SMTP id hq11so10257847vcb.24 for ; Mon, 21 Jul 2014 05:20:25 -0700 (PDT) X-Received: by 10.221.26.10 with SMTP id rk10mr28826074vcb.0.1405945225198; Mon, 21 Jul 2014 05:20:25 -0700 (PDT) X-Forwarded-To: patchwork-forward@linaro.org X-Forwarded-For: patch@linaro.org patchwork-forward@linaro.org Delivered-To: patch@linaro.org Received: by 10.221.37.5 with SMTP id tc5csp107333vcb; Mon, 21 Jul 2014 05:20:24 -0700 (PDT) X-Received: by 10.52.36.131 with SMTP id q3mr1089040vdj.90.1405945224656; Mon, 21 Jul 2014 05:20:24 -0700 (PDT) Received: from lists.xen.org (lists.xen.org. [50.57.142.19]) by mx.google.com with ESMTPS id k8si11162889vdv.58.2014.07.21.05.20.24 for (version=TLSv1 cipher=RC4-SHA bits=128/128); Mon, 21 Jul 2014 05:20:24 -0700 (PDT) Received-SPF: none (google.com: xen-devel-bounces@lists.xen.org does not designate permitted sender hosts) client-ip=50.57.142.19; Received: from localhost ([127.0.0.1] helo=lists.xen.org) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1X9CXv-0006ov-7o; Mon, 21 Jul 2014 12:18:23 +0000 Received: from mail6.bemta3.messagelabs.com ([195.245.230.39]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1X9CXt-0006oq-Ql for xen-devel@lists.xen.org; Mon, 21 Jul 2014 12:18:21 +0000 Received: from [85.158.137.68:43799] by server-2.bemta-3.messagelabs.com id D2/82-09149-D050DC35; Mon, 21 Jul 2014 12:18:21 +0000 X-Env-Sender: Ian.Campbell@citrix.com X-Msg-Ref: server-5.tower-31.messagelabs.com!1405945098!17106632!1 X-Originating-IP: [66.165.176.63] X-SpamReason: No, hits=0.0 required=7.0 tests=sa_preprocessor: VHJ1c3RlZCBJUDogNjYuMTY1LjE3Ni42MyA9PiAzMDYwNDg=\n X-StarScan-Received: X-StarScan-Version: 6.11.3; banners=-,-,- X-VirusChecked: Checked Received: (qmail 8213 invoked from network); 21 Jul 2014 12:18:20 -0000 Received: from smtp02.citrix.com (HELO SMTP02.CITRIX.COM) (66.165.176.63) by server-5.tower-31.messagelabs.com with RC4-SHA encrypted SMTP; 21 Jul 2014 12:18:20 -0000 X-IronPort-AV: E=Sophos;i="5.01,700,1400025600"; d="scan'208";a="154638659" Received: from accessns.citrite.net (HELO FTLPEX01CL03.citrite.net) ([10.9.154.239]) by FTLPIPO02.CITRIX.COM with ESMTP; 21 Jul 2014 12:18:18 +0000 Received: from kazak.uk.xensource.com (10.80.2.80) by FTLPEX01CL03.citrite.net (10.13.107.80) with Microsoft SMTP Server id 14.3.181.6; Mon, 21 Jul 2014 08:18:17 -0400 Message-ID: <1405945096.25022.46.camel@kazak.uk.xensource.com> From: Ian Campbell To: Julien Grall Date: Mon, 21 Jul 2014 13:18:16 +0100 In-Reply-To: <53CCFF1C.10607@linaro.org> References: <1405688876.491.10.camel@kazak.uk.xensource.com> <1405688900-11769-9-git-send-email-ian.campbell@citrix.com> <53C98BB4.5070302@linaro.org> <1405943128.25022.37.camel@kazak.uk.xensource.com> <53CCFF1C.10607@linaro.org> Organization: Citrix Systems, Inc. X-Mailer: Evolution 3.12.2-1 MIME-Version: 1.0 X-Originating-IP: [10.80.2.80] X-DLP: MIA1 Cc: stefano.stabellini@eu.citrix.com, tim@xen.org, xen-devel@lists.xen.org Subject: Re: [Xen-devel] [PATCH v3 09/10] xen: arm: update multiboot device tree bindings. X-BeenThere: xen-devel@lists.xen.org X-Mailman-Version: 2.1.13 Precedence: list List-Id: List-Unsubscribe: , List-Post: , List-Help: , List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org X-Removed-Original-Auth: Dkim didn't pass. X-Original-Sender: ian.campbell@citrix.com X-Original-Authentication-Results: mx.google.com; spf=pass (google.com: domain of patch+caf_=patchwork-forward=linaro.org@linaro.org designates 209.85.220.179 as permitted sender) smtp.mail=patch+caf_=patchwork-forward=linaro.org@linaro.org Mailing-list: list patchwork-forward@linaro.org; contact patchwork-forward+owners@linaro.org X-Google-Group-Id: 836684582541 List-Archive: On Mon, 2014-07-21 at 12:53 +0100, Julien Grall wrote: > On 07/21/2014 12:45 PM, Ian Campbell wrote: > > On Fri, 2014-07-18 at 22:03 +0100, Julien Grall wrote: > >> Hi Ian, > >> > >> On 18/07/14 14:08, Ian Campbell wrote: > >>> Signed-off-by: Ian Campbell > >> > >> Acked-by: Julien Grall > > > > Thanks. I've applied patches 1..9 of this series. I'll resend 10 ASAP > > with the change you've suggested. > > > > On my potential backports list I've got: > > xen: arm: implement generic multiboot compatibility strings > > xen: arm: /chosen/module@N/bootargs bootprotcol node is not deprecated > > > > I don't think anything else here is suitable for backport. Let me know > > if you think something is (or isn't) > > I would also update the document bindings in > docs/misc/arm/device-tree/booting.txt. > > The patch #9 looks the good one for this purpose. Minus the unspecified > type example. I think it is acceptable to point people to the latest version of the doc in the dev branch as the canonical copy. The latest version already needs to properly describe the mechanisms for backwards compatibility anyway and trying to backport only the docs updates which match backported bits of code is liable to get fiddly quite fast and/or require new patches etc, I'd rather not do this. > BTW, I don't find anything in the documentation talking about bootmodule > type detection by ordering. Is this intended? No, I should have done this and forgot, thanks for the reminder. See below. Ian. 8<-------------------------- >From f9e80ead57b9f739c3041fe5abc4b23c8f0eb18f Mon Sep 17 00:00:00 2001 From: Ian Campbell Date: Mon, 21 Jul 2014 13:16:31 +0100 Subject: [PATCH] xen: arm: document boot module compatibility based on ordering Signed-off-by: Ian Campbell Acked-by: Julien Grall Acked-by: Julien Grall --- docs/misc/arm/device-tree/booting.txt | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/docs/misc/arm/device-tree/booting.txt b/docs/misc/arm/device-tree/booting.txt index d967061..ad98bf3 100644 --- a/docs/misc/arm/device-tree/booting.txt +++ b/docs/misc/arm/device-tree/booting.txt @@ -23,7 +23,13 @@ Each node contains the following properties: compatible string (if one applies) in addition to the generic string (which must always be present). - Xen 4.4 supported a different set of legacy compatible strings + Xen will assume that the first module which lacks a more + specific compatible string is a "multiboot,kernel" and that + the second such is a "multiboot,ramdisk". Any subsequent + modules which lack a specific compatiblity string will not + receive any special treatment. + + Xen 4.4 supported a different set of legacy compatible strings which remain supported such that systems supporting both 4.4 and later can use a single DTB.