Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18;
Date:   Mon, 30 May 2022 22:51:20 +0200
From:   Fabien Parent <fparent@baylibre.com>
To:     Uwe =?utf-8?Q?Kleine-K=C3=B6nig?= <u.kleine-koenig@pengutronix.de>
Cc:     Thierry Reding <thierry.reding@gmail.com>,
        Lee Jones <lee.jones@linaro.org>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Matthias Brugger <matthias.bgg@gmail.com>,
        linux-pwm@vger.kernel.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
        linux-mediatek@lists.infradead.org
Subject: Re: [PATCH 1/2] dt-bindings: pwm: add MT8365 SoC binding
Message-ID: <20220530205120.q3gi2fhua5lbxnmj@helium.lan>
References: <20220530202136.906407-1-fparent@baylibre.com>
 <20220530203101.5wq52q5pq523ewuu@pengutronix.de>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="kbnb43vxe2gkvrvk"
Content-Disposition: inline
In-Reply-To: <20220530203101.5wq52q5pq523ewuu@pengutronix.de>
Precedence: bulk


--kbnb43vxe2gkvrvk
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Mon, May 30, 2022 at 10:31:01PM +0200, Uwe Kleine-K=F6nig wrote:
> On Mon, May 30, 2022 at 10:21:34PM +0200, Fabien Parent wrote:
> > Add binding documentation for the MT8365 SoC.
> >=20
> > Signed-off-by: Fabien Parent <fparent@baylibre.com>
> > ---
> >  Documentation/devicetree/bindings/pwm/pwm-mediatek.txt | 2 ++
> >  1 file changed, 2 insertions(+)
> >=20
> > diff --git a/Documentation/devicetree/bindings/pwm/pwm-mediatek.txt b/D=
ocumentation/devicetree/bindings/pwm/pwm-mediatek.txt
> > index 25ed214473d7..ac5c58bbb22d 100644
> > --- a/Documentation/devicetree/bindings/pwm/pwm-mediatek.txt
> > +++ b/Documentation/devicetree/bindings/pwm/pwm-mediatek.txt
> > @@ -8,6 +8,7 @@ Required properties:
> >     - "mediatek,mt7628-pwm": found on mt7628 SoC.
> >     - "mediatek,mt7629-pwm": found on mt7629 SoC.
> >     - "mediatek,mt8183-pwm": found on mt8183 SoC.
> > +   - "mediatek,mt8365-pwm": found on mt8365 SoC.
> >     - "mediatek,mt8516-pwm": found on mt8516 SoC.
> >   - reg: physical base address and length of the controller's registers.
> >   - #pwm-cells: must be 2. See pwm.yaml in this directory for a descrip=
tion of
> > @@ -17,6 +18,7 @@ Required properties:
> >                  has no clocks
> >     - "top": the top clock generator
> >     - "main": clock used by the PWM core
> > +   - "pwm1-3": the five per PWM clocks for mt8365
>=20
> I think you either want s/3/5/ or s/five/three/ here?!

Thanks. Fixed in v2.

BR,
Fabien

>=20
> Best regards
> Uwe
>=20
> --=20
> Pengutronix e.K.                           | Uwe Kleine-K=F6nig          =
  |
> Industrial Linux Solutions                 | https://www.pengutronix.de/ |


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

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

iQIzBAABCgAdFiEEQyJHfU3UNW7IG61XVJKc6g3CeAAFAmKVLkcACgkQVJKc6g3C
eAD8MhAAq04bhlW59dA7G2QxeUOL60bMUzB6aHtyUWFSLDgMwO410j+eNRD2IT4w
pcuyVtI/DxqdHKqEGxD0on8zwRF2yEN3jxwdKFxoQ7+CDINxWKqyvk6HAZtNmEw8
eEq5xNXGTowOIwkvVoKETjRbnzEnmB89kLYCg+SqlE+8H2LW8GjhhGtFU2Q/8K7v
hsAzwb7rWNwnYLoVdl1gu1he78z//mq5hyNzWVspVS3ZCeekxmd/0dlG+Gz6sEwv
EVJFljX3PwhZZsWYGZGql/FiSwslrtZIDLTvv5UQK8vID7/NhE8lBRsreSiwVux6
8wqvB34HO3eDnRjflHpgyjnzJGCuQa/DN05paCv14abQAig9eSdhbmLpLnp/6N1j
/AQ2jCu8sNhJUzqetocp87qEmzvlYgKbvSv7NfiCklk1MU5cv/KB1tF+V1MpiSTO
P5t6hZVqMG3511ES0Im1VUCQ2WqsMVPhe2PAHmek0hJoUyPIkFGOz0bg04/hDG1s
H25LL/dHcHk8pU3SaM0QnLmxmLYmzEZ9nX4EK8hK0IyhRir3G6u1ruEkr46/VczL
9SNUPumDv3nQvJ9QcrsNooDdPUo0enRX+IiZe2G7QzZCyPW6LdCQFFgPBYAKSjf/
oUONf/ORm1vqWTP7DxtQRTsKiuyeDat+Yt3DgIE4ArZKbUP5LfI=
=qbYd
-----END PGP SIGNATURE-----

--kbnb43vxe2gkvrvk--