Re: Re: [PATCH v2 1/2] dt-bindings: usb: Add Eswin EIC7700 USB controller

[zhangsenchuan <zhangsenchuan@eswincomputing.com>]

> -----原始邮件-----
> 发件人: "Krzysztof Kozlowski" <krzk@kernel.org>
> 发送时间:2025-07-30 15:46:02 (星期三)
> 收件人: zhangsenchuan@eswincomputing.com, gregkh@linuxfoundation.org, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, Thinh.Nguyen@synopsys.com, p.zabel@pengutronix.de
> 抄送: ningyu@eswincomputing.com, linmin@eswincomputing.com, yangwei1@eswincomputing.com, pinkesh.vaghela@einfochips.com
> 主题: Re: [PATCH v2 1/2] dt-bindings: usb: Add Eswin EIC7700 USB controller
> 
> On 30/07/2025 09:40, zhangsenchuan@eswincomputing.com wrote:
> > +
> > +examples:
> > +  - |
> > +    soc {
> > +        #address-cells = <2>;
> > +        #size-cells = <2>;
> 
> This wasn't here. Drop.
> 
> > +
> > +        usb {
> > +            compatible = "eswin,eic7700-dwc3";
> > +            clocks = <&fixed_factor_u_clk_1m_div24>,
> > +                     <&gate_clk_hsp_aclk>,
> > +                     <&gate_clk_hsp_cfgclk>;
> > +            clock-names = "suspend","aclk", "cfg";
> > +            eswin,hsp-sp-csr = <&hsp_sp_csr 0x800 0x808 0x83c 0x840>;
> > +            resets = <&reset 84>;
> > +            reset-names = "vaux";
> > +            ranges;
> 
> Same comment: follow DTS coding style. Where should ranges or reg be placed?

According to the "dts-coding-style.html", the order should be like below, is this correct?

compatible = "eswin,eic7700-dwc3";
ranges;
#address-cells = <1>;
#size-cells = <1>;

> 
> > +            #address-cells = <2>;
> > +            #size-cells = <2>;
> > +
> > +            usb@50480000 {
> > +                compatible = "snps,dwc3";
> > +                reg = <0x0 0x50480000 0x0 0x10000>;
> 
> Nothing improved and you did not respond to the feedback.

I'm very sorry that I haven't given you any feedback here. I'm not quite clear about what you mean by feedback.
In v1 patch, you mentioned "Fold the child node into the parent"，could you please tell me what's the exact
meaning of this comment?
> 
> 
> Best regards,
> Senchuan Zhang