All of lore.kernel.org
 help / color / mirror / Atom feed
From: lukma@denx.de (Lukasz Majewski)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/2] ARM: dts: Add the support for display5 equipped with Mitsubishi (800x480) display
Date: Fri, 17 Nov 2017 15:13:34 +0100	[thread overview]
Message-ID: <20171117151334.041c4684@jawa> (raw)
In-Reply-To: <20171117131047.GE25390@dragon>

Hi Shawn,

> On Fri, Nov 17, 2017 at 11:24:03AM +0100, Lukasz Majewski wrote:
> > This is the support for aa070mc01-ca1 Mitsubishi display.
> > 
> > Signed-off-by: Lukasz Majewski <lukma@denx.de>
> > ---
> >  arch/arm/boot/dts/Makefile                         |  1 +
> >  .../imx6q-display5-mitsubishi-aa070-800x480.dts    | 51
> > ++++++++++++++++++++++ 2 files changed, 52 insertions(+)
> >  create mode 100644
> > arch/arm/boot/dts/imx6q-display5-mitsubishi-aa070-800x480.dts
> > 
> > diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
> > index d9d0c82e7d20..10b3ad078dea 100644
> > --- a/arch/arm/boot/dts/Makefile
> > +++ b/arch/arm/boot/dts/Makefile
> > @@ -407,6 +407,7 @@ dtb-$(CONFIG_SOC_IMX6Q) += \
> >  	imx6q-cm-fx6.dtb \
> >  	imx6q-cubox-i.dtb \
> >  	imx6q-dfi-fs700-m60.dtb \
> > +	imx6q-display5-mitsubishi-aa070-800x480.dtb \
> >  	imx6q-display5-tianma-tm070-1280x768.dtb \
> >  	imx6q-dmo-edmqmx6.dtb \
> >  	imx6q-evi.dtb \  
> 
> <snip>
> 
> > +#include "imx6q-display5.dtsi"
> > +
> > +&panel {
> > +	compatible = "mitsubishi,aa070mc01-ca1";
> > +};  
> 
> How many DTB files do you want to add for this display5 board?  

This is a single board, which can support panels which I've posted.

Those dtbs I do pack to u-boot' FIT image and don't need to worry that
some display is not working (after setting bootloader env variable).

> It's
> obvious no fun to maintain a bunch of DTB files for all the panels
> this board can support.

Please note that those files excessively use #include directive, and
there is not so much code added (the longest part is license and
comments).

>  Let's stop right here.

If I may ask - how shall I proceed? What would be your suggestion to
move on (dtb overlays) ?


> 
> Shawn
> 
> > +
> > +&ldb {
> > +	lvds0: lvds-channel at 0 {
> > +		fsl,data-mapping = "spwg";
> > +		fsl,data-width = <24>;
> > +	};
> > +};
> > -- 
> > 2.11.0
> >   



Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20171117/a2bd84d1/attachment.sig>

WARNING: multiple messages have this Message-ID (diff)
From: Lukasz Majewski <lukma-ynQEQJNshbs@public.gmane.org>
To: Shawn Guo <shawnguo-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
Cc: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
	Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
	Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>,
	Russell King <linux-I+IVW8TIWO2tmTQ+vhA3Yw@public.gmane.org>,
	Sascha Hauer <kernel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>,
	Fabio Estevam <fabio.estevam-3arQi8VN3Tc@public.gmane.org>,
	devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Subject: Re: [PATCH 1/2] ARM: dts: Add the support for display5 equipped with Mitsubishi (800x480) display
Date: Fri, 17 Nov 2017 15:13:34 +0100	[thread overview]
Message-ID: <20171117151334.041c4684@jawa> (raw)
In-Reply-To: <20171117131047.GE25390@dragon>

[-- Attachment #1: Type: text/plain, Size: 2287 bytes --]

Hi Shawn,

> On Fri, Nov 17, 2017 at 11:24:03AM +0100, Lukasz Majewski wrote:
> > This is the support for aa070mc01-ca1 Mitsubishi display.
> > 
> > Signed-off-by: Lukasz Majewski <lukma-ynQEQJNshbs@public.gmane.org>
> > ---
> >  arch/arm/boot/dts/Makefile                         |  1 +
> >  .../imx6q-display5-mitsubishi-aa070-800x480.dts    | 51
> > ++++++++++++++++++++++ 2 files changed, 52 insertions(+)
> >  create mode 100644
> > arch/arm/boot/dts/imx6q-display5-mitsubishi-aa070-800x480.dts
> > 
> > diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
> > index d9d0c82e7d20..10b3ad078dea 100644
> > --- a/arch/arm/boot/dts/Makefile
> > +++ b/arch/arm/boot/dts/Makefile
> > @@ -407,6 +407,7 @@ dtb-$(CONFIG_SOC_IMX6Q) += \
> >  	imx6q-cm-fx6.dtb \
> >  	imx6q-cubox-i.dtb \
> >  	imx6q-dfi-fs700-m60.dtb \
> > +	imx6q-display5-mitsubishi-aa070-800x480.dtb \
> >  	imx6q-display5-tianma-tm070-1280x768.dtb \
> >  	imx6q-dmo-edmqmx6.dtb \
> >  	imx6q-evi.dtb \  
> 
> <snip>
> 
> > +#include "imx6q-display5.dtsi"
> > +
> > +&panel {
> > +	compatible = "mitsubishi,aa070mc01-ca1";
> > +};  
> 
> How many DTB files do you want to add for this display5 board?  

This is a single board, which can support panels which I've posted.

Those dtbs I do pack to u-boot' FIT image and don't need to worry that
some display is not working (after setting bootloader env variable).

> It's
> obvious no fun to maintain a bunch of DTB files for all the panels
> this board can support.

Please note that those files excessively use #include directive, and
there is not so much code added (the longest part is license and
comments).

>  Let's stop right here.

If I may ask - how shall I proceed? What would be your suggestion to
move on (dtb overlays) ?


> 
> Shawn
> 
> > +
> > +&ldb {
> > +	lvds0: lvds-channel@0 {
> > +		fsl,data-mapping = "spwg";
> > +		fsl,data-width = <24>;
> > +	};
> > +};
> > -- 
> > 2.11.0
> >   



Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd-ynQEQJNshbs@public.gmane.org

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

WARNING: multiple messages have this Message-ID (diff)
From: Lukasz Majewski <lukma@denx.de>
To: Shawn Guo <shawnguo@kernel.org>
Cc: Rob Herring <robh@kernel.org>, Rob Herring <robh+dt@kernel.org>,
	Mark Rutland <mark.rutland@arm.com>,
	Russell King <linux@armlinux.org.uk>,
	Sascha Hauer <kernel@pengutronix.de>,
	Fabio Estevam <fabio.estevam@nxp.com>,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 1/2] ARM: dts: Add the support for display5 equipped with Mitsubishi (800x480) display
Date: Fri, 17 Nov 2017 15:13:34 +0100	[thread overview]
Message-ID: <20171117151334.041c4684@jawa> (raw)
In-Reply-To: <20171117131047.GE25390@dragon>

[-- Attachment #1: Type: text/plain, Size: 2245 bytes --]

Hi Shawn,

> On Fri, Nov 17, 2017 at 11:24:03AM +0100, Lukasz Majewski wrote:
> > This is the support for aa070mc01-ca1 Mitsubishi display.
> > 
> > Signed-off-by: Lukasz Majewski <lukma@denx.de>
> > ---
> >  arch/arm/boot/dts/Makefile                         |  1 +
> >  .../imx6q-display5-mitsubishi-aa070-800x480.dts    | 51
> > ++++++++++++++++++++++ 2 files changed, 52 insertions(+)
> >  create mode 100644
> > arch/arm/boot/dts/imx6q-display5-mitsubishi-aa070-800x480.dts
> > 
> > diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
> > index d9d0c82e7d20..10b3ad078dea 100644
> > --- a/arch/arm/boot/dts/Makefile
> > +++ b/arch/arm/boot/dts/Makefile
> > @@ -407,6 +407,7 @@ dtb-$(CONFIG_SOC_IMX6Q) += \
> >  	imx6q-cm-fx6.dtb \
> >  	imx6q-cubox-i.dtb \
> >  	imx6q-dfi-fs700-m60.dtb \
> > +	imx6q-display5-mitsubishi-aa070-800x480.dtb \
> >  	imx6q-display5-tianma-tm070-1280x768.dtb \
> >  	imx6q-dmo-edmqmx6.dtb \
> >  	imx6q-evi.dtb \  
> 
> <snip>
> 
> > +#include "imx6q-display5.dtsi"
> > +
> > +&panel {
> > +	compatible = "mitsubishi,aa070mc01-ca1";
> > +};  
> 
> How many DTB files do you want to add for this display5 board?  

This is a single board, which can support panels which I've posted.

Those dtbs I do pack to u-boot' FIT image and don't need to worry that
some display is not working (after setting bootloader env variable).

> It's
> obvious no fun to maintain a bunch of DTB files for all the panels
> this board can support.

Please note that those files excessively use #include directive, and
there is not so much code added (the longest part is license and
comments).

>  Let's stop right here.

If I may ask - how shall I proceed? What would be your suggestion to
move on (dtb overlays) ?


> 
> Shawn
> 
> > +
> > +&ldb {
> > +	lvds0: lvds-channel@0 {
> > +		fsl,data-mapping = "spwg";
> > +		fsl,data-width = <24>;
> > +	};
> > +};
> > -- 
> > 2.11.0
> >   



Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd@denx.de

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

  reply	other threads:[~2017-11-17 14:13 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-11-17 10:24 [PATCH 1/2] ARM: dts: Add the support for display5 equipped with Mitsubishi (800x480) display Lukasz Majewski
2017-11-17 10:24 ` Lukasz Majewski
2017-11-17 10:24 ` Lukasz Majewski
2017-11-17 10:24 ` [PATCH 2/2] ARM: dts: Add the support for display5 equipped with Tianma " Lukasz Majewski
2017-11-17 10:24   ` Lukasz Majewski
2017-11-17 10:24   ` Lukasz Majewski
2017-11-17 13:10 ` [PATCH 1/2] ARM: dts: Add the support for display5 equipped with Mitsubishi " Shawn Guo
2017-11-17 13:10   ` Shawn Guo
2017-11-17 13:10   ` Shawn Guo
2017-11-17 14:13   ` Lukasz Majewski [this message]
2017-11-17 14:13     ` Lukasz Majewski
2017-11-17 14:13     ` Lukasz Majewski
2017-11-17 14:36     ` Lucas Stach
2017-11-17 14:36       ` Lucas Stach
2017-11-17 14:36       ` Lucas Stach
2017-11-17 15:03       ` Lukasz Majewski
2017-11-17 15:03         ` Lukasz Majewski
2017-11-17 15:03         ` Lukasz Majewski
2017-11-17 15:07         ` Lucas Stach
2017-11-17 15:07           ` Lucas Stach
2017-11-17 15:07           ` Lucas Stach
2017-11-17 15:27           ` Lukasz Majewski
2017-11-17 15:27             ` Lukasz Majewski
2017-11-17 15:27             ` Lukasz Majewski

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=20171117151334.041c4684@jawa \
    --to=lukma@denx.de \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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.