Skip to content
Snippets Groups Projects
Commit 6cd68c81 authored by Michal Simek's avatar Michal Simek
Browse files

fpga: xilinx: Check if fpga operations are defined


Ensure that operations are correctly setup.

Signed-off-by: default avatarMichal Simek <michal.simek@xilinx.com>
Reviewed-by: default avatarSimon Glass <sjg@chromium.org>
parent 345f9e19
No related branches found
No related tags found
No related merge requests found
...@@ -139,6 +139,11 @@ int xilinx_load(xilinx_desc *desc, const void *buf, size_t bsize, ...@@ -139,6 +139,11 @@ int xilinx_load(xilinx_desc *desc, const void *buf, size_t bsize,
return FPGA_FAIL; return FPGA_FAIL;
} }
if (!desc->operations || !desc->operations->load) {
printf("%s: Missing load operation\n", __func__);
return FPGA_FAIL;
}
return desc->operations->load(desc, buf, bsize, bstype); return desc->operations->load(desc, buf, bsize, bstype);
} }
...@@ -151,8 +156,10 @@ int xilinx_loadfs(xilinx_desc *desc, const void *buf, size_t bsize, ...@@ -151,8 +156,10 @@ int xilinx_loadfs(xilinx_desc *desc, const void *buf, size_t bsize,
return FPGA_FAIL; return FPGA_FAIL;
} }
if (!desc->operations->loadfs) if (!desc->operations || !desc->operations->loadfs) {
printf("%s: Missing loadfs operation\n", __func__);
return FPGA_FAIL; return FPGA_FAIL;
}
return desc->operations->loadfs(desc, buf, bsize, fpga_fsinfo); return desc->operations->loadfs(desc, buf, bsize, fpga_fsinfo);
} }
...@@ -165,6 +172,11 @@ int xilinx_dump(xilinx_desc *desc, const void *buf, size_t bsize) ...@@ -165,6 +172,11 @@ int xilinx_dump(xilinx_desc *desc, const void *buf, size_t bsize)
return FPGA_FAIL; return FPGA_FAIL;
} }
if (!desc->operations || !desc->operations->dump) {
printf("%s: Missing dump operation\n", __func__);
return FPGA_FAIL;
}
return desc->operations->dump(desc, buf, bsize); return desc->operations->dump(desc, buf, bsize);
} }
...@@ -228,7 +240,8 @@ int xilinx_info(xilinx_desc *desc) ...@@ -228,7 +240,8 @@ int xilinx_info(xilinx_desc *desc)
if (desc->iface_fns) { if (desc->iface_fns) {
printf ("Device Function Table @ 0x%p\n", desc->iface_fns); printf ("Device Function Table @ 0x%p\n", desc->iface_fns);
desc->operations->info(desc); if (desc->operations && desc->operations->info)
desc->operations->info(desc);
} else } else
printf ("No Device Function Table.\n"); printf ("No Device Function Table.\n");
......
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