Calling 'mdio read ...' currently leads to a data abort when no mdio bus is found.
To fix this, check if 'bus' is a valid pointer before accessing it. Signed-off-by: Simon Goldschmidt <simon.k.r.goldschm...@gmail.com> --- cmd/mdio.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/cmd/mdio.c b/cmd/mdio.c index 5e219f699d..e1f771d3ed 100644 --- a/cmd/mdio.c +++ b/cmd/mdio.c @@ -263,6 +263,11 @@ static int do_mdio(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) break; } + if (!bus) { + puts("No MDIO bus found\n"); + return CMD_RET_FAILURE; + } + if (op[0] == 'l') { mdio_list_devices(); -- 2.20.1 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot