diff options
| author | David Gibson <david@gibson.dropbear.id.au> | 2008-02-18 18:06:31 +1100 |
|---|---|---|
| committer | Jon Loeliger <jdl@freescale.com> | 2008-02-18 08:27:13 -0600 |
| commit | 8a4e75049dd0543aab4f286fafa41f0b404c59b4 (patch) | |
| tree | adb8aa080209f68b9af42b9a84117d78528b27fe /libfdt/fdt_rw.c | |
| parent | a90b5905fede1ebd2d6365103cc1d9b78f76217c (diff) | |
| download | dtc-8a4e75049dd0543aab4f286fafa41f0b404c59b4.zip dtc-8a4e75049dd0543aab4f286fafa41f0b404c59b4.tar.gz dtc-8a4e75049dd0543aab4f286fafa41f0b404c59b4.tar.bz2 | |
libfdt: Trivial cleanup for CHECK_HEADER)
Currently the CHECK_HEADER() macro is defined local to fdt_ro.c.
However, there are a handful of functions (fdt_move, rw_check_header,
fdt_open_into) from other files which could also use it (currently
they open-code something more-or-less identical). Therefore, this
patch moves CHECK_HEADER() to libfdt_internal.h and uses it in those
places.
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Diffstat (limited to 'libfdt/fdt_rw.c')
| -rw-r--r-- | libfdt/fdt_rw.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/libfdt/fdt_rw.c b/libfdt/fdt_rw.c index ac50195..0df472b 100644 --- a/libfdt/fdt_rw.c +++ b/libfdt/fdt_rw.c @@ -69,10 +69,8 @@ static int _blocks_misordered(const void *fdt, static int rw_check_header(void *fdt) { - int err; + CHECK_HEADER(fdt); - if ((err = fdt_check_header(fdt))) - return err; if (fdt_version(fdt) < 17) return -FDT_ERR_BADVERSION; if (_blocks_misordered(fdt, sizeof(struct fdt_reserve_entry), @@ -399,9 +397,7 @@ int fdt_open_into(const void *fdt, void *buf, int bufsize) int newsize; void *tmp; - err = fdt_check_header(fdt); - if (err) - return err; + CHECK_HEADER(fdt); mem_rsv_size = (fdt_num_mem_rsv(fdt)+1) * sizeof(struct fdt_reserve_entry); |