src: Use #include <timer.h> when appropriate

Also, extra-lines added or removed and local includes moved down.

Change-Id: I5e739233f3742fd68d537f671642bb04886e3009
Signed-off-by: Elyes HAOUAS <ehaouas@noos.fr>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/32009
Reviewed-by: Nico Huber <nico.h@gmx.de>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>

src/commonlib/storage/sd.c

@@ -24,9 +24,9 @@
 #include <commonlib/storage.h>
 #include <delay.h>
 #include <endian.h>
+
 #include "sd_mmc.h"
 #include "storage.h"
-#include <timer.h>
 
 int sd_send_if_cond(struct storage_media *media)
 {

src/commonlib/storage/sd_mmc.c

@@ -24,11 +24,11 @@
 #include <commonlib/storage.h>
 #include <delay.h>
 #include <endian.h>
+#include <string.h>
+
 #include "mmc.h"
 #include "sd_mmc.h"
 #include "storage.h"
-#include <string.h>
-#include <timer.h>
 
 uint64_t sd_mmc_extract_uint32_bits(const uint32_t *array, int start, int count)
 {

src/commonlib/storage/sdhci_adma.c

@@ -22,11 +22,11 @@
 #include <commonlib/storage.h>
 #include <delay.h>
 #include <endian.h>
+#include <string.h>
+
 #include "sdhci.h"
 #include "sd_mmc.h"
 #include "storage.h"
-#include <string.h>
-#include <timer.h>
 
 static void sdhci_alloc_adma_descs(struct sdhci_ctrlr *sdhci_ctrlr,
 	u32 need_descriptors)

src/cpu/allwinner/a10/timer.c

@@ -17,11 +17,10 @@
  *
  */
 
-#include "timer.h"
-
 #include <device/mmio.h>
 #include <delay.h>
-#include <timer.h>
+
+#include "timer.h"
 
 struct a1x_timer_module *const timer_module = (void *)A1X_TIMER_BASE;
 struct a1x_timer *const tmr0 =

src/cpu/x86/mp_init.c

@@ -36,6 +36,7 @@
 #include <smp/atomic.h>
 #include <smp/spinlock.h>
 #include <symbols.h>
+#include <timer.h>
 #include <thread.h>
 
 #define MAX_APIC_IDS 256

src/drivers/i2c/tpm/tis.c

@@ -18,15 +18,14 @@
 #include <string.h>
 #include <assert.h>
 #include <commonlib/endian.h>
+#include <console/console.h>
 #include <delay.h>
 #include <device/i2c_simple.h>
 #include <endian.h>
 #include <lib.h>
 #include <security/tpm/tis.h>
-#include "tpm.h"
-#include <timer.h>
 
-#include <console/console.h>
+#include "tpm.h"
 
 /* global structure for tpm chip data */
 static struct tpm_chip g_chip CAR_GLOBAL;

src/lib/thread.c

@@ -12,6 +12,7 @@
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * GNU General Public License for more details.
  */
+
 #include <stddef.h>
 #include <stdint.h>
 #include <stdlib.h>
@@ -19,6 +20,7 @@
 #include <bootstate.h>
 #include <console/console.h>
 #include <thread.h>
+#include <timer.h>
 
 static void idle_thread_init(void);
 

src/soc/cavium/cn81xx/include/soc/timer.h

@@ -18,7 +18,6 @@
 
 #include <inttypes.h>
 #include <types.h>
-#include <timer.h>
 #include <delay.h>
 
 /* Watchdog functions */

src/soc/intel/apollolake/pmutil.c

@@ -35,8 +35,8 @@
 #include <soc/pci_devs.h>
 #include <soc/pm.h>
 #include <soc/smbus.h>
-#include <timer.h>
 #include <security/vboot/vbnv.h>
+
 #include "chip.h"
 
 static uintptr_t read_pmc_mmio_bar(void)

src/soc/intel/cannonlake/pmutil.c

@@ -38,8 +38,8 @@
 #include <soc/pci_devs.h>
 #include <soc/pm.h>
 #include <soc/smbus.h>
-#include <timer.h>
 #include <security/vboot/vbnv.h>
+
 #include "chip.h"
 
 /*

src/soc/intel/cannonlake/reset.c

@@ -20,7 +20,6 @@
 #include <fsp/util.h>
 #include <soc/intel/common/reset.h>
 #include <string.h>
-#include <timer.h>
 #include <soc/pci_devs.h>
 
 /* Reset Request  */

src/soc/intel/common/block/systemagent/systemagent_early.c

@@ -22,8 +22,8 @@
 #include <soc/iomap.h>
 #include <soc/pci_devs.h>
 #include <soc/systemagent.h>
+
 #include "systemagent_def.h"
-#include <timer.h>
 
 #if !ENV_RAMSTAGE
 void bootblock_systemagent_early_init(void)

src/soc/intel/icelake/pmutil.c

@@ -37,8 +37,8 @@
 #include <soc/pci_devs.h>
 #include <soc/pm.h>
 #include <soc/smbus.h>
-#include <timer.h>
 #include <security/vboot/vbnv.h>
+
 #include "chip.h"
 
 /*

src/soc/intel/icelake/reset.c

@@ -20,7 +20,6 @@
 #include <fsp/util.h>
 #include <soc/intel/common/reset.h>
 #include <string.h>
-#include <timer.h>
 #include <soc/pci_devs.h>
 
 /* Reset Request  */

src/soc/intel/quark/storage_test.c

@@ -25,6 +25,7 @@
 #include <soc/iomap.h>
 #include <soc/pci_devs.h>
 #include <soc/storage_test.h>
+#include <timer.h>
 #include <string.h>
 
 #if CONFIG(STORAGE_LOG)

src/soc/intel/skylake/pmutil.c

@@ -37,8 +37,8 @@
 #include <soc/pm.h>
 #include <soc/pmc.h>
 #include <soc/smbus.h>
-#include <timer.h>
 #include <security/vboot/vbnv.h>
+
 #include "chip.h"
 
 /*

src/soc/intel/skylake/reset.c

@@ -20,7 +20,6 @@
 #include <soc/intel/common/reset.h>
 #include <soc/me.h>
 #include <soc/pm.h>
-#include <timer.h>
 
 static void do_force_global_reset(void)
 {

src/soc/mediatek/common/include/soc/pmic_wrap_common.h

@@ -17,7 +17,6 @@
 #define SOC_MEDIATEK_PMIC_WRAP_COMMON_H
 
 #include <console/console.h>
-#include <timer.h>
 
 #define PWRAPTAG                "[PWRAP] "
 #define pwrap_err(fmt, arg ...) printk(BIOS_ERR, PWRAPTAG "ERROR,line=%d" fmt, \

src/soc/mediatek/common/include/soc/rtc_common.h

@@ -21,7 +21,6 @@
 #include <console/console.h>
 #include <delay.h>
 #include <rtc.h>
-#include <timer.h>
 #include <soc/pmic_wrap_common.h>
 
 #define RTCTAG			"[RTC]"

src/soc/mediatek/common/pmic_wrap.c

@@ -16,6 +16,7 @@
 #include <device/mmio.h>
 #include <assert.h>
 #include <soc/pmic_wrap.h>
+#include <timer.h>
 
 u32 wait_for_state_idle(u32 timeout_us, void *wacs_register,
 			void *wacs_vldclr_register, u32 *read_reg)

src/soc/mediatek/common/rtc.c

@@ -16,6 +16,7 @@
 #include <soc/rtc_common.h>
 #include <soc/rtc.h>
 #include <soc/pmic_wrap.h>
+#include <timer.h>
 
 /* ensure rtc write success */
 int rtc_busy_wait(void)

src/soc/mediatek/mt8183/pmic_wrap.c

@@ -17,6 +17,7 @@
 #include <soc/infracfg.h>
 #include <soc/pll.h>
 #include <soc/pmic_wrap.h>
+#include <timer.h>
 
 #define PRIORITY_FIELD(x)		((x % 4) * 8)
 #define PRIORITY_IN(id, priority)	(id << PRIORITY_FIELD(priority))

src/soc/nvidia/tegra210/romstage.c

@@ -28,7 +28,6 @@
 #include <soc/romstage.h>
 #include <soc/nvidia/tegra/apbmisc.h>
 #include <symbols.h>
-#include <timer.h>
 #include <vendorcode/google/chromeos/chromeos.h>
 
 void __weak romstage_mainboard_init(void)

src/soc/nvidia/tegra210/soc.c

@@ -26,7 +26,6 @@
 #include <soc/cpu.h>
 #include <soc/mc.h>
 #include <soc/nvidia/tegra/apbmisc.h>
-#include <timer.h>
 #include <soc/sdram.h>
 #include <soc/sdram_configs.h>
 

src/soc/nvidia/tegra210/spi.c

@@ -29,8 +29,6 @@
 #include <stdint.h>
 #include <stdlib.h>
 #include <symbols.h>
-#include <timer.h>
-
 
 #if defined(CONFIG_DEBUG_SPI) && CONFIG_DEBUG_SPI
 # define DEBUG_SPI(x,...)	printk(BIOS_DEBUG, "TEGRA_SPI: " x)

src/soc/qualcomm/ipq40xx/blobs_init.c

@@ -19,9 +19,7 @@
 #include <cbfs.h>
 #include <console/console.h>
 #include <string.h>
-#include <timer.h>
 #include <program_loading.h>
-
 #include <soc/iomap.h>
 #include <soc/soc_services.h>
 

src/soc/qualcomm/sdm845/clock.c

@@ -17,10 +17,8 @@
 #include <types.h>
 #include <console/console.h>
 #include <delay.h>
-#include <timer.h>
 #include <commonlib/helpers.h>
 #include <assert.h>
-
 #include <soc/clock.h>
 
 #define DIV(div) (2*div - 1)

src/soc/qualcomm/sdm845/gpio.c

@@ -16,8 +16,6 @@
 #include <device/mmio.h>
 #include <types.h>
 #include <delay.h>
-#include <timer.h>
-
 #include <gpio.h>
 
 void gpio_configure(gpio_t gpio, uint32_t func, uint32_t pull,

src/soc/samsung/exynos5420/dp.c

@@ -23,7 +23,6 @@
 #include <soc/sysreg.h>
 #include <stdlib.h>
 #include <string.h>
-#include <timer.h>
 
 /*
  * Here is the rough outline of how we bring up the display:

src/soc/samsung/exynos5420/dp_lowlevel.c

@@ -23,7 +23,6 @@
 #include <soc/power.h>
 #include <soc/sysreg.h>
 #include <stdlib.h>
-#include <timer.h>
 
 /* FIXME: I think the DP controller shouldn't be hardcoded here... */
 static struct exynos_dp * const dp_regs = (void *)EXYNOS5_DP1_BASE;

src/soc/samsung/exynos5420/fimd.c

@@ -24,7 +24,6 @@
 #include <soc/fimd.h>
 #include <soc/periph.h>
 #include <soc/sysreg.h>
-#include <timer.h>
 
 /* fairly useful debugging stuff. */
 #if 0