2021-10-27 21:50:36

by Francesco Dolcini

[permalink] [raw]
Subject: [PATCH] mfd: stmpe: Support disabling sub-functions

From: Oleksandr Suvorov <[email protected]>

Add support of sub-functions disabling. It allows one to define
an stmpe sub-function device in devicetree, but keep it disabled.

Signed-off-by: Oleksandr Suvorov <[email protected]>
Cc: Oleksandr Suvorov <[email protected]>
Signed-off-by: Francesco Dolcini <[email protected]>
---
Hello,
the main reason for this patch is to allow enabling/disabling sub-fuctions
using DTS overlay to enable more flexibility on SoM/Carrier boards
combinations.
---
drivers/mfd/stmpe.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mfd/stmpe.c b/drivers/mfd/stmpe.c
index e928df95e316..aeb9ea55f97d 100644
--- a/drivers/mfd/stmpe.c
+++ b/drivers/mfd/stmpe.c
@@ -1361,7 +1361,7 @@ static void stmpe_of_probe(struct stmpe_platform_data *pdata,

pdata->autosleep = (pdata->autosleep_timeout) ? true : false;

- for_each_child_of_node(np, child) {
+ for_each_available_child_of_node(np, child) {
if (of_node_name_eq(child, "stmpe_gpio")) {
pdata->blocks |= STMPE_BLOCK_GPIO;
} else if (of_node_name_eq(child, "stmpe_keypad")) {
--
2.25.1


2021-10-28 00:39:05

by Linus Walleij

[permalink] [raw]
Subject: Re: [PATCH] mfd: stmpe: Support disabling sub-functions

On Wed, Oct 27, 2021 at 10:23 AM Francesco Dolcini
<[email protected]> wrote:

> From: Oleksandr Suvorov <[email protected]>
>
> Add support of sub-functions disabling. It allows one to define
> an stmpe sub-function device in devicetree, but keep it disabled.
>
> Signed-off-by: Oleksandr Suvorov <[email protected]>
> Cc: Oleksandr Suvorov <[email protected]>
> Signed-off-by: Francesco Dolcini <[email protected]>

Looks helpful.
Reviewed-by: Linus Walleij <[email protected]>

Yours,
Linus Walleij

2021-11-01 09:55:35

by Francesco Dolcini

[permalink] [raw]
Subject: Re: [PATCH] mfd: stmpe: Support disabling sub-functions

On Wed, Oct 27, 2021 at 10:21:55AM +0200, Francesco Dolcini wrote:
> From: Oleksandr Suvorov <[email protected]>
>
> Add support of sub-functions disabling. It allows one to define
> an stmpe sub-function device in devicetree, but keep it disabled.
>
> Signed-off-by: Oleksandr Suvorov <[email protected]>
> Cc: Oleksandr Suvorov <[email protected]>
> Signed-off-by: Francesco Dolcini <[email protected]>

Hello Lee,
any chance you could take this one line patch for this merge window?

Francesco

2021-11-01 12:14:39

by Lee Jones

[permalink] [raw]
Subject: Re: [PATCH] mfd: stmpe: Support disabling sub-functions

On Mon, 01 Nov 2021, Francesco Dolcini wrote:

> On Wed, Oct 27, 2021 at 10:21:55AM +0200, Francesco Dolcini wrote:
> > From: Oleksandr Suvorov <[email protected]>
> >
> > Add support of sub-functions disabling. It allows one to define
> > an stmpe sub-function device in devicetree, but keep it disabled.
> >
> > Signed-off-by: Oleksandr Suvorov <[email protected]>
> > Cc: Oleksandr Suvorov <[email protected]>
> > Signed-off-by: Francesco Dolcini <[email protected]>
>
> Hello Lee,
> any chance you could take this one line patch for this merge window?

For a patch sent 0.5 weeks before the merge-window? Nope. :)

--
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog

2021-11-02 08:24:06

by Francesco Dolcini

[permalink] [raw]
Subject: Re: [PATCH] mfd: stmpe: Support disabling sub-functions

On Mon, Nov 01, 2021 at 12:13:23PM +0000, Lee Jones wrote:
> On Mon, 01 Nov 2021, Francesco Dolcini wrote:
> > Hello Lee,
> > any chance you could take this one line patch for this merge window?
>
> For a patch sent 0.5 weeks before the merge-window? Nope. :)

Hello Lee,
no problem, I thought it was that trivial to not create any worries.
I'll resend it in a while unless you take it before.

Francesco

2021-11-02 10:50:30

by Lee Jones

[permalink] [raw]
Subject: Re: [PATCH] mfd: stmpe: Support disabling sub-functions

On Tue, 02 Nov 2021, Francesco Dolcini wrote:

> On Mon, Nov 01, 2021 at 12:13:23PM +0000, Lee Jones wrote:
> > On Mon, 01 Nov 2021, Francesco Dolcini wrote:
> > > Hello Lee,
> > > any chance you could take this one line patch for this merge window?
> >
> > For a patch sent 0.5 weeks before the merge-window? Nope. :)
>
> Hello Lee,
> no problem, I thought it was that trivial to not create any worries.
> I'll resend it in a while unless you take it before.

I like *all* patches to have a soak in -next.

You don't need to resend it. I'll apply it in due course.

--
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog

2021-11-24 15:45:41

by Lee Jones

[permalink] [raw]
Subject: Re: [PATCH] mfd: stmpe: Support disabling sub-functions

On Wed, 27 Oct 2021, Francesco Dolcini wrote:

> From: Oleksandr Suvorov <[email protected]>
>
> Add support of sub-functions disabling. It allows one to define
> an stmpe sub-function device in devicetree, but keep it disabled.
>
> Signed-off-by: Oleksandr Suvorov <[email protected]>
> Cc: Oleksandr Suvorov <[email protected]>
> Signed-off-by: Francesco Dolcini <[email protected]>
> ---
> Hello,
> the main reason for this patch is to allow enabling/disabling sub-fuctions
> using DTS overlay to enable more flexibility on SoM/Carrier boards
> combinations.
> ---
> drivers/mfd/stmpe.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)

Applied, thanks.

--
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog