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:   Tue, 4 Apr 2023 21:52:54 +0200
From:   Thierry Reding <thierry.reding@gmail.com>
To:     Rob Herring <robh@kernel.org>
Cc:     Sudeep Holla <sudeep.holla@arm.com>,
        Cristian Marussi <cristian.marussi@arm.com>,
        Jonathan Hunter <jonathanh@nvidia.com>,
        devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
        linux-kernel@vger.kernel.org, linux-tegra@vger.kernel.org
Subject: Re: [PATCH] firmware: Use of_property_present() for testing DT
 property presence
Message-ID: <ZCyAFhY0xVy8Q0Yk@orome>
References: <20230310144704.1542045-1-robh@kernel.org>
 <20230323120030.7mk62xk7tco32zw2@bogus>
 <ZCwmWoZhMOlHnnzf@orome>
 <20230404165846.GA49361-robh@kernel.org>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
        protocol="application/pgp-signature"; boundary="EV9WiWkvTmZhpxvQ"
Content-Disposition: inline
In-Reply-To: <20230404165846.GA49361-robh@kernel.org>
User-Agent: Mutt/2.2.10 (2023-03-25)
Precedence: bulk


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

On Tue, Apr 04, 2023 at 11:58:46AM -0500, Rob Herring wrote:
> On Tue, Apr 04, 2023 at 03:30:02PM +0200, Thierry Reding wrote:
> > On Thu, Mar 23, 2023 at 12:00:30PM +0000, Sudeep Holla wrote:
> > > On Fri, Mar 10, 2023 at 08:47:04AM -0600, Rob Herring wrote:
> > > > It is preferred to use typed property access functions (i.e.
> > > > of_property_read_<type> functions) rather than low-level
> > > > of_get_property/of_find_property functions for reading properties. =
As
> > > > part of this, convert of_get_property/of_find_property calls to the
> > > > recently added of_property_present() helper when we just want to te=
st
> > > > for presence of a property and nothing more.
> > > >=20
> > > > Signed-off-by: Rob Herring <robh@kernel.org>
> > > > ---
> > > >  drivers/firmware/arm_scmi/optee.c | 2 +-
> > >=20
> > > Acked-by: Sudeep Holla <sudeep.holla@arm.com>(for the SCMI part)
> > >=20
> > > I am assuming you will route this as it has other changes as well in =
the
> > > patch.
> >=20
> > I can also pick this up along with the Tegra changes if Rob doesn't
> > want to take it. But also feel free to take this through whatever tree
> > works best, for the Tegra parts:
> >=20
> > Acked-by: Thierry Reding <treding@nvidia.com>
>=20
> You can take it. Thanks.

Applied now, thanks.

Thierry

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

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

iQIzBAABCAAdFiEEiOrDCAFJzPfAjcif3SOs138+s6EFAmQsgBYACgkQ3SOs138+
s6EnWg//c7WQrukK4feUgU8dH0K8IO0ecq7TdKAaG2qzAY/nmrcVweH4fkoOQzu5
RMw7p2Cwc6gS+oty/bX/jWBuFAKVYfyjDCYmRPbvhwQD4DxvD6uT7v+DmIGA5aCu
5cwT/I5vLLqJZhxbmecB+ByLRe2EO8TKTCKs7G3iydO/OUt8wW3DeOzGf/Rr3568
+ZntYrBRXvJ6KBBJ8ghTr54daiJK2AN6QGX8GFvtzxGZ77733DPuOI5LImK/j6vw
czfc0BpUWTmNnlKAm82b729R32gTGWHlE50tb6MLQaO4Z0qjt97K+fCxPfcuVut+
2wqMMYRT+ZyI07ySV5o3YKuXYFoqBAZeHEkuQ3OUoYH6KS47x6A86HbXhfohU6wT
iNstMGArkdF4eFOc846WZuKDqiY3PFCp3fNN/pF8T4Jkt77tuM1EDU6O5/pDkTg9
FZ7KOFRqbNV00SnM52M+8TJ99GaugoxmXbsxX/aQh3rqiqOBrBbdGjF9UbphFtQD
lx7ZSSgTCtuMOBl5gbDeJCxBsfZCnsPdtFA2LzsR44dTzOyOaGWNkzd65EjVz0gG
P4/mcvxoxMj9h08QHuzHtmKIxam9x34UyCoXdmYM1LhKYi/SHsSNX8Ehk6oU09xb
xVMIeWlKk5hO5hmr7IdyO0vFR0wPjzvrOYKCBJOK/kKNsH4hBi0=
=M5/M
-----END PGP SIGNATURE-----

--EV9WiWkvTmZhpxvQ--