Andy Green wrote: > I think maybe you misunderstood this -- the only edit in there
I guess I wasn't clear :) I spotted the GLAMO entires while looking at your patch, that's why I put this there as a reminder, for further study. >> should probably be "struct _fiq_ipc". In general, I think we should > > I think you missed an edit on that line :-) Ah yes :) > Fair enough, with the attached patch I changed it to use fiq_ipc_t which > is the method used in plenty of other places in the kernel Ick. "struct foo_t" ain't right. There's a lot of it in the Atheros code, but that's all. (And that one has special dispension for not following any coding style right now, because we all know that it'll eventually get torn to pieces by the sharks on linux-kernel anyway ;-) "foo_t" is actually reserved by POSIX, so even the common practice of mimicking the way how "standard" typedef'ed types are named is questionable at best. (And clearly wrong in user space.) So, how about just "struct fiq_ipc" ? - Werner
