OSDN Git Service

Staging: bcm: Fixed indention for inner if-block
authorMatthias Beyer <mail@beyermatthias.de>
Fri, 27 Jun 2014 09:55:14 +0000 (11:55 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 29 Jun 2014 21:29:41 +0000 (14:29 -0700)
The inner if-statement was aligned just like the outer one. Why?

This indention was introduced by

    f34c488c3894968e8cdbdc3b1ed617d78315cace

which is a indention-fix patch itself. That's why I'm curious about it.

I did not merge these nested if-statements, as I don't know if I'm
destroying logical seperated checks with it.

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/bcm/DDRInit.c

index 4c7f518..ac45ead 100644 (file)
@@ -1216,11 +1216,11 @@ int download_ddr_settings(struct bcm_mini_adapter *Adapter)
                if (!retval) {
                        if (bOverrideSelfRefresh && (psDDRSetting->ulRegAddress == 0x0F007018)) {
                                value = (psDDRSetting->ulRegValue | (1<<8));
-                       if (STATUS_SUCCESS != wrmalt(Adapter, ul_ddr_setting_load_addr,
-                                       &value, sizeof(value))) {
-                               BCM_DEBUG_PRINT(Adapter, DBG_TYPE_PRINTK, 0, 0, "%s:%d\n", __func__, __LINE__);
-                               break;
-                       }
+                               if (STATUS_SUCCESS != wrmalt(Adapter, ul_ddr_setting_load_addr,
+                                               &value, sizeof(value))) {
+                                       BCM_DEBUG_PRINT(Adapter, DBG_TYPE_PRINTK, 0, 0, "%s:%d\n", __func__, __LINE__);
+                                       break;
+                               }
                        } else {
                                value = psDDRSetting->ulRegValue;