On Fri, Dec 04, 2015 at 08:15:04AM -0800, Andy Lutomirski wrote: > On Fri, Dec 4, 2015 at 12:39 AM, Pali Rohár <pali.rohar@xxxxxxxxx> wrote: > > On Thursday 03 December 2015 15:38:56 Darren Hart wrote: > >> Pali, this appears to have the one change you asked for (0x%x instead of %d). > >> I've added the Reviewed-by: Pali Rohár <pali.rohar@xxxxxxxxx> you provided > >> previously pending this change. > > > > Ok. > > > >> > + /* Uninitialized entries are 0 aka KEY_RESERVED. */ > >> > + u16 keycode = (bios_entry->keycode < > >> > + ARRAY_SIZE(bios_to_linux_keycode)) ? > >> > + bios_to_linux_keycode[bios_entry->keycode] : > >> > + KEY_RESERVED; > >> > + BUILD_BUG_ON(KEY_RESERVED != 0); > > > > Anyway, Darren what do you think is that BUILD_BUG_ON check needed? > > > > Sorry, my bad. IIRC Darren made that change, I made the other change, > and then I resent and we lost this one. > > Darren, want to fix it in your tree? If not, I'll include the > BUILD_BUG_ON deletion in v3. Thanks for the catch Pali. I've removed this in my tree. No need to resend. -- Darren Hart Intel Open Source Technology Center -- To unsubscribe from this list: send the line "unsubscribe platform-driver-x86" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
