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 2F97AC43334 for ; Wed, 6 Jul 2022 14:59:07 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233459AbiGFO7G (ORCPT ); Wed, 6 Jul 2022 10:59:06 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:46040 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233277AbiGFO7F (ORCPT ); Wed, 6 Jul 2022 10:59:05 -0400 Received: from mail-lf1-x12b.google.com (mail-lf1-x12b.google.com [IPv6:2a00:1450:4864:20::12b]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id A746C25C79 for ; Wed, 6 Jul 2022 07:59:03 -0700 (PDT) Received: by mail-lf1-x12b.google.com with SMTP id y16so26344979lfb.9 for ; Wed, 06 Jul 2022 07:59:03 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=message-id:date:mime-version:user-agent:subject:content-language:to :cc:references:from:in-reply-to:content-transfer-encoding; bh=cmKQ1A+SNcVV+xdQAtk9f96FGO9npHWf2vTO1h3im4s=; b=bJ2wmt19f7E1iCV0cGWdHwG61juT9VIkFewxrXGstKTVLXv0jkpZkQy6KxA/5C4qCk Jp3/f2gVaIhNHYyeq8rvmfsLz9PtwYUqeV2kuy4xPxHBteIXHDQaLGKsHvXyB78CBRhO Mko6wPrrUlSKk6yGd6ObodHqj7Kucs5yHvjwp5If/uKb9PILyEzFwUY3Ql3yIl0UjmZI 4ichYDbs91f7sSkVOXgKmZM7DWZmVf8I5Ow/GLh+VNIoBbKnJ2pKssLSyol/GOyj/y7h +lIkCKVw7m1OxBhsAZXLvfeINFzlzcpA0J9BaRHgB9QVfHD+DgIaRwbU7pfIlLZIJER9 X+zg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:message-id:date:mime-version:user-agent:subject :content-language:to:cc:references:from:in-reply-to :content-transfer-encoding; bh=cmKQ1A+SNcVV+xdQAtk9f96FGO9npHWf2vTO1h3im4s=; b=zrxdabbp/e8rOAEDVts3TZhcTBYEnB/G6jlnsIHbKHY1aKjvrUqwSLw6vQfhfxfuAz ChBFPtktJ1Z/g165xBBreYVK0d1iwN1jeAxF+w4HpRPC1TgV+l0QizZ59LfThJcor46h HJ2xyXKfyiP8/ZXjOz3ZwD3DUFovMO3UUlRa/KaMA78cACboYzupdqd31DzAw/mz+7xH aAKV0jwlA2bqzhW39ZV31VxmloZY5IzcRXNEj7x2tcbcNf2xc2JGQOQ1SGL5jzR4FE8o I4bLQqzKQJVg3GwKkat1clEAFrTgY8J7TXO/CrJ32tO0X6ZEIGG+Tr/gH0xJ+PdxUfwz Wtnw== X-Gm-Message-State: AJIora9pE6Szj5r9H+RF9y5vyARnlnuVLIVUo4QICEdq7H3C0qAFR8zr HqHKHuuJNhI1aERwvQw2ip0BrA== X-Google-Smtp-Source: AGRyM1vgEuf632zLZr+I1Gp3QidCXy8XtIkAtWyjsW3vgP/zQ1uvi/Tv6A+oTDrNIBJQQDoRMtGnjg== X-Received: by 2002:a05:6512:1698:b0:47f:b5a6:6870 with SMTP id bu24-20020a056512169800b0047fb5a66870mr25958708lfb.578.1657119542015; Wed, 06 Jul 2022 07:59:02 -0700 (PDT) Received: from [192.168.1.52] ([84.20.121.239]) by smtp.gmail.com with ESMTPSA id o14-20020ac25e2e000000b0047f93edb9f5sm6304967lfg.181.2022.07.06.07.59.00 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 06 Jul 2022 07:59:01 -0700 (PDT) Message-ID: <75f8b257-7e0a-d871-ab30-37a72f7da56e@linaro.org> Date: Wed, 6 Jul 2022 16:59:00 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.11.0 Subject: Re: [PATCH v2 2/3] dt-bindings: pci: QCOM Adding sc7280 aggre0, aggre1 clocks Content-Language: en-US To: Krishna Chaitanya Chundru , helgaas@kernel.org Cc: linux-pci@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, mka@chromium.org, quic_vbadigan@quicinc.com, quic_hemantk@quicinc.com, quic_nitegupt@quicinc.com, quic_skananth@quicinc.com, quic_ramkri@quicinc.com, manivannan.sadhasivam@linaro.org, swboyd@chromium.org, dmitry.baryshkov@linaro.org, Andy Gross , Bjorn Andersson , Bjorn Helgaas , Rob Herring , Krzysztof Kozlowski , Stanimir Varbanov , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" References: <1656062391-14567-1-git-send-email-quic_krichai@quicinc.com> <1656691899-21315-1-git-send-email-quic_krichai@quicinc.com> <1656691899-21315-3-git-send-email-quic_krichai@quicinc.com> <1fb5f0c6-ff72-b9ba-175a-b5197ed658a7@linaro.org> <9de4c3a0-eb95-f4e9-b828-2343241fff41@quicinc.com> From: Krzysztof Kozlowski In-Reply-To: <9de4c3a0-eb95-f4e9-b828-2343241fff41@quicinc.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-pci@vger.kernel.org On 06/07/2022 13:55, Krishna Chaitanya Chundru wrote: > > On 7/4/2022 1:54 PM, Krzysztof Kozlowski wrote: >> On 01/07/2022 18:11, Krishna chaitanya chundru wrote: >>> Adding aggre0 and aggre1 clock entries to PCIe node. >>> >>> Signed-off-by: Krishna chaitanya chundru >>> --- >>> Documentation/devicetree/bindings/pci/qcom,pcie.yaml | 6 ++++-- >>> 1 file changed, 4 insertions(+), 2 deletions(-) >>> >>> diff --git a/Documentation/devicetree/bindings/pci/qcom,pcie.yaml b/Documentation/devicetree/bindings/pci/qcom,pcie.yaml >>> index 0b69b12..8f29bdd 100644 >>> --- a/Documentation/devicetree/bindings/pci/qcom,pcie.yaml >>> +++ b/Documentation/devicetree/bindings/pci/qcom,pcie.yaml >>> @@ -423,8 +423,8 @@ allOf: >>> then: >>> properties: >>> clocks: >>> - minItems: 11 >>> - maxItems: 11 >>> + minItems: 13 >>> + maxItems: 13 >>> clock-names: >>> items: >>> - const: pipe # PIPE clock >>> @@ -437,6 +437,8 @@ allOf: >>> - const: bus_slave # Slave AXI clock >>> - const: slave_q2a # Slave Q2A clock >>> - const: tbu # PCIe TBU clock >>> + - const: aggre0 # Aggre NoC PCIe CENTER SF AXI clock >>> + - const: aggre1 # Aggre NoC PCIe1 AXI clock >> You ignored my comments from v1 - please don't. This is not accepted. >> >> Also, please do not send new versions of patchset as reply to some other >> threads. It's extremely confusing to find it under something else. >> >> Best regards, >> Krzysztof > Hi > > Krzysztof, > > Sorry for confusion created which replying this patch. > > The only comment I got from v1 from you is to run make dtbs_check. > > I ran that command I found the errors and fixed them and I ran the make dtbs_check again > before on v2 and made sure there are no errors. > > Can you please tell me is there any steps I missed. The comment was: "This won't work. You need to update other entry." and then a conditional: "If you test it with `make dtbs_check` you will see the errors." So let's run it together: /home/krzk/dev/linux/linux/out/arch/arm64/boot/dts/qcom/sc7280-idp.dtb: pci@1c08000: clocks: [[42, 55], [42, 56], [41, 0], [39, 0], [42, 50], [42, 52], [42, 53], [42, 57], [42, 58], [42, 177], [42, 178], [42, 8], [42, 21]] is too long From schema: /home/krzk/dev/linux/linux/Documentation/devicetree/bindings/pci/qcom,pcie.yaml /home/krzk/dev/linux/linux/out/arch/arm64/boot/dts/qcom/sc7280-idp.dtb: pci@1c08000: clock-names: ['pipe', 'pipe_mux', 'phy_pipe', 'ref', 'aux', 'cfg', 'bus_master', 'bus_slave', 'slave_q2a', 'tbu', 'aggre0', 'aggre1', 'ddrss_sf_tbu'] is too long clocks and clock-names can be maximum 12 items, as defined by schema in "properties:" section. You cannot extend it in one place to 13 but leave 12 in other, because both constraints are applicable. If you test it, you will see the errors. Best regards, Krzysztof