* [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).