Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-12 Thread Jon Masters
On 09/11/2014 10:15 AM, Will Deacon wrote: > On Tue, Sep 09, 2014 at 05:04:15PM +0100, Jon Masters wrote: >> On 09/09/2014 12:00 PM, Hanjun Guo wrote: >>> No, you didn't miss it, I'm still working on the new version, sorry I didn't >>> clarify that in my previous email. >> >> Thanks. If you could

Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-12 Thread Jon Masters
On 09/11/2014 10:15 AM, Will Deacon wrote: On Tue, Sep 09, 2014 at 05:04:15PM +0100, Jon Masters wrote: On 09/09/2014 12:00 PM, Hanjun Guo wrote: No, you didn't miss it, I'm still working on the new version, sorry I didn't clarify that in my previous email. Thanks. If you could copy me on

Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-11 Thread Will Deacon
On Tue, Sep 09, 2014 at 05:04:15PM +0100, Jon Masters wrote: > On 09/09/2014 12:00 PM, Hanjun Guo wrote: > > No, you didn't miss it, I'm still working on the new version, sorry I didn't > > clarify that in my previous email. > > Thanks. If you could copy me on the next posting that would rock. In

Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-11 Thread Grant Likely
On Mon, 1 Sep 2014 22:57:47 +0800, Hanjun Guo wrote: > MADT contains the information for MPIDR which is essential for > SMP initialization, parse the GIC cpu interface structures to > get the MPIDR value and map it to cpu_logical_map(), and add > enabled cpu with valid MPIDR into

Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-11 Thread Grant Likely
On Mon, 1 Sep 2014 22:57:47 +0800, Hanjun Guo hanjun@linaro.org wrote: MADT contains the information for MPIDR which is essential for SMP initialization, parse the GIC cpu interface structures to get the MPIDR value and map it to cpu_logical_map(), and add enabled cpu with valid MPIDR

Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-11 Thread Will Deacon
On Tue, Sep 09, 2014 at 05:04:15PM +0100, Jon Masters wrote: On 09/09/2014 12:00 PM, Hanjun Guo wrote: No, you didn't miss it, I'm still working on the new version, sorry I didn't clarify that in my previous email. Thanks. If you could copy me on the next posting that would rock. In a few

Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-09 Thread Jon Masters
On 09/09/2014 12:52 PM, Lorenzo Pieralisi wrote: > On Thu, Sep 04, 2014 at 04:29:15PM +0100, Hanjun Guo wrote: + + /* Get cpu_ops include the boot CPU */ + cpu_ops[cpu] = cpu_get_ops("psci"); + if (!cpu_ops[cpu]) + return -EINVAL; + + /* CPU 0 was

Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-09 Thread Jon Masters
On 09/09/2014 12:52 PM, Lorenzo Pieralisi wrote: > On Thu, Sep 04, 2014 at 04:29:15PM +0100, Hanjun Guo wrote: >> Hi Lorenzo, + if (!enabled) + return -EINVAL; + + if (enabled_cpus >= NR_CPUS) { + pr_warn("NR_CPUS limit of %d reached, Processor

Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-09 Thread Lorenzo Pieralisi
On Thu, Sep 04, 2014 at 04:29:15PM +0100, Hanjun Guo wrote: > Hi Lorenzo, > > On 2014?09?04? 01:21, Lorenzo Pieralisi wrote: > > On Mon, Sep 01, 2014 at 03:57:47PM +0100, Hanjun Guo wrote: > >> MADT contains the information for MPIDR which is essential for > >> SMP initialization, parse the GIC

Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-09 Thread Hanjun Guo
On 2014年09月10日 00:04, Jon Masters wrote: > On 09/09/2014 12:00 PM, Hanjun Guo wrote: >> On 2014年09月09日 13:44, Jon Masters wrote: >>> On 09/09/2014 12:57 AM, Hanjun Guo wrote: Hi Jon, On 2014年09月09日 12:23, Jon Masters wrote: > On 09/01/2014 10:57 AM, Hanjun Guo wrote: >> MADT

Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-09 Thread Jon Masters
On 09/09/2014 12:00 PM, Hanjun Guo wrote: > On 2014年09月09日 13:44, Jon Masters wrote: >> On 09/09/2014 12:57 AM, Hanjun Guo wrote: >>> Hi Jon, >>> >>> On 2014年09月09日 12:23, Jon Masters wrote: On 09/01/2014 10:57 AM, Hanjun Guo wrote: > MADT contains the information for MPIDR which is

Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-09 Thread Hanjun Guo
On 2014年09月09日 13:44, Jon Masters wrote: > On 09/09/2014 12:57 AM, Hanjun Guo wrote: >> Hi Jon, >> >> On 2014年09月09日 12:23, Jon Masters wrote: >>> On 09/01/2014 10:57 AM, Hanjun Guo wrote: MADT contains the information for MPIDR which is essential for SMP initialization, parse the GIC

Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-09 Thread Hanjun Guo
On 2014年09月09日 13:44, Jon Masters wrote: On 09/09/2014 12:57 AM, Hanjun Guo wrote: Hi Jon, On 2014年09月09日 12:23, Jon Masters wrote: On 09/01/2014 10:57 AM, Hanjun Guo wrote: MADT contains the information for MPIDR which is essential for SMP initialization, parse the GIC cpu interface

Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-09 Thread Jon Masters
On 09/09/2014 12:00 PM, Hanjun Guo wrote: On 2014年09月09日 13:44, Jon Masters wrote: On 09/09/2014 12:57 AM, Hanjun Guo wrote: Hi Jon, On 2014年09月09日 12:23, Jon Masters wrote: On 09/01/2014 10:57 AM, Hanjun Guo wrote: MADT contains the information for MPIDR which is essential for SMP

Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-09 Thread Hanjun Guo
On 2014年09月10日 00:04, Jon Masters wrote: On 09/09/2014 12:00 PM, Hanjun Guo wrote: On 2014年09月09日 13:44, Jon Masters wrote: On 09/09/2014 12:57 AM, Hanjun Guo wrote: Hi Jon, On 2014年09月09日 12:23, Jon Masters wrote: On 09/01/2014 10:57 AM, Hanjun Guo wrote: MADT contains the information for

Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-09 Thread Lorenzo Pieralisi
On Thu, Sep 04, 2014 at 04:29:15PM +0100, Hanjun Guo wrote: Hi Lorenzo, On 2014?09?04? 01:21, Lorenzo Pieralisi wrote: On Mon, Sep 01, 2014 at 03:57:47PM +0100, Hanjun Guo wrote: MADT contains the information for MPIDR which is essential for SMP initialization, parse the GIC cpu

Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-09 Thread Jon Masters
On 09/09/2014 12:52 PM, Lorenzo Pieralisi wrote: On Thu, Sep 04, 2014 at 04:29:15PM +0100, Hanjun Guo wrote: Hi Lorenzo, + if (!enabled) + return -EINVAL; + + if (enabled_cpus = NR_CPUS) { + pr_warn(NR_CPUS limit of %d reached, Processor %d/0x%llx ignored.\n, +

Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-09 Thread Jon Masters
On 09/09/2014 12:52 PM, Lorenzo Pieralisi wrote: On Thu, Sep 04, 2014 at 04:29:15PM +0100, Hanjun Guo wrote: + + /* Get cpu_ops include the boot CPU */ + cpu_ops[cpu] = cpu_get_ops(psci); + if (!cpu_ops[cpu]) + return -EINVAL; + + /* CPU 0 was already initialized */ + if

Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-08 Thread Jon Masters
On 09/09/2014 12:57 AM, Hanjun Guo wrote: > Hi Jon, > > On 2014年09月09日 12:23, Jon Masters wrote: >> On 09/01/2014 10:57 AM, Hanjun Guo wrote: >>> MADT contains the information for MPIDR which is essential for >>> SMP initialization, parse the GIC cpu interface structures to >>> get the MPIDR

Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-08 Thread Jon Masters
On 09/09/2014 01:11 AM, Hanjun Guo wrote: > On 2014年09月09日 12:29, Jon Masters wrote: >> Hi Hanjun, Lorenzo, > > Hi Jon, > >> >> Resending due to my mail client removing list CCs...sorry about that. >> >> On 09/04/2014 11:29 AM, Hanjun Guo wrote: >> > + } else { > + /* Fist GICC

Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-08 Thread Hanjun Guo
On 2014年09月09日 12:29, Jon Masters wrote: > Hi Hanjun, Lorenzo, Hi Jon, > > Resending due to my mail client removing list CCs...sorry about that. > > On 09/04/2014 11:29 AM, Hanjun Guo wrote: > + } else { + /* Fist GICC entry must be BSP as ACPI spec said */ >>> s/Fist/First/

Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-08 Thread Hanjun Guo
Hi Jon, On 2014年09月09日 12:23, Jon Masters wrote: > On 09/01/2014 10:57 AM, Hanjun Guo wrote: >> MADT contains the information for MPIDR which is essential for >> SMP initialization, parse the GIC cpu interface structures to >> get the MPIDR value and map it to cpu_logical_map(), and add >>

Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-08 Thread Jon Masters
Hi Hanjun, Lorenzo, Resending due to my mail client removing list CCs...sorry about that. On 09/04/2014 11:29 AM, Hanjun Guo wrote: >>> + } else { >>> + /* Fist GICC entry must be BSP as ACPI spec said */ >> s/Fist/First/ >> >>> + if (cpu_logical_map(0) != mpidr) { >>> +

Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-08 Thread Jon Masters
On 09/01/2014 10:57 AM, Hanjun Guo wrote: > MADT contains the information for MPIDR which is essential for > SMP initialization, parse the GIC cpu interface structures to > get the MPIDR value and map it to cpu_logical_map(), and add > enabled cpu with valid MPIDR into cpu_possible_map. > > ACPI

Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-08 Thread Jon Masters
On 09/01/2014 10:57 AM, Hanjun Guo wrote: MADT contains the information for MPIDR which is essential for SMP initialization, parse the GIC cpu interface structures to get the MPIDR value and map it to cpu_logical_map(), and add enabled cpu with valid MPIDR into cpu_possible_map. ACPI 5.1

Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-08 Thread Jon Masters
Hi Hanjun, Lorenzo, Resending due to my mail client removing list CCs...sorry about that. On 09/04/2014 11:29 AM, Hanjun Guo wrote: + } else { + /* Fist GICC entry must be BSP as ACPI spec said */ s/Fist/First/ + if (cpu_logical_map(0) != mpidr) { +

Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-08 Thread Hanjun Guo
Hi Jon, On 2014年09月09日 12:23, Jon Masters wrote: On 09/01/2014 10:57 AM, Hanjun Guo wrote: MADT contains the information for MPIDR which is essential for SMP initialization, parse the GIC cpu interface structures to get the MPIDR value and map it to cpu_logical_map(), and add enabled cpu

Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-08 Thread Hanjun Guo
On 2014年09月09日 12:29, Jon Masters wrote: Hi Hanjun, Lorenzo, Hi Jon, Resending due to my mail client removing list CCs...sorry about that. On 09/04/2014 11:29 AM, Hanjun Guo wrote: + } else { + /* Fist GICC entry must be BSP as ACPI spec said */ s/Fist/First/ + if

Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-08 Thread Jon Masters
On 09/09/2014 01:11 AM, Hanjun Guo wrote: On 2014年09月09日 12:29, Jon Masters wrote: Hi Hanjun, Lorenzo, Hi Jon, Resending due to my mail client removing list CCs...sorry about that. On 09/04/2014 11:29 AM, Hanjun Guo wrote: + } else { + /* Fist GICC entry must be BSP as ACPI

Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-08 Thread Jon Masters
On 09/09/2014 12:57 AM, Hanjun Guo wrote: Hi Jon, On 2014年09月09日 12:23, Jon Masters wrote: On 09/01/2014 10:57 AM, Hanjun Guo wrote: MADT contains the information for MPIDR which is essential for SMP initialization, parse the GIC cpu interface structures to get the MPIDR value and map it

Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-04 Thread Hanjun Guo
Hi Lorenzo, On 2014年09月04日 01:21, Lorenzo Pieralisi wrote: > On Mon, Sep 01, 2014 at 03:57:47PM +0100, Hanjun Guo wrote: >> MADT contains the information for MPIDR which is essential for >> SMP initialization, parse the GIC cpu interface structures to >> get the MPIDR value and map it to

Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-04 Thread Hanjun Guo
Hi Lorenzo, On 2014年09月04日 01:21, Lorenzo Pieralisi wrote: On Mon, Sep 01, 2014 at 03:57:47PM +0100, Hanjun Guo wrote: MADT contains the information for MPIDR which is essential for SMP initialization, parse the GIC cpu interface structures to get the MPIDR value and map it to

Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-03 Thread Lorenzo Pieralisi
On Mon, Sep 01, 2014 at 03:57:47PM +0100, Hanjun Guo wrote: > MADT contains the information for MPIDR which is essential for > SMP initialization, parse the GIC cpu interface structures to > get the MPIDR value and map it to cpu_logical_map(), and add > enabled cpu with valid MPIDR into

Re: [PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-03 Thread Lorenzo Pieralisi
On Mon, Sep 01, 2014 at 03:57:47PM +0100, Hanjun Guo wrote: MADT contains the information for MPIDR which is essential for SMP initialization, parse the GIC cpu interface structures to get the MPIDR value and map it to cpu_logical_map(), and add enabled cpu with valid MPIDR into

[PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-01 Thread Hanjun Guo
MADT contains the information for MPIDR which is essential for SMP initialization, parse the GIC cpu interface structures to get the MPIDR value and map it to cpu_logical_map(), and add enabled cpu with valid MPIDR into cpu_possible_map. ACPI 5.1 only has two explicit methods to boot up SMP, PSCI

[PATCH v3 09/17] ARM64 / ACPI: Parse MADT for SMP initialization

2014-09-01 Thread Hanjun Guo
MADT contains the information for MPIDR which is essential for SMP initialization, parse the GIC cpu interface structures to get the MPIDR value and map it to cpu_logical_map(), and add enabled cpu with valid MPIDR into cpu_possible_map. ACPI 5.1 only has two explicit methods to boot up SMP, PSCI