Re: [PATCH v6 35/37] cxlflash: Fix to avoid corrupting port selection mask

2015-10-22 Thread Manoj Kumar
Acked-by: Manoj Kumar On 10/21/2015 3:16 PM, Matthew R. Ochs wrote: The port selection mask of a LUN can be corrupted when the manage LUN ioctl (DK_CXLFLASH_MANAGE_LUN) is issued more than once for any device. ___

Re: [PATCH v6 37/37] cxlflash: Fix to avoid bypassing context cleanup

2015-10-22 Thread Manoj Kumar
Acked-by: Manoj Kumar On 10/21/2015 3:16 PM, Matthew R. Ochs wrote: Contexts may be skipped over for cleanup in situations where contention for the adapter's table-list mutex is experienced in the presence of a signal during the execution of the release handler.

Re: [PATCH V4 00/31] powerpc/mm: Update page table format for book3s 64

2015-10-22 Thread Denis Kirjanov
On 10/17/15, Aneesh Kumar K.V wrote: > Hi All, > > This patch series attempt to update book3s 64 linux page table format to > make it more flexible. Our current pte format is very restrictive and we > overload multiple pte bits. This is due to the non-availability

Re: Time to remove platforms/cell?

2015-10-22 Thread Geoff Levand
Hi, On Fri, 2015-10-09 at 23:13 +0200, Marc Dietrich wrote: > Am Freitag, 9. Oktober 2015, 10:45:42 schrieb Geoff Levand: > > With the 4.2-rc4 kernel, kexec seems to work when CONFIG_SPU_FS=n. > > I > > have not tried with petitboot release white-09.09.01-15.56 yet though. > > I guess you mean

Re: [PATCH v6 36/37] cxlflash: Fix to avoid lock instrumentation rejection

2015-10-22 Thread Manoj Kumar
Acked-by: Manoj Kumar On 10/21/2015 3:16 PM, Matthew R. Ochs wrote: When running with lock instrumentation (e.g. lockdep), some of the instrumentation can become disabled at probe time for a cxlflash adapter. This is due to a missing lock registration for the

Re: [RFC PATCH 0/8] clk: qoriq: Move chip-specific knowledge into driver

2015-10-22 Thread Michael Turquette
Quoting Scott Wood (2015-10-09 16:57:44) > On Thu, 2015-10-01 at 19:26 -0500, Scott Wood wrote: > > [Resending to updated e-mail address] > > > > On Tue, 2015-08-11 at 11:25 -0700, Michael Turquette wrote: > > > Hi Scott, > > > > > > Quoting Scott Wood (2015-06-18 19:49:10) > > > > The existing

Re: [PATCH tip/locking/core v4 1/6] powerpc: atomic: Make *xchg and *cmpxchg a full barrier

2015-10-22 Thread Boqun Feng
On Wed, Oct 21, 2015 at 09:48:25PM +0200, Peter Zijlstra wrote: > On Wed, Oct 21, 2015 at 12:35:23PM -0700, Paul E. McKenney wrote: > > > > > > I ask this because I recall Peter once bought up a discussion: > > > > > > > > > > > > https://lkml.org/lkml/2015/8/26/596 > > > > So a full barrier on

Re: [PATCH V7 0/6] Redesign SR-IOV on PowerNV

2015-10-22 Thread Gavin Shan
On Thu, Oct 22, 2015 at 09:22:13AM +0800, Wei Yang wrote: >In original design, it tries to group VFs to enable more number of VFs in the >system, when VF BAR is bigger than 64MB. This design has a flaw in which one >error on a VF will interfere other VFs in the same group. > >This patch series

Re: [PATCH v6 22/22] of/platform: Defer probes of registered devices

2015-10-22 Thread Scott Wood
On Thu, 2015-10-22 at 15:04 +0200, Tomeu Vizoso wrote: > On 22 October 2015 at 00:51, Scott Wood wrote: > > On Wed, 2015-10-21 at 08:44 -0500, Rob Herring wrote: > > > On Wed, Oct 21, 2015 at 12:54 AM, Scott Wood > > > wrote: > > > > On Mon,

[PATCH] ASoC: fsl: Use #ifdef instead of #if for CONFIG_PM_SLEEP

2015-10-22 Thread Nicolin Chen
Change them to #ifdef as CONFIG_PM_SLEEP might not be defined at all. Reported-by: kbuild test robot Signed-off-by: Nicolin Chen --- sound/soc/fsl/fsl_esai.c | 2 +- sound/soc/fsl/fsl_sai.c | 2 +- 2 files changed, 2 insertions(+), 2

Re: [PATCH] powerpc: Introduce the use of the managed version of kzalloc

2015-10-22 Thread Anatolij Gustschin
On Tue, 27 May 2014 01:51:31 +0530 Himangi Saraogi wrote: ... > > Signed-off-by: Himangi Saraogi > Acked-by: Julia Lawall > --- > Not compile tested > arch/powerpc/platforms/52xx/mpc52xx_gpt.c | 6 ++ > 1 file changed, 2

Re: [PATCH] powerpc: mpc512x: drop bogus and unused psc register bit definitions

2015-10-22 Thread Anatolij Gustschin
On Tue, 21 Jul 2015 09:21:13 +0200 Uwe Kleine-König wrote: > These were introduced in commit 25ae3a0739c6 ("[POWERPC] mpc512x: Add > MPC512x PSC support to MPC52xx psc driver") and never used. Moreover > according to the datasheet[1] MEMERROR is bit 25 (0x40) and

Re: [PATCH v6 36/37] cxlflash: Fix to avoid lock instrumentation rejection

2015-10-22 Thread Andrew Donnellan
On 22/10/15 07:16, Matthew R. Ochs wrote: When running with lock instrumentation (e.g. lockdep), some of the instrumentation can become disabled at probe time for a cxlflash adapter. This is due to a missing lock registration for the tmf_slock. The fix is to call spin_lock_init() for the

Re: [PATCH 9/9] powerpc: optimise csum_partial() call when len is constant

2015-10-22 Thread Scott Wood
On Tue, 2015-09-22 at 16:34 +0200, Christophe Leroy wrote: > csum_partial is often called for small fixed length packets > for which it is suboptimal to use the generic csum_partial() > function. > > For instance, in my configuration, I got: > * One place calling it with constant len 4 > * Seven

[GIT PULL] Please pull powerpc/linux.git powerpc-4.3-5 tag

2015-10-22 Thread Michael Ellerman
Hi Linus, Please pull some more powerpc fixes for 4.3: The following changes since commit abb39bc792aa8e9143e78a39ec13d7de1391f4b4: selftests/powerpc: Fix build failure of load_unaligned_zeropad test (2015-10-13 15:08:37 +1100) are available in the git repository at:

Re: [PATCH v6 16/37] cxlflash: Fix async interrupt bypass logic

2015-10-22 Thread Andrew Donnellan
On 22/10/15 07:13, Matthew R. Ochs wrote: A bug was introduced earlier in the development cycle when cleaning up logic statements. Instead of skipping bits that are not set, set bits are skipped, causing async interrupts to not be handled correctly. To fix, simply add back in the proper

Re: [PATCH v6 35/37] cxlflash: Fix to avoid corrupting port selection mask

2015-10-22 Thread Andrew Donnellan
On 22/10/15 07:16, Matthew R. Ochs wrote: The port selection mask of a LUN can be corrupted when the manage LUN ioctl (DK_CXLFLASH_MANAGE_LUN) is issued more than once for any device. > ... Signed-off-by: Matthew R. Ochs Reviewed-by: Andrew Donnellan

Re: Time to remove platforms/cell?

2015-10-22 Thread Michael Ellerman
On Thu, 2015-10-22 at 12:15 -0700, Geoff Levand wrote: > Hi, > > On Fri, 2015-10-09 at 23:13 +0200, Marc Dietrich wrote: > > Am Freitag, 9. Oktober 2015, 10:45:42 schrieb Geoff Levand: > > > With the 4.2-rc4 kernel, kexec seems to work when CONFIG_SPU_FS=n. > > > I > > > have not tried with

Re: [PATCH v12 5/6] QE: use subsys_initcall to init qe

2015-10-22 Thread Scott Wood
On Wed, 2015-10-14 at 15:16 +0800, Zhao Qiang wrote: > diff --git a/arch/powerpc/sysdev/qe_lib/qe.c > > b/arch/powerpc/sysdev/qe_lib/qe.c > index c2518cd..3f9f596 100644 > --- a/arch/powerpc/sysdev/qe_lib/qe.c > +++ b/arch/powerpc/sysdev/qe_lib/qe.c > @@ -671,6 +671,21 @@ unsigned int

Re: [PATCH v12 3/6] CPM/QE: use genalloc to manage CPM/QE muram

2015-10-22 Thread Scott Wood
On Wed, 2015-10-14 at 15:16 +0800, Zhao Qiang wrote: > Use genalloc to manage CPM/QE muram instead of rheap. > > Signed-off-by: Zhao Qiang > --- > Changes for v9: > - splitted from patch 3/5, modify cpm muram management functions. > Changes for v10: > -

Re: [PATCH v12 4/6] QE/CPM: move muram management functions to qe_common

2015-10-22 Thread Scott Wood
On Wed, 2015-10-14 at 15:16 +0800, Zhao Qiang wrote: > QE and CPM have the same muram, they use the same management > functions. Now QE support both ARM and PowerPC, it is necessary > to move QE to "driver/soc", so move the muram management functions > from cpm_common to qe_common for preparing to

Re: [PATCH 6/9] powerpc32: optimise a few instructions in csum_partial()

2015-10-22 Thread Scott Wood
On Tue, 2015-09-22 at 16:34 +0200, Christophe Leroy wrote: > r5 does contain the value to be updated, so lets use r5 all way long > for that. It makes the code more readable. > > To avoid confusion, it is better to use adde instead of addc > > The first addition is useless. Its only purpose is

[PATCH] clk: qoriq: Fix wrong data in p2041_cmux_grp2

2015-10-22 Thread Scott Wood
Signed-off-by: Scott Wood --- This is on top of http://patchwork.ozlabs.org/patch/519801/ and I'll apply them together. drivers/clk/clk-qoriq.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/clk/clk-qoriq.c b/drivers/clk/clk-qoriq.c

Re: [PATCH v12 6/6] QE: Move QE from arch/powerpc to drivers/soc

2015-10-22 Thread Scott Wood
On Wed, Oct 14, 2015 at 03:16:08PM +0800, Zhao Qiang wrote: > diff --git a/arch/powerpc/platforms/Kconfig b/arch/powerpc/platforms/Kconfig > index 01626be7..c9541a5 100644 > --- a/arch/powerpc/platforms/Kconfig > +++ b/arch/powerpc/platforms/Kconfig > @@ -272,25 +272,6 @@ config TAU_AVERAGE > >

Re: [PATCH 3/9] powerpc32: checksum_wrappers_64 becomes checksum_wrappers

2015-10-22 Thread Scott Wood
On Tue, 2015-09-22 at 16:34 +0200, Christophe Leroy wrote: > The powerpc64 checksum wrapper functions adds csum_and_copy_to_user() > which otherwise is implemented in include/net/checksum.h by using > csum_partial() then copy_to_user() > > Those two wrapper fonctions are also applicable to

Re: [PATCH 8/9] powerpc: simplify csum_add(a, b) in case a or b is constant 0

2015-10-22 Thread Scott Wood
On Tue, 2015-09-22 at 16:34 +0200, Christophe Leroy wrote: > Simplify csum_add(a, b) in case a or b is constant 0 > > Signed-off-by: Christophe Leroy > --- > arch/powerpc/include/asm/checksum.h | 6 ++ > 1 file changed, 6 insertions(+) > > diff --git

[PATCH v3 05/18] powerpc/85xx: Load all early TLB entries at once

2015-10-22 Thread Scott Wood
Use an AS=1 trampoline TLB entry to allow all normal TLB1 entries to be loaded at once. This avoids the need to keep the translation that code is executing from in the same TLB entry in the final TLB configuration as during early boot, which in turn is helpful for relocatable kernels (e.g. kdump)

Re: [v2] powerpc/e6500: hw tablewalk: make sure we invalidate and write to the same tlb entry

2015-10-22 Thread Kevin Hao
On Fri, Oct 16, 2015 at 07:01:55PM -0500, Scott Wood wrote: > On Tue, Aug 18, 2015 at 03:55:56PM +0800, Kevin Hao wrote: > > diff --git a/arch/powerpc/mm/tlb_low_64e.S b/arch/powerpc/mm/tlb_low_64e.S > > index e4185581c5a7..3a5b89dfb5a1 100644 > > --- a/arch/powerpc/mm/tlb_low_64e.S > > +++

Re: [PATCH v6 22/22] of/platform: Defer probes of registered devices

2015-10-22 Thread Tomeu Vizoso
On 22 October 2015 at 00:51, Scott Wood wrote: > On Wed, 2015-10-21 at 08:44 -0500, Rob Herring wrote: >> On Wed, Oct 21, 2015 at 12:54 AM, Scott Wood >> wrote: >> > On Mon, 2015-09-21 at 16:03 +0200, Tomeu Vizoso wrote: >> > > Instead of trying

Re: [PATCH v4 2/2] powerpc/512x: add a device tree binding for LocalPlus Bus FIFO

2015-10-22 Thread Anatolij Gustschin
On Mon, 12 Oct 2015 00:08:02 +0300 Alexander Popov wrote: > Add a device tree binding for Freescale MPC512x LocalPlus Bus FIFO and > introduce the document describing that binding. > > Signed-off-by: Alexander Popov > --- >

[PATCH v3] powerpc/e6500: hw tablewalk: make sure we invalidate and write to the same tlb entry

2015-10-22 Thread Kevin Hao
In order to workaround Erratum A-008139, we have to invalidate the tlb entry with tlbilx before overwriting. Due to the performance consideration, we don't add any memory barrier when acquire/release the tcd lock. This means the two load instructions for esel_next do have the possibility to return

Re: [PATCH v4 1/2] powerpc/512x: add LocalPlus Bus FIFO device driver

2015-10-22 Thread Anatolij Gustschin
On Mon, 12 Oct 2015 00:08:01 +0300 Alexander Popov wrote: > This driver for Freescale MPC512x LocalPlus Bus FIFO (called SCLPC > in the Reference Manual) allows Direct Memory Access transfers > between RAM and peripheral devices on LocalPlus Bus. > > Signed-off-by:

Re: [RESEND PATCH 1/2] powerpc: platforms: mpc52xx_lpbfifo: Fix module autoload for OF platform driver

2015-10-22 Thread Anatolij Gustschin
On Tue, 20 Oct 2015 16:04:12 +0100 Luis de Bethencourt wrote: > From: Luis de Bethencourt > > This platform driver has a OF device ID table but the OF module > alias information is not created so module autoloading won't work. > > Signed-off-by: