On Thu, Jun 18, 2020 at 08:28:50PM -0400, Changming Liu wrote: > This patch series changes all appropriate instances > of signed char arrays or buffer to unsigned char. > > For each patch, if changing one variable to u8 > involves its callers/callees, then those changes > are included in that patch as well. > > This doesn't apply to ioctl functions, since > the types for buffer of ioctl-like functions > needs to be char* instead of u8* to keep > the compiler happy. Why is that? What is forcing those types to be that way? These are all self-contained in the driver itself, so they should be able to be changed, right? Do you have an example of a function that you want to change but somehow can not? thanks, greg k-h