diff options
| author | Michael Tretter <m.tretter@pengutronix.de> | 2021-08-19 10:12:49 +0200 |
|---|---|---|
| committer | Sascha Hauer <s.hauer@pengutronix.de> | 2021-08-23 15:51:36 +0200 |
| commit | 3b1ac08954d4518e019331fb34ba9b0677055145 (patch) | |
| tree | 5e7f91485852a0bf10653de90d9f3ecc6809a6f3 | |
| parent | c53e1fc545e686e1f48c8efb9057fc72e158f183 (diff) | |
| download | barebox-3b1ac08954d4518e019331fb34ba9b0677055145.tar.gz barebox-3b1ac08954d4518e019331fb34ba9b0677055145.tar.xz | |
firmware: zynqmp-fpga: initialize flags at function start
The ZYNQMP_FPGA_BIT_ONLY_BIN flag is always set when programming the
FPGA. Simplify the code by initializing the flags with
ZYNQMP_FPGA_BIT_ONLY_BIN already set.
Signed-off-by: Michael Tretter <m.tretter@pengutronix.de>
Link: https://lore.barebox.org/20210819081251.726840-2-m.tretter@pengutronix.de
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
| -rw-r--r-- | drivers/firmware/zynqmp-fpga.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/firmware/zynqmp-fpga.c b/drivers/firmware/zynqmp-fpga.c index 0fc229bfd3..736d1950fa 100644 --- a/drivers/firmware/zynqmp-fpga.c +++ b/drivers/firmware/zynqmp-fpga.c @@ -205,7 +205,7 @@ static int fpgamgr_program_finish(struct firmware_handler *fh) enum xilinx_byte_order byte_order; u64 addr; int status = 0; - u8 flags = 0; + u8 flags = ZYNQMP_FPGA_BIT_ONLY_BIN; if (!mgr->buf) { status = -ENOBUFS; @@ -259,9 +259,6 @@ static int fpgamgr_program_finish(struct firmware_handler *fh) addr = (u64)buf_aligned; - /* we do not provide a header */ - flags |= ZYNQMP_FPGA_BIT_ONLY_BIN; - if (!(mgr->features & ZYNQMP_PM_FEATURE_SIZE_NOT_NEEDED) && buf_size) { status = mgr->eemi_ops->fpga_load(addr, (u32)(uintptr_t)buf_size, |