On Wed, Aug 20, 2008 at 10:45:47PM -0700, Jared Hulbert wrote: >+/****************************************************************************** >+ * >+ * axfs_copy_block_data >+ * >+ * Description: Helper function to read data from block device >+ * >+ * Parameters: >+ * (IN) sb - pointer to super block structure. >+ * >+ * (IN) dst_addr - pointer to buffer into which data is to be read. >+ * >+ * (IN) boffset - offset within block device >+ * >+ * (IN) len - length of data to be read >+ * >+ * Returns: >+ * 0 or error number >+ * >+ *****************************************************************************/ >+int axfs_copy_block(struct super_block *sb, void *dst_addr, u64 fsoffset, >+ u64 len) mismatch between documentation and implementation WRT the function name ;) >+{ >+ struct axfs_super *sbi = AXFS_SB(sb); >+ u64 boffset = AXFS_FSOFFSET_2_DEVOFFSET(sbi, fsoffset); >+ u64 blocks; >+ u64 blksize = sb->s_blocksize; >+ unsigned long dst; >+ unsigned long src; >+ sector_t block; >+ size_t bytes; >+ struct buffer_head *bh; >+ u64 copied = 0; >+ >+ if (len == 0) >+ return 0; >+ >+ blocks = len / blksize; >+ if ((len % blksize) > 0) >+ blocks += 1; >+ >+ while (copied < len) { >+ /* Explicit casting for ARM linker errors. */ did it try to emit some external div()? Is this still needed? -- To unsubscribe from this list: send the line "unsubscribe linux-embedded" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html