Messages in this thread | | | Date | Tue, 08 May 2007 17:19:19 -0600 | From | Paul Fulghum <> | Subject | Re: [PATCH] synclink_gt add compat_ioctl |
| |
Arnd Bergmann wrote: > To solve this, you can to change include/linux/Kbuild to list > synclink.h as unifdef-y instead of header-y, and put the parts > that you don't want to be in user space inside of #ifdef __KERNEL__. > > Alternatively, you can put these kernel-internal definitions into > a private header file in drivers/char that does not get installed > in the first place. That would be particularly useful if you can > also move other parts of linux/synclink.h into the private header, > when they are not part of the external ABI.
Understood. That is the last piece to the puzzle.
I think the first approach would be better as synclink.h is all interface definitions. The kernel specific parts are in the individual synclink drivers (such as register definitions, etc). The only exception to this is now the ioctl32 structures and I would hate to break out a whole new header just for that.
Thanks again for your targeted and informative help. (Thanks to Andrew for your patience in dealing with a patch that never should have been submitted.)
-- Paul
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |