arm: socfpga: fpga: fix type of local variable
authorSimon Goldschmidt <simon.k.r.goldschmidt@gmail.com>
Mon, 15 Oct 2018 18:35:10 +0000 (20:35 +0200)
committerMarek Vasut <marex@denx.de>
Wed, 31 Oct 2018 00:41:10 +0000 (01:41 +0100)
The 'status' variable in 'socfpga_load()' for both gen5 and arria10
is of type 'unsigned long' while it is always used as 'int' only.
Change it to 'int'.

Signed-off-by: Simon Goldschmidt <simon.k.r.goldschmidt@gmail.com>
drivers/fpga/socfpga_arria10.c
drivers/fpga/socfpga_gen5.c

index d6b5949..114dd91 100644 (file)
@@ -453,7 +453,7 @@ int fpgamgr_program_finish(void)
  */
 int socfpga_load(Altera_desc *desc, const void *rbf_data, size_t rbf_size)
 {
-       unsigned long status;
+       int status;
 
        /* disable all signals from hps peripheral controller to fpga */
        writel(0, &system_manager_base->fpgaintf_en_global);
index 184de74..6d16e0b 100644 (file)
@@ -204,7 +204,7 @@ static int fpgamgr_program_poll_usermode(void)
  */
 int socfpga_load(Altera_desc *desc, const void *rbf_data, size_t rbf_size)
 {
-       unsigned long status;
+       int status;
 
        if ((uint32_t)rbf_data & 0x3) {
                puts("FPGA: Unaligned data, realign to 32bit boundary.\n");