Am Donnerstag, 28. Juli 2011, 18:08:27 schrieb David Herrmann: > +static void handler_accel(struct wiimote_data *wdata, const __u8 *payload) > +{ > + __u16 x, y, z; > + > + if (!(wdata->state.flags & WIIPROTO_FLAG_ACCEL)) > + return; > + > + /* > + * payload is: BB BB XX YY ZZ > + * Buttons data contains LSBs > + */ > + > + x = payload[2] << 2; > + y = payload[3] << 2; > + z = payload[4] << 2; > + > + x |= (payload[0] >> 5) & 0x3; > + y |= (payload[1] >> 4) & 0x2; > + z |= (payload[1] >> 5) & 0x2; Could you make the comments a bit clearer. Those last lines are impossible to understand. Regards Oliver -- - - - SUSE LINUX Products GmbH, GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer, HRB 16746 (AG Nürnberg) Maxfeldstraße 5 90409 Nürnberg Germany - - - -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html