All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] staging: wlan-ng: cfg80211.c: fixed a line over 80 characters long
@ 2017-06-09  1:38 Elias Carter
  2017-06-09  5:41 ` Greg KH
  0 siblings, 1 reply; 4+ messages in thread
From: Elias Carter @ 2017-06-09  1:38 UTC (permalink / raw)
  To: gregkh; +Cc: devel, linux-kernel, edcarter

From: edcarter <edcarter@ualberta.ca>

Fixed a coding style issue where a line was longer than 80 characters.

Signed-off-by: Elias Carter <edcarter@ualberta.ca>
---
 drivers/staging/wlan-ng/cfg80211.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/staging/wlan-ng/cfg80211.c b/drivers/staging/wlan-ng/cfg80211.c
index 178f6f5..e82043e 100644
--- a/drivers/staging/wlan-ng/cfg80211.c
+++ b/drivers/staging/wlan-ng/cfg80211.c
@@ -692,7 +692,8 @@ static const struct cfg80211_ops prism2_usb_cfg_ops = {
 };
 
 /* Functions to create/free wiphy interface */
-static struct wiphy *wlan_create_wiphy(struct device *dev, struct wlandevice *wlandev)
+static struct wiphy *wlan_create_wiphy(struct device *dev,
+				       struct wlandevice *wlandev)
 {
 	struct wiphy *wiphy;
 	struct prism2_wiphy_private *priv;
-- 
2.7.4

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

* Re: [PATCH] staging: wlan-ng: cfg80211.c: fixed a line over 80 characters long
  2017-06-09  1:38 [PATCH] staging: wlan-ng: cfg80211.c: fixed a line over 80 characters long Elias Carter
@ 2017-06-09  5:41 ` Greg KH
  2017-06-09  6:25   ` Elias Carter
  0 siblings, 1 reply; 4+ messages in thread
From: Greg KH @ 2017-06-09  5:41 UTC (permalink / raw)
  To: Elias Carter; +Cc: devel, linux-kernel

On Thu, Jun 08, 2017 at 06:38:30PM -0700, Elias Carter wrote:
> From: edcarter <edcarter@ualberta.ca>
> 
> Fixed a coding style issue where a line was longer than 80 characters.
> 
> Signed-off-by: Elias Carter <edcarter@ualberta.ca>

Your from and signed-off-by names have to match :(

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

* Re: [PATCH] staging: wlan-ng: cfg80211.c: fixed a line over 80 characters long
  2017-06-09  5:41 ` Greg KH
@ 2017-06-09  6:25   ` Elias Carter
  2017-06-09  6:48     ` Greg KH
  0 siblings, 1 reply; 4+ messages in thread
From: Elias Carter @ 2017-06-09  6:25 UTC (permalink / raw)
  To: Greg KH; +Cc: devel, linux-kernel

On Fri, Jun 09, 2017 at 07:41:12AM +0200, Greg KH wrote:
> On Thu, Jun 08, 2017 at 06:38:30PM -0700, Elias Carter wrote:
> > From: edcarter <edcarter@ualberta.ca>
> > 
> > Fixed a coding style issue where a line was longer than 80 characters.
> > 
> > Signed-off-by: Elias Carter <edcarter@ualberta.ca>
> 
> Your from and signed-off-by names have to match :(

Hi Greg, thanks for the feedback.

I fixed the issue but I was wondering if I should include the patch in a
reply to this email or resend a new patch?

I am just trying to find my way around the development process.

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

* Re: [PATCH] staging: wlan-ng: cfg80211.c: fixed a line over 80 characters long
  2017-06-09  6:25   ` Elias Carter
@ 2017-06-09  6:48     ` Greg KH
  0 siblings, 0 replies; 4+ messages in thread
From: Greg KH @ 2017-06-09  6:48 UTC (permalink / raw)
  To: Elias Carter; +Cc: devel, linux-kernel

On Thu, Jun 08, 2017 at 11:25:55PM -0700, Elias Carter wrote:
> On Fri, Jun 09, 2017 at 07:41:12AM +0200, Greg KH wrote:
> > On Thu, Jun 08, 2017 at 06:38:30PM -0700, Elias Carter wrote:
> > > From: edcarter <edcarter@ualberta.ca>
> > > 
> > > Fixed a coding style issue where a line was longer than 80 characters.
> > > 
> > > Signed-off-by: Elias Carter <edcarter@ualberta.ca>
> > 
> > Your from and signed-off-by names have to match :(
> 
> Hi Greg, thanks for the feedback.
> 
> I fixed the issue but I was wondering if I should include the patch in a
> reply to this email or resend a new patch?

Totally new patch.

> I am just trying to find my way around the development process.

You did read Documenation/development_process/ right?

Also, please use scripts/get_maintainer.pl to determine who else to cc:
patches on.

good luck!

greg k-h

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

end of thread, other threads:[~2017-06-09  6:49 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-06-09  1:38 [PATCH] staging: wlan-ng: cfg80211.c: fixed a line over 80 characters long Elias Carter
2017-06-09  5:41 ` Greg KH
2017-06-09  6:25   ` Elias Carter
2017-06-09  6:48     ` Greg KH

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.