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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 38FF4C0015E for ; Wed, 5 Jul 2023 06:17:17 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230407AbjGEGRP (ORCPT ); Wed, 5 Jul 2023 02:17:15 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:38440 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231540AbjGEGRP (ORCPT ); Wed, 5 Jul 2023 02:17:15 -0400 Received: from mail-lj1-x22c.google.com (mail-lj1-x22c.google.com [IPv6:2a00:1450:4864:20::22c]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id DBE8210F5 for ; Tue, 4 Jul 2023 23:17:10 -0700 (PDT) Received: by mail-lj1-x22c.google.com with SMTP id 38308e7fff4ca-2b69f958ef3so101710061fa.1 for ; Tue, 04 Jul 2023 23:17:10 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1688537829; x=1691129829; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=90vFHFNv8Z4EnqFJciqEXi399O0ahUbhYDsrlxPDfUA=; b=ZmcAsyvfEgsv3Kwm+U6DlC5Ix5sl36izoQPtKwWwQXUXXk9e96NWXqqiH/MdmdOQ68 6Y+kREJdwA7kgrbS+bngxnH+GlQfk/5CvT4xdIbRB8j0e9JsV5KwoLN4h3w4XeVrPOZ9 gyopY24vIoXs2o/zSgUQJO189T7kUPedathiv/f1g4jMjrWmJTTsrwIuoM36YCk4epoK 3qtnWlIf12SBeOj7oQ1uxCrRsUkLVuE1VzzGPYF/nditBmMYJ2kca1xN+Xl3xlpwHCJd iF0A6izZqoI/LQL0dwhehqI/cgJnzGT/EjBz3nXouh5WwOSXCIM9ILveifP2hrDqEh7P CbOA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1688537829; x=1691129829; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=90vFHFNv8Z4EnqFJciqEXi399O0ahUbhYDsrlxPDfUA=; b=WPwV9ZCp7dEoXelu5WxWfOPZ5b9OONsX0/moL2iECo8ea0kHSEHuQzQ+VWI6b8kGOc fT+IhMUeS/CUsRKn1DN6iS/aRGgTRnC/Aqb644PbDRfwoN7SSpi9s3nmbsaof36iedmx dEB1+md2xK/sfzi2yHRr/dck+9PaV1EnnF9RfPuSP/A09pbj6mtZQ3opP6dhzD3F1lQP vBMG9LPTvU8rFx3qZ1Kk/jcBEnP7gkgoHqylfUg6hPrQ6TAAK0OyKKMjuVQlVA8XrKAs xc2P+yYQo+dnpf4c0CF/yYNZBzi5/7vFX8VW5VfHSUaR2EXHwwHr4y3nJ7r06zgT+2wX ryTA== X-Gm-Message-State: ABy/qLZrKbY+O5TDsu7pjmxmToEewNmgWSRevDdDNyYm8hAlDZjWC7wf NWDt3XXGi9xXwSp5cYL2m7+kCA== X-Google-Smtp-Source: APBJJlFEGjzSwAu/PuXg2rA//CTojQCgKRBCyvzFiOBPJoQUTeMenTuSrLxZqCx2g7wC+7Wnju1qjA== X-Received: by 2002:a2e:6a0f:0:b0:2b6:cf18:77e0 with SMTP id f15-20020a2e6a0f000000b002b6cf1877e0mr10155024ljc.48.1688537829112; Tue, 04 Jul 2023 23:17:09 -0700 (PDT) Received: from [192.168.1.20] ([178.197.219.26]) by smtp.gmail.com with ESMTPSA id r19-20020aa7cfd3000000b0051e06693590sm4552748edy.91.2023.07.04.23.17.07 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 04 Jul 2023 23:17:08 -0700 (PDT) Message-ID: Date: Wed, 5 Jul 2023 08:17:06 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.12.0 Subject: Re: [PATCH V7 1/2] dt-bindings: firmware: bootstats: Add the dtschema Content-Language: en-US To: Souradeep Chowdhury , Andy Gross , Konrad Dybcio , Krzysztof Kozlowski , Bjorn Andersson , Rob Herring , Arnd Bergmann Cc: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, Sibi Sankar , Rajendra Nayak References: From: Krzysztof Kozlowski In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On 05/07/2023 07:32, Souradeep Chowdhury wrote: > Add the device tree bindings for boot stats. This has 2 properties > pre-abl-time and abl-time which records the timestamps for boot > stats. > > Signed-off-by: Souradeep Chowdhury > Link: https://lore.kernel.org/linux-arm-kernel/7d397e67-5d56-4975-98af-1ac9746c07f4@app.fastmail.com/ > --- > .../bindings/firmware/qcom,bootstats.yaml | 38 +++++++++++++++++++ > 1 file changed, 38 insertions(+) > create mode 100644 Documentation/devicetree/bindings/firmware/qcom,bootstats.yaml > > diff --git a/Documentation/devicetree/bindings/firmware/qcom,bootstats.yaml b/Documentation/devicetree/bindings/firmware/qcom,bootstats.yaml > new file mode 100644 > index 000000000000..22e697524058 > --- /dev/null > +++ b/Documentation/devicetree/bindings/firmware/qcom,bootstats.yaml > @@ -0,0 +1,38 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/firmware/qcom,bootstats.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Boot Stats This explains nothing... > + > +maintainers: > + - Souradeep Chowdhury > + > +description: | Drop | > + Qualcomm's proprietary Android boot-loaders capture boot time Wrong indentation. > + stats, like the time when the bootloader started execution and > + at what point the bootloader handed over control to the kernel. > + This is captured in the unit of ms in devicetree property. > + > +properties: > + pre-abl-time: > + description: The property to store the starting point of abl in ms. String instead of ms? No, this is the craziest idea I saw recently. Use -ms suffix. > + $ref: /schemas/types.yaml#/definitions/string-array > + > + abl-time: > + description: The property to store the duration of abl in ms. > + $ref: /schemas/types.yaml#/definitions/string-array I have no clue what this entire binding is about. Nothing can bind to it, no usage explained. Properties are not used to "store the duration". This does not look like suitable for DT, drop entire binding. > + > +required: > + - pre-abl-time > + - abl-time > + > +additionalProperties: false > + > +examples: > + - | > + bootstats { > + pre-abl-time="17627"; > + abl-time="26748"; Missing spaces. Open existing DTS and look at existing coding style. Best regards, Krzysztof