Chen, Justin | 10 Sep 02:49
Picon
Favicon

RE: [PATCH] acpiphp: Match the variable types for ia64

>> On the ia64, since the "unsigned long" is u64, the above
>mismatch may cause the problem if the _SUN returns the value
>in 64 bits.
>>
>> The following patch change the definition in acpiphp.h to
>match the type of sun with other places.
>
>Does HP intend to have slot numbers larger than 4 billion?
>The ACPI 3.0 spec is mute on how large the 'integer' returned
>from _SUN is -- presumably it can be 64 bit.  But if we're
>going to permit _SUN to return 64-bit, we should make it u64
>everywhere, not just 64-bit for 64-bit machines.
>

HP has a new developing system does report the _SUN in 64 bits.  That's why the mismatch shows up.  I can try to
persuade the FW not to use the 64 bits.  But isn't this patch making more sense to have the variables in same type?

Justin

Gmane