Re: [PATCH] SN: Add initial ACPI support

From: Jes Sorensen <>
Date: 2006-01-15 23:34:54
>>>>> "Keith" == Keith Owens <> writes:


Just a general comment on top of Keith's.

Keith> John Keller (on Sat, 14 Jan 2006 11:37:47 -0600 (CST)) wrote:
>> +inline uint64_t +sal_ioif_init(void)

Keith> Make that static with no inline, let gcc decide if it should be
Keith> inlined or not.

Please stick to using the preferred kernel internal types, u64/u32/u16
etc. I noticed you switch u64 to uint64_t in many places which is the
wrong way to go.

To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to
More majordomo info at
Received on Sun Jan 15 23:36:33 2006

This archive was generated by hypermail 2.1.8 : 2006-01-15 23:36:42 EST