On Wed, Jan 04, 2023 at 05:15:31PM +0200, Ilpo Järvinen wrote:
> Change tty_termios_hw_change() return to bool and convert it to
> a or-chained return statement.
>
> Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@xxxxxxxxxxxxxxx>
> ---
> drivers/tty/tty_ioctl.c | 10 ++++------
> include/linux/tty.h | 2 +-
> 2 files changed, 5 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/tty/tty_ioctl.c b/drivers/tty/tty_ioctl.c
> index ce511557b98b..4a80fabf6d5e 100644
> --- a/drivers/tty/tty_ioctl.c
> +++ b/drivers/tty/tty_ioctl.c
> @@ -270,13 +270,11 @@ EXPORT_SYMBOL(tty_termios_copy_hw);
> * between the two termios structures, or a speed change is needed.
> */
>
> -int tty_termios_hw_change(const struct ktermios *a, const struct ktermios *b)
> +bool tty_termios_hw_change(const struct ktermios *a, const struct ktermios *b)
> {
> - if (a->c_ispeed != b->c_ispeed || a->c_ospeed != b->c_ospeed)
> - return 1;
> - if ((a->c_cflag ^ b->c_cflag) & ~(HUPCL | CREAD | CLOCAL))
> - return 1;
> - return 0;
> + return a->c_ispeed != b->c_ispeed ||
> + a->c_ospeed != b->c_ospeed ||
> + (a->c_cflag ^ b->c_cflag) & ~(HUPCL | CREAD | CLOCAL);
Please keep the current structure which is more readable rather than
merge everything into one long expression.
Johan
