From: maxime.ripard@bootlin.com (Maxime Ripard)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2 6/9] drm/panel: Add Netron DY E231732
Date: Thu, 3 May 2018 15:09:58 +0200 [thread overview]
Message-ID: <20180503130958.gh6mgbhon3uaha75@flea> (raw)
In-Reply-To: <89cac24ddcb5f4ac710a697a4433c927e5c7e71e.camel@paulk.fr>
Hi,
On Wed, May 02, 2018 at 09:20:30AM +0200, Paul Kocialkowski wrote:
> Le vendredi 09 septembre 2016 ? 16:35 +0200, Maxime Ripard a ?crit :
> > On Wed, Sep 07, 2016 at 12:01:56AM +0800, Chen-Yu Tsai wrote:
> > > Hi,
> > >
> > > On Tue, Sep 6, 2016 at 10:46 PM, Maxime Ripard
> > > <maxime.ripard@free-electrons.com> wrote:
> > > > The E231732 is a 7" panel with a resolution of 800x480.
> > >
> > > From what I could make out of an archived version of Netron's
> > > website
> > > (it's unreachable from my place), they are a manufacturer of printed
> > > ribbon cables, not LCD panels. This is probably a no-go.
> >
> > I don't know. I haven't been able to find any website for Netron DY,
> > however, googling the part number I used find numerous matches on ebay
> > and alibaba.
>
> While starting a port for the A13-based TZX-Q8-713B7 tablet, it appears
> that my LCD cable also has the Netron-DY E231732 marking, but is
> definitely different from the one on the SinA33. So I think Chen-Yu's
> comment was founded and "Netron DY E231732" does not reflect the panel
> name.
>
> Here are some details regarding the LCD cable on both devices:
>
> The SinA33 has a 1024x600 panel with the following markings:
> 7300101463
> E231732
> NETRON-DY <> 1
> 94V-0\1310
>
> The TZX-Q8-713B7 has a 800x480 panel with the following markings:
> 7300101448
> E231732
> NETRON-DY <> 2
> 94V-0\1249
>
> So it seems that the distinctive part of the name is
> 7300101463/7300101448.
>
> Also, it seems that various online shops list the panels with a name
> like "E231732 7300101463". So maybe we could change the bindings to
> have:
>
> - netron-dy,e231732-7300101463
> - netron-dy,e231732-7300101448
>
> What do you think? Does that make sense?
>
> We should probably also fallback netron-dy,e231732 to the sina33's panel
> in order to avoid breaking the ABI.
That's kind of unfortunate, and maybe the netron part is just the
vendor of the flex cable.
In your case, I guess the best work around this would be to call it
TZX-Q8-713B7-panel, if it's shipped with a single resolution.
Maxime
--
Maxime Ripard, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20180503/5652e833/attachment-0001.sig>
next prev parent reply other threads:[~2018-05-03 13:09 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-09-06 14:46 [PATCH v2 0/9] drm/sun4i: Introduce A33 display driver Maxime Ripard
2016-09-06 14:46 ` [PATCH v2 1/9] drm/sun4i: support TCONs without channel 1 Maxime Ripard
2016-09-06 14:46 ` [PATCH v2 2/9] drm/sun4i: support A33 tcon Maxime Ripard
2016-09-06 15:07 ` Chen-Yu Tsai
2016-09-06 15:16 ` Chen-Yu Tsai
2016-09-08 7:29 ` Maxime Ripard
2016-11-05 15:54 ` Chen-Yu Tsai
2016-11-06 18:50 ` Maxime Ripard
2016-09-06 14:46 ` [PATCH v2 3/9] drm/sun4i: backend: Handle the SAT Maxime Ripard
2016-09-06 15:13 ` Chen-Yu Tsai
2016-09-06 14:46 ` [PATCH v2 4/9] drm/sun4i: Add a DRC driver Maxime Ripard
2016-09-06 15:10 ` Chen-Yu Tsai
2016-09-06 20:38 ` [linux-sunxi] " Peter Korsgaard
2016-09-06 14:46 ` [PATCH v2 5/9] of: Add vendor prefix for Netron DY Maxime Ripard
2016-10-19 12:39 ` Thierry Reding
2016-10-19 13:03 ` Rob Herring
2017-01-26 9:53 ` Thierry Reding
2016-09-06 14:46 ` [PATCH v2 6/9] drm/panel: Add Netron DY E231732 Maxime Ripard
2016-09-06 16:01 ` Chen-Yu Tsai
2016-09-09 14:35 ` Maxime Ripard
2018-05-02 7:20 ` Paul Kocialkowski
2018-05-03 13:09 ` Maxime Ripard [this message]
2017-01-26 9:54 ` Thierry Reding
2017-01-26 10:51 ` Maxime Ripard
2016-09-06 14:46 ` [PATCH v2 7/9] ARM: sun8i: a33: Add display pipeline Maxime Ripard
2016-09-06 15:24 ` Chen-Yu Tsai
2016-09-06 14:46 ` [PATCH v2 8/9] ARM: sun8i: a23/a33: Add RGB666 pins Maxime Ripard
2016-09-06 14:46 ` [PATCH v2 9/9] ARM: sun8i: sina33: Enable display Maxime Ripard
2017-01-26 10:53 ` Maxime Ripard
2016-09-08 8:12 ` [PATCH v2 0/9] drm/sun4i: Introduce A33 display driver Maxime Ripard
2016-09-08 20:19 ` 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=20180503130958.gh6mgbhon3uaha75@flea \
--to=maxime.ripard@bootlin.com \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox