On 12/05/2010 03:09 PM, Colin Cross wrote: > +static int __init tegra_irq_debug_init(void) > +{ > + struct dentry *d; > + > + d = debugfs_create_file("wake_irq", 0755, NULL, NULL, > + &tegra_wake_irq_debug_fops); Why not use S_IRUGO? And why is this file executable? -- Sent by an employee of the Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum. -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html