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;
Message-ID: <afc9cd15-a895-792f-3f49-1dca3694b0f3@nvidia.com>
Date:   Sat, 9 Jul 2022 08:03:45 +0530
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101
 Thunderbird/91.3.2
Subject: Re: [PATCH V4 3/9] dt-bindings: PCI: tegra234: Add schema for
 tegra234 endpoint mode
Content-Language: en-US
To:     Rob Herring <robh@kernel.org>
Cc:     kthota@nvidia.com, sagar.tv@gmail.com, lpieralisi@kernel.org,
        bhelgaas@google.com, krzysztof.kozlowski+dt@linaro.org,
        kishon@ti.com, robh+dt@kernel.org, linux-pci@vger.kernel.org,
        devicetree@vger.kernel.org, jonathanh@nvidia.com,
        gustavo.pimentel@synopsys.com, linux-tegra@vger.kernel.org,
        thierry.reding@gmail.com, mmaddireddy@nvidia.com,
        jingoohan1@gmail.com, kw@linux.com, linux-kernel@vger.kernel.org
References: <20220707081301.29961-1-vidyas@nvidia.com>
 <20220707081301.29961-4-vidyas@nvidia.com>
 <1657196871.476299.1454231.nullmailer@robh.at.kernel.org>
From:   Vidya Sagar <vidyas@nvidia.com>
In-Reply-To: <1657196871.476299.1454231.nullmailer@robh.at.kernel.org>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?WEZEdmNQT0oyaXgyeWRpMVFpU29pR3pZeGhhNHNHWTZaaXMwTUlYWjlZKysy?=
 =?utf-8?B?bFcyYm43NHZ3Vm1XM3A5eitSaWZNc2dwcEhnNDZ2Q2JIQjIySDhBZVhhT0lh?=
 =?utf-8?B?d2Mzd1NNaXByU2d2U3RJTFhaYmNmZndjd2psQVp1aWpLd2YwdFlWdHM4ME03?=
 =?utf-8?B?ZmhYSWtJYjBlNDJQbThNV3BuSVpWdHVlYnB3blFodm0zanYvMlNTMktGNDlB?=
 =?utf-8?B?eUpyTkdHcjR5RU1FcDlCbWtmMll4dSsyRnlRZklYR3FHMmxwMWU0WjhyVjNO?=
 =?utf-8?B?K2tsSlV5aHEzV2h2a0M3aVE1T1lJTkxCbE1VallOczcyYW9mTUJZdXRpMUNk?=
 =?utf-8?B?bFJOOHl0cG1hUjl5N3RKRWJpVzNIbFlIM1VqbW1wYlh6VmJvVnZCb1BvL3pq?=
 =?utf-8?B?NTZmMEZiV1dsUUNFUi90UWVwQ0tDZ1ZjNVJHOFVsYWpFMEhyNGFSOXE5V3lm?=
 =?utf-8?B?c3lWZEg2Nzg0Q3g2RjA2bWZ5ZzVFYStrRFppbEJoa1Jsc0M2U0FSaGZweXJL?=
 =?utf-8?B?RzhhNWV3Z2dKd3VnV2U2OFE3THZlWnBPS0ZPekdqS3B4OE1nM2JlUHRJQmUw?=
 =?utf-8?B?dk91bFBDTkRaaXhDbGRod1RFL2czN0ZMTEU3WnluMTlsM0FJTmdWd2drRmlE?=
 =?utf-8?B?UndkeXA5NDJIQUJPdkw0enVPdWV3WVdCeW5HeWpJcmNiZHdGbi9zV1NCeitT?=
 =?utf-8?B?ei9MWkRGYUVPQUdQMHRXQVU0MklkYWwxbTl4ZGJmeW1lbXlYczhIekJpZTlY?=
 =?utf-8?B?YzdtYXlSV1VUQklSdDRCTnNPUi9Yc2R2TzRmcGg5RVRkZVNITUhHNXY2U2xW?=
 =?utf-8?B?bk1xazlBRzl4VnhuQThHcVo5TmZEdjBCY0R0bE5haGJLemV5YzhHTlY2ZTha?=
 =?utf-8?B?TG1qcTNFV09tQUowMktJM0FOWmZFaldmQmN1UjdteVV0aGJhQnoyMVRuRnRt?=
 =?utf-8?B?VllTdjBGbEhwaS9kMHlsSTkycjE2NnRMc3I4UUdnQVBRYXBiTmtXMHYvbU0r?=
 =?utf-8?B?WmVkOVh5c0NjQm9iWVVmbXQwU0d5WWNLckk3Z2xuUVFwTlhud1NtN0lvU200?=
 =?utf-8?B?MjBMZU42TWpNMW9FN0t1ai80Y2FVRm5JVElJMHRWQ01JV3lUd0JzZ3VxMDhX?=
 =?utf-8?B?UGcydjhGU3lCcG9qTjgydWg1K2dNVElUY1k3M2tXcDhESVE2aUszL1d2UUlB?=
 =?utf-8?B?bVFBK3Vram5ybXRYOVJDbnZhRWUvdDVyaHdHRDJNRVlWK0d3eklQTmRJL2tX?=
 =?utf-8?B?dU1kdUlQcGxlZnRLM3BVRVlYekVMeVh1UmF6Z0xvejg0TnR6SDZHRTIrdGQ5?=
 =?utf-8?B?aXI3NG9icGI5bDNUSkNoanJtNnJlajIzTEVJWUtlU25tdy8xQlBCQ2ErMVAw?=
 =?utf-8?B?b3BFaVROempaSEZFeEpGc1c4OUFoSjN3Rjk1ZE51YVBwdzh5V29XTFBVcGg4?=
 =?utf-8?B?SktVU2RnN3hGcWg1SHdVQXFBZjl3R29SV0ZLWlZBVzR2RlUyQnNLMi9rMUc5?=
 =?utf-8?B?YUNIZzc5SFhmdXZ5ZjBXWk5menJzelZJVmRFOFoxUlZOMFdhWEloYmFFOHpu?=
 =?utf-8?B?MUZCcjRaYnV2SmRoU2IrRjVMWFAzUVF0Lzg5YisxRnNOQ2Q0Q2ZGK0Fub0Vl?=
 =?utf-8?B?NU8ySHl6VjZwRk55ZmFheC9Md0tPYnRMN0g1dkl3Tm5scW9KdVhCNWQ4K1NL?=
 =?utf-8?B?U3hvSzBjcHJaSXVnN0FHSjl3OXN6ckR1ZmVjUy9Fdndndkd0Y2RYNG1vWXBs?=
 =?utf-8?B?RE9NMXhVL3pQUnoveUUyWmk2Y204Y3J2V3VUNTRDSzlCOS9LVUdWM1dRYzlr?=
 =?utf-8?B?QVlmeGxNZThxK2hDeVk2c2VCcGREYnJid0E4T2MxcmFCVHFrVW10UEJZY2Uw?=
 =?utf-8?B?SEFPL3BuTkVQdmFBdWYrRjFMMU50UERSeEJSWVhGaDIra0lBM0F0REhndC9M?=
 =?utf-8?B?S202ZWF0VDQwc3d5ZVlERVZqYjdPYUl5aDRMYktrTVFNSlRJbWpuU3laeXRI?=
 =?utf-8?B?RFk1Z1J0SEtNQTVtQk9mM083SkV0d3dFZHg5TXRoUWtPUVZvN3EwWnh2QW9O?=
 =?utf-8?B?VkRjZE93K200bG9zYUtqeDNydG8vcXRXSWhscnMvUlFqUS9wME5LN3k3eGQ4?=
 =?utf-8?Q?oNkNfANhQWE/ERbrJfXY2HH7o?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: c71f9681-6009-42d8-0448-08da61537988
X-MS-Exchange-CrossTenant-AuthSource: BN8PR12MB2900.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 Jul 2022 02:33:57.9233
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: wHsHDYwNhmvr9MHRwY/14+sK50VylWkpI1Aim+amY8MrifBH0CJ/B6PcesHt2DG1xEVgntkkhQ1CNOaW1g57vQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR12MB3823
X-Spam-Status: No, score=-1.8 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FORGED_SPF_HELO,
        NICE_REPLY_A,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,
        SPF_NONE,T_SCC_BODY_TEXT_LINE 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: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


On 7/7/2022 5:57 PM, Rob Herring wrote:
> External email: Use caution opening links or attachments
> 
> 
> On Thu, 07 Jul 2022 13:42:55 +0530, Vidya Sagar wrote:
>> Add support for PCIe controllers that operate in the endpoint mode
>> in tegra234 chipset.
>>
>> Signed-off-by: Vidya Sagar <vidyas@nvidia.com>
>> ---
>> V4:
>> * Rebased on top of previous patch
>>
>> V3:
>> * New patch in this series
>>
>>   .../bindings/pci/nvidia,tegra194-pcie-ep.yaml | 123 +++++++++++++++++-
>>   1 file changed, 117 insertions(+), 6 deletions(-)
>>
> 
> My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check'
> on your patch (DT_CHECKER_FLAGS is new in v5.13):
> 
> yamllint warnings/errors:
> 
> dtschema/dtc warnings/errors:
> /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/pci/nvidia,tegra194-pcie-ep.example.dtb: pcie-ep@141a0000: Unevaluated properties are not allowed ('nvidia,enable-ext-refclk' was unexpected)
>          From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/pci/nvidia,tegra194-pcie-ep.yaml
> 
> doc reference errors (make refcheckdocs):
> 
> See https://patchwork.ozlabs.org/patch/
> 
> This check can fail if there are any dependencies. The base for a patch
> series is generally the most recent rc1.
> 
> If you already ran 'make dt_binding_check' and didn't see the above
> error(s), then make sure 'yamllint' is installed and dt-schema is up to
> date:
> 
> pip3 install dtschema --upgrade
> 
> Please check and re-submit.

I did run 'make dt_binding_check' before sending the patches for review 
and didn't observe any issue. Just to make sure that I observe the same 
issue locally, I updated dtschema and gave DT_CHECKER_FLAGS=-m and yet 
didn't observe the tool reporting any issue.
FWIW, I think I missed adding the documentation for 
'nvidia,enable-ext-refclk' property. I'll add it and send again, but 
before that, I would like to see if my change addresses the issue 
correctly and also there are no other issues. Could you please help me 
with why I'm not observing the issue locally?

FWIW, Here is the output I get with V4 patch.

vidya@vidya-virtual-machine:~/x/linux-next$ make DT_CHECKER_FLAGS=-m
dt_binding_check
DT_SCHEMA_FILES=Documentation/devicetree/bindings/pci/nvidia,tegra194-pcie-ep.yaml

    LINT    Documentation/devicetree/bindings
    CHKDT   Documentation/devicetree/bindings/processed-schema.json
    SCHEMA  Documentation/devicetree/bindings/processed-schema.json
/home/vidya/x/linux-next/Documentation/devicetree/bindings/display/tegra/nvidia,tegra124-sor.yaml:
ignoring, error in schema: allOf: 1: if: not: properties
    DTEX
Documentation/devicetree/bindings/pci/nvidia,tegra194-pcie-ep.example.dts
    DTC
Documentation/devicetree/bindings/pci/nvidia,tegra194-pcie-ep.example.dtb
    CHECK
Documentation/devicetree/bindings/pci/nvidia,tegra194-pcie-ep.example.dtb
vidya@vidya-virtual-machine:~/x/linux-next$

Thanks,
Vidya Sagar

>