From patchwork Thu Aug 3 22:43:03 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Rob Herring (Arm)" X-Patchwork-Id: 13340972 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id BC9DBC001DF for ; Thu, 3 Aug 2023 22:43:45 +0000 (UTC) Received: by smtp.kernel.org (Postfix) id A2421C433C9; Thu, 3 Aug 2023 22:43:45 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id D967FC433C8; Thu, 3 Aug 2023 22:43:41 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1691102625; bh=KtCTm8P3ZFPnN+VJ1KJBv1l1cSj885PBIvibZEA9CFY=; h=From:Date:Subject:References:In-Reply-To:List-Id:To:Cc:From; b=bo7MVsa1PBJS3oAt6C7lE2MQrPn6pxc9efA2qZ5huiRANMhVjXByXKZhBwqAIi+9x u9UVdv6CUAVhQ77sdLFdEJm/QkWOk0ykZZM37LvQqzGXcxMRs/LjozPEYSrAKQ9nSB 5oVbcE+G5NTPAL2bWzIv1hpCBrkL5QePQJjoyc+Aw4UbH6QC8INDzHLfwZWMWR+wSG tlrAwAZESxEw3s4Vn/zqmLiHA3BMg9GjO3Lrzia8fM4hsOXgD6HBbctAhm7lfO7FOH dGzlCLuYxPdd+JK/DHcz70G23se9J5yx1ib0FW9v5Vh8brWKm4w8myjmjyZrSjqu0d vRUhW/xtmS34w== Received: (nullmailer pid 3693796 invoked by uid 1000); Thu, 03 Aug 2023 22:42:55 -0000 From: Rob Herring Date: Thu, 03 Aug 2023 16:43:03 -0600 Subject: [PATCH v2 23/23] soc: fsl: Explicitly include correct DT includes MIME-Version: 1.0 Message-Id: <20230803-dt-header-cleanups-for-soc-v2-23-d8de2cc88bff@kernel.org> References: <20230803-dt-header-cleanups-for-soc-v2-0-d8de2cc88bff@kernel.org> In-Reply-To: <20230803-dt-header-cleanups-for-soc-v2-0-d8de2cc88bff@kernel.org> List-Id: To: soc@kernel.org, Patrice Chotard , Tsahee Zidenberg , Antoine Tenart , Jisheng Zhang , Sebastian Hesselbarth , Andrew Lunn , Gregory Clement , Jean-Marie Verdun , Nick Hawkins , Lubomir Rintel , Linus Walleij , Avi Fishman , Tomer Maimon , Tali Perry , Patrick Venture , Nancy Yuen , Benjamin Fair , Daniel Mack , Haojian Zhuang , Robert Jarzmik , Heiko Stuebner , Liviu Dudau , Sudeep Holla , Lorenzo Pieralisi , Viresh Kumar , Shiraz Has him , Stuart Yoder , Laurentiu Tudor , Jay Fang , Chen-Yu Tsai , Jernej Skrabec , Samuel Holland , Cristian Marussi , Shawn Guo , Sascha Hauer , Fabio Estevam , Matthias Brugger , AngeloGioacchino Del Regno , Florian Fainelli , Dinh Nguyen , Thierry Reding , Jonathan Hunter , Michal Simek , Joel Stanley , Andrew Jeffery , Ulf Hansson , Li Yang , Qiang Zhao Cc: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, openbmc@lists.ozlabs.org, linux-rockchip@lists.infradead.org, linux-sunxi@lists.linux.dev, linux-mediatek@lists.infradead.org, linux-rpi-kernel@lists.infradead.org, linux-tegra@vger.kernel.org, linux-aspeed@lists.ozlabs.org, linux-pm@vger.kernel.org X-Mailer: b4 0.13-dev 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. Signed-off-by: Rob Herring --- v2: - Add qe.c --- drivers/soc/fsl/dpaa2-console.c | 3 ++- drivers/soc/fsl/qe/qe.c | 3 ++- drivers/soc/fsl/qe/qe_common.c | 1 - drivers/soc/fsl/qe/qe_tdm.c | 4 +--- 4 files changed, 5 insertions(+), 6 deletions(-) diff --git a/drivers/soc/fsl/dpaa2-console.c b/drivers/soc/fsl/dpaa2-console.c index 53917410f2bd..1dca693b6b38 100644 --- a/drivers/soc/fsl/dpaa2-console.c +++ b/drivers/soc/fsl/dpaa2-console.c @@ -9,9 +9,10 @@ #define pr_fmt(fmt) "dpaa2-console: " fmt #include -#include +#include #include #include +#include #include #include #include diff --git a/drivers/soc/fsl/qe/qe.c b/drivers/soc/fsl/qe/qe.c index 58746e570d14..3ee0c7c1e9a4 100644 --- a/drivers/soc/fsl/qe/qe.c +++ b/drivers/soc/fsl/qe/qe.c @@ -25,7 +25,8 @@ #include #include #include -#include +#include +#include #include #include diff --git a/drivers/soc/fsl/qe/qe_common.c b/drivers/soc/fsl/qe/qe_common.c index a0cb8e746879..9729ce86db59 100644 --- a/drivers/soc/fsl/qe/qe_common.c +++ b/drivers/soc/fsl/qe/qe_common.c @@ -16,7 +16,6 @@ #include #include #include -#include #include #include #include diff --git a/drivers/soc/fsl/qe/qe_tdm.c b/drivers/soc/fsl/qe/qe_tdm.c index 7d7d78d3ee50..a3b691875c8e 100644 --- a/drivers/soc/fsl/qe/qe_tdm.c +++ b/drivers/soc/fsl/qe/qe_tdm.c @@ -9,9 +9,7 @@ */ #include #include -#include -#include -#include +#include #include static int set_tdm_framer(const char *tdm_framer_type)