Commit dfb10421 authored by Hugo Villeneuve's avatar Hugo Villeneuve Committed by Greg Kroah-Hartman
Browse files

serial: sc16is7xx: add explicit return for some switch default cases 



Allows to simplify code by removing the break statement in the default
switch/case in some functions.

Reviewed-by: default avatarAndy Shevchenko <andy.shevchenko@gmail.com>
Suggested-by: default avatarAndy Shevchenko <andy.shevchenko@gmail.com>
Signed-off-by: default avatarHugo Villeneuve <hvilleneuve@dimonoff.com>
Link: https://lore.kernel.org/r/20231221231823.2327894-12-hugo@hugovil.com


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent acd7f118

drivers/tty/serial/sc16is7xx.c

+2 −6
Original line number Diff line number Diff line
@@ -460,11 +460,9 @@ static bool sc16is7xx_regmap_volatile(struct device *dev, unsigned int reg) 
	case SC16IS7XX_IOCONTROL_REG:

		return true;

	default:

		break;

	}



		return false;

	}

}



static bool sc16is7xx_regmap_precious(struct device *dev, unsigned int reg)

{
@@ -472,11 +470,9 @@ static bool sc16is7xx_regmap_precious(struct device *dev, unsigned int reg) 
	case SC16IS7XX_RHR_REG:

		return true;

	default:

		break;

	}



		return false;

	}

}



static bool sc16is7xx_regmap_noinc(struct device *dev, unsigned int reg)

{