Skip to content

Commit 6fe13aa

Browse files
raagjadavandy-shev
authored andcommitted
pinctrl: cherryview: reduce scope of PIN_CONFIG_BIAS_HIGH_IMPEDANCE case
We have a couple of pinconfig cases inside the braces which are meant for PIN_CONFIG_BIAS_HIGH_IMPEDANCE case. Although it is valid C, it makes the code less readable and prone to misinterpretation. Limit the braces to PIN_CONFIG_BIAS_HIGH_IMPEDANCE case to avoid this. Signed-off-by: Raag Jadav <raag.jadav@intel.com> Acked-by: Mika Westerberg <mika.westerberg@linux.intel.com> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
1 parent e0ba736 commit 6fe13aa

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

drivers/pinctrl/intel/pinctrl-cherryview.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -894,6 +894,7 @@ static int chv_config_get(struct pinctrl_dev *pctldev, unsigned int pin,
894894
return -EINVAL;
895895

896896
break;
897+
}
897898

898899
case PIN_CONFIG_DRIVE_PUSH_PULL:
899900
if (ctrl1 & CHV_PADCTRL1_ODEN)
@@ -904,7 +905,6 @@ static int chv_config_get(struct pinctrl_dev *pctldev, unsigned int pin,
904905
if (!(ctrl1 & CHV_PADCTRL1_ODEN))
905906
return -EINVAL;
906907
break;
907-
}
908908

909909
default:
910910
return -ENOTSUPP;

0 commit comments

Comments
 (0)