On 5/2/2019 3:53 PM, Colin King wrote:
From: Colin Ian King
There is a spelling mistake in a pr_err message, fix it.
Signed-off-by: Colin Ian King
Reviewed-by: Mukesh Ojha
Cheers,
-Mukesh
---
arch/powerpc/kvm/book3s_xive_native.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion
: In function
'ddw_memory_hotplug_max':
arch/powerpc/platforms/pseries/iommu.c:948:7: warning: variable 'ranges'
set but not used [-Wunused-but-set-variable]
int ranges, n_mem_addr_cells, n_mem_size_cells, len;
^~
Signed-off-by: Qian Cai
Reviewed-by: Mukesh Ojha
platforms/pseries/pmem.c:109:6: warning: variable 'count'
set but not used [-Wunused-but-set-variable]
Signed-off-by: Qian Cai
Reviewed-by: Mukesh Ojha
Cheers,
-Mukesh
---
arch/powerpc/platforms/pseries/pmem.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/
On 4/5/2019 6:20 PM, Fabio Estevam wrote:
On Fri, Apr 5, 2019 at 9:47 AM Mukesh Ojha wrote:
Don't change entire thing with //. only above line with the license is
enough.
Please apply this rule every patch of yours.
Not really. This is the style preferred in this subsystem.
you
On 4/5/2019 5:19 PM, Andra Danciu wrote:
Adopt the SPDX license identifier headers to ease license compliance
management.
Signed-off-by: Andra Danciu
---
sound/soc/fsl/efika-audio-fabric.c | 17 ++---
1 file changed, 6 insertions(+), 11 deletions(-)
diff --git a/sound/soc/fsl/
On 4/5/2019 5:19 PM, Andra Danciu wrote:
Adopt the SPDX license identifier headers to ease license compliance
management.
Signed-off-by: Andra Danciu
Reviewed-by: Mukesh Ojha
Cheers,
-Mukesh
---
sound/soc/fsl/imx-pcm.h | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff
On 4/5/2019 5:19 PM, Andra Danciu wrote:
Adopt the SPDX license identifier headers to ease license compliance
management.
Signed-off-by: Andra Danciu
Reviewed-by: Mukesh Ojha
Cheers,
-Mukesh
---
sound/soc/fsl/imx-pcm.h | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff
sed [-Wunused-but-set-variable]
It's never used since introduction in
commit ed651a10875f ("ibmvnic: Updated reset handling")
Signed-off-by: YueHaibing
Reviewed-by: Mukesh Ojha
Cheers,
-Mukesh
---
drivers/net/ethernet/ibm/ibmvnic.c | 2 --
1 file changed, 2 deletions(-)
diff
;)
Signed-off-by: Julia Lawall
Reviewed-by: Mukesh Ojha
Cheers,
-Mukesh
---
arch/powerpc/platforms/83xx/usb.c |4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff -u -p a/arch/powerpc/platforms/83xx/usb.c
b/arch/powerpc/platforms/83xx/usb.c
--- a/arch/powerpc/platforms/83xx/usb.c
++
_decompress':
drivers/crypto/nx/nx-842-pseries.c:430:15: warning: variable 'max_sync_size'
set but not used [-Wunused-but-set-variable]
They are not used any more and can be removed.
Signed-off-by: YueHaibing
Reviewed-by: Mukesh Ojha
Cheers,
-Mukesh
---
drivers/crypto/nx/nx-842
able]
drivers/misc/ocxl/link.c:187:6: warning: variable 'lpid' set but not used
[-Wunused-but-set-variable]
They are never used and can be removed.
Signed-off-by: YueHaibing
Reviewed-by: Mukesh Ojha
Cheers,
-Mukesh
---
drivers/misc/ocxl/link.c | 4 +---
1 file changed, 1 in
On 3/28/2019 2:58 PM, Colin King wrote:
From: Colin Ian King
There is a spelling mistake in a dev_err message. Fix this.
Signed-off-by: Colin Ian King
Reviewed-by: Mukesh Ojha
-Mukesh
---
sound/soc/fsl/imx-audmix.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
On 3/23/2019 6:20 PM, jagdsh.li...@gmail.com wrote:
From: Jagadeesh Pagadala
Remove duplicate headers which are included twice.
Signed-off-by: Jagadeesh Pagadala
Reviewed-by: Mukesh Ojha
-Mukesh
---
arch/powerpc/kernel/time.c | 1 -
arch/powerpc/lib/code-patching.c | 1
ms/embedded6xx/holly.c:196:17: error: no previous
prototype for 'holly_restart' [-Werror=missing-prototypes]
Signed-off-by: Mathieu Malaterre
Reviewed-by: Mukesh Ojha
---
v2: Split the patch in two operations
arch/powerpc/platforms/embedded6xx/holly.c | 7 ---
1 file changed,
3:6: error: no previous
prototype for 'holly_halt' [-Werror=missing-prototypes]
Signed-off-by: Mathieu Malaterre
Reviewed-by: Mukesh Ojha
-Mukesh
---
arch/powerpc/platforms/embedded6xx/holly.c | 12
1 file changed, 12 deletions(-)
diff --git a/arch/powerpc/p
d while returning -1 or 0 from handler.
Signed-off-by: Mukesh Ojha
Reviewed-by: Vasant Hegde
---
arch/powerpc/platforms/powernv/opal-dump.c | 9 +++--
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/arch/powerpc/platforms/powernv/opal-dump.c
b/arch/powerpc/platforms/power
Moves the return value check of 'opal_dump_info' to a proper place which
was previously unnecessarily filling all the dump info even on failure.
Signed-off-by: Mukesh Ojha
Acked-by: Stewart Smith
Acked-by: Jeremy Kerr
---
arch/powerpc/platforms/powernv/opal-dump.c | 9 ++-
On Thursday 16 February 2017 08:10 AM, Jeremy Kerr wrote:
Hi Mukesh,
The return value of an interrupt handler is the special type
irqreturn_t. An interrupt handler can return two special values,
IRQ_NONE or IRQ_HANDLED.
Yep, you can assume that the reader knows that level of the interrupt
ha
Hi Jeremy,
On Wednesday 15 February 2017 08:48 AM, Jeremy Kerr wrote:
Hi Mukesh,
Converts all the return explicit number to a more proper IRQ_HANDLED,
which looks proper incase of interrupt handler returning case.
This looks good to me, but can you describe the effects of those changes
to th
On Wednesday 15 February 2017 10:38 AM, Stewart Smith wrote:
Mukesh Ojha writes:
Converts all the return explicit number to a more proper IRQ_HANDLED,
which looks proper incase of interrupt handler returning case.
Signed-off-by: Mukesh Ojha
Reviewed-by: Vasant Hegde
---
arch/powerpc
Hi Micheal,
Can you please look at this patchset?
-Mukesh
On Tuesday 06 December 2016 12:07 PM, Mukesh Ojha wrote:
Hi Michael,
Can you please have a look at this patchset as there is no
functional changes involve with this?
Thanks,
Mukesh
On Thursday 01 December 2016 02:38 PM, Mukesh
return -EIO;
- case OPAL_INTERNAL_ERROR: return -EIO;
+ case OPAL_UNSUPPORTED:
+ case OPAL_HARDWARE:
+ case OPAL_INTERNAL_ERROR:
+ case OPAL_WRONG_STATE: return -EIO;
Looks good.
Reviewed-by: Mukesh Ojha
default:
Hi Michael,
Can you please have a look at this patchset as there is no
functional changes involve with this?
Thanks,
Mukesh
On Thursday 01 December 2016 02:38 PM, Mukesh Ojha wrote:
Moves the return value check of 'opal_dump_info' to a proper place which
was previously unn
Converts all the return explicit number to a more proper IRQ_HANDLED,
which looks proper incase of interrupt handler returning case.
Signed-off-by: Mukesh Ojha
Reviewed-by: Vasant Hegde
---
arch/powerpc/platforms/powernv/opal-dump.c | 9 +++--
1 file changed, 3 insertions(+), 6 deletions
Moves the return value check of 'opal_dump_info' to a proper place which
was previously unnecessarily filling all the dump info even on failure.
Signed-off-by: Mukesh Ojha
---
arch/powerpc/platforms/powernv/opal-dump.c | 9 ++---
1 file changed, 6 insertions(+), 3 deletions(-)
di
7; which was missing earlier.
Signed-off-by: Mukesh Ojha
Cc: sta...@vger.kernel.org
---
arch/powerpc/platforms/powernv/opal-dump.c | 7 ++-
arch/powerpc/platforms/powernv/opal-elog.c | 7 ++-
2 files changed, 12 insertions(+), 2 deletions(-)
diff --git a/arch/powerpc/platforms/powernv
26 matches
Mail list logo
