From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-14.0 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,INCLUDES_CR_TRAILER,INCLUDES_PATCH,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id B320CC433B4 for ; Fri, 2 Apr 2021 18:07:48 +0000 (UTC) Received: from desiato.infradead.org (desiato.infradead.org [90.155.92.199]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 446B761103 for ; Fri, 2 Apr 2021 18:07:48 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 446B761103 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=desiato.20200630; h=Sender:Content-Transfer-Encoding :Content-Type:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:Message-ID: Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=6cvoKuKSygBdYo33zz3QNELgd4G7GIPzqApIkh/L4ps=; b=VsH/3G87cBL0z5iRnIpPIUj3X Phjxv0RtvZXrItUW96bACQ84OTPHUsv94NLsa28J7APJIgmri79w8Ux+LueE231XPUAu/HAFjwQAS +cHYIWzCSy7pIUj9NIvk7Rbrk7pEPvfT7FS5vs3I3luSC1rM/i/93eDt8uHVcUikXZGQHwyzZPYMl pFr7NRWLa0r0Ej03UK0g9dPd1yXj2W+d8zDhSHaGaaA0/3HUOWGrQj4yVN4vhdnVDhLn/ERqOA365 mDnB+5j3csqOLYYLrAq3OOVKq6QyY8cCQK9ejLBlgnSTN91Ntx8cqirsdE10BchNu1Jv4N4alRXhr Lb0w6j7ow==; Received: from localhost ([::1] helo=desiato.infradead.org) by desiato.infradead.org with esmtp (Exim 4.94 #2 (Red Hat Linux)) id 1lSOBY-00DY6m-Vn; Fri, 02 Apr 2021 18:06:22 +0000 Received: from mail.kernel.org ([198.145.29.99]) by desiato.infradead.org with esmtps (Exim 4.94 #2 (Red Hat Linux)) id 1lSOBT-00DY5q-Dp; Fri, 02 Apr 2021 18:06:17 +0000 Received: by mail.kernel.org (Postfix) with ESMTPSA id 5798E610A5; Fri, 2 Apr 2021 18:06:11 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1617386774; bh=g6R99wNEiaWYjUZA/Zr//MAlmyK7L1YWL5mD/61jbv4=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=EGaw/CfI/k5x+byWWJbrGybqVaGb31AIwkEo5rGJ1eH835kv0HMdK73qwOtsRY6Nu RQ8AO0DR6v/KMLAnvA+IvKSgfp/Kw8+TLNOmSAE6RaUN10rLyeLjyhI2xu24+DcpC+ CH1kCSTuoDB3lW/nuboMLmkgZRyBQZnZTD269RhTjbkqfXP9PEBf4P+z1gtzNpaIUy 5i4GQ61FF1alW0z5K/fQX8HzYDusa74cH97ptCilGabfCE0pEzRu09C8ffzOaPD3iW RCXpfM6DHSk+3CQfGpSUrJA128uEmtUXVi/5V8Lt675cUgb6HsVmLpu2ecbbnGH4op oTml45c4736AQ== Date: Fri, 2 Apr 2021 23:36:07 +0530 From: Manivannan Sadhasivam To: Cristian Ciocaltea Cc: Rob Herring , Andreas =?iso-8859-1?Q?F=E4rber?= , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-actions@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 1/6] dt-bindings: reserved-memory: Add Owl SoC serial number binding Message-ID: <20210402180607.GB31906@thinkpad> References: <20210401170704.GA610119@robh.at.kernel.org> <20210401174001.GA2016477@BV030612LT> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20210401174001.GA2016477@BV030612LT> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210402_190615_938416_7495FD8F X-CRM114-Status: GOOD ( 25.08 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Thu, Apr 01, 2021 at 08:40:01PM +0300, Cristian Ciocaltea wrote: > On Thu, Apr 01, 2021 at 12:07:04PM -0500, Rob Herring wrote: > > On Tue, Mar 30, 2021 at 04:48:16PM +0300, Cristian Ciocaltea wrote: > > > Add devicetree binding for the Actions Semi Owl SoC serial number > > > reserved-memory range. > > > > > > Signed-off-by: Cristian Ciocaltea > > > --- > > > .../actions,owl-soc-serial.yaml | 53 +++++++++++++++++++ > > > 1 file changed, 53 insertions(+) > > > create mode 100644 Documentation/devicetree/bindings/reserved-memory/actions,owl-soc-serial.yaml > > > > > > diff --git a/Documentation/devicetree/bindings/reserved-memory/actions,owl-soc-serial.yaml b/Documentation/devicetree/bindings/reserved-memory/actions,owl-soc-serial.yaml > > > new file mode 100644 > > > index 000000000000..41b71f47ee6c > > > --- /dev/null > > > +++ b/Documentation/devicetree/bindings/reserved-memory/actions,owl-soc-serial.yaml > > > @@ -0,0 +1,53 @@ > > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > > > +%YAML 1.2 > > > +--- > > > +$id: http://devicetree.org/schemas/reserved-memory/actions,owl-soc-serial.yaml# > > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > > + > > > +title: Actions Semi Owl reserved-memory for SoC serial number > > > + > > > +maintainers: > > > + - Cristian Ciocaltea > > > + > > > +description: | > > > + Provide access to the memory region where the two parts of the Actions > > > + Semi Owl SoC serial number (low & high) can be read from. This information > > > + is provided by the bootloader, hence expose it under /reserved-memory node. > > > + > > > + Please refer to reserved-memory.txt in this directory for common binding > > > + part and usage. > > > + > > > + This is currently supported only on the S500 SoC variant. > > > + > > > +properties: > > > + compatible: > > > + oneOf: > > > + - const: actions,owl-soc-serial > > > + - items: > > > + - enum: > > > + - actions,s500-soc-serial > > > + - const: actions,owl-soc-serial > > > + > > > + reg: > > > + maxItems: 1 > > > + > > > +required: > > > + - compatible > > > + - reg > > > + > > > +additionalProperties: true > > > + > > > +examples: > > > + - | > > > + reserved-memory { > > > + #address-cells = <1>; > > > + #size-cells = <1>; > > > + ranges; > > > + > > > + soc_serial: soc-serial@800 { > > > + compatible = "actions,s500-soc-serial", "actions,owl-soc-serial"; > > > + reg = <0x800 0x8>; > > > > You end up wasting a whole page of memory for 8 bytes. It may be better > > to copy this to a DT property ('serial-number' is already a defined root > > property). > > Actually there is more information provided by the vendor bootloader Then you should call it as socinfo or something not soc_serial. Thanks, Mani > in this memory page, so we might use it once we are able to decode it. > For the moment I could only identify the serial number. > > Thanks, > Cristi > > > Rob _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel