Hi Morimoto-San and Shimoda-San, Thanks for the feedback. > Subject: Re: [PATCH v6 1/7] dt-bindings: usb: hd3ss3220 device tree binding > document > > > Hi > > > > +Required properties: > > > + - compatible: Must be "ti,hd3ss3220". > > > + - reg: I2C slave address, must be 0x47 or 0x67 based on ADDR pin. > > > + - interrupts: <a b> 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 ? Yes, I need to fix the extra space. > > 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. OK. I will use single port on the next patch series. Regards, Biju