There is a typo in a pr_err message. Fix it.
Signed-off-by: Colin Ian King
---
arch/powerpc/platforms/book3s/vas-api.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/powerpc/platforms/book3s/vas-api.c
b/arch/powerpc/platforms/book3s/vas-api.c
index 0b6365d85d11
There is a spelling mistake in the help message. Fix it.
Signed-off-by: Colin Ian King
---
tools/testing/selftests/powerpc/dexcr/chdexcr.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/testing/selftests/powerpc/dexcr/chdexcr.c
b/tools/testing/selftests/powerpc/dexcr
There is a statement with two semicolons. Remove the second one, it
is redundant.
Signed-off-by: Colin Ian King
---
drivers/tty/hvc/hvc_xen.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/tty/hvc/hvc_xen.c b/drivers/tty/hvc/hvc_xen.c
index 0e497501f8e3
never read [deadcode.DeadStores]
drivers/soc/fsl/dpio/qbman-portal.c:820:3: warning: Value stored to 'p'
is never read [deadcode.DeadStores]
Signed-off-by: Colin Ian King
---
drivers/soc/fsl/dpio/qbman-portal.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/soc/fsl/dpio/qbman
There is a spelling mistake of the EACCES error name, fix it.
Signed-off-by: Colin Ian King
---
tools/testing/selftests/powerpc/papr_sysparm/papr_sysparm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/testing/selftests/powerpc/papr_sysparm/papr_sysparm.c
b/tools
The pointer p is being assigned a value that is never read, the assignment
is redundant and can be removed. Cleans up clang scan build warning:
drivers/soc/fsl/dpio/qbman-portal.c:662:3: warning: Value stored to 'p'
is never read [deadcode.DeadStores]
Signed-off-by: Colin Ian King
--
: Either
the condition '!iov' is redundant or there is possible null pointer
dereference: iov. [nullPointerRedundantCheck]
num_vfs = iov->num_vfs;
^
Fixes: 052da31d45fc ("powerpc/powernv/sriov: De-indent setup and teardown")
Signed-off-by: Colin Ian King
---
arch/po
There is a spelling mistake in a perror message. Fix it.
Signed-off-by: Colin Ian King
---
tools/testing/selftests/powerpc/ptrace/core-pkey.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/testing/selftests/powerpc/ptrace/core-pkey.c
b/tools/testing/selftests/powerpc
There are a few spelling mistakes in error messages. Fix them.
Signed-off-by: Colin Ian King
---
tools/testing/selftests/powerpc/security/spectre_v2.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/tools/testing/selftests/powerpc/security/spectre_v2.c
b/tools/testing
On 14/11/17 13:45, Julia Lawall wrote:
>
>
> On Tue, 14 Nov 2017, Colin King wrote:
>
>> From: Colin Ian King
>>
>> Trivial fix to spelling mistake in pr_err error message text
>>
>> Signed-off-by: Colin Ian King
>> ---
>> drivers/c
On 24/10/16 23:20, Joe Perches wrote:
> On Mon, 2016-10-24 at 15:13 -0700, Kees Cook wrote:
>> On Mon, Oct 24, 2016 at 3:02 PM, Colin King wrote:
>>> From: Colin Ian King
>>>
>>> trivial fix to spelling mistake in pr_debug message
>>>
>>> Si
On 12/09/16 17:34, Joe Perches wrote:
> On Mon, 2016-09-12 at 11:12 +0100, Colin King wrote:
>> From: Colin Ian King
>> The message is missing a \n, add it.
>
> Was this found via inspection or a tool?
>
grep
On 01/07/16 06:27, Stephen Rothwell wrote:
> Hi all,
>
> After merging the powerpc tree, today's linux-next build (powerpc
> allyesconfig) failed like this:
>
> arch/powerpc/kernel/fadump.c: In function 'fadump_invalidate_dump':
> arch/powerpc/kernel/fadump.c:1014:2: error: expected ';' before '}
On 27/06/16 12:20, Michael Ellerman wrote:
> On Mon, 2016-06-27 at 03:51 -0700, Joe Perches wrote:
>> On Mon, 2016-06-27 at 11:38 +0100, Colin Ian King wrote:
>>> On 26/06/16 05:19, Michael Ellerman wrote:
>>>> On Fri, 2016-24-06 at 17:43:00 UTC, Colin King wrote:
On 26/06/16 05:19, Michael Ellerman wrote:
> On Fri, 2016-24-06 at 17:43:00 UTC, Colin King wrote:
>> From: Colin Ian King
>>
>> trivial fix to spelling mistake "rgistration" and minor clean up
>> of the printk error message
>
> Can you also:
> - u
sage.
>
> Not that it's particularly meaningful for such a trivial fix, but FWIW
:
>
> Reviewed-by: Daniel Axtens
>
> Regards,
> Daniel
>
> Colin King writes:
>
>> From: Colin Ian King
>>
>> pi_buff is being memset before it is sanity chec
On the unlikely event that drv is null, the current code will
perform a null pointer dereference with it when printing a dev_dbg
message. Instead, the BUG_ON check on drv should be performed
before we emit the dev_dbg message.
Signed-off-by: Colin Ian King
Signed-off-by: Geoff Levand
d way as opposed to just removing it and
then hitting an issue with a null ptr deference.
>
>> Signed-off-by: Colin Ian King
>> ---
>> drivers/ps3/ps3-vuart.c | 3 +--
>> 1 file changed, 1 insertion(+), 2 deletions(-)
>>
>> diff --git a/drivers/ps3/ps3-vua
18 matches
Mail list logo