Re: [PATCH] staging: ccree: Fix lines longer than 80 characters

2017-10-26 Thread Stephen Brennan
Apologies for the noise, this was the wrong patch. Please ignore this.

[PATCH] staging: ccree: Fix lines longer than 80 characters

2017-10-26 Thread Stephen Brennan
Simply break down some long lines and tab-indent them. Signed-off-by: Stephen Brennan --- I'm learning the patch submission process, and this is my first patch. I know it's trivial but I'm just trying to get my feet wet. Thanks in advance for taking the time to review this! drivers/staging/ccre

Re: [PATCH] staging: ccree: Fix lines longer than 80 characters

2017-10-24 Thread Gilad Ben-Yossef
On Mon, Oct 23, 2017 at 6:00 PM, Stephen Brennan wrote: > Hi Gilad, > > Thanks for the quick reply, I really appreciate your taking time to help a > newbie get started. I've made the appropriate changes and re-submitted. It is completely my pleasure. Thanks, > >> TIP: if you run the scripts/get_m

Re: [PATCH] staging: ccree: Fix lines longer than 80 characters

2017-10-24 Thread Gilad Ben-Yossef
Hi Tobin, On Tue, Oct 24, 2017 at 6:02 AM, Tobin C. Harding wrote: > On Mon, Oct 23, 2017 at 07:53:18AM -0700, Stephen Brennan wrote: >> Simply break down some long lines and tab-indent them. > > Hi Stephen, > > Welcome to the Linux kernel. Great that you have put in a patch, you are, > however,

Re: [PATCH] staging: ccree: Fix lines longer than 80 characters

2017-10-23 Thread Tobin C. Harding
On Mon, Oct 23, 2017 at 07:53:18AM -0700, Stephen Brennan wrote: > Simply break down some long lines and tab-indent them. Hi Stephen, Welcome to the Linux kernel. Great that you have put in a patch, you are, however, unlikely to see success fixing 'line over 80' warnings. There are a bunch of ar

Re: [PATCH] staging: ccree: Fix lines longer than 80 characters

2017-10-23 Thread Stephen Brennan
Hi Gilad, Thanks for the quick reply, I really appreciate your taking time to help a newbie get started. I've made the appropriate changes and re-submitted. > TIP: if you run the scripts/get_maintainers.pl script on your patch it > will tell you exactly which > list and which people your patch ne

[PATCH] staging: ccree: Fix lines longer than 80 characters

2017-10-23 Thread Stephen Brennan
Simply break down some long lines and tab-indent them. Signed-off-by: Stephen Brennan --- I'm learning the patch submission process, and this is my first patch. I know it's trivial but I'm just trying to get my feet wet. Thanks in advance for taking the time to review this! drivers/staging/ccre

Re: [PATCH] staging: ccree: Fix lines longer than 80 characters

2017-10-21 Thread Gilad Ben-Yossef
Hello Stephen, Thank you for your patch! sorry for not responding to your first post. I seem to have missed that email completely. The CryptoCell driver is currently in the staging process for inclusion in the Linux kernel. As such, patches such as these are discussed in the staging mailing list

Re: [PATCH] staging: ccree: Fix lines longer than 80 characters

2017-10-20 Thread Stephen Brennan
Hello, Just bumping this patch. I know it's only a very trivial change that shuts up a checkpatch warning. Please let me know if I can do anything to help. Thanks, Stephen

[PATCH] staging: ccree: Fix lines longer than 80 characters

2017-10-09 Thread Stephen Brennan
--- I'm new to kernel development and hoping to start with some simple changes to get familiar with the process. Please let me know if there's anything I can do to improve this very trivial patch! drivers/staging/ccree/ssi_pm.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff -