serial: max310x: 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.

Suggested-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Signed-off-by: Hugo Villeneuve <hvilleneuve@dimonoff.com>
Link: https://lore.kernel.org/r/20240118152213.2644269-11-hugo@hugovil.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/tty/serial/max310x.c b/drivers/tty/serial/max310x.c
index 9ef146f09d5b2d024ea11715eb2238a843edb3ed..048ae432ba4847a060059fe85db9e18eeaf377e8 100644 (file)
--- a/drivers/tty/serial/max310x.c
+++ b/drivers/tty/serial/max310x.c
@@ -483,10 +483,8 @@ static bool max310x_reg_writeable(struct device *dev, unsigned int reg)
        case MAX310X_RXFIFOLVL_REG:
                return false;
        default:
-               break;
+               return true;
        }
-
-       return true;
 }
 
 static bool max310x_reg_volatile(struct device *dev, unsigned int reg)
@@ -505,10 +503,8 @@ static bool max310x_reg_volatile(struct device *dev, unsigned int reg)
        case MAX310X_REG_1F:
                return true;
        default:
-               break;
+               return false;
        }
-
-       return false;
 }
 
 static bool max310x_reg_precious(struct device *dev, unsigned int reg)
@@ -520,10 +516,8 @@ static bool max310x_reg_precious(struct device *dev, unsigned int reg)
        case MAX310X_STS_IRQSTS_REG:
                return true;
        default:
-               break;
+               return false;
        }
-
-       return false;
 }
 
 static bool max310x_reg_noinc(struct device *dev, unsigned int reg)