Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:6 as permitted sender) client-ip=2620:137:e000::3:6;
Date:   Wed, 18 Oct 2023 22:52:32 +0200
From:   Uwe =?utf-8?Q?Kleine-K=C3=B6nig?= <u.kleine-koenig@pengutronix.de>
To:     Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Cc:     Devi Priya <quic_devipriy@quicinc.com>, agross@kernel.org,
        andersson@kernel.org, konrad.dybcio@linaro.org, lee@kernel.org,
        robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org,
        conor+dt@kernel.org, thierry.reding@gmail.com,
        ndesaulniers@google.com, trix@redhat.com, baruch@tkos.co.il,
        linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org, llvm@lists.linux.dev,
        linux-pwm@vger.kernel.org, nathan@kernel.org
Subject: Re: [PATCH V15 0/4] Add PWM support for IPQ chipsets
Message-ID: <20231018205232.p5d4xaqsd5fcbvfi@pengutronix.de>
References: <20231005160550.2423075-1-quic_devipriy@quicinc.com>
 <f87f480d-6775-44c9-8a2b-05d47bb0dc5d@linaro.org>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="6foex2ayshpkzjy2"
Content-Disposition: inline
In-Reply-To: <f87f480d-6775-44c9-8a2b-05d47bb0dc5d@linaro.org>
Precedence: bulk


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

Hello,

On Wed, Oct 18, 2023 at 06:29:30PM +0200, Krzysztof Kozlowski wrote:
> On 05/10/2023 18:05, Devi Priya wrote:
> > Add PWM driver and binding support for IPQ chipsets.
> > Also, add support for pwm node in ipq6018.
> >=20
>=20
> You need to clearly mark dependencies.

This is something I wouldn't blame Devi for. The dependency is not very
obvious and its kind of normal and expected for a patch series to have
dependencies. *shrug*

> Next is now broken because of this patchset.

If I understand correctly this affects "only" the dtb check targets,
right? Is this bad enough that it needs an urgent fix? I would expect it
doesn't hurt much, am I right here?

I just looked into patch #2 and had a few comments for it.

Best regards
Uwe

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

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

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

iQEzBAABCgAdFiEEP4GsaTp6HlmJrf7Tj4D7WH0S/k4FAmUwRZAACgkQj4D7WH0S
/k4XJQgAp4VfEVwgDzWlPPCnQWB0m+MuYItgUfBPVM+8B48xCCnkP5UocEn4hI3q
Zjek7v+tQBAXIckXu6RndUjFF2H5PrV4f2EpUAyVtyONFLA6zl4fd3LxiXZsOS6/
LeTITpIekRlPZFRh3pvscS8O2/ZkXfU3PYV2jb1Ch4woaVYAui2IcmcqsyWVUE9J
Wd4hLPLNHCAoK4aYz3xEnNOM2/x9LT2KqxndiTFtpZS6auOf+dWVJndAehIPbXsC
mzSNBelj8MItVL8kucOY34s6DLj1GPTSSklE1yyM3W9TQ7Nn0xKbbaem/XqkG/Gv
lfiB9x/hb7yFBv3crIPZfiRd5kLgmg==
=XXnl
-----END PGP SIGNATURE-----

--6foex2ayshpkzjy2--