yaffs Handle ENOSPC properly during object creation
[yaffs2.git] / yaffs_vfs_multi.c
index 7571b3e8c80e647db117c7bc829eb4efe3918b0b..acf43480b69bd13ad931be86d2e9364184f865f3 100644 (file)
@@ -1,7 +1,7 @@
 /*
  * YAFFS: Yet Another Flash File System. A NAND-flash specific file system.
  *
- * Copyright (C) 2002-2010 Aleph One Ltd.
+ * Copyright (C) 2002-2011 Aleph One Ltd.
  *   for Toby Churchill Ltd and Brightstar Engineering
  *
  * Created by Charles Manning <charles@aleph1.co.uk>
  * the VFS.
  *
  * Special notes:
- * >> 2.4: sb->u.generic_sbp points to the yaffs_dev_t associated with
+ * >> 2.4: sb->u.generic_sbp points to the struct yaffs_dev associated with
  *         this superblock
- * >> 2.6: sb->s_fs_info  points to the yaffs_dev_t associated with this
+ * >> 2.6: sb->s_fs_info  points to the struct yaffs_dev associated with this
  *         superblock
- * >> inode->u.generic_ip points to the associated yaffs_obj_t.
+ * >> inode->u.generic_ip points to the associated struct yaffs_obj.
  */
 
 /*
 
 #if (LINUX_VERSION_CODE >= KERNEL_VERSION(2, 6, 10))
 #define YAFFS_COMPILE_BACKGROUND
-#if (LINUX_VERSION_CODE >= KERNEL_VERSION(2,6, 23))
+#if (LINUX_VERSION_CODE >= KERNEL_VERSION(2, 6, 23))
 #define YAFFS_COMPILE_FREEZER
 #endif
 #endif
 
-#if (LINUX_VERSION_CODE >= KERNEL_VERSION(2,6,28))
+#if (LINUX_VERSION_CODE >= KERNEL_VERSION(2, 6, 28))
 #define YAFFS_COMPILE_EXPORTFS
 #endif
 
-#if (LINUX_VERSION_CODE > KERNEL_VERSION(2,6,35))
+#if (LINUX_VERSION_CODE > KERNEL_VERSION(2, 6, 35))
 #define YAFFS_USE_SETATTR_COPY
 #define YAFFS_USE_TRUNCATE_SETSIZE
 #endif
-#if (LINUX_VERSION_CODE > KERNEL_VERSION(2,6,35))
+#if (LINUX_VERSION_CODE > KERNEL_VERSION(2, 6, 35))
 #define YAFFS_HAS_EVICT_INODE
 #endif
 
-#if (LINUX_VERSION_CODE > KERNEL_VERSION(2,6,13))
+#if (LINUX_VERSION_CODE > KERNEL_VERSION(2, 6, 13))
 #define YAFFS_NEW_FOLLOW_LINK 1
 #else
 #define YAFFS_NEW_FOLLOW_LINK 0
 #define YPROC_ROOT  NULL
 #endif
 
-#if (LINUX_VERSION_CODE < KERNEL_VERSION(2,6,26))
+#if (LINUX_VERSION_CODE < KERNEL_VERSION(2, 6, 26))
 #define Y_INIT_TIMER(a)        init_timer(a)
 #else
 #define Y_INIT_TIMER(a)        init_timer_on_stack(a)
@@ -151,7 +151,7 @@ static uint32_t YCALCBLOCKS(uint64_t partition_size, uint32_t block_size)
 {
        uint64_t result = partition_size;
        do_div(result, block_size);
-       return (uint32_t)result;
+       return (uint32_t) result;
 }
 #else
 #define YCALCBLOCKS(s, b) ((s)/(b))
@@ -163,6 +163,7 @@ static uint32_t YCALCBLOCKS(uint64_t partition_size, uint32_t block_size)
 #include "yportenv.h"
 #include "yaffs_trace.h"
 #include "yaffs_guts.h"
+#include "yaffs_attribs.h"
 
 #include "yaffs_linux.h"
 
@@ -204,25 +205,25 @@ static struct inode *yaffs_iget(struct super_block *sb, unsigned long ino);
 #endif
 
 #if (LINUX_VERSION_CODE > KERNEL_VERSION(2, 6, 18))
-#define yaffs_InodeToObjectLV(iptr) ((iptr)->i_private)
+#define yaffs_inode_to_obj_lv(iptr) ((iptr)->i_private)
 #else
-#define yaffs_InodeToObjectLV(iptr) ((iptr)->u.generic_ip)
+#define yaffs_inode_to_obj_lv(iptr) ((iptr)->u.generic_ip)
 #endif
 
-#define yaffs_InodeToObject(iptr) ((yaffs_obj_t *)(yaffs_InodeToObjectLV(iptr)))
-#define yaffs_dentry_to_obj(dptr) yaffs_InodeToObject((dptr)->d_inode)
+#define yaffs_inode_to_obj(iptr) \
+       ((struct yaffs_obj *)(yaffs_inode_to_obj_lv(iptr)))
+#define yaffs_dentry_to_obj(dptr) yaffs_inode_to_obj((dptr)->d_inode)
 
 #if (LINUX_VERSION_CODE > KERNEL_VERSION(2, 5, 0))
-#define yaffs_SuperToDevice(sb)        ((yaffs_dev_t *)sb->s_fs_info)
+#define yaffs_super_to_dev(sb) ((struct yaffs_dev *)sb->s_fs_info)
 #else
-#define yaffs_SuperToDevice(sb)        ((yaffs_dev_t *)sb->u.generic_sbp)
+#define yaffs_super_to_dev(sb) ((struct yaffs_dev *)sb->u.generic_sbp)
 #endif
 
-
 #define update_dir_time(dir) do {\
                        (dir)->i_ctime = (dir)->i_mtime = CURRENT_TIME; \
-               } while(0)
-               
+               } while (0)
+
 static void yaffs_put_super(struct super_block *sb);
 
 static ssize_t yaffs_file_write(struct file *f, const char *buf, size_t n,
@@ -240,7 +241,7 @@ static int yaffs_file_flush(struct file *file);
 static int yaffs_sync_object(struct file *file, int datasync);
 #else
 static int yaffs_sync_object(struct file *file, struct dentry *dentry,
-                               int datasync);
+                            int datasync);
 #endif
 
 static int yaffs_readdir(struct file *f, void *dirent, filldir_t filldir);
@@ -249,24 +250,24 @@ static int yaffs_readdir(struct file *f, void *dirent, filldir_t filldir);
 static int yaffs_create(struct inode *dir, struct dentry *dentry, int mode,
                        struct nameidata *n);
 static struct dentry *yaffs_lookup(struct inode *dir, struct dentry *dentry,
-                                       struct nameidata *n);
+                                  struct nameidata *n);
 #else
 static int yaffs_create(struct inode *dir, struct dentry *dentry, int mode);
 static struct dentry *yaffs_lookup(struct inode *dir, struct dentry *dentry);
 #endif
 static int yaffs_link(struct dentry *old_dentry, struct inode *dir,
-                       struct dentry *dentry);
+                     struct dentry *dentry);
 static int yaffs_unlink(struct inode *dir, struct dentry *dentry);
 static int yaffs_symlink(struct inode *dir, struct dentry *dentry,
-                       const char *symname);
+                        const char *symname);
 static int yaffs_mkdir(struct inode *dir, struct dentry *dentry, int mode);
 
 #if (LINUX_VERSION_CODE > KERNEL_VERSION(2, 5, 0))
 static int yaffs_mknod(struct inode *dir, struct dentry *dentry, int mode,
-                       dev_t dev);
+                      dev_t dev);
 #else
 static int yaffs_mknod(struct inode *dir, struct dentry *dentry, int mode,
-                       int dev);
+                      int dev);
 #endif
 static int yaffs_rename(struct inode *old_dir, struct dentry *old_dentry,
                        struct inode *new_dir, struct dentry *new_dentry);
@@ -307,32 +308,31 @@ static int yaffs_writepage(struct page *page);
 #endif
 
 #ifdef CONFIG_YAFFS_XATTR
-int yaffs_setxattr(struct dentry *dentry, const char *name,
-                       const void *value, size_t size, int flags);
-ssize_t yaffs_getxattr(struct dentry *dentry, const char *name, void *buff,
-                       size_t size);
-int yaffs_removexattr(struct dentry *dentry, const char *name);
-ssize_t yaffs_listxattr(struct dentry *dentry, char *buff, size_t size);
+static int yaffs_setxattr(struct dentry *dentry, const char *name,
+                  const void *value, size_t size, int flags);
+static ssize_t yaffs_getxattr(struct dentry *dentry, const char *name,
+                               void *buff, size_t size);
+static int yaffs_removexattr(struct dentry *dentry, const char *name);
+static ssize_t yaffs_listxattr(struct dentry *dentry, char *buff, size_t size);
 #endif
 
-
 #if (YAFFS_USE_WRITE_BEGIN_END != 0)
 static int yaffs_write_begin(struct file *filp, struct address_space *mapping,
-                               loff_t pos, unsigned len, unsigned flags,
-                               struct page **pagep, void **fsdata);
+                            loff_t pos, unsigned len, unsigned flags,
+                            struct page **pagep, void **fsdata);
 static int yaffs_write_end(struct file *filp, struct address_space *mapping,
-                               loff_t pos, unsigned len, unsigned copied,
-                               struct page *pg, void *fsdadata);
+                          loff_t pos, unsigned len, unsigned copied,
+                          struct page *pg, void *fsdadata);
 #else
 static int yaffs_prepare_write(struct file *f, struct page *pg,
-                               unsigned offset, unsigned to);
+                              unsigned offset, unsigned to);
 static int yaffs_commit_write(struct file *f, struct page *pg, unsigned offset,
-                               unsigned to);
+                             unsigned to);
 
 #endif
 
-static int yaffs_readlink(struct dentry *dentry, char __user *buffer,
-                               int buflen);
+static int yaffs_readlink(struct dentry *dentry, char __user * buffer,
+                         int buflen);
 #if (YAFFS_NEW_FOLLOW_LINK == 1)
 void yaffs_put_link(struct dentry *dentry, struct nameidata *nd, void *alias);
 static void *yaffs_follow_link(struct dentry *dentry, struct nameidata *nd);
@@ -340,13 +340,10 @@ static void *yaffs_follow_link(struct dentry *dentry, struct nameidata *nd);
 static int yaffs_follow_link(struct dentry *dentry, struct nameidata *nd);
 #endif
 
-static void yaffs_touch_super(yaffs_dev_t *dev);
-
-static loff_t yaffs_dir_llseek(struct file *file, loff_t offset, int origin);
+static void yaffs_touch_super(struct yaffs_dev *dev);
 
 static int yaffs_vfs_setattr(struct inode *, struct iattr *);
 
-
 static struct address_space_operations yaffs_file_address_operations = {
        .readpage = yaffs_readpage,
        .writepage = yaffs_writepage,
@@ -359,7 +356,6 @@ static struct address_space_operations yaffs_file_address_operations = {
 #endif
 };
 
-
 #if (LINUX_VERSION_CODE > KERNEL_VERSION(2, 6, 22))
 static const struct file_operations yaffs_file_operations = {
        .read = do_sync_read,
@@ -401,17 +397,16 @@ static const struct file_operations yaffs_file_operations = {
 };
 #endif
 
-#if (LINUX_VERSION_CODE < KERNEL_VERSION(2,6,25))
+#if (LINUX_VERSION_CODE < KERNEL_VERSION(2, 6, 25))
 static void zero_user_segment(struct page *page, unsigned start, unsigned end)
 {
-       void * kaddr = kmap_atomic(page, KM_USER0);
+       void *kaddr = kmap_atomic(page, KM_USER0);
        memset(kaddr + start, 0, end - start);
        kunmap_atomic(kaddr, KM_USER0);
        flush_dcache_page(page);
 }
 #endif
 
-
 static const struct inode_operations yaffs_file_inode_operations = {
        .setattr = yaffs_setattr,
 #ifdef CONFIG_YAFFS_XATTR
@@ -460,7 +455,7 @@ static const struct file_operations yaffs_dir_operations = {
        .read = generic_read_dir,
        .readdir = yaffs_readdir,
        .fsync = yaffs_sync_object,
-       .llseek = yaffs_dir_llseek,
+       .llseek = generic_file_llseek,
 };
 
 static const struct super_operations yaffs_super_ops = {
@@ -483,11 +478,10 @@ static const struct super_operations yaffs_super_ops = {
        .write_super = yaffs_write_super,
 };
 
-
-static  int yaffs_vfs_setattr(struct inode *inode, struct iattr *attr)
+static int yaffs_vfs_setattr(struct inode *inode, struct iattr *attr)
 {
-#ifdef  YAFFS_USE_SETATTR_COPY
-       setattr_copy(inode,attr);
+#ifdef YAFFS_USE_SETATTR_COPY
+       setattr_copy(inode, attr);
        return 0;
 #else
        return inode_setattr(inode, attr);
@@ -495,54 +489,58 @@ static  int yaffs_vfs_setattr(struct inode *inode, struct iattr *attr)
 
 }
 
-static  int yaffs_vfs_setsize(struct inode *inode, loff_t newsize)
+static int yaffs_vfs_setsize(struct inode *inode, loff_t newsize)
 {
-#ifdef  YAFFS_USE_TRUNCATE_SETSIZE
-       truncate_setsize(inode,newsize);
+#ifdef YAFFS_USE_TRUNCATE_SETSIZE
+       truncate_setsize(inode, newsize);
        return 0;
 #else
-       truncate_inode_pages(&inode->i_data,newsize);
+       truncate_inode_pages(&inode->i_data, newsize);
        return 0;
 #endif
 
 }
 
-static unsigned yaffs_gc_control_callback(yaffs_dev_t *dev)
+static unsigned yaffs_gc_control_callback(struct yaffs_dev *dev)
 {
        return yaffs_gc_control;
 }
-                                                                                                                       
-static void yaffs_gross_lock(yaffs_dev_t *dev)
+
+static void yaffs_gross_lock(struct yaffs_dev *dev)
 {
-       T(YAFFS_TRACE_LOCK, (TSTR("yaffs locking %p\n"), current));
-       down(&(yaffs_dev_to_lc(dev)->grossLock));
-       T(YAFFS_TRACE_LOCK, (TSTR("yaffs locked %p\n"), current));
+       yaffs_trace(YAFFS_TRACE_LOCK, "yaffs locking %p", current);
+       mutex_lock(&(yaffs_dev_to_lc(dev)->gross_lock));
+       yaffs_trace(YAFFS_TRACE_LOCK, "yaffs locked %p", current);
 }
 
-static void yaffs_gross_unlock(yaffs_dev_t *dev)
+static void yaffs_gross_unlock(struct yaffs_dev *dev)
 {
-       T(YAFFS_TRACE_LOCK, (TSTR("yaffs unlocking %p\n"), current));
-       up(&(yaffs_dev_to_lc(dev)->grossLock));
+       yaffs_trace(YAFFS_TRACE_LOCK, "yaffs unlocking %p", current);
+       mutex_unlock(&(yaffs_dev_to_lc(dev)->gross_lock));
 }
 
 #ifdef YAFFS_COMPILE_EXPORTFS
 
-static struct inode *
-yaffs2_nfs_get_inode(struct super_block *sb, uint64_t ino, uint32_t generation)
+static struct inode *yaffs2_nfs_get_inode(struct super_block *sb, uint64_t ino,
+                                         uint32_t generation)
 {
        return Y_IGET(sb, ino);
 }
 
-static struct dentry *
-yaffs2_fh_to_dentry(struct super_block *sb, struct fid *fid, int fh_len, int fh_type)
+static struct dentry *yaffs2_fh_to_dentry(struct super_block *sb,
+                                         struct fid *fid, int fh_len,
+                                         int fh_type)
 {
-       return generic_fh_to_dentry(sb, fid, fh_len, fh_type, yaffs2_nfs_get_inode) ;
+       return generic_fh_to_dentry(sb, fid, fh_len, fh_type,
+                                   yaffs2_nfs_get_inode);
 }
 
-static struct dentry *
- yaffs2_fh_to_parent(struct super_block *sb, struct fid *fid, int fh_len, int fh_type)
+static struct dentry *yaffs2_fh_to_parent(struct super_block *sb,
+                                         struct fid *fid, int fh_len,
+                                         int fh_type)
 {
-       return generic_fh_to_parent(sb, fid, fh_len, fh_type, yaffs2_nfs_get_inode);
+       return generic_fh_to_parent(sb, fid, fh_len, fh_type,
+                                   yaffs2_nfs_get_inode);
 }
 
 struct dentry *yaffs2_get_parent(struct dentry *dentry)
@@ -552,10 +550,10 @@ struct dentry *yaffs2_get_parent(struct dentry *dentry)
        struct dentry *parent = ERR_PTR(-ENOENT);
        struct inode *inode;
        unsigned long parent_ino;
-       yaffs_obj_t *d_obj;
-       yaffs_obj_t *parent_obj;
+       struct yaffs_obj *d_obj;
+       struct yaffs_obj *parent_obj;
 
-       d_obj = yaffs_InodeToObject(dentry->d_inode);
+       d_obj = yaffs_inode_to_obj(dentry->d_inode);
 
        if (d_obj) {
                parent_obj = d_obj->parent;
@@ -582,12 +580,11 @@ struct dentry *yaffs2_get_parent(struct dentry *dentry)
  * using the default functions of exportfs.
  */
 
-static struct export_operations yaffs_export_ops =
-{
+static struct export_operations yaffs_export_ops = {
        .fh_to_dentry = yaffs2_fh_to_dentry,
        .fh_to_parent = yaffs2_fh_to_parent,
        .get_parent = yaffs2_get_parent,
-} ;
+};
 
 #endif
 
@@ -603,41 +600,42 @@ static struct export_operations yaffs_export_ops =
  * next object before the object is deleted.
  *
  * Many readdirs (and thus seach conexts) may be alive simulateously so
- * each yaffs_dev_t has a list of these.
+ * each struct yaffs_dev has a list of these.
  *
  * A seach context lives for the duration of a readdir.
  *
  * All these functions must be called while yaffs is locked.
  */
 
-struct yaffs_SearchContext {
-       yaffs_dev_t *dev;
-       yaffs_obj_t *dirObj;
-       yaffs_obj_t *nextReturn;
-       struct ylist_head others;
+struct yaffs_search_context {
+       struct yaffs_dev *dev;
+       struct yaffs_obj *dir_obj;
+       struct yaffs_obj *next_return;
+       struct list_head others;
 };
 
 /*
- * yaffs_NewSearch() creates a new search context, initialises it and
+ * yaffs_new_search() creates a new search context, initialises it and
  * adds it to the device's search context list.
  *
  * Called at start of readdir.
  */
-static struct yaffs_SearchContext * yaffs_NewSearch(yaffs_obj_t *dir)
+static struct yaffs_search_context *yaffs_new_search(struct yaffs_obj *dir)
 {
-       yaffs_dev_t *dev = dir->my_dev;
-       struct yaffs_SearchContext *sc = YMALLOC(sizeof(struct yaffs_SearchContext));
-       if(sc){
-               sc->dirObj = dir;
+       struct yaffs_dev *dev = dir->my_dev;
+       struct yaffs_search_context *sc =
+           kmalloc(sizeof(struct yaffs_search_context), GFP_NOFS);
+       if (sc) {
+               sc->dir_obj = dir;
                sc->dev = dev;
-               if( ylist_empty(&sc->dirObj->variant.dir_variant.children))
-                       sc->nextReturn = NULL;
+               if (list_empty(&sc->dir_obj->variant.dir_variant.children))
+                       sc->next_return = NULL;
                else
-                       sc->nextReturn = ylist_entry(
-                                dir->variant.dir_variant.children.next,
-                               yaffs_obj_t,siblings);
-               YINIT_LIST_HEAD(&sc->others);
-               ylist_add(&sc->others,&(yaffs_dev_to_lc(dev)->searchContexts));
+                       sc->next_return =
+                           list_entry(dir->variant.dir_variant.children.next,
+                                      struct yaffs_obj, siblings);
+               INIT_LIST_HEAD(&sc->others);
+               list_add(&sc->others, &(yaffs_dev_to_lc(dev)->search_contexts));
        }
        return sc;
 }
@@ -645,11 +643,11 @@ static struct yaffs_SearchContext * yaffs_NewSearch(yaffs_obj_t *dir)
 /*
  * yaffs_search_end() disposes of a search context and cleans up.
  */
-static void yaffs_search_end(struct yaffs_SearchContext * sc)
+static void yaffs_search_end(struct yaffs_search_context *sc)
 {
-       if(sc){
-               ylist_del(&sc->others);
-               YFREE(sc);
+       if (sc) {
+               list_del(&sc->others);
+               kfree(sc);
        }
 }
 
@@ -658,22 +656,23 @@ static void yaffs_search_end(struct yaffs_SearchContext * sc)
  * Called when the search iterates or when an object removal causes
  * the search context to be moved to the next object.
  */
-static void yaffs_search_advance(struct yaffs_SearchContext *sc)
+static void yaffs_search_advance(struct yaffs_search_context *sc)
 {
-        if(!sc)
-                return;
+       if (!sc)
+               return;
 
-        if( sc->nextReturn == NULL ||
-                ylist_empty(&sc->dirObj->variant.dir_variant.children))
-                sc->nextReturn = NULL;
-        else {
-                struct ylist_head *next = sc->nextReturn->siblings.next;
+       if (sc->next_return == NULL ||
+           list_empty(&sc->dir_obj->variant.dir_variant.children))
+               sc->next_return = NULL;
+       else {
+               struct list_head *next = sc->next_return->siblings.next;
 
-                if( next == &sc->dirObj->variant.dir_variant.children)
-                        sc->nextReturn = NULL; /* end of list */
-                else
-                        sc->nextReturn = ylist_entry(next,yaffs_obj_t,siblings);
-        }
+               if (next == &sc->dir_obj->variant.dir_variant.children)
+                       sc->next_return = NULL; /* end of list */
+               else
+                       sc->next_return =
+                           list_entry(next, struct yaffs_obj, siblings);
+       }
 }
 
 /*
@@ -681,38 +680,35 @@ static void yaffs_search_advance(struct yaffs_SearchContext *sc)
  * We check open search contexts and advance any which are currently
  * on the object being iterated.
  */
-static void yaffs_remove_obj_callback(yaffs_obj_t *obj)
+static void yaffs_remove_obj_callback(struct yaffs_obj *obj)
 {
 
-        struct ylist_head *i;
-        struct yaffs_SearchContext *sc;
-        struct ylist_head *search_contexts = &(yaffs_dev_to_lc(obj->my_dev)->searchContexts);
-
+       struct list_head *i;
+       struct yaffs_search_context *sc;
+       struct list_head *search_contexts =
+           &(yaffs_dev_to_lc(obj->my_dev)->search_contexts);
 
-        /* Iterate through the directory search contexts.
-         * If any are currently on the object being removed, then advance
-         * the search context to the next object to prevent a hanging pointer.
-         */
-         ylist_for_each(i, search_contexts) {
-                if (i) {
-                        sc = ylist_entry(i, struct yaffs_SearchContext,others);
-                        if(sc->nextReturn == obj)
-                                yaffs_search_advance(sc);
-                }
+       /* Iterate through the directory search contexts.
+        * If any are currently on the object being removed, then advance
+        * the search context to the next object to prevent a hanging pointer.
+        */
+       list_for_each(i, search_contexts) {
+               sc = list_entry(i, struct yaffs_search_context, others);
+               if (sc->next_return == obj)
+                       yaffs_search_advance(sc);
        }
 
 }
 
-
 /*-----------------------------------------------------------------*/
 
-static int yaffs_readlink(struct dentry *dentry, char __user *buffer,
-                       int buflen)
+static int yaffs_readlink(struct dentry *dentry, char __user * buffer,
+                         int buflen)
 {
        unsigned char *alias;
        int ret;
 
-       yaffs_dev_t *dev = yaffs_dentry_to_obj(dentry)->my_dev;
+       struct yaffs_dev *dev = yaffs_dentry_to_obj(dentry)->my_dev;
 
        yaffs_gross_lock(dev);
 
@@ -730,13 +726,16 @@ static int yaffs_readlink(struct dentry *dentry, char __user *buffer,
 
 #if (YAFFS_NEW_FOLLOW_LINK == 1)
 static void *yaffs_follow_link(struct dentry *dentry, struct nameidata *nd)
+{
+       void *ret;
 #else
 static int yaffs_follow_link(struct dentry *dentry, struct nameidata *nd)
-#endif
 {
+       int ret
+#endif
        unsigned char *alias;
-       int ret;
-       yaffs_dev_t *dev = yaffs_dentry_to_obj(dentry)->my_dev;
+       int ret_int = 0;
+       struct yaffs_dev *dev = yaffs_dentry_to_obj(dentry)->my_dev;
 
        yaffs_gross_lock(dev);
 
@@ -744,31 +743,35 @@ static int yaffs_follow_link(struct dentry *dentry, struct nameidata *nd)
        yaffs_gross_unlock(dev);
 
        if (!alias) {
-               ret = -ENOMEM;
+               ret_int = -ENOMEM;
                goto out;
        }
-
 #if (YAFFS_NEW_FOLLOW_LINK == 1)
        nd_set_link(nd, alias);
-       ret = (int)alias;
+       ret = alias;
 out:
-       return ERR_PTR(ret);
+       if (ret_int)
+               ret = ERR_PTR(ret_int);
+       return ret;
 #else
        ret = vfs_follow_link(nd, alias);
        kfree(alias);
 out:
+       if (ret_int)
+               ret = ret_int;
        return ret;
 #endif
 }
 
 #if (YAFFS_NEW_FOLLOW_LINK == 1)
-void yaffs_put_link(struct dentry *dentry, struct nameidata *nd, void *alias) {
+void yaffs_put_link(struct dentry *dentry, struct nameidata *nd, void *alias)
+{
        kfree(alias);
 }
 #endif
 
 struct inode *yaffs_get_inode(struct super_block *sb, int mode, int dev,
-                               yaffs_obj_t *obj);
+                             struct yaffs_obj *obj);
 
 /*
  * Lookup is used to find objects in the fs
@@ -776,54 +779,37 @@ struct inode *yaffs_get_inode(struct super_block *sb, int mode, int dev,
 #if (LINUX_VERSION_CODE > KERNEL_VERSION(2, 5, 0))
 
 static struct dentry *yaffs_lookup(struct inode *dir, struct dentry *dentry,
-                               struct nameidata *n)
+                                  struct nameidata *n)
 #else
 static struct dentry *yaffs_lookup(struct inode *dir, struct dentry *dentry)
 #endif
 {
-       yaffs_obj_t *obj;
+       struct yaffs_obj *obj;
        struct inode *inode = NULL;     /* NCB 2.5/2.6 needs NULL here */
 
-       yaffs_dev_t *dev = yaffs_InodeToObject(dir)->my_dev;
+       struct yaffs_dev *dev = yaffs_inode_to_obj(dir)->my_dev;
 
-       if(current != yaffs_dev_to_lc(dev)->readdirProcess)
+       if (current != yaffs_dev_to_lc(dev)->readdir_process)
                yaffs_gross_lock(dev);
 
-       T(YAFFS_TRACE_OS,
-               (TSTR("yaffs_lookup for %d:%s\n"),
-               yaffs_InodeToObject(dir)->obj_id, dentry->d_name.name));
+       yaffs_trace(YAFFS_TRACE_OS, "yaffs_lookup for %d:%s",
+               yaffs_inode_to_obj(dir)->obj_id, dentry->d_name.name);
 
-       obj = yaffs_find_by_name(yaffs_InodeToObject(dir),
-                                       dentry->d_name.name);
+       obj = yaffs_find_by_name(yaffs_inode_to_obj(dir), dentry->d_name.name);
 
        obj = yaffs_get_equivalent_obj(obj);    /* in case it was a hardlink */
 
        /* Can't hold gross lock when calling yaffs_get_inode() */
-       if(current != yaffs_dev_to_lc(dev)->readdirProcess)
+       if (current != yaffs_dev_to_lc(dev)->readdir_process)
                yaffs_gross_unlock(dev);
 
        if (obj) {
-               T(YAFFS_TRACE_OS,
-                       (TSTR("yaffs_lookup found %d\n"), obj->obj_id));
+               yaffs_trace(YAFFS_TRACE_OS,
+                       "yaffs_lookup found %d", obj->obj_id);
 
                inode = yaffs_get_inode(dir->i_sb, obj->yst_mode, 0, obj);
-
-               if (inode) {
-                       T(YAFFS_TRACE_OS,
-                               (TSTR("yaffs_loookup dentry \n")));
-/* #if 0 asserted by NCB for 2.5/6 compatability - falls through to
- * d_add even if NULL inode */
-#if 0
-                       /*dget(dentry); // try to solve directory bug */
-                       d_add(dentry, inode);
-
-                       /* return dentry; */
-                       return NULL;
-#endif
-               }
-
        } else {
-               T(YAFFS_TRACE_OS,(TSTR("yaffs_lookup not found\n")));
+               yaffs_trace(YAFFS_TRACE_OS, "yaffs_lookup not found");
 
        }
 
@@ -834,7 +820,6 @@ static struct dentry *yaffs_lookup(struct inode *dir, struct dentry *dentry)
        return NULL;
 }
 
-
 #ifdef YAFFS_HAS_PUT_INODE
 
 /* For now put inode is just for debugging
@@ -842,21 +827,20 @@ static struct dentry *yaffs_lookup(struct inode *dir, struct dentry *dentry)
  */
 static void yaffs_put_inode(struct inode *inode)
 {
-       T(YAFFS_TRACE_OS,
-               (TSTR("yaffs_put_inode: ino %d, count %d\n"), (int)inode->i_ino,
-               atomic_read(&inode->i_count)));
+       yaffs_trace(YAFFS_TRACE_OS,
+               "yaffs_put_inode: ino %d, count %d"),
+               (int)inode->i_ino, atomic_read(&inode->i_count);
 
 }
 #endif
 
-
-static void yaffs_unstitch_obj(struct inode *inode, yaffs_obj_t *obj)
+static void yaffs_unstitch_obj(struct inode *inode, struct yaffs_obj *obj)
 {
        /* Clear the association between the inode and
-        * the yaffs_obj_t.
+        * the struct yaffs_obj.
         */
        obj->my_inode = NULL;
-       yaffs_InodeToObjectLV(inode) = NULL;
+       yaffs_inode_to_obj_lv(inode) = NULL;
 
        /* If the object freeing was deferred, then the real
         * free happens now.
@@ -870,25 +854,25 @@ static void yaffs_unstitch_obj(struct inode *inode, yaffs_obj_t *obj)
  * yaffs_clear_inode() and yaffs_delete_inode()
  *
  */
-static void yaffs_evict_inode( struct inode *inode)
+static void yaffs_evict_inode(struct inode *inode)
 {
-       yaffs_obj_t *obj;
-       yaffs_dev_t *dev;
+       struct yaffs_obj *obj;
+       struct yaffs_dev *dev;
        int deleteme = 0;
 
-       obj = yaffs_InodeToObject(inode);
+       obj = yaffs_inode_to_obj(inode);
 
-       T(YAFFS_TRACE_OS,
-               (TSTR("yaffs_evict_inode: ino %d, count %d %s\n"), (int)inode->i_ino,
-               atomic_read(&inode->i_count),
-               obj ? "object exists" : "null object"));
+       yaffs_trace(YAFFS_TRACE_OS,
+               "yaffs_evict_inode: ino %d, count %d %s",
+               (int)inode->i_ino, atomic_read(&inode->i_count),
+               obj ? "object exists" : "null object");
 
        if (!inode->i_nlink && !is_bad_inode(inode))
                deleteme = 1;
-       truncate_inode_pages(&inode->i_data,0);
+       truncate_inode_pages(&inode->i_data, 0);
        end_writeback(inode);
 
-       if(deleteme && obj){
+       if (deleteme && obj) {
                dev = obj->my_dev;
                yaffs_gross_lock(dev);
                yaffs_del_obj(obj);
@@ -897,11 +881,9 @@ static void yaffs_evict_inode( struct inode *inode)
        if (obj) {
                dev = obj->my_dev;
                yaffs_gross_lock(dev);
-               yaffs_unstitch_obj(inode,obj);
+               yaffs_unstitch_obj(inode, obj);
                yaffs_gross_unlock(dev);
        }
-
-
 }
 #else
 
@@ -914,20 +896,20 @@ static void yaffs_evict_inode( struct inode *inode)
 
 static void yaffs_clear_inode(struct inode *inode)
 {
-       yaffs_obj_t *obj;
-       yaffs_dev_t *dev;
+       struct yaffs_obj *obj;
+       struct yaffs_dev *dev;
 
-       obj = yaffs_InodeToObject(inode);
+       obj = yaffs_inode_to_obj(inode);
 
-       T(YAFFS_TRACE_OS,
-               (TSTR("yaffs_clear_inode: ino %d, count %d %s\n"), (int)inode->i_ino,
-               atomic_read(&inode->i_count),
-               obj ? "object exists" : "null object"));
+       yaffs_trace(YAFFS_TRACE_OS,
+               "yaffs_clear_inode: ino %d, count %d %s",
+               (int)inode->i_ino, atomic_read(&inode->i_count),
+               obj ? "object exists" : "null object");
 
        if (obj) {
                dev = obj->my_dev;
                yaffs_gross_lock(dev);
-               yaffs_unstitch_obj(inode,obj);
+               yaffs_unstitch_obj(inode, obj);
                yaffs_gross_unlock(dev);
        }
 
@@ -940,13 +922,13 @@ static void yaffs_clear_inode(struct inode *inode)
  */
 static void yaffs_delete_inode(struct inode *inode)
 {
-       yaffs_obj_t *obj = yaffs_InodeToObject(inode);
-       yaffs_dev_t *dev;
+       struct yaffs_obj *obj = yaffs_inode_to_obj(inode);
+       struct yaffs_dev *dev;
 
-       T(YAFFS_TRACE_OS,
-               (TSTR("yaffs_delete_inode: ino %d, count %d %s\n"), (int)inode->i_ino,
-               atomic_read(&inode->i_count),
-               obj ? "object exists" : "null object"));
+       yaffs_trace(YAFFS_TRACE_OS,
+               "yaffs_delete_inode: ino %d, count %d %s",
+               (int)inode->i_ino, atomic_read(&inode->i_count),
+               obj ? "object exists" : "null object");
 
        if (obj) {
                dev = obj->my_dev;
@@ -961,20 +943,20 @@ static void yaffs_delete_inode(struct inode *inode)
 }
 #endif
 
-
 #if (LINUX_VERSION_CODE > KERNEL_VERSION(2, 6, 17))
 static int yaffs_file_flush(struct file *file, fl_owner_t id)
 #else
 static int yaffs_file_flush(struct file *file)
 #endif
 {
-       yaffs_obj_t *obj = yaffs_dentry_to_obj(file->f_dentry);
+       struct yaffs_obj *obj = yaffs_dentry_to_obj(file->f_dentry);
 
-       yaffs_dev_t *dev = obj->my_dev;
+       struct yaffs_dev *dev = obj->my_dev;
 
-       T(YAFFS_TRACE_OS,
-               (TSTR("yaffs_file_flush object %d (%s)\n"), obj->obj_id,
-               obj->dirty ? "dirty" : "clean"));
+       yaffs_trace(YAFFS_TRACE_OS,
+               "yaffs_file_flush object %d (%s)",
+               obj->obj_id,
+               obj->dirty ? "dirty" : "clean");
 
        yaffs_gross_lock(dev);
 
@@ -989,16 +971,16 @@ static int yaffs_readpage_nolock(struct file *f, struct page *pg)
 {
        /* Lifted from jffs2 */
 
-       yaffs_obj_t *obj;
+       struct yaffs_obj *obj;
        unsigned char *pg_buf;
        int ret;
 
-       yaffs_dev_t *dev;
+       struct yaffs_dev *dev;
 
-       T(YAFFS_TRACE_OS,
-               (TSTR("yaffs_readpage_nolock at %08x, size %08x\n"),
-               (unsigned)(pg->index << PAGE_CACHE_SHIFT),
-               (unsigned)PAGE_CACHE_SIZE));
+       yaffs_trace(YAFFS_TRACE_OS,
+               "yaffs_readpage_nolock at %08x, size %08x",
+               (unsigned)(pg->index << PAGE_CACHE_SHIFT),
+               (unsigned)PAGE_CACHE_SIZE);
 
        obj = yaffs_dentry_to_obj(f->f_dentry);
 
@@ -1017,8 +999,7 @@ static int yaffs_readpage_nolock(struct file *f, struct page *pg)
        yaffs_gross_lock(dev);
 
        ret = yaffs_file_rd(obj, pg_buf,
-                               pg->index << PAGE_CACHE_SHIFT,
-                               PAGE_CACHE_SIZE);
+                           pg->index << PAGE_CACHE_SHIFT, PAGE_CACHE_SIZE);
 
        yaffs_gross_unlock(dev);
 
@@ -1036,7 +1017,7 @@ static int yaffs_readpage_nolock(struct file *f, struct page *pg)
        flush_dcache_page(pg);
        kunmap(pg);
 
-       T(YAFFS_TRACE_OS, (TSTR("yaffs_readpage_nolock done\n")));
+       yaffs_trace(YAFFS_TRACE_OS, "yaffs_readpage_nolock done");
        return ret;
 }
 
@@ -1051,9 +1032,9 @@ static int yaffs_readpage(struct file *f, struct page *pg)
 {
        int ret;
 
-       T(YAFFS_TRACE_OS, (TSTR("yaffs_readpage\n")));
-       ret=yaffs_readpage_unlock(f, pg);
-       T(YAFFS_TRACE_OS, (TSTR("yaffs_readpage done\n")));
+       yaffs_trace(YAFFS_TRACE_OS, "yaffs_readpage");
+       ret = yaffs_readpage_unlock(f, pg);
+       yaffs_trace(YAFFS_TRACE_OS, "yaffs_readpage done");
        return ret;
 }
 
@@ -1065,13 +1046,13 @@ static int yaffs_writepage(struct page *page, struct writeback_control *wbc)
 static int yaffs_writepage(struct page *page)
 #endif
 {
-       yaffs_dev_t *dev;
+       struct yaffs_dev *dev;
        struct address_space *mapping = page->mapping;
        struct inode *inode;
        unsigned long end_index;
        char *buffer;
-       yaffs_obj_t *obj;
-       int nWritten = 0;
+       struct yaffs_obj *obj;
+       int n_written = 0;
        unsigned n_bytes;
        loff_t i_size;
 
@@ -1084,20 +1065,20 @@ static int yaffs_writepage(struct page *page)
 
        end_index = i_size >> PAGE_CACHE_SHIFT;
 
-       if(page->index < end_index)
+       if (page->index < end_index)
                n_bytes = PAGE_CACHE_SIZE;
        else {
-               n_bytes = i_size & (PAGE_CACHE_SIZE -1);
+               n_bytes = i_size & (PAGE_CACHE_SIZE - 1);
 
                if (page->index > end_index || !n_bytes) {
-                       T(YAFFS_TRACE_OS,
-                               (TSTR("yaffs_writepage at %08x, inode size = %08x!!!\n"),
+                       yaffs_trace(YAFFS_TRACE_OS,
+                               "yaffs_writepage at %08x, inode size = %08x!!",
                                (unsigned)(page->index << PAGE_CACHE_SHIFT),
-                               (unsigned)inode->i_size));
-                       T(YAFFS_TRACE_OS,
-                               (TSTR("                -> don't care!!\n")));
+                               (unsigned)inode->i_size);
+                       yaffs_trace(YAFFS_TRACE_OS,
+                               "                -> don't care!!");
 
-                       zero_user_segment(page,0,PAGE_CACHE_SIZE);
+                       zero_user_segment(page, 0, PAGE_CACHE_SIZE);
                        set_page_writeback(page);
                        unlock_page(page);
                        end_page_writeback(page);
@@ -1105,32 +1086,32 @@ static int yaffs_writepage(struct page *page)
                }
        }
 
-       if(n_bytes != PAGE_CACHE_SIZE)
-               zero_user_segment(page,n_bytes,PAGE_CACHE_SIZE);
+       if (n_bytes != PAGE_CACHE_SIZE)
+               zero_user_segment(page, n_bytes, PAGE_CACHE_SIZE);
 
        get_page(page);
 
        buffer = kmap(page);
 
-       obj = yaffs_InodeToObject(inode);
+       obj = yaffs_inode_to_obj(inode);
        dev = obj->my_dev;
        yaffs_gross_lock(dev);
 
-       T(YAFFS_TRACE_OS,
-               (TSTR("yaffs_writepage at %08x, size %08x\n"),
-               (unsigned)(page->index << PAGE_CACHE_SHIFT), n_bytes));
-       T(YAFFS_TRACE_OS,
-               (TSTR("writepag0: obj = %05x, ino = %05x\n"),
-               (int)obj->variant.file_variant.file_size, (int)inode->i_size));
+       yaffs_trace(YAFFS_TRACE_OS,
+               "yaffs_writepage at %08x, size %08x",
+               (unsigned)(page->index << PAGE_CACHE_SHIFT), n_bytes);
+       yaffs_trace(YAFFS_TRACE_OS,
+               "writepag0: obj = %05x, ino = %05x",
+               (int)obj->variant.file_variant.file_size, (int)inode->i_size);
 
-       nWritten = yaffs_wr_file(obj, buffer,
-                       page->index << PAGE_CACHE_SHIFT, n_bytes, 0);
+       n_written = yaffs_wr_file(obj, buffer,
+                                 page->index << PAGE_CACHE_SHIFT, n_bytes, 0);
 
        yaffs_touch_super(dev);
 
-       T(YAFFS_TRACE_OS,
-               (TSTR("writepag1: obj = %05x, ino = %05x\n"),
-               (int)obj->variant.file_variant.file_size, (int)inode->i_size));
+       yaffs_trace(YAFFS_TRACE_OS,
+               "writepag1: obj = %05x, ino = %05x",
+               (int)obj->variant.file_variant.file_size, (int)inode->i_size);
 
        yaffs_gross_unlock(dev);
 
@@ -1140,14 +1121,13 @@ static int yaffs_writepage(struct page *page)
        end_page_writeback(page);
        put_page(page);
 
-       return (nWritten == n_bytes) ? 0 : -ENOSPC;
+       return (n_written == n_bytes) ? 0 : -ENOSPC;
 }
 
-
 #if (YAFFS_USE_WRITE_BEGIN_END > 0)
 static int yaffs_write_begin(struct file *filp, struct address_space *mapping,
-                               loff_t pos, unsigned len, unsigned flags,
-                               struct page **pagep, void **fsdata)
+                            loff_t pos, unsigned len, unsigned flags,
+                            struct page **pagep, void **fsdata)
 {
        struct page *pg = NULL;
        pgoff_t index = pos >> PAGE_CACHE_SHIFT;
@@ -1164,12 +1144,12 @@ static int yaffs_write_begin(struct file *filp, struct address_space *mapping,
 
        *pagep = pg;
        if (!pg) {
-               ret =  -ENOMEM;
+               ret = -ENOMEM;
                goto out;
        }
-       T(YAFFS_TRACE_OS,
-               (TSTR("start yaffs_write_begin index %d(%x) uptodate %d\n"),
-               (int)index,(int)index,Page_Uptodate(pg) ? 1 : 0));
+       yaffs_trace(YAFFS_TRACE_OS,
+               "start yaffs_write_begin index %d(%x) uptodate %d",
+               (int)index, (int)index, Page_Uptodate(pg) ? 1 : 0);
 
        /* Get fs space */
        space_held = yaffs_hold_space(filp);
@@ -1188,13 +1168,13 @@ static int yaffs_write_begin(struct file *filp, struct address_space *mapping,
                goto out;
 
        /* Happy path return */
-       T(YAFFS_TRACE_OS, (TSTR("end yaffs_write_begin - ok\n")));
+       yaffs_trace(YAFFS_TRACE_OS, "end yaffs_write_begin - ok");
 
        return 0;
 
 out:
-       T(YAFFS_TRACE_OS,
-               (TSTR("end yaffs_write_begin fail returning %d\n"), ret));
+       yaffs_trace(YAFFS_TRACE_OS,
+               "end yaffs_write_begin fail returning %d", ret);
        if (space_held)
                yaffs_release_space(filp);
        if (pg) {
@@ -1207,9 +1187,9 @@ out:
 #else
 
 static int yaffs_prepare_write(struct file *f, struct page *pg,
-                               unsigned offset, unsigned to)
+                              unsigned offset, unsigned to)
 {
-       T(YAFFS_TRACE_OS, (TSTR("yaffs_prepair_write\n")));
+       yaffs_trace(YAFFS_TRACE_OS, "yaffs_prepair_write");
 
        if (!Page_Uptodate(pg))
                return yaffs_readpage_nolock(f, pg);
@@ -1219,8 +1199,8 @@ static int yaffs_prepare_write(struct file *f, struct page *pg,
 
 #if (YAFFS_USE_WRITE_BEGIN_END > 0)
 static int yaffs_write_end(struct file *filp, struct address_space *mapping,
-                               loff_t pos, unsigned len, unsigned copied,
-                               struct page *pg, void *fsdadata)
+                          loff_t pos, unsigned len, unsigned copied,
+                          struct page *pg, void *fsdadata)
 {
        int ret = 0;
        void *addr, *kva;
@@ -1229,19 +1209,17 @@ static int yaffs_write_end(struct file *filp, struct address_space *mapping,
        kva = kmap(pg);
        addr = kva + offset_into_page;
 
-       T(YAFFS_TRACE_OS,
-               ("yaffs_write_end addr %p pos %x n_bytes %d\n",
-               addr,(unsigned)pos, copied));
+       yaffs_trace(YAFFS_TRACE_OS,
+               "yaffs_write_end addr %p pos %x n_bytes %d",
+               addr, (unsigned)pos, copied);
 
        ret = yaffs_file_write(filp, addr, copied, &pos);
 
        if (ret != copied) {
-               T(YAFFS_TRACE_OS,
-                       (TSTR("yaffs_write_end not same size ret %d  copied %d\n"),
-                       ret, copied));
+               yaffs_trace(YAFFS_TRACE_OS,
+                       "yaffs_write_end not same size ret %d  copied %d",
+                       ret, copied);
                SetPageError(pg);
-       } else {
-               /* Nothing */
        }
 
        kunmap(pg);
@@ -1254,13 +1232,13 @@ static int yaffs_write_end(struct file *filp, struct address_space *mapping,
 #else
 
 static int yaffs_commit_write(struct file *f, struct page *pg, unsigned offset,
-                               unsigned to)
+                             unsigned to)
 {
        void *addr, *kva;
 
        loff_t pos = (((loff_t) pg->index) << PAGE_CACHE_SHIFT) + offset;
        int n_bytes = to - offset;
-       int nWritten;
+       int n_written;
 
        unsigned spos = pos;
        unsigned saddr;
@@ -1268,41 +1246,37 @@ static int yaffs_commit_write(struct file *f, struct page *pg, unsigned offset,
        kva = kmap(pg);
        addr = kva + offset;
 
-       saddr = (unsigned) addr;
+       saddr = (unsigned)addr;
 
-       T(YAFFS_TRACE_OS,
-               (TSTR("yaffs_commit_write addr %x pos %x n_bytes %d\n"),
-               saddr, spos, n_bytes));
+       yaffs_trace(YAFFS_TRACE_OS,
+               "yaffs_commit_write addr %x pos %x n_bytes %d",
+               saddr, spos, n_bytes);
 
-       nWritten = yaffs_file_write(f, addr, n_bytes, &pos);
+       n_written = yaffs_file_write(f, addr, n_bytes, &pos);
 
-       if (nWritten != n_bytes) {
-               T(YAFFS_TRACE_OS,
-                       (TSTR("yaffs_commit_write not same size nWritten %d  n_bytes %d\n"),
-                       nWritten, n_bytes));
+       if (n_written != n_bytes) {
+               yaffs_trace(YAFFS_TRACE_OS,
+                       "yaffs_commit_write not same size n_written %d  n_bytes %d",
+                       n_written, n_bytes);
                SetPageError(pg);
-       } else {
-               /* Nothing */
        }
-
        kunmap(pg);
 
-       T(YAFFS_TRACE_OS,
-               (TSTR("yaffs_commit_write returning %d\n"),
-               nWritten == n_bytes ? 0 : nWritten));
+       yaffs_trace(YAFFS_TRACE_OS,
+               "yaffs_commit_write returning %d",
+               n_written == n_bytes ? 0 : n_written);
 
-       return nWritten == n_bytes ? 0 : nWritten;
+       return n_written == n_bytes ? 0 : n_written;
 }
 #endif
 
-
-static void yaffs_fill_inode_from_obj(struct inode *inode, yaffs_obj_t *obj)
+static void yaffs_fill_inode_from_obj(struct inode *inode,
+                                     struct yaffs_obj *obj)
 {
        if (inode && obj) {
 
-
                /* Check mode against the variant type and attempt to repair if broken. */
-               __u32 mode = obj->yst_mode;
+               u32 mode = obj->yst_mode;
                switch (obj->variant_type) {
                case YAFFS_OBJECT_TYPE_FILE:
                        if (!S_ISREG(mode)) {
@@ -1362,26 +1336,26 @@ static void yaffs_fill_inode_from_obj(struct inode *inode, yaffs_obj_t *obj)
 
                inode->i_nlink = yaffs_get_obj_link_count(obj);
 
-               T(YAFFS_TRACE_OS,
-                       (TSTR("yaffs_fill_inode mode %x uid %d gid %d size %d count %d\n"),
+               yaffs_trace(YAFFS_TRACE_OS,
+                       "yaffs_fill_inode mode %x uid %d gid %d size %d count %d",
                        inode->i_mode, inode->i_uid, inode->i_gid,
-                       (int)inode->i_size, atomic_read(&inode->i_count)));
+                       (int)inode->i_size, atomic_read(&inode->i_count));
 
                switch (obj->yst_mode & S_IFMT) {
                default:        /* fifo, device or socket */
 #if (LINUX_VERSION_CODE > KERNEL_VERSION(2, 5, 0))
                        init_special_inode(inode, obj->yst_mode,
-                                       old_decode_dev(obj->yst_rdev));
+                                          old_decode_dev(obj->yst_rdev));
 #else
                        init_special_inode(inode, obj->yst_mode,
-                                       (dev_t) (obj->yst_rdev));
+                                          (dev_t) (obj->yst_rdev));
 #endif
                        break;
                case S_IFREG:   /* file */
                        inode->i_op = &yaffs_file_inode_operations;
                        inode->i_fop = &yaffs_file_operations;
                        inode->i_mapping->a_ops =
-                               &yaffs_file_address_operations;
+                           &yaffs_file_address_operations;
                        break;
                case S_IFDIR:   /* directory */
                        inode->i_op = &yaffs_dir_inode_operations;
@@ -1392,38 +1366,38 @@ static void yaffs_fill_inode_from_obj(struct inode *inode, yaffs_obj_t *obj)
                        break;
                }
 
-               yaffs_InodeToObjectLV(inode) = obj;
+               yaffs_inode_to_obj_lv(inode) = obj;
 
                obj->my_inode = inode;
 
        } else {
-               T(YAFFS_TRACE_OS,
-                       (TSTR("yaffs_FileInode invalid parameters\n")));
+               yaffs_trace(YAFFS_TRACE_OS,
+                       "yaffs_fill_inode invalid parameters");
        }
 
 }
 
 struct inode *yaffs_get_inode(struct super_block *sb, int mode, int dev,
-                               yaffs_obj_t *obj)
+                             struct yaffs_obj *obj)
 {
        struct inode *inode;
 
        if (!sb) {
-               T(YAFFS_TRACE_OS,
-                       (TSTR("yaffs_get_inode for NULL super_block!!\n")));
+               yaffs_trace(YAFFS_TRACE_OS,
+                       "yaffs_get_inode for NULL super_block!!");
                return NULL;
 
        }
 
        if (!obj) {
-               T(YAFFS_TRACE_OS,
-                       (TSTR("yaffs_get_inode for NULL object!!\n")));
+               yaffs_trace(YAFFS_TRACE_OS,
+                       "yaffs_get_inode for NULL object!!");
                return NULL;
 
        }
 
-       T(YAFFS_TRACE_OS,
-               (TSTR("yaffs_get_inode for object %d\n"), obj->obj_id));
+       yaffs_trace(YAFFS_TRACE_OS,
+               "yaffs_get_inode for object %d", obj->obj_id);
 
        inode = Y_IGET(sb, obj->obj_id);
        if (IS_ERR(inode))
@@ -1431,21 +1405,27 @@ struct inode *yaffs_get_inode(struct super_block *sb, int mode, int dev,
 
        /* NB Side effect: iget calls back to yaffs_read_inode(). */
        /* iget also increments the inode's i_count */
-       /* NB You can't be holding grossLock or deadlock will happen! */
+       /* NB You can't be holding gross_lock or deadlock will happen! */
 
        return inode;
 }
 
 static ssize_t yaffs_file_write(struct file *f, const char *buf, size_t n,
-                               loff_t *pos)
+                               loff_t * pos)
 {
-       yaffs_obj_t *obj;
-       int nWritten, ipos;
+       struct yaffs_obj *obj;
+       int n_written, ipos;
        struct inode *inode;
-       yaffs_dev_t *dev;
+       struct yaffs_dev *dev;
 
        obj = yaffs_dentry_to_obj(f->f_dentry);
 
+       if (!obj) {
+               yaffs_trace(YAFFS_TRACE_OS,
+                       "yaffs_file_write: hey obj is null!");
+                return -EINVAL;
+        }
+
        dev = obj->my_dev;
 
        yaffs_gross_lock(dev);
@@ -1457,39 +1437,33 @@ static ssize_t yaffs_file_write(struct file *f, const char *buf, size_t n,
        else
                ipos = *pos;
 
-       if (!obj)
-               T(YAFFS_TRACE_OS,
-                       (TSTR("yaffs_file_write: hey obj is null!\n")));
-       else
-               T(YAFFS_TRACE_OS,
-                       (TSTR("yaffs_file_write about to write writing %u(%x) bytes"
-                       "to object %d at %d(%x)\n"),
-                       (unsigned) n, (unsigned) n, obj->obj_id, ipos,ipos));
+       yaffs_trace(YAFFS_TRACE_OS,
+               "yaffs_file_write about to write writing %u(%x) bytes to object %d at %d(%x)",
+               (unsigned)n, (unsigned)n, obj->obj_id, ipos, ipos);
 
-       nWritten = yaffs_wr_file(obj, buf, ipos, n, 0);
+       n_written = yaffs_wr_file(obj, buf, ipos, n, 0);
 
        yaffs_touch_super(dev);
 
-       T(YAFFS_TRACE_OS,
-               (TSTR("yaffs_file_write: %d(%x) bytes written\n"),
-               (unsigned )n,(unsigned)n));
+       yaffs_trace(YAFFS_TRACE_OS,
+               "yaffs_file_write: %d(%x) bytes written",
+               (unsigned)n, (unsigned)n);
 
-       if (nWritten > 0) {
-               ipos += nWritten;
+       if (n_written > 0) {
+               ipos += n_written;
                *pos = ipos;
                if (ipos > inode->i_size) {
                        inode->i_size = ipos;
                        inode->i_blocks = (ipos + 511) >> 9;
 
-                       T(YAFFS_TRACE_OS,
-                               (TSTR("yaffs_file_write size updated to %d bytes, "
-                               "%d blocks\n"),
-                               ipos, (int)(inode->i_blocks)));
+                       yaffs_trace(YAFFS_TRACE_OS,
+                               "yaffs_file_write size updated to %d bytes, %d blocks",
+                               ipos, (int)(inode->i_blocks));
                }
 
        }
        yaffs_gross_unlock(dev);
-       return (nWritten == 0) && (n > 0) ? -ENOSPC : nWritten;
+       return (n_written == 0) && (n > 0) ? -ENOSPC : n_written;
 }
 
 /* Space holding and freeing is done to ensure we have space available for write_begin/end */
@@ -1498,12 +1472,11 @@ static ssize_t yaffs_file_write(struct file *f, const char *buf, size_t n,
 
 static ssize_t yaffs_hold_space(struct file *f)
 {
-       yaffs_obj_t *obj;
-       yaffs_dev_t *dev;
+       struct yaffs_obj *obj;
+       struct yaffs_dev *dev;
 
        int n_free_chunks;
 
-
        obj = yaffs_dentry_to_obj(f->f_dentry);
 
        dev = obj->my_dev;
@@ -1519,9 +1492,8 @@ static ssize_t yaffs_hold_space(struct file *f)
 
 static void yaffs_release_space(struct file *f)
 {
-       yaffs_obj_t *obj;
-       yaffs_dev_t *dev;
-
+       struct yaffs_obj *obj;
+       struct yaffs_dev *dev;
 
        obj = yaffs_dentry_to_obj(f->f_dentry);
 
@@ -1529,46 +1501,18 @@ static void yaffs_release_space(struct file *f)
 
        yaffs_gross_lock(dev);
 
-
        yaffs_gross_unlock(dev);
 }
 
-
-static loff_t yaffs_dir_llseek(struct file *file, loff_t offset, int origin)
-{
-       long long retval;
-
-       lock_kernel();
-
-       switch (origin){
-       case 2:
-               offset += i_size_read(file->f_path.dentry->d_inode);
-               break;
-       case 1:
-               offset += file->f_pos;
-       }
-       retval = -EINVAL;
-
-       if (offset >= 0){
-               if (offset != file->f_pos)
-                       file->f_pos = offset;
-
-               retval = offset;
-       }
-       unlock_kernel();
-       return retval;
-}
-
-
 static int yaffs_readdir(struct file *f, void *dirent, filldir_t filldir)
 {
-       yaffs_obj_t *obj;
-       yaffs_dev_t *dev;
-        struct yaffs_SearchContext *sc;
+       struct yaffs_obj *obj;
+       struct yaffs_dev *dev;
+       struct yaffs_search_context *sc;
        struct inode *inode = f->f_dentry->d_inode;
        unsigned long offset, curoffs;
-       yaffs_obj_t *l;
-        int retVal = 0;
+       struct yaffs_obj *l;
+       int ret_val = 0;
 
        char name[YAFFS_MAX_NAME_LENGTH + 1];
 
@@ -1577,24 +1521,25 @@ static int yaffs_readdir(struct file *f, void *dirent, filldir_t filldir)
 
        yaffs_gross_lock(dev);
 
-       yaffs_dev_to_lc(dev)->readdirProcess = current;
+       yaffs_dev_to_lc(dev)->readdir_process = current;
 
        offset = f->f_pos;
 
-        sc = yaffs_NewSearch(obj);
-        if(!sc){
-                retVal = -ENOMEM;
-                goto out;
-        }
+       sc = yaffs_new_search(obj);
+       if (!sc) {
+               ret_val = -ENOMEM;
+               goto out;
+       }
 
-       T(YAFFS_TRACE_OS, (TSTR("yaffs_readdir: starting at %d\n"), (int)offset));
+       yaffs_trace(YAFFS_TRACE_OS,
+               "yaffs_readdir: starting at %d", (int)offset);
 
        if (offset == 0) {
-               T(YAFFS_TRACE_OS,
-                       (TSTR("yaffs_readdir: entry . ino %d \n"),
-                       (int)inode->i_ino));
+               yaffs_trace(YAFFS_TRACE_OS,
+                       "yaffs_readdir: entry . ino %d",
+                       (int)inode->i_ino);
                yaffs_gross_unlock(dev);
-               if (filldir(dirent, ".", 1, offset, inode->i_ino, DT_DIR) < 0){
+               if (filldir(dirent, ".", 1, offset, inode->i_ino, DT_DIR) < 0) {
                        yaffs_gross_lock(dev);
                        goto out;
                }
@@ -1603,12 +1548,13 @@ static int yaffs_readdir(struct file *f, void *dirent, filldir_t filldir)
                f->f_pos++;
        }
        if (offset == 1) {
-               T(YAFFS_TRACE_OS,
-                       (TSTR("yaffs_readdir: entry .. ino %d \n"),
-                       (int)f->f_dentry->d_parent->d_inode->i_ino));
+               yaffs_trace(YAFFS_TRACE_OS,
+                       "yaffs_readdir: entry .. ino %d",
+                       (int)f->f_dentry->d_parent->d_inode->i_ino);
                yaffs_gross_unlock(dev);
                if (filldir(dirent, "..", 2, offset,
-                       f->f_dentry->d_parent->d_inode->i_ino, DT_DIR) < 0){
+                           f->f_dentry->d_parent->d_inode->i_ino,
+                           DT_DIR) < 0) {
                        yaffs_gross_lock(dev);
                        goto out;
                }
@@ -1627,49 +1573,44 @@ static int yaffs_readdir(struct file *f, void *dirent, filldir_t filldir)
                f->f_version = inode->i_version;
        }
 
-       while(sc->nextReturn){
+       while (sc->next_return) {
                curoffs++;
-                l = sc->nextReturn;
+               l = sc->next_return;
                if (curoffs >= offset) {
-                        int this_inode = yaffs_get_obj_inode(l);
-                        int this_type = yaffs_get_obj_type(l);
+                       int this_inode = yaffs_get_obj_inode(l);
+                       int this_type = yaffs_get_obj_type(l);
 
-                       yaffs_get_obj_name(l, name,
-                                           YAFFS_MAX_NAME_LENGTH + 1);
-                       T(YAFFS_TRACE_OS,
-                         (TSTR("yaffs_readdir: %s inode %d\n"),
-                         name, yaffs_get_obj_inode(l)));
+                       yaffs_get_obj_name(l, name, YAFFS_MAX_NAME_LENGTH + 1);
+                       yaffs_trace(YAFFS_TRACE_OS,
+                               "yaffs_readdir: %s inode %d",
+                               name, yaffs_get_obj_inode(l));
 
-                        yaffs_gross_unlock(dev);
+                       yaffs_gross_unlock(dev);
 
                        if (filldir(dirent,
-                                       name,
-                                       strlen(name),
-                                       offset,
-                                       this_inode,
-                                       this_type) < 0){
+                                   name,
+                                   strlen(name),
+                                   offset, this_inode, this_type) < 0) {
                                yaffs_gross_lock(dev);
                                goto out;
                        }
 
-                        yaffs_gross_lock(dev);
+                       yaffs_gross_lock(dev);
 
                        offset++;
                        f->f_pos++;
                }
-                yaffs_search_advance(sc);
+               yaffs_search_advance(sc);
        }
 
 out:
        yaffs_search_end(sc);
-       yaffs_dev_to_lc(dev)->readdirProcess = NULL;
+       yaffs_dev_to_lc(dev)->readdir_process = NULL;
        yaffs_gross_unlock(dev);
 
-       return retVal;
+       return ret_val;
 }
 
-
-
 /*
  * File creation. Allocate an inode, and we're done..
  */
@@ -1682,39 +1623,40 @@ out:
 
 #if (LINUX_VERSION_CODE > KERNEL_VERSION(2, 5, 0))
 static int yaffs_mknod(struct inode *dir, struct dentry *dentry, int mode,
-                       dev_t rdev)
+                      dev_t rdev)
 #else
 static int yaffs_mknod(struct inode *dir, struct dentry *dentry, int mode,
-                       int rdev)
+                      int rdev)
 #endif
 {
        struct inode *inode;
 
-       yaffs_obj_t *obj = NULL;
-       yaffs_dev_t *dev;
+       struct yaffs_obj *obj = NULL;
+       struct yaffs_dev *dev;
 
-       yaffs_obj_t *parent = yaffs_InodeToObject(dir);
+       struct yaffs_obj *parent = yaffs_inode_to_obj(dir);
 
        int error = -ENOSPC;
        uid_t uid = YCRED(current)->fsuid;
-       gid_t gid = (dir->i_mode & S_ISGID) ? dir->i_gid : YCRED(current)->fsgid;
+       gid_t gid =
+           (dir->i_mode & S_ISGID) ? dir->i_gid : YCRED(current)->fsgid;
 
        if ((dir->i_mode & S_ISGID) && S_ISDIR(mode))
                mode |= S_ISGID;
 
        if (parent) {
-               T(YAFFS_TRACE_OS,
-                       (TSTR("yaffs_mknod: parent object %d type %d\n"),
-                       parent->obj_id, parent->variant_type));
+               yaffs_trace(YAFFS_TRACE_OS,
+                       "yaffs_mknod: parent object %d type %d",
+                       parent->obj_id, parent->variant_type);
        } else {
-               T(YAFFS_TRACE_OS,
-                       (TSTR("yaffs_mknod: could not get parent object\n")));
+               yaffs_trace(YAFFS_TRACE_OS,
+                       "yaffs_mknod: could not get parent object");
                return -EPERM;
        }
 
-       T(YAFFS_TRACE_OS, (TSTR("yaffs_mknod: making oject for %s, "
-                       "mode %x dev %x\n"),
-                       dentry->d_name.name, mode, rdev));
+       yaffs_trace(YAFFS_TRACE_OS,
+               "yaffs_mknod: making oject for %s, mode %x dev %x",
+               dentry->d_name.name, mode, rdev);
 
        dev = parent->my_dev;
 
@@ -1723,28 +1665,29 @@ static int yaffs_mknod(struct inode *dir, struct dentry *dentry, int mode,
        switch (mode & S_IFMT) {
        default:
                /* Special (socket, fifo, device...) */
-               T(YAFFS_TRACE_OS, (TSTR("yaffs_mknod: making special\n")));
+               yaffs_trace(YAFFS_TRACE_OS, "yaffs_mknod: making special");
 #if (LINUX_VERSION_CODE > KERNEL_VERSION(2, 5, 0))
-               obj = yaffs_create_special(parent, dentry->d_name.name, mode, uid,
-                               gid, old_encode_dev(rdev));
+               obj =
+                   yaffs_create_special(parent, dentry->d_name.name, mode, uid,
+                                        gid, old_encode_dev(rdev));
 #else
-               obj = yaffs_create_special(parent, dentry->d_name.name, mode, uid,
-                               gid, rdev);
+               obj =
+                   yaffs_create_special(parent, dentry->d_name.name, mode, uid,
+                                        gid, rdev);
 #endif
                break;
        case S_IFREG:           /* file          */
-               T(YAFFS_TRACE_OS, (TSTR("yaffs_mknod: making file\n")));
+               yaffs_trace(YAFFS_TRACE_OS, "yaffs_mknod: making file");
                obj = yaffs_create_file(parent, dentry->d_name.name, mode, uid,
-                               gid);
+                                       gid);
                break;
        case S_IFDIR:           /* directory */
-               T(YAFFS_TRACE_OS,
-                       (TSTR("yaffs_mknod: making directory\n")));
+               yaffs_trace(YAFFS_TRACE_OS, "yaffs_mknod: making directory");
                obj = yaffs_create_dir(parent, dentry->d_name.name, mode,
-                                       uid, gid);
+                                      uid, gid);
                break;
        case S_IFLNK:           /* symlink */
-               T(YAFFS_TRACE_OS, (TSTR("yaffs_mknod: making symlink\n")));
+               yaffs_trace(YAFFS_TRACE_OS, "yaffs_mknod: making symlink");
                obj = NULL;     /* Do we ever get here? */
                break;
        }
@@ -1756,14 +1699,13 @@ static int yaffs_mknod(struct inode *dir, struct dentry *dentry, int mode,
                inode = yaffs_get_inode(dir->i_sb, mode, rdev, obj);
                d_instantiate(dentry, inode);
                update_dir_time(dir);
-               T(YAFFS_TRACE_OS,
-                       (TSTR("yaffs_mknod created object %d count = %d\n"),
-                       obj->obj_id, atomic_read(&inode->i_count)));
+               yaffs_trace(YAFFS_TRACE_OS,
+                       "yaffs_mknod created object %d count = %d",
+                       obj->obj_id, atomic_read(&inode->i_count));
                error = 0;
-               yaffs_fill_inode_from_obj(dir,parent);
+               yaffs_fill_inode_from_obj(dir, parent);
        } else {
-               T(YAFFS_TRACE_OS,
-                       (TSTR("yaffs_mknod failed making object\n")));
+               yaffs_trace(YAFFS_TRACE_OS, "yaffs_mknod failed making object");
                error = -ENOMEM;
        }
 
@@ -1772,10 +1714,10 @@ static int yaffs_mknod(struct inode *dir, struct dentry *dentry, int mode,
 
 static int yaffs_mkdir(struct inode *dir, struct dentry *dentry, int mode)
 {
-       int retVal;
-       T(YAFFS_TRACE_OS, (TSTR("yaffs_mkdir\n")));
-       retVal = yaffs_mknod(dir, dentry, mode | S_IFDIR, 0);
-       return retVal;
+       int ret_val;
+       yaffs_trace(YAFFS_TRACE_OS, "yaffs_mkdir");
+       ret_val = yaffs_mknod(dir, dentry, mode | S_IFDIR, 0);
+       return ret_val;
 }
 
 #if (LINUX_VERSION_CODE > KERNEL_VERSION(2, 5, 0))
@@ -1785,29 +1727,27 @@ static int yaffs_create(struct inode *dir, struct dentry *dentry, int mode,
 static int yaffs_create(struct inode *dir, struct dentry *dentry, int mode)
 #endif
 {
-       T(YAFFS_TRACE_OS,(TSTR("yaffs_create\n")));
+       yaffs_trace(YAFFS_TRACE_OS, "yaffs_create");
        return yaffs_mknod(dir, dentry, mode | S_IFREG, 0);
 }
 
 static int yaffs_unlink(struct inode *dir, struct dentry *dentry)
 {
-       int retVal;
+       int ret_val;
 
-       yaffs_dev_t *dev;
-       yaffs_obj_t *obj;
+       struct yaffs_dev *dev;
+       struct yaffs_obj *obj;
 
-       T(YAFFS_TRACE_OS,
-               (TSTR("yaffs_unlink %d:%s\n"),
-               (int)(dir->i_ino),
-               dentry->d_name.name));
-       obj = yaffs_InodeToObject(dir);
+       yaffs_trace(YAFFS_TRACE_OS, "yaffs_unlink %d:%s",
+               (int)(dir->i_ino), dentry->d_name.name);
+       obj = yaffs_inode_to_obj(dir);
        dev = obj->my_dev;
 
        yaffs_gross_lock(dev);
 
-       retVal = yaffs_unlinker(obj, dentry->d_name.name);
+       ret_val = yaffs_unlinker(obj, dentry->d_name.name);
 
-       if (retVal == YAFFS_OK) {
+       if (ret_val == YAFFS_OK) {
                dentry->d_inode->i_nlink--;
                dir->i_version++;
                yaffs_gross_unlock(dev);
@@ -1823,37 +1763,38 @@ static int yaffs_unlink(struct inode *dir, struct dentry *dentry)
  * Create a link...
  */
 static int yaffs_link(struct dentry *old_dentry, struct inode *dir,
-                       struct dentry *dentry)
+                     struct dentry *dentry)
 {
        struct inode *inode = old_dentry->d_inode;
-       yaffs_obj_t *obj = NULL;
-       yaffs_obj_t *link = NULL;
-       yaffs_dev_t *dev;
+       struct yaffs_obj *obj = NULL;
+       struct yaffs_obj *link = NULL;
+       struct yaffs_dev *dev;
 
-       T(YAFFS_TRACE_OS, (TSTR("yaffs_link\n")));
+       yaffs_trace(YAFFS_TRACE_OS, "yaffs_link");
 
-       obj = yaffs_InodeToObject(inode);
+       obj = yaffs_inode_to_obj(inode);
        dev = obj->my_dev;
 
        yaffs_gross_lock(dev);
 
-       if (!S_ISDIR(inode->i_mode))            /* Don't link directories */
-               link = yaffs_link_obj(yaffs_InodeToObject(dir), dentry->d_name.name,
-                       obj);
+       if (!S_ISDIR(inode->i_mode))    /* Don't link directories */
+               link =
+                   yaffs_link_obj(yaffs_inode_to_obj(dir), dentry->d_name.name,
+                                  obj);
 
        if (link) {
                old_dentry->d_inode->i_nlink = yaffs_get_obj_link_count(obj);
                d_instantiate(dentry, old_dentry->d_inode);
                atomic_inc(&old_dentry->d_inode->i_count);
-               T(YAFFS_TRACE_OS,
-                       (TSTR("yaffs_link link count %d i_count %d\n"),
+               yaffs_trace(YAFFS_TRACE_OS,
+                       "yaffs_link link count %d i_count %d",
                        old_dentry->d_inode->i_nlink,
-                       atomic_read(&old_dentry->d_inode->i_count)));
+                       atomic_read(&old_dentry->d_inode->i_count));
        }
 
        yaffs_gross_unlock(dev);
 
-       if (link){
+       if (link) {
                update_dir_time(dir);
                return 0;
        }
@@ -1862,19 +1803,20 @@ static int yaffs_link(struct dentry *old_dentry, struct inode *dir,
 }
 
 static int yaffs_symlink(struct inode *dir, struct dentry *dentry,
-                               const char *symname)
+                        const char *symname)
 {
-       yaffs_obj_t *obj;
-       yaffs_dev_t *dev;
+       struct yaffs_obj *obj;
+       struct yaffs_dev *dev;
        uid_t uid = YCRED(current)->fsuid;
-       gid_t gid = (dir->i_mode & S_ISGID) ? dir->i_gid : YCRED(current)->fsgid;
+       gid_t gid =
+           (dir->i_mode & S_ISGID) ? dir->i_gid : YCRED(current)->fsgid;
 
-       T(YAFFS_TRACE_OS, (TSTR("yaffs_symlink\n")));
+       yaffs_trace(YAFFS_TRACE_OS, "yaffs_symlink");
 
-       dev = yaffs_InodeToObject(dir)->my_dev;
+       dev = yaffs_inode_to_obj(dir)->my_dev;
        yaffs_gross_lock(dev);
-       obj = yaffs_create_symlink(yaffs_InodeToObject(dir), dentry->d_name.name,
-                               S_IFLNK | S_IRWXUGO, uid, gid, symname);
+       obj = yaffs_create_symlink(yaffs_inode_to_obj(dir), dentry->d_name.name,
+                                  S_IFLNK | S_IRWXUGO, uid, gid, symname);
        yaffs_gross_unlock(dev);
 
        if (obj) {
@@ -1883,10 +1825,10 @@ static int yaffs_symlink(struct inode *dir, struct dentry *dentry,
                inode = yaffs_get_inode(dir->i_sb, obj->yst_mode, 0, obj);
                d_instantiate(dentry, inode);
                update_dir_time(dir);
-               T(YAFFS_TRACE_OS, (TSTR("symlink created OK\n")));
+               yaffs_trace(YAFFS_TRACE_OS, "symlink created OK");
                return 0;
        } else {
-               T(YAFFS_TRACE_OS, (TSTR("symlink not created\n")));
+               yaffs_trace(YAFFS_TRACE_OS, "symlink not created");
        }
 
        return -ENOMEM;
@@ -1896,12 +1838,12 @@ static int yaffs_symlink(struct inode *dir, struct dentry *dentry,
 static int yaffs_sync_object(struct file *file, int datasync)
 #else
 static int yaffs_sync_object(struct file *file, struct dentry *dentry,
-                               int datasync)
+                            int datasync)
 #endif
 {
 
-       yaffs_obj_t *obj;
-       yaffs_dev_t *dev;
+       struct yaffs_obj *obj;
+       struct yaffs_dev *dev;
 #if (LINUX_VERSION_CODE > KERNEL_VERSION(2, 6, 34))
        struct dentry *dentry = file->f_path.dentry;
 #endif
@@ -1910,8 +1852,8 @@ static int yaffs_sync_object(struct file *file, struct dentry *dentry,
 
        dev = obj->my_dev;
 
-       T(YAFFS_TRACE_OS | YAFFS_TRACE_SYNC,
-               (TSTR("yaffs_sync_object\n")));
+       yaffs_trace(YAFFS_TRACE_OS | YAFFS_TRACE_SYNC,
+               "yaffs_sync_object");
        yaffs_gross_lock(dev);
        yaffs_flush_file(obj, 1, datasync);
        yaffs_gross_unlock(dev);
@@ -1926,46 +1868,44 @@ static int yaffs_sync_object(struct file *file, struct dentry *dentry,
 static int yaffs_rename(struct inode *old_dir, struct dentry *old_dentry,
                        struct inode *new_dir, struct dentry *new_dentry)
 {
-       yaffs_dev_t *dev;
-       int retVal = YAFFS_FAIL;
-       yaffs_obj_t *target;
+       struct yaffs_dev *dev;
+       int ret_val = YAFFS_FAIL;
+       struct yaffs_obj *target;
 
-       T(YAFFS_TRACE_OS, (TSTR("yaffs_rename\n")));
-       dev = yaffs_InodeToObject(old_dir)->my_dev;
+       yaffs_trace(YAFFS_TRACE_OS, "yaffs_rename");
+       dev = yaffs_inode_to_obj(old_dir)->my_dev;
 
        yaffs_gross_lock(dev);
 
        /* Check if the target is an existing directory that is not empty. */
-       target = yaffs_find_by_name(yaffs_InodeToObject(new_dir),
-                               new_dentry->d_name.name);
-
-
+       target = yaffs_find_by_name(yaffs_inode_to_obj(new_dir),
+                                   new_dentry->d_name.name);
 
        if (target && target->variant_type == YAFFS_OBJECT_TYPE_DIRECTORY &&
-               !ylist_empty(&target->variant.dir_variant.children)) {
+           !list_empty(&target->variant.dir_variant.children)) {
 
-               T(YAFFS_TRACE_OS, (TSTR("target is non-empty dir\n")));
+               yaffs_trace(YAFFS_TRACE_OS, "target is non-empty dir");
 
-               retVal = YAFFS_FAIL;
+               ret_val = YAFFS_FAIL;
        } else {
                /* Now does unlinking internally using shadowing mechanism */
-               T(YAFFS_TRACE_OS, (TSTR("calling yaffs_rename_obj\n")));
+               yaffs_trace(YAFFS_TRACE_OS, "calling yaffs_rename_obj");
 
-               retVal = yaffs_rename_obj(yaffs_InodeToObject(old_dir),
-                               old_dentry->d_name.name,
-                               yaffs_InodeToObject(new_dir),
-                               new_dentry->d_name.name);
+               ret_val = yaffs_rename_obj(yaffs_inode_to_obj(old_dir),
+                                          old_dentry->d_name.name,
+                                          yaffs_inode_to_obj(new_dir),
+                                          new_dentry->d_name.name);
        }
        yaffs_gross_unlock(dev);
 
-       if (retVal == YAFFS_OK) {
+       if (ret_val == YAFFS_OK) {
                if (target) {
                        new_dentry->d_inode->i_nlink--;
                        mark_inode_dirty(new_dentry->d_inode);
                }
-               
+
                update_dir_time(old_dir);
-               if(old_dir != new_dir)
+               if (old_dir != new_dir)
                        update_dir_time(new_dir);
                return 0;
        } else {
@@ -1977,37 +1917,38 @@ static int yaffs_setattr(struct dentry *dentry, struct iattr *attr)
 {
        struct inode *inode = dentry->d_inode;
        int error = 0;
-       yaffs_dev_t *dev;
+       struct yaffs_dev *dev;
 
-       T(YAFFS_TRACE_OS,
-               (TSTR("yaffs_setattr of object %d\n"),
-               yaffs_InodeToObject(inode)->obj_id));
+       yaffs_trace(YAFFS_TRACE_OS,
+               "yaffs_setattr of object %d",
+               yaffs_inode_to_obj(inode)->obj_id);
 
-       /* Fail if a requested resize >= 2GB */         
-       if (attr->ia_valid & ATTR_SIZE &&
-               (attr->ia_size >> 31))
+       /* Fail if a requested resize >= 2GB */
+       if (attr->ia_valid & ATTR_SIZE && (attr->ia_size >> 31))
                error = -EINVAL;
 
        if (error == 0)
                error = inode_change_ok(inode, attr);
        if (error == 0) {
                int result;
-               if (!error){
+               if (!error) {
                        error = yaffs_vfs_setattr(inode, attr);
-                       T(YAFFS_TRACE_OS,(TSTR("inode_setattr called\n")));
-                       if (attr->ia_valid & ATTR_SIZE){
-                               yaffs_vfs_setsize(inode,attr->ia_size);
-                               inode->i_blocks = (inode->i_size + 511) >> 9;
+                       yaffs_trace(YAFFS_TRACE_OS, "inode_setattr called");
+                       if (attr->ia_valid & ATTR_SIZE) {
+                               yaffs_vfs_setsize(inode, attr->ia_size);
+                               inode->i_blocks = (inode->i_size + 511) >> 9;
                        }
                }
-               dev = yaffs_InodeToObject(inode)->my_dev;
-               if (attr->ia_valid & ATTR_SIZE){
-                       T(YAFFS_TRACE_OS,(TSTR("resize to %d(%x)\n"),
-                               (int)(attr->ia_size),(int)(attr->ia_size)));
+               dev = yaffs_inode_to_obj(inode)->my_dev;
+               if (attr->ia_valid & ATTR_SIZE) {
+                       yaffs_trace(YAFFS_TRACE_OS,
+                               "resize to %d(%x)",
+                               (int)(attr->ia_size),
+                               (int)(attr->ia_size));
                }
                yaffs_gross_lock(dev);
-               result = yaffs_set_attribs(yaffs_InodeToObject(inode), attr);
-               if(result == YAFFS_OK) {
+               result = yaffs_set_attribs(yaffs_inode_to_obj(inode), attr);
+               if (result == YAFFS_OK) {
                        error = 0;
                } else {
                        error = -EPERM;
@@ -2016,56 +1957,50 @@ static int yaffs_setattr(struct dentry *dentry, struct iattr *attr)
 
        }
 
-       T(YAFFS_TRACE_OS,
-               (TSTR("yaffs_setattr done returning %d\n"),error));
+       yaffs_trace(YAFFS_TRACE_OS, "yaffs_setattr done returning %d", error);
 
        return error;
 }
 
 #ifdef CONFIG_YAFFS_XATTR
-int yaffs_setxattr(struct dentry *dentry, const char *name,
-                       const void *value, size_t size, int flags)
+static int yaffs_setxattr(struct dentry *dentry, const char *name,
+                  const void *value, size_t size, int flags)
 {
        struct inode *inode = dentry->d_inode;
        int error = 0;
-       yaffs_dev_t *dev;
-       yaffs_obj_t *obj = yaffs_InodeToObject(inode);
-
-       T(YAFFS_TRACE_OS,
-               (TSTR("yaffs_setxattr of object %d\n"),
-               obj->obj_id));
+       struct yaffs_dev *dev;
+       struct yaffs_obj *obj = yaffs_inode_to_obj(inode);
 
+       yaffs_trace(YAFFS_TRACE_OS, "yaffs_setxattr of object %d", obj->obj_id);
 
        if (error == 0) {
                int result;
                dev = obj->my_dev;
                yaffs_gross_lock(dev);
                result = yaffs_set_xattrib(obj, name, value, size, flags);
-               if(result == YAFFS_OK)
+               if (result == YAFFS_OK)
                        error = 0;
-               else if(result < 0)
+               else if (result < 0)
                        error = result;
                yaffs_gross_unlock(dev);
 
        }
-       T(YAFFS_TRACE_OS,
-               (TSTR("yaffs_setxattr done returning %d\n"),error));
+       yaffs_trace(YAFFS_TRACE_OS, "yaffs_setxattr done returning %d", error);
 
        return error;
 }
 
-
-ssize_t yaffs_getxattr(struct dentry *dentry, const char *name, void *buff,
-                       size_t size)
+static ssize_t yaffs_getxattr(struct dentry * dentry, const char *name,
+                       void *buff, size_t size)
 {
        struct inode *inode = dentry->d_inode;
        int error = 0;
-       yaffs_dev_t *dev;
-       yaffs_obj_t *obj = yaffs_InodeToObject(inode);
+       struct yaffs_dev *dev;
+       struct yaffs_obj *obj = yaffs_inode_to_obj(inode);
 
-       T(YAFFS_TRACE_OS,
-               (TSTR("yaffs_getxattr \"%s\" from object %d\n"),
-               name, obj->obj_id));
+       yaffs_trace(YAFFS_TRACE_OS,
+               "yaffs_getxattr \"%s\" from object %d",
+               name, obj->obj_id);
 
        if (error == 0) {
                dev = obj->my_dev;
@@ -2074,53 +2009,48 @@ ssize_t yaffs_getxattr(struct dentry *dentry, const char *name, void *buff,
                yaffs_gross_unlock(dev);
 
        }
-       T(YAFFS_TRACE_OS,
-               (TSTR("yaffs_getxattr done returning %d\n"),error));
+       yaffs_trace(YAFFS_TRACE_OS, "yaffs_getxattr done returning %d", error);
 
        return error;
 }
 
-int yaffs_removexattr(struct dentry *dentry, const char *name)
+static int yaffs_removexattr(struct dentry *dentry, const char *name)
 {
        struct inode *inode = dentry->d_inode;
        int error = 0;
-       yaffs_dev_t *dev;
-       yaffs_obj_t *obj = yaffs_InodeToObject(inode);
-
-       T(YAFFS_TRACE_OS,
-               (TSTR("yaffs_removexattr of object %d\n"),
-               obj->obj_id));
+       struct yaffs_dev *dev;
+       struct yaffs_obj *obj = yaffs_inode_to_obj(inode);
 
+       yaffs_trace(YAFFS_TRACE_OS,
+               "yaffs_removexattr of object %d", obj->obj_id);
 
        if (error == 0) {
                int result;
                dev = obj->my_dev;
                yaffs_gross_lock(dev);
                result = yaffs_remove_xattrib(obj, name);
-               if(result == YAFFS_OK)
+               if (result == YAFFS_OK)
                        error = 0;
-               else if(result < 0)
+               else if (result < 0)
                        error = result;
                yaffs_gross_unlock(dev);
 
        }
-       T(YAFFS_TRACE_OS,
-               (TSTR("yaffs_removexattr done returning %d\n"),error));
+       yaffs_trace(YAFFS_TRACE_OS,
+               "yaffs_removexattr done returning %d", error);
 
        return error;
 }
 
-ssize_t yaffs_listxattr(struct dentry *dentry, char *buff, size_t size)
+static ssize_t yaffs_listxattr(struct dentry * dentry, char *buff, size_t size)
 {
        struct inode *inode = dentry->d_inode;
        int error = 0;
-       yaffs_dev_t *dev;
-       yaffs_obj_t *obj = yaffs_InodeToObject(inode);
-
-       T(YAFFS_TRACE_OS,
-               (TSTR("yaffs_listxattr of object %d\n"),
-               obj->obj_id));
+       struct yaffs_dev *dev;
+       struct yaffs_obj *obj = yaffs_inode_to_obj(inode);
 
+       yaffs_trace(YAFFS_TRACE_OS,
+               "yaffs_listxattr of object %d", obj->obj_id);
 
        if (error == 0) {
                dev = obj->my_dev;
@@ -2129,31 +2059,30 @@ ssize_t yaffs_listxattr(struct dentry *dentry, char *buff, size_t size)
                yaffs_gross_unlock(dev);
 
        }
-       T(YAFFS_TRACE_OS,
-               (TSTR("yaffs_listxattr done returning %d\n"),error));
+       yaffs_trace(YAFFS_TRACE_OS,
+               "yaffs_listxattr done returning %d", error);
 
        return error;
 }
 
 #endif
 
-
 #if (LINUX_VERSION_CODE > KERNEL_VERSION(2, 6, 17))
 static int yaffs_statfs(struct dentry *dentry, struct kstatfs *buf)
 {
-       yaffs_dev_t *dev = yaffs_dentry_to_obj(dentry)->my_dev;
+       struct yaffs_dev *dev = yaffs_dentry_to_obj(dentry)->my_dev;
        struct super_block *sb = dentry->d_sb;
 #elif (LINUX_VERSION_CODE > KERNEL_VERSION(2, 5, 0))
 static int yaffs_statfs(struct super_block *sb, struct kstatfs *buf)
 {
-       yaffs_dev_t *dev = yaffs_SuperToDevice(sb);
+       struct yaffs_dev *dev = yaffs_super_to_dev(sb);
 #else
 static int yaffs_statfs(struct super_block *sb, struct statfs *buf)
 {
-       yaffs_dev_t *dev = yaffs_SuperToDevice(sb);
+       struct yaffs_dev *dev = yaffs_super_to_dev(sb);
 #endif
 
-       T(YAFFS_TRACE_OS, (TSTR("yaffs_statfs\n")));
+       yaffs_trace(YAFFS_TRACE_OS, "yaffs_statfs");
 
        yaffs_gross_lock(dev);
 
@@ -2164,40 +2093,43 @@ static int yaffs_statfs(struct super_block *sb, struct statfs *buf)
        if (dev->data_bytes_per_chunk & (dev->data_bytes_per_chunk - 1)) {
                /* Do this if chunk size is not a power of 2 */
 
-               uint64_t bytesInDev;
-               uint64_t bytesFree;
+               uint64_t bytes_in_dev;
+               uint64_t bytes_free;
 
-               bytesInDev = ((uint64_t)((dev->param.end_block - dev->param.start_block + 1))) *
-                       ((uint64_t)(dev->param.chunks_per_block * dev->data_bytes_per_chunk));
+               bytes_in_dev =
+                   ((uint64_t)
+                    ((dev->param.end_block - dev->param.start_block +
+                      1))) * ((uint64_t) (dev->param.chunks_per_block *
+                                          dev->data_bytes_per_chunk));
 
-               do_div(bytesInDev, sb->s_blocksize); /* bytesInDev becomes the number of blocks */
-               buf->f_blocks = bytesInDev;
+               do_div(bytes_in_dev, sb->s_blocksize);  /* bytes_in_dev becomes the number of blocks */
+               buf->f_blocks = bytes_in_dev;
 
-               bytesFree  = ((uint64_t)(yaffs_get_n_free_chunks(dev))) *
-                       ((uint64_t)(dev->data_bytes_per_chunk));
+               bytes_free = ((uint64_t) (yaffs_get_n_free_chunks(dev))) *
+                   ((uint64_t) (dev->data_bytes_per_chunk));
 
-               do_div(bytesFree, sb->s_blocksize);
+               do_div(bytes_free, sb->s_blocksize);
 
-               buf->f_bfree = bytesFree;
+               buf->f_bfree = bytes_free;
 
        } else if (sb->s_blocksize > dev->data_bytes_per_chunk) {
 
                buf->f_blocks =
-                       (dev->param.end_block - dev->param.start_block + 1) *
-                       dev->param.chunks_per_block /
-                       (sb->s_blocksize / dev->data_bytes_per_chunk);
+                   (dev->param.end_block - dev->param.start_block + 1) *
+                   dev->param.chunks_per_block /
+                   (sb->s_blocksize / dev->data_bytes_per_chunk);
                buf->f_bfree =
-                       yaffs_get_n_free_chunks(dev) /
-                       (sb->s_blocksize / dev->data_bytes_per_chunk);
+                   yaffs_get_n_free_chunks(dev) /
+                   (sb->s_blocksize / dev->data_bytes_per_chunk);
        } else {
                buf->f_blocks =
-                       (dev->param.end_block - dev->param.start_block + 1) *
-                       dev->param.chunks_per_block *
-                       (dev->data_bytes_per_chunk / sb->s_blocksize);
+                   (dev->param.end_block - dev->param.start_block + 1) *
+                   dev->param.chunks_per_block *
+                   (dev->data_bytes_per_chunk / sb->s_blocksize);
 
                buf->f_bfree =
-                       yaffs_get_n_free_chunks(dev) *
-                       (dev->data_bytes_per_chunk / sb->s_blocksize);
+                   yaffs_get_n_free_chunks(dev) *
+                   (dev->data_bytes_per_chunk / sb->s_blocksize);
        }
 
        buf->f_files = 0;
@@ -2208,84 +2140,80 @@ static int yaffs_statfs(struct super_block *sb, struct statfs *buf)
        return 0;
 }
 
-
-
 static void yaffs_flush_inodes(struct super_block *sb)
 {
        struct inode *iptr;
-       yaffs_obj_t *obj;
-       
-       list_for_each_entry(iptr,&sb->s_inodes, i_sb_list){
-               obj = yaffs_InodeToObject(iptr);
-               if(obj){
-                       T(YAFFS_TRACE_OS, (TSTR("flushing obj %d\n"),
-                               obj->obj_id));
-                       yaffs_flush_file(obj,1,0);
+       struct yaffs_obj *obj;
+
+       list_for_each_entry(iptr, &sb->s_inodes, i_sb_list) {
+               obj = yaffs_inode_to_obj(iptr);
+               if (obj) {
+                       yaffs_trace(YAFFS_TRACE_OS,
+                               "flushing obj %d",
+                               obj->obj_id);
+                       yaffs_flush_file(obj, 1, 0);
                }
        }
 }
 
-
 static void yaffs_flush_super(struct super_block *sb, int do_checkpoint)
 {
-       yaffs_dev_t *dev = yaffs_SuperToDevice(sb);     
-       if(!dev)
+       struct yaffs_dev *dev = yaffs_super_to_dev(sb);
+       if (!dev)
                return;
-       
+
        yaffs_flush_inodes(sb);
        yaffs_update_dirty_dirs(dev);
        yaffs_flush_whole_cache(dev);
-       if(do_checkpoint)
+       if (do_checkpoint)
                yaffs_checkpoint_save(dev);
 }
 
-
-static unsigned yaffs_bg_gc_urgency(yaffs_dev_t *dev)
+static unsigned yaffs_bg_gc_urgency(struct yaffs_dev *dev)
 {
-       unsigned erasedChunks = dev->n_erased_blocks * dev->param.chunks_per_block;
-       struct yaffs_LinuxContext *context = yaffs_dev_to_lc(dev);
-       unsigned scatteredFree = 0; /* Free chunks not in an erased block */
+       unsigned erased_chunks =
+           dev->n_erased_blocks * dev->param.chunks_per_block;
+       struct yaffs_linux_context *context = yaffs_dev_to_lc(dev);
+       unsigned scattered = 0; /* Free chunks not in an erased block */
 
-       if(erasedChunks < dev->n_free_chunks)
-               scatteredFree = (dev->n_free_chunks - erasedChunks);
+       if (erased_chunks < dev->n_free_chunks)
+               scattered = (dev->n_free_chunks - erased_chunks);
 
-       if(!context->bgRunning)
+       if (!context->bg_running)
                return 0;
-       else if(scatteredFree < (dev->param.chunks_per_block * 2))
+       else if (scattered < (dev->param.chunks_per_block * 2))
                return 0;
-       else if(erasedChunks > dev->n_free_chunks/2)
+       else if (erased_chunks > dev->n_free_chunks / 2)
                return 0;
-       else if(erasedChunks > dev->n_free_chunks/4)
+       else if (erased_chunks > dev->n_free_chunks / 4)
                return 1;
        else
                return 2;
 }
 
-static int yaffs_do_sync_fs(struct super_block *sb,
-                               int request_checkpoint)
+static int yaffs_do_sync_fs(struct super_block *sb, int request_checkpoint)
 {
 
-       yaffs_dev_t *dev = yaffs_SuperToDevice(sb);
+       struct yaffs_dev *dev = yaffs_super_to_dev(sb);
        unsigned int oneshot_checkpoint = (yaffs_auto_checkpoint & 4);
        unsigned gc_urgent = yaffs_bg_gc_urgency(dev);
        int do_checkpoint;
 
-       T(YAFFS_TRACE_OS | YAFFS_TRACE_SYNC | YAFFS_TRACE_BACKGROUND,
-               (TSTR("yaffs_do_sync_fs: gc-urgency %d %s %s%s\n"),
+       yaffs_trace(YAFFS_TRACE_OS | YAFFS_TRACE_SYNC | YAFFS_TRACE_BACKGROUND,
+               "yaffs_do_sync_fs: gc-urgency %d %s %s%s",
                gc_urgent,
                sb->s_dirt ? "dirty" : "clean",
                request_checkpoint ? "checkpoint requested" : "no checkpoint",
-               oneshot_checkpoint ? " one-shot" : "" ));
+               oneshot_checkpoint ? " one-shot" : "");
 
        yaffs_gross_lock(dev);
        do_checkpoint = ((request_checkpoint && !gc_urgent) ||
-                       oneshot_checkpoint) &&
-                       !dev->is_checkpointed;
+                        oneshot_checkpoint) && !dev->is_checkpointed;
 
        if (sb->s_dirt || do_checkpoint) {
                yaffs_flush_super(sb, !dev->is_checkpointed && do_checkpoint);
                sb->s_dirt = 0;
-               if(oneshot_checkpoint)
+               if (oneshot_checkpoint)
                        yaffs_auto_checkpoint &= ~4;
        }
        yaffs_gross_unlock(dev);
@@ -2299,7 +2227,7 @@ static int yaffs_do_sync_fs(struct super_block *sb,
  * yaffs_bg_start() launches the background thread.
  * yaffs_bg_stop() cleans up the background thread.
  *
- * NB: 
+ * NB:
  * The thread should only run after the yaffs is initialised
  * The thread should be stopped before yaffs is unmounted.
  * The thread should not do any writing while the fs is in read only.
@@ -2314,74 +2242,74 @@ void yaffs_background_waker(unsigned long data)
 
 static int yaffs_bg_thread_fn(void *data)
 {
-       yaffs_dev_t *dev = (yaffs_dev_t *)data;
-       struct yaffs_LinuxContext *context = yaffs_dev_to_lc(dev);
+       struct yaffs_dev *dev = (struct yaffs_dev *)data;
+       struct yaffs_linux_context *context = yaffs_dev_to_lc(dev);
        unsigned long now = jiffies;
        unsigned long next_dir_update = now;
        unsigned long next_gc = now;
        unsigned long expires;
        unsigned int urgency;
 
-       int gcResult;
+       int gc_result;
        struct timer_list timer;
 
-       T(YAFFS_TRACE_BACKGROUND,
-               (TSTR("yaffs_background starting for dev %p\n"),
-               (void *)dev));
+       yaffs_trace(YAFFS_TRACE_BACKGROUND,
+               "yaffs_background starting for dev %p", (void *)dev);
 
 #ifdef YAFFS_COMPILE_FREEZER
        set_freezable();
 #endif
-       while(context->bgRunning){
-               T(YAFFS_TRACE_BACKGROUND,
-                       (TSTR("yaffs_background\n")));
+       while (context->bg_running) {
+               yaffs_trace(YAFFS_TRACE_BACKGROUND, "yaffs_background");
 
-               if(kthread_should_stop())
+               if (kthread_should_stop())
                        break;
 
 #ifdef YAFFS_COMPILE_FREEZER
-               if(try_to_freeze())
+               if (try_to_freeze())
                        continue;
 #endif
                yaffs_gross_lock(dev);
 
                now = jiffies;
 
-               if(time_after(now, next_dir_update) && yaffs_bg_enable){
+               if (time_after(now, next_dir_update) && yaffs_bg_enable) {
                        yaffs_update_dirty_dirs(dev);
                        next_dir_update = now + HZ;
                }
 
-               if(time_after(now,next_gc) && yaffs_bg_enable){
-                       if(!dev->is_checkpointed){
+               if (time_after(now, next_gc) && yaffs_bg_enable) {
+                       if (!dev->is_checkpointed) {
                                urgency = yaffs_bg_gc_urgency(dev);
-                               gcResult = yaffs_bg_gc(dev, urgency);
-                               if(urgency > 1)
-                                       next_gc = now + HZ/20+1;
-                               else if(urgency > 0)
-                                       next_gc = now + HZ/10+1;
+                               gc_result = yaffs_bg_gc(dev, urgency);
+                               if (urgency > 1)
+                                       next_gc = now + HZ / 20 + 1;
+                               else if (urgency > 0)
+                                       next_gc = now + HZ / 10 + 1;
                                else
                                        next_gc = now + HZ * 2;
-                       } else /*
-                               * gc not running so set to next_dir_update
-                               * to cut down on wake ups
-                               */
+                       } else  {
+                               /*
+                                * gc not running so set to next_dir_update
+                                * to cut down on wake ups
+                                */
                                next_gc = next_dir_update;
+                        }
                }
                yaffs_gross_unlock(dev);
 #if 1
                expires = next_dir_update;
-               if (time_before(next_gc,expires))
+               if (time_before(next_gc, expires))
                        expires = next_gc;
-               if(time_before(expires,now))
+               if (time_before(expires, now))
                        expires = now + HZ;
 
                Y_INIT_TIMER(&timer);
-               timer.expires = expires+1;
-               timer.data = (unsigned long) current;
+               timer.expires = expires + 1;
+               timer.data = (unsigned long)current;
                timer.function = yaffs_background_waker;
 
-                set_current_state(TASK_INTERRUPTIBLE);
+               set_current_state(TASK_INTERRUPTIBLE);
                add_timer(&timer);
                schedule();
                del_timer_sync(&timer);
@@ -2393,36 +2321,37 @@ static int yaffs_bg_thread_fn(void *data)
        return 0;
 }
 
-static int yaffs_bg_start(yaffs_dev_t *dev)
+static int yaffs_bg_start(struct yaffs_dev *dev)
 {
        int retval = 0;
-       struct yaffs_LinuxContext *context = yaffs_dev_to_lc(dev);
+       struct yaffs_linux_context *context = yaffs_dev_to_lc(dev);
 
-       if(dev->read_only)
+       if (dev->read_only)
                return -1;
 
-       context->bgRunning = 1;
+       context->bg_running = 1;
 
-       context->bgThread = kthread_run(yaffs_bg_thread_fn,
-                               (void *)dev,"yaffs-bg-%d",context->mount_id);
+       context->bg_thread = kthread_run(yaffs_bg_thread_fn,
+                                        (void *)dev, "yaffs-bg-%d",
+                                        context->mount_id);
 
-       if(IS_ERR(context->bgThread)){
-               retval = PTR_ERR(context->bgThread);
-               context->bgThread = NULL;
-               context->bgRunning = 0;
+       if (IS_ERR(context->bg_thread)) {
+               retval = PTR_ERR(context->bg_thread);
+               context->bg_thread = NULL;
+               context->bg_running = 0;
        }
        return retval;
 }
 
-static void yaffs_bg_stop(yaffs_dev_t *dev)
+static void yaffs_bg_stop(struct yaffs_dev *dev)
 {
-       struct yaffs_LinuxContext *ctxt = yaffs_dev_to_lc(dev);
+       struct yaffs_linux_context *ctxt = yaffs_dev_to_lc(dev);
 
-       ctxt->bgRunning = 0;
+       ctxt->bg_running = 0;
 
-       if( ctxt->bgThread){
-               kthread_stop(ctxt->bgThread);
-               ctxt->bgThread = NULL;
+       if (ctxt->bg_thread) {
+               kthread_stop(ctxt->bg_thread);
+               ctxt->bg_thread = NULL;
        }
 }
 #else
@@ -2431,17 +2360,16 @@ static int yaffs_bg_thread_fn(void *data)
        return 0;
 }
 
-static int yaffs_bg_start(yaffs_dev_t *dev)
+static int yaffs_bg_start(struct yaffs_dev *dev)
 {
        return 0;
 }
 
-static void yaffs_bg_stop(yaffs_dev_t *dev)
+static void yaffs_bg_stop(struct yaffs_dev *dev)
 {
 }
 #endif
 
-
 #if (LINUX_VERSION_CODE > KERNEL_VERSION(2, 6, 17))
 static void yaffs_write_super(struct super_block *sb)
 #else
@@ -2450,9 +2378,9 @@ static int yaffs_write_super(struct super_block *sb)
 {
        unsigned request_checkpoint = (yaffs_auto_checkpoint >= 2);
 
-       T(YAFFS_TRACE_OS | YAFFS_TRACE_SYNC | YAFFS_TRACE_BACKGROUND,
-               (TSTR("yaffs_write_super%s\n"),
-               request_checkpoint ? " checkpt" : ""));
+       yaffs_trace(YAFFS_TRACE_OS | YAFFS_TRACE_SYNC | YAFFS_TRACE_BACKGROUND,
+               "yaffs_write_super %s",
+               request_checkpoint ? " checkpt" : "");
 
        yaffs_do_sync_fs(sb, request_checkpoint);
 
@@ -2461,7 +2389,6 @@ static int yaffs_write_super(struct super_block *sb)
 #endif
 }
 
-
 #if (LINUX_VERSION_CODE > KERNEL_VERSION(2, 6, 17))
 static int yaffs_sync_fs(struct super_block *sb, int wait)
 #else
@@ -2470,9 +2397,8 @@ static int yaffs_sync_fs(struct super_block *sb)
 {
        unsigned request_checkpoint = (yaffs_auto_checkpoint >= 1);
 
-       T(YAFFS_TRACE_OS | YAFFS_TRACE_SYNC,
-               (TSTR("yaffs_sync_fs%s\n"),
-               request_checkpoint ? " checkpt" : ""));
+       yaffs_trace(YAFFS_TRACE_OS | YAFFS_TRACE_SYNC,
+               "yaffs_sync_fs%s", request_checkpoint ? " checkpt" : "");
 
        yaffs_do_sync_fs(sb, request_checkpoint);
 
@@ -2484,11 +2410,10 @@ static int yaffs_sync_fs(struct super_block *sb)
 static struct inode *yaffs_iget(struct super_block *sb, unsigned long ino)
 {
        struct inode *inode;
-       yaffs_obj_t *obj;
-       yaffs_dev_t *dev = yaffs_SuperToDevice(sb);
+       struct yaffs_obj *obj;
+       struct yaffs_dev *dev = yaffs_super_to_dev(sb);
 
-       T(YAFFS_TRACE_OS,
-               (TSTR("yaffs_iget for %lu\n"), ino));
+       yaffs_trace(YAFFS_TRACE_OS, "yaffs_iget for %lu", ino);
 
        inode = iget_locked(sb, ino);
        if (!inode)
@@ -2522,68 +2447,66 @@ static void yaffs_read_inode(struct inode *inode)
         * need to lock again.
         */
 
-       yaffs_obj_t *obj;
-       yaffs_dev_t *dev = yaffs_SuperToDevice(inode->i_sb);
+       struct yaffs_obj *obj;
+       struct yaffs_dev *dev = yaffs_super_to_dev(inode->i_sb);
 
-       T(YAFFS_TRACE_OS,
-               (TSTR("yaffs_read_inode for %d\n"), (int)inode->i_ino));
+       yaffs_trace(YAFFS_TRACE_OS,
+               "yaffs_read_inode for %d", (int)inode->i_ino);
 
-       if(current != yaffs_dev_to_lc(dev)->readdirProcess)
+       if (current != yaffs_dev_to_lc(dev)->readdir_process)
                yaffs_gross_lock(dev);
 
        obj = yaffs_find_by_number(dev, inode->i_ino);
 
        yaffs_fill_inode_from_obj(inode, obj);
 
-       if(current != yaffs_dev_to_lc(dev)->readdirProcess)
+       if (current != yaffs_dev_to_lc(dev)->readdir_process)
                yaffs_gross_unlock(dev);
 }
 
 #endif
 
-static YLIST_HEAD(yaffs_context_list);
-struct semaphore yaffs_context_lock;
+static LIST_HEAD(yaffs_context_list);
+struct mutex yaffs_context_lock;
 
 static void yaffs_put_super(struct super_block *sb)
 {
-       yaffs_dev_t *dev = yaffs_SuperToDevice(sb);
+       struct yaffs_dev *dev = yaffs_super_to_dev(sb);
 
-       T(YAFFS_TRACE_OS, (TSTR("yaffs_put_super\n")));
+       yaffs_trace(YAFFS_TRACE_OS, "yaffs_put_super");
 
-       T(YAFFS_TRACE_OS | YAFFS_TRACE_BACKGROUND,
-               (TSTR("Shutting down yaffs background thread\n")));
+       yaffs_trace(YAFFS_TRACE_OS | YAFFS_TRACE_BACKGROUND,
+               "Shutting down yaffs background thread");
        yaffs_bg_stop(dev);
-       T(YAFFS_TRACE_OS | YAFFS_TRACE_BACKGROUND,
-               (TSTR("yaffs background thread shut down\n")));
+       yaffs_trace(YAFFS_TRACE_OS | YAFFS_TRACE_BACKGROUND,
+               "yaffs background thread shut down");
 
        yaffs_gross_lock(dev);
 
-       yaffs_flush_super(sb,1);
-
-       if (yaffs_dev_to_lc(dev)->putSuperFunc)
-               yaffs_dev_to_lc(dev)->putSuperFunc(sb);
+       yaffs_flush_super(sb, 1);
 
+       if (yaffs_dev_to_lc(dev)->put_super_fn)
+               yaffs_dev_to_lc(dev)->put_super_fn(sb);
 
        yaffs_deinitialise(dev);
 
        yaffs_gross_unlock(dev);
 
-       down(&yaffs_context_lock);
-       ylist_del_init(&(yaffs_dev_to_lc(dev)->contextList));
-       up(&yaffs_context_lock);
+       mutex_lock(&yaffs_context_lock);
+       list_del_init(&(yaffs_dev_to_lc(dev)->context_list));
+       mutex_unlock(&yaffs_context_lock);
 
-       if (yaffs_dev_to_lc(dev)->spareBuffer) {
-               YFREE(yaffs_dev_to_lc(dev)->spareBuffer);
-               yaffs_dev_to_lc(dev)->spareBuffer = NULL;
+       if (yaffs_dev_to_lc(dev)->spare_buffer) {
+               kfree(yaffs_dev_to_lc(dev)->spare_buffer);
+               yaffs_dev_to_lc(dev)->spare_buffer = NULL;
        }
 
        kfree(dev);
 }
 
-
-static void yaffs_MTDPutSuper(struct super_block *sb)
+static void yaffs_mtd_put_super(struct super_block *sb)
 {
-       struct mtd_info *mtd = yaffs_dev_to_mtd(yaffs_SuperToDevice(sb));
+       struct mtd_info *mtd = yaffs_dev_to_mtd(yaffs_super_to_dev(sb));
 
        if (mtd->sync)
                mtd->sync(mtd);
@@ -2591,17 +2514,16 @@ static void yaffs_MTDPutSuper(struct super_block *sb)
        put_mtd_device(mtd);
 }
 
-
-static void yaffs_touch_super(yaffs_dev_t *dev)
+static void yaffs_touch_super(struct yaffs_dev *dev)
 {
-       struct super_block *sb = yaffs_dev_to_lc(dev)->superBlock;
+       struct super_block *sb = yaffs_dev_to_lc(dev)->super;
 
-       T(YAFFS_TRACE_OS, (TSTR("yaffs_touch_super() sb = %p\n"), sb));
+       yaffs_trace(YAFFS_TRACE_OS, "yaffs_touch_super() sb = %p", sb);
        if (sb)
                sb->s_dirt = 1;
 }
 
-typedef struct {
+struct yaffs_options {
        int inband_tags;
        int skip_checkpoint_read;
        int skip_checkpoint_write;
@@ -2612,10 +2534,11 @@ typedef struct {
        int lazy_loading_overridden;
        int empty_lost_and_found;
        int empty_lost_and_found_overridden;
-} yaffs_options;
+};
 
 #define MAX_OPT_LEN 30
-static int yaffs_parse_options(yaffs_options *options, const char *options_str)
+static int yaffs_parse_options(struct yaffs_options *options,
+                              const char *options_str)
 {
        char cur_opt[MAX_OPT_LEN + 1];
        int p;
@@ -2627,7 +2550,7 @@ static int yaffs_parse_options(yaffs_options *options, const char *options_str)
                memset(cur_opt, 0, MAX_OPT_LEN + 1);
                p = 0;
 
-               while(*options_str == ',')
+               while (*options_str == ',')
                        options_str++;
 
                while (*options_str && *options_str != ',') {
@@ -2638,38 +2561,38 @@ static int yaffs_parse_options(yaffs_options *options, const char *options_str)
                        options_str++;
                }
 
-               if (!strcmp(cur_opt, "inband-tags"))
+               if (!strcmp(cur_opt, "inband-tags")) {
                        options->inband_tags = 1;
-               else if (!strcmp(cur_opt, "tags-ecc-off")){
+               } else if (!strcmp(cur_opt, "tags-ecc-off")) {
                        options->tags_ecc_on = 0;
-                       options->tags_ecc_overridden=1;
-               } else if (!strcmp(cur_opt, "tags-ecc-on")){
+                       options->tags_ecc_overridden = 1;
+               } else if (!strcmp(cur_opt, "tags-ecc-on")) {
                        options->tags_ecc_on = 1;
                        options->tags_ecc_overridden = 1;
-               } else if (!strcmp(cur_opt, "lazy-loading-off")){
+               } else if (!strcmp(cur_opt, "lazy-loading-off")) {
                        options->lazy_loading_enabled = 0;
-                       options->lazy_loading_overridden=1;
-               } else if (!strcmp(cur_opt, "lazy-loading-on")){
+                       options->lazy_loading_overridden = 1;
+               } else if (!strcmp(cur_opt, "lazy-loading-on")) {
                        options->lazy_loading_enabled = 1;
                        options->lazy_loading_overridden = 1;
-               } else if (!strcmp(cur_opt, "empty-lost-and-found-off")){
+               } else if (!strcmp(cur_opt, "empty-lost-and-found-off")) {
                        options->empty_lost_and_found = 0;
-                       options->empty_lost_and_found_overridden=1;
-               } else if (!strcmp(cur_opt, "empty-lost-and-found-on")){
+                       options->empty_lost_and_found_overridden = 1;
+               } else if (!strcmp(cur_opt, "empty-lost-and-found-on")) {
                        options->empty_lost_and_found = 1;
-                       options->empty_lost_and_found_overridden=1;
-               } else if (!strcmp(cur_opt, "no-cache"))
+                       options->empty_lost_and_found_overridden = 1;
+               } else if (!strcmp(cur_opt, "no-cache")) {
                        options->no_cache = 1;
-               else if (!strcmp(cur_opt, "no-checkpoint-read"))
+               } else if (!strcmp(cur_opt, "no-checkpoint-read")) {
                        options->skip_checkpoint_read = 1;
-               else if (!strcmp(cur_opt, "no-checkpoint-write"))
+               } else if (!strcmp(cur_opt, "no-checkpoint-write")) {
                        options->skip_checkpoint_write = 1;
-               else if (!strcmp(cur_opt, "no-checkpoint")) {
+               else if (!strcmp(cur_opt, "no-checkpoint")) {
                        options->skip_checkpoint_read = 1;
                        options->skip_checkpoint_write = 1;
                } else {
                        printk(KERN_INFO "yaffs: Bad mount option \"%s\"\n",
-                                       cur_opt);
+                              cur_opt);
                        error = 1;
                }
        }
@@ -2678,51 +2601,52 @@ static int yaffs_parse_options(yaffs_options *options, const char *options_str)
 }
 
 static struct super_block *yaffs_internal_read_super(int yaffs_version,
-                                               struct super_block *sb,
-                                               void *data, int silent)
+                                                    struct super_block *sb,
+                                                    void *data, int silent)
 {
-       int nBlocks;
+       int n_blocks;
        struct inode *inode = NULL;
        struct dentry *root;
-       yaffs_dev_t *dev = 0;
+       struct yaffs_dev *dev = 0;
        char devname_buf[BDEVNAME_SIZE + 1];
        struct mtd_info *mtd;
        int err;
        char *data_str = (char *)data;
-       struct yaffs_LinuxContext *context = NULL;
-       yaffs_param_t *param;
+       struct yaffs_linux_context *context = NULL;
+       struct yaffs_param *param;
 
        int read_only = 0;
 
-       yaffs_options options;
+       struct yaffs_options options;
 
        unsigned mount_id;
        int found;
-       struct yaffs_LinuxContext *context_iterator;
-       struct ylist_head *l;
+       struct yaffs_linux_context *context_iterator;
+       struct list_head *l;
+
+       if (!sb) {
+               printk(KERN_INFO "yaffs: sb is NULL\n");
+               return NULL;
+        }
 
        sb->s_magic = YAFFS_MAGIC;
        sb->s_op = &yaffs_super_ops;
        sb->s_flags |= MS_NOATIME;
 
-       read_only =((sb->s_flags & MS_RDONLY) != 0);
-
+       read_only = ((sb->s_flags & MS_RDONLY) != 0);
 
 #ifdef YAFFS_COMPILE_EXPORTFS
        sb->s_export_op = &yaffs_export_ops;
 #endif
 
-       if (!sb)
-               printk(KERN_INFO "yaffs: sb is NULL\n");
-       else if (!sb->s_dev)
+       if (!sb->s_dev)
                printk(KERN_INFO "yaffs: sb->s_dev is NULL\n");
        else if (!yaffs_devname(sb, devname_buf))
                printk(KERN_INFO "yaffs: devname is NULL\n");
        else
                printk(KERN_INFO "yaffs: dev is %d name is \"%s\" %s\n",
                       sb->s_dev,
-                      yaffs_devname(sb, devname_buf),
-                      read_only ? "ro" : "rw");
+                      yaffs_devname(sb, devname_buf), read_only ? "ro" : "rw");
 
        if (!data_str)
                data_str = "";
@@ -2736,20 +2660,18 @@ static struct super_block *yaffs_internal_read_super(int yaffs_version,
                return NULL;
        }
 
-
        sb->s_blocksize = PAGE_CACHE_SIZE;
        sb->s_blocksize_bits = PAGE_CACHE_SHIFT;
 
-       T(YAFFS_TRACE_OS,
-               (TSTR("yaffs_read_super: Using yaffs%d\n"), yaffs_version));
-       T(YAFFS_TRACE_OS,
-               (TSTR("yaffs_read_super: block size %d\n"),
-               (int)(sb->s_blocksize)));
+       yaffs_trace(YAFFS_TRACE_OS,
+               "yaffs_read_super: Using yaffs%d", yaffs_version);
+       yaffs_trace(YAFFS_TRACE_OS,
+               "yaffs_read_super: block size %d", (int)(sb->s_blocksize));
 
-       T(YAFFS_TRACE_ALWAYS,
-               (TSTR("yaffs: Attempting MTD mount of %u.%u,\"%s\"\n"),
-              MAJOR(sb->s_dev), MINOR(sb->s_dev),
-              yaffs_devname(sb, devname_buf)));
+       yaffs_trace(YAFFS_TRACE_ALWAYS,
+               "yaffs: Attempting MTD mount of %u.%u,\"%s\"",
+               MAJOR(sb->s_dev), MINOR(sb->s_dev),
+               yaffs_devname(sb, devname_buf));
 
        /* Check it's an mtd device..... */
        if (MAJOR(sb->s_dev) != MTD_BLOCK_MAJOR)
@@ -2758,123 +2680,119 @@ static struct super_block *yaffs_internal_read_super(int yaffs_version,
        /* Get the device */
        mtd = get_mtd_device(NULL, MINOR(sb->s_dev));
        if (!mtd) {
-               T(YAFFS_TRACE_ALWAYS,
-                       (TSTR("yaffs: MTD device #%u doesn't appear to exist\n"),
-                       MINOR(sb->s_dev)));
+               yaffs_trace(YAFFS_TRACE_ALWAYS,
+                       "yaffs: MTD device #%u doesn't appear to exist",
+                       MINOR(sb->s_dev));
                return NULL;
        }
        /* Check it's NAND */
        if (mtd->type != MTD_NANDFLASH) {
-               T(YAFFS_TRACE_ALWAYS,
-                       (TSTR("yaffs: MTD device is not NAND it's type %d\n"),
-                       mtd->type));
+               yaffs_trace(YAFFS_TRACE_ALWAYS,
+                       "yaffs: MTD device is not NAND it's type %d",
+                       mtd->type);
                return NULL;
        }
 
-       T(YAFFS_TRACE_OS, (TSTR(" erase %p\n"), mtd->erase));
-       T(YAFFS_TRACE_OS, (TSTR(" read %p\n"), mtd->read));
-       T(YAFFS_TRACE_OS, (TSTR(" write %p\n"), mtd->write));
-       T(YAFFS_TRACE_OS, (TSTR(" readoob %p\n"), mtd->read_oob));
-       T(YAFFS_TRACE_OS, (TSTR(" writeoob %p\n"), mtd->write_oob));
-       T(YAFFS_TRACE_OS, (TSTR(" block_isbad %p\n"), mtd->block_isbad));
-       T(YAFFS_TRACE_OS, (TSTR(" block_markbad %p\n"), mtd->block_markbad));
-       T(YAFFS_TRACE_OS, (TSTR(" %s %d\n"), WRITE_SIZE_STR, WRITE_SIZE(mtd)));
-       T(YAFFS_TRACE_OS, (TSTR(" oobsize %d\n"), mtd->oobsize));
-       T(YAFFS_TRACE_OS, (TSTR(" erasesize %d\n"), mtd->erasesize));
+       yaffs_trace(YAFFS_TRACE_OS, " erase %p", mtd->erase);
+       yaffs_trace(YAFFS_TRACE_OS, " read %p", mtd->read);
+       yaffs_trace(YAFFS_TRACE_OS, " write %p", mtd->write);
+       yaffs_trace(YAFFS_TRACE_OS, " readoob %p", mtd->read_oob);
+       yaffs_trace(YAFFS_TRACE_OS, " writeoob %p", mtd->write_oob);
+       yaffs_trace(YAFFS_TRACE_OS, " block_isbad %p", mtd->block_isbad);
+       yaffs_trace(YAFFS_TRACE_OS, " block_markbad %p", mtd->block_markbad);
+       yaffs_trace(YAFFS_TRACE_OS, " %s %d", WRITE_SIZE_STR, WRITE_SIZE(mtd));
+       yaffs_trace(YAFFS_TRACE_OS, " oobsize %d", mtd->oobsize);
+       yaffs_trace(YAFFS_TRACE_OS, " erasesize %d", mtd->erasesize);
 #if LINUX_VERSION_CODE < KERNEL_VERSION(2, 6, 29)
-       T(YAFFS_TRACE_OS, (TSTR(" size %u\n"), mtd->size));
+       yaffs_trace(YAFFS_TRACE_OS, " size %u", mtd->size);
 #else
-       T(YAFFS_TRACE_OS, (TSTR(" size %lld\n"), mtd->size));
+       yaffs_trace(YAFFS_TRACE_OS, " size %lld", mtd->size);
 #endif
 
 #ifdef CONFIG_YAFFS_AUTO_YAFFS2
 
        if (yaffs_version == 1 && WRITE_SIZE(mtd) >= 2048) {
-               T(YAFFS_TRACE_ALWAYS,
-                       (TSTR("yaffs: auto selecting yaffs2\n")));
+               yaffs_trace(YAFFS_TRACE_ALWAYS, "auto selecting yaffs2");
                yaffs_version = 2;
        }
 
        /* Added NCB 26/5/2006 for completeness */
-       if (yaffs_version == 2 && !options.inband_tags && WRITE_SIZE(mtd) == 512) {
-               T(YAFFS_TRACE_ALWAYS,
-                       (TSTR("yaffs: auto selecting yaffs1\n")));
+       if (yaffs_version == 2 && !options.inband_tags
+           && WRITE_SIZE(mtd) == 512) {
+               yaffs_trace(YAFFS_TRACE_ALWAYS, "auto selecting yaffs1");
                yaffs_version = 1;
        }
-
 #endif
 
        if (yaffs_version == 2) {
                /* Check for version 2 style functions */
                if (!mtd->erase ||
                    !mtd->block_isbad ||
-                   !mtd->block_markbad ||
-                   !mtd->read ||
-                   !mtd->write ||
+                   !mtd->block_markbad || !mtd->read || !mtd->write ||
 #if (LINUX_VERSION_CODE > KERNEL_VERSION(2, 6, 17))
                    !mtd->read_oob || !mtd->write_oob) {
 #else
                    !mtd->write_ecc ||
                    !mtd->read_ecc || !mtd->read_oob || !mtd->write_oob) {
 #endif
-                       T(YAFFS_TRACE_ALWAYS,
-                         (TSTR("yaffs: MTD device does not support required "
-                          "functions\n")));
+                       yaffs_trace(YAFFS_TRACE_ALWAYS,
+                               "MTD device does not support required functions"
+                       );
                        return NULL;
                }
 
                if ((WRITE_SIZE(mtd) < YAFFS_MIN_YAFFS2_CHUNK_SIZE ||
-                   mtd->oobsize < YAFFS_MIN_YAFFS2_SPARE_SIZE) &&
+                    mtd->oobsize < YAFFS_MIN_YAFFS2_SPARE_SIZE) &&
                    !options.inband_tags) {
-                       T(YAFFS_TRACE_ALWAYS,
-                         (TSTR("yaffs: MTD device does not have the "
-                          "right page sizes\n")));
+                       yaffs_trace(YAFFS_TRACE_ALWAYS,
+                               "MTD device does not have the right page sizes"
+                       );
                        return NULL;
                }
        } else {
                /* Check for V1 style functions */
-               if (!mtd->erase ||
-                   !mtd->read ||
-                   !mtd->write ||
+               if (!mtd->erase || !mtd->read || !mtd->write ||
 #if (LINUX_VERSION_CODE > KERNEL_VERSION(2, 6, 17))
                    !mtd->read_oob || !mtd->write_oob) {
 #else
                    !mtd->write_ecc ||
                    !mtd->read_ecc || !mtd->read_oob || !mtd->write_oob) {
 #endif
-                       T(YAFFS_TRACE_ALWAYS,
-                         (TSTR("yaffs: MTD device does not support required "
-                          "functions\n")));
+                       yaffs_trace(YAFFS_TRACE_ALWAYS,
+                               "MTD device does not support required functions"
+                       );
                        return NULL;
                }
 
                if (WRITE_SIZE(mtd) < YAFFS_BYTES_PER_CHUNK ||
                    mtd->oobsize != YAFFS_BYTES_PER_SPARE) {
-                       T(YAFFS_TRACE_ALWAYS,
-                         (TSTR("yaffs: MTD device does not support have the "
-                          "right page sizes\n")));
+                       yaffs_trace(YAFFS_TRACE_ALWAYS,
+                               "MTD device does not support have the right page sizes"
+                       );
                        return NULL;
                }
        }
 
        /* OK, so if we got here, we have an MTD that's NAND and looks
         * like it has the right capabilities
-        * Set the yaffs_dev_t up for mtd
+        * Set the struct yaffs_dev up for mtd
         */
 
-       if (!read_only && !(mtd->flags & MTD_WRITEABLE)){
+       if (!read_only && !(mtd->flags & MTD_WRITEABLE)) {
                read_only = 1;
-               printk(KERN_INFO "yaffs: mtd is read only, setting superblock read only");
+               printk(KERN_INFO
+                      "yaffs: mtd is read only, setting superblock read only\n"
+               );
                sb->s_flags |= MS_RDONLY;
        }
 
-       dev = kmalloc(sizeof(yaffs_dev_t), GFP_KERNEL);
-       context = kmalloc(sizeof(struct yaffs_LinuxContext),GFP_KERNEL);
-       
-       if(!dev || !context ){
-               if(dev)
+       dev = kmalloc(sizeof(struct yaffs_dev), GFP_KERNEL);
+       context = kmalloc(sizeof(struct yaffs_linux_context), GFP_KERNEL);
+
+       if (!dev || !context) {
+               if (dev)
                        kfree(dev);
-               if(context)
+               if (context)
                        kfree(context);
                dev = NULL;
                context = NULL;
@@ -2882,19 +2800,19 @@ static struct super_block *yaffs_internal_read_super(int yaffs_version,
 
        if (!dev) {
                /* Deep shit could not allocate device structure */
-               T(YAFFS_TRACE_ALWAYS,
-                 (TSTR("yaffs_read_super: Failed trying to allocate "
-                  "yaffs_dev_t. \n")));
+               yaffs_trace(YAFFS_TRACE_ALWAYS,
+                       "yaffs_read_super: Failed trying to allocate struct yaffs_dev."
+               );
                return NULL;
        }
-       memset(dev, 0, sizeof(yaffs_dev_t));
+       memset(dev, 0, sizeof(struct yaffs_dev));
        param = &(dev->param);
 
-       memset(context,0,sizeof(struct yaffs_LinuxContext));
+       memset(context, 0, sizeof(struct yaffs_linux_context));
        dev->os_context = context;
-       YINIT_LIST_HEAD(&(context->contextList));
+       INIT_LIST_HEAD(&(context->context_list));
        context->dev = dev;
-       context->superBlock = sb;
+       context->super = sb;
 
        dev->read_only = read_only;
 
@@ -2903,16 +2821,18 @@ static struct super_block *yaffs_internal_read_super(int yaffs_version,
 #else
        sb->u.generic_sbp = dev;
 #endif
-       
+
        dev->driver_context = mtd;
        param->name = mtd->name;
 
        /* Set up the memory size parameters.... */
 
-       nBlocks = YCALCBLOCKS(mtd->size, (YAFFS_CHUNKS_PER_BLOCK * YAFFS_BYTES_PER_CHUNK));
+       n_blocks =
+           YCALCBLOCKS(mtd->size,
+                       (YAFFS_CHUNKS_PER_BLOCK * YAFFS_BYTES_PER_CHUNK));
 
        param->start_block = 0;
-       param->end_block = nBlocks - 1;
+       param->end_block = n_blocks - 1;
        param->chunks_per_block = YAFFS_CHUNKS_PER_BLOCK;
        param->total_bytes_per_chunk = YAFFS_BYTES_PER_CHUNK;
        param->n_reserved_blocks = 5;
@@ -2925,7 +2845,7 @@ static struct super_block *yaffs_internal_read_super(int yaffs_version,
 #ifdef CONFIG_YAFFS_XATTR
        param->enable_xattr = 1;
 #endif
-       if(options.lazy_loading_overridden)
+       if (options.lazy_loading_overridden)
                param->disable_lazy_load = !options.lazy_loading_enabled;
 
 #ifdef CONFIG_YAFFS_DISABLE_TAGS_ECC
@@ -2937,7 +2857,7 @@ static struct super_block *yaffs_internal_read_super(int yaffs_version,
        param->defered_dir_update = 1;
 #endif
 
-       if(options.tags_ecc_overridden)
+       if (options.tags_ecc_overridden)
                param->no_tags_ecc = !options.tags_ecc_on;
 
 #ifdef CONFIG_YAFFS_EMPTY_LOST_AND_FOUND
@@ -2954,18 +2874,17 @@ static struct super_block *yaffs_internal_read_super(int yaffs_version,
        param->always_check_erased = 1;
 #endif
 
-       if(options.empty_lost_and_found_overridden)
+       if (options.empty_lost_and_found_overridden)
                param->empty_lost_n_found = options.empty_lost_and_found;
 
        /* ... and the functions. */
        if (yaffs_version == 2) {
-               param->write_chunk_tags_fn =
-                   nandmtd2_WriteChunkWithTagsToNAND;
-               param->read_chunk_tags_fn =
-                   nandmtd2_ReadChunkWithTagsFromNAND;
-               param->bad_block_fn = nandmtd2_MarkNANDBlockBad;
-               param->query_block_fn = nandmtd2_QueryNANDBlock;
-               yaffs_dev_to_lc(dev)->spareBuffer = YMALLOC(mtd->oobsize);
+               param->write_chunk_tags_fn = nandmtd2_write_chunk_tags;
+               param->read_chunk_tags_fn = nandmtd2_read_chunk_tags;
+               param->bad_block_fn = nandmtd2_mark_block_bad;
+               param->query_block_fn = nandmtd2_query_block;
+               yaffs_dev_to_lc(dev)->spare_buffer =
+                               kmalloc(mtd->oobsize, GFP_NOFS);
                param->is_yaffs2 = 1;
 #if (LINUX_VERSION_CODE > KERNEL_VERSION(2, 6, 17))
                param->total_bytes_per_chunk = mtd->writesize;
@@ -2974,36 +2893,33 @@ static struct super_block *yaffs_internal_read_super(int yaffs_version,
                param->total_bytes_per_chunk = mtd->oobblock;
                param->chunks_per_block = mtd->erasesize / mtd->oobblock;
 #endif
-               nBlocks = YCALCBLOCKS(mtd->size, mtd->erasesize);
+               n_blocks = YCALCBLOCKS(mtd->size, mtd->erasesize);
 
                param->start_block = 0;
-               param->end_block = nBlocks - 1;
+               param->end_block = n_blocks - 1;
        } else {
 #if (LINUX_VERSION_CODE > KERNEL_VERSION(2, 6, 17))
                /* use the MTD interface in yaffs_mtdif1.c */
-               param->write_chunk_tags_fn =
-                       nandmtd1_WriteChunkWithTagsToNAND;
-               param->read_chunk_tags_fn =
-                       nandmtd1_ReadChunkWithTagsFromNAND;
-               param->bad_block_fn = nandmtd1_MarkNANDBlockBad;
-               param->query_block_fn = nandmtd1_QueryNANDBlock;
+               param->write_chunk_tags_fn = nandmtd1_write_chunk_tags;
+               param->read_chunk_tags_fn = nandmtd1_read_chunk_tags;
+               param->bad_block_fn = nandmtd1_mark_block_bad;
+               param->query_block_fn = nandmtd1_query_block;
 #else
-               param->write_chunk_fn = nandmtd_WriteChunkToNAND;
-               param->read_chunk_fn = nandmtd_ReadChunkFromNAND;
+               param->write_chunk_fn = nandmtd_write_chunk;
+               param->read_chunk_fn = nandmtd_read_chunk;
 #endif
                param->is_yaffs2 = 0;
        }
        /* ... and common functions */
-       param->erase_fn = nandmtd_EraseBlockInNAND;
-       param->initialise_flash_fn = nandmtd_InitialiseNAND;
+       param->erase_fn = nandmtd_erase_block;
+       param->initialise_flash_fn = nandmtd_initialise;
 
-       yaffs_dev_to_lc(dev)->putSuperFunc = yaffs_MTDPutSuper;
+       yaffs_dev_to_lc(dev)->put_super_fn = yaffs_mtd_put_super;
 
        param->sb_dirty_fn = yaffs_touch_super;
        param->gc_control = yaffs_gc_control_callback;
 
-       yaffs_dev_to_lc(dev)->superBlock= sb;
-       
+       yaffs_dev_to_lc(dev)->super = sb;
 
 #ifndef CONFIG_YAFFS_DOES_ECC
        param->use_nand_ecc = 1;
@@ -3016,50 +2932,51 @@ static struct super_block *yaffs_internal_read_super(int yaffs_version,
        param->skip_checkpt_rd = options.skip_checkpoint_read;
        param->skip_checkpt_wr = options.skip_checkpoint_write;
 
-       down(&yaffs_context_lock);
+       mutex_lock(&yaffs_context_lock);
        /* Get a mount id */
        found = 0;
-       for(mount_id=0; ! found; mount_id++){
+       for (mount_id = 0; !found; mount_id++) {
                found = 1;
-               ylist_for_each(l,&yaffs_context_list){
-                       context_iterator = ylist_entry(l,struct yaffs_LinuxContext,contextList);
-                       if(context_iterator->mount_id == mount_id)
+               list_for_each(l, &yaffs_context_list) {
+                       context_iterator =
+                           list_entry(l, struct yaffs_linux_context,
+                                      context_list);
+                       if (context_iterator->mount_id == mount_id)
                                found = 0;
                }
        }
        context->mount_id = mount_id;
 
-       ylist_add_tail(&(yaffs_dev_to_lc(dev)->contextList), &yaffs_context_list);
-       up(&yaffs_context_lock);
+       list_add_tail(&(yaffs_dev_to_lc(dev)->context_list),
+                     &yaffs_context_list);
+       mutex_unlock(&yaffs_context_lock);
 
-        /* Directory search handling...*/
-        YINIT_LIST_HEAD(&(yaffs_dev_to_lc(dev)->searchContexts));
-        param->remove_obj_fn = yaffs_remove_obj_callback;
+       /* Directory search handling... */
+       INIT_LIST_HEAD(&(yaffs_dev_to_lc(dev)->search_contexts));
+       param->remove_obj_fn = yaffs_remove_obj_callback;
 
-       init_MUTEX(&(yaffs_dev_to_lc(dev)->grossLock));
+       mutex_init(&(yaffs_dev_to_lc(dev)->gross_lock));
 
        yaffs_gross_lock(dev);
 
        err = yaffs_guts_initialise(dev);
 
-       T(YAFFS_TRACE_OS,
-         (TSTR("yaffs_read_super: guts initialised %s\n"),
-          (err == YAFFS_OK) ? "OK" : "FAILED"));
-          
-       if(err == YAFFS_OK)
+       yaffs_trace(YAFFS_TRACE_OS,
+               "yaffs_read_super: guts initialised %s",
+               (err == YAFFS_OK) ? "OK" : "FAILED");
+
+       if (err == YAFFS_OK)
                yaffs_bg_start(dev);
-               
-       if(!context->bgThread)
-               param->defered_dir_update = 0;
 
+       if (!context->bg_thread)
+               param->defered_dir_update = 0;
 
        /* Release lock before yaffs_get_inode() */
        yaffs_gross_unlock(dev);
 
        /* Create root inode */
        if (err == YAFFS_OK)
-               inode = yaffs_get_inode(sb, S_IFDIR | 0755, 0,
-                                       yaffs_root(dev));
+               inode = yaffs_get_inode(sb, S_IFDIR | 0755, 0, yaffs_root(dev));
 
        if (!inode)
                return NULL;
@@ -3067,11 +2984,11 @@ static struct super_block *yaffs_internal_read_super(int yaffs_version,
        inode->i_op = &yaffs_dir_inode_operations;
        inode->i_fop = &yaffs_dir_operations;
 
-       T(YAFFS_TRACE_OS, (TSTR("yaffs_read_super: got root inode\n")));
+       yaffs_trace(YAFFS_TRACE_OS, "yaffs_read_super: got root inode");
 
        root = d_alloc_root(inode);
 
-       T(YAFFS_TRACE_OS, (TSTR("yaffs_read_super: d_alloc_root done\n")));
+       yaffs_trace(YAFFS_TRACE_OS, "yaffs_read_super: d_alloc_root done");
 
        if (!root) {
                iput(inode);
@@ -3079,15 +2996,14 @@ static struct super_block *yaffs_internal_read_super(int yaffs_version,
        }
        sb->s_root = root;
        sb->s_dirt = !dev->is_checkpointed;
-       T(YAFFS_TRACE_ALWAYS,
-               (TSTR("yaffs_read_super: is_checkpointed %d\n"),
-               dev->is_checkpointed));
+       yaffs_trace(YAFFS_TRACE_ALWAYS,
+               "yaffs_read_super: is_checkpointed %d",
+               dev->is_checkpointed);
 
-       T(YAFFS_TRACE_OS, (TSTR("yaffs_read_super: done\n")));
+       yaffs_trace(YAFFS_TRACE_OS, "yaffs_read_super: done");
        return sb;
 }
 
-
 #if (LINUX_VERSION_CODE > KERNEL_VERSION(2, 5, 0))
 static int yaffs_internal_read_super_mtd(struct super_block *sb, void *data,
                                         int silent)
@@ -3133,7 +3049,6 @@ static DECLARE_FSTYPE(yaffs_fs_type, "yaffs", yaffs_read_super,
                      FS_REQUIRES_DEV);
 #endif
 
-
 #ifdef CONFIG_YAFFS_YAFFS2
 
 #if (LINUX_VERSION_CODE > KERNEL_VERSION(2, 5, 0))
@@ -3145,11 +3060,11 @@ static int yaffs2_internal_read_super_mtd(struct super_block *sb, void *data,
 
 #if (LINUX_VERSION_CODE > KERNEL_VERSION(2, 6, 17))
 static int yaffs2_read_super(struct file_system_type *fs,
-                       int flags, const char *dev_name, void *data,
-                       struct vfsmount *mnt)
+                            int flags, const char *dev_name, void *data,
+                            struct vfsmount *mnt)
 {
        return get_sb_bdev(fs, flags, dev_name, data,
-                       yaffs2_internal_read_super_mtd, mnt);
+                          yaffs2_internal_read_super_mtd, mnt);
 }
 #else
 static struct super_block *yaffs2_read_super(struct file_system_type *fs,
@@ -3180,66 +3095,79 @@ static DECLARE_FSTYPE(yaffs2_fs_type, "yaffs2", yaffs2_read_super,
                      FS_REQUIRES_DEV);
 #endif
 
-#endif                         /* CONFIG_YAFFS_YAFFS2 */
+#endif /* CONFIG_YAFFS_YAFFS2 */
 
 static struct proc_dir_entry *my_proc_entry;
-static struct proc_dir_entry *debug_proc_entry;
-
-static char *yaffs_dump_dev_part0(char *buf, yaffs_dev_t * dev)
-{
-       buf += sprintf(buf, "start_block......... %d\n", dev->param.start_block);
-       buf += sprintf(buf, "end_block........... %d\n", dev->param.end_block);
-       buf += sprintf(buf, "total_bytes_per_chunk. %d\n", dev->param.total_bytes_per_chunk);
-       buf += sprintf(buf, "use_nand_ecc......... %d\n", dev->param.use_nand_ecc);
-       buf += sprintf(buf, "no_tags_ecc.......... %d\n", dev->param.no_tags_ecc);
-       buf += sprintf(buf, "is_yaffs2........... %d\n", dev->param.is_yaffs2);
-       buf += sprintf(buf, "inband_tags......... %d\n", dev->param.inband_tags);
-       buf += sprintf(buf, "empty_lost_n_found.. %d\n", dev->param.empty_lost_n_found);
-       buf += sprintf(buf, "disable_lazy_load.... %d\n", dev->param.disable_lazy_load);
-       buf += sprintf(buf, "refresh_period...... %d\n", dev->param.refresh_period);
-       buf += sprintf(buf, "n_caches..... %d\n", dev->param.n_caches);
-       buf += sprintf(buf, "n_reserved_blocks.... %d\n", dev->param.n_reserved_blocks);
-       buf += sprintf(buf, "always_check_erased.. %d\n", dev->param.always_check_erased);
 
+static char *yaffs_dump_dev_part0(char *buf, struct yaffs_dev *dev)
+{
+       struct yaffs_param *param = &dev->param;
+
+       buf += sprintf(buf, "start_block.......... %d\n", param->start_block);
+       buf += sprintf(buf, "end_block............ %d\n", param->end_block);
+       buf += sprintf(buf, "total_bytes_per_chunk %d\n",
+                               param->total_bytes_per_chunk);
+       buf += sprintf(buf, "use_nand_ecc......... %d\n", param->use_nand_ecc);
+       buf += sprintf(buf, "no_tags_ecc.......... %d\n", param->no_tags_ecc);
+       buf += sprintf(buf, "is_yaffs2............ %d\n", param->is_yaffs2);
+       buf += sprintf(buf, "inband_tags.......... %d\n", param->inband_tags);
+       buf += sprintf(buf, "empty_lost_n_found... %d\n",
+                               param->empty_lost_n_found);
+       buf += sprintf(buf, "disable_lazy_load.... %d\n",
+                               param->disable_lazy_load);
+       buf += sprintf(buf, "refresh_period....... %d\n",
+                               param->refresh_period);
+       buf += sprintf(buf, "n_caches............. %d\n", param->n_caches);
+       buf += sprintf(buf, "n_reserved_blocks.... %d\n",
+                               param->n_reserved_blocks);
+       buf += sprintf(buf, "always_check_erased.. %d\n",
+                               param->always_check_erased);
        buf += sprintf(buf, "\n");
 
        return buf;
 }
 
-
-static char *yaffs_dump_dev_part1(char *buf, yaffs_dev_t * dev)
+static char *yaffs_dump_dev_part1(char *buf, struct yaffs_dev *dev)
 {
-       buf += sprintf(buf, "data_bytes_per_chunk. %d\n", dev->data_bytes_per_chunk);
-       buf += sprintf(buf, "chunk_grp_bits..... %d\n", dev->chunk_grp_bits);
-       buf += sprintf(buf, "chunk_grp_size..... %d\n", dev->chunk_grp_size);
+       buf += sprintf(buf, "data_bytes_per_chunk. %d\n",
+                               dev->data_bytes_per_chunk);
+       buf += sprintf(buf, "chunk_grp_bits....... %d\n", dev->chunk_grp_bits);
+       buf += sprintf(buf, "chunk_grp_size....... %d\n", dev->chunk_grp_size);
        buf += sprintf(buf, "n_erased_blocks...... %d\n", dev->n_erased_blocks);
-       buf += sprintf(buf, "blocks_in_checkpt. %d\n", dev->blocks_in_checkpt);
+       buf += sprintf(buf, "blocks_in_checkpt.... %d\n",
+                               dev->blocks_in_checkpt);
        buf += sprintf(buf, "\n");
-       buf += sprintf(buf, "n_tnodes............ %d\n", dev->n_tnodes);
-       buf += sprintf(buf, "n_obj........... %d\n", dev->n_obj);
+       buf += sprintf(buf, "n_tnodes............. %d\n", dev->n_tnodes);
+       buf += sprintf(buf, "n_obj................ %d\n", dev->n_obj);
        buf += sprintf(buf, "n_free_chunks........ %d\n", dev->n_free_chunks);
        buf += sprintf(buf, "\n");
        buf += sprintf(buf, "n_page_writes........ %u\n", dev->n_page_writes);
        buf += sprintf(buf, "n_page_reads......... %u\n", dev->n_page_reads);
-       buf += sprintf(buf, "n_erasures..... %u\n", dev->n_erasures);
+       buf += sprintf(buf, "n_erasures........... %u\n", dev->n_erasures);
        buf += sprintf(buf, "n_gc_copies.......... %u\n", dev->n_gc_copies);
-       buf += sprintf(buf, "all_gcs............. %u\n", dev->all_gcs);
-       buf += sprintf(buf, "passive_gc_count......... %u\n", dev->passive_gc_count);
-       buf += sprintf(buf, "oldest_dirty_gc_count..... %u\n", dev->oldest_dirty_gc_count);
+       buf += sprintf(buf, "all_gcs.............. %u\n", dev->all_gcs);
+       buf += sprintf(buf, "passive_gc_count..... %u\n",
+                               dev->passive_gc_count);
+       buf += sprintf(buf, "oldest_dirty_gc_count %u\n",
+                               dev->oldest_dirty_gc_count);
        buf += sprintf(buf, "n_gc_blocks.......... %u\n", dev->n_gc_blocks);
-       buf += sprintf(buf, "bg_gcs...... %u\n", dev->bg_gcs);
-       buf += sprintf(buf, "n_retired_writes..... %u\n", dev->n_retired_writes);
-       buf += sprintf(buf, "nRetireBlocks...... %u\n", dev->n_retired_blocks);
-       buf += sprintf(buf, "n_ecc_fixed........... %u\n", dev->n_ecc_fixed);
-       buf += sprintf(buf, "n_ecc_unfixed......... %u\n", dev->n_ecc_unfixed);
-       buf += sprintf(buf, "n_tags_ecc_fixed....... %u\n", dev->n_tags_ecc_fixed);
-       buf += sprintf(buf, "n_tags_ecc_unfixed..... %u\n", dev->n_tags_ecc_unfixed);
-       buf += sprintf(buf, "cache_hits.......... %u\n", dev->cache_hits);
+       buf += sprintf(buf, "bg_gcs............... %u\n", dev->bg_gcs);
+       buf += sprintf(buf, "n_retired_writes..... %u\n",
+                               dev->n_retired_writes);
+       buf += sprintf(buf, "n_retired_blocks..... %u\n",
+                               dev->n_retired_blocks);
+       buf += sprintf(buf, "n_ecc_fixed.......... %u\n", dev->n_ecc_fixed);
+       buf += sprintf(buf, "n_ecc_unfixed........ %u\n", dev->n_ecc_unfixed);
+       buf += sprintf(buf, "n_tags_ecc_fixed..... %u\n",
+                               dev->n_tags_ecc_fixed);
+       buf += sprintf(buf, "n_tags_ecc_unfixed... %u\n",
+                               dev->n_tags_ecc_unfixed);
+       buf += sprintf(buf, "cache_hits........... %u\n", dev->cache_hits);
        buf += sprintf(buf, "n_deleted_files...... %u\n", dev->n_deleted_files);
-       buf += sprintf(buf, "n_unlinked_files..... %u\n", dev->n_unlinked_files);
-       buf += sprintf(buf, "refresh_count....... %u\n", dev->refresh_count);
-       buf +=
-           sprintf(buf, "nBackgroudDeletions %u\n", dev->n_bg_deletions);
+       buf += sprintf(buf, "n_unlinked_files..... %u\n",
+                               dev->n_unlinked_files);
+       buf += sprintf(buf, "refresh_count........ %u\n", dev->refresh_count);
+       buf += sprintf(buf, "n_bg_deletions....... %u\n", dev->n_bg_deletions);
 
        return buf;
 }
@@ -3248,7 +3176,7 @@ static int yaffs_proc_read(char *page,
                           char **start,
                           off_t offset, int count, int *eof, void *data)
 {
-       struct ylist_head *item;
+       struct list_head *item;
        char *buf = page;
        int step = offset;
        int n = 0;
@@ -3263,63 +3191,41 @@ static int yaffs_proc_read(char *page,
 
        /* Print header first */
        if (step == 0)
-               buf += sprintf(buf, "Multi-version YAFFS built:" __DATE__ " " __TIME__"\n");
+               buf +=
+                   sprintf(buf,
+                           "Multi-version YAFFS built:" __DATE__ " " __TIME__
+                           "\n");
        else if (step == 1)
-               buf += sprintf(buf,"\n");
+               buf += sprintf(buf, "\n");
        else {
-               step-=2;
-               
-               down(&yaffs_context_lock);
+               step -= 2;
+
+               mutex_lock(&yaffs_context_lock);
 
                /* Locate and print the Nth entry.  Order N-squared but N is small. */
-               ylist_for_each(item, &yaffs_context_list) {
-                       struct yaffs_LinuxContext *dc = ylist_entry(item, struct yaffs_LinuxContext, contextList);
-                       yaffs_dev_t *dev = dc->dev;
+               list_for_each(item, &yaffs_context_list) {
+                       struct yaffs_linux_context *dc =
+                           list_entry(item, struct yaffs_linux_context,
+                                      context_list);
+                       struct yaffs_dev *dev = dc->dev;
 
                        if (n < (step & ~1)) {
-                               n+=2;
+                               n += 2;
                                continue;
                        }
-                       if((step & 1)==0){
-                               buf += sprintf(buf, "\nDevice %d \"%s\"\n", n, dev->param.name);
+                       if ((step & 1) == 0) {
+                               buf +=
+                                   sprintf(buf, "\nDevice %d \"%s\"\n", n,
+                                           dev->param.name);
                                buf = yaffs_dump_dev_part0(buf, dev);
-                       } else
+                       } else {
                                buf = yaffs_dump_dev_part1(buf, dev);
-                       
+                        }
+
                        break;
                }
-               up(&yaffs_context_lock);
-       }
-
-       return buf - page < count ? buf - page : count;
-}
-
-static int yaffs_stats_proc_read(char *page,
-                               char **start,
-                               off_t offset, int count, int *eof, void *data)
-{
-       struct ylist_head *item;
-       char *buf = page;
-       int n = 0;
-
-       down(&yaffs_context_lock);
-
-       /* Locate and print the Nth entry.  Order N-squared but N is small. */
-       ylist_for_each(item, &yaffs_context_list) {
-               struct yaffs_LinuxContext *dc = ylist_entry(item, struct yaffs_LinuxContext, contextList);
-               yaffs_dev_t *dev = dc->dev;
-
-               int erasedChunks;
-
-               erasedChunks = dev->n_erased_blocks * dev->param.chunks_per_block;
-               
-               buf += sprintf(buf,"%d, %d, %d, %u, %u, %u, %u\n",
-                               n, dev->n_free_chunks, erasedChunks,
-                               dev->bg_gcs, dev->oldest_dirty_gc_count,
-                               dev->n_obj, dev->n_tnodes);
+               mutex_unlock(&yaffs_context_lock);
        }
-       up(&yaffs_context_lock);
-
 
        return buf - page < count ? buf - page : count;
 }
@@ -3352,15 +3258,14 @@ static struct {
        {"os", YAFFS_TRACE_OS},
        {"scan_debug", YAFFS_TRACE_SCAN_DEBUG},
        {"scan", YAFFS_TRACE_SCAN},
+       {"mount", YAFFS_TRACE_MOUNT},
        {"tracing", YAFFS_TRACE_TRACING},
        {"sync", YAFFS_TRACE_SYNC},
        {"write", YAFFS_TRACE_WRITE},
-
        {"verify", YAFFS_TRACE_VERIFY},
        {"verify_nand", YAFFS_TRACE_VERIFY_NAND},
        {"verify_full", YAFFS_TRACE_VERIFY_FULL},
        {"verify_all", YAFFS_TRACE_VERIFY_ALL},
-
        {"all", 0xffffffff},
        {"none", 0},
        {NULL, 0},
@@ -3368,7 +3273,7 @@ static struct {
 
 #define MAX_MASK_NAME_LENGTH 40
 static int yaffs_proc_write_trace_options(struct file *file, const char *buf,
-                                        unsigned long count, void *data)
+                                         unsigned long count, void *data)
 {
        unsigned rg = 0, mask_bitfield;
        char *end;
@@ -3410,15 +3315,17 @@ static int yaffs_proc_write_trace_options(struct file *file, const char *buf,
                        done = 0;
                } else {
                        for (x = buf + pos, i = 0;
-                           (*x == '_' || (*x >= 'a' && *x <= 'z')) &&
-                           i < MAX_MASK_NAME_LENGTH; x++, i++, pos++)
+                            (*x == '_' || (*x >= 'a' && *x <= 'z')) &&
+                            i < MAX_MASK_NAME_LENGTH; x++, i++, pos++)
                                substring[i] = *x;
                        substring[i] = '\0';
 
                        for (i = 0; mask_flags[i].mask_name != NULL; i++) {
-                               if (strcmp(substring, mask_flags[i].mask_name) == 0) {
+                               if (strcmp(substring, mask_flags[i].mask_name)
+                                   == 0) {
                                        mask_name = mask_flags[i].mask_name;
-                                       mask_bitfield = mask_flags[i].mask_bitfield;
+                                       mask_bitfield =
+                                           mask_flags[i].mask_bitfield;
                                        done = 0;
                                        break;
                                }
@@ -3453,18 +3360,18 @@ static int yaffs_proc_write_trace_options(struct file *file, const char *buf,
                        char flag;
                        flag = ((rg & mask_flags[i].mask_bitfield) ==
                                mask_flags[i].mask_bitfield) ? '+' : '-';
-                       printk(KERN_DEBUG "%c%s\n", flag, mask_flags[i].mask_name);
+                       printk(KERN_DEBUG "%c%s\n", flag,
+                              mask_flags[i].mask_name);
                }
        }
 
        return count;
 }
 
-
 static int yaffs_proc_write(struct file *file, const char *buf,
-                                        unsigned long count, void *data)
+                           unsigned long count, void *data)
 {
-        return yaffs_proc_write_trace_options(file, buf, count, data);
+       return yaffs_proc_write_trace_options(file, buf, count, data);
 }
 
 /* Stuff to handle installation of file systems */
@@ -3484,41 +3391,28 @@ static int __init init_yaffs_fs(void)
        int error = 0;
        struct file_system_to_install *fsinst;
 
-       T(YAFFS_TRACE_ALWAYS,
-         (TSTR("yaffs built " __DATE__ " " __TIME__ " Installing. \n")));
+       yaffs_trace(YAFFS_TRACE_ALWAYS,
+               "yaffs built " __DATE__ " " __TIME__ " Installing.");
 
 #ifdef CONFIG_YAFFS_ALWAYS_CHECK_CHUNK_ERASED
-       T(YAFFS_TRACE_ALWAYS,
-         (TSTR(" \n\n\n\nYAFFS-WARNING CONFIG_YAFFS_ALWAYS_CHECK_CHUNK_ERASED selected.\n\n\n\n")));
+       yaffs_trace(YAFFS_TRACE_ALWAYS,
+               " \n\n\n\nYAFFS-WARNING CONFIG_YAFFS_ALWAYS_CHECK_CHUNK_ERASED selected.\n\n\n\n"
+       );
 #endif
 
-
-
-
-       init_MUTEX(&yaffs_context_lock);
+       mutex_init(&yaffs_context_lock);
 
        /* Install the proc_fs entries */
        my_proc_entry = create_proc_entry("yaffs",
-                                              S_IRUGO | S_IFREG,
-                                              YPROC_ROOT);
+                                         S_IRUGO | S_IFREG, YPROC_ROOT);
 
        if (my_proc_entry) {
                my_proc_entry->write_proc = yaffs_proc_write;
                my_proc_entry->read_proc = yaffs_proc_read;
                my_proc_entry->data = NULL;
-       } else
-               return -ENOMEM;
-
-       debug_proc_entry = create_proc_entry("yaffs_stats",
-                                              S_IRUGO | S_IFREG,
-                                              YPROC_ROOT);
-
-       if (debug_proc_entry) {
-               debug_proc_entry->write_proc = NULL;
-               debug_proc_entry->read_proc = yaffs_stats_proc_read;
-               debug_proc_entry->data = NULL;
-       } else
+       } else {
                return -ENOMEM;
+        }
 
        /* Now add the file system entries */
 
@@ -3552,11 +3446,10 @@ static void __exit exit_yaffs_fs(void)
 
        struct file_system_to_install *fsinst;
 
-       T(YAFFS_TRACE_ALWAYS,
-               (TSTR("yaffs built " __DATE__ " " __TIME__ " removing. \n")));
+       yaffs_trace(YAFFS_TRACE_ALWAYS,
+               "yaffs built " __DATE__ " " __TIME__ " removing.");
 
        remove_proc_entry("yaffs", YPROC_ROOT);
-       remove_proc_entry("yaffs_stats", YPROC_ROOT);
 
        fsinst = fs_to_install;
 
@@ -3570,8 +3463,8 @@ static void __exit exit_yaffs_fs(void)
 }
 
 module_init(init_yaffs_fs)
-module_exit(exit_yaffs_fs)
+    module_exit(exit_yaffs_fs)
 
-MODULE_DESCRIPTION("YAFFS2 - a NAND specific flash file system");
-MODULE_AUTHOR("Charles Manning, Aleph One Ltd., 2002-2010");
+    MODULE_DESCRIPTION("YAFFS2 - a NAND specific flash file system");
+MODULE_AUTHOR("Charles Manning, Aleph One Ltd., 2002-2011");
 MODULE_LICENSE("GPL");