fpga: altera: More indentation trimdown

Further improve the indentation in the rest of the file, where
the indentation is initially a bit less brutal. There is no
functional change in this patch.

Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Chin Liang See <clsee@altera.com>
Cc: Dinh Nguyen <dinguyen@altera.com>
Cc: Albert Aribaud <albert.u.boot@aribaud.net>
Cc: Tom Rini <trini@ti.com>
Cc: Wolfgang Denk <wd@denx.de>
Cc: Pavel Machek <pavel@denx.de>
Acked-by: Pavel Machek <pavel@denx.de>

diff --git a/drivers/fpga/altera.c b/drivers/fpga/altera.c
index 6394afe095a7ce5994d275969be406b909620033..b0f323fbef37126e5a927f662597cf421fd57504 100644 (file)
--- a/drivers/fpga/altera.c
+++ b/drivers/fpga/altera.c
@@ -26,40 +26,41 @@ int altera_load(Altera_desc *desc, const void *buf, size_t bsize)
 {
        int ret_val = FPGA_FAIL;        /* assume a failure */
 
-       if (!altera_validate (desc, (char *)__func__)) {
+       if (!altera_validate(desc, (char *)__func__)) {
                printf("%s: Invalid device descriptor\n", __func__);
-       } else {
-               switch (desc->family) {
-               case Altera_ACEX1K:
-               case Altera_CYC2:
+               return FPGA_FAIL;
+       }
+
+       switch (desc->family) {
+       case Altera_ACEX1K:
+       case Altera_CYC2:
 #if defined(CONFIG_FPGA_ACEX1K)
-                       debug_cond(FPGA_DEBUG,
-                                  "%s: Launching the ACEX1K Loader...\n",
-                                  __func__);
-                       ret_val = ACEX1K_load (desc, buf, bsize);
+               debug_cond(FPGA_DEBUG,
+                          "%s: Launching the ACEX1K Loader...\n",
+                          __func__);
+               ret_val = ACEX1K_load (desc, buf, bsize);
 #elif defined(CONFIG_FPGA_CYCLON2)
-                       debug_cond(FPGA_DEBUG,
-                                  "%s: Launching the CYCLONE II Loader...\n",
-                                  __func__);
-                       ret_val = CYC2_load (desc, buf, bsize);
+               debug_cond(FPGA_DEBUG,
+                          "%s: Launching the CYCLONE II Loader...\n",
+                          __func__);
+               ret_val = CYC2_load (desc, buf, bsize);
 #else
-                       printf("%s: No support for ACEX1K devices.\n",
-                              __func__);
+               printf("%s: No support for ACEX1K devices.\n",
+                      __func__);
 #endif
-                       break;
+               break;
 
 #if defined(CONFIG_FPGA_STRATIX_II)
-               case Altera_StratixII:
-                       debug_cond(FPGA_DEBUG,
-                                  "%s: Launching the Stratix II Loader...\n",
-                                  __func__);
-                       ret_val = StratixII_load (desc, buf, bsize);
-                       break;
+       case Altera_StratixII:
+               debug_cond(FPGA_DEBUG,
+                          "%s: Launching the Stratix II Loader...\n",
+                          __func__);
+               ret_val = StratixII_load (desc, buf, bsize);
+               break;
 #endif
-               default:
-                       printf("%s: Unsupported family type, %d\n",
-                              __func__, desc->family);
-               }
+       default:
+               printf("%s: Unsupported family type, %d\n",
+                      __func__, desc->family);
        }
 
        return ret_val;
@@ -71,125 +72,126 @@ int altera_dump(Altera_desc *desc, const void *buf, size_t bsize)
 
        if (!altera_validate (desc, (char *)__func__)) {
                printf("%s: Invalid device descriptor\n", __func__);
-       } else {
-               switch (desc->family) {
-               case Altera_ACEX1K:
+               return FPGA_FAIL;
+       }
+
+       switch (desc->family) {
+       case Altera_ACEX1K:
 #if defined(CONFIG_FPGA_ACEX)
-                       debug_cond(FPGA_DEBUG,
-                                  "%s: Launching the ACEX1K Reader...\n",
-                                  __func__);
-                       ret_val = ACEX1K_dump (desc, buf, bsize);
+               debug_cond(FPGA_DEBUG,
+                          "%s: Launching the ACEX1K Reader...\n",
+                          __func__);
+               ret_val = ACEX1K_dump (desc, buf, bsize);
 #else
-                       printf("%s: No support for ACEX1K devices.\n",
-                              __func__);
+               printf("%s: No support for ACEX1K devices.\n",
+                      __func__);
 #endif
-                       break;
+               break;
 
 #if defined(CONFIG_FPGA_STRATIX_II)
-               case Altera_StratixII:
-                       debug_cond(FPGA_DEBUG,
-                                  "%s: Launching the Stratix II Reader...\n",
-                                  __func__);
-                       ret_val = StratixII_dump (desc, buf, bsize);
-                       break;
+       case Altera_StratixII:
+               debug_cond(FPGA_DEBUG,
+                          "%s: Launching the Stratix II Reader...\n",
+                          __func__);
+               ret_val = StratixII_dump (desc, buf, bsize);
+               break;
 #endif
-               default:
-                       printf("%s: Unsupported family type, %d\n",
-                              __func__, desc->family);
-               }
+       default:
+               printf("%s: Unsupported family type, %d\n",
+                      __func__, desc->family);
        }
 
        return ret_val;
 }
 
-int altera_info( Altera_desc *desc )
+int altera_info(Altera_desc *desc)
 {
        int ret_val = FPGA_FAIL;
 
-       if (altera_validate (desc, (char *)__func__)) {
-               printf("Family:        \t");
-               switch (desc->family) {
-               case Altera_ACEX1K:
-                       printf("ACEX1K\n");
-                       break;
-               case Altera_CYC2:
-                       printf("CYCLON II\n");
-                       break;
-               case Altera_StratixII:
-                       printf("Stratix II\n");
-                       break;
-                       /* Add new family types here */
-               default:
-                       printf("Unknown family type, %d\n", desc->family);
-               }
+       if (!altera_validate (desc, (char *)__func__)) {
+               printf("%s: Invalid device descriptor\n", __func__);
+               return FPGA_FAIL;
+       }
 
-               printf("Interface type:\t");
-               switch (desc->iface) {
-               case passive_serial:
-                       printf("Passive Serial (PS)\n");
-                       break;
-               case passive_parallel_synchronous:
-                       printf("Passive Parallel Synchronous (PPS)\n");
-                       break;
-               case passive_parallel_asynchronous:
-                       printf("Passive Parallel Asynchronous (PPA)\n");
-                       break;
-               case passive_serial_asynchronous:
-                       printf("Passive Serial Asynchronous (PSA)\n");
-                       break;
-               case altera_jtag_mode:          /* Not used */
-                       printf("JTAG Mode\n");
-                       break;
-               case fast_passive_parallel:
-                       printf("Fast Passive Parallel (FPP)\n");
-                       break;
-               case fast_passive_parallel_security:
-                       printf
-                           ("Fast Passive Parallel with Security (FPPS) \n");
-                       break;
-                       /* Add new interface types here */
-               default:
-                       printf("Unsupported interface type, %d\n", desc->iface);
-               }
+       printf("Family:        \t");
+       switch (desc->family) {
+       case Altera_ACEX1K:
+               printf("ACEX1K\n");
+               break;
+       case Altera_CYC2:
+               printf("CYCLON II\n");
+               break;
+       case Altera_StratixII:
+               printf("Stratix II\n");
+               break;
+               /* Add new family types here */
+       default:
+               printf("Unknown family type, %d\n", desc->family);
+       }
 
-               printf("Device Size:   \t%zd bytes\n"
-                      "Cookie:        \t0x%x (%d)\n",
-                      desc->size, desc->cookie, desc->cookie);
+       printf("Interface type:\t");
+       switch (desc->iface) {
+       case passive_serial:
+               printf("Passive Serial (PS)\n");
+               break;
+       case passive_parallel_synchronous:
+               printf("Passive Parallel Synchronous (PPS)\n");
+               break;
+       case passive_parallel_asynchronous:
+               printf("Passive Parallel Asynchronous (PPA)\n");
+               break;
+       case passive_serial_asynchronous:
+               printf("Passive Serial Asynchronous (PSA)\n");
+               break;
+       case altera_jtag_mode:          /* Not used */
+               printf("JTAG Mode\n");
+               break;
+       case fast_passive_parallel:
+               printf("Fast Passive Parallel (FPP)\n");
+               break;
+       case fast_passive_parallel_security:
+               printf("Fast Passive Parallel with Security (FPPS)\n");
+               break;
+               /* Add new interface types here */
+       default:
+               printf("Unsupported interface type, %d\n", desc->iface);
+       }
+
+       printf("Device Size:   \t%zd bytes\n"
+              "Cookie:        \t0x%x (%d)\n",
+              desc->size, desc->cookie, desc->cookie);
 
-               if (desc->iface_fns) {
-                       printf("Device Function Table @ 0x%p\n", desc->iface_fns);
-                       switch (desc->family) {
-                       case Altera_ACEX1K:
-                       case Altera_CYC2:
+       if (desc->iface_fns) {
+               printf("Device Function Table @ 0x%p\n", desc->iface_fns);
+               switch (desc->family) {
+               case Altera_ACEX1K:
+               case Altera_CYC2:
 #if defined(CONFIG_FPGA_ACEX1K)
-                               ACEX1K_info(desc);
+                       ACEX1K_info(desc);
 #elif defined(CONFIG_FPGA_CYCLON2)
-                               CYC2_info(desc);
+                       CYC2_info(desc);
 #else
-                               /* just in case */
-                               printf("%s: No support for ACEX1K devices.\n",
-                                               __func__);
+                       /* just in case */
+                       printf("%s: No support for ACEX1K devices.\n",
+                                       __func__);
 #endif
-                               break;
+                       break;
 #if defined(CONFIG_FPGA_STRATIX_II)
-                       case Altera_StratixII:
-                               StratixII_info(desc);
-                               break;
+               case Altera_StratixII:
+                       StratixII_info(desc);
+                       break;
 #endif
-                               /* Add new family types here */
-                       default:
-                               /* we don't need a message here - we give one up above */
-                               break;
-                       }
-               } else {
-                       printf("No Device Function Table.\n");
+                       /* Add new family types here */
+               default:
+                       /* we don't need a message here - we give one up above */
+                       break;
                }
-
-               ret_val = FPGA_SUCCESS;
        } else {
-               printf("%s: Invalid device descriptor\n", __func__);
+               printf("No Device Function Table.\n");
        }
 
+       ret_val = FPGA_SUCCESS;
+
        return ret_val;
 }