Received: by 2002:a05:6a10:9848:0:0:0:0 with SMTP id x8csp691605pxf; Thu, 1 Apr 2021 11:02:12 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxqDH50V9aTV+imaxQTHc/yzj59FkXzW3jqhqL1gQJRQkiVBJI5E+TTQJ+8X/NbyX2PrqgW X-Received: by 2002:a17:906:8583:: with SMTP id v3mr10624199ejx.361.1617300132106; Thu, 01 Apr 2021 11:02:12 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1617300132; cv=none; d=google.com; s=arc-20160816; b=f61Box9gns7GhHIwmHNSNQ5Y6nYPfC6yG/neP04Kq9OV0Tuu7PwrHRkApK/ZvX5drl OMVRd2r5G9UIVXIiAgyKLs3wbGoKNRSDwA5RkL9iNwXrSGKknX1GjQo2VeQWGFTO1I6c 2FOtm3ODZJqh7CFRu2EAzBXpyd44gvqo764t8JYTp/4ioNzjjs34kxbWJVw1ocq8MCCl 84mbvhqewAw0Dz9CSFf9La28am90YgA7Ylo4YecBZ9WZ7ew7IXI3bsCMPuO1VgEhNFoJ dOvjrWXJfXFFGBGeHkuFfNraMpIunF7AzbgZSlrUZpDTO1/GK4froqK6TK2tVkPkkVxQ UJ9g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date; bh=pBHNevxyUCCPbFPV2Zb7KUDNOhpcn/AKmNJD00Mb8Us=; b=RxGoqYJsfsZixY8uoX2bzjIYvDR0ocb5nNC/Ado61J0JGyIMltwlqzj0PEW8u4yfcG rDPpINwygXSgBRDxFwFaAi4GIl5rUTRWUpQ95ZigZxSgYY5VElM5wBhW/kbhRzu8wDFU F7lKlbjrEg9R6w4IAdMg4ogAabtWmfWmhrmLfPuo1AKXzDcfumtqbbfLL4GsDzJs0zEj rVIQM7p3T0IxhHEZf1m17PMhd5IJp5eRyobrpuPc6VbV9f8OPwyA/daSgEhASVhNko2d nm7kUN0ahmCO5fs2wrxzSVk07Z9kn3xcNz+WVqYj2SBopC5nT/aF6lqvvy7x6UObmlzv DWIg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id w12si4839790edx.14.2021.04.01.11.01.47; Thu, 01 Apr 2021 11:02:12 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S237525AbhDASAT (ORCPT + 99 others); Thu, 1 Apr 2021 14:00:19 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:58342 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S236316AbhDARoU (ORCPT ); Thu, 1 Apr 2021 13:44:20 -0400 Received: from mail-ot1-x336.google.com (mail-ot1-x336.google.com [IPv6:2607:f8b0:4864:20::336]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id D8A61C0319C1; Thu, 1 Apr 2021 10:16:09 -0700 (PDT) Received: by mail-ot1-x336.google.com with SMTP id m21-20020a9d7ad50000b02901b83efc84a0so2726723otn.10; Thu, 01 Apr 2021 10:16:09 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=pBHNevxyUCCPbFPV2Zb7KUDNOhpcn/AKmNJD00Mb8Us=; b=pHRqoUUn2yHtI6TXAo3BT4Zo/1ZUyAFKfF9fQpbiQVgJQmy2YCfAnNKnS8MsfYNFWi K09YlE2Y7XjxodJ6jYsduUXP9vMT+dK8AyTMovvnE9ftsfxQzStCM+kdLUb+sw1rngJx pqKswVRKG/nRxzTrEUqPEybeyb9U7FtDJsBbM6ziiiqbHmMmltIzr0Xi/RaaokdNsS3i 4px6N/0JF3p44YnIEt08VCn1oSBtNmfDN43kZdJ0mRCiM35xhV7sdfQwteZzEFhECwIc WWwpR4JHIcX5MUPdJV93EDZJf7gh22ixAq7ryc3rgPCzRG6xOh+niFy+Ec/EPp3zzMCy AKdQ== X-Gm-Message-State: AOAM530i9e4nr28vXY1VyljhhyMPWvdy1swaUd1gIuP+sdpRNUkzNEqo H7p4i05chikB64tveTQI0oIWDQr76g== X-Received: by 2002:a9d:7481:: with SMTP id t1mr7796126otk.208.1617297366044; Thu, 01 Apr 2021 10:16:06 -0700 (PDT) Received: from robh.at.kernel.org (24-155-109-49.dyn.grandenetworks.net. [24.155.109.49]) by smtp.gmail.com with ESMTPSA id f8sm1241470otp.71.2021.04.01.10.16.03 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 01 Apr 2021 10:16:04 -0700 (PDT) Received: (nullmailer pid 633945 invoked by uid 1000); Thu, 01 Apr 2021 17:16:03 -0000 Date: Thu, 1 Apr 2021 12:16:03 -0500 From: Rob Herring To: cristian.birsan@microchip.com Cc: linux@roeck-us.net, heikki.krogerus@linux.intel.com, gregkh@linuxfoundation.org, linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org Subject: Re: [RFC PATCH v2 1/2] dt-bindings: usb: Add DT bindings for Microchip sama7g5 tcpc Message-ID: <20210401171603.GA624021@robh.at.kernel.org> References: <20210330205442.981649-1-cristian.birsan@microchip.com> <20210330205442.981649-2-cristian.birsan@microchip.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210330205442.981649-2-cristian.birsan@microchip.com> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Mar 30, 2021 at 11:54:41PM +0300, cristian.birsan@microchip.com wrote: > From: Cristian Birsan > > This patch adds DT bindings for the new Microchip USB Type-C Port > Controller (TCPC) embedded in sama7g5 SoC. > > Signed-off-by: Cristian Birsan > --- > .../bindings/usb/microchip,sama7g5-tcpc.yaml | 90 +++++++++++++++++++ > 1 file changed, 90 insertions(+) > create mode 100644 Documentation/devicetree/bindings/usb/microchip,sama7g5-tcpc.yaml > > diff --git a/Documentation/devicetree/bindings/usb/microchip,sama7g5-tcpc.yaml b/Documentation/devicetree/bindings/usb/microchip,sama7g5-tcpc.yaml > new file mode 100644 > index 000000000000..2d3289dd9ac9 > --- /dev/null > +++ b/Documentation/devicetree/bindings/usb/microchip,sama7g5-tcpc.yaml > @@ -0,0 +1,90 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: "http://devicetree.org/schemas/usb/microchip,sama7g5-tcpc.yaml#" > +$schema: "http://devicetree.org/meta-schemas/core.yaml#" > + > +title: Microchip SAMA7G5 Type-C port controller bindings > + > +maintainers: > + - Cristian Birsan > + > +properties: > + compatible: > + enum: > + - microchip,sama7g5-tcpc > + > + reg: > + maxItems: 1 > + > + clocks: > + items: > + - description: Peripheral Bus Clock > + - description: Generic Clock > + > + clock-names: > + items: > + - const: pclk > + - const: gclk > + > + vbus-supply: > + description: vbus power supply (5V) phandle This belongs in the connector node unless this chip uses Vbus as a supply. > + > + microchip,vbus-gpio: > + description: vbus detect gpio We have standard property for this in the connector node. > + > + connector: > + type: object > + > + allOf: > + - $ref: ../connector/usb-connector.yaml You can drop 'allOf'. And add: unevaluatedProperties: false > + > + properties: > + compatible: > + const: usb-c-connector > + > + power-role: true > + > + required: > + - compatible > + > +required: > + - compatible > + - reg > + - connector > + > +additionalProperties: false > + > +examples: > + - | > + #include > + #include > + tcpcb: tcpcb@e0844000 { > + compatible = "microchip,sama7g5-tcpc"; > + reg = <0xe0844000 0x3fff>; > + clocks = <&pmc PMC_TYPE_PERIPHERAL 95>, <&pmc PMC_TYPE_GCK 95>; > + clock-names = "pclk", "gclk"; > + > + usb_con: connector { > + #address-cells = <1>; > + #size-cells = <0>; > + compatible = "usb-c-connector"; > + label = "USB-C"; > + data-role = "device"; > + power-role = "sink"; > + try-power-role = "sink"; > + source-pdos = + PDO_FIXED_USB_COMM)>; > + sink-pdos = + PDO_FIXED_USB_COMM)>; > + op-sink-microwatt = <2500000>; > + > + port@1 { > + reg = <1>; > + usb_con_hs: endpoint { > + remote-endpoint = <&usb_phy1>; > + }; > + }; > + }; > + }; > +... > -- > 2.25.1 >