| Message ID | 1610509395-107554-1-git-send-email-abaci-bugfix@linux.alibaba.com |
|---|---|
| State | New |
| Headers | show |
| Series | [v2] pinctrl: sprd: Simplify bool comparison | expand |
On Wed, Jan 13, 2021 at 11:43 AM Yang Li <abaci-bugfix@linux.alibaba.com> wrote: > > Fix the following coccicheck warning: > ./drivers/pinctrl/sprd/pinctrl-sprd.c:690:8-23: WARNING: Comparison to > bool > > Reported-by: Abaci Robot <abaci@linux.alibaba.com> > Signed-off-by: Yang Li <abaci-bugfix@linux.alibaba.com> You should keep other guy's reviewed-by or acked-by tag for the following version if no other big changes. So again Reviewed-by: Baolin Wang <baolin.wang7@gmail.com> > --- > Changes in v2: > - make "pinctrl: sprd:" as subject prefix > > drivers/pinctrl/sprd/pinctrl-sprd.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/pinctrl/sprd/pinctrl-sprd.c b/drivers/pinctrl/sprd/pinctrl-sprd.c > index 08dc193..dca7a50 100644 > --- a/drivers/pinctrl/sprd/pinctrl-sprd.c > +++ b/drivers/pinctrl/sprd/pinctrl-sprd.c > @@ -687,7 +687,7 @@ static int sprd_pinconf_set(struct pinctrl_dev *pctldev, unsigned int pin_id, > shift = INPUT_SCHMITT_SHIFT; > break; > case PIN_CONFIG_BIAS_PULL_UP: > - if (is_sleep_config == true) { > + if (is_sleep_config) { > val |= SLEEP_PULL_UP; > mask = SLEEP_PULL_UP_MASK; > shift = SLEEP_PULL_UP_SHIFT; > -- > 1.8.3.1 >
diff --git a/drivers/pinctrl/sprd/pinctrl-sprd.c b/drivers/pinctrl/sprd/pinctrl-sprd.c index 08dc193..dca7a50 100644 --- a/drivers/pinctrl/sprd/pinctrl-sprd.c +++ b/drivers/pinctrl/sprd/pinctrl-sprd.c @@ -687,7 +687,7 @@ static int sprd_pinconf_set(struct pinctrl_dev *pctldev, unsigned int pin_id, shift = INPUT_SCHMITT_SHIFT; break; case PIN_CONFIG_BIAS_PULL_UP: - if (is_sleep_config == true) { + if (is_sleep_config) { val |= SLEEP_PULL_UP; mask = SLEEP_PULL_UP_MASK; shift = SLEEP_PULL_UP_SHIFT;
Fix the following coccicheck warning: ./drivers/pinctrl/sprd/pinctrl-sprd.c:690:8-23: WARNING: Comparison to bool Reported-by: Abaci Robot <abaci@linux.alibaba.com> Signed-off-by: Yang Li <abaci-bugfix@linux.alibaba.com> --- Changes in v2: - make "pinctrl: sprd:" as subject prefix drivers/pinctrl/sprd/pinctrl-sprd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)