Re: [patch] dma-debug: debugfs_create_bool() takes a u32 pointer

From: Dan Carpenter
Date: Fri Mar 02 2012 - 07:53:56 EST


On Fri, Mar 02, 2012 at 06:22:03AM -0500, Neil Horman wrote:
> This certainly works, but a quick search of call sites shows this isn't the only
> place someone passes a bool to debugfs_create_bool. I think the better solution
> would be to change debugfs_create_bool and its corresponding read/write methods
> to actually work with a bool type, and then clean all the call sites to pass in
> a bool properly, instead of a u32.

There is only one other call site which gets this wrong, and I sent
a fix for that today as well. But yes, you're probably right that
would be cleaner.

regards,
dan carpenter

Attachment: signature.asc
Description: Digital signature