On Sun, Nov 19, 2017 at 12:31:45AM +0000, Diego Elio Pettenò wrote: > Please do not drop the explicit documentation of the diff. Looking at what > a driver does in three years is not going to be obvious, whether you know > HID or not. I'm sure there are situations where this is true but I've just gone through the first six HID drivers in the order that grep found them in when I searched for "report_fixup" and in all but one I was either informed of the issue and the fix using a short and concise comment, annotations on the replacement descriptor or the commit message. The one report_fixup which was unclear had been moved out of hid-input-quirks.c (when it was a thing) and I didn't actually feel like finding its original commit message, it could probably have been clarified by a rather simple comment. Even the existing report_fixup of the BM084 has a one line comment which entirely explains what the problem is. I really don't think a big diagram showing the difference between two report descriptors, which may or may not completely overlap with the report descriptors of other devices exhibiting a similar issue, is necessary. Please also note that the documentation is not gone, it's just shortened and generalised, there is still more than enough information to work out exactly what the report change is. -- Tomasz Kramkowski | GPG: 40B037BA0A5B8680 | Web: https://the-tk.com/ -- 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
