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
On Mon, Feb 27, 2017 at 4:21 PM, Greg Kroah-Hartman
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 rather
>> than BUG() or BUG_
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
>> +
On Mon, Feb 27, 2017 at 10:42 PM, Greg KH wrote:
> On Sun, Feb 26, 2017 at 01:22:18PM +0800, Man Choy wrote:
>> Fix following checkpatch warning:
>>
>> WARNING: line over 80 characters
>> +>attributes)
&g
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
Fix following checkpatch warning:
WARNING: line over 80 characters
+>attributes) <
0)
total: 0 errors, 1 warnings, 485 lines checked
Signed-off-by: Man Choy
---
drivers/staging/speakup/synth.c | 4 ++--
1 file changed, 2 insertions
Fix following warning:
WARNING: line over 80 characters
+>attributes) <
0)
Signed-off-by: Man Choy <manch...@gmail.com>
---
drivers/staging/speakup/synth.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --g
Fix following warning:
WARNING: line over 80 characters
+>attributes) <
0)
Signed-off-by: Man Choy
---
drivers/staging/speakup/synth.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/s
On Thu, Feb 23, 2017 at 9:34 PM, Joe Perches <j...@perches.com> wrote:
> On Thu, 2017-02-23 at 21:15 +0800, Man Choy wrote:
> []
>> diff --git a/drivers/staging/speakup/synth.c
>> b/drivers/staging/speakup/synth.c
> []
>> @@ -407,7 +407,7 @@ static int do_synt
On Thu, Feb 23, 2017 at 9:34 PM, Joe Perches wrote:
> On Thu, 2017-02-23 at 21:15 +0800, Man Choy wrote:
> []
>> diff --git a/drivers/staging/speakup/synth.c
>> b/drivers/staging/speakup/synth.c
> []
>> @@ -407,7 +407,7 @@ static int do_synth_init
Signed-off-by: Man Choy <manch...@gmail.com>
---
drivers/staging/speakup/synth.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/speakup/synth.c b/drivers/staging/speakup/synth.c
index 1cd38e7..968d8b6 100644
--- a/drivers/staging/speakup/synth.c
Signed-off-by: Man Choy
---
drivers/staging/speakup/synth.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/speakup/synth.c b/drivers/staging/speakup/synth.c
index 1cd38e7..968d8b6 100644
--- a/drivers/staging/speakup/synth.c
+++ b/drivers/staging/speakup
Signed-off-by: Man Choy <manch...@gmail.com>
---
drivers/staging/speakup/synth.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/speakup/synth.c b/drivers/staging/speakup/synth.c
index a61c02b..1cd38e7 100644
--- a/drivers/staging/speakup/synth.c
+++ b/d
Signed-off-by: Man Choy
---
drivers/staging/speakup/synth.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/speakup/synth.c b/drivers/staging/speakup/synth.c
index a61c02b..1cd38e7 100644
--- a/drivers/staging/speakup/synth.c
+++ b/drivers/staging/speakup
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);
^
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);
^
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
On Mon, Feb 13, 2017 at 8:03 PM, Greg Kroah-Hartman
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 the cleanup. How do I
>> doubl
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
On Sun, Feb 12, 2017 at 8:16 PM, Greg Kroah-Hartman
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 = NULL;
>>
>> ERROR: do no
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 ++--
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
---
drivers/staging/bcm2835-audio/bcm2835.c | 4 ++--
1 file changed, 2
22 matches
Mail list logo
