Re: [PATCH] bcm2048: Fix checkpatch checks

2017-02-27 Thread Man Choy
On Mon, Feb 27, 2017 at 4:21 PM, Greg Kroah-Hartman <gre...@linuxfoundation.org> wrote: > On Sat, Feb 18, 2017 at 11:52:37AM +0800, Man Choy wrote: >> Fix following checks: >> >> CHECK: Avoid crashing the kernel - try using WARN_ON & recovery code

Re: [PATCH] speakup: synth: Fix checkpatch warning

2017-02-27 Thread Man Choy
On Mon, Feb 27, 2017 at 10:42 PM, Greg KH <gre...@linuxfoundation.org> wrote: > On Sun, Feb 26, 2017 at 01:22:18PM +0800, Man Choy wrote: >> Fix following checkpatch warning: >> >> WARNING: line over 80 characters >> +

[PATCH] speakup: synth: Fix checkpatch warning

2017-02-25 Thread Man Choy
Fix following checkpatch warning: WARNING: line over 80 characters +>attributes) < 0) total: 0 errors, 1 warnings, 485 lines checked Signed-off-by: Man Choy <manch...@gmail.com> --- drivers/staging/speakup/synth.c | 4 ++-- 1

[PATCH] bcm2048: Fix checkpatch checks

2017-02-17 Thread Man Choy
Fix following checks: CHECK: Avoid crashing the kernel - try using WARN_ON & recovery code rather than BUG() or BUG_ON() + BUG_ON((index+2) >= BCM2048_MAX_RDS_RT); CHECK: spaces preferred around that '+' (ctx:VxV) + BUG_ON((index+2) >= BCM2048_MAX_RDS_RT); ^

Re: [PATCH] bcm2835-audio: Fix checkpatch errors

2017-02-13 Thread Man Choy
On Mon, Feb 13, 2017 at 8:03 PM, Greg Kroah-Hartman <gre...@linuxfoundation.org> wrote: > On Mon, Feb 13, 2017 at 08:49:06AM +0800, Man Choy wrote: >> A quick question, I just updated my linux-next tree last night, >> checkout tag "next-20170210" and working on

Re: [PATCH] bcm2835-audio: Fix checkpatch errors

2017-02-12 Thread Man Choy
On Sun, Feb 12, 2017 at 8:16 PM, Greg Kroah-Hartman <gre...@linuxfoundation.org> wrote: > On Sun, Feb 12, 2017 at 04:34:09AM +0800, Man Choy wrote: >> Fix following errors: >> >> ERROR: do not initialise statics to NULL >> +static struct snd_card *g_card = NUL

[PATCH] bcm2835-audio: Fix checkpatch errors

2017-02-12 Thread Man Choy
Fix following errors: ERROR: do not initialise statics to NULL +static struct snd_card *g_card = NULL; ERROR: do not initialise statics to NULL +static struct bcm2835_chip *g_chip = NULL; Signed-off-by: Man Choy <manch...@gmail.com> --- drivers/staging/bcm2835-audio/bcm2835.c | 4 ++--