Shouldn't this be: if (AccessByteWidth < 256) { ObjDesc->CommonField.AccessByteWidth = (UINT8) AccessByteWidth; } >-----Original Message----- >From: Lin, Ming M >Sent: Wednesday, May 26, 2010 12:32 AM >To: Alexey Starikovskiy >Cc: Moore, Robert; Len Brown; Linux-acpi@xxxxxxxxxxxxxxx >Subject: Re: [PATCH] ACPICA: Handle large field apertures. > >On Wed, 2010-05-26 at 15:30 +0800, Alexey Starikovskiy wrote: >> Hi Ming, >> >> 26.05.2010 06:49, Lin Ming пишет: >> > Better to test AccessByteWidth as below, >> Why not just 'if (AccessByteWidth > 255)' ? > >OK, that's more clear. > >> > >> > + if (!(AccessByteWidth & ~0xFF)) { >> > >> -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html