On Thursday 05 February 2015 10:38:16 Jassi Brar wrote: > Passing pointer to a structure on stack will be automatically punished > as a random crash/behavior. > We just need to make sure no client passes a value directly in 'void > *data', right? Yes. > > I know typedef's are frowned upon, but how bad is the following option? > typedef void* mbox_data_info > int mbox_send_message(struct mbox_chan *chan, mbox_data_info data); I don't see how that would help. I believe typedefs for pointer types are particularly bad because they hide what is going on. If we want to enforce the use of pointers to point to data, having the pointer in the function prototype should be enough as a hint to driver developers ;-) Arnd -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html