lkml.org 
[lkml]   [2008]   [Mar]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] drivers/char/synclink drivers/pcmcia/synclink - BOOLEAN to bool conversion
Joe Perches wrote:
> Remove more TRUE/FALSE defines and uses
> Remove == TRUE tests
> Convert BOOLEAN to bool
> Convert int to bool where appropriate
>
> Signed-off-by: Joe Perches <joe@perches.com>
>
> include/linux/synclink.h | 4 -
> drivers/char/pcmcia/synclink_cs.c | 125 +++++++++---------
> drivers/char/synclink.c | 258 ++++++++++++++++++-------------------
> drivers/char/synclink_gt.c | 88 +++++++-------
> drivers/char/synclinkmp.c | 157 +++++++++++-----------
> 5 files changed, 312 insertions(+), 320 deletions(-)
>
...
> diff --git a/drivers/char/synclink_gt.c b/drivers/char/synclink_gt.c
...
> @@ -4475,14 +4476,13 @@ static void reset_rbufs(struct slgt_info *info)
> /*
> * pass receive HDLC frame to upper layer
> *
> - * return 1 if frame available, otherwise 0
> + * return true if frame available, otherwise false
> */
> static int rx_get_frame(struct slgt_info *info)
> {
> unsigned int start, end;
> unsigned short status;
> unsigned int framesize = 0;
> - int rc = 0;
> unsigned long flags;
> struct tty_struct *tty = info->tty;
> unsigned char addr_field = 0xff;
> @@ -4606,10 +4606,10 @@ check_again:
> }
> }
> free_rbufs(info, start, end);
> - rc = 1;
> + return true;
>
> cleanup:
> - return rc;
> + return false;
> }

If you return bool values from this function
it would make sense to also change the return value
of the function declaration to bool.

Overall the patch seems reasonable. I need to spend
time tomorrow going over it more carefully,
compiling it, and testing it with hardware.

You may wish to shorten the subject line to:
[PATCH] synclink drivers bool conversion

It is more readable and covers all of the
patch changes including int to bool conversions.

--
Paul


\
 
 \ /
  Last update: 2008-03-09 16:09    [W:0.065 / U:0.020 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site