On Thu, Mar 06, 2014 at 04:19:50AM +0000, quwenruo@xxxxxxxxxxxxxx wrote: > @@ -23,11 +23,13 @@ > struct btrfs_workqueue; > /* Internal use only */ > struct __btrfs_workqueue; > +struct btrfs_work; > +typedef void (*btrfs_func_t)(struct btrfs_work *arg); I don't see what's wrong with the non-typedef type, CodingStyle discourages from using typedefs in general (Chapter 5). The name btrfs_func_t is a generic, if you really need to use a typedef here, please change it to something closer to the workqueues, eg. btrfs_work_func_t. -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
