From mboxrd@z Thu Jan 1 00:00:00 1970 From: rpjday@crashcourse.ca (Robert P. J. Day) Date: Tue, 3 Feb 2015 08:44:50 -0500 (EST) Subject: why is "const u32 (*tab)[256]" not kerneldoc-able? In-Reply-To: <87bnlbuncq.fsf@nemi.mork.no> References: <87fvanuo9h.fsf@nemi.mork.no> <87bnlbuncq.fsf@nemi.mork.no> Message-ID: To: kernelnewbies@lists.kernelnewbies.org List-Id: kernelnewbies.lists.kernelnewbies.org On Tue, 3 Feb 2015, Bj?rn Mork wrote: > Bj?rn Mork writes: > > > Maybe the attached simple patch is good enough? Completely untested > > except for a single run against that file.... > > Ah, umh, no. That "const u32 (*)[256] tab" does not look correct: > > > crc32_le_generic(9) Kernel Hacker's Manual crc32_le_generic(9) > > NAME > crc32_le_generic - Calculate bitwise little-endian Ethernet AUTODIN II CRC32/CRC32C > > SYNOPSIS > u32 __pure crc32_le_generic (u32 crc, unsigned char const *p, size_t len, const u32 (*)[256] tab, u32 polynomial); > > ARGUMENTS > crc seed value for computation. ~0 for Ethernet, sometimes 0 for other uses, or the previous crc32/crc32c value > if computing incrementally. > > p pointer to buffer over which CRC32/CRC32C is run > > len length of buffer p > > tab little-endian Ethernet table > > polynomial CRC32/CRC32c LE polynomial > > > > I guess a proper fix is needed. actually, i just found where this is a known issue: http://www.spinics.net/lists/linux-doc/msg09364.html rday -- ======================================================================== Robert P. J. Day Ottawa, Ontario, CANADA http://crashcourse.ca Twitter: http://twitter.com/rpjday LinkedIn: http://ca.linkedin.com/in/rpjday ========================================================================