On 11/2/2010 1:19 PM, Randy Dunlap wrote: > On Tue, 2 Nov 2010 13:05:34 -0400 Chris Metcalf wrote: >> This change enables the Linux driver to access i2c devices via >> the Tilera hypervisor's virtualized API. >> >> Note that the arch/tile/include/hv/ headers are "upstream" headers >> that likely benefit less from LKML review. >> >> [...] >> +++ b/arch/tile/include/hv/drv_i2cm_intf.h >> [...] >> +/** > In kernel source code, "/**" is used to begin kernel-doc notation blocks. > Please don't use it for non-kernel-doc comments. > > (multiple times...) These headers are essentially "upstream" for the Linux port; they are owned by the Tilera hypervisor. The "/**" comments in those files are Doxygen comments used to generate standalone API documentation for the hypervisor. The same answer applies to your comment later in the file where you pointed out some brace-positioning issues and the use of typedefs, since the coding style for those headers is different the Linux coding style. (Note also that these headers are using two-space indents rather than tab indents.) To avoid stylistic clash, we could have released the hypervisor headers as a separate tarball (somewhere) and required that the kernel build take an environment variable or some such to locate them, but it seemed simpler to incorporate them into the Tilera Linux code so that the kernel can build in a reasonably self-contained way. The hypervisor header model is that the client can use any internally consistent set of hypervisor headers and just register the version number associated with that set of headers when starting up. -- Chris Metcalf, Tilera Corp. http://www.tilera.com -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
