2020-01-10 15:01:15

by Matthias Brugger

[permalink] [raw]
Subject: [PATCH] dt-bindings: mfd: mediatek: Add MT6397 Pin Controller

From: Matthias Brugger <[email protected]>

The MT6397 mfd includes a pin controller. Add binding
a description for it.

Signed-off-by: Matthias Brugger <[email protected]>

---

Documentation/devicetree/bindings/mfd/mt6397.txt | 5 +++++
1 file changed, 5 insertions(+)

diff --git a/Documentation/devicetree/bindings/mfd/mt6397.txt b/Documentation/devicetree/bindings/mfd/mt6397.txt
index a9b105ac00a8..ce22fca9d48b 100644
--- a/Documentation/devicetree/bindings/mfd/mt6397.txt
+++ b/Documentation/devicetree/bindings/mfd/mt6397.txt
@@ -54,6 +54,11 @@ Optional subnodes:
- compatible: "mediatek,mt6323-pwrc"
For details, see ../power/reset/mt6323-poweroff.txt

+- pin-controller
+ Required properties:
+ - compatible: "mediatek,mt6397-pinctrl"
+ For details, see ../pinctrl/pinctrl-mt65xx.txt
+
Example:
pwrap: pwrap@1000f000 {
compatible = "mediatek,mt8135-pwrap";
--
2.24.0


2020-01-13 07:09:35

by Nicolas Boichat

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: mfd: mediatek: Add MT6397 Pin Controller

On Fri, Jan 10, 2020 at 11:00 PM <[email protected]> wrote:
>
> From: Matthias Brugger <[email protected]>
>
> The MT6397 mfd includes a pin controller. Add binding
> a description for it.
>
> Signed-off-by: Matthias Brugger <[email protected]>

Reviewed-by: Nicolas Boichat <[email protected]>

>
> ---
>
> Documentation/devicetree/bindings/mfd/mt6397.txt | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/mfd/mt6397.txt b/Documentation/devicetree/bindings/mfd/mt6397.txt
> index a9b105ac00a8..ce22fca9d48b 100644
> --- a/Documentation/devicetree/bindings/mfd/mt6397.txt
> +++ b/Documentation/devicetree/bindings/mfd/mt6397.txt
> @@ -54,6 +54,11 @@ Optional subnodes:
> - compatible: "mediatek,mt6323-pwrc"
> For details, see ../power/reset/mt6323-poweroff.txt
>
> +- pin-controller
> + Required properties:
> + - compatible: "mediatek,mt6397-pinctrl"
> + For details, see ../pinctrl/pinctrl-mt65xx.txt
> +
> Example:
> pwrap: pwrap@1000f000 {
> compatible = "mediatek,mt8135-pwrap";
> --
> 2.24.0
>

2020-01-15 15:21:24

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: mfd: mediatek: Add MT6397 Pin Controller

On Fri, 10 Jan 2020 15:59:51 +0100, [email protected] wrote:
> From: Matthias Brugger <[email protected]>
>
> The MT6397 mfd includes a pin controller. Add binding
> a description for it.
>
> Signed-off-by: Matthias Brugger <[email protected]>
>
> ---
>
> Documentation/devicetree/bindings/mfd/mt6397.txt | 5 +++++
> 1 file changed, 5 insertions(+)
>

Acked-by: Rob Herring <[email protected]>

2020-05-21 10:19:22

by Matthias Brugger

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: mfd: mediatek: Add MT6397 Pin Controller

Hi Lee,

On 15/01/2020 16:19, Rob Herring wrote:
> On Fri, 10 Jan 2020 15:59:51 +0100, [email protected] wrote:
>> From: Matthias Brugger <[email protected]>
>>
>> The MT6397 mfd includes a pin controller. Add binding
>> a description for it.
>>
>> Signed-off-by: Matthias Brugger <[email protected]>
>>
>> ---
>>
>> Documentation/devicetree/bindings/mfd/mt6397.txt | 5 +++++
>> 1 file changed, 5 insertions(+)
>>
>
> Acked-by: Rob Herring <[email protected]>
>

It looks like this fall through the cracks.
Would you consider to queue it or do you have further comments?

Regards,
Matthias

2020-05-22 07:39:33

by Lee Jones

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: mfd: mediatek: Add MT6397 Pin Controller

On Thu, 21 May 2020, Matthias Brugger wrote:

> Hi Lee,
>
> On 15/01/2020 16:19, Rob Herring wrote:
> > On Fri, 10 Jan 2020 15:59:51 +0100, [email protected] wrote:
> >> From: Matthias Brugger <[email protected]>
> >>
> >> The MT6397 mfd includes a pin controller. Add binding
> >> a description for it.
> >>
> >> Signed-off-by: Matthias Brugger <[email protected]>
> >>
> >> ---
> >>
> >> Documentation/devicetree/bindings/mfd/mt6397.txt | 5 +++++
> >> 1 file changed, 5 insertions(+)
> >>
> >
> > Acked-by: Rob Herring <[email protected]>
> >
>
> It looks like this fall through the cracks.
> Would you consider to queue it or do you have further comments?

There is a current issue where Rob and I are both taking patches.
Sometimes we both assume the other will take the patch.

Apologies.

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

2020-05-22 07:40:22

by Lee Jones

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: mfd: mediatek: Add MT6397 Pin Controller

On Fri, 10 Jan 2020, [email protected] wrote:

> From: Matthias Brugger <[email protected]>
>
> The MT6397 mfd includes a pin controller. Add binding
> a description for it.
>
> Signed-off-by: Matthias Brugger <[email protected]>
>
> ---
>
> Documentation/devicetree/bindings/mfd/mt6397.txt | 5 +++++
> 1 file changed, 5 insertions(+)

Applied, thanks.

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