[PATCH][next] powerpc/pseries/vas: Fix typo "mmaping" -> "mmapping"

2025-01-06 Thread Colin Ian King
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

[PATCH][next] selftests/powerpc/dexcr: Fix spelling mistake "predicition" -> "prediction"

2024-05-08 Thread Colin Ian King
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

[PATCH][next] tty: hvc: Remove second semicolon

2024-03-15 Thread Colin Ian King
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

[PATCH][next] soc: fsl: dpio: remove redundant assignment to pointer p

2024-02-23 Thread Colin Ian King
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

[PATCH][next] powerpc/selftests: Fix spelling mistake "EACCESS" -> "EACCES"

2023-12-15 Thread Colin Ian King
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

[PATCH][next] soc: fsl: dpio: remove redundant assignment to pointer p

2023-11-11 Thread Colin Ian King
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 --

[PATCH][next] powerpc/powernv/sriov: perform null check on iov before dereferencing iov

2023-06-08 Thread 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

[PATCH] selftests/powerpc: Fix spelling mistake "mmaping" -> "mmapping"

2022-10-21 Thread Colin Ian King
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

[PATCH] selftests/powerpc/pmu: fix spelling mistake "mis-match" -> "mismatch"

2022-03-20 Thread Colin Ian King
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

Re: [PATCH] crypto/nx: fix spelling mistake: "availavle" -> "available"

2017-11-14 Thread Colin Ian King
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

Re: [PATCH] powerpc/pseries: fix spelling mistake: "Attemping" -> "Attempting"

2016-10-24 Thread Colin Ian King
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

Re: [PATCH] powerpc/32: add missing \n at end of printk warning message

2016-09-12 Thread Colin Ian King
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

Re: linux-next: build failure after merge of the powerpc tree

2016-07-01 Thread Colin Ian King
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 '}

Re: powerpc/fadump: trivial fix of spelling mistake, clean up message

2016-06-27 Thread Colin Ian King
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:

Re: powerpc/fadump: trivial fix of spelling mistake, clean up message

2016-06-27 Thread Colin Ian King
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

Re: [PATCH] powerpc/pseries/hvcserver: don't memset pi_buff if it is null

2015-10-11 Thread Colin Ian King
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

[PATCH 2/3] drivers/ps3: Fix ps3-vuart null dereference

2015-09-14 Thread Colin Ian King
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

Re: [PATCH] ps3-vuart: BUG_ON on null drv before dereferencing it on dev_dbg

2015-09-01 Thread Colin Ian King
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