> > Why not skip the pointer here? > > well, the idea was to keep the memory footprint low. As these values > are only needed at init, then I freed them once I finished using them. > I can of course skip the pointer, but in that case, wouldn't the > struct declaration be worthless? My bad, I misread the placement of the free() statement. I was also concerned about memory, since HID is big enough a memory hog as it is. Barring the added complexity of this patch, it now makes more sense. Acked-by: Henrik Rydberg <rydberg@xxxxxxxxxxx> Thanks, Henrik -- 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