On Fri, Apr 15, 2022 at 10:08:32PM +0530, Vignesh Raghavendra wrote:
On 15/04/22 9:57 pm, Tom Rini wrote:
On Fri, Apr 15, 2022 at 07:39:24PM +0530, Vignesh Raghavendra wrote:
[snip]
diff --git a/arch/arm/mach-k3/am625_init.c b/arch/arm/mach-k3/am625_init.c new file mode 100644 index 0000000000..ee902a774e --- /dev/null +++ b/arch/arm/mach-k3/am625_init.c @@ -0,0 +1,254 @@ +// SPDX-License-Identifier: GPL-2.0 +/*
- AM625: SoC specific initialization
- Copyright (C) 2020-2022 Texas Instruments Incorporated - https://www.ti.com/
- Suman Anna s-anna@ti.com
- */
+#include <common.h> +#include <spl.h> +#include <asm/io.h> +#include <asm/arch/hardware.h> +#include <asm/arch/sysfw-loader.h> +#include "common.h" +#include <dm.h> +#include <dm/uclass-internal.h> +#include <dm/pinctrl.h>
Er, common.h twice?
Second one refers to arch/arm/mach-k3/common.h
OK, lets rename that, and see what info there we can put elsewhere.
Please re-check this, and then also drop common.h (here and elsewhere) as it only includes other headers and we should avoid it when possible.
Sure, will do.
In fact, checkpatch.pl complains about both dm.h and common.h so please address series-wide, thanks.
Unfortunately, it doesn't ;(
Ah, right. It only checks for inside of headers. But still should be avoided.