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: <c2112fcd-187a-73dd-22e4-631500a6b648@nvidia.com>
Date:   Mon, 24 Oct 2022 16:58:53 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.2.2
Subject: Re: [PATCH 02/11] dt-bindings: usb: Add NVIDIA Tegra XUSB host
 controller binding
Content-Language: en-US
To:     Rob Herring <robh@kernel.org>, Wayne Chang <waynec@nvidia.com>
Cc:     linux-phy@lists.infradead.org, robh+dt@kernel.org,
        gregkh@linuxfoundation.org, kishon@ti.com, vkoul@kernel.org,
        linux-tegra@vger.kernel.org, linux-kernel@vger.kernel.org,
        singhanc@nvidia.com, linux-i2c@vger.kernel.org,
        mathias.nyman@intel.com, heikki.krogerus@linux.intel.com,
        linux-usb@vger.kernel.org, thierry.reding@gmail.com,
        devicetree@vger.kernel.org, treding@nvidia.com,
        krzysztof.kozlowski+dt@linaro.org, balbi@kernel.org,
        ajayg@nvidia.com, jckuo@nvidia.com, p.zabel@pengutronix.de
References: <20221024074128.1113554-1-waynec@nvidia.com>
 <20221024074128.1113554-3-waynec@nvidia.com>
 <166661789886.1678573.2345753570567106623.robh@kernel.org>
From:   Jon Hunter <jonathanh@nvidia.com>
In-Reply-To: <166661789886.1678573.2345753570567106623.robh@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?LzFyWnpRdHVSanR0ZStxZlBBQXRVaTNJOWoxelduWWczYmY1VFExaE1YS3Ns?=
 =?utf-8?B?MXN6VTdjSWYzVjNCZ3c2U2pUZ1YzRVpKbkFEQTF6bmozV09WZUFvWmlJTDZO?=
 =?utf-8?B?clhJWG5RSmxreDd4UUJQYUZ4VFcrOExnZURmTjYwc2ErV1JrRW9Yd0l1anhJ?=
 =?utf-8?B?bzd1Sjc5WG1uRFNNTnpqWklkTXFxSTNSbHZrSWkvS3Yxb3NnazJNZFc4S250?=
 =?utf-8?B?NHgzVytiVDJlQXVEQ3kzZ2U4cW5SbGZYL3E5UG5IbEVwTlJDQjNsclRtbjZp?=
 =?utf-8?B?djVDTS9vazlmaGd1cXJrVGZhTFo3aW1kc0YwRUt4UjlWckhqUTd3Z1NFa1dJ?=
 =?utf-8?B?UEJQNEFYOVcxQjROUzFQU0xLTTFnbWNweTh0bkhZNEdWWENTR0x4eE9yVnBR?=
 =?utf-8?B?aHluRGxRZUdaVk1LTFZvQmd4R3ZleWZ1UXM3RStObXp3bHlacEF1TFF0RHls?=
 =?utf-8?B?bkR6VVVmY2FudSt5RHpvSjh0WHRLK1RwSHdzNjhTWGJUUkorQzgxTDRISDV6?=
 =?utf-8?B?MUdWV24rbXU1N3dyeTJtUWZxTjBqcGlSOWRsNUxxRDR3R0E1V0UwcENiQVZF?=
 =?utf-8?B?d1BGU2Z6Tkdyem8wOGNwMURBSGVFZGUwM2grd09KQlFwZ2lKYnlvMmpnSmNq?=
 =?utf-8?B?VEpCUWh0Mjg3WUlmcmRSc3NIekJDR0ZQOTRqR0phSndLa1JSRVZGbHovdWhw?=
 =?utf-8?B?Rk5QWVNTVjhnem5mUjBhcDNFWlNwemlNakVzVkw2djBaMXNVL0hMd2hWVlkv?=
 =?utf-8?B?QkZmdk1OQkpQVkdWMkQ0SDlqMUNYa2Y2UXprUHFKTGJXOE9lVFZVMWx0VEFw?=
 =?utf-8?B?NHUyQlNZRWsvMmJRZVVtcFJUQXV5VW5MZjJSbGhRMjBrMHcra1lMVnFpLyt1?=
 =?utf-8?B?QkRQbDYvOUkzNGo0TUhpUGZ2MVd3aDZQNHg5NTlITVlhQ3ZxTDZ3eE9HZG1z?=
 =?utf-8?B?NmhjM3Y0OEsxcGVYWHdtVjloTmlqT2xaSnJ5bmQvT09IbVpRdTdkVkhXeWg2?=
 =?utf-8?B?UTV4MU1ZSEJ3T1NhbFlmcVJMbHBnRXZqVW94WHBHampPWFpkZVpseVFZR3JE?=
 =?utf-8?B?NXBBYU8xWEpCL21yMHJDdkliYnNqK1lyQVBlSkJaalBMMGd2RVdXcHY3em16?=
 =?utf-8?B?MXRoOHJMYW1SN1NYVFA3U1RlcXlCQmNIWUl1TUt0QlgyL09Hai84ZkttMGlN?=
 =?utf-8?B?MDF2c1FvWU1ibytDOGFiUUxlb3NRTHk2WjJzb25NZ052UlpKb0dBRk0rQTRk?=
 =?utf-8?B?K21DUE54NWt0L2grclM2SU1icjk5VWo2RGpsd0JQdVBTVE5teUw3UXA2L01G?=
 =?utf-8?B?enA0MDk0djlZVUVjb0UzWDR4RXlpQ1RRWExadW1hWGF4bWQwazltMkNGQ3dI?=
 =?utf-8?B?TmprZzlwN05lODc2dERXNUowVEFpZGtYQW1MMFpaR3d0RXRqbkhoSkJZVkIw?=
 =?utf-8?B?dCtubzJkNHg1WVVUdzc0QkVnNzVNdHZXZmozRm5XTFRvbjYwdy9JbzdkRWpG?=
 =?utf-8?B?cHVjTGt4SzBzR1M5d1RDVDdFN1NERjNWOFN2VEZzVXY3UHhFMURVWHlHdzJS?=
 =?utf-8?B?a0h3Wi9QeTJoY0dYcGgvY1E0K2d0NjNxRjZGZmxSZ1lYUTJIQ2w0cHB0Ujhj?=
 =?utf-8?B?OTFmellNRnZjV3V6OHZNYjZQTnVpcnNQVFl6K0lXdi9oNzBzaGNGMFlyaCs4?=
 =?utf-8?B?eVo5bDdkdkJmV2tOZVgwVkJWV1BVVkFvSXRKZWRCaUczWkY3RlFJZHQyeWdh?=
 =?utf-8?B?M2JobUxWVTFxOUZIZy9SQnovNm95TENIZXFMTXA3WjFxNlNyR3lwVjRIVWcy?=
 =?utf-8?B?NEFkMVRhZDFaWGlQVzFnN3cxR2t0eWFTY3hPZFRqdUU4MUVFUmRDeHJPZnRZ?=
 =?utf-8?B?ZFN0eVkzOTRFaGVWdjU1WElDdUFkSUpyZ09ldnZENlgrZ25yVVZoSSt2MzVY?=
 =?utf-8?B?TE4wY0pyTDh2RGxvYy9mNWlpY09WS0JRRnRDejQ1SjhuTUZBdDZXcGpRSzlk?=
 =?utf-8?B?SFVjTDJVQTZxVFRUeEEvUUUrNzhKc1ZXYjZ1U0x3bUlrSnRoSFRKRlI0L2tD?=
 =?utf-8?B?UUZXbHptc2d6bXN5SWpNSkZkajA2ZnJCMHQ0KzhncHpGZDhFeGFEQ29CeCsy?=
 =?utf-8?B?ZFZhZmRJOWJab2Z0VE1WWUpBTStQb2lyMklrZ05LaVBVNkR6TDVyWW9pMmFl?=
 =?utf-8?B?a0FDZEI4TmpsQ2t0aENoRUZHOTNMMVBNNWk3amJoSGhxYnc0QlYzdy93TTBr?=
 =?utf-8?B?R0Fub3lmWVE0MTJYaEVnMDBtNnlnPT0=?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 06dd5b24-97a2-4488-ebbd-08dab5d8aaad
X-MS-Exchange-CrossTenant-AuthSource: CO6PR12MB5444.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 24 Oct 2022 15:59:01.0883
 (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: eBMGav6l7zisExDnJQMbxZNYMLv/OP+yQHxlEBP7usCiUwAUa6lafhZVMbt4RpVbiZO/wQbk9UMa38kZ0zaTOg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY5PR12MB4323
X-Spam-Status: No, score=-1.6 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,SPF_HELO_PASS,SPF_NONE autolearn=no 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 24/10/2022 14:30, Rob Herring wrote:
> On Mon, 24 Oct 2022 15:41:19 +0800, Wayne Chang wrote:
>> Add device-tree binding documentation for the XUSB host controller present
>> on Tegra194 and Tegra234 SoC. This controller supports the USB 3.1
>> specification.
>>
>> Signed-off-by: Wayne Chang <waynec@nvidia.com>
>> ---
>>   .../bindings/usb/nvidia,tegra-xhci.yaml       | 213 ++++++++++++++++++
>>   1 file changed, 213 insertions(+)
>>   create mode 100644 Documentation/devicetree/bindings/usb/nvidia,tegra-xhci.yaml
>>
> 
> 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:
> Error: Documentation/devicetree/bindings/usb/nvidia,tegra-xhci.example.dts:36.27-28 syntax error
> FATAL ERROR: Unable to parse input tree
> make[1]: *** [scripts/Makefile.lib:406: Documentation/devicetree/bindings/usb/nvidia,tegra-xhci.example.dtb] Error 1
> make[1]: *** Waiting for unfinished jobs....
> make: *** [Makefile:1492: dt_binding_check] Error 2


I believe that this is because another DT patch [0] is missing (see 
patch 0/11). Thierry has just picked this up for -next and so hopefully 
will resolve this.

Cheers
Jon

[0] 
https://lore.kernel.org/all/20221003125141.123759-1-jonathanh@nvidia.com/

-- 
nvpublic