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;
Date:   Mon, 31 Jul 2023 17:12:24 +0200
From:   Sebastian Reichel <sebastian.reichel@collabora.com>
To:     Conor Dooley <conor@kernel.org>
Cc:     Heiko Stuebner <heiko@sntech.de>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Thinh Nguyen <Thinh.Nguyen@synopsys.com>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Conor Dooley <conor+dt@kernel.org>,
        linux-rockchip@lists.infradead.org, linux-usb@vger.kernel.org,
        devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org, kernel@collabora.com
Subject: Re: [PATCH v2 1/3] dt-bindings: usb: add rk3588 compatible to
 rockchip,dwc3
Message-ID: <20230731151224.2vlk7bnabrqhz3pr@mercury.elektranox.org>
References: <20230720173643.69553-1-sebastian.reichel@collabora.com>
 <20230720173643.69553-2-sebastian.reichel@collabora.com>
 <20230722-coleslaw-breeder-40827e23f717@spud>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="m6dzip35wy4zbmme"
Content-Disposition: inline
In-Reply-To: <20230722-coleslaw-breeder-40827e23f717@spud>
Precedence: bulk


--m6dzip35wy4zbmme
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

Hi,

On Sat, Jul 22, 2023 at 12:42:09PM +0100, Conor Dooley wrote:
> On Thu, Jul 20, 2023 at 07:36:41PM +0200, Sebastian Reichel wrote:
> > RK3588 has three DWC3 controllers. Two of them are fully functional in
> > host, device and OTG mode including USB2 support. They are connected to
> > dedicated PHYs, that also support USB-C's DisplayPort alternate mode.
> >=20
> > The third controller is connected to one of the combphy's shared
> > with PCIe and SATA. It can only be used in host mode and does not
> > support USB2. Compared to the other controllers this one needs
> > some extra clocks.
> >=20
> > Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>
>=20
> I feel like I say it a bunch for some of these Rockchip bindings
> patches, but if you're adding more clocks for some SoCs, should some
> per-SoC constraints not also be added?

The extra clocks are not actually needed by all the USB3 controllers
in the SoC. Only one of three USB3 controllers needs them. In v1 I
used different compatible values to narrow the clock binding down
and Krzysztof asked to remove that. So please tell me what it should
look like.

Greetings,

-- Sebastian

> > ---
> >  .../devicetree/bindings/usb/rockchip,dwc3.yaml        | 11 +++++++++--
> >  1 file changed, 9 insertions(+), 2 deletions(-)
> >=20
> > diff --git a/Documentation/devicetree/bindings/usb/rockchip,dwc3.yaml b=
/Documentation/devicetree/bindings/usb/rockchip,dwc3.yaml
> > index 291844c8f3e1..cbc3e55e05e1 100644
> > --- a/Documentation/devicetree/bindings/usb/rockchip,dwc3.yaml
> > +++ b/Documentation/devicetree/bindings/usb/rockchip,dwc3.yaml
> > @@ -30,6 +30,7 @@ select:
> >          enum:
> >            - rockchip,rk3328-dwc3
> >            - rockchip,rk3568-dwc3
> > +          - rockchip,rk3588-dwc3
> >    required:
> >      - compatible
> > =20
> > @@ -39,6 +40,7 @@ properties:
> >        - enum:
> >            - rockchip,rk3328-dwc3
> >            - rockchip,rk3568-dwc3
> > +          - rockchip,rk3588-dwc3
> >        - const: snps,dwc3
> > =20
> >    reg:
> > @@ -58,7 +60,9 @@ properties:
> >            Master/Core clock, must to be >=3D 62.5 MHz for SS
> >            operation and >=3D 30MHz for HS operation
> >        - description:
> > -          Controller grf clock
> > +          Controller grf clock OR UTMI clock
> > +      - description:
> > +          PIPE clock
> > =20
> >    clock-names:
> >      minItems: 3
> > @@ -66,7 +70,10 @@ properties:
> >        - const: ref_clk
> >        - const: suspend_clk
> >        - const: bus_clk
> > -      - const: grf_clk
> > +      - enum:
> > +          - grf_clk
> > +          - utmi
> > +      - const: pipe
> > =20
> >    power-domains:
> >      maxItems: 1
> > --=20
> > 2.40.1
> >=20


--m6dzip35wy4zbmme
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQIzBAABCgAdFiEE72YNB0Y/i3JqeVQT2O7X88g7+poFAmTHz0AACgkQ2O7X88g7
+pqKjg/+OYBPYuk3hSKFWkamZHuGn2rDB7jeHnXzTF7PG/ehiqqifOqn2ro376hZ
Ays6PKMQR+4a+te/CSgLYvH5jc0AB+4v6tA5olENnyRF8XfMaUB6LGPv/HDBWU1v
yhmleCXSeBGcJwpoxsB8UMhhUdzaXlLJ3tRfrtsJJnSOL2Hu8FHif/5RQ8BIoZg/
Q89X9a2zYUlevNHiK63HIjih60/FagN7FvfuPRxz0OpZYewsFIJVfX6SltPuUUNa
p6HuuEcbVzWklTshBax2IKSfT/DHEZAtdcuRaUhZ/7Xiy0BPvOHQQIunh8oKzLde
MquuX8YesnPH/VQzWIElIUOej2RsOMNYE/o/Fw8jRL/j2jQtvUWYHr/dbQy940BW
Pvci2+YO5TQtscLPIAngEqD65VGco9pzbZo9C2cMNXik/LmE+AGomcAFt/Ea4o3l
Cq3UL87xcO/nIuj/wsaNfH0o9CZSbl6tOx4QTgt4EouXDk4kh3prLslOEnhhBGT+
zB4Zbk7ZwH6Q8gWZdo5ED7JiXRsVx6W2NHdFAAkZMq2B/NCKykV/MI2hDDoO6AXC
wd6cGFMCQKrmNO1bWtPBL3jBwwQadZf3M4wtBagB3AsE5UUhc+5JwjZEKWIKfnd3
pRKq0qyv6KC/QaySyXYf1d3094JZL3Ks4rvKAlOHlKuHYesdzWU=
=qk2b
-----END PGP SIGNATURE-----

--m6dzip35wy4zbmme--