linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] dt-bindings: mfd: mediatek: Add MT6397 Pin Controller
@ 2020-01-10 14:59 matthias.bgg
  2020-01-13  7:08 ` Nicolas Boichat
                   ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: matthias.bgg @ 2020-01-10 14:59 UTC (permalink / raw)
  To: Lee Jones
  Cc: Matthias Brugger, Mark Rutland, Rob Herring, devicetree,
	linux-arm-kernel, linux-kernel, linux-mediatek

From: Matthias Brugger <matthias.bgg@gmail.com>

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

Signed-off-by: Matthias Brugger <matthias.bgg@gmail.com>

---

 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


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH] dt-bindings: mfd: mediatek: Add MT6397 Pin Controller
  2020-01-10 14:59 [PATCH] dt-bindings: mfd: mediatek: Add MT6397 Pin Controller matthias.bgg
@ 2020-01-13  7:08 ` Nicolas Boichat
  2020-01-15 15:19 ` Rob Herring
  2020-05-22  7:37 ` Lee Jones
  2 siblings, 0 replies; 6+ messages in thread
From: Nicolas Boichat @ 2020-01-13  7:08 UTC (permalink / raw)
  To: matthias.bgg
  Cc: Lee Jones, Matthias Brugger, Mark Rutland, Rob Herring,
	Devicetree List, linux-arm Mailing List, lkml,
	moderated list:ARM/Mediatek SoC support

On Fri, Jan 10, 2020 at 11:00 PM <matthias.bgg@kernel.org> wrote:
>
> From: Matthias Brugger <matthias.bgg@gmail.com>
>
> The MT6397 mfd includes a pin controller. Add binding
> a description for it.
>
> Signed-off-by: Matthias Brugger <matthias.bgg@gmail.com>

Reviewed-by: Nicolas Boichat <drinkcat@chromium.org>

>
> ---
>
>  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
>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] dt-bindings: mfd: mediatek: Add MT6397 Pin Controller
  2020-01-10 14:59 [PATCH] dt-bindings: mfd: mediatek: Add MT6397 Pin Controller matthias.bgg
  2020-01-13  7:08 ` Nicolas Boichat
@ 2020-01-15 15:19 ` Rob Herring
  2020-05-21 10:16   ` Matthias Brugger
  2020-05-22  7:37 ` Lee Jones
  2 siblings, 1 reply; 6+ messages in thread
From: Rob Herring @ 2020-01-15 15:19 UTC (permalink / raw)
  To: matthias.bgg
  Cc: Lee Jones, Matthias Brugger, Mark Rutland, devicetree,
	linux-arm-kernel, linux-kernel, linux-mediatek

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

Acked-by: Rob Herring <robh@kernel.org>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] dt-bindings: mfd: mediatek: Add MT6397 Pin Controller
  2020-01-15 15:19 ` Rob Herring
@ 2020-05-21 10:16   ` Matthias Brugger
  2020-05-22  7:37     ` Lee Jones
  0 siblings, 1 reply; 6+ messages in thread
From: Matthias Brugger @ 2020-05-21 10:16 UTC (permalink / raw)
  To: matthias.bgg, Lee Jones
  Cc: Rob Herring, Mark Rutland, devicetree, linux-arm-kernel,
	linux-kernel, linux-mediatek

Hi Lee,

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

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

Regards,
Matthias

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] dt-bindings: mfd: mediatek: Add MT6397 Pin Controller
  2020-05-21 10:16   ` Matthias Brugger
@ 2020-05-22  7:37     ` Lee Jones
  0 siblings, 0 replies; 6+ messages in thread
From: Lee Jones @ 2020-05-22  7:37 UTC (permalink / raw)
  To: Matthias Brugger
  Cc: matthias.bgg, Rob Herring, Mark Rutland, devicetree,
	linux-arm-kernel, linux-kernel, linux-mediatek

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, matthias.bgg@kernel.org wrote:
> >> From: Matthias Brugger <matthias.bgg@gmail.com>
> >>
> >> The MT6397 mfd includes a pin controller. Add binding
> >> a description for it.
> >>
> >> Signed-off-by: Matthias Brugger <matthias.bgg@gmail.com>
> >>
> >> ---
> >>
> >>  Documentation/devicetree/bindings/mfd/mt6397.txt | 5 +++++
> >>  1 file changed, 5 insertions(+)
> >>
> > 
> > Acked-by: Rob Herring <robh@kernel.org>
> > 
> 
> 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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] dt-bindings: mfd: mediatek: Add MT6397 Pin Controller
  2020-01-10 14:59 [PATCH] dt-bindings: mfd: mediatek: Add MT6397 Pin Controller matthias.bgg
  2020-01-13  7:08 ` Nicolas Boichat
  2020-01-15 15:19 ` Rob Herring
@ 2020-05-22  7:37 ` Lee Jones
  2 siblings, 0 replies; 6+ messages in thread
From: Lee Jones @ 2020-05-22  7:37 UTC (permalink / raw)
  To: matthias.bgg
  Cc: Matthias Brugger, Mark Rutland, Rob Herring, devicetree,
	linux-arm-kernel, linux-kernel, linux-mediatek

On Fri, 10 Jan 2020, matthias.bgg@kernel.org wrote:

> From: Matthias Brugger <matthias.bgg@gmail.com>
> 
> The MT6397 mfd includes a pin controller. Add binding
> a description for it.
> 
> Signed-off-by: Matthias Brugger <matthias.bgg@gmail.com>
> 
> ---
> 
>  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

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2020-05-22  7:37 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-01-10 14:59 [PATCH] dt-bindings: mfd: mediatek: Add MT6397 Pin Controller matthias.bgg
2020-01-13  7:08 ` Nicolas Boichat
2020-01-15 15:19 ` Rob Herring
2020-05-21 10:16   ` Matthias Brugger
2020-05-22  7:37     ` Lee Jones
2020-05-22  7:37 ` Lee Jones

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).