All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] Staging: asus_oled: asus_oled.c removed an unnessecary space, making it checkpatch clean
@ 2010-04-09 13:01 Gunnar Gudvardarson
  2010-04-09 17:22 ` Greg KH
  0 siblings, 1 reply; 2+ messages in thread
From: Gunnar Gudvardarson @ 2010-04-09 13:01 UTC (permalink / raw)
  To: gregkh; +Cc: devel, linux-kernel, Gunnar Gudvardarson

This is the first linux kernel patch i have ever made, i don't even
know the C programming language except reading the syntax (basic)

It didn't compile on my machine due to invalid configuration of something
on my machine, because i have not yet learned how to configure the compiler.

Signed-off-by: Gunnar Gudvardarson <gunni@wave.is>
---
 drivers/staging/asus_oled/asus_oled.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/staging/asus_oled/asus_oled.c b/drivers/staging/asus_oled/asus_oled.c
index 7ebecc9..5b279fb 100644
--- a/drivers/staging/asus_oled/asus_oled.c
+++ b/drivers/staging/asus_oled/asus_oled.c
@@ -771,7 +771,7 @@ static struct usb_driver oled_driver = {
 };
 
 static CLASS_ATTR_STRING(version, S_IRUGO,
-		 	ASUS_OLED_UNDERSCORE_NAME " " ASUS_OLED_VERSION);
+			ASUS_OLED_UNDERSCORE_NAME " " ASUS_OLED_VERSION);
 
 static int __init asus_oled_init(void)
 {
-- 
1.6.3.3


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

* Re: [PATCH] Staging: asus_oled: asus_oled.c removed an unnessecary space, making it checkpatch clean
  2010-04-09 13:01 [PATCH] Staging: asus_oled: asus_oled.c removed an unnessecary space, making it checkpatch clean Gunnar Gudvardarson
@ 2010-04-09 17:22 ` Greg KH
  0 siblings, 0 replies; 2+ messages in thread
From: Greg KH @ 2010-04-09 17:22 UTC (permalink / raw)
  To: Gunnar Gudvardarson; +Cc: devel, linux-kernel

On Fri, Apr 09, 2010 at 01:01:09PM +0000, Gunnar Gudvardarson wrote:
> This is the first linux kernel patch i have ever made, i don't even
> know the C programming language except reading the syntax (basic)

Then I would strongly encourage you to either learn it, or not modify
the code.

> It didn't compile on my machine due to invalid configuration of something
> on my machine, because i have not yet learned how to configure the compiler.

Especially as you were not able to build test the patch, which is a
basic requirement for submitting code changes.

Try reading the kernel file, Documentation/HOWTO for pointers on how to
find information to learn the C language, and to learn how to build a
kernel on your own.

best of luck,

greg k-h

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

end of thread, other threads:[~2010-04-09 18:00 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-04-09 13:01 [PATCH] Staging: asus_oled: asus_oled.c removed an unnessecary space, making it checkpatch clean Gunnar Gudvardarson
2010-04-09 17:22 ` 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.