Skip to content
Snippets Groups Projects
Commit 021f6df6 authored by Anton Vorontsov's avatar Anton Vorontsov Committed by Kim Phillips
Browse files

83xx: mpc8315erdb: fix silly thinko in fdt_tsec1_fixup


The thinko was quite silly indeed, I messed with !ptr. Normally this
would trigger some fault, but in U-Boot NULL pointer is equal to phys
0, so the code was working still, just didn't actually test mpc8315erdb
environment variable value. Heh.

Signed-off-by: default avatarAnton Vorontsov <avorontsov@ru.mvista.com>
Signed-off-by: default avatarKim Phillips <kim.phillips@freescale.com>
parent 25f5f0d4
No related branches found
No related tags found
No related merge requests found
...@@ -128,15 +128,16 @@ void fdt_tsec1_fixup(void *fdt, bd_t *bd) ...@@ -128,15 +128,16 @@ void fdt_tsec1_fixup(void *fdt, bd_t *bd)
const char *path; const char *path;
int ret; int ret;
if (!mpc8315erdb) { if (!mpc8315erdb)
if (!strcmp(mpc8315erdb, "tsec1")) { return;
return;
} else if (strcmp(mpc8315erdb, "ulpi")) { if (!strcmp(mpc8315erdb, "tsec1")) {
printf("WARNING: wrong `mpc8315erdb' environment " return;
"variable specified: `%s'. Should be `ulpi' " } else if (strcmp(mpc8315erdb, "ulpi")) {
"or `tsec1'.\n", mpc8315erdb); printf("WARNING: wrong `mpc8315erdb' environment "
return; "variable specified: `%s'. Should be `ulpi' "
} "or `tsec1'.\n", mpc8315erdb);
return;
} }
ret = fdt_path_offset(fdt, "/aliases"); ret = fdt_path_offset(fdt, "/aliases");
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment