From:   Leo Li <leoyang.li@nxp.com>
To:     Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "linuxppc-dev@lists.ozlabs.org" <linuxppc-dev@lists.ozlabs.org>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>
CC:     Qiang Zhao <qiang.zhao@nxp.com>
Subject: RE: [PATCH v2 3/3] soc: fsl: Replace kernel.h with the necessary
 inclusions
Thread-Topic: [PATCH v2 3/3] soc: fsl: Replace kernel.h with the necessary
 inclusions
Thread-Index: AQHX1iIbcPueVtt/aUW4L8kSf69QdKwEfA2AgACzPEA=
Date:   Mon, 15 Nov 2021 22:24:36 +0000
Message-ID: <AS8PR04MB8946B4800AE34258F7F2BA688F989@AS8PR04MB8946.eurprd04.prod.outlook.com>
References: <20211110105952.62013-1-andriy.shevchenko@linux.intel.com>
 <20211110105952.62013-3-andriy.shevchenko@linux.intel.com>
 <YZJExzxJ4j8g6jEY@smile.fi.intel.com>
In-Reply-To: <YZJExzxJ4j8g6jEY@smile.fi.intel.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?2C2NQwv4l9IjQMZX87nMAj06jcxGjoX7wlRBfxejNj+quwZxVYUEqFVF/BUf?=
 =?us-ascii?Q?jAKRtRAvVX8V5TA4BP0lbfTp8HAxJhRgGiide+dmBjbi1Q/mJfI9PzrIVXUg?=
 =?us-ascii?Q?i1/ffSha2HTR95U9/0oiOMcVGWhtvf43dMI54A3Z0bxRhv2i6Vi/uI3xS4R4?=
 =?us-ascii?Q?H5iqQkfxP7Yej1zl4+wUGnrCmnGbt4DT8uRL/HCk55709+WYyGaBPFNYbv8b?=
 =?us-ascii?Q?ErL60PBDg+vadnewDizD20D1Tse+YWk6I+Mmow58wU67NIUPDlm3jPt2sIsP?=
 =?us-ascii?Q?R+rxgobpCnVWgyZR9wVBLRwSN9r1G7SUJSd2zH2GRzhgpBNnfpPOnNKKMOta?=
 =?us-ascii?Q?eUTE40JSFdR6/P+Ift1ikqWD+3XWMNVWrfjmnvOQQNSUE3LWtxz2SDp9K8QX?=
 =?us-ascii?Q?BARFd87lWPLEtOAH7sUvkWZf7oo0Gc+ZCuyL1CPGW7qQ/wlYJ6dDbDGQ2/1q?=
 =?us-ascii?Q?MbxkGreMILlFqDvcWI4dp1Ghm6gcGZEGX0kZirvyGmOvR/00YMiol0O9fahd?=
 =?us-ascii?Q?FD7wOOHEvZzICb4SF9z3kQg7Yxmtv8u6TD5AWXiTfHbvpgCH5+CTgqkJGm22?=
 =?us-ascii?Q?YoRLhzXvC8+KfLzsA+RuXRaghzMbfaM3WrqVn8wWKtzvkIlhbwrwNhDF5FHB?=
 =?us-ascii?Q?NTTEMVBH5t4nAmMji4yIhb9sPul+Iw4esqukFOzXsBoQzkc35ngPmN5BT3at?=
 =?us-ascii?Q?S3hX/pa99aHv+pKHZCSLX5q7BiekC8Jc8j1jP6DiFsEfAmc4rN0XUr0CQ05r?=
 =?us-ascii?Q?boIfTJTfFkXIqlolDmI32E6XbdPYtD2COuyTauH91X96CsZHUxjg/2v8dCp/?=
 =?us-ascii?Q?NwFJ4OTxdJGccG6ftSIhh+60u198PGT4SLyYdFfhzAzrtXBCRfu1rFlUgwhi?=
 =?us-ascii?Q?ajBwhVZ5M+YQqNFGzo5Qlk2vQFiluoAkimM5Js7ptz46uDk47aLHOc8rIHge?=
 =?us-ascii?Q?B4LbB86C+PFc9Z3fGLAIR9fwi2uFd0tkkK0zUS2Qpk3t7AxRKVS01BSd1CEj?=
 =?us-ascii?Q?79wwdA5IoRBtT7EheGLkfAhlEya7WnvMVJFthMPlBV0qq3ppUa73F3hLrhN9?=
 =?us-ascii?Q?+fXdTvvX/TAyu4gI0ERJXlPGjS3UxVxJQxWDAvOwhhtzju72kUnYPMD5aQX4?=
 =?us-ascii?Q?29VV2Li94nG680rWuxVuto3XJ4viTcGntxBeq0ATJiNzUkZKpeZLwp04PBzI?=
 =?us-ascii?Q?ygIrKMy2U7T1d4kbdQq64g2DaM30j3cgjJ3zAcjdKR+0ywPRdqb+DpmQ5AAT?=
 =?us-ascii?Q?IX14JKhsZwgQKZbYPi7iSKC6VBHxfWp3IhzJizPjSq8YeWcFBIDgAdM4L/7P?=
 =?us-ascii?Q?WaxY6La7eKjCWiiCT+lRI6ROFK8LK79tvh2jtlLSnhiV8ipERORUCRdi354E?=
 =?us-ascii?Q?6E570Lj/iHjSA+VfURwnScqQe2x3LyGEwVVqBv+rb5rRATzZWfff6QRiQtJc?=
 =?us-ascii?Q?qy/f1lKhYM/wApkeaYZmqSoFwP9lXtmNBU0FPKEDZQPL2idD9dwFbLG2hSUA?=
 =?us-ascii?Q?4ne/kQbhEtgmkFdaIQPUk0CdWSHkTf/rRRU/NU9276U5yH2wNR6HNoH5CPy5?=
 =?us-ascii?Q?gQl2nV9KaB4r+64nbZuhqg2pTsjqNrdIPYrrndil9Z+C9mo+R1GLrLqop1XJ?=
 =?us-ascii?Q?vw=3D=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: AS8PR04MB8946.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 42868ce8-1ad7-4ce1-322e-08d9a886b50e
X-MS-Exchange-CrossTenant-originalarrivaltime: 15 Nov 2021 22:24:36.4743
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: riONMCzcU0/vJlBCvAsJPacf99LYHi0CKFDULCC/14R1EpMwPTCrvFuzKL/8nvLd3c/t+CSrXfMfZQgV8ea8nQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS8PR04MB8849
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


> -----Original Message-----
> From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> Sent: Monday, November 15, 2021 5:30 AM
> To: linux-kernel@vger.kernel.org; linuxppc-dev@lists.ozlabs.org; linux-ar=
m-
> kernel@lists.infradead.org
> Cc: Leo Li <leoyang.li@nxp.com>; Qiang Zhao <qiang.zhao@nxp.com>
> Subject: Re: [PATCH v2 3/3] soc: fsl: Replace kernel.h with the necessary
> inclusions
>=20
> On Wed, Nov 10, 2021 at 12:59:52PM +0200, Andy Shevchenko wrote:
> > When kernel.h is used in the headers it adds a lot into dependency
> > hell, especially when there are circular dependencies are involved.
> >
> > Replace kernel.h inclusion with the list of what is really being used.
> >
> > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> > ---
> > v2: updated Cc list based on previous changes to MAINTAINERS
>=20
> Any comments on this, please?
>=20
> I really want to decrease amount of kernel.h usage in the common headers.
> So others won't copy'n'paste bad example.

There seems to be no problem with the patch although I didn't get time to r=
eally compile with it applied.

Will pick them up later after build test.

Regards,
Leo