On 2015年03月12日 17:51, Lorenzo Pieralisi wrote:
On Wed, Mar 11, 2015 at 12:39:39PM +0000, Hanjun Guo wrote:
CPU hardware ID (phys_id) is defined as u32 in structure acpi_processor, but phys_id is used as int in acpi processor driver, so it will lead to some inconsistence for the drivers.
Furthermore, to cater for ACPI arch ports that implement 64 bits CPU ids a generic CPU physical id type is required.
So introduce typedef u32 phys_cpuid_t for x86 and ia64, and introduce a macro CPU_PHYS_ID_INVALID as (u32)(-1), use phys_cpuid_t when phys_id defined in acpi processor driver, and replace CPU_PHYS_ID_INVALID as -1 for phys_id, this will solve the inconsistence in acpi processor driver, and will prepare for the ACPI on ARM64 for the 64 bit CPU hardware ID in the following patch.
Nit: Commit log above is stale (since you moved the phys_cpuid_t to common declaration for all archs that do not define PHYS_CPUID_INVALID and the macro name itself changed) so you should rewrite it before
Oops, I will send a updated patch only for this one, good catch.
asking to merge it. Furthermore you should not describe how you implemented the change, you should describe why.
I'm prefer writing the commit log as following way (not always) :) - why the patch is needed - what the patch implements - How the patch is implemented
Other than that:
Acked-by: Lorenzo Pieralisi lorenzo.pieralisi@arm.com
Thanks!
Hanjun