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, 17 Jul 2023 09:18:06 +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>,
        Shawn Guo <shawnguo@kernel.org>,
        Sascha Hauer <s.hauer@pengutronix.de>,
        Pengutronix Kernel Team <kernel@pengutronix.de>,
        Fabio Estevam <festevam@gmail.com>,
        NXP Linux Team <linux-imx@nxp.com>,
        Matthias Brugger <matthias.bgg@gmail.com>,
        AngeloGioacchino Del Regno 
        <angelogioacchino.delregno@collabora.com>,
        Florian Fainelli <florian.fainelli@broadcom.com>,
        Broadcom internal kernel review list 
        <bcm-kernel-feedback-list@broadcom.com>,
        Dinh Nguyen <dinguyen@kernel.org>,
        Jonathan Hunter <jonathanh@nvidia.com>,
        Michal Simek <michal.simek@amd.com>,
        devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
        linux-kernel@vger.kernel.org, linux-mediatek@lists.infradead.org,
        linux-rpi-kernel@lists.infradead.org, linux-tegra@vger.kernel.org
Subject: Re: [PATCH] firmware: Explicitly include correct DT includes
Message-ID: <ZLTrLq5Hp1sHckMJ@orome>
References: <20230714174438.4054854-1-robh@kernel.org>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
        protocol="application/pgp-signature"; boundary="TiNl3sY2kGGAUSEk"
Content-Disposition: inline
In-Reply-To: <20230714174438.4054854-1-robh@kernel.org>
User-Agent: Mutt/2.2.10 (2023-03-25)
Precedence: bulk


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

On Fri, Jul 14, 2023 at 11:44:37AM -0600, Rob Herring wrote:
> The DT of_device.h and of_platform.h date back to the separate
> of_platform_bus_type before it as merged into the regular platform bus.
> As part of that merge prepping Arm DT support 13 years ago, they
> "temporarily" include each other. They also include platform_device.h
> and of.h. As a result, there's a pretty much random mix of those include
> files used throughout the tree. In order to detangle these headers and
> replace the implicit includes with struct declarations, users need to
> explicitly include the correct includes.
>=20
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
>  drivers/firmware/arm_scmi/driver.c | 4 ++--
>  drivers/firmware/imx/imx-dsp.c     | 1 -
>  drivers/firmware/imx/imx-scu-irq.c | 1 +
>  drivers/firmware/imx/imx-scu.c     | 1 +
>  drivers/firmware/mtk-adsp-ipc.c    | 1 -
>  drivers/firmware/raspberrypi.c     | 1 +
>  drivers/firmware/scpi_pm_domain.c  | 3 ++-
>  drivers/firmware/stratix10-rsu.c   | 1 -
>  drivers/firmware/tegra/bpmp.c      | 3 +--
>  drivers/firmware/xilinx/zynqmp.c   | 1 +
>  10 files changed, 9 insertions(+), 8 deletions(-)

Acked-by: Thierry Reding <treding@nvidia.com>

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

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

iQIzBAABCAAdFiEEiOrDCAFJzPfAjcif3SOs138+s6EFAmS06y4ACgkQ3SOs138+
s6H3vxAAt+q3FQ+MBS2z+u74V8zvjMkyymL79TZvk56JPufzsPYsS6gvjgQeEJg8
R/slJzQCTxr3jPo3Z+wNWyfyEfE/1M58LbOBii5LVVxiQadLFmzqS/x9Zhy+LOVq
LI1xBxmIE7fiC98G8rS5E0F/N8PPUm2KiNOtAelj0xeiy3TkG+kdNeTDQlt+yuHx
tSTz992PllOwY2WSlYhXGwKJXZmibO5zmb6z1gWXYZZnzzIuyL2KeIuF0pRw/4Y9
HoWX10J/985/8uaeWWci4ucp87WAYX0Z1/DfLBVjVFXA7rDcw5To/JwvwR8JkgXo
2iF65qla1iOdqpq8KzWf485wdHISrWNwSiTLmhzeMNW3SagUNlsGtQCyCmDhIXD5
A2nj+9NUWQmVlhkEbEMcJug9GUNtluuFCyWz6F7ipkz+l6r0hxFAUVDlntgWDt7w
xRdCGvohvX3squFZ2NxeU0AX7NGxBRyEVlnmExUYjsdlYRrNFATYrV0Xyu1GKTjt
QcaBe5bhIx9anBIZCfbb2OjHXYguN9LGABN0CepVZEJQ+92CHsS9o5fAMasquHTg
BKnkPKO9/hwB+oY/kV6DgMCReLLL9Xl1CCKW1IktCempLLX+M7oTpXSR1Gt4Iolj
CUo463ao93rGLkBXs7nlXnzDf1iykUqesITmjzEcbV9NwI1AxFw=
=lAG5
-----END PGP SIGNATURE-----

--TiNl3sY2kGGAUSEk--