Question On staging/drivers/staging/*/TODO

2018-08-27 Thread Ray Clinton
Hi, I'm brand new to kernel dev and after going through Kernel Newbies am trying to find a good first patch to work. I just wanted to know if the TODO files, mentioned in the subject, are up-to-date. If not, would you have any suggestions on where to find a good first patch to work on? Thanks so

Re: Question On staging/drivers/staging/*/TODO

2018-08-27 Thread Ray Clinton
Great, thanks so much for the quick reply! Ray On Mon, Aug 27, 2018 at 2:34 PM Greg Kroah-Hartman wrote: > > On Mon, Aug 27, 2018 at 02:26:48PM -0400, Ray Clinton wrote: > > Hi, > > > > I'm brand new to kernel dev and after going through Kernel Newbies > > a

[PATCH] Trivial numbering change in comments.

2018-08-28 Thread Ray Clinton
his is my first actual try at one and thought I might start out small (and I don't think you can get smaller than a single character change). Any and all advice (about this patch, email, or anything else) is very welcome. Thanks so much! Ray Signed-off-by: Ray Clinton --- drivers/staging/and

Re: Kernel-only deployments?

2018-08-23 Thread Ray Clinton
On Thu, Aug 23, 2018 at 7:44 PM Paul E. McKenney wrote: > Does anyone do kernel-only deployments, for example, setting up an > embedded device having a Linux kernel and absolutely no userspace > whatsoever? To be honest I'm a total newb to kernel dev, so much so that I copied and pasted the above

[PATCH 2/4] staging: comedi: Correct multiline dereference as per TODO

2018-08-30 Thread Ray Clinton
ff-by: Ray Clinton --- drivers/staging/comedi/drivers/ni_labpc_common.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/comedi/drivers/ni_labpc_common.c b/drivers/staging/comedi/drivers/ni_labpc_common.c index 7fa2d39..0fad809 100644 --- a/drivers/staging/c

[PATCH 3/4] staging: comedi: Correct multiline dereference as per TODO

2018-08-30 Thread Ray Clinton
Using checkpatch.pl I was able to find a multiline dereference which goes again the coding style for the kernel. I'm still working on my email client so the indentation looks bad here (in gmail) but the arguments for ni_tio_set_mite_channel should go just under the opening ( Signed-off-by

[PATCH 4/4] staging: comedi: Correct multiline dereference as per TODO

2018-08-30 Thread Ray Clinton
ff-by: Ray Clinton --- drivers/staging/comedi/drivers/rtd520.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/comedi/drivers/rtd520.c b/drivers/staging/comedi/drivers/rtd520.c index bb400e0..b53bf5d 100644 --- a/drivers/staging/comedi/drivers/rtd520.c

[PATCH 1/4] staging: comedi: Correct multiline dereference as per TODO

2018-08-30 Thread Ray Clinton
ff-by: Ray Clinton --- drivers/staging/comedi/drivers/dt3000.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/comedi/drivers/dt3000.c b/drivers/staging/comedi/drivers/dt3000.c index 2edf3ee..6bc5c86 100644 --- a/drivers/staging/comedi/drivers/dt3000.c

[PATCH 0/4] staging: comedi: Correct multiline dereference as per TODO

2018-08-30 Thread Ray Clinton
Hi, I've been working on the correcting the checkpatch.pl failures found in staging/drivers/staging/comedi of 4.18.0 and have a few patches to submit for review. The files that I changed are: drivers/dt3000.c drivers/ni_labpc_common.c drivers/ni_mio_common.c drivers/dt3000.c While there were a