[PATCH 1/2] staging: vt6656: aes_ccmp, baseband, bssdb, card: Change BOOL to int

Dan Carpenter dan.carpenter at oracle.com
Mon Dec 5 06:55:18 UTC 2011


On Sun, Dec 04, 2011 at 11:12:46PM -0200, Marcos Paulo de Souza wrote:
> This patch change all references of BOOL to int. As asked by the TODO
> file of the driver, we need to remove the ttype.h, and this file do a
> typedef of int to BOOL. We don't need this.
> 

Why not use bool?  Especially inside structs bool takes up less
space than an int.  Also static analysis tools care about the type.

Changing the type from int to bool is a code change so it needs
careful auditing.  It's not a problem if you don't have this
hardware, but you've been sending quite a few patches for it, so I'm
curious if you do?  If you did, and you made a code change then it
helps us to know if you have tested it.

regards,
dan carpenter

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://driverdev.linuxdriverproject.org/pipermail/driverdev-devel/attachments/20111205/42ebaf11/attachment.asc>


More information about the devel mailing list