Change in coreboot[master]: src: Remove unused 'include <stdint.h>

[HAOUAS Elyes (Code Review)]

HAOUAS Elyes has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/41827 )

Change subject: src: Remove unused 'include <stdint.h>
......................................................................

src: Remove unused 'include <stdint.h>

Found using:
diff <(git grep -l '#include (<stdint.h>|<types.h>)' -- src/) <(git grep -l 'int8_t|int16_t|int32_t|int64_t|intptr_t|intmax_t|s8|u8|s16|u16|s32|u32|s64|u64|INT8_MIN|INT8_MAX|INT16_MIN|INT16_MAX|INT32_MIN|INT32_MAX|INT64_MIN|INT64_MAX|INTMAX_MIN|INTMAX_MAX' -- src/) |grep -v vendorcode |grep '<'

Change-Id: I5e14bf4887c7d2644a64f4d58c6d8763eb74d2ed
Signed-off-by: Elyes HAOUAS 
---
M src/acpi/soundwire.c
M src/drivers/intel/soundwire/soundwire.c
M src/soc/intel/jasperlake/me.c
3 files changed, 0 insertions(+), 3 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/27/41827/1

diff --git a/src/acpi/soundwire.c b/src/acpi/soundwire.c
index 1151a18..341114a 100644
--- a/src/acpi/soundwire.c
+++ b/src/acpi/soundwire.c
@@ -6,7 +6,6 @@
 #include <commonlib/helpers.h>
 #include <device/soundwire.h>
 #include <stdbool.h>
-#include <stdint.h>
 #include <string.h>

/* Specification-defined prefix for SoundWire properties. */
diff --git a/src/drivers/intel/soundwire/soundwire.c b/src/drivers/intel/soundwire/soundwire.c
index ab09ff4..34ecd86 100644
--- a/src/drivers/intel/soundwire/soundwire.c
+++ b/src/drivers/intel/soundwire/soundwire.c
@@ -8,7 +8,6 @@
 #include <device/path.h>
 #include <device/soundwire.h>
 #include <stdbool.h>
-#include <stdint.h>

#include "soundwire.h"
 #include "chip.h"
diff --git a/src/soc/intel/jasperlake/me.c b/src/soc/intel/jasperlake/me.c
index c8496f9..0797993 100644
--- a/src/soc/intel/jasperlake/me.c
+++ b/src/soc/intel/jasperlake/me.c
@@ -5,7 +5,6 @@
 #include <intelblocks/cse.h>
 #include <console/console.h>
 #include <soc/me.h>
-#include <stdint.h>

static void dump_me_status(void *unused)
 {

[Paul Menzel (Code Review)]

Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41827 )

Change subject: src: Remove unused 'include <stdint.h>
......................................................................

Patch Set 1: Code-Review+1

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41827 )

Change subject: src: Remove unused 'include <stdint.h>
......................................................................

Patch Set 5: Code-Review+1

(1 comment)

https://review.coreboot.org/c/coreboot/+/41827/5/src/acpi/soundwire.c 
File src/acpi/soundwire.c:

https://review.coreboot.org/c/coreboot/+/41827/5/src/acpi/soundwire.c@a168 
PS5, Line 168: size_t
hmmm...

[HAOUAS Elyes (Code Review)]

HAOUAS Elyes has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41827 )

Change subject: src: Remove unused 'include <stdint.h>
......................................................................

Patch Set 5:

(1 comment)

https://review.coreboot.org/c/coreboot/+/41827/5/src/acpi/soundwire.c 
File src/acpi/soundwire.c:

https://review.coreboot.org/c/coreboot/+/41827/5/src/acpi/soundwire.c@a168 
PS5, Line 168: size_t
> hmmm...

stddef.h
maybe for another patch :)

[HAOUAS Elyes (Code Review)]

HAOUAS Elyes has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41827 )

Change subject: src: Remove unused 'include <stdint.h>
......................................................................

Patch Set 5:

(1 comment)

https://review.coreboot.org/c/coreboot/+/41827/5/src/acpi/soundwire.c 
File src/acpi/soundwire.c:

https://review.coreboot.org/c/coreboot/+/41827/5/src/acpi/soundwire.c@a168 
PS5, Line 168: size_t
> stddef.h […]

please see https://review.coreboot.org/c/coreboot/+/42008

[HAOUAS Elyes (Code Review)]

Hello build bot (Jenkins), Paul Menzel, Angel Pons, Patrick Rudolph,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/41827

to look at the new patch set (#6).

Change subject: src: Remove unused 'include <stdint.h>
......................................................................

src: Remove unused 'include <stdint.h>

Found using:
diff <(git grep -l '#include (<stdint.h>|<types.h>)' -- src/) <(git grep -l 'int8_t|int16_t|int32_t|int64_t|intptr_t|intmax_t|s8|u8|s16|u16|s32|u32|s64|u64|INT8_MIN|INT8_MAX|INT16_MIN|INT16_MAX|INT32_MIN|INT32_MAX|INT64_MIN|INT64_MAX|INTMAX_MIN|INTMAX_MAX' -- src/) |grep -v vendorcode |grep '<'

Change-Id: I5e14bf4887c7d2644a64f4d58c6d8763eb74d2ed
Signed-off-by: Elyes HAOUAS 
---
M src/acpi/soundwire.c
M src/drivers/intel/soundwire/soundwire.c
M src/soc/intel/jasperlake/me.c
3 files changed, 1 insertion(+), 3 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/27/41827/6

[HAOUAS Elyes (Code Review)]

Hello build bot (Jenkins), Paul Menzel, Angel Pons, Patrick Rudolph,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/41827

to look at the new patch set (#7).

Change subject: src: Remove unused 'include <stdint.h>
......................................................................

src: Remove unused 'include <stdint.h>

Found using:
diff <(git grep -l '#include (<stdint.h>|<types.h>)' -- src/) <(git grep -l 'int8_t|int16_t|int32_t|int64_t|intptr_t|intmax_t|s8|u8|s16|u16|s32|u32|s64|u64|INT8_MIN|INT8_MAX|INT16_MIN|INT16_MAX|INT32_MIN|INT32_MAX|INT64_MIN|INT64_MAX|INTMAX_MIN|INTMAX_MAX' -- src/) |grep -v vendorcode |grep '<'

Aslo add missing <stddef.h> for 'size_t' in "src/acpi/soundwire.c".

Change-Id: I5e14bf4887c7d2644a64f4d58c6d8763eb74d2ed
Signed-off-by: Elyes HAOUAS 
---
M src/acpi/soundwire.c
M src/drivers/intel/soundwire/soundwire.c
M src/soc/intel/jasperlake/me.c
3 files changed, 1 insertion(+), 3 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/27/41827/7

[HAOUAS Elyes (Code Review)]

Hello build bot (Jenkins), Paul Menzel, Angel Pons, Patrick Rudolph,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/41827

to look at the new patch set (#8).

Change subject: src: Remove unused 'include <stdint.h>
......................................................................

src: Remove unused 'include <stdint.h>

Found using:
diff <(git grep -l '#include (<stdint.h>|<types.h>)' -- src/) <(git grep -l 'int8_t|int16_t|int32_t|int64_t|intptr_t|intmax_t|s8|u8|s16|u16|s32|u32|s64|u64|INT8_MIN|INT8_MAX|INT16_MIN|INT16_MAX|INT32_MIN|INT32_MAX|INT64_MIN|INT64_MAX|INTMAX_MIN|INTMAX_MAX' -- src/) |grep -v vendorcode |grep '<'

Also add missing <stddef.h> for 'size_t' in "src/acpi/soundwire.c".

Change-Id: I5e14bf4887c7d2644a64f4d58c6d8763eb74d2ed
Signed-off-by: Elyes HAOUAS 
---
M src/acpi/soundwire.c
M src/drivers/intel/soundwire/soundwire.c
M src/soc/intel/jasperlake/me.c
3 files changed, 1 insertion(+), 3 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/27/41827/8

[HAOUAS Elyes (Code Review)]

HAOUAS Elyes has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41827 )

Change subject: src: Remove unused 'include <stdint.h>
......................................................................

Patch Set 8:

(1 comment)

https://review.coreboot.org/c/coreboot/+/41827/5/src/acpi/soundwire.c 
File src/acpi/soundwire.c:

https://review.coreboot.org/c/coreboot/+/41827/5/src/acpi/soundwire.c@a168 
PS5, Line 168: size_t
> please see https://review.coreboot. […]

<stddef.h> added.
Thx

Change in coreboot[master]: src: Remove unused 'include <stdint.h>'

[HAOUAS Elyes (Code Review)]

Hello build bot (Jenkins), Paul Menzel, Angel Pons, Patrick Rudolph,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/41827

to look at the new patch set (#11).

Change subject: src: Remove unused 'include <stdint.h>'
......................................................................

src: Remove unused 'include <stdint.h>'

Found using:
diff <(git grep -l '#include (<stdint.h>|<types.h>)' -- src/) <(git grep -l 'int8_t|int16_t|int32_t|int64_t|intptr_t|intmax_t|s8|u8|s16|u16|s32|u32|s64|u64|INT8_MIN|INT8_MAX|INT16_MIN|INT16_MAX|INT32_MIN|INT32_MAX|INT64_MIN|INT64_MAX|INTMAX_MIN|INTMAX_MAX' -- src/) |grep -v vendorcode |grep '<'

Also add missing <stddef.h> for 'size_t' in "src/acpi/soundwire.c".

Change-Id: I5e14bf4887c7d2644a64f4d58c6d8763eb74d2ed
Signed-off-by: Elyes HAOUAS 
---
M src/acpi/soundwire.c
M src/drivers/intel/soundwire/soundwire.c
M src/soc/intel/jasperlake/me.c
3 files changed, 1 insertion(+), 3 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/27/41827/11

Change in coreboot[master]: src: Remove unused 'include <stdint.h>'

[HAOUAS Elyes (Code Review)]

Hello build bot (Jenkins), Paul Menzel, Angel Pons, Patrick Rudolph,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/41827

to look at the new patch set (#12).

Change subject: src: Remove unused 'include <stdint.h>'
......................................................................

src: Remove unused 'include <stdint.h>'

Found using:
diff <(git grep -l '#include <stdint.h>' -- src/) <(git grep -l 'int8_t|int16_t|int32_t|int64_t|intptr_t|intmax_t|s8|u8|s16|u16|s32|u32|s64|u64|INT8_MIN|INT8_MAX|INT16_MIN|INT16_MAX|INT32_MIN|INT32_MAX|INT64_MIN|INT64_MAX|INTMAX_MIN|INTMAX_MAX' -- src/) |grep -v vendorcode |grep '<'

Also add missing <stddef.h> for 'size_t' in "src/acpi/soundwire.c".

Change-Id: I5e14bf4887c7d2644a64f4d58c6d8763eb74d2ed
Signed-off-by: Elyes HAOUAS 
---
M src/acpi/soundwire.c
M src/drivers/intel/soundwire/soundwire.c
M src/soc/intel/jasperlake/me.c
3 files changed, 1 insertion(+), 3 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/27/41827/12

[HAOUAS Elyes (Code Review)]

Hello build bot (Jenkins), Paul Menzel, Angel Pons, Patrick Rudolph,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/41827

to look at the new patch set (#13).

Change subject: src: Remove unused 'include <stdint.h>
......................................................................

src: Remove unused 'include <stdint.h>

Found using:
diff <(git grep -l '#include (<stdint.h>|<types.h>)' -- src/) <(git grep -l 'int8_t|int16_t|int32_t|int64_t|intptr_t|intmax_t|s8|u8|s16|u16|s32|u32|s64|u64|INT8_MIN|INT8_MAX|INT16_MIN|INT16_MAX|INT32_MIN|INT32_MAX|INT64_MIN|INT64_MAX|INTMAX_MIN|INTMAX_MAX' -- src/) |grep -v vendorcode |grep '<'

Also add missing <stddef.h> for 'size_t' in "src/acpi/soundwire.c".

Change-Id: I5e14bf4887c7d2644a64f4d58c6d8763eb74d2ed
Signed-off-by: Elyes HAOUAS 
---
M src/acpi/soundwire.c
M src/drivers/intel/soundwire/soundwire.c
M src/soc/intel/jasperlake/me.c
3 files changed, 1 insertion(+), 3 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/27/41827/13

[HAOUAS Elyes (Code Review)]

Hello build bot (Jenkins), Paul Menzel, Angel Pons, Patrick Rudolph,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/41827

to look at the new patch set (#15).

Change subject: src: Remove unused 'include <stdint.h>
......................................................................

src: Remove unused 'include <stdint.h>

Found using:
diff <(git grep -l '#include <stdint.h>' -- src/) <(git grep -l 'int8_t|int16_t|int32_t|int64_t|intptr_t|intmax_t|s8|u8|s16|u16|s32|u32|s64|u64|INT8_MIN|INT8_MAX|INT16_MIN|INT16_MAX|INT32_MIN|INT32_MAX|INT64_MIN|INT64_MAX|INTMAX_MIN|INTMAX_MAX' -- src/) |grep -v vendorcode |grep '<'

Also add missing <stddef.h> for 'size_t' in "src/acpi/soundwire.c".

Change-Id: I5e14bf4887c7d2644a64f4d58c6d8763eb74d2ed
Signed-off-by: Elyes HAOUAS 
---
M src/acpi/soundwire.c
M src/arch/arm/include/armv4/arch/cache.h
M src/arch/arm64/include/armv8/arch/cache.h
M src/arch/x86/include/arch/ebda.h
M src/commonlib/include/commonlib/region.h
M src/drivers/i2c/rt1011/chip.h
M src/drivers/intel/fsp1_1/include/fsp/car.h
M src/drivers/intel/ptt/ptt.h
M src/drivers/intel/soundwire/soundwire.c
M src/mainboard/intel/cannonlake_rvp/variants/baseboard/include/baseboard/variants.h
M src/mainboard/intel/coffeelake_rvp/variants/baseboard/include/baseboard/variants.h
M src/mainboard/intel/glkrvp/variants/baseboard/include/baseboard/variants.h
M src/mainboard/intel/icelake_rvp/variants/baseboard/include/baseboard/variants.h
M src/mainboard/intel/jasperlake_rvp/board_id.h
M src/mainboard/intel/tglrvp/board_id.h
M src/mainboard/intel/tglrvp/variants/baseboard/include/baseboard/variants.h
M src/northbridge/intel/sandybridge/romstage.c
M src/security/memory/memory.h
M src/security/vboot/vboot_common.h
M src/soc/amd/common/block/include/amdblocks/acpi.h
M src/soc/amd/common/block/include/amdblocks/agesawrapper.h
M src/soc/amd/common/block/include/amdblocks/s3_resume.h
M src/soc/intel/baytrail/include/soc/romstage.h
M src/soc/intel/braswell/include/soc/romstage.h
M src/soc/intel/broadwell/include/soc/romstage.h
M src/soc/intel/cannonlake/include/soc/pch.h
M src/soc/intel/common/block/include/intelblocks/pcie_rp.h
M src/soc/intel/common/block/smbus/smbuslib.h
M src/soc/intel/common/mma.h
M src/soc/intel/common/pch/include/intelpch/lockdown.h
M src/soc/intel/icelake/include/soc/pch.h
M src/soc/intel/jasperlake/include/soc/espi.h
M src/soc/intel/jasperlake/include/soc/pch.h
M src/soc/intel/jasperlake/me.c
M src/soc/intel/tigerlake/include/soc/espi.h
M src/soc/intel/tigerlake/include/soc/pch.h
M src/soc/mediatek/mt8173/include/soc/addressmap.h
M src/soc/nvidia/tegra/usb.h
M src/soc/nvidia/tegra124/include/soc/gpio.h
M src/soc/nvidia/tegra124/include/soc/pingroup.h
M src/soc/nvidia/tegra124/include/soc/pinmux.h
M src/soc/nvidia/tegra210/include/soc/ccplex.h
M src/soc/nvidia/tegra210/include/soc/pinmux.h
M src/soc/qualcomm/qcs405/include/soc/addressmap.h
M src/soc/qualcomm/sc7180/include/soc/addressmap.h
M src/soc/qualcomm/sdm845/include/soc/addressmap.h
M src/southbridge/amd/cimx/sb800/cfg.h
47 files changed, 1 insertion(+), 49 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/27/41827/15

[HAOUAS Elyes (Code Review)]

Hello build bot (Jenkins), Lee Leahy, Paul Menzel, Huang Jin, Julius Werner, Angel Pons, Aaron Durbin, Patrick Rudolph,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/41827

to look at the new patch set (#16).

Change subject: src: Remove unused 'include <stdint.h>
......................................................................

src: Remove unused 'include <stdint.h>

Found using:
diff <(git grep -l '#include <stdint.h>' -- src/) <(git grep -l 'int8_t|int16_t|int32_t|int64_t|intptr_t|intmax_t|s8|u8|s16|u16|s32|u32|s64|u64|INT8_MIN|INT8_MAX|INT16_MIN|INT16_MAX|INT32_MIN|INT32_MAX|INT64_MIN|INT64_MAX|INTMAX_MIN|INTMAX_MAX' -- src/) |grep -v vendorcode |grep '<'

Also add missing <stddef.h>.

Change-Id: I5e14bf4887c7d2644a64f4d58c6d8763eb74d2ed
Signed-off-by: Elyes HAOUAS 
---
M src/acpi/soundwire.c
M src/arch/arm/include/armv4/arch/cache.h
M src/arch/arm64/include/armv8/arch/cache.h
M src/arch/x86/include/arch/ebda.h
M src/commonlib/include/commonlib/region.h
M src/drivers/i2c/rt1011/chip.h
M src/drivers/intel/fsp1_1/include/fsp/car.h
M src/drivers/intel/ptt/ptt.h
M src/drivers/intel/soundwire/soundwire.c
M src/mainboard/intel/cannonlake_rvp/variants/baseboard/include/baseboard/variants.h
M src/mainboard/intel/coffeelake_rvp/variants/baseboard/include/baseboard/variants.h
M src/mainboard/intel/glkrvp/variants/baseboard/include/baseboard/variants.h
M src/mainboard/intel/icelake_rvp/variants/baseboard/include/baseboard/variants.h
M src/mainboard/intel/jasperlake_rvp/board_id.h
M src/mainboard/intel/tglrvp/board_id.h
M src/mainboard/intel/tglrvp/variants/baseboard/include/baseboard/variants.h
M src/northbridge/intel/sandybridge/romstage.c
M src/security/memory/memory.h
M src/security/vboot/vboot_common.h
M src/soc/amd/common/block/include/amdblocks/acpi.h
M src/soc/amd/common/block/include/amdblocks/agesawrapper.h
M src/soc/amd/common/block/include/amdblocks/s3_resume.h
M src/soc/intel/baytrail/include/soc/romstage.h
M src/soc/intel/braswell/include/soc/romstage.h
M src/soc/intel/broadwell/include/soc/romstage.h
M src/soc/intel/cannonlake/include/soc/pch.h
M src/soc/intel/common/block/include/intelblocks/pcie_rp.h
M src/soc/intel/common/block/smbus/smbuslib.h
M src/soc/intel/common/mma.h
M src/soc/intel/common/pch/include/intelpch/lockdown.h
M src/soc/intel/icelake/include/soc/pch.h
M src/soc/intel/jasperlake/include/soc/espi.h
M src/soc/intel/jasperlake/include/soc/pch.h
M src/soc/intel/jasperlake/me.c
M src/soc/intel/tigerlake/include/soc/espi.h
M src/soc/intel/tigerlake/include/soc/pch.h
M src/soc/mediatek/mt8173/include/soc/addressmap.h
M src/soc/nvidia/tegra/usb.h
M src/soc/nvidia/tegra124/include/soc/gpio.h
M src/soc/nvidia/tegra124/include/soc/pingroup.h
M src/soc/nvidia/tegra124/include/soc/pinmux.h
M src/soc/nvidia/tegra210/include/soc/ccplex.h
M src/soc/nvidia/tegra210/include/soc/pinmux.h
M src/soc/qualcomm/qcs405/include/soc/addressmap.h
M src/soc/qualcomm/sc7180/include/soc/addressmap.h
M src/soc/qualcomm/sdm845/include/soc/addressmap.h
M src/southbridge/amd/cimx/sb800/cfg.h
47 files changed, 1 insertion(+), 49 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/27/41827/16

[HAOUAS Elyes (Code Review)]

Hello build bot (Jenkins), Lee Leahy, Paul Menzel, Huang Jin, Julius Werner, Angel Pons, Aaron Durbin, Patrick Rudolph,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/41827

to look at the new patch set (#17).

Change subject: src: Remove unused 'include <stdint.h>
......................................................................

src: Remove unused 'include <stdint.h>

Found using:
diff <(git grep -l '#include <stdint.h>' -- src/) <(git grep -l 'int8_t|int16_t|int32_t|int64_t|intptr_t|intmax_t|s8|u8|s16|u16|s32|u32|s64|u64|INT8_MIN|INT8_MAX|INT16_MIN|INT16_MAX|INT32_MIN|INT32_MAX|INT64_MIN|INT64_MAX|INTMAX_MIN|INTMAX_MAX' -- src/) |grep -v vendorcode |grep '<'

Also add missing <stddef.h>.

Change-Id: I5e14bf4887c7d2644a64f4d58c6d8763eb74d2ed
Signed-off-by: Elyes HAOUAS 
---
M src/acpi/soundwire.c
M src/arch/arm/include/armv4/arch/cache.h
M src/arch/arm64/include/armv8/arch/cache.h
M src/arch/x86/include/arch/ebda.h
M src/commonlib/include/commonlib/region.h
M src/drivers/i2c/rt1011/chip.h
M src/drivers/intel/fsp1_1/include/fsp/car.h
M src/drivers/intel/ptt/ptt.h
M src/drivers/intel/soundwire/soundwire.c
M src/mainboard/intel/cannonlake_rvp/variants/baseboard/include/baseboard/variants.h
M src/mainboard/intel/coffeelake_rvp/variants/baseboard/include/baseboard/variants.h
M src/mainboard/intel/glkrvp/variants/baseboard/include/baseboard/variants.h
M src/mainboard/intel/icelake_rvp/variants/baseboard/include/baseboard/variants.h
M src/mainboard/intel/jasperlake_rvp/board_id.h
M src/mainboard/intel/tglrvp/board_id.h
M src/mainboard/intel/tglrvp/variants/baseboard/include/baseboard/variants.h
M src/northbridge/intel/sandybridge/romstage.c
M src/security/memory/memory.h
M src/security/vboot/vboot_common.h
M src/soc/amd/common/block/include/amdblocks/agesawrapper.h
M src/soc/amd/common/block/include/amdblocks/s3_resume.h
M src/soc/intel/baytrail/include/soc/romstage.h
M src/soc/intel/braswell/include/soc/romstage.h
M src/soc/intel/broadwell/include/soc/romstage.h
M src/soc/intel/cannonlake/include/soc/pch.h
M src/soc/intel/common/block/include/intelblocks/pcie_rp.h
M src/soc/intel/common/block/smbus/smbuslib.h
M src/soc/intel/common/mma.h
M src/soc/intel/common/pch/include/intelpch/lockdown.h
M src/soc/intel/icelake/include/soc/pch.h
M src/soc/intel/jasperlake/include/soc/espi.h
M src/soc/intel/jasperlake/include/soc/pch.h
M src/soc/intel/jasperlake/me.c
M src/soc/intel/tigerlake/include/soc/espi.h
M src/soc/intel/tigerlake/include/soc/pch.h
M src/soc/mediatek/mt8173/include/soc/addressmap.h
M src/soc/nvidia/tegra/usb.h
M src/soc/nvidia/tegra124/include/soc/gpio.h
M src/soc/nvidia/tegra124/include/soc/pingroup.h
M src/soc/nvidia/tegra124/include/soc/pinmux.h
M src/soc/nvidia/tegra210/include/soc/ccplex.h
M src/soc/nvidia/tegra210/include/soc/pinmux.h
M src/soc/qualcomm/qcs405/include/soc/addressmap.h
M src/soc/qualcomm/sc7180/include/soc/addressmap.h
M src/soc/qualcomm/sdm845/include/soc/addressmap.h
M src/southbridge/amd/cimx/sb800/cfg.h
46 files changed, 1 insertion(+), 48 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/27/41827/17

[HAOUAS Elyes (Code Review)]

HAOUAS Elyes has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/41827 )

Change subject: src: Remove unused 'include <stdint.h>
......................................................................

Abandoned

see 42340

[HAOUAS Elyes (Code Review)]

HAOUAS Elyes has restored this change. ( https://review.coreboot.org/c/coreboot/+/41827 )

Change subject: src: Remove unused 'include <stdint.h>
......................................................................

Restored

[HAOUAS Elyes (Code Review)]

Hello build bot (Jenkins), Lee Leahy, Paul Menzel, Huang Jin, Julius Werner, Angel Pons, Aaron Durbin, Patrick Rudolph,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/41827

to look at the new patch set (#18).

Change subject: src: Remove unused 'include <stdint.h>
......................................................................

src: Remove unused 'include <stdint.h>

Found using:
diff <(git grep -l '#include <stdint.h>' -- src/) <(git grep -l 'int8_t|int16_t|int32_t|int64_t|intptr_t|intmax_t|s8|u8|s16|u16|s32|u32|s64|u64|INT8_MIN|INT8_MAX|INT16_MIN|INT16_MAX|INT32_MIN|INT32_MAX|INT64_MIN|INT64_MAX|INTMAX_MIN|INTMAX_MAX' -- src/) |grep -v vendorcode |grep '<'

Change-Id: I5e14bf4887c7d2644a64f4d58c6d8763eb74d2ed
Signed-off-by: Elyes HAOUAS 
---
M src/arch/arm/include/armv4/arch/cache.h
M src/arch/arm64/include/armv8/arch/cache.h
M src/arch/x86/include/arch/ebda.h
M src/commonlib/include/commonlib/region.h
M src/drivers/i2c/rt1011/chip.h
M src/drivers/intel/fsp1_1/include/fsp/car.h
M src/drivers/intel/ptt/ptt.h
M src/drivers/intel/soundwire/soundwire.c
M src/mainboard/google/slippy/variants/falco/romstage.c
M src/mainboard/google/slippy/variants/leon/romstage.c
M src/mainboard/google/slippy/variants/wolf/romstage.c
M src/mainboard/intel/cannonlake_rvp/variants/baseboard/include/baseboard/variants.h
M src/mainboard/intel/coffeelake_rvp/variants/baseboard/include/baseboard/variants.h
M src/mainboard/intel/glkrvp/variants/baseboard/include/baseboard/variants.h
M src/mainboard/intel/icelake_rvp/variants/baseboard/include/baseboard/variants.h
M src/mainboard/intel/jasperlake_rvp/board_id.h
M src/mainboard/intel/tglrvp/board_id.h
M src/mainboard/intel/tglrvp/variants/baseboard/include/baseboard/variants.h
M src/northbridge/intel/sandybridge/romstage.c
M src/security/memory/memory.h
M src/security/vboot/vboot_common.h
M src/soc/amd/common/block/include/amdblocks/agesawrapper.h
M src/soc/amd/common/block/include/amdblocks/s3_resume.h
M src/soc/amd/picasso/psp_verstage/printk.c
M src/soc/amd/picasso/psp_verstage/reset.c
M src/soc/intel/baytrail/include/soc/romstage.h
M src/soc/intel/braswell/include/soc/romstage.h
M src/soc/intel/broadwell/include/soc/romstage.h
M src/soc/intel/cannonlake/include/soc/pch.h
M src/soc/intel/common/block/include/intelblocks/pcie_rp.h
M src/soc/intel/common/block/smbus/smbuslib.h
M src/soc/intel/common/mma.h
M src/soc/intel/common/pch/include/intelpch/lockdown.h
M src/soc/intel/icelake/include/soc/pch.h
M src/soc/intel/jasperlake/include/soc/espi.h
M src/soc/intel/jasperlake/include/soc/pch.h
M src/soc/intel/jasperlake/me.c
M src/soc/intel/tigerlake/include/soc/espi.h
M src/soc/intel/tigerlake/include/soc/pch.h
M src/soc/mediatek/mt8173/include/soc/addressmap.h
M src/soc/nvidia/tegra/usb.h
M src/soc/nvidia/tegra124/include/soc/gpio.h
M src/soc/nvidia/tegra124/include/soc/pingroup.h
M src/soc/nvidia/tegra124/include/soc/pinmux.h
M src/soc/nvidia/tegra210/include/soc/ccplex.h
M src/soc/nvidia/tegra210/include/soc/pinmux.h
M src/soc/qualcomm/qcs405/include/soc/addressmap.h
M src/soc/qualcomm/sc7180/include/soc/addressmap.h
M src/soc/qualcomm/sdm845/include/soc/addressmap.h
M src/southbridge/amd/cimx/sb800/cfg.h
50 files changed, 0 insertions(+), 53 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/27/41827/18

[Patrick Georgi (Code Review)]

Patrick Georgi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41827 )

Change subject: src: Remove unused 'include <stdint.h>
......................................................................

Patch Set 19: Code-Review+2

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/41827 )

Change subject: src: Remove unused 'include <stdint.h>
......................................................................

src: Remove unused 'include <stdint.h>

Found using:
diff <(git grep -l '#include <stdint.h>' -- src/) <(git grep -l 'int8_t|int16_t|int32_t|int64_t|intptr_t|intmax_t|s8|u8|s16|u16|s32|u32|s64|u64|INT8_MIN|INT8_MAX|INT16_MIN|INT16_MAX|INT32_MIN|INT32_MAX|INT64_MIN|INT64_MAX|INTMAX_MIN|INTMAX_MAX' -- src/) |grep -v vendorcode |grep '<'

Change-Id: I5e14bf4887c7d2644a64f4d58c6d8763eb74d2ed
Signed-off-by: Elyes HAOUAS 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/41827
Reviewed-by: Patrick Georgi 
Tested-by: build bot (Jenkins) 
---
M src/arch/arm/include/armv4/arch/cache.h
M src/arch/arm64/include/armv8/arch/cache.h
M src/arch/x86/include/arch/ebda.h
M src/commonlib/include/commonlib/region.h
M src/drivers/i2c/rt1011/chip.h
M src/drivers/intel/fsp1_1/include/fsp/car.h
M src/drivers/intel/ptt/ptt.h
M src/drivers/intel/soundwire/soundwire.c
M src/mainboard/google/slippy/variants/falco/romstage.c
M src/mainboard/google/slippy/variants/leon/romstage.c
M src/mainboard/google/slippy/variants/wolf/romstage.c
M src/mainboard/intel/cannonlake_rvp/variants/baseboard/include/baseboard/variants.h
M src/mainboard/intel/coffeelake_rvp/variants/baseboard/include/baseboard/variants.h
M src/mainboard/intel/glkrvp/variants/baseboard/include/baseboard/variants.h
M src/mainboard/intel/icelake_rvp/variants/baseboard/include/baseboard/variants.h
M src/mainboard/intel/jasperlake_rvp/board_id.h
M src/mainboard/intel/tglrvp/board_id.h
M src/mainboard/intel/tglrvp/variants/baseboard/include/baseboard/variants.h
M src/northbridge/intel/sandybridge/romstage.c
M src/security/memory/memory.h
M src/security/vboot/vboot_common.h
M src/soc/amd/common/block/include/amdblocks/agesawrapper.h
M src/soc/amd/common/block/include/amdblocks/s3_resume.h
M src/soc/amd/picasso/psp_verstage/printk.c
M src/soc/amd/picasso/psp_verstage/reset.c
M src/soc/intel/baytrail/include/soc/romstage.h
M src/soc/intel/braswell/include/soc/romstage.h
M src/soc/intel/broadwell/include/soc/romstage.h
M src/soc/intel/cannonlake/include/soc/pch.h
M src/soc/intel/common/block/include/intelblocks/pcie_rp.h
M src/soc/intel/common/block/smbus/smbuslib.h
M src/soc/intel/common/mma.h
M src/soc/intel/common/pch/include/intelpch/lockdown.h
M src/soc/intel/icelake/include/soc/pch.h
M src/soc/intel/jasperlake/include/soc/espi.h
M src/soc/intel/jasperlake/include/soc/pch.h
M src/soc/intel/jasperlake/me.c
M src/soc/intel/tigerlake/include/soc/espi.h
M src/soc/intel/tigerlake/include/soc/pch.h
M src/soc/mediatek/mt8173/include/soc/addressmap.h
M src/soc/nvidia/tegra/usb.h
M src/soc/nvidia/tegra124/include/soc/gpio.h
M src/soc/nvidia/tegra124/include/soc/pingroup.h
M src/soc/nvidia/tegra124/include/soc/pinmux.h
M src/soc/nvidia/tegra210/include/soc/ccplex.h
M src/soc/nvidia/tegra210/include/soc/pinmux.h
M src/soc/qualcomm/qcs405/include/soc/addressmap.h
M src/soc/qualcomm/sc7180/include/soc/addressmap.h
M src/soc/qualcomm/sdm845/include/soc/addressmap.h
M src/southbridge/amd/cimx/sb800/cfg.h
50 files changed, 0 insertions(+), 53 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Patrick Georgi: Looks good to me, approved

diff --git a/src/arch/arm/include/armv4/arch/cache.h b/src/arch/arm/include/armv4/arch/cache.h
index 8807beb..8aa619d 100644
--- a/src/arch/arm/include/armv4/arch/cache.h
+++ b/src/arch/arm/include/armv4/arch/cache.h
@@ -7,7 +7,6 @@
 #define ARM_CACHE_H

#include <stddef.h>
-#include <stdint.h>

/*
  * Cache maintenance API
diff --git a/src/arch/arm64/include/armv8/arch/cache.h b/src/arch/arm64/include/armv8/arch/cache.h
index a78d03c..bb9d6fe 100644
--- a/src/arch/arm64/include/armv8/arch/cache.h
+++ b/src/arch/arm64/include/armv8/arch/cache.h
@@ -11,7 +11,6 @@
 #ifndef __ASSEMBLER__

#include <stddef.h>
-#include <stdint.h>
 #include <arch/barrier.h>

/* dcache clean by virtual address to PoC */
diff --git a/src/arch/x86/include/arch/ebda.h b/src/arch/x86/include/arch/ebda.h
index 71426b1..e625d4a 100644
--- a/src/arch/x86/include/arch/ebda.h
+++ b/src/arch/x86/include/arch/ebda.h
@@ -3,7 +3,6 @@
 #ifndef __ARCH_EBDA_H
 #define __ARCH_EBDA_H

-#include <stdint.h>
 #include <stddef.h>

#define X86_BDA_SIZE		0x200
diff --git a/src/commonlib/include/commonlib/region.h b/src/commonlib/include/commonlib/region.h
index c7c339f..b9a984f 100644
--- a/src/commonlib/include/commonlib/region.h
+++ b/src/commonlib/include/commonlib/region.h
@@ -4,7 +4,6 @@
 #define _REGION_H_

#include <sys/types.h>
-#include <stdint.h>
 #include <stddef.h>
 #include <stdbool.h>
 #include <commonlib/mem_pool.h>
diff --git a/src/drivers/i2c/rt1011/chip.h b/src/drivers/i2c/rt1011/chip.h
index 9d1a121..e5dee97 100644
--- a/src/drivers/i2c/rt1011/chip.h
+++ b/src/drivers/i2c/rt1011/chip.h
@@ -4,7 +4,6 @@
  * Realtek RT1011 audio codec devicetree bindings
  */

-#include <stdint.h>

struct drivers_i2c_rt1011_config {
    const char *name; /* ACPI Device Name */
diff --git a/src/drivers/intel/fsp1_1/include/fsp/car.h b/src/drivers/intel/fsp1_1/include/fsp/car.h
index db203ae..ceb87d8 100644
--- a/src/drivers/intel/fsp1_1/include/fsp/car.h
+++ b/src/drivers/intel/fsp1_1/include/fsp/car.h
@@ -4,7 +4,6 @@
 #define FSP1_1_CAR_H

#include <fsp/api.h>
-#include <stdint.h>

/* Per stage calls from the above two functions. The void * return from
  * cache_as_ram_stage_main() is the stack pointer to use in RAM after
diff --git a/src/drivers/intel/ptt/ptt.h b/src/drivers/intel/ptt/ptt.h
index 7bc878d..b447d58 100644
--- a/src/drivers/intel/ptt/ptt.h
+++ b/src/drivers/intel/ptt/ptt.h
@@ -8,7 +8,6 @@
  * is still in the preboot phase.
  *
  */
-#include <stdint.h>
 /*
  * ptt_active
  *
diff --git a/src/drivers/intel/soundwire/soundwire.c b/src/drivers/intel/soundwire/soundwire.c
index ab09ff4..34ecd86 100644
--- a/src/drivers/intel/soundwire/soundwire.c
+++ b/src/drivers/intel/soundwire/soundwire.c
@@ -8,7 +8,6 @@
 #include <device/path.h>
 #include <device/soundwire.h>
 #include <stdbool.h>
-#include <stdint.h>

#include "soundwire.h"
 #include "chip.h"
diff --git a/src/mainboard/google/slippy/variants/falco/romstage.c b/src/mainboard/google/slippy/variants/falco/romstage.c
index e4af352..eba4a8b 100644
--- a/src/mainboard/google/slippy/variants/falco/romstage.c
+++ b/src/mainboard/google/slippy/variants/falco/romstage.c
@@ -1,7 +1,6 @@
 /* SPDX-License-Identifier: GPL-2.0-only */

#include <stddef.h>
-#include <stdint.h>
 #include <string.h>
 #include <cbfs.h>
 #include <console/console.h>
diff --git a/src/mainboard/google/slippy/variants/leon/romstage.c b/src/mainboard/google/slippy/variants/leon/romstage.c
index 7a13b90..2b1b5ca 100644
--- a/src/mainboard/google/slippy/variants/leon/romstage.c
+++ b/src/mainboard/google/slippy/variants/leon/romstage.c
@@ -1,6 +1,5 @@
 /* SPDX-License-Identifier: GPL-2.0-only */

-#include <stdint.h>
 #include <string.h>
 #include <cbfs.h>
 #include <console/console.h>
diff --git a/src/mainboard/google/slippy/variants/wolf/romstage.c b/src/mainboard/google/slippy/variants/wolf/romstage.c
index 4e02f73..405e86a 100644
--- a/src/mainboard/google/slippy/variants/wolf/romstage.c
+++ b/src/mainboard/google/slippy/variants/wolf/romstage.c
@@ -1,6 +1,5 @@
 /* SPDX-License-Identifier: GPL-2.0-only */

-#include <stdint.h>
 #include <string.h>
 #include <cbfs.h>
 #include <console/console.h>
diff --git a/src/mainboard/intel/cannonlake_rvp/variants/baseboard/include/baseboard/variants.h b/src/mainboard/intel/cannonlake_rvp/variants/baseboard/include/baseboard/variants.h
index af9ba07..f77d38b 100644
--- a/src/mainboard/intel/cannonlake_rvp/variants/baseboard/include/baseboard/variants.h
+++ b/src/mainboard/intel/cannonlake_rvp/variants/baseboard/include/baseboard/variants.h
@@ -4,7 +4,6 @@
 #define __BASEBOARD_VARIANTS_H__

#include <soc/gpio.h>
-#include <stdint.h>
 #include <vendorcode/google/chromeos/chromeos.h>

/* The next set of functions return the gpio table and fill in the number of
diff --git a/src/mainboard/intel/coffeelake_rvp/variants/baseboard/include/baseboard/variants.h b/src/mainboard/intel/coffeelake_rvp/variants/baseboard/include/baseboard/variants.h
index 48abbfe..b7a1616 100644
--- a/src/mainboard/intel/coffeelake_rvp/variants/baseboard/include/baseboard/variants.h
+++ b/src/mainboard/intel/coffeelake_rvp/variants/baseboard/include/baseboard/variants.h
@@ -5,7 +5,6 @@

#include <soc/cnl_memcfg_init.h>
 #include <soc/gpio.h>
-#include <stdint.h>
 #include <vendorcode/google/chromeos/chromeos.h>

/* The next set of functions return the gpio table and fill in the number of
diff --git a/src/mainboard/intel/glkrvp/variants/baseboard/include/baseboard/variants.h b/src/mainboard/intel/glkrvp/variants/baseboard/include/baseboard/variants.h
index 178064f..f1df937 100644
--- a/src/mainboard/intel/glkrvp/variants/baseboard/include/baseboard/variants.h
+++ b/src/mainboard/intel/glkrvp/variants/baseboard/include/baseboard/variants.h
@@ -5,7 +5,6 @@

#include <soc/gpio.h>
 #include <soc/meminit.h>
-#include <stdint.h>
 #include <vendorcode/google/chromeos/chromeos.h>

/**
diff --git a/src/mainboard/intel/icelake_rvp/variants/baseboard/include/baseboard/variants.h b/src/mainboard/intel/icelake_rvp/variants/baseboard/include/baseboard/variants.h
index 16a4cbc..ccbadca 100644
--- a/src/mainboard/intel/icelake_rvp/variants/baseboard/include/baseboard/variants.h
+++ b/src/mainboard/intel/icelake_rvp/variants/baseboard/include/baseboard/variants.h
@@ -4,7 +4,6 @@
 #define __BASEBOARD_VARIANTS_H__

#include <soc/gpio.h>
-#include <stdint.h>
 #include <vendorcode/google/chromeos/chromeos.h>

/* The next set of functions return the gpio table and fill in the number of
diff --git a/src/mainboard/intel/jasperlake_rvp/board_id.h b/src/mainboard/intel/jasperlake_rvp/board_id.h
index 20483c8..55b4f52 100644
--- a/src/mainboard/intel/jasperlake_rvp/board_id.h
+++ b/src/mainboard/intel/jasperlake_rvp/board_id.h
@@ -3,7 +3,6 @@
 #ifndef _MAINBOARD_COMMON_BOARD_ID_H_
 #define _MAINBOARD_COMMON_BOARD_ID_H_

-#include <stdint.h>

/* Board/FAB ID Command */
 #define EC_FAB_ID_CMD	0x0D
diff --git a/src/mainboard/intel/tglrvp/board_id.h b/src/mainboard/intel/tglrvp/board_id.h
index b87f99a..c8d6cd2 100644
--- a/src/mainboard/intel/tglrvp/board_id.h
+++ b/src/mainboard/intel/tglrvp/board_id.h
@@ -3,7 +3,6 @@
 #ifndef _MAINBOARD_COMMON_BOARD_ID_H_
 #define _MAINBOARD_COMMON_BOARD_ID_H_

-#include <stdint.h>

/* Board/FAB ID Command */
 #define EC_FAB_ID_CMD	0x0D
diff --git a/src/mainboard/intel/tglrvp/variants/baseboard/include/baseboard/variants.h b/src/mainboard/intel/tglrvp/variants/baseboard/include/baseboard/variants.h
index 0da97cd..bad9e9e 100644
--- a/src/mainboard/intel/tglrvp/variants/baseboard/include/baseboard/variants.h
+++ b/src/mainboard/intel/tglrvp/variants/baseboard/include/baseboard/variants.h
@@ -5,7 +5,6 @@

#include <soc/gpio.h>
 #include <soc/meminit.h>
-#include <stdint.h>
 #include <vendorcode/google/chromeos/chromeos.h>

/* The next set of functions return the gpio table and fill in the number of
diff --git a/src/northbridge/intel/sandybridge/romstage.c b/src/northbridge/intel/sandybridge/romstage.c
index b9841b5..5c0dd5c 100644
--- a/src/northbridge/intel/sandybridge/romstage.c
+++ b/src/northbridge/intel/sandybridge/romstage.c
@@ -1,6 +1,5 @@
 /* SPDX-License-Identifier: GPL-2.0-only */

-#include <stdint.h>
 #include <console/console.h>
 #include <cf9_reset.h>
 #include <device/pci_ops.h>
diff --git a/src/security/memory/memory.h b/src/security/memory/memory.h
index fea341e..442ace1 100644
--- a/src/security/memory/memory.h
+++ b/src/security/memory/memory.h
@@ -1,5 +1,4 @@
 /* SPDX-License-Identifier: GPL-2.0-only */

-#include <stdint.h>

bool security_clear_dram_request(void);
diff --git a/src/security/vboot/vboot_common.h b/src/security/vboot/vboot_common.h
index 66e96e4..f2cff65 100644
--- a/src/security/vboot/vboot_common.h
+++ b/src/security/vboot/vboot_common.h
@@ -3,7 +3,6 @@
 #define __VBOOT_VBOOT_COMMON_H__

#include <commonlib/region.h>
-#include <stdint.h>
 #include <vb2_api.h>

/*
diff --git a/src/soc/amd/common/block/include/amdblocks/agesawrapper.h b/src/soc/amd/common/block/include/amdblocks/agesawrapper.h
index 348798e..8883893 100644
--- a/src/soc/amd/common/block/include/amdblocks/agesawrapper.h
+++ b/src/soc/amd/common/block/include/amdblocks/agesawrapper.h
@@ -3,7 +3,6 @@
 #ifndef __AGESAWRAPPER_H__
 #define __AGESAWRAPPER_H__

-#include <stdint.h>
 #include <agesa_headers.h>

enum {
diff --git a/src/soc/amd/common/block/include/amdblocks/s3_resume.h b/src/soc/amd/common/block/include/amdblocks/s3_resume.h
index 3989508..305ed04 100644
--- a/src/soc/amd/common/block/include/amdblocks/s3_resume.h
+++ b/src/soc/amd/common/block/include/amdblocks/s3_resume.h
@@ -3,7 +3,6 @@
 #ifndef __AMD_S3_RESUME_H__
 #define __AMD_S3_RESUME_H__

-#include <stdint.h>
 #include <agesa_headers.h>

AGESA_STATUS OemInitResume(S3_DATA_BLOCK *dataBlock);
diff --git a/src/soc/amd/picasso/psp_verstage/printk.c b/src/soc/amd/picasso/psp_verstage/printk.c
index 2252d44..2f9d30a 100644
--- a/src/soc/amd/picasso/psp_verstage/printk.c
+++ b/src/soc/amd/picasso/psp_verstage/printk.c
@@ -4,7 +4,6 @@
 #include <console/console.h>
 #include <console/streams.h>
 #include <stdarg.h>
-#include <stdint.h>

void console_hw_init(void)
 {
diff --git a/src/soc/amd/picasso/psp_verstage/reset.c b/src/soc/amd/picasso/psp_verstage/reset.c
index 5619e45..7cd7907 100644
--- a/src/soc/amd/picasso/psp_verstage/reset.c
+++ b/src/soc/amd/picasso/psp_verstage/reset.c
@@ -3,8 +3,6 @@
 #include <bl_uapp/bl_syscall_public.h>
 #include <console/console.h>
 #include <reset.h>
-#include <stdint.h>
-

void do_board_reset(void)
 {
diff --git a/src/soc/intel/baytrail/include/soc/romstage.h b/src/soc/intel/baytrail/include/soc/romstage.h
index a7b7fce..156fc31 100644
--- a/src/soc/intel/baytrail/include/soc/romstage.h
+++ b/src/soc/intel/baytrail/include/soc/romstage.h
@@ -3,7 +3,6 @@
 #ifndef _BAYTRAIL_ROMSTAGE_H_
 #define _BAYTRAIL_ROMSTAGE_H_

-#include <stdint.h>
 #include <soc/mrc_wrapper.h>

void mainboard_fill_mrc_params(struct mrc_params *mp);
diff --git a/src/soc/intel/braswell/include/soc/romstage.h b/src/soc/intel/braswell/include/soc/romstage.h
index de890fd..2cf9afa 100644
--- a/src/soc/intel/braswell/include/soc/romstage.h
+++ b/src/soc/intel/braswell/include/soc/romstage.h
@@ -3,7 +3,6 @@
 #ifndef _SOC_ROMSTAGE_H_
 #define _SOC_ROMSTAGE_H_

-#include <stdint.h>
 #include <fsp/romstage.h>
 #include <fsp/util.h>
 #include <soc/pm.h>
diff --git a/src/soc/intel/broadwell/include/soc/romstage.h b/src/soc/intel/broadwell/include/soc/romstage.h
index 2c7b0a4..f45419c 100644
--- a/src/soc/intel/broadwell/include/soc/romstage.h
+++ b/src/soc/intel/broadwell/include/soc/romstage.h
@@ -3,7 +3,6 @@
 #ifndef _BROADWELL_ROMSTAGE_H_
 #define _BROADWELL_ROMSTAGE_H_

-#include <stdint.h>
 #include <soc/pei_data.h>

struct chipset_power_state;
diff --git a/src/soc/intel/cannonlake/include/soc/pch.h b/src/soc/intel/cannonlake/include/soc/pch.h
index 754eabb..778b269 100644
--- a/src/soc/intel/cannonlake/include/soc/pch.h
+++ b/src/soc/intel/cannonlake/include/soc/pch.h
@@ -3,7 +3,6 @@
 #ifndef _SOC_CANNONLAKE_PCH_H_
 #define _SOC_CANNONLAKE_PCH_H_

-#include <stdint.h>

#define PCH_H				1
 #define PCH_LP				2
diff --git a/src/soc/intel/common/block/include/intelblocks/pcie_rp.h b/src/soc/intel/common/block/include/intelblocks/pcie_rp.h
index c75e055..5f5aab5 100644
--- a/src/soc/intel/common/block/include/intelblocks/pcie_rp.h
+++ b/src/soc/intel/common/block/include/intelblocks/pcie_rp.h
@@ -3,7 +3,6 @@
 #ifndef SOC_INTEL_COMMON_BLOCK_PCIE_RP_H
 #define SOC_INTEL_COMMON_BLOCK_PCIE_RP_H

-#include <stdint.h>

/*
  * The PCIe Root Ports usually come in groups of up to 8 PCI-device
diff --git a/src/soc/intel/common/block/smbus/smbuslib.h b/src/soc/intel/common/block/smbus/smbuslib.h
index 2d8774c..05cecf3 100644
--- a/src/soc/intel/common/block/smbus/smbuslib.h
+++ b/src/soc/intel/common/block/smbus/smbuslib.h
@@ -3,7 +3,6 @@
 #ifndef SOC_INTEL_COMMON_BLOCK_SMBUS__LIB_H
 #define SOC_INTEL_COMMON_BLOCK_SMBUS__LIB_H

-#include <stdint.h>

/* SMBus IO Base Address */
 #define SMBUS_IO_BASE	0xefa0
diff --git a/src/soc/intel/common/mma.h b/src/soc/intel/common/mma.h
index 2e01210..e58edaf 100644
--- a/src/soc/intel/common/mma.h
+++ b/src/soc/intel/common/mma.h
@@ -3,7 +3,6 @@
 #ifndef _SOC_MMA_H_
 #define _SOC_MMA_H_

-#include <stdint.h>
 #include <commonlib/region.h>

struct mma_config_param {
diff --git a/src/soc/intel/common/pch/include/intelpch/lockdown.h b/src/soc/intel/common/pch/include/intelpch/lockdown.h
index 72deebd..17b8cbc 100644
--- a/src/soc/intel/common/pch/include/intelpch/lockdown.h
+++ b/src/soc/intel/common/pch/include/intelpch/lockdown.h
@@ -3,7 +3,6 @@
 #ifndef SOC_INTEL_COMMON_PCH_LOCKDOWN_H
 #define SOC_INTEL_COMMON_PCH_LOCKDOWN_H

-#include <stdint.h>

/*
  * This function will get lockdown config specific to soc.
diff --git a/src/soc/intel/icelake/include/soc/pch.h b/src/soc/intel/icelake/include/soc/pch.h
index c66808b..cbba2b4 100644
--- a/src/soc/intel/icelake/include/soc/pch.h
+++ b/src/soc/intel/icelake/include/soc/pch.h
@@ -3,7 +3,6 @@
 #ifndef _SOC_ICELAKE_PCH_H_
 #define _SOC_ICELAKE_PCH_H_

-#include <stdint.h>

#define PCH_H				1
 #define PCH_LP				2
diff --git a/src/soc/intel/jasperlake/include/soc/espi.h b/src/soc/intel/jasperlake/include/soc/espi.h
index 7797534..5c99adb 100644
--- a/src/soc/intel/jasperlake/include/soc/espi.h
+++ b/src/soc/intel/jasperlake/include/soc/espi.h
@@ -3,7 +3,6 @@
 #ifndef _SOC_JASPERLAKE_ESPI_H_
 #define _SOC_JASPERLAKE_ESPI_H_

-#include <stdint.h>

/* PCI Configuration Space (D31:F0): ESPI */
 #define SCI_IRQ_SEL		(7 << 0)
diff --git a/src/soc/intel/jasperlake/include/soc/pch.h b/src/soc/intel/jasperlake/include/soc/pch.h
index 1a6396d..db62c86 100644
--- a/src/soc/intel/jasperlake/include/soc/pch.h
+++ b/src/soc/intel/jasperlake/include/soc/pch.h
@@ -3,7 +3,6 @@
 #ifndef _SOC_JASPERLAKE_PCH_H_
 #define _SOC_JASPERLAKE_PCH_H_

-#include <stdint.h>

#define PCIE_CLK_NOTUSED		0xFF
 #define PCIE_CLK_LAN			0x70
diff --git a/src/soc/intel/jasperlake/me.c b/src/soc/intel/jasperlake/me.c
index e4dc93f..b40b4dc 100644
--- a/src/soc/intel/jasperlake/me.c
+++ b/src/soc/intel/jasperlake/me.c
@@ -4,7 +4,6 @@
 #include <intelblocks/cse.h>
 #include <console/console.h>
 #include <soc/me.h>
-#include <stdint.h>

static void dump_me_status(void *unused)
 {
diff --git a/src/soc/intel/tigerlake/include/soc/espi.h b/src/soc/intel/tigerlake/include/soc/espi.h
index 449eb63..6a86147 100644
--- a/src/soc/intel/tigerlake/include/soc/espi.h
+++ b/src/soc/intel/tigerlake/include/soc/espi.h
@@ -9,7 +9,6 @@
 #ifndef _SOC_TIGERLAKE_ESPI_H_
 #define _SOC_TIGERLAKE_ESPI_H_

-#include <stdint.h>

/* PCI Configuration Space (D31:F0): ESPI */
 #define SCI_IRQ_SEL		(7 << 0)
diff --git a/src/soc/intel/tigerlake/include/soc/pch.h b/src/soc/intel/tigerlake/include/soc/pch.h
index 457b616..ad0186a 100644
--- a/src/soc/intel/tigerlake/include/soc/pch.h
+++ b/src/soc/intel/tigerlake/include/soc/pch.h
@@ -3,7 +3,6 @@
 #ifndef _SOC_TIGERLAKE_PCH_H_
 #define _SOC_TIGERLAKE_PCH_H_

-#include <stdint.h>

#define PCIE_CLK_NOTUSED		0xFF
 #define PCIE_CLK_LAN			0x70
diff --git a/src/soc/mediatek/mt8173/include/soc/addressmap.h b/src/soc/mediatek/mt8173/include/soc/addressmap.h
index 4b2e150..772f6f2 100644
--- a/src/soc/mediatek/mt8173/include/soc/addressmap.h
+++ b/src/soc/mediatek/mt8173/include/soc/addressmap.h
@@ -4,7 +4,6 @@
 #define __SOC_MEDIATEK_MT8173_INCLUDE_SOC_ADDRESS_MAP_H__

#include <stddef.h>
-#include <stdint.h>

enum {
    VER_BASE	= 0x08000000,
diff --git a/src/soc/nvidia/tegra/usb.h b/src/soc/nvidia/tegra/usb.h
index 2a14fef..387fb67 100644
--- a/src/soc/nvidia/tegra/usb.h
+++ b/src/soc/nvidia/tegra/usb.h
@@ -3,7 +3,6 @@
 #ifndef __SOC_NVIDIA_TEGRA_USB_H__
 #define __SOC_NVIDIA_TEGRA_USB_H__

-#include <stdint.h>

enum usb_phy_type {		/* For use in lpm_ctrl[31:29] */
    USB_PHY_UTMIP = 0,
diff --git a/src/soc/nvidia/tegra124/include/soc/gpio.h b/src/soc/nvidia/tegra124/include/soc/gpio.h
index 10612c4..6b66a98 100644
--- a/src/soc/nvidia/tegra124/include/soc/gpio.h
+++ b/src/soc/nvidia/tegra124/include/soc/gpio.h
@@ -5,7 +5,6 @@

#include <soc/nvidia/tegra/gpio.h>
 #include <soc/pinmux.h>	/* for pinmux constants in GPIO macro */
-#include <stdint.h>

/* GPIO index constants. */

diff --git a/src/soc/nvidia/tegra124/include/soc/pingroup.h b/src/soc/nvidia/tegra124/include/soc/pingroup.h
index d9a94a8..7b1fb16 100644
--- a/src/soc/nvidia/tegra124/include/soc/pingroup.h
+++ b/src/soc/nvidia/tegra124/include/soc/pingroup.h
@@ -4,7 +4,6 @@
 #define __SOC_NVIDIA_TEGRA124_PINGROUP_H__

#include <soc/nvidia/tegra/pingroup.h>
-#include <stdint.h>

enum {
    PINGROUP_AO1_INDEX = 0,		/* offset 0x868 */
diff --git a/src/soc/nvidia/tegra124/include/soc/pinmux.h b/src/soc/nvidia/tegra124/include/soc/pinmux.h
index f5f01ef..e0dfc9d 100644
--- a/src/soc/nvidia/tegra124/include/soc/pinmux.h
+++ b/src/soc/nvidia/tegra124/include/soc/pinmux.h
@@ -4,7 +4,6 @@
 #define __SOC_NVIDIA_TEGRA124_PINMUX_H__

#include <soc/nvidia/tegra/pinmux.h>
-#include <stdint.h>

enum {
    PINMUX_FUNC_MASK = 3 << 0,
diff --git a/src/soc/nvidia/tegra210/include/soc/ccplex.h b/src/soc/nvidia/tegra210/include/soc/ccplex.h
index 2fb3087..3975f44 100644
--- a/src/soc/nvidia/tegra210/include/soc/ccplex.h
+++ b/src/soc/nvidia/tegra210/include/soc/ccplex.h
@@ -3,7 +3,6 @@
 #ifndef __SOC_NVIDIA_TEGRA210_CCPLEX_H__
 #define __SOC_NVIDIA_TEGRA210_CCPLEX_H__

-#include <stdint.h>

#define MTS_LOAD_ADDRESS 0x82000000

diff --git a/src/soc/nvidia/tegra210/include/soc/pinmux.h b/src/soc/nvidia/tegra210/include/soc/pinmux.h
index 4c8c4be..195aeeb 100644
--- a/src/soc/nvidia/tegra210/include/soc/pinmux.h
+++ b/src/soc/nvidia/tegra210/include/soc/pinmux.h
@@ -3,7 +3,6 @@
 #ifndef __SOC_NVIDIA_TEGRA210_PINMUX_H__
 #define __SOC_NVIDIA_TEGRA210_PINMUX_H__

-#include <stdint.h>
 #include <soc/nvidia/tegra/gpio.h>
 #include <soc/nvidia/tegra/pinmux.h>

diff --git a/src/soc/qualcomm/qcs405/include/soc/addressmap.h b/src/soc/qualcomm/qcs405/include/soc/addressmap.h
index 8e1700c..1d46071 100644
--- a/src/soc/qualcomm/qcs405/include/soc/addressmap.h
+++ b/src/soc/qualcomm/qcs405/include/soc/addressmap.h
@@ -3,7 +3,6 @@
 #ifndef __SOC_QUALCOMM_QCS405_ADDRESS_MAP_H__
 #define __SOC_QUALCOMM_QCS405_ADDRESS_MAP_H__

-#include <stdint.h>

#define QSPI_BASE			0x88DF000
 #define TLMM_EAST_TILE_BASE		0x7B00000
diff --git a/src/soc/qualcomm/sc7180/include/soc/addressmap.h b/src/soc/qualcomm/sc7180/include/soc/addressmap.h
index c5b48bf..2b65e99 100644
--- a/src/soc/qualcomm/sc7180/include/soc/addressmap.h
+++ b/src/soc/qualcomm/sc7180/include/soc/addressmap.h
@@ -3,7 +3,6 @@
 #ifndef _SOC_QUALCOMM_SC7180_ADDRESS_MAP_H_
 #define _SOC_QUALCOMM_SC7180_ADDRESS_MAP_H_

-#include <stdint.h>

#define AOSS_CC_BASE			0x0C2A0000
 #define GCC_BASE			0x00100000
diff --git a/src/soc/qualcomm/sdm845/include/soc/addressmap.h b/src/soc/qualcomm/sdm845/include/soc/addressmap.h
index bddb551..ba6bc8c 100644
--- a/src/soc/qualcomm/sdm845/include/soc/addressmap.h
+++ b/src/soc/qualcomm/sdm845/include/soc/addressmap.h
@@ -3,7 +3,6 @@
 #ifndef __SOC_QUALCOMM_SDM845_ADDRESS_MAP_H__
 #define __SOC_QUALCOMM_SDM845_ADDRESS_MAP_H__

-#include <stdint.h>

#define QSPI_BASE			0x88DF000
 #define TLMM_EAST_TILE_BASE             0x03500000
diff --git a/src/southbridge/amd/cimx/sb800/cfg.h b/src/southbridge/amd/cimx/sb800/cfg.h
index 3c6cda1..25b2a45 100644
--- a/src/southbridge/amd/cimx/sb800/cfg.h
+++ b/src/southbridge/amd/cimx/sb800/cfg.h
@@ -1,11 +1,8 @@
 /* SPDX-License-Identifier: GPL-2.0-only */

-
 #ifndef _SB800_CFG_H_
 #define _SB800_CFG_H_

-#include <stdint.h>
-
 /**
  * @brief South Bridge CIMx configuration
  *