[PATCH v2 04/20] fip: drop typedefs
Sascha Hauer
s.hauer at pengutronix.de
Wed Feb 12 06:09:17 PST 2025
Do not typedef struct types.
Signed-off-by: Sascha Hauer <s.hauer at pengutronix.de>
---
commands/fiptool.c | 28 ++++++++++++++--------------
include/fiptool.h | 26 +++++++++++++-------------
lib/fip.c | 44 ++++++++++++++++++++++----------------------
3 files changed, 49 insertions(+), 49 deletions(-)
diff --git a/commands/fiptool.c b/commands/fiptool.c
index 2d766f895b..a4370c9eef 100644
--- a/commands/fiptool.c
+++ b/commands/fiptool.c
@@ -31,7 +31,7 @@ typedef struct cmd {
} cmd_t;
-static int write_image_to_file(const image_t *image, const char *filename)
+static int write_image_to_file(const struct fip_image *image, const char *filename)
{
int fd;
@@ -53,7 +53,7 @@ static int write_image_to_file(const image_t *image, const char *filename)
static int info_cmd(struct fip_state *fip, int argc, char *argv[])
{
- image_desc_t *desc;
+ struct fip_image_desc *desc;
fip_toc_header_t toc_header;
int ret;
@@ -74,7 +74,7 @@ static int info_cmd(struct fip_state *fip, int argc, char *argv[])
(unsigned long long)toc_header.flags);
for (desc = fip->image_desc_head; desc != NULL; desc = desc->next) {
- image_t *image = desc->image;
+ struct fip_image *image = desc->image;
if (image == NULL)
continue;
@@ -193,7 +193,7 @@ static __maybe_unused int create_cmd(struct fip_state *fip, int argc, char *argv
while ((opt = getopt(argc, argv, "e:p:a:b:")) > 0) {
switch (opt) {
case 'e': {
- image_desc_t *desc;
+ struct fip_image_desc *desc;
desc = lookup_image_desc_from_opt(fip, &optarg);
if (!desc)
@@ -214,7 +214,7 @@ static __maybe_unused int create_cmd(struct fip_state *fip, int argc, char *argv
char name[UUID_STRING_LEN + 1];
char filename[PATH_MAX] = { 0 };
uuid_t uuid = uuid_null;
- image_desc_t *desc;
+ struct fip_image_desc *desc;
parse_blob_opt(optarg, &uuid,
filename, sizeof(filename));
@@ -264,7 +264,7 @@ static __maybe_unused int update_cmd(struct fip_state *fip, int argc, char *argv
while ((opt = getopt(argc, argv, "e:p:b:a:o:")) > 0) {
switch (opt) {
case 'e': {
- image_desc_t *desc;
+ struct fip_image_desc *desc;
desc = lookup_image_desc_from_opt(fip, &optarg);
if (!desc)
@@ -281,7 +281,7 @@ static __maybe_unused int update_cmd(struct fip_state *fip, int argc, char *argv
char name[UUID_STRING_LEN + 1];
char filename[PATH_MAX] = { 0 };
uuid_t uuid = uuid_null;
- image_desc_t *desc;
+ struct fip_image_desc *desc;
parse_blob_opt(optarg, &uuid,
filename, sizeof(filename));
@@ -339,7 +339,7 @@ static __maybe_unused int update_cmd(struct fip_state *fip, int argc, char *argv
static int unpack_cmd(struct fip_state *fip, int argc, char *argv[])
{
char outdir[PATH_MAX] = { 0 };
- image_desc_t *desc;
+ struct fip_image_desc *desc;
int fflag = 0;
int unpack_all = 1;
int ret, opt;
@@ -350,7 +350,7 @@ static int unpack_cmd(struct fip_state *fip, int argc, char *argv[])
while ((opt = getopt(argc, argv, "e:b:fo:")) > 0) {
switch (opt) {
case 'e': {
- image_desc_t *desc;
+ struct fip_image_desc *desc;
desc = lookup_image_desc_from_opt(fip, &optarg);
if (!desc)
@@ -363,7 +363,7 @@ static int unpack_cmd(struct fip_state *fip, int argc, char *argv[])
char name[UUID_STRING_LEN + 1];
char filename[PATH_MAX] = { 0 };
uuid_t uuid = uuid_null;
- image_desc_t *desc;
+ struct fip_image_desc *desc;
parse_blob_opt(optarg, &uuid,
filename, sizeof(filename));
@@ -411,7 +411,7 @@ static int unpack_cmd(struct fip_state *fip, int argc, char *argv[])
/* Unpack all specified images. */
for (desc = fip->image_desc_head; desc != NULL; desc = desc->next) {
char file[PATH_MAX];
- image_t *image = desc->image;
+ struct fip_image *image = desc->image;
if (!unpack_all && desc->action != DO_UNPACK)
continue;
@@ -447,7 +447,7 @@ static __maybe_unused int remove_cmd(struct fip_state *fip, int argc, char *argv
{
char outfile[PATH_MAX] = { 0 };
fip_toc_header_t toc_header;
- image_desc_t *desc;
+ struct fip_image_desc *desc;
long align = 1;
int ret, opt, fflag = 0;
@@ -457,7 +457,7 @@ static __maybe_unused int remove_cmd(struct fip_state *fip, int argc, char *argv
while ((opt = getopt(argc, argv, "e:a:b:fo:")) > 0) {
switch (opt) {
case 'e': {
- image_desc_t *desc;
+ struct fip_image_desc *desc;
desc = lookup_image_desc_from_opt(fip, &optarg);
if (!desc)
@@ -473,7 +473,7 @@ static __maybe_unused int remove_cmd(struct fip_state *fip, int argc, char *argv
case 'b': {
char name[UUID_STRING_LEN + 1], filename[PATH_MAX];
uuid_t uuid = uuid_null;
- image_desc_t *desc;
+ struct fip_image_desc *desc;
parse_blob_opt(optarg, &uuid,
filename, sizeof(filename));
diff --git a/include/fiptool.h b/include/fiptool.h
index 704845f456..6f1d69d178 100644
--- a/include/fiptool.h
+++ b/include/fiptool.h
@@ -16,23 +16,23 @@ enum {
DO_REMOVE = 3
};
-typedef struct image_desc {
+struct fip_image_desc {
uuid_t uuid;
char *name;
char *cmdline_name;
int action;
char *action_arg;
- struct image *image;
- struct image_desc *next;
-} image_desc_t;
+ struct fip_image *image;
+ struct fip_image_desc *next;
+};
-typedef struct image {
+struct fip_image {
struct fip_toc_entry toc_e;
void *buffer;
-} image_t;
+};
struct fip_state {
- image_desc_t *image_desc_head;
+ struct fip_image_desc *image_desc_head;
size_t nr_image_descs;
int verbose;
};
@@ -45,24 +45,24 @@ struct fip_state {
} \
} while (0)
-image_desc_t *new_image_desc(const uuid_t *uuid,
+struct fip_image_desc *new_image_desc(const uuid_t *uuid,
const char *name, const char *cmdline_name);
-void set_image_desc_action(image_desc_t *desc, int action,
+void set_image_desc_action(struct fip_image_desc *desc, int action,
const char *arg);
-void free_image_desc(image_desc_t *desc);
+void free_image_desc(struct fip_image_desc *desc);
-void add_image_desc(struct fip_state *fip, image_desc_t *desc);
+void add_image_desc(struct fip_state *fip, struct fip_image_desc *desc);
void free_image_descs(struct fip_state *fip);
void fill_image_descs(struct fip_state *fip);
-image_desc_t *lookup_image_desc_from_uuid(struct fip_state *fip,
+struct fip_image_desc *lookup_image_desc_from_uuid(struct fip_state *fip,
const uuid_t *uuid);
-image_desc_t *lookup_image_desc_from_opt(struct fip_state *fip, char **arg);
+struct fip_image_desc *lookup_image_desc_from_opt(struct fip_state *fip, char **arg);
int parse_fip(struct fip_state *fip,
const char *filename, fip_toc_header_t *toc_header_out);
diff --git a/lib/fip.c b/lib/fip.c
index ae7ff8b3ad..e7589ff4c2 100644
--- a/lib/fip.c
+++ b/lib/fip.c
@@ -26,10 +26,10 @@
#include <fip.h>
#include <fiptool.h>
-image_desc_t *new_image_desc(const uuid_t *uuid,
+struct fip_image_desc *new_image_desc(const uuid_t *uuid,
const char *name, const char *cmdline_name)
{
- image_desc_t *desc;
+ struct fip_image_desc *desc;
desc = xzalloc(sizeof(*desc));
memcpy(&desc->uuid, uuid, sizeof(uuid_t));
@@ -39,7 +39,7 @@ image_desc_t *new_image_desc(const uuid_t *uuid,
return desc;
}
-void set_image_desc_action(image_desc_t *desc, int action,
+void set_image_desc_action(struct fip_image_desc *desc, int action,
const char *arg)
{
ASSERT(desc != NULL);
@@ -52,7 +52,7 @@ void set_image_desc_action(image_desc_t *desc, int action,
desc->action_arg = xstrdup(arg);
}
-void free_image_desc(image_desc_t *desc)
+void free_image_desc(struct fip_image_desc *desc)
{
free(desc->name);
free(desc->cmdline_name);
@@ -64,9 +64,9 @@ void free_image_desc(image_desc_t *desc)
free(desc);
}
-void add_image_desc(struct fip_state *fip, image_desc_t *desc)
+void add_image_desc(struct fip_state *fip, struct fip_image_desc *desc)
{
- image_desc_t **p = &fip->image_desc_head;
+ struct fip_image_desc **p = &fip->image_desc_head;
while (*p)
p = &(*p)->next;
@@ -78,7 +78,7 @@ void add_image_desc(struct fip_state *fip, image_desc_t *desc)
void free_image_descs(struct fip_state *fip)
{
- image_desc_t *desc = fip->image_desc_head, *tmp;
+ struct fip_image_desc *desc = fip->image_desc_head, *tmp;
while (desc != NULL) {
tmp = desc->next;
@@ -96,7 +96,7 @@ void fill_image_descs(struct fip_state *fip)
for (toc_entry = toc_entries;
toc_entry->cmdline_name != NULL;
toc_entry++) {
- image_desc_t *desc;
+ struct fip_image_desc *desc;
desc = new_image_desc(&toc_entry->uuid,
toc_entry->name,
@@ -106,7 +106,7 @@ void fill_image_descs(struct fip_state *fip)
for (toc_entry = plat_def_toc_entries;
toc_entry->cmdline_name != NULL;
toc_entry++) {
- image_desc_t *desc;
+ struct fip_image_desc *desc;
desc = new_image_desc(&toc_entry->uuid,
toc_entry->name,
@@ -115,10 +115,10 @@ void fill_image_descs(struct fip_state *fip)
}
}
-image_desc_t *lookup_image_desc_from_uuid(struct fip_state *fip,
+struct fip_image_desc *lookup_image_desc_from_uuid(struct fip_state *fip,
const uuid_t *uuid)
{
- image_desc_t *desc;
+ struct fip_image_desc *desc;
for (desc = fip->image_desc_head; desc != NULL; desc = desc->next)
if (memcmp(&desc->uuid, uuid, sizeof(uuid_t)) == 0)
@@ -126,10 +126,10 @@ image_desc_t *lookup_image_desc_from_uuid(struct fip_state *fip,
return NULL;
}
-image_desc_t *lookup_image_desc_from_opt(struct fip_state *fip, char **arg)
+struct fip_image_desc *lookup_image_desc_from_opt(struct fip_state *fip, char **arg)
{
int len = 0;
- image_desc_t *desc;
+ struct fip_image_desc *desc;
char *eq;
eq = strchrnul(*arg, '=');
@@ -200,8 +200,8 @@ int parse_fip(struct fip_state *fip,
/* Walk through each ToC entry in the file. */
while ((char *)toc_entry + sizeof(*toc_entry) - 1 < bufend) {
- image_t *image;
- image_desc_t *desc;
+ struct fip_image *image;
+ struct fip_image_desc *desc;
/* Found the ToC terminator, we are done. */
if (memcmp(&toc_entry->uuid, &uuid_null, sizeof(uuid_t)) == 0) {
@@ -260,10 +260,10 @@ int parse_fip(struct fip_state *fip,
return 0;
}
-static image_t *read_image_from_file(const uuid_t *uuid, const char *filename)
+static struct fip_image *read_image_from_file(const uuid_t *uuid, const char *filename)
{
struct stat st;
- image_t *image;
+ struct fip_image *image;
int fd;
ASSERT(uuid != NULL);
@@ -298,7 +298,7 @@ int pack_images(struct fip_state *fip,
uint64_t toc_flags, unsigned long align)
{
int fd;
- image_desc_t *desc;
+ struct fip_image_desc *desc;
fip_toc_header_t *toc_header;
fip_toc_entry_t *toc_entry;
char *buf;
@@ -325,7 +325,7 @@ int pack_images(struct fip_state *fip,
entry_offset = buf_size;
for (desc = fip->image_desc_head; desc != NULL; desc = desc->next) {
- image_t *image = desc->image;
+ struct fip_image *image = desc->image;
if (image == NULL || (image->toc_e.size == 0ULL))
continue;
@@ -361,7 +361,7 @@ int pack_images(struct fip_state *fip,
pr_verbose("Payload size: %llu bytes\n", payload_size);
for (desc = fip->image_desc_head; desc != NULL; desc = desc->next) {
- image_t *image = desc->image;
+ struct fip_image *image = desc->image;
if (image == NULL)
continue;
@@ -396,11 +396,11 @@ int pack_images(struct fip_state *fip,
*/
int update_fip(struct fip_state *fip)
{
- image_desc_t *desc;
+ struct fip_image_desc *desc;
/* Add or replace images in the FIP file. */
for (desc = fip->image_desc_head; desc != NULL; desc = desc->next) {
- image_t *image;
+ struct fip_image *image;
if (desc->action != DO_PACK)
continue;
--
2.39.5
More information about the barebox
mailing list