2018-12-18 02:35:07

by Chuanjia Liu

Subject: [PATCH v2] pinctrl:mediatek:add judgment conditions

From: Chuanjia Liu <[email protected]>

This patch avoid return err when virtual gpio set SMT.

Signed-off-by: Chuanjia Liu <[email protected]>
---
change note:
v2 : 1. delete virtual gpio number
2. add judgment conditions
---
drivers/pinctrl/mediatek/pinctrl-mtk-common-v2.c | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/drivers/pinctrl/mediatek/pinctrl-mtk-common-v2.c b/drivers/pinctrl/mediatek/pinctrl-mtk-common-v2.c
index 4a9e0d4c2bbc..a576e417744a 100644
--- a/drivers/pinctrl/mediatek/pinctrl-mtk-common-v2.c
+++ b/drivers/pinctrl/mediatek/pinctrl-mtk-common-v2.c
@@ -290,7 +290,11 @@ static int mtk_xt_set_gpio_as_eint(void *data, unsigned long eint_n)
return err;

err = mtk_hw_set_value(hw, desc, PINCTRL_PIN_REG_SMT, MTK_ENABLE);
- if (err)
+
+ /* SMT doesn't support virtual GPIO, add err != -ENOTSUPP is used for
+ * avoid return err when virtual gpio set SMT.
+ */
+ if (err && err != -ENOTSUPP)
return err;

return 0;
--
2.19.1

2018-12-23 16:28:38

by Sean Wang

Subject: Re: [PATCH v2] pinctrl:mediatek:add judgment conditions

On Mon, Dec 17, 2018 at 6:32 PM <[email protected]> wrote:
>

I thought the subject still is much general, it should be further
improved with something like "add EINT support to virtual GPIOs" and
should say more about what virtual GPIOs means by MTKers in the patch
description because "virtual" is a usual term used throughout the
kernel and that would easily cause others to fail to interpret
correctly.

> From: Chuanjia Liu <[email protected]>
>
> This patch avoid return err when virtual gpio set SMT.
>
> Signed-off-by: Chuanjia Liu <[email protected]>
> ---
> change note:
> v2 : 1. delete virtual gpio number
> 2. add judgment conditions
> ---
> drivers/pinctrl/mediatek/pinctrl-mtk-common-v2.c | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/pinctrl/mediatek/pinctrl-mtk-common-v2.c b/drivers/pinctrl/mediatek/pinctrl-mtk-common-v2.c
> index 4a9e0d4c2bbc..a576e417744a 100644
> --- a/drivers/pinctrl/mediatek/pinctrl-mtk-common-v2.c
> +++ b/drivers/pinctrl/mediatek/pinctrl-mtk-common-v2.c
> @@ -290,7 +290,11 @@ static int mtk_xt_set_gpio_as_eint(void *data, unsigned long eint_n)
> return err;
>
> err = mtk_hw_set_value(hw, desc, PINCTRL_PIN_REG_SMT, MTK_ENABLE);
> - if (err)
> +
> + /* SMT doesn't support virtual GPIO, add err != -ENOTSUPP is used for
> + * avoid return err when virtual gpio set SMT.
> + */

I'd prefer to the comments as the below

/*
SMT is supposed to be supported by every real GPIO and doesn't support
virtual GPIOs, so the extra condition err != -ENOTSUPP is just for
adding EINT support to these virtual GPIOs. It should add an extra
flag in the pin descriptor when more pins with distinctive
characteristic come out.
*/

> + if (err && err != -ENOTSUPP)
> return err;
>
> return 0;
> --
> 2.19.1
>

2018-12-25 03:43:48

by Chuanjia Liu

Subject: Re: [PATCH v2] pinctrl:mediatek:add judgment conditions

On Sun, 2018-12-23 at 02:50 +0800, Sean Wang wrote:
> On Mon, Dec 17, 2018 at 6:32 PM <[email protected]> wrote:
> >
>
> I thought the subject still is much general, it should be further
> improved with something like "add EINT support to virtual GPIOs" and
> should say more about what virtual GPIOs means by MTKers in the patch
> description because "virtual" is a usual term used throughout the
> kernel and that would easily cause others to fail to interpret
> correctly.
>
Thank you for your advice,I will modify subject and description.
> > From: Chuanjia Liu <[email protected]>
> >
> > This patch avoid return err when virtual gpio set SMT.
> >
> > Signed-off-by: Chuanjia Liu <[email protected]>
> > ---
> > change note:
> > v2 : 1. delete virtual gpio number
> > 2. add judgment conditions
> > ---
> > drivers/pinctrl/mediatek/pinctrl-mtk-common-v2.c | 6 +++++-
> > 1 file changed, 5 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/pinctrl/mediatek/pinctrl-mtk-common-v2.c b/drivers/pinctrl/mediatek/pinctrl-mtk-common-v2.c
> > index 4a9e0d4c2bbc..a576e417744a 100644
> > --- a/drivers/pinctrl/mediatek/pinctrl-mtk-common-v2.c
> > +++ b/drivers/pinctrl/mediatek/pinctrl-mtk-common-v2.c
> > @@ -290,7 +290,11 @@ static int mtk_xt_set_gpio_as_eint(void *data, unsigned long eint_n)
> > return err;
> >
> > err = mtk_hw_set_value(hw, desc, PINCTRL_PIN_REG_SMT, MTK_ENABLE);
> > - if (err)
> > +
> > + /* SMT doesn't support virtual GPIO, add err != -ENOTSUPP is used for
> > + * avoid return err when virtual gpio set SMT.
> > + */
>
> I'd prefer to the comments as the below
>
> /*
> SMT is supposed to be supported by every real GPIO and doesn't support
> virtual GPIOs, so the extra condition err != -ENOTSUPP is just for
> adding EINT support to these virtual GPIOs. It should add an extra
> flag in the pin descriptor when more pins with distinctive
> characteristic come out.
> */
>
Thank you for your advice,I will modify comments.
> > + if (err && err != -ENOTSUPP)
> > return err;
> >
> > return 0;
> > --
> > 2.19.1
> >