In rzg2l_pinconf_set(), there are no new variables defined in the case
statement for PIN_CONFIG_INPUT_ENABLE so no additional scope is needed.

Signed-off-by: Paul Barker <paul.barker...@bp.renesas.com>
Reviewed-by: Marek Vasut <marek.vasut+rene...@mailbox.org>
---
Changes v1->v2:
  - Split out of series adding RZ/G2L Ethernet support [1]
  - Added Marek's Reviewed-by tag

[1]: 
https://lore.kernel.org/all/20241024152448.102-1-paul.barker...@bp.renesas.com/

 drivers/pinctrl/renesas/rzg2l-pfc.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/pinctrl/renesas/rzg2l-pfc.c 
b/drivers/pinctrl/renesas/rzg2l-pfc.c
index af371bd0ff1e..3c751e9473a5 100644
--- a/drivers/pinctrl/renesas/rzg2l-pfc.c
+++ b/drivers/pinctrl/renesas/rzg2l-pfc.c
@@ -381,7 +381,7 @@ static int rzg2l_pinconf_set(struct udevice *dev, unsigned 
int pin_selector,
        }
 
        switch (param) {
-       case PIN_CONFIG_INPUT_ENABLE: {
+       case PIN_CONFIG_INPUT_ENABLE:
                if (!(cfg & PIN_CFG_IEN)) {
                        dev_err(dev, "pin does not support IEN\n");
                        return -EINVAL;
@@ -391,7 +391,6 @@ static int rzg2l_pinconf_set(struct udevice *dev, unsigned 
int pin_selector,
                        port_offset, pin, argument);
                rzg2l_rmw_pin_config(data, IEN(port_offset), pin, IEN_MASK, 
!!argument);
                break;
-       }
 
        case PIN_CONFIG_POWER_SOURCE: {
                bool support_2500 = false;
-- 
2.43.0

Reply via email to