From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: 21 May 2019 17:08:41 +0900 Message-ID: <877eaktf8i.wl-kuninori.morimoto.gx@renesas.com> From: Kuninori Morimoto Subject: Re: [PATCH v6 1/7] dt-bindings: usb: hd3ss3220 device tree binding document In-Reply-To: References: <1557922152-16449-1-git-send-email-biju.das@bp.renesas.com> <1557922152-16449-2-git-send-email-biju.das@bp.renesas.com> MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") Content-Type: text/plain; charset=US-ASCII To: Yoshihiro Shimoda Cc: Biju Das , Rob Herring , Mark Rutland , Greg Kroah-Hartman , Heikki Krogerus , Felipe Balbi , "linux-usb@vger.kernel.org" , "devicetree@vger.kernel.org" , Simon Horman , Geert Uytterhoeven , Chris Paterson , Fabrizio Castro , "linux-renesas-soc@vger.kernel.org" List-ID: Hi > > +Required properties: > > + - compatible: Must be "ti,hd3ss3220". > > + - reg: I2C slave address, must be 0x47 or 0x67 based on ADDR pin. > > + - interrupts: where a is the interrupt number and b represents an > > + encoding of the sense and level information for the interrupt. > > + > > +Required sub-node: > > + - connector : The "usb-c-connector" attached to the hd3ss3220 chip. The > > s/connector :/connector:/ Maybe it is for alignment ? > According to the connector/usb-connector.txt, should the connector node > have ports, port@1 and an endpoint nodes like below? "ports" is needed if it has multiple "port", otherwise, single port is allowed from OF-graph point of view. Thank you for your help !! Best regards --- Kuninori Morimoto