Received: by 2002:a05:6358:1087:b0:cb:c9d3:cd90 with SMTP id j7csp2404258rwi; Fri, 28 Oct 2022 06:46:34 -0700 (PDT) X-Google-Smtp-Source: AMsMyM6OhdGm9hwPp0iuLkbwdffdvO4G3DCiikzoF+SS49X1ReT06GI6bSa17jct13Db0W866Jun X-Received: by 2002:a17:907:7244:b0:78d:cedc:7a9e with SMTP id ds4-20020a170907724400b0078dcedc7a9emr46294100ejc.600.1666964773682; Fri, 28 Oct 2022 06:46:13 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1666964773; cv=none; d=google.com; s=arc-20160816; b=WLnL+yCPbR21JkAOKKqQ3GNm85+Q5Tm5QkQHhn2i5bzsNAslN8Xal37y/2d3s0EE+q y41TulKmo1P5MlWGgfPb+zgBA6vc2A0e1G2diuSO46YkXavDnXwu67Jxx7+DgpEZplqx w+4oBsmYTu+8xfXgXaEScJP8FGHg6X5dhPS2GtPTclxRWH8zWhwBFaKHdt6VlkIp78hp bFVsN6Ctvwd5dc2Csm6guSrfZxaL7tXfMt/yiniRlWVnMqxeIxW01jOrFinL5c/wXdRi M3aNM6DSOJM+tpclhjjLlzlmYgDpfgrL2SZeBLKyFP0LHzSYzsvj09aVvOQ0HS5Ji9Xg CaTQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:in-reply-to:from :references:cc:to:content-language:subject:user-agent:mime-version :date:message-id:dkim-signature; bh=m1zeisbWzfTRLqshyHyu2ONVYhtME7k5k9XJadTce2s=; b=pI6eLwGoZL7uGRMtAUFjBwsdmykL4/rQwOy/WzNRp+TXAxAzL1H18Qh1ic/utzC+HH +krDvWvdRx6OnzOHFZUBocKO6hvyLJCyEaTw0stwCBTBrCoUVOjHx1alsQCQTEfHGvmr wUek/t8v8UuNCoJihAbhDtQTWcnW2gvIi1jpxVRTePIkU8BwwLWCLBv+avkJUfEVP7AF cUpKhoQGj/GO9mdhCViiDiQ61hXpVoFxf9gp1TVynpWEJwgZZQiBfiowUNk153+Ui0ha lkRRSCemkjfHkHooJtZSO5xW/yxVfRFFnzHAUWFfs5nbioX1ScdlFNUrhD7wD4Ct2qN5 AqhA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=PFKyKgk7; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id b17-20020a056402279100b00462a20f860bsi5409126ede.539.2022.10.28.06.45.48; Fri, 28 Oct 2022 06:46:13 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=PFKyKgk7; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229980AbiJ1Mr1 (ORCPT + 99 others); Fri, 28 Oct 2022 08:47:27 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:48660 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229494AbiJ1Mr0 (ORCPT ); Fri, 28 Oct 2022 08:47:26 -0400 Received: from mail-qk1-x72e.google.com (mail-qk1-x72e.google.com [IPv6:2607:f8b0:4864:20::72e]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6F84573322 for ; Fri, 28 Oct 2022 05:47:25 -0700 (PDT) Received: by mail-qk1-x72e.google.com with SMTP id 8so3300834qka.1 for ; Fri, 28 Oct 2022 05:47:25 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; 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=m1zeisbWzfTRLqshyHyu2ONVYhtME7k5k9XJadTce2s=; b=PFKyKgk7W8wOY0MRB/S6pACCrIjHCkTlMW+Yhg3RLzDCGfn2zH66SiaRGnDFFeNNpS AaEGFW39LYwKXiQGZsgVVTIY2+ZyKPxoe8eP3Q633BpMZsCSOJwWOR0hb5ad3LYuHJSL RAoSEhyRCATvIUtlvm+WsOtX10V7fHg5dNTQK5a4jEciPzwN5t/Wkye+8I/tdCmWX7QK 5pRM57LlOhRw+5TgEViDRB7aR1Pk0pIQEhWnugXoSTXI7DKCRdGXKRvTed2NLGt7A5Ro 5m6YRSgUewjzkZnzx9Ti005uE5lr9S84Yec6On5Y/Q94mBJhda73rlWT/iaggTcmG9iH XCGw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; 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=m1zeisbWzfTRLqshyHyu2ONVYhtME7k5k9XJadTce2s=; b=YniMr7PCDozpRd6uTiUiPiKFX2ncrFOM3sgdFxkc61FaX8avUrlRJRPrOZIh1MqgA9 X6yOOpv1JcqFq2qoaUfPKvlSZYGOHsZ3TuAjgftqlgBP35S/PkT8lWtd/VmNbqwwPmko 1njHqqTvfKlzQW64c29LDkIWUZb7bX0889SDuQHk4JUWMQF8IIuHczsmjylVx8wPFTLx o5B/n48FXOtQisgtg/EkIDUKi3RT//mfQs2nvW6K8TZLrXt8/IneMNwfP9mf0mS3LbXP GSmUztS/FOEg9KmKXMt/N0jrRexhOO5QRrin6FTRjiVMCmcXgpKzJjeOIExmZDFwSmoz fVqw== X-Gm-Message-State: ACrzQf1zx5sKxa1g0+VQ9wkY5JOsZOlBMih1T5RQ0y6YfDUyJMyTQS1X v94x9nsnFBlWwzrZQsmnJiAx5w== X-Received: by 2002:a37:5a42:0:b0:6ee:8f0a:9319 with SMTP id o63-20020a375a42000000b006ee8f0a9319mr37789772qkb.315.1666961244601; Fri, 28 Oct 2022 05:47:24 -0700 (PDT) Received: from [192.168.1.11] ([64.57.193.93]) by smtp.gmail.com with ESMTPSA id w10-20020a05620a444a00b006f9e103260dsm2942489qkp.91.2022.10.28.05.47.23 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 28 Oct 2022 05:47:24 -0700 (PDT) Message-ID: <20019699-40a7-37d6-4b9b-06446183338f@linaro.org> Date: Fri, 28 Oct 2022 08:47:22 -0400 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.4.1 Subject: Re: [PATCH V1 1/2] dt-bindings: PCI: tegra234: Add ECAM support Content-Language: en-US To: Vidya Sagar , robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, thierry.reding@gmail.com, jonathanh@nvidia.com, mperttunen@nvidia.com Cc: devicetree@vger.kernel.org, linux-tegra@vger.kernel.org, linux-kernel@vger.kernel.org, kthota@nvidia.com, mmaddireddy@nvidia.com, sagar.tv@gmail.com References: <20221027051214.22003-1-vidyas@nvidia.com> <20221027051214.22003-2-vidyas@nvidia.com> <4579404d-b3f7-8342-6704-2644f5cd489d@linaro.org> <7229a2a6-d646-a8f5-50a2-cca6c44d16d5@nvidia.com> From: Krzysztof Kozlowski In-Reply-To: <7229a2a6-d646-a8f5-50a2-cca6c44d16d5@nvidia.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,RCVD_IN_DNSWL_NONE, SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 28/10/2022 08:09, Vidya Sagar wrote: > > > On 10/28/2022 2:48 AM, Krzysztof Kozlowski wrote: >> External email: Use caution opening links or attachments >> >> >> On 27/10/2022 01:12, Vidya Sagar wrote: >>> Add support for ECAM aperture for Tegra234. >>> >>> Signed-off-by: Vidya Sagar >>> --- >>> .../devicetree/bindings/pci/nvidia,tegra194-pcie.yaml | 10 ++++++++-- >>> .../devicetree/bindings/pci/snps,dw-pcie.yaml | 2 +- >>> 2 files changed, 9 insertions(+), 3 deletions(-) >>> >>> diff --git a/Documentation/devicetree/bindings/pci/nvidia,tegra194-pcie.yaml b/Documentation/devicetree/bindings/pci/nvidia,tegra194-pcie.yaml >>> index 75da3e8eecb9..15cc2d2055bb 100644 >>> --- a/Documentation/devicetree/bindings/pci/nvidia,tegra194-pcie.yaml >>> +++ b/Documentation/devicetree/bindings/pci/nvidia,tegra194-pcie.yaml >>> @@ -27,6 +27,7 @@ properties: >>> - nvidia,tegra234-pcie >>> >>> reg: >>> + minItems: 4 >>> items: >>> - description: controller's application logic registers >>> - description: configuration registers >>> @@ -35,13 +36,17 @@ properties: >>> available for software access. >>> - description: aperture where the Root Port's own configuration >>> registers are available. >>> + - description: aperture to access the configuration space through ECAM. >>> + This is applicable only for Tegra234. >> >> Then restrict it per compatible in allOf >> > > Thanks Krzysztof for your review. > For a similar change here > https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20220707081301.29961-3-vidyas@nvidia.com/ > Rob had said that may be it is not worth it. > Let me know if you really want this change and I'll push a new patch. > That's a bit different. Restricting it per variant works and should be trivial. Best regards, Krzysztof