diff mbox

[U-Boot] fpga: Use enum for bitstream command types

Message ID 60a1e3df2695f775c63bcd0650671ea9a205fa89.1483950073.git.michal.simek@xilinx.com
State Accepted
Commit 5cf22289aeaf0c84dcb55c629ae4ed72112024cc
Delegated to: Michal Simek
Headers show

Commit Message

Michal Simek Jan. 9, 2017, 8:21 a.m. UTC
Using enum simplify handling of different bitstream command
types.

Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---

 cmd/fpga.c | 20 +++++++++++---------
 1 file changed, 11 insertions(+), 9 deletions(-)

Comments

Simon Glass Jan. 19, 2017, 1:57 p.m. UTC | #1
On 9 January 2017 at 01:21, Michal Simek <michal.simek@xilinx.com> wrote:
> Using enum simplify handling of different bitstream command
> types.
>
> Signed-off-by: Michal Simek <michal.simek@xilinx.com>
> ---
>
>  cmd/fpga.c | 20 +++++++++++---------
>  1 file changed, 11 insertions(+), 9 deletions(-)

Reviewed-by: Simon Glass <sjg@chromium.org>
diff mbox

Patch

diff --git a/cmd/fpga.c b/cmd/fpga.c
index 8956eb1b654a..016349f56086 100644
--- a/cmd/fpga.c
+++ b/cmd/fpga.c
@@ -18,15 +18,17 @@ 
 static int fpga_get_op(char *opstr);
 
 /* Local defines */
-#define FPGA_NONE   -1
-#define FPGA_INFO   0
-#define FPGA_LOAD   1
-#define FPGA_LOADB  2
-#define FPGA_DUMP   3
-#define FPGA_LOADMK 4
-#define FPGA_LOADP  5
-#define FPGA_LOADBP 6
-#define FPGA_LOADFS 7
+enum {
+	FPGA_NONE = -1,
+	FPGA_INFO,
+	FPGA_LOAD,
+	FPGA_LOADB,
+	FPGA_DUMP,
+	FPGA_LOADMK,
+	FPGA_LOADP,
+	FPGA_LOADBP,
+	FPGA_LOADFS,
+};
 
 /* ------------------------------------------------------------------------- */
 /* command form: