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:   Wed, 13 Jul 2022 08:11:43 +0200
From:   Uwe =?utf-8?Q?Kleine-K=C3=B6nig?= <u.kleine-koenig@pengutronix.de>
To:     Ben Dooks <ben.dooks@sifive.com>
Cc:     Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
        linux-pwm@vger.kernel.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org, Lee Jones <lee.jones@linaro.org>,
        Thierry Reding <thierry.reding@gmail.com>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Greentime Hu <greentime.hu@sifive.com>,
        Jude Onyenegecha <jude.onyenegecha@sifive.com>,
        Sudip Mukherjee <sudip.mukherjee@sifive.com>,
        William Salmon <william.salmon@sifive.com>,
        Adnan Chowdhury <adnan.chowdhury@sifive.com>
Subject: Re: [PATCH 6/7] pwm: dwc: remove the CONFIG_OF in timer clock
Message-ID: <20220713061143.qvgt5wry3onbswlg@pengutronix.de>
References: <20220712100113.569042-1-ben.dooks@sifive.com>
 <20220712100113.569042-7-ben.dooks@sifive.com>
 <bcbd1b29-7c6d-1d1d-2c72-b8818e99c65c@linaro.org>
 <77c97281-96d9-c969-18cf-6e609098aa57@sifive.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="yywwgvrzbnvxjnxs"
Content-Disposition: inline
In-Reply-To: <77c97281-96d9-c969-18cf-6e609098aa57@sifive.com>
Precedence: bulk


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

On Tue, Jul 12, 2022 at 11:20:23AM +0100, Ben Dooks wrote:
> On 12/07/2022 11:09, Krzysztof Kozlowski wrote:
> > On 12/07/2022 12:01, Ben Dooks wrote:
> > > We should probably change from the #ifdef added earlier in
> > > 49a0f4692a8752c7b03cb26d54282bee5c8c71bb ("wm: dwc: add timer clock")
> > > and just have it always in the dwc data so if we have a system with
> > > both PCI and OF probing it should work
> > >=20
> > > -- consider merging with original patch
> >=20
> > Missing SoB. Please run checkpatch.
>=20
> This was meant to be an RFC about whether it should be a single patch
> or merged back into the previous one.

+1 for merging these

Best regards
Uwe

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

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

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

iQEzBAABCgAdFiEEfnIqFpAYrP8+dKQLwfwUeK3K7AkFAmLOYhwACgkQwfwUeK3K
7Anlkgf+PzfFU+vh+ZG2uD+0UJ1DqmgG+ptqG5hMxqBIBY95jJTmFQhXsDZ4MXGG
cAp8rQObOhcDNgR8QBXVghQpeFMFdAPnV4j3bLERToB0Mqatd6JXi8VyN0ji27or
RbCiDnSf+gBWa6t9tk6i6PHi3Eg1qMyRUtRmORruJNt1d/B2XKz6v2eAWWHIsJH0
6Sf2U8Niqg8q26bMZJqe1UHxXok1omEQj/7yjp5MwM+gpNUQ8kBTcc/9SB3Vao4v
Tm0ikwxMYVsBQEez7dnx3flHmwoKhJHt1uwKi7DPQVA3Qf1G/NiALr32gqk0etqc
dZ61QOY3ZKu18/Hzgiste1wMU02GbQ==
=Yud/
-----END PGP SIGNATURE-----

--yywwgvrzbnvxjnxs--