From: Simon Glass <sjg@chromium.org>
To: U-Boot Mailing List <u-boot@lists.denx.de>
Cc: Patrice Chotard <patrice.chotard@foss.st.com>,
Tom Rini <trini@konsulko.com>, Simon Glass <sjg@chromium.org>
Subject: [PATCH v2 09/18] pxe: Tidy up code style a little in pxe_utils
Date: Mon, 27 Sep 2021 09:23:51 -0600 [thread overview]
Message-ID: <20210927092350.v2.9.Ib56ccfab71a5b43d9d35c5d9bd88813e6dfd9ffb@changeid> (raw)
In-Reply-To: <20210927152400.433569-1-sjg@chromium.org>
There are a few more blank lines than makes sense for readability. Also
free() handles a NULL pointer so drop the pointless checks.
Signed-off-by: Simon Glass <sjg@chromium.org>
---
(no changes since v1)
boot/pxe_utils.c | 66 ++++++++++--------------------------------------
1 file changed, 13 insertions(+), 53 deletions(-)
diff --git a/boot/pxe_utils.c b/boot/pxe_utils.c
index 7a2213a5925..9f3edeab06a 100644
--- a/boot/pxe_utils.c
+++ b/boot/pxe_utils.c
@@ -51,7 +51,6 @@ int format_mac_pxe(char *outbuf, size_t outbuf_len)
if (outbuf_len < 21) {
printf("outbuf is too small (%zd < 21)\n", outbuf_len);
-
return -ENOSPC;
}
@@ -91,12 +90,10 @@ static int get_bootfile_path(const char *file_path, char *bootfile_path,
goto ret;
bootfile = from_env("bootfile");
-
if (!bootfile)
goto ret;
last_slash = strrchr(bootfile, '/');
-
if (!last_slash)
goto ret;
@@ -140,7 +137,6 @@ static int get_relfile(struct pxe_context *ctx, const char *file_path,
err = get_bootfile_path(file_path, relfile, sizeof(relfile),
ctx->allow_abs_path);
-
if (err < 0)
return err;
@@ -181,7 +177,6 @@ int get_pxe_file(struct pxe_context *ctx, const char *file_path,
char *buf;
err = get_relfile(ctx, file_path, file_addr);
-
if (err < 0)
return err;
@@ -190,7 +185,6 @@ int get_pxe_file(struct pxe_context *ctx, const char *file_path,
* and add the NUL byte.
*/
tftp_filesize = from_env("filesize");
-
if (!tftp_filesize)
return -ENOENT;
@@ -253,7 +247,6 @@ static int get_relfile_envaddr(struct pxe_context *ctx, const char *file_path,
char *envaddr;
envaddr = from_env(envaddr_name);
-
if (!envaddr)
return -ENOENT;
@@ -276,7 +269,6 @@ static struct pxe_label *label_create(void)
struct pxe_label *label;
label = malloc(sizeof(struct pxe_label));
-
if (!label)
return NULL;
@@ -300,30 +292,14 @@ static struct pxe_label *label_create(void)
*/
static void label_destroy(struct pxe_label *label)
{
- if (label->name)
- free(label->name);
-
- if (label->kernel)
- free(label->kernel);
-
- if (label->config)
- free(label->config);
-
- if (label->append)
- free(label->append);
-
- if (label->initrd)
- free(label->initrd);
-
- if (label->fdt)
- free(label->fdt);
-
- if (label->fdtdir)
- free(label->fdtdir);
-
- if (label->fdtoverlays)
- free(label->fdtoverlays);
-
+ free(label->name);
+ free(label->kernel);
+ free(label->config);
+ free(label->append);
+ free(label->initrd);
+ free(label->fdt);
+ free(label->fdtdir);
+ free(label->fdtoverlays);
free(label);
}
@@ -359,7 +335,6 @@ static int label_localboot(struct pxe_label *label)
char *localcmd;
localcmd = from_env("localcmd");
-
if (!localcmd)
return -ENOENT;
@@ -718,8 +693,8 @@ static int label_boot(struct pxe_context *ctx, struct pxe_label *label)
unmap_sysmem(buf);
cleanup:
- if (fit_addr)
- free(fit_addr);
+ free(fit_addr);
+
return 1;
}
@@ -832,7 +807,6 @@ static char *get_string(char **p, struct token *t, char delim, int lower)
*/
b = *p;
e = *p;
-
while (*e) {
if ((delim == ' ' && isspace(*e)) || delim == *e)
break;
@@ -858,11 +832,8 @@ static char *get_string(char **p, struct token *t, char delim, int lower)
t->val[len] = '\0';
- /*
- * Update *p so the caller knows where to continue scanning.
- */
+ /* Update *p so the caller knows where to continue scanning */
*p = e;
-
t->type = T_STRING;
return t->val;
@@ -988,7 +959,6 @@ static int parse_integer(char **c, int *dst)
char *s = *c;
get_token(c, &t, L_SLITERAL);
-
if (t.type != T_STRING) {
printf("Expected string: %.*s\n", (int)(*c - s), s);
return -EINVAL;
@@ -1022,14 +992,12 @@ static int handle_include(struct pxe_context *ctx, char **c, unsigned long base,
int ret;
err = parse_sliteral(c, &include_path);
-
if (err < 0) {
printf("Expected include path: %.*s\n", (int)(*c - s), s);
return err;
}
err = get_pxe_file(ctx, include_path, base);
-
if (err < 0) {
printf("Couldn't retrieve %s\n", include_path);
return err;
@@ -1079,7 +1047,6 @@ static int parse_menu(struct pxe_context *ctx, char **c, struct pxe_menu *cfg,
printf("Ignoring malformed menu command: %.*s\n",
(int)(*c - s), s);
}
-
if (err < 0)
return err;
@@ -1353,11 +1320,8 @@ void destroy_pxe_menu(struct pxe_menu *cfg)
struct list_head *pos, *n;
struct pxe_label *label;
- if (cfg->title)
- free(cfg->title);
-
- if (cfg->default_label)
- free(cfg->default_label);
+ free(cfg->title);
+ free(cfg->default_label);
list_for_each_safe(pos, n, &cfg->labels) {
label = list_entry(pos, struct pxe_label, list);
@@ -1375,7 +1339,6 @@ struct pxe_menu *parse_pxefile(struct pxe_context *ctx, unsigned long menucfg)
int r;
cfg = malloc(sizeof(struct pxe_menu));
-
if (!cfg)
return NULL;
@@ -1386,7 +1349,6 @@ struct pxe_menu *parse_pxefile(struct pxe_context *ctx, unsigned long menucfg)
buf = map_sysmem(menucfg, 0);
r = parse_pxefile_top(ctx, buf, menucfg, cfg, 1);
unmap_sysmem(buf);
-
if (r < 0) {
destroy_pxe_menu(cfg);
return NULL;
@@ -1413,7 +1375,6 @@ static struct menu *pxe_menu_to_menu(struct pxe_menu *cfg)
*/
m = menu_create(cfg->title, DIV_ROUND_UP(cfg->timeout, 10),
cfg->prompt, NULL, label_print, NULL, NULL);
-
if (!m)
return NULL;
@@ -1492,7 +1453,6 @@ void handle_pxe_menu(struct pxe_context *ctx, struct pxe_menu *cfg)
return;
err = menu_get_choice(m, &choice);
-
menu_destroy(m);
/*
--
2.33.0.685.g46640cef36-goog
next prev parent reply other threads:[~2021-09-27 15:28 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-09-27 15:23 [PATCH v2 00/18] pxe: Refactoring to tidy up and prepare for bootflow Simon Glass
2021-09-27 15:23 ` [PATCH v2 01/18] Create a new boot/ directory Simon Glass
2021-09-27 15:23 ` [PATCH v2 02/18] pxe: Move API comments to the header files Simon Glass
2021-09-27 15:23 ` [PATCH v2 03/18] pxe: Use a context pointer Simon Glass
2021-09-27 15:23 ` [PATCH v2 04/18] pxe: Move do_getfile() into the context Simon Glass
2021-09-27 15:23 ` [PATCH v2 05/18] pxe: Add a userdata field to " Simon Glass
2021-09-27 15:23 ` [PATCH v2 06/18] pxe: Tidy up the is_pxe global Simon Glass
2021-09-27 15:23 ` [PATCH v2 07/18] pxe: Move pxe_utils files Simon Glass
2021-09-27 15:23 ` [PATCH v2 08/18] pxe: Tidy up some comments in pxe_utils Simon Glass
2021-09-27 15:23 ` Simon Glass [this message]
2021-09-27 15:23 ` [PATCH v2 10/18] pxe: Move common parsing coding into pxe_util Simon Glass
2021-09-27 15:23 ` [PATCH v2 11/18] pxe: Clean up the use of bootfile Simon Glass
2021-09-27 15:23 ` [PATCH v2 12/18] pxe: Drop get_bootfile_path() Simon Glass
2021-09-27 15:23 ` [PATCH v2 13/18] lib: Add tests for simple_itoa() Simon Glass
2021-09-27 15:23 ` [PATCH v2 14/18] lib: Add a function to convert a string to a hex value Simon Glass
2021-09-27 15:23 ` [PATCH v2 15/18] pxe: Return the file size from the getfile() function Simon Glass
2021-09-27 15:23 ` [PATCH v2 16/18] pxe: Refactor sysboot to have one helper Simon Glass
2021-09-27 15:23 ` [PATCH v2 17/18] doc: Move distro boot doc to rST Simon Glass
2021-09-27 15:24 ` [PATCH v2 18/18] pxe: Allow calling the pxe_get logic directly Simon Glass
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20210927092350.v2.9.Ib56ccfab71a5b43d9d35c5d9bd88813e6dfd9ffb@changeid \
--to=sjg@chromium.org \
--cc=patrice.chotard@foss.st.com \
--cc=trini@konsulko.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).