Re: [PATCH 18/27] powerpc, smpboot: Use generic SMP booting infrastructure

2012-06-01 Thread Paul Mackerras
On Fri, Jun 01, 2012 at 02:44:23PM +0530, Srivatsa S. Bhat wrote: > From: Nikunj A. Dadhania > > Convert powerpc to use the generic framework to boot secondary CPUs. > > Signed-off-by: Nikunj A. Dadhania Acked-by: Paul Mackerras ___ Linuxppc-dev mai

Serial RAPID IO kernel hang on maintenance read transaction

2012-06-01 Thread Proicou, Mike
I've been struggling with a kernel hang during bootup + enumeration of a Rapid IO system. My current system contains a N.A.T MCH (using the IDT/Tundra Tsi 578 switch) and a Vadatech AMC719 card using the Freescale P4080 processor. There will be other cards added to the system, but I'm testing

Re: [PATCH v5 5/5] powerpc/85xx: add support to JOG feature using cpufreq interface

2012-06-01 Thread Scott Wood
On 05/11/2012 06:53 AM, Zhao Chenhui wrote: > Some 85xx silicons like MPC8536 and P1022 have a JOG feature, which provides > a dynamic mechanism to lower or raise the CPU core clock at runtime. Is there a reason P1023 isn't supported? > This patch adds the support to change CPU frequency using th

Re: [RFC] [PATCH] powerpc: Add MSR_DE to MSR_KERNEL

2012-06-01 Thread Benjamin Herrenschmidt
On Fri, 2012-06-01 at 17:42 -0500, Scott Wood wrote: > On 06/01/2012 05:30 PM, Benjamin Herrenschmidt wrote: > > BTW. My point of view is that this whole business about MSR:DE is a HW > > design bug. There should be -no- (absolutely 0) interaction between the > > SW state and the HW debugger for no

Re: [RFC] [PATCH] powerpc: Add MSR_DE to MSR_KERNEL

2012-06-01 Thread Scott Wood
On 06/01/2012 05:30 PM, Benjamin Herrenschmidt wrote: > BTW. My point of view is that this whole business about MSR:DE is a HW > design bug. There should be -no- (absolutely 0) interaction between the > SW state and the HW debugger for normal operations unless the user of > the debugger explicitly

Re: [RFC] [PATCH] powerpc: Add MSR_DE to MSR_KERNEL

2012-06-01 Thread Benjamin Herrenschmidt
On Fri, 2012-06-01 at 11:28 -0500, Scott Wood wrote: > > It's not really about CodeWarrior -- it's needed for any external > debug > on these chips. > > Those chips are commercial products too, BTW. :-) As long as it's not code to specifically interact with the CW software it's ok. I don't have

Re: power management patch set for mpc85xx

2012-06-01 Thread Benjamin Herrenschmidt
On Sat, 2012-06-02 at 08:17 +1000, Benjamin Herrenschmidt wrote: > On Fri, 2012-06-01 at 10:29 +, Zhao Chenhui-B35336 wrote: > > Hi Ben and Paul, > > > > I am sorry to trouble you. It seems that Kumar is busy recently. > > > > Could you have a review on the following patches? These patches >

Re: power management patch set for mpc85xx

2012-06-01 Thread Benjamin Herrenschmidt
On Fri, 2012-06-01 at 10:29 +, Zhao Chenhui-B35336 wrote: > Hi Ben and Paul, > > I am sorry to trouble you. It seems that Kumar is busy recently. > > Could you have a review on the following patches? These patches > implement the power management support on MPC85xx platform. At this point, I

Re: [PATCH v5 4/5] fsl_pmc: Add API to enable device as wakeup event source

2012-06-01 Thread Scott Wood
On 05/11/2012 06:53 AM, Zhao Chenhui wrote: > Add APIs for setting wakeup source and lossless Ethernet in low power modes. > These APIs can be used by wake-on-packet feature. > > Signed-off-by: Dave Liu > Signed-off-by: Li Yang > Signed-off-by: Jin Qing > Signed-off-by: Zhao Chenhui > --- > a

Re: [PATCH v5 3/5] powerpc/85xx: add sleep and deep sleep support

2012-06-01 Thread Scott Wood
On 05/11/2012 06:53 AM, Zhao Chenhui wrote: > From: Li Yang > > In sleep PM mode, the clocks of e500 core and unused IP blocks is > turned off. IP blocks which are allowed to wake up the processor > are still running. > > Some Freescale chips like MPC8536 and P1022 has deep sleep PM mode > in ad

Re: [PATCH v5 2/5] powerpc/85xx: add HOTPLUG_CPU support

2012-06-01 Thread Scott Wood
On 05/11/2012 06:53 AM, Zhao Chenhui wrote: \> +#if defined(CONFIG_FSL_BOOKE) || defined(CONFIG_6xx) > +extern void __flush_disable_L1(void); > +#endif Prototypes aren't normally guarded by ifdefs. > +static void __cpuinit smp_85xx_mach_cpu_die(void) > +{ > + unsigned int cpu = smp_processor_

Re: [PATCH 03/27] smpboot: Define and use cpu_state per-cpu variable in generic code

2012-06-01 Thread David Daney
On 06/01/2012 02:10 AM, Srivatsa S. Bhat wrote: The per-cpu variable cpu_state is used in x86 and also used in other architectures, to track the state of the cpu during bringup and hotplug. Pull it out into generic code. Cc: Tony Luck Cc: Fenghua Yu Cc: Ralf Baechle Cc: Benjamin Herrenschmidt Cc

Re: [RFC] [PATCH] powerpc: Add MSR_DE to MSR_KERNEL

2012-06-01 Thread Scott Wood
On 06/01/2012 04:14 AM, Benjamin Herrenschmidt wrote: > On Wed, 2012-05-30 at 09:26 -0400, Bob Cochran wrote: >> I believe that additional patches are required for CodeWarrior to >> work >> properly (e.g., assembly start up). I think the patches should come >> from Freescale. For whatever reaso

Re: [PATCH v5 1/5] powerpc/85xx: implement hardware timebase sync

2012-06-01 Thread Scott Wood
On 05/11/2012 06:53 AM, Zhao Chenhui wrote: > #ifdef CONFIG_KEXEC > +static struct ccsr_guts __iomem *guts; > +static u64 timebase; > +static int tb_req; > +static int tb_valid; > + > +static void mpc85xx_timebase_freeze(int freeze) Why is this under CONFIG_KEXEC? It'll also be needed for CPU ho

Re: power management patch set for mpc85xx

2012-06-01 Thread Kumar Gala
On Jun 1, 2012, at 5:29 AM, Zhao Chenhui-B35336 wrote: > Hi Ben and Paul, > > I am sorry to trouble you. It seems that Kumar is busy recently. > > Could you have a review on the following patches? These patches > implement the power management support on MPC85xx platform. > > http://patchwork.

RE: kernel panic during kernel module load (powerpc specific part)

2012-06-01 Thread Wrobel Heinz-R39252
> > I believe that the basic premise is that you should provide a directly > > reachable copy of the save/rstore functions, even if this means that > you need several copies of the functions. > > I just fixed a very similar problem with grub2 in fact. It was using r0 > and trashing the saved LR th

Re: [RFC] [PATCH] powerpc: Add MSR_DE to MSR_KERNEL

2012-06-01 Thread Joakim Tjernlund
Benjamin Herrenschmidt wrote on 2012/06/01 11:14:49: > > On Wed, 2012-05-30 at 09:26 -0400, Bob Cochran wrote: > > I believe that additional patches are required for CodeWarrior to > > work > > properly (e.g., assembly start up). I think the patches should come > > from Freescale. For whatever r

[PATCH 03/27] smpboot: Define and use cpu_state per-cpu variable in generic code

2012-06-01 Thread Srivatsa S. Bhat
The per-cpu variable cpu_state is used in x86 and also used in other architectures, to track the state of the cpu during bringup and hotplug. Pull it out into generic code. Cc: Tony Luck Cc: Fenghua Yu Cc: Ralf Baechle Cc: Benjamin Herrenschmidt Cc: Paul Mundt Cc: Chris Metcalf Cc: Thomas Gl

Re: Re[2]: [RFC] [PATCH] powerpc: Add MSR_DE to MSR_KERNEL

2012-06-01 Thread Joakim Tjernlund
Benjamin Herrenschmidt wrote on 2012/06/01 11:12:51: > > On Thu, 2012-05-31 at 11:05 +0200, Joakim Tjernlund wrote: > > Abatron Support wrote on 2012/05/30 14:08:26: > > > > > > >> I have tested this briefly with BDI2000 on P2010(e500) and > > > >> it works for me. I don't know if there are any b

power management patch set for mpc85xx

2012-06-01 Thread Zhao Chenhui-B35336
Hi Ben and Paul, I am sorry to trouble you. It seems that Kumar is busy recently. Could you have a review on the following patches? These patches implement the power management support on MPC85xx platform. http://patchwork.ozlabs.org/patch/158484/ http://patchwork.ozlabs.org/patch/158485/ http:/

[PATCH] powerpc/time: Sanity check of decrementer expiration is necessary

2012-06-01 Thread Paul Mackerras
This reverts 68568add2c ("powerpc/time: Remove unnecessary sanity check of decrementer expiration"). We do need to check whether we have reached the expiration time of the next event, because we sometimes get an early decrementer interrupt, most notably when we set the decrementer to 1 in arch_irq

[PATCH 18/27] powerpc, smpboot: Use generic SMP booting infrastructure

2012-06-01 Thread Srivatsa S. Bhat
From: Nikunj A. Dadhania Convert powerpc to use the generic framework to boot secondary CPUs. Signed-off-by: Nikunj A. Dadhania Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: Thomas Gleixner Cc: Yong Zhang Cc: Paul Gortmaker Cc: linuxppc-dev@lists.ozlabs.org Signed-off-by: Srivatsa S. B

Re: kernel panic during kernel module load (powerpc specific part)

2012-06-01 Thread Benjamin Herrenschmidt
On Thu, 2012-05-31 at 13:04 +0200, Gabriel Paubert wrote: > I believe that the basic premise is that you should provide a directly > reachable copy > of the save/rstore functions, even if this means that you need several copies > of the functions. I just fixed a very similar problem with grub2

Re: [RFC] [PATCH] powerpc: Add MSR_DE to MSR_KERNEL

2012-06-01 Thread Benjamin Herrenschmidt
On Wed, 2012-05-30 at 09:26 -0400, Bob Cochran wrote: > I believe that additional patches are required for CodeWarrior to > work > properly (e.g., assembly start up). I think the patches should come > from Freescale. For whatever reason, they include them in their SDK, > but haven't submitted

Re: Re[2]: [RFC] [PATCH] powerpc: Add MSR_DE to MSR_KERNEL

2012-06-01 Thread Benjamin Herrenschmidt
On Thu, 2012-05-31 at 11:05 +0200, Joakim Tjernlund wrote: > Abatron Support wrote on 2012/05/30 14:08:26: > > > > >> I have tested this briefly with BDI2000 on P2010(e500) and > > >> it works for me. I don't know if there are any bad side effects, > > >> therfore > > >> this RFC. > > > > > We use