All of lore.kernel.org
 help / color / mirror / Atom feed
From: Chen-Yu Tsai <wens@csie.org>
To: Maxime Ripard <maxime.ripard@free-electrons.com>
Cc: Daniel Vetter <daniel.vetter@intel.com>,
	David Airlie <airlied@linux.ie>,
	Thierry Reding <thierry.reding@gmail.com>,
	Chen-Yu Tsai <wens@csie.org>, Rob Herring <robh+dt@kernel.org>,
	dri-devel <dri-devel@lists.freedesktop.org>,
	linux-arm-kernel <linux-arm-kernel@lists.infradead.org>,
	linux-kernel <linux-kernel@vger.kernel.org>,
	linux-sunxi <linux-sunxi@googlegroups.com>,
	Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Subject: Re: [linux-sunxi] [PATCH 6/7] ARM: sun8i: a33: Add RGB666 pins
Date: Fri, 2 Sep 2016 09:44:17 +0800	[thread overview]
Message-ID: <CAGb2v64kbkyS8eDtNrsok5UU9BTVr0mJBi_4bn1TnaxHmzfSTg@mail.gmail.com> (raw)
In-Reply-To: <20160901153204.11217-7-maxime.ripard@free-electrons.com>

Hi,

On Thu, Sep 1, 2016 at 11:32 PM, Maxime Ripard
<maxime.ripard@free-electrons.com> wrote:
> The LCD output needs to be muxed. Add the proper pinctrl node.
>
> Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com>
> ---
>  arch/arm/boot/dts/sun8i-a33.dtsi | 10 ++++++++++
>  1 file changed, 10 insertions(+)
>
> diff --git a/arch/arm/boot/dts/sun8i-a33.dtsi b/arch/arm/boot/dts/sun8i-a33.dtsi
> index 5f9dbd17eb50..d5f93c05846f 100644
> --- a/arch/arm/boot/dts/sun8i-a33.dtsi
> +++ b/arch/arm/boot/dts/sun8i-a33.dtsi
> @@ -300,6 +300,16 @@
>         interrupts = <GIC_SPI 15 IRQ_TYPE_LEVEL_HIGH>,
>                      <GIC_SPI 17 IRQ_TYPE_LEVEL_HIGH>;
>
> +       tcon0_rgb666_pins_a: tcon0_rgb666@0 {

This is the only possible combination. You can drop the _a and @0.

Also this can be shared with A23, as they are pin compatible, and I
also matched the datasheets.

Otherwise

Acked-by: Chen-Yu Tsai <wens@csie.org>

> +               allwinner,pins = "PD2", "PD3", "PD4", "PD5", "PD6", "PD7",
> +                                "PD10", "PD11", "PD12", "PD13", "PD14", "PD15",
> +                                "PD18", "PD19", "PD20", "PD21", "PD22", "PD23",
> +                                "PD24", "PD25", "PD26", "PD27";
> +               allwinner,function = "lcd0";
> +               allwinner,drive = <SUN4I_PINCTRL_10_MA>;
> +               allwinner,pull = <SUN4I_PINCTRL_NO_PULL>;
> +       };
> +
>         uart0_pins_b: uart0@1 {
>                 allwinner,pins = "PB0", "PB1";
>                 allwinner,function = "uart0";
> --
> 2.9.2
>
> --
> You received this message because you are subscribed to the Google Groups "linux-sunxi" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe@googlegroups.com.
> For more options, visit https://groups.google.com/d/optout.

WARNING: multiple messages have this Message-ID (diff)
From: wens@csie.org (Chen-Yu Tsai)
To: linux-arm-kernel@lists.infradead.org
Subject: [linux-sunxi] [PATCH 6/7] ARM: sun8i: a33: Add RGB666 pins
Date: Fri, 2 Sep 2016 09:44:17 +0800	[thread overview]
Message-ID: <CAGb2v64kbkyS8eDtNrsok5UU9BTVr0mJBi_4bn1TnaxHmzfSTg@mail.gmail.com> (raw)
In-Reply-To: <20160901153204.11217-7-maxime.ripard@free-electrons.com>

Hi,

On Thu, Sep 1, 2016 at 11:32 PM, Maxime Ripard
<maxime.ripard@free-electrons.com> wrote:
> The LCD output needs to be muxed. Add the proper pinctrl node.
>
> Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com>
> ---
>  arch/arm/boot/dts/sun8i-a33.dtsi | 10 ++++++++++
>  1 file changed, 10 insertions(+)
>
> diff --git a/arch/arm/boot/dts/sun8i-a33.dtsi b/arch/arm/boot/dts/sun8i-a33.dtsi
> index 5f9dbd17eb50..d5f93c05846f 100644
> --- a/arch/arm/boot/dts/sun8i-a33.dtsi
> +++ b/arch/arm/boot/dts/sun8i-a33.dtsi
> @@ -300,6 +300,16 @@
>         interrupts = <GIC_SPI 15 IRQ_TYPE_LEVEL_HIGH>,
>                      <GIC_SPI 17 IRQ_TYPE_LEVEL_HIGH>;
>
> +       tcon0_rgb666_pins_a: tcon0_rgb666 at 0 {

This is the only possible combination. You can drop the _a and @0.

Also this can be shared with A23, as they are pin compatible, and I
also matched the datasheets.

Otherwise

Acked-by: Chen-Yu Tsai <wens@csie.org>

> +               allwinner,pins = "PD2", "PD3", "PD4", "PD5", "PD6", "PD7",
> +                                "PD10", "PD11", "PD12", "PD13", "PD14", "PD15",
> +                                "PD18", "PD19", "PD20", "PD21", "PD22", "PD23",
> +                                "PD24", "PD25", "PD26", "PD27";
> +               allwinner,function = "lcd0";
> +               allwinner,drive = <SUN4I_PINCTRL_10_MA>;
> +               allwinner,pull = <SUN4I_PINCTRL_NO_PULL>;
> +       };
> +
>         uart0_pins_b: uart0 at 1 {
>                 allwinner,pins = "PB0", "PB1";
>                 allwinner,function = "uart0";
> --
> 2.9.2
>
> --
> You received this message because you are subscribed to the Google Groups "linux-sunxi" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe at googlegroups.com.
> For more options, visit https://groups.google.com/d/optout.

WARNING: multiple messages have this Message-ID (diff)
From: Chen-Yu Tsai <wens-jdAy2FN1RRM@public.gmane.org>
To: Maxime Ripard
	<maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
Cc: Daniel Vetter
	<daniel.vetter-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>,
	David Airlie <airlied-cv59FeDIM0c@public.gmane.org>,
	Thierry Reding
	<thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	Chen-Yu Tsai <wens-jdAy2FN1RRM@public.gmane.org>,
	Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
	dri-devel
	<dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org>,
	linux-arm-kernel
	<linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>,
	linux-kernel
	<linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	linux-sunxi <linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org>,
	Thomas Petazzoni
	<thomas.petazzoni-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
Subject: Re: [PATCH 6/7] ARM: sun8i: a33: Add RGB666 pins
Date: Fri, 2 Sep 2016 09:44:17 +0800	[thread overview]
Message-ID: <CAGb2v64kbkyS8eDtNrsok5UU9BTVr0mJBi_4bn1TnaxHmzfSTg@mail.gmail.com> (raw)
In-Reply-To: <20160901153204.11217-7-maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>

Hi,

On Thu, Sep 1, 2016 at 11:32 PM, Maxime Ripard
<maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> wrote:
> The LCD output needs to be muxed. Add the proper pinctrl node.
>
> Signed-off-by: Maxime Ripard <maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
> ---
>  arch/arm/boot/dts/sun8i-a33.dtsi | 10 ++++++++++
>  1 file changed, 10 insertions(+)
>
> diff --git a/arch/arm/boot/dts/sun8i-a33.dtsi b/arch/arm/boot/dts/sun8i-a33.dtsi
> index 5f9dbd17eb50..d5f93c05846f 100644
> --- a/arch/arm/boot/dts/sun8i-a33.dtsi
> +++ b/arch/arm/boot/dts/sun8i-a33.dtsi
> @@ -300,6 +300,16 @@
>         interrupts = <GIC_SPI 15 IRQ_TYPE_LEVEL_HIGH>,
>                      <GIC_SPI 17 IRQ_TYPE_LEVEL_HIGH>;
>
> +       tcon0_rgb666_pins_a: tcon0_rgb666@0 {

This is the only possible combination. You can drop the _a and @0.

Also this can be shared with A23, as they are pin compatible, and I
also matched the datasheets.

Otherwise

Acked-by: Chen-Yu Tsai <wens-jdAy2FN1RRM@public.gmane.org>

> +               allwinner,pins = "PD2", "PD3", "PD4", "PD5", "PD6", "PD7",
> +                                "PD10", "PD11", "PD12", "PD13", "PD14", "PD15",
> +                                "PD18", "PD19", "PD20", "PD21", "PD22", "PD23",
> +                                "PD24", "PD25", "PD26", "PD27";
> +               allwinner,function = "lcd0";
> +               allwinner,drive = <SUN4I_PINCTRL_10_MA>;
> +               allwinner,pull = <SUN4I_PINCTRL_NO_PULL>;
> +       };
> +
>         uart0_pins_b: uart0@1 {
>                 allwinner,pins = "PB0", "PB1";
>                 allwinner,function = "uart0";
> --
> 2.9.2
>
> --
> You received this message because you are subscribed to the Google Groups "linux-sunxi" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org
> For more options, visit https://groups.google.com/d/optout.

  reply	other threads:[~2016-09-02  1:44 UTC|newest]

Thread overview: 96+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-09-01 15:31 [PATCH 0/7] drm/sun4i: Introduce A33 display driver Maxime Ripard
2016-09-01 15:31 ` Maxime Ripard
2016-09-01 15:31 ` Maxime Ripard
2016-09-01 15:31 ` [PATCH 1/7] drm/sun4i: support TCONs without channel 1 Maxime Ripard
2016-09-01 15:31   ` Maxime Ripard
2016-09-01 15:31   ` Maxime Ripard
2016-09-02  1:47   ` Chen-Yu Tsai
2016-09-02  1:47     ` Chen-Yu Tsai
2016-09-02  1:47     ` Chen-Yu Tsai
2016-09-01 15:31 ` [PATCH 2/7] drm/sun4i: support A33 tcon Maxime Ripard
2016-09-01 15:31   ` Maxime Ripard
2016-09-01 15:31   ` Maxime Ripard
2016-09-02  6:02   ` Chen-Yu Tsai
2016-09-02  6:02     ` Chen-Yu Tsai
2016-09-02  6:02     ` Chen-Yu Tsai
2016-09-05 20:22     ` Maxime Ripard
2016-09-05 20:22       ` Maxime Ripard
2016-09-05 20:22       ` Maxime Ripard
2016-09-01 15:32 ` [PATCH 3/7] drm/sun4i: Add SAT and DRC drivers Maxime Ripard
2016-09-01 15:32   ` Maxime Ripard
2016-09-01 15:32   ` Maxime Ripard
2016-09-02  6:45   ` Chen-Yu Tsai
2016-09-02  6:45     ` Chen-Yu Tsai
2016-09-02  6:45     ` Chen-Yu Tsai
2016-09-05 20:27     ` Maxime Ripard
2016-09-05 20:27       ` Maxime Ripard
2016-09-05 20:27       ` Maxime Ripard
2016-09-04 20:03   ` [linux-sunxi] " Peter Korsgaard
2016-09-04 20:03     ` Peter Korsgaard
2016-09-04 20:03     ` [linux-sunxi] " Peter Korsgaard
2016-09-06 13:59     ` Maxime Ripard
2016-09-06 13:59       ` Maxime Ripard
2016-09-06 13:59       ` [linux-sunxi] " Maxime Ripard
2016-09-01 15:32 ` [PATCH 4/7] drm/panel: Add Sinlinx SinA33 7" panel Maxime Ripard
2016-09-01 15:32   ` Maxime Ripard
2016-09-01 15:32   ` Maxime Ripard
2016-09-02  7:01   ` Chen-Yu Tsai
2016-09-02  7:01     ` Chen-Yu Tsai
2016-09-04 17:03   ` Icenowy Zheng
2016-09-04 17:03     ` Icenowy Zheng
2016-09-05 20:02     ` Maxime Ripard
2016-09-05 20:02       ` Maxime Ripard
2016-09-05 20:02       ` Maxime Ripard
2016-09-06  2:53       ` Chen-Yu Tsai
2016-09-06  2:53         ` Chen-Yu Tsai
2016-09-06  2:53         ` Chen-Yu Tsai
2016-09-06  9:12       ` Thierry Reding
2016-09-06  9:12         ` Thierry Reding
2016-09-06  9:12         ` Thierry Reding
2016-09-06 14:33         ` Maxime Ripard
2016-09-06 14:33           ` Maxime Ripard
2016-09-06 14:33           ` Maxime Ripard
2016-09-01 15:32 ` [PATCH 5/7] ARM: sun8i: a33: Add display pipeline Maxime Ripard
2016-09-01 15:32   ` Maxime Ripard
2016-09-01 15:32   ` Maxime Ripard
2016-09-02  6:28   ` Chen-Yu Tsai
2016-09-02  6:28     ` Chen-Yu Tsai
2016-09-02  6:28     ` Chen-Yu Tsai
2016-09-05 20:21     ` Maxime Ripard
2016-09-05 20:21       ` Maxime Ripard
2016-09-05 20:21       ` Maxime Ripard
2016-09-06  2:51       ` Chen-Yu Tsai
2016-09-06  2:51         ` Chen-Yu Tsai
2016-09-06  2:51         ` Chen-Yu Tsai
2016-09-01 15:32 ` [PATCH 6/7] ARM: sun8i: a33: Add RGB666 pins Maxime Ripard
2016-09-01 15:32   ` Maxime Ripard
2016-09-01 15:32   ` Maxime Ripard
2016-09-02  1:44   ` Chen-Yu Tsai [this message]
2016-09-02  1:44     ` Chen-Yu Tsai
2016-09-02  1:44     ` [linux-sunxi] " Chen-Yu Tsai
2016-09-01 15:32 ` [PATCH 7/7] ARM: sun8i: sina33: Enable display Maxime Ripard
2016-09-01 15:32   ` Maxime Ripard
2016-09-01 15:32   ` Maxime Ripard
2016-09-02  1:30 ` [PATCH 0/7] drm/sun4i: Introduce A33 display driver Icenowy Zheng
2016-09-02  1:30   ` Icenowy Zheng
2016-09-02 19:06   ` Maxime Ripard
2016-09-02 19:06     ` Maxime Ripard
2016-09-02 19:06     ` Maxime Ripard
2016-09-03  1:43     ` Chen-Yu Tsai
2016-09-03  1:43       ` Chen-Yu Tsai
2016-09-03  1:43       ` Chen-Yu Tsai
2016-09-05 20:37       ` Maxime Ripard
2016-09-05 20:37         ` Maxime Ripard
2016-09-05 20:37         ` Maxime Ripard
2016-09-06  2:50         ` Chen-Yu Tsai
2016-09-06  2:50           ` Chen-Yu Tsai
2016-09-06  2:50           ` Chen-Yu Tsai
2016-09-06 18:54           ` Maxime Ripard
2016-09-06 18:54             ` Maxime Ripard
2016-09-06 18:54             ` Maxime Ripard
2016-09-07  4:49             ` Chen-Yu Tsai
2016-09-07  4:49               ` Chen-Yu Tsai
2016-09-07  4:49               ` Chen-Yu Tsai
2016-09-12  9:56               ` Maxime Ripard
2016-09-12  9:56                 ` Maxime Ripard
2016-09-12  9:56                 ` Maxime Ripard

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAGb2v64kbkyS8eDtNrsok5UU9BTVr0mJBi_4bn1TnaxHmzfSTg@mail.gmail.com \
    --to=wens@csie.org \
    --cc=airlied@linux.ie \
    --cc=daniel.vetter@intel.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-sunxi@googlegroups.com \
    --cc=maxime.ripard@free-electrons.com \
    --cc=robh+dt@kernel.org \
    --cc=thierry.reding@gmail.com \
    --cc=thomas.petazzoni@free-electrons.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.