diff options
| author | Dan Handley <dan.handley@arm.com> | 2014-04-09 13:14:54 +0100 |
|---|---|---|
| committer | Dan Handley <dan.handley@arm.com> | 2014-05-06 13:57:48 +0100 |
| commit | 97043ac98e13a726dbf8b3b41654dca759e3da2c (patch) | |
| tree | 18133148dea88d9a2313113111b24b56e8130505 /drivers/io | |
| parent | fb037bfb7cbf7b404c069b4ebac5a10059d948b1 (diff) | |
| download | platform_external_arm-trusted-firmware-97043ac98e13a726dbf8b3b41654dca759e3da2c.tar.gz platform_external_arm-trusted-firmware-97043ac98e13a726dbf8b3b41654dca759e3da2c.tar.bz2 platform_external_arm-trusted-firmware-97043ac98e13a726dbf8b3b41654dca759e3da2c.zip | |
Reduce deep nesting of header files
Reduce the number of header files included from other header
files as much as possible without splitting the files. Use forward
declarations where possible. This allows removal of some unnecessary
"#ifndef __ASSEMBLY__" statements.
Also, review the .c and .S files for which header files really need
including and reorder the #include statements alphabetically.
Fixes ARM-software/tf-issues#31
Change-Id: Iec92fb976334c77453e010b60bcf56f3be72bd3e
Diffstat (limited to 'drivers/io')
| -rw-r--r-- | drivers/io/io_fip.c | 14 | ||||
| -rw-r--r-- | drivers/io/io_memmap.c | 6 | ||||
| -rw-r--r-- | drivers/io/io_semihosting.c | 2 |
3 files changed, 11 insertions, 11 deletions
diff --git a/drivers/io/io_fip.c b/drivers/io/io_fip.c index 482e06848..b2dcfccfd 100644 --- a/drivers/io/io_fip.c +++ b/drivers/io/io_fip.c @@ -28,17 +28,17 @@ * POSSIBILITY OF SUCH DAMAGE. */ -#include <stdint.h> -#include <uuid.h> -#include <errno.h> -#include <string.h> #include <assert.h> -#include <platform.h> +#include <debug.h> +#include <errno.h> #include <firmware_image_package.h> -#include <io_storage.h> #include <io_driver.h> #include <io_fip.h> -#include <debug.h> +#include <io_storage.h> +#include <platform.h> +#include <stdint.h> +#include <string.h> +#include <uuid.h> /* Useful for printing UUIDs when debugging.*/ #define PRINT_UUID2(x) \ diff --git a/drivers/io/io_memmap.c b/drivers/io/io_memmap.c index 2e70e9699..a40e61207 100644 --- a/drivers/io/io_memmap.c +++ b/drivers/io/io_memmap.c @@ -29,10 +29,10 @@ */ #include <assert.h> -#include <string.h> -#include <io_storage.h> -#include <io_driver.h> #include <debug.h> +#include <io_driver.h> +#include <io_storage.h> +#include <string.h> /* As we need to be able to keep state for seek, only one file can be open * at a time. Make this a structure and point to the entity->info. When we diff --git a/drivers/io/io_semihosting.c b/drivers/io/io_semihosting.c index 89c844b2b..a37693df6 100644 --- a/drivers/io/io_semihosting.c +++ b/drivers/io/io_semihosting.c @@ -29,8 +29,8 @@ */ #include <assert.h> -#include <io_storage.h> #include <io_driver.h> +#include <io_storage.h> #include <semihosting.h> |