Received: by 2002:a25:ad19:0:0:0:0:0 with SMTP id y25csp62739ybi; Thu, 1 Aug 2019 14:40:45 -0700 (PDT) X-Google-Smtp-Source: APXvYqzUo6Wr7LbSVsJ1EntAiIDNf5m4fZ0EuQDCEbyIBQfaTfNKufqM0DnpPA5chFClfONJf8mq X-Received: by 2002:a63:481c:: with SMTP id v28mr8906259pga.50.1564695645039; Thu, 01 Aug 2019 14:40:45 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1564695645; cv=none; d=google.com; s=arc-20160816; b=h6cP/Gn41Sn8/UNYXdS8wihHfIe9JlesMTk80XqhnWMPTOsCHDcp8M2ZYjqLJ5gKsT /dgnQJd4nxLrw+5ygyu/9m54hDad3WzYQYY4HYPU9F4s6sZ9uATykNYCCywfZg3rhTV4 h9qLhZJ9Komz605VlK/HfmRftMzt7vLpKosu+SaRkFP+7jcqTt9XmGKE6WyI7a7hUqaL ULOfQ+yt99PJa7Ywoovu9DbQNHQeMMbazEn/6/u2BCk6a+zyCB7mRsDKbhj+IWIh3RN0 Rmsq0ttT31AvJEsg+IIupsjYXrGxCgG1+HZwFPWJEaDxiWgQLz/Kox/LIJFomrT9DqDb CsJg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:dkim-signature; bh=vuaVHewX2373t+xnC+Iap1nqe3pd2bZCWl+GzakvMfk=; b=p5xf+4YxhWtg3JEzgahDZONANsWuQngToQi8G7j/r3+cBFqup3tx7ZQGP1nHWqTZCP cIkF2HzhYoFiPXHhKG9n3p9sD55wBrzqSvqe3v55a23Yy3TSBsOvfgZYZwNhr0HyDTIj WaqTH5DJtpDtw9mk+XwEM3CSPx8wpjnyGkc4zFzPb6iw4v/QtKYMwSxjIk1tZqfofPt5 KxlorMpWWiSS498PViYNuC7A2uCaLT+1KSwpZSKojNHn3ctPED6R8uj/vbJ81kf3To9L AHQxejpKT1fAN41F4vKW6bDC5aHx8pHHw7Gxl2bawz7uSKF7hhBBKq65b9bLos0NuN7h n/9w== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=NXLsozSY; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id 22si35383794pgu.226.2019.08.01.14.40.29; Thu, 01 Aug 2019 14:40:45 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=NXLsozSY; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1731470AbfHAPuw (ORCPT + 99 others); Thu, 1 Aug 2019 11:50:52 -0400 Received: from mail.kernel.org ([198.145.29.99]:48782 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1731079AbfHAPuw (ORCPT ); Thu, 1 Aug 2019 11:50:52 -0400 Received: from mail-qt1-f175.google.com (mail-qt1-f175.google.com [209.85.160.175]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 285B821773; Thu, 1 Aug 2019 15:50:51 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1564674651; bh=mo2HdMSRCqt3TFJkQl9d7X3tKGeVpM+cbgnJbsSOiBU=; h=References:In-Reply-To:From:Date:Subject:To:Cc:From; b=NXLsozSYOIx9rgmDZGvJH1orQh3Sih5ok+LDLGRki2c9GNYyHcBujEiSfTrb4L0hE Kq/zLCt7vLGKd95Jmjf+KBCOpxM7BmwGOZi5zcqTEOBKLZXpvIR9vZWaZOFuOFOAp4 nKbypj4GdUDLiW2dD2yPDj6ZdsQV8WJLxPGE3UsA= Received: by mail-qt1-f175.google.com with SMTP id k10so1562479qtq.1; Thu, 01 Aug 2019 08:50:51 -0700 (PDT) X-Gm-Message-State: APjAAAVyg7KFyXzeE7nAjgxIEhUb9rc3CuK/OEHuRDwbUAb+hdJkQbBi GnMoegWd4iqSHVypdqjAKtBfSAoiPNd8TCTr7A== X-Received: by 2002:a0c:acef:: with SMTP id n44mr94852273qvc.39.1564674650265; Thu, 01 Aug 2019 08:50:50 -0700 (PDT) MIME-Version: 1.0 References: <20190801005843.10343-1-atish.patra@wdc.com> <20190801005843.10343-6-atish.patra@wdc.com> In-Reply-To: <20190801005843.10343-6-atish.patra@wdc.com> From: Rob Herring Date: Thu, 1 Aug 2019 09:50:37 -0600 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH v3 5/5] dt-bindings: Update the riscv,isa string description To: Atish Patra Cc: "linux-kernel@vger.kernel.org" , Paul Walmsley , Albert Ou , Allison Randal , Anup Patel , Daniel Lezcano , devicetree@vger.kernel.org, Enrico Weigelt , Gary Guo , Greg Kroah-Hartman , Johan Hovold , linux-riscv@lists.infradead.org, Mark Rutland , Palmer Dabbelt , Thomas Gleixner , Yangtao Li Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Jul 31, 2019 at 6:58 PM Atish Patra wrote: > > Since the RISC-V specification states that ISA description strings are > case-insensitive, there's no functional difference between mixed-case, > upper-case, and lower-case ISA strings. Thus, to simplify parsing, > specify that the letters present in "riscv,isa" must be all lowercase. > > Suggested-by: Paul Walmsley > Signed-off-by: Atish Patra > --- > Documentation/devicetree/bindings/riscv/cpus.yaml | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/riscv/cpus.yaml b/Documentation/devicetree/bindings/riscv/cpus.yaml > index c899111aa5e3..4f0acb00185a 100644 > --- a/Documentation/devicetree/bindings/riscv/cpus.yaml > +++ b/Documentation/devicetree/bindings/riscv/cpus.yaml > @@ -46,10 +46,12 @@ properties: > - rv64imafdc > description: > Identifies the specific RISC-V instruction set architecture > - supported by the hart. These are documented in the RISC-V > + supported by the hart. These are documented in the RISC-V > User-Level ISA document, available from > https://riscv.org/specifications/ > > + Letters in the riscv,isa string must be all lowercase. > + The schemas are case sensitive this looks pretty pointless without the context of the commit msg. Can you prefix with 'While the specification is case insensitive, " For some background, FDT generally always has been case sensitive too (dtc won't merge/override nodes/properties with differing case). It's really only some older true OF systems that were case insensitive. The kernel had a mixture of case sensitive and insensitive comparisons somewhat depending on the arch and whether of_prop_cmp/of_node_cmp or str*cmp functions were used. There's been a lot of clean-up and now most comparisons are case sensitive with only Sparc having some deviation. Rob