linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 2/2] drivers:staging:wlan_ng Fix no space is necessary after a cast
@ 2015-10-27 11:24 Bogicevic Sasa
  2015-10-27 11:35 ` Albino B Neto
  0 siblings, 1 reply; 5+ messages in thread
From: Bogicevic Sasa @ 2015-10-27 11:24 UTC (permalink / raw)
  To: gregkh; +Cc: linux-kernel, Bogicevic Sasa

This fixes "No space is necessary after a cast" messages from
checkpatch.pl

Signed-off-by: Bogicevic Sasa <brutallesale@gmail.com>
---
 drivers/staging/wlan-ng/prism2usb.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/staging/wlan-ng/prism2usb.c b/drivers/staging/wlan-ng/prism2usb.c
index 9167f6e..1485c00 100644
--- a/drivers/staging/wlan-ng/prism2usb.c
+++ b/drivers/staging/wlan-ng/prism2usb.c
@@ -6,7 +6,7 @@
 
 #define PRISM_DEV(vid, pid, name)		\
 	{ USB_DEVICE(vid, pid),			\
-	.driver_info = (unsigned long) name }
+	.driver_info = (unsigned long)name }
 
 static struct usb_device_id usb_prism_tbl[] = {
 	PRISM_DEV(0x04bb, 0x0922, "IOData AirPort WN-B11/USBS"),
@@ -142,7 +142,7 @@ static void prism2sta_disconnect_usb(struct usb_interface *interface)
 {
 	wlandevice_t *wlandev;
 
-	wlandev = (wlandevice_t *) usb_get_intfdata(interface);
+	wlandev = (wlandevice_t *)usb_get_intfdata(interface);
 	if (wlandev != NULL) {
 		LIST_HEAD(cleanlist);
 		struct list_head *entry;
@@ -237,7 +237,7 @@ static int prism2sta_suspend(
 	hfa384x_t *hw = NULL;
 	wlandevice_t *wlandev;
 
-	wlandev = (wlandevice_t *) usb_get_intfdata(interface);
+	wlandev = (wlandevice_t *)usb_get_intfdata(interface);
 	if (!wlandev)
 		return -ENODEV;
 
@@ -260,7 +260,7 @@ static int prism2sta_resume(struct usb_interface *interface)
 	hfa384x_t *hw = NULL;
 	wlandevice_t *wlandev;
 
-	wlandev = (wlandevice_t *) usb_get_intfdata(interface);
+	wlandev = (wlandevice_t *)usb_get_intfdata(interface);
 	if (!wlandev)
 		return -ENODEV;
 
-- 
1.9.1


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

* Re: [PATCH 2/2] drivers:staging:wlan_ng Fix no space is necessary after a cast
  2015-10-27 11:24 [PATCH 2/2] drivers:staging:wlan_ng Fix no space is necessary after a cast Bogicevic Sasa
@ 2015-10-27 11:35 ` Albino B Neto
  2015-10-27 19:41   ` Greg KH
  0 siblings, 1 reply; 5+ messages in thread
From: Albino B Neto @ 2015-10-27 11:35 UTC (permalink / raw)
  To: Bogicevic Sasa; +Cc: Greg KH, LKML

2015-10-27 9:24 GMT-02:00 Bogicevic Sasa <brutallesale@gmail.com>:
> This fixes "No space is necessary after a cast" messages from
> checkpatch.pl
>
> Signed-off-by: Bogicevic Sasa <brutallesale@gmail.com>

Tips for read: https://www.kernel.org/doc/Documentation/CodingStyle

   Albino

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

* Re: [PATCH 2/2] drivers:staging:wlan_ng Fix no space is necessary after a cast
  2015-10-27 11:35 ` Albino B Neto
@ 2015-10-27 19:41   ` Greg KH
  2015-10-27 22:26     ` Bogicevic Sasa
  0 siblings, 1 reply; 5+ messages in thread
From: Greg KH @ 2015-10-27 19:41 UTC (permalink / raw)
  To: Albino B Neto; +Cc: Bogicevic Sasa, LKML

On Tue, Oct 27, 2015 at 09:35:35AM -0200, Albino B Neto wrote:
> 2015-10-27 9:24 GMT-02:00 Bogicevic Sasa <brutallesale@gmail.com>:
> > This fixes "No space is necessary after a cast" messages from
> > checkpatch.pl
> >
> > Signed-off-by: Bogicevic Sasa <brutallesale@gmail.com>
> 
> Tips for read: https://www.kernel.org/doc/Documentation/CodingStyle

What does that mean?  What is wrong with this patch?

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

* Re: [PATCH 2/2] drivers:staging:wlan_ng Fix no space is necessary after a cast
  2015-10-27 19:41   ` Greg KH
@ 2015-10-27 22:26     ` Bogicevic Sasa
  2015-10-28  5:41       ` Sudip Mukherjee
  0 siblings, 1 reply; 5+ messages in thread
From: Bogicevic Sasa @ 2015-10-27 22:26 UTC (permalink / raw)
  To: Greg KH; +Cc: Albino B Neto, LKML

It should be fine, I meant in a sence nothing is accepted from me yet
On 10/28, Greg KH wrote:
>On Tue, Oct 27, 2015 at 09:35:35AM -0200, Albino B Neto wrote:
>> 2015-10-27 9:24 GMT-02:00 Bogicevic Sasa <brutallesale@gmail.com>:
>> > This fixes "No space is necessary after a cast" messages from
>> > checkpatch.pl
>> >
>> > Signed-off-by: Bogicevic Sasa <brutallesale@gmail.com>
>>
>> Tips for read: https://www.kernel.org/doc/Documentation/CodingStyle
>
>What does that mean?  What is wrong with this patch?

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

* Re: [PATCH 2/2] drivers:staging:wlan_ng Fix no space is necessary after a cast
  2015-10-27 22:26     ` Bogicevic Sasa
@ 2015-10-28  5:41       ` Sudip Mukherjee
  0 siblings, 0 replies; 5+ messages in thread
From: Sudip Mukherjee @ 2015-10-28  5:41 UTC (permalink / raw)
  To: Bogicevic Sasa; +Cc: Greg KH, Albino B Neto, LKML

On Tue, Oct 27, 2015 at 11:26:48PM +0100, Bogicevic Sasa wrote:
> It should be fine, I meant in a sence nothing is accepted from me yet

It doesnot generate any warning in my setup.
And please donot top post while replying.

regards
sudip

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

end of thread, other threads:[~2015-10-28  5:41 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-10-27 11:24 [PATCH 2/2] drivers:staging:wlan_ng Fix no space is necessary after a cast Bogicevic Sasa
2015-10-27 11:35 ` Albino B Neto
2015-10-27 19:41   ` Greg KH
2015-10-27 22:26     ` Bogicevic Sasa
2015-10-28  5:41       ` Sudip Mukherjee

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