Re: [PATCH v2] powerpc: don't enable cpu hotplug on mpic-based pseries

2007-10-10 Thread Olof Johansson
On Thu, Oct 11, 2007 at 03:52:04PM +1000, Paul Mackerras wrote: > Olof Johansson writes: > > > Don't allow cpu hotplug on systems lacking XICS interrupt controller, > > since current code is hardcoded for it. > ... > > + for (np = NULL; (np = of_find_node_by_name(np, > > +

Re: [PATCH v2] powerpc: don't enable cpu hotplug on mpic-based pseries

2007-10-10 Thread Paul Mackerras
Olof Johansson writes: > Don't allow cpu hotplug on systems lacking XICS interrupt controller, > since current code is hardcoded for it. ... > + for (np = NULL; (np = of_find_node_by_name(np, > +"interrupt-controller"));) { Looks like

Re: [PATCH v2] powerpc: don't enable cpu hotplug on mpic-based pseries

2007-10-10 Thread Olof Johansson
On Wed, Oct 10, 2007 at 05:08:44AM -0500, Milton Miller wrote: > Olof's patch searched the device-tree again, looking for an mpic. This > code instead checks that we found an xics the first time by checking the > init function. I'm glad you find the kernel so perfect that your best use of time

Re: [PATCH v2] powerpc: don't enable cpu hotplug on mpic-based pseries

2007-10-10 Thread Milton Miller
Don't allow cpu hotplug on pSeries systems lacking XICS interrupt controller, since current code is hardcoded to call xics routines. Signed-off-by: Milton Miller <[EMAIL PROTECTED]> -- Olof's patch searched the device-tree again, looking for an mpic. This code instead checks that we found an

Re: [PATCH v2] powerpc: don't enable cpu hotplug on mpic-based pseries

2007-10-10 Thread Milton Miller
Don't allow cpu hotplug on pSeries systems lacking XICS interrupt controller, since current code is hardcoded to call xics routines. Signed-off-by: Milton Miller [EMAIL PROTECTED] -- Olof's patch searched the device-tree again, looking for an mpic. This code instead checks that we found an

Re: [PATCH v2] powerpc: don't enable cpu hotplug on mpic-based pseries

2007-10-10 Thread Olof Johansson
On Wed, Oct 10, 2007 at 05:08:44AM -0500, Milton Miller wrote: Olof's patch searched the device-tree again, looking for an mpic. This code instead checks that we found an xics the first time by checking the init function. I'm glad you find the kernel so perfect that your best use of time is

Re: [PATCH v2] powerpc: don't enable cpu hotplug on mpic-based pseries

2007-10-10 Thread Paul Mackerras
Olof Johansson writes: Don't allow cpu hotplug on systems lacking XICS interrupt controller, since current code is hardcoded for it. ... + for (np = NULL; (np = of_find_node_by_name(np, +interrupt-controller));) { Looks like

Re: [PATCH v2] powerpc: don't enable cpu hotplug on mpic-based pseries

2007-10-10 Thread Olof Johansson
On Thu, Oct 11, 2007 at 03:52:04PM +1000, Paul Mackerras wrote: Olof Johansson writes: Don't allow cpu hotplug on systems lacking XICS interrupt controller, since current code is hardcoded for it. ... + for (np = NULL; (np = of_find_node_by_name(np, +

[PATCH v2] powerpc: don't enable cpu hotplug on mpic-based pseries

2007-10-09 Thread Olof Johansson
Don't allow cpu hotplug on systems lacking XICS interrupt controller, since current code is hardcoded for it. Signed-off-by: Olof Johansson <[EMAIL PROTECTED]> --- On Wed, Oct 10, 2007 at 10:18:26AM +1000, Stephen Rothwell wrote: > > + struct device_node *np; > > + const char *typep; > >

[PATCH v2] powerpc: don't enable cpu hotplug on mpic-based pseries

2007-10-09 Thread Olof Johansson
Don't allow cpu hotplug on systems lacking XICS interrupt controller, since current code is hardcoded for it. Signed-off-by: Olof Johansson [EMAIL PROTECTED] --- On Wed, Oct 10, 2007 at 10:18:26AM +1000, Stephen Rothwell wrote: + struct device_node *np; + const char *typep; + +