Am 12.12.2012 10:51, schrieb Lars-Peter Clausen:

Looks good, but as I wrote during the last review the __devinits need to go.
A few other suggerstions online

Oh, have forgotten it. Ok, will make a v4, changing the other few things too.

+       case HID_USAGE_SENSOR_TIME_MONTH:
+               time_buf->tm_mon = --*(u8 *)raw_data;

Time sensors are sending the month in the range 1-12, not 0-11.

Regards,

Alexander

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to