Re: [PATCH] Staging: bcm: DDRInit: fixed bracing issues.

2013-12-17 Thread Greg KH 
On Mon, Dec 16, 2013 at 03:51:58AM -0500, Gary Rookard wrote:
 removed braces from single statement conditonals.
 if, else, while opening brace placement was moved to last 
 on first line of each, and closing brace moved to first on
 new line to follow the linux kernel coding style thus removing
 checkpatch errors respectfully.
 
 Signed-off-by: Gary Alan Rookard garyrook...@gmail.com
 ---
 On branch staging-next
  drivers/staging/bcm/DDRInit.c | 112 
 --
  1 file changed, 31 insertions(+), 81 deletions(-)

Does not apply :(

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[PATCH] Staging: bcm: DDRInit: fixed bracing issues.

2013-12-16 Thread Gary Rookard 
removed braces from single statement conditonals.
if, else, while opening brace placement was moved to last 
on first line of each, and closing brace moved to first on
new line to follow the linux kernel coding style thus removing
checkpatch errors respectfully.

Signed-off-by: Gary Alan Rookard garyrook...@gmail.com
---
On branch staging-next
 drivers/staging/bcm/DDRInit.c | 112 --
 1 file changed, 31 insertions(+), 81 deletions(-)

diff --git a/drivers/staging/bcm/DDRInit.c b/drivers/staging/bcm/DDRInit.c
index 41132b5..16a2d3d 100644
--- a/drivers/staging/bcm/DDRInit.c
+++ b/drivers/staging/bcm/DDRInit.c
@@ -739,11 +739,9 @@ int ddr_init(struct bcm_mini_adapter *Adapter)
UINT uiClockSetting = 0;
int retval = STATUS_SUCCESS;
 
-   switch (Adapter-chip_id)
-   {
+   switch (Adapter-chip_id) {
case 0xbece3200:
-   switch (Adapter-DDRSetting)
-   {
+   switch (Adapter-DDRSetting) {
case DDR_80_MHZ:
psDDRSetting = asT3LP_DDRSetting80MHz;
RegCount = (sizeof(asT3LP_DDRSetting80MHz)/
@@ -759,17 +757,13 @@ int ddr_init(struct bcm_mini_adapter *Adapter)
RegCount = (sizeof(asT3LP_DDRSetting133MHz)/
sizeof(struct bcm_ddr_setting));
if (Adapter-bMipsConfig == MIPS_200_MHZ)
-   {
uiClockSetting = 0x03F13652;
-   }
else
-   {
uiClockSetting = 0x03F1365B;
-   }
break;
default:
return -EINVAL;
-   }
+   }
 
break;
case T3LPB:
@@ -782,22 +776,20 @@ int ddr_init(struct bcm_mini_adapter *Adapter)
 */
if ((Adapter-chip_id !=  BCS220_2) 
(Adapter-chip_id !=  BCS220_2BC) 
-   (Adapter-chip_id != BCS220_3))
-   {
+   (Adapter-chip_id != BCS220_3)) {
retval = rdmalt(Adapter,(UINT)0x0f000830, uiResetValue, 
sizeof(uiResetValue));
if (retval  0) {
BCM_DEBUG_PRINT(Adapter, CMHOST, RDM, DBG_LVL_ALL, 
%s:%d RDM failed\n, __func__, __LINE__);
return retval;
-   }
+   }
uiResetValue |= 0x44;
retval = wrmalt(Adapter,(UINT)0x0f000830, uiResetValue, 
sizeof(uiResetValue));
if (retval  0) {
BCM_DEBUG_PRINT(Adapter, CMHOST, RDM, DBG_LVL_ALL, 
%s:%d RDM failed\n, __func__, __LINE__);
return retval;
-   }
}
-   switch (Adapter-DDRSetting)
-   {
+   }
+   switch (Adapter-DDRSetting) {
 
 
 
@@ -817,13 +809,9 @@ int ddr_init(struct bcm_mini_adapter *Adapter)
sizeof(struct bcm_ddr_setting));
 
if (Adapter-bMipsConfig == MIPS_200_MHZ)
-   {
uiClockSetting = 0x03F13652;
-   }
else
-   {
uiClockSetting = 0x03F1365B;
-   }
break;
 
case DDR_160_MHZ:
@@ -831,14 +819,10 @@ int ddr_init(struct bcm_mini_adapter *Adapter)
RegCount = 
sizeof(asT3LPB_DDRSetting160MHz)/sizeof(struct bcm_ddr_setting);
 
if (Adapter-bMipsConfig == MIPS_200_MHZ)
-   {
uiClockSetting = 0x03F137D2;
-   }
else
-   {
uiClockSetting = 0x03F137DB;
-   }
-   }
+   }
break;
 
case 0xbece0110:
@@ -847,8 +831,7 @@ int ddr_init(struct bcm_mini_adapter *Adapter)
case 0xbece0130:
case 0xbece0300:
BCM_DEBUG_PRINT(Adapter,DBG_TYPE_INITEXIT, DRV_ENTRY, DBG_LVL_ALL, DDR 
Setting: %x\n, Adapter-DDRSetting);
-   switch (Adapter-DDRSetting)
-   {
+   switch (Adapter-DDRSetting) {
case DDR_80_MHZ:
psDDRSetting = asT3_DDRSetting80MHz;
RegCount = (sizeof(asT3_DDRSetting80MHz)/
@@ -868,9 +851,7 @@ int ddr_init(struct bcm_mini_adapter *Adapter)
return -EINVAL;
}
case 0xbece0310:
-   {
-   switch (Adapter-DDRSetting)
-   {
+   switch