LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* Re: [PATCH v5 0/5] Add Mediatek SoC Pinctrl/GPIO/EINT driver for MT8135.
       [not found] <1421818097-9281-1-git-send-email-hongzhou.yang@mediatek.com>
@ 2015-01-21  5:39 ` Yingjoe Chen
       [not found] ` <1421818097-9281-2-git-send-email-hongzhou.yang@mediatek.com>
                   ` (4 subsequent siblings)
  5 siblings, 0 replies; 18+ messages in thread
From: Yingjoe Chen @ 2015-01-21  5:39 UTC (permalink / raw)
  To: Hongzhou Yang
  Cc: Rob Herring, Linus Walleij, Matthias Brugger, Sascha Hauer,
	Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala, Russell King,
	Grant Likely, Catalin Marinas, Vladimir Murzin, Ashwin Chaugule,
	devicetree, linux-kernel, linux-arm-kernel, dandan.he,
	alan.cheng, toby.liu, maoguang.meng, eddie.huang,
	Jean-Christophe PLAGNIOL-VILLARD

On Wed, 2015-01-21 at 13:28 +0800, Hongzhou Yang wrote:
> This is v5 of add Mediatek SoC Pinctrl/GPIO/EINT driver for MT8135.
> It is based on Joe.C' basic device tree support.
> See http://lists.infradead.org/pipermail/linux-arm-kernel/2014-October/296093.html

Hi,

A little correction.
I just talked to Hongzhou, this series is based on 3.19-rc1 instead of
my old patch. Sorry for the confusion.

Joe.C



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

* Re: [PATCH v5 1/5] ARM: mediatek: Add config options for mediatek SoCs.
       [not found] ` <1421818097-9281-2-git-send-email-hongzhou.yang@mediatek.com>
@ 2015-01-22 12:54   ` Matthias Brugger
  2015-01-27 10:16     ` Paul Bolle
  0 siblings, 1 reply; 18+ messages in thread
From: Matthias Brugger @ 2015-01-22 12:54 UTC (permalink / raw)
  To: Hongzhou Yang
  Cc: Rob Herring, Linus Walleij, Sascha Hauer, Pawel Moll,
	Mark Rutland, Ian Campbell, Kumar Gala, Russell King,
	Grant Likely, Joe.C, Catalin Marinas, Vladimir Murzin,
	Ashwin Chaugule, devicetree, linux-kernel, linux-arm-kernel,
	dandan.he, alan.cheng, toby.liu, maoguang.meng, huang eddie,
	Jean-Christophe PLAGNIOL-VILLARD

2015-01-21 6:28 GMT+01:00 Hongzhou Yang <hongzhou.yang@mediatek.com>:
> From: Yingjoe Chen <yingjoe.chen@mediatek.com>
>
> The upcoming MTK pinctrl driver have a big pin table for each SoC
> and we don't want to bloat the kernel binary if we don't need it.
> Add config options so we can build for one SoC only.
>
> Acked-by: Linus Walleij <linus.walleij@linaro.org>
> Signed-off-by: Yingjoe Chen <yingjoe.chen@mediatek.com>
> Signed-off-by: Hongzhou Yang <hongzhou.yang@mediatek.com>

Applied to v3.20-next/soc

> ---
>  arch/arm/mach-mediatek/Kconfig | 22 +++++++++++++++++++++-
>  1 file changed, 21 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm/mach-mediatek/Kconfig b/arch/arm/mach-mediatek/Kconfig
> index f73f588..f7e463c 100644
> --- a/arch/arm/mach-mediatek/Kconfig
> +++ b/arch/arm/mach-mediatek/Kconfig
> @@ -1,6 +1,26 @@
> -config ARCH_MEDIATEK
> +menuconfig ARCH_MEDIATEK
>         bool "Mediatek MT65xx & MT81xx SoC" if ARCH_MULTI_V7
>         select ARM_GIC
>         select MTK_TIMER
>         help
>           Support for Mediatek MT65xx & MT81xx SoCs
> +
> +if ARCH_MEDIATEK
> +
> +config MACH_MT6589
> +       bool "MediaTek MT6589 SoCs support"
> +       default ARCH_MEDIATEK
> +
> +config MACH_MT6592
> +       bool "MediaTek MT6592 SoCs support"
> +       default ARCH_MEDIATEK
> +
> +config MACH_MT8127
> +       bool "MediaTek MT8127 SoCs support"
> +       default ARCH_MEDIATEK
> +
> +config MACH_MT8135
> +       bool "MediaTek MT8135 SoCs support"
> +       default ARCH_MEDIATEK
> +
> +endif
> --
> 1.8.1.1.dirty
>



-- 
motzblog.wordpress.com

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

* Re: [PATCH v5 1/5] ARM: mediatek: Add config options for mediatek SoCs.
  2015-01-22 12:54   ` [PATCH v5 1/5] ARM: mediatek: Add config options for mediatek SoCs Matthias Brugger
@ 2015-01-27 10:16     ` Paul Bolle
  2015-01-27 10:26       ` Yingjoe Chen
  0 siblings, 1 reply; 18+ messages in thread
From: Paul Bolle @ 2015-01-27 10:16 UTC (permalink / raw)
  To: Joe. C
  Cc: Valentin Rothberg, Matthias Brugger, Hongzhou Yang, Rob Herring,
	Linus Walleij, Sascha Hauer, Pawel Moll, Mark Rutland,
	Ian Campbell, Kumar Gala, Russell King, Grant Likely,
	Catalin Marinas, Vladimir Murzin, Ashwin Chaugule, devicetree,
	linux-kernel, linux-arm-kernel, dandan.he, alan.cheng, toby.liu,
	maoguang.meng, huang eddie, Jean-Christophe PLAGNIOL-VILLARD

Joe,

On Thu, 2015-01-22 at 13:54 +0100, Matthias Brugger wrote:
> 2015-01-21 6:28 GMT+01:00 Hongzhou Yang <hongzhou.yang@mediatek.com>:
> > From: Yingjoe Chen <yingjoe.chen@mediatek.com>
> >
> > The upcoming MTK pinctrl driver have a big pin table for each SoC
> > and we don't want to bloat the kernel binary if we don't need it.
> > Add config options so we can build for one SoC only.
> >
> > Acked-by: Linus Walleij <linus.walleij@linaro.org>
> > Signed-off-by: Yingjoe Chen <yingjoe.chen@mediatek.com>
> > Signed-off-by: Hongzhou Yang <hongzhou.yang@mediatek.com>
> 
> Applied to v3.20-next/soc

This became commit ad8a221e1f49 ("ARM: mediatek: Add config options for
mediatek SoCs.") in today's linux-next (ie, next-20150127). I noticed
because a script I use to check linux-next spotted a problem with it.

> > ---
> >  arch/arm/mach-mediatek/Kconfig | 22 +++++++++++++++++++++-
> >  1 file changed, 21 insertions(+), 1 deletion(-)
> >
> > diff --git a/arch/arm/mach-mediatek/Kconfig b/arch/arm/mach-mediatek/Kconfig
> > index f73f588..f7e463c 100644
> > --- a/arch/arm/mach-mediatek/Kconfig
> > +++ b/arch/arm/mach-mediatek/Kconfig
> > @@ -1,6 +1,26 @@
> > -config ARCH_MEDIATEK
> > +menuconfig ARCH_MEDIATEK
> >         bool "Mediatek MT65xx & MT81xx SoC" if ARCH_MULTI_V7
> >         select ARM_GIC
> >         select MTK_TIMER
> >         help
> >           Support for Mediatek MT65xx & MT81xx SoCs
> > +
> > +if ARCH_MEDIATEK
> > +
> > +config MACH_MT6589
> > +       bool "MediaTek MT6589 SoCs support"
> > +       default ARCH_MEDIATEK
> > +
> > +config MACH_MT6592
> > +       bool "MediaTek MT6592 SoCs support"
> > +       default ARCH_MEDIATEK
> > +
> > +config MACH_MT8127
> > +       bool "MediaTek MT8127 SoCs support"
> > +       default ARCH_MEDIATEK
> > +
> > +config MACH_MT8135
> > +       bool "MediaTek MT8135 SoCs support"
> > +       default ARCH_MEDIATEK
> > +
> > +endif

None of these four new MACH_MT* Kconfig symbols are currently used in
linux-next. I assume that patches that actually use them (either as a
Kconfig symbol or as a CONFIG_* macro) are still pending. Is that
correct?

Thanks,


Paul Bolle


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

* Re: [PATCH v5 1/5] ARM: mediatek: Add config options for mediatek SoCs.
  2015-01-27 10:16     ` Paul Bolle
@ 2015-01-27 10:26       ` Yingjoe Chen
  2015-01-27 10:34         ` Paul Bolle
  0 siblings, 1 reply; 18+ messages in thread
From: Yingjoe Chen @ 2015-01-27 10:26 UTC (permalink / raw)
  To: Paul Bolle
  Cc: Valentin Rothberg, Matthias Brugger, Hongzhou Yang, Rob Herring,
	Linus Walleij, Sascha Hauer, Pawel Moll, Mark Rutland,
	Ian Campbell, Kumar Gala, Russell King, Grant Likely,
	Catalin Marinas, Vladimir Murzin, Ashwin Chaugule, devicetree,
	linux-kernel, linux-arm-kernel, dandan.he, alan.cheng, toby.liu,
	maoguang.meng, huang eddie, Jean-Christophe PLAGNIOL-VILLARD


Hi,

On Tue, 2015-01-27 at 11:16 +0100, Paul Bolle wrote:
> Joe,
> 
> On Thu, 2015-01-22 at 13:54 +0100, Matthias Brugger wrote:
> > 2015-01-21 6:28 GMT+01:00 Hongzhou Yang <hongzhou.yang@mediatek.com>:
> > > From: Yingjoe Chen <yingjoe.chen@mediatek.com>
> > >
> > > The upcoming MTK pinctrl driver have a big pin table for each SoC
> > > and we don't want to bloat the kernel binary if we don't need it.
> > > Add config options so we can build for one SoC only.
> > >
> > > Acked-by: Linus Walleij <linus.walleij@linaro.org>
> > > Signed-off-by: Yingjoe Chen <yingjoe.chen@mediatek.com>
> > > Signed-off-by: Hongzhou Yang <hongzhou.yang@mediatek.com>
> > 
> > Applied to v3.20-next/soc
> 
> This became commit ad8a221e1f49 ("ARM: mediatek: Add config options for
> mediatek SoCs.") in today's linux-next (ie, next-20150127). I noticed
> because a script I use to check linux-next spotted a problem with it.
> 
> > > ---
> > >  arch/arm/mach-mediatek/Kconfig | 22 +++++++++++++++++++++-
> > >  1 file changed, 21 insertions(+), 1 deletion(-)
> > >
> > > diff --git a/arch/arm/mach-mediatek/Kconfig b/arch/arm/mach-mediatek/Kconfig
> > > index f73f588..f7e463c 100644
> > > --- a/arch/arm/mach-mediatek/Kconfig
> > > +++ b/arch/arm/mach-mediatek/Kconfig
> > > @@ -1,6 +1,26 @@
> > > -config ARCH_MEDIATEK
> > > +menuconfig ARCH_MEDIATEK
> > >         bool "Mediatek MT65xx & MT81xx SoC" if ARCH_MULTI_V7
> > >         select ARM_GIC
> > >         select MTK_TIMER
> > >         help
> > >           Support for Mediatek MT65xx & MT81xx SoCs
> > > +
> > > +if ARCH_MEDIATEK
> > > +
> > > +config MACH_MT6589
> > > +       bool "MediaTek MT6589 SoCs support"
> > > +       default ARCH_MEDIATEK
> > > +
> > > +config MACH_MT6592
> > > +       bool "MediaTek MT6592 SoCs support"
> > > +       default ARCH_MEDIATEK
> > > +
> > > +config MACH_MT8127
> > > +       bool "MediaTek MT8127 SoCs support"
> > > +       default ARCH_MEDIATEK
> > > +
> > > +config MACH_MT8135
> > > +       bool "MediaTek MT8135 SoCs support"
> > > +       default ARCH_MEDIATEK
> > > +
> > > +endif
> 
> None of these four new MACH_MT* Kconfig symbols are currently used in
> linux-next. I assume that patches that actually use them (either as a
> Kconfig symbol or as a CONFIG_* macro) are still pending. Is that
> correct?

Yes, it is used in patch 3 in this series[1], and is discussed in [2].

Joe.C

[1]
http://lists.infradead.org/pipermail/linux-arm-kernel/2015-January/318453.html
[2]
http://lists.infradead.org/pipermail/linux-arm-kernel/2015-January/315616.html





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

* Re: [PATCH v5 1/5] ARM: mediatek: Add config options for mediatek SoCs.
  2015-01-27 10:26       ` Yingjoe Chen
@ 2015-01-27 10:34         ` Paul Bolle
  2015-01-27 12:38           ` Matthias Brugger
  0 siblings, 1 reply; 18+ messages in thread
From: Paul Bolle @ 2015-01-27 10:34 UTC (permalink / raw)
  To: Yingjoe Chen
  Cc: Valentin Rothberg, Matthias Brugger, Hongzhou Yang, Rob Herring,
	Linus Walleij, Sascha Hauer, Pawel Moll, Mark Rutland,
	Ian Campbell, Kumar Gala, Russell King, Grant Likely,
	Catalin Marinas, Vladimir Murzin, Ashwin Chaugule, devicetree,
	linux-kernel, linux-arm-kernel, dandan.he, alan.cheng, toby.liu,
	maoguang.meng, huang eddie, Jean-Christophe PLAGNIOL-VILLARD

Hi Joe,

On Tue, 2015-01-27 at 18:26 +0800, Yingjoe Chen wrote:
> On Tue, 2015-01-27 at 11:16 +0100, Paul Bolle wrote:
> > On Thu, 2015-01-22 at 13:54 +0100, Matthias Brugger wrote:
> > > 2015-01-21 6:28 GMT+01:00 Hongzhou Yang <hongzhou.yang@mediatek.com>:
> > > > From: Yingjoe Chen <yingjoe.chen@mediatek.com>
> > > >
> > > > The upcoming MTK pinctrl driver have a big pin table for each SoC
> > > > and we don't want to bloat the kernel binary if we don't need it.
> > > > Add config options so we can build for one SoC only.
> > > >
> > > > Acked-by: Linus Walleij <linus.walleij@linaro.org>
> > > > Signed-off-by: Yingjoe Chen <yingjoe.chen@mediatek.com>
> > > > Signed-off-by: Hongzhou Yang <hongzhou.yang@mediatek.com>
> > > 
> > > Applied to v3.20-next/soc
> > 
> > This became commit ad8a221e1f49 ("ARM: mediatek: Add config options for
> > mediatek SoCs.") in today's linux-next (ie, next-20150127). I noticed
> > because a script I use to check linux-next spotted a problem with it.
> > 
> > > > ---
> > > >  arch/arm/mach-mediatek/Kconfig | 22 +++++++++++++++++++++-
> > > >  1 file changed, 21 insertions(+), 1 deletion(-)
> > > >
> > > > diff --git a/arch/arm/mach-mediatek/Kconfig b/arch/arm/mach-mediatek/Kconfig
> > > > index f73f588..f7e463c 100644
> > > > --- a/arch/arm/mach-mediatek/Kconfig
> > > > +++ b/arch/arm/mach-mediatek/Kconfig
> > > > @@ -1,6 +1,26 @@
> > > > -config ARCH_MEDIATEK
> > > > +menuconfig ARCH_MEDIATEK
> > > >         bool "Mediatek MT65xx & MT81xx SoC" if ARCH_MULTI_V7
> > > >         select ARM_GIC
> > > >         select MTK_TIMER
> > > >         help
> > > >           Support for Mediatek MT65xx & MT81xx SoCs
> > > > +
> > > > +if ARCH_MEDIATEK
> > > > +
> > > > +config MACH_MT6589
> > > > +       bool "MediaTek MT6589 SoCs support"
> > > > +       default ARCH_MEDIATEK
> > > > +
> > > > +config MACH_MT6592
> > > > +       bool "MediaTek MT6592 SoCs support"
> > > > +       default ARCH_MEDIATEK
> > > > +
> > > > +config MACH_MT8127
> > > > +       bool "MediaTek MT8127 SoCs support"
> > > > +       default ARCH_MEDIATEK
> > > > +
> > > > +config MACH_MT8135
> > > > +       bool "MediaTek MT8135 SoCs support"
> > > > +       default ARCH_MEDIATEK
> > > > +
> > > > +endif
> > 
> > None of these four new MACH_MT* Kconfig symbols are currently used in
> > linux-next. I assume that patches that actually use them (either as a
> > Kconfig symbol or as a CONFIG_* macro) are still pending. Is that
> > correct?
> 
> Yes, it is used in patch 3 in this series[1], and is discussed in [2].
> 
> Joe.C
> 
> [1]
> http://lists.infradead.org/pipermail/linux-arm-kernel/2015-January/318453.html

That only seems to add a user for MACH_MT8135, not for the other three
symbols. The other three symbols will be used too, won't they?

> [2]
> http://lists.infradead.org/pipermail/linux-arm-kernel/2015-January/315616.html

(For the record, that is a message regarding v4 of this patch.)


Paul Bolle


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

* Re: [PATCH v5 1/5] ARM: mediatek: Add config options for mediatek SoCs.
  2015-01-27 10:34         ` Paul Bolle
@ 2015-01-27 12:38           ` Matthias Brugger
  2015-01-27 12:47             ` Paul Bolle
  0 siblings, 1 reply; 18+ messages in thread
From: Matthias Brugger @ 2015-01-27 12:38 UTC (permalink / raw)
  To: Paul Bolle
  Cc: Yingjoe Chen, Valentin Rothberg, Hongzhou Yang, Rob Herring,
	Linus Walleij, Sascha Hauer, Pawel Moll, Mark Rutland,
	Ian Campbell, Kumar Gala, Russell King, Grant Likely,
	Catalin Marinas, Vladimir Murzin, Ashwin Chaugule, devicetree,
	linux-kernel, linux-arm-kernel, dandan.he, alan.cheng, toby.liu,
	maoguang.meng, huang eddie, Jean-Christophe PLAGNIOL-VILLARD,
	Howard Chen

2015-01-27 11:34 GMT+01:00 Paul Bolle <pebolle@tiscali.nl>:
> Hi Joe,
>
> On Tue, 2015-01-27 at 18:26 +0800, Yingjoe Chen wrote:
>> On Tue, 2015-01-27 at 11:16 +0100, Paul Bolle wrote:
>> > On Thu, 2015-01-22 at 13:54 +0100, Matthias Brugger wrote:
>> > > 2015-01-21 6:28 GMT+01:00 Hongzhou Yang <hongzhou.yang@mediatek.com>:
>> > > > From: Yingjoe Chen <yingjoe.chen@mediatek.com>
>> > > >
>> > > > The upcoming MTK pinctrl driver have a big pin table for each SoC
>> > > > and we don't want to bloat the kernel binary if we don't need it.
>> > > > Add config options so we can build for one SoC only.
>> > > >
>> > > > Acked-by: Linus Walleij <linus.walleij@linaro.org>
>> > > > Signed-off-by: Yingjoe Chen <yingjoe.chen@mediatek.com>
>> > > > Signed-off-by: Hongzhou Yang <hongzhou.yang@mediatek.com>
>> > >
>> > > Applied to v3.20-next/soc
>> >
>> > This became commit ad8a221e1f49 ("ARM: mediatek: Add config options for
>> > mediatek SoCs.") in today's linux-next (ie, next-20150127). I noticed
>> > because a script I use to check linux-next spotted a problem with it.
>> >
>> > > > ---
>> > > >  arch/arm/mach-mediatek/Kconfig | 22 +++++++++++++++++++++-
>> > > >  1 file changed, 21 insertions(+), 1 deletion(-)
>> > > >
>> > > > diff --git a/arch/arm/mach-mediatek/Kconfig b/arch/arm/mach-mediatek/Kconfig
>> > > > index f73f588..f7e463c 100644
>> > > > --- a/arch/arm/mach-mediatek/Kconfig
>> > > > +++ b/arch/arm/mach-mediatek/Kconfig
>> > > > @@ -1,6 +1,26 @@
>> > > > -config ARCH_MEDIATEK
>> > > > +menuconfig ARCH_MEDIATEK
>> > > >         bool "Mediatek MT65xx & MT81xx SoC" if ARCH_MULTI_V7
>> > > >         select ARM_GIC
>> > > >         select MTK_TIMER
>> > > >         help
>> > > >           Support for Mediatek MT65xx & MT81xx SoCs
>> > > > +
>> > > > +if ARCH_MEDIATEK
>> > > > +
>> > > > +config MACH_MT6589
>> > > > +       bool "MediaTek MT6589 SoCs support"
>> > > > +       default ARCH_MEDIATEK
>> > > > +
>> > > > +config MACH_MT6592
>> > > > +       bool "MediaTek MT6592 SoCs support"
>> > > > +       default ARCH_MEDIATEK
>> > > > +
>> > > > +config MACH_MT8127
>> > > > +       bool "MediaTek MT8127 SoCs support"
>> > > > +       default ARCH_MEDIATEK
>> > > > +
>> > > > +config MACH_MT8135
>> > > > +       bool "MediaTek MT8135 SoCs support"
>> > > > +       default ARCH_MEDIATEK
>> > > > +
>> > > > +endif
>> >
>> > None of these four new MACH_MT* Kconfig symbols are currently used in
>> > linux-next. I assume that patches that actually use them (either as a
>> > Kconfig symbol or as a CONFIG_* macro) are still pending. Is that
>> > correct?
>>
>> Yes, it is used in patch 3 in this series[1], and is discussed in [2].
>>
>> Joe.C
>>
>> [1]
>> http://lists.infradead.org/pipermail/linux-arm-kernel/2015-January/318453.html
>
> That only seems to add a user for MACH_MT8135, not for the other three
> symbols. The other three symbols will be used too, won't they?
>
>> [2]
>> http://lists.infradead.org/pipermail/linux-arm-kernel/2015-January/315616.html
>
> (For the record, that is a message regarding v4 of this patch.)
>
>
> Paul Bolle
>

Yes, this symbols are needed when we implement the pinctrl driver for
the other platforms.
I will implement mt6589 as soon as the pinctrl driver got merged.
I suppose Howard will go for the mt6592.

Cheers,
Matthias

-- 
motzblog.wordpress.com

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

* Re: [PATCH v5 1/5] ARM: mediatek: Add config options for mediatek SoCs.
  2015-01-27 12:38           ` Matthias Brugger
@ 2015-01-27 12:47             ` Paul Bolle
  0 siblings, 0 replies; 18+ messages in thread
From: Paul Bolle @ 2015-01-27 12:47 UTC (permalink / raw)
  To: Matthias Brugger
  Cc: Yingjoe Chen, Valentin Rothberg, Hongzhou Yang, Rob Herring,
	Linus Walleij, Sascha Hauer, Pawel Moll, Mark Rutland,
	Ian Campbell, Kumar Gala, Russell King, Grant Likely,
	Catalin Marinas, Vladimir Murzin, Ashwin Chaugule, devicetree,
	linux-kernel, linux-arm-kernel, dandan.he, alan.cheng, toby.liu,
	maoguang.meng, huang eddie, Jean-Christophe PLAGNIOL-VILLARD,
	Howard Chen

Matthias,

On Tue, 2015-01-27 at 13:38 +0100, Matthias Brugger wrote:
> Yes, this symbols are needed when we implement the pinctrl driver for
> the other platforms.
> I will implement mt6589 as soon as the pinctrl driver got merged.
> I suppose Howard will go for the mt6592.

I see. Unless the users of MACH_MT6589, MACH_MT6592, MACH_MT8127, and
MACH_MT8135 take very long to land in linux-next (or in mainline) I
won't bother you again about this.

Thanks!


Paul Bolle


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

* Re: [PATCH v5 2/5] dt-bindings: Add pinctrl bindings for mt65xx/mt81xx.
       [not found] ` <1421818097-9281-3-git-send-email-hongzhou.yang@mediatek.com>
@ 2015-01-27 14:19   ` Linus Walleij
  2015-01-28  7:49     ` Sascha Hauer
  2015-01-28  7:48   ` Sascha Hauer
  2015-02-10  8:01   ` Linus Walleij
  2 siblings, 1 reply; 18+ messages in thread
From: Linus Walleij @ 2015-01-27 14:19 UTC (permalink / raw)
  To: Hongzhou Yang, Sascha Hauer
  Cc: Rob Herring, Matthias Brugger, Pawel Moll, Mark Rutland,
	Ian Campbell, Kumar Gala, Russell King, Grant Likely, Joe.C,
	Catalin Marinas, Vladimir Murzin, Ashwin Chaugule, devicetree,
	linux-kernel, linux-arm-kernel, dandan.he, alan.cheng, toby.liu,
	maoguang.meng, huang eddie, Jean-Christophe PLAGNIOL-VILLARD

On Wed, Jan 21, 2015 at 6:28 AM, Hongzhou Yang
<hongzhou.yang@mediatek.com> wrote:

> From: Hongzhou Yang <hongzhou.yang@mediatek.com>
>
> Add devicetree bindings for Mediatek SoC pinctrl driver.
>
> Signed-off-by: Hongzhou Yang <hongzhou.yang@mediatek.com>

Sascha can you ACK this binding?
If you confirm it will cover also your usecase I'm gonna be happy
with this...

Yours,
Linus Walleij

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

* Re: [PATCH v5 2/5] dt-bindings: Add pinctrl bindings for mt65xx/mt81xx.
       [not found] ` <1421818097-9281-3-git-send-email-hongzhou.yang@mediatek.com>
  2015-01-27 14:19   ` [PATCH v5 2/5] dt-bindings: Add pinctrl bindings for mt65xx/mt81xx Linus Walleij
@ 2015-01-28  7:48   ` Sascha Hauer
  2015-02-10  8:01   ` Linus Walleij
  2 siblings, 0 replies; 18+ messages in thread
From: Sascha Hauer @ 2015-01-28  7:48 UTC (permalink / raw)
  To: Hongzhou Yang
  Cc: Rob Herring, Linus Walleij, Matthias Brugger, Mark Rutland,
	devicetree, Vladimir Murzin, Russell King, Pawel Moll,
	Ian Campbell, Catalin Marinas, eddie.huang, linux-kernel,
	alan.cheng, maoguang.meng, Ashwin Chaugule, toby.liu,
	Sascha Hauer, Kumar Gala, Grant Likely, Joe.C, dandan.he,
	Jean-Christophe PLAGNIOL-VILLARD, linux-arm-kernel

On Wed, Jan 21, 2015 at 01:28:14PM +0800, Hongzhou Yang wrote:
> From: Hongzhou Yang <hongzhou.yang@mediatek.com>
> 
> Add devicetree bindings for Mediatek SoC pinctrl driver.
> 
> Signed-off-by: Hongzhou Yang <hongzhou.yang@mediatek.com>

This basically follows the pinmux binding I suggested as generic pinmux
binding for per-pin type controllers, so:

Acked-by: Sascha Hauer <s.hauer@pengutronix.de>

Sascha


-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

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

* Re: [PATCH v5 2/5] dt-bindings: Add pinctrl bindings for mt65xx/mt81xx.
  2015-01-27 14:19   ` [PATCH v5 2/5] dt-bindings: Add pinctrl bindings for mt65xx/mt81xx Linus Walleij
@ 2015-01-28  7:49     ` Sascha Hauer
  0 siblings, 0 replies; 18+ messages in thread
From: Sascha Hauer @ 2015-01-28  7:49 UTC (permalink / raw)
  To: Linus Walleij
  Cc: Hongzhou Yang, Sascha Hauer, Rob Herring, Matthias Brugger,
	Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala, Russell King,
	Grant Likely, Joe.C, Catalin Marinas, Vladimir Murzin,
	Ashwin Chaugule, devicetree, linux-kernel, linux-arm-kernel,
	dandan.he, alan.cheng, toby.liu, maoguang.meng, huang eddie,
	Jean-Christophe PLAGNIOL-VILLARD

On Tue, Jan 27, 2015 at 03:19:01PM +0100, Linus Walleij wrote:
> On Wed, Jan 21, 2015 at 6:28 AM, Hongzhou Yang
> <hongzhou.yang@mediatek.com> wrote:
> 
> > From: Hongzhou Yang <hongzhou.yang@mediatek.com>
> >
> > Add devicetree bindings for Mediatek SoC pinctrl driver.
> >
> > Signed-off-by: Hongzhou Yang <hongzhou.yang@mediatek.com>
> 
> Sascha can you ACK this binding?

Just did that.

> If you confirm it will cover also your usecase I'm gonna be happy
> with this...

\o/

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

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

* Re: [PATCH v5 2/5] dt-bindings: Add pinctrl bindings for mt65xx/mt81xx.
       [not found] ` <1421818097-9281-3-git-send-email-hongzhou.yang@mediatek.com>
  2015-01-27 14:19   ` [PATCH v5 2/5] dt-bindings: Add pinctrl bindings for mt65xx/mt81xx Linus Walleij
  2015-01-28  7:48   ` Sascha Hauer
@ 2015-02-10  8:01   ` Linus Walleij
  2015-03-08  8:16     ` Matthias Brugger
  2 siblings, 1 reply; 18+ messages in thread
From: Linus Walleij @ 2015-02-10  8:01 UTC (permalink / raw)
  To: Hongzhou Yang
  Cc: Rob Herring, Matthias Brugger, Sascha Hauer, Pawel Moll,
	Mark Rutland, Ian Campbell, Kumar Gala, Russell King,
	Grant Likely, Joe.C, Catalin Marinas, Vladimir Murzin,
	Ashwin Chaugule, devicetree, linux-kernel, linux-arm-kernel,
	dandan.he, alan.cheng, toby.liu, maoguang.meng, huang eddie,
	Jean-Christophe PLAGNIOL-VILLARD

On Wed, Jan 21, 2015 at 1:28 PM, Hongzhou Yang
<hongzhou.yang@mediatek.com> wrote:

> From: Hongzhou Yang <hongzhou.yang@mediatek.com>
>
> Add devicetree bindings for Mediatek SoC pinctrl driver.
>
> Signed-off-by: Hongzhou Yang <hongzhou.yang@mediatek.com>

OK applied this patch for v3.21 now, relying on Sascha's ACK.

Yours,
Linus Walleij

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

* Re: [PATCH v5 5/5] ARM: dts: mt8135: Add pinctrl/GPIO/EINT node for mt8135.
       [not found] ` <1421818097-9281-6-git-send-email-hongzhou.yang@mediatek.com>
@ 2015-02-10  8:13   ` Linus Walleij
  2015-03-06 12:41     ` Matthias Brugger
  0 siblings, 1 reply; 18+ messages in thread
From: Linus Walleij @ 2015-02-10  8:13 UTC (permalink / raw)
  To: Hongzhou Yang
  Cc: Rob Herring, Matthias Brugger, Sascha Hauer, Pawel Moll,
	Mark Rutland, Ian Campbell, Kumar Gala, Russell King,
	Grant Likely, Joe.C, Catalin Marinas, Vladimir Murzin,
	Ashwin Chaugule, devicetree, linux-kernel, linux-arm-kernel,
	dandan.he, alan.cheng, toby.liu, maoguang.meng, huang eddie,
	Jean-Christophe PLAGNIOL-VILLARD

On Wed, Jan 21, 2015 at 1:28 PM, Hongzhou Yang
<hongzhou.yang@mediatek.com> wrote:

> From: Hongzhou Yang <hongzhou.yang@mediatek.com>
>
> Add pinctrl,GPIO and EINT node to mt8135.dtsi.
>
> Signed-off-by: Hongzhou Yang <hongzhou.yang@mediatek.com>

Acked-by: Linus Walleij <linus.walleij@linaro.org>

The pinctrl driver portions are merged to the pin control tree.

Yours,
Linus Walleij

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

* Re: [PATCH v5 3/5] ARM: mediatek: Add Pinctrl/GPIO driver for mt8135.
       [not found] ` <1421818097-9281-4-git-send-email-hongzhou.yang@mediatek.com>
@ 2015-02-10  8:17   ` Linus Walleij
       [not found]     ` <1423644176.1967.12.camel@mussux00.eus.mediatek.inc>
  0 siblings, 1 reply; 18+ messages in thread
From: Linus Walleij @ 2015-02-10  8:17 UTC (permalink / raw)
  To: Hongzhou Yang
  Cc: Rob Herring, Matthias Brugger, Sascha Hauer, Pawel Moll,
	Mark Rutland, Ian Campbell, Kumar Gala, Russell King,
	Grant Likely, Joe.C, Catalin Marinas, Vladimir Murzin,
	Ashwin Chaugule, devicetree, linux-kernel, linux-arm-kernel,
	dandan.he, alan.cheng, toby.liu, maoguang.meng, huang eddie,
	Jean-Christophe PLAGNIOL-VILLARD

On Wed, Jan 21, 2015 at 1:28 PM, Hongzhou Yang
<hongzhou.yang@mediatek.com> wrote:

> From: Hongzhou Yang <hongzhou.yang@mediatek.com>
>
> The mediatek SoCs have GPIO controller that handle both the muxing and GPIOs.
>
> The GPIO controller have pinmux, pull enable, pull select, direction and output high/low control.
>
> This driver include common driver and mt8135 part.
> The common driver include the pinctrl driver and GPIO driver.
> The mt8135 part contain its special device data.
>
> Signed-off-by: Hongzhou Yang <hongzhou.yang@mediatek.com>

I have merged this patch for v3.21, except:

>  arch/arm/mach-mediatek/Kconfig                |    1 +
(...)
 --- a/arch/arm/mach-mediatek/Kconfig
> +++ b/arch/arm/mach-mediatek/Kconfig
> @@ -1,6 +1,7 @@
>  menuconfig ARCH_MEDIATEK
>         bool "Mediatek MT65xx & MT81xx SoC" if ARCH_MULTI_V7
>         select ARM_GIC
> +       select PINCTRL
>         select MTK_TIMER
>         help
>           Support for Mediatek MT65xx & MT81xx SoCs

This, which should come in through the ARM SoC tree. It's not
dependent on this series anyway.

This will appear after the merge window as we start the v3.21 cycle.

Yours,
Linus Walleij

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

* Re: [PATCH v5 4/5] ARM: mediatek: Add EINT support to MTK pinctrl driver.
       [not found] ` <1421818097-9281-5-git-send-email-hongzhou.yang@mediatek.com>
@ 2015-02-10  8:17   ` Linus Walleij
  0 siblings, 0 replies; 18+ messages in thread
From: Linus Walleij @ 2015-02-10  8:17 UTC (permalink / raw)
  To: Hongzhou Yang
  Cc: Rob Herring, Matthias Brugger, Sascha Hauer, Pawel Moll,
	Mark Rutland, Ian Campbell, Kumar Gala, Russell King,
	Grant Likely, Joe.C, Catalin Marinas, Vladimir Murzin,
	Ashwin Chaugule, devicetree, linux-kernel, linux-arm-kernel,
	dandan.he, alan.cheng, toby.liu, maoguang.meng, huang eddie,
	Jean-Christophe PLAGNIOL-VILLARD

On Wed, Jan 21, 2015 at 1:28 PM, Hongzhou Yang
<hongzhou.yang@mediatek.com> wrote:

> From: Maoguang Meng <maoguang.meng@mediatek.com>
>
> MTK SoC support external interrupt(EINT) from most SoC pins.
> Add EINT support to pinctrl driver.
>
> Signed-off-by: Maoguang Meng <maoguang.meng@mediatek.com>
> Signed-off-by: Hongzhou Yang <hongzhou.yang@mediatek.com>

Patch applied.

Yours,
Linus Walleij

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

* Re: [PATCH v5 3/5] ARM: mediatek: Add Pinctrl/GPIO driver for mt8135.
       [not found]     ` <1423644176.1967.12.camel@mussux00.eus.mediatek.inc>
@ 2015-02-11 14:45       ` Linus Walleij
  0 siblings, 0 replies; 18+ messages in thread
From: Linus Walleij @ 2015-02-11 14:45 UTC (permalink / raw)
  To: Hongzhou Yang
  Cc: Rob Herring, Matthias Brugger, Sascha Hauer, Pawel Moll,
	Mark Rutland, Ian Campbell, Kumar Gala, Russell King,
	Grant Likely, Joe.C, Catalin Marinas, Vladimir Murzin,
	Ashwin Chaugule, devicetree, linux-kernel, linux-arm-kernel,
	dandan.he, alan.cheng, toby.liu, maoguang.meng, huang eddie,
	Jean-Christophe PLAGNIOL-VILLARD

On Wed, Feb 11, 2015 at 4:42 PM, Hongzhou Yang
<hongzhou.yang@mediatek.com> wrote:

> Due to pinconf relate API changed at kernel-3.20, a build error
> happened.
> Do I need to send patch v6?

I discussed with your colleagues and I think it's simplest just to make
a small fix patch on top of what is found on the mtk-staging branch
and I'll apply it there.

Yours,
Linus Walleij

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

* Re: [PATCH v5 5/5] ARM: dts: mt8135: Add pinctrl/GPIO/EINT node for mt8135.
  2015-02-10  8:13   ` [PATCH v5 5/5] ARM: dts: mt8135: Add pinctrl/GPIO/EINT node for mt8135 Linus Walleij
@ 2015-03-06 12:41     ` Matthias Brugger
  0 siblings, 0 replies; 18+ messages in thread
From: Matthias Brugger @ 2015-03-06 12:41 UTC (permalink / raw)
  To: Linus Walleij
  Cc: Hongzhou Yang, Rob Herring, Sascha Hauer, Pawel Moll,
	Mark Rutland, Ian Campbell, Kumar Gala, Russell King,
	Grant Likely, Joe.C, Catalin Marinas, Vladimir Murzin,
	Ashwin Chaugule, devicetree, linux-kernel, linux-arm-kernel,
	dandan.he, alan.cheng, toby.liu, maoguang.meng, huang eddie,
	Jean-Christophe PLAGNIOL-VILLARD

2015-02-10 9:13 GMT+01:00 Linus Walleij <linus.walleij@linaro.org>:
> On Wed, Jan 21, 2015 at 1:28 PM, Hongzhou Yang
> <hongzhou.yang@mediatek.com> wrote:
>
>> From: Hongzhou Yang <hongzhou.yang@mediatek.com>
>>
>> Add pinctrl,GPIO and EINT node to mt8135.dtsi.
>>
>> Signed-off-by: Hongzhou Yang <hongzhou.yang@mediatek.com>
>
> Acked-by: Linus Walleij <linus.walleij@linaro.org>

merged to v4.0-next/dts

>
> The pinctrl driver portions are merged to the pin control tree.
>
> Yours,
> Linus Walleij



-- 
motzblog.wordpress.com

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

* Re: [PATCH v5 2/5] dt-bindings: Add pinctrl bindings for mt65xx/mt81xx.
  2015-02-10  8:01   ` Linus Walleij
@ 2015-03-08  8:16     ` Matthias Brugger
  2015-03-09 17:37       ` Linus Walleij
  0 siblings, 1 reply; 18+ messages in thread
From: Matthias Brugger @ 2015-03-08  8:16 UTC (permalink / raw)
  To: Linus Walleij
  Cc: Hongzhou Yang, Rob Herring, Sascha Hauer, Pawel Moll,
	Mark Rutland, Ian Campbell, Kumar Gala, Russell King,
	Grant Likely, Joe.C, Catalin Marinas, Vladimir Murzin,
	Ashwin Chaugule, devicetree, linux-kernel, linux-arm-kernel,
	dandan.he, alan.cheng, toby.liu, maoguang.meng, huang eddie,
	Jean-Christophe PLAGNIOL-VILLARD

Hi Linus,

2015-02-10 9:01 GMT+01:00 Linus Walleij <linus.walleij@linaro.org>:
> On Wed, Jan 21, 2015 at 1:28 PM, Hongzhou Yang
> <hongzhou.yang@mediatek.com> wrote:
>
>> From: Hongzhou Yang <hongzhou.yang@mediatek.com>
>>
>> Add devicetree bindings for Mediatek SoC pinctrl driver.
>>
>> Signed-off-by: Hongzhou Yang <hongzhou.yang@mediatek.com>
>
> OK applied this patch for v3.21 now, relying on Sascha's ACK.

I can see the the pinctrl driver parts in your tree repository [0],
but not in linux-next.
Do think of merging them in the next merge window?

Thanks,
Matthias

[0] https://git.kernel.org/cgit/linux/kernel/git/linusw/linux-pinctrl.git/log/?h=mtk-staging
>
> Yours,
> Linus Walleij



-- 
motzblog.wordpress.com

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

* Re: [PATCH v5 2/5] dt-bindings: Add pinctrl bindings for mt65xx/mt81xx.
  2015-03-08  8:16     ` Matthias Brugger
@ 2015-03-09 17:37       ` Linus Walleij
  0 siblings, 0 replies; 18+ messages in thread
From: Linus Walleij @ 2015-03-09 17:37 UTC (permalink / raw)
  To: Matthias Brugger
  Cc: Hongzhou Yang, Rob Herring, Sascha Hauer, Pawel Moll,
	Mark Rutland, Ian Campbell, Kumar Gala, Russell King,
	Grant Likely, Joe.C, Catalin Marinas, Vladimir Murzin,
	Ashwin Chaugule, devicetree, linux-kernel, linux-arm-kernel,
	dandan.he, alan.cheng, toby.liu, maoguang.meng, huang eddie,
	Jean-Christophe PLAGNIOL-VILLARD

On Sun, Mar 8, 2015 at 9:16 AM, Matthias Brugger <matthias.bgg@gmail.com> wrote:
> Hi Linus,
>
> 2015-02-10 9:01 GMT+01:00 Linus Walleij <linus.walleij@linaro.org>:
>> On Wed, Jan 21, 2015 at 1:28 PM, Hongzhou Yang
>> <hongzhou.yang@mediatek.com> wrote:
>>
>>> From: Hongzhou Yang <hongzhou.yang@mediatek.com>
>>>
>>> Add devicetree bindings for Mediatek SoC pinctrl driver.
>>>
>>> Signed-off-by: Hongzhou Yang <hongzhou.yang@mediatek.com>
>>
>> OK applied this patch for v3.21 now, relying on Sascha's ACK.
>
> I can see the the pinctrl driver parts in your tree repository [0],
> but not in linux-next.
> Do think of merging them in the next merge window?

Oops too stressed just forgot to merge them into my devel
branch.

Merged now. Will push from devel to for-next when the
zeroday builders say it all compiles.

Yours,
Linus Walleij

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

end of thread, other threads:[~2015-03-09 17:37 UTC | newest]

Thread overview: 18+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <1421818097-9281-1-git-send-email-hongzhou.yang@mediatek.com>
2015-01-21  5:39 ` [PATCH v5 0/5] Add Mediatek SoC Pinctrl/GPIO/EINT driver for MT8135 Yingjoe Chen
     [not found] ` <1421818097-9281-2-git-send-email-hongzhou.yang@mediatek.com>
2015-01-22 12:54   ` [PATCH v5 1/5] ARM: mediatek: Add config options for mediatek SoCs Matthias Brugger
2015-01-27 10:16     ` Paul Bolle
2015-01-27 10:26       ` Yingjoe Chen
2015-01-27 10:34         ` Paul Bolle
2015-01-27 12:38           ` Matthias Brugger
2015-01-27 12:47             ` Paul Bolle
     [not found] ` <1421818097-9281-3-git-send-email-hongzhou.yang@mediatek.com>
2015-01-27 14:19   ` [PATCH v5 2/5] dt-bindings: Add pinctrl bindings for mt65xx/mt81xx Linus Walleij
2015-01-28  7:49     ` Sascha Hauer
2015-01-28  7:48   ` Sascha Hauer
2015-02-10  8:01   ` Linus Walleij
2015-03-08  8:16     ` Matthias Brugger
2015-03-09 17:37       ` Linus Walleij
     [not found] ` <1421818097-9281-6-git-send-email-hongzhou.yang@mediatek.com>
2015-02-10  8:13   ` [PATCH v5 5/5] ARM: dts: mt8135: Add pinctrl/GPIO/EINT node for mt8135 Linus Walleij
2015-03-06 12:41     ` Matthias Brugger
     [not found] ` <1421818097-9281-4-git-send-email-hongzhou.yang@mediatek.com>
2015-02-10  8:17   ` [PATCH v5 3/5] ARM: mediatek: Add Pinctrl/GPIO driver " Linus Walleij
     [not found]     ` <1423644176.1967.12.camel@mussux00.eus.mediatek.inc>
2015-02-11 14:45       ` Linus Walleij
     [not found] ` <1421818097-9281-5-git-send-email-hongzhou.yang@mediatek.com>
2015-02-10  8:17   ` [PATCH v5 4/5] ARM: mediatek: Add EINT support to MTK pinctrl driver Linus Walleij

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