linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 02/13] staging: vt6656: clean up MACvSetBBType
@ 2013-06-05 20:15 Malcolm Priestley
  2013-06-05 22:45 ` Greg KH
  0 siblings, 1 reply; 2+ messages in thread
From: Malcolm Priestley @ 2013-06-05 20:15 UTC (permalink / raw)
  To: gregkh; +Cc: linux-wireless

Remove internal white space and camel case.

Camel case changes;
pDevice -> priv
pbyData -> data
byType -> type

Functional change
CONTROLnsRequestOut Length = ARRAY_SIZE(data)

Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
---
 drivers/staging/vt6656/mac.c | 18 +++++++-----------
 1 file changed, 7 insertions(+), 11 deletions(-)

diff --git a/drivers/staging/vt6656/mac.c b/drivers/staging/vt6656/mac.c
index 89d700f..0f8132b8 100644
--- a/drivers/staging/vt6656/mac.c
+++ b/drivers/staging/vt6656/mac.c
@@ -84,20 +84,16 @@ void MACbShutdown(struct vnt_private *pDevice)
                         );
 }
 
-void MACvSetBBType(struct vnt_private *pDevice, u8 byType)
+void MACvSetBBType(struct vnt_private *priv, u8 type)
 {
-	u8 pbyData[2];
+	u8 data[2];
 
-    pbyData[0] = byType;
-    pbyData[1] = EnCFG_BBType_MASK;
+	data[0] = type;
+	data[1] = EnCFG_BBType_MASK;
 
-    CONTROLnsRequestOut(pDevice,
-                        MESSAGE_TYPE_WRITE_MASK,
-                        MAC_REG_ENCFG0,
-                        MESSAGE_REQUEST_MACREG,
-                        2,
-                        pbyData
-                        );
+	CONTROLnsRequestOut(priv, MESSAGE_TYPE_WRITE_MASK,
+		MAC_REG_ENCFG0, MESSAGE_REQUEST_MACREG,
+		ARRAY_SIZE(data), data);
 }
 
 /*
-- 
1.8.1.2




^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH 02/13] staging: vt6656: clean up MACvSetBBType
  2013-06-05 20:15 [PATCH 02/13] staging: vt6656: clean up MACvSetBBType Malcolm Priestley
@ 2013-06-05 22:45 ` Greg KH
  0 siblings, 0 replies; 2+ messages in thread
From: Greg KH @ 2013-06-05 22:45 UTC (permalink / raw)
  To: Malcolm Priestley; +Cc: linux-wireless

On Wed, Jun 05, 2013 at 09:15:12PM +0100, Malcolm Priestley wrote:
> Remove internal white space and camel case.
> 
> Camel case changes;
> pDevice -> priv
> pbyData -> data
> byType -> type
> 
> Functional change
> CONTROLnsRequestOut Length = ARRAY_SIZE(data)

Don't do more than one thing in a patch, don't mix coding style cleanups
with function changes please.

Please redo this whole series, odds are, you did the same thing with all
of these, right?

thanks,

greg k-h

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2013-06-05 22:45 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-06-05 20:15 [PATCH 02/13] staging: vt6656: clean up MACvSetBBType Malcolm Priestley
2013-06-05 22:45 ` Greg KH

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).