From: Maarten ter Huurne <maarten@treewalker.org>
To: Paul Cercueil <paul@crapouillou.net>, Sam Ravnborg <sam@ravnborg.org>
Cc: "Thierry Reding" <thierry.reding@gmail.com>,
"David Airlie" <airlied@linux.ie>,
"Daniel Vetter" <daniel@ffwll.ch>,
"Rob Herring" <robh+dt@kernel.org>,
"Andrzej Hajda" <a.hajda@samsung.com>,
"Neil Armstrong" <narmstrong@baylibre.com>,
"Laurent Pinchart" <Laurent.pinchart@ideasonboard.com>,
"Jonas Karlman" <jonas@kwiboo.se>,
"Jernej Skrabec" <jernej.skrabec@siol.net>,
"Maarten Lankhorst" <maarten.lankhorst@linux.intel.com>,
"Maxime Ripard" <mripard@kernel.org>,
"Thomas Zimmermann" <tzimmermann@suse.de>,
"Noralf Trønnes" <noralf@tronnes.org>,
od@zcrc.me, dri-devel@lists.freedesktop.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/6] dt-bindings: display: Document NewVision NV3052C DT node
Date: Mon, 27 Jul 2020 21:24:52 +0200 [thread overview]
Message-ID: <2191202.OZe27ScSiE@hyperion> (raw)
In-Reply-To: <20200727191052.GA1011220@ravnborg.org>
On Monday, 27 July 2020 21:10:52 CEST Sam Ravnborg wrote:
> > +description: |
> > + This is a driver for 320x240 TFT panels,
>
> The binding describes the HW, not the driver. So please re-phrase this
> part.
>
> This datasheet:
> https://www.phoenixdisplay.com/wp-content/uploads/2019/05/NV3052C-Dat
> asheet-V0.2.pdf tells that the driver supports additional resoltions.
> I guess the 320x240 resolution is limited to the leadtek panel.
The word "driver" is overloaded ;)
I guess "driver IC" would make it clearer.
Bye,
Maarten
WARNING: multiple messages have this Message-ID (diff)
From: Maarten ter Huurne <maarten@treewalker.org>
To: Paul Cercueil <paul@crapouillou.net>, Sam Ravnborg <sam@ravnborg.org>
Cc: devicetree@vger.kernel.org,
Jernej Skrabec <jernej.skrabec@siol.net>,
od@zcrc.me, Thomas Zimmermann <tzimmermann@suse.de>,
Neil Armstrong <narmstrong@baylibre.com>,
David Airlie <airlied@linux.ie>,
dri-devel@lists.freedesktop.org, Jonas Karlman <jonas@kwiboo.se>,
linux-kernel@vger.kernel.org, Andrzej Hajda <a.hajda@samsung.com>,
Rob Herring <robh+dt@kernel.org>,
Thierry Reding <thierry.reding@gmail.com>,
Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
Subject: Re: [PATCH 1/6] dt-bindings: display: Document NewVision NV3052C DT node
Date: Mon, 27 Jul 2020 21:24:52 +0200 [thread overview]
Message-ID: <2191202.OZe27ScSiE@hyperion> (raw)
In-Reply-To: <20200727191052.GA1011220@ravnborg.org>
On Monday, 27 July 2020 21:10:52 CEST Sam Ravnborg wrote:
> > +description: |
> > + This is a driver for 320x240 TFT panels,
>
> The binding describes the HW, not the driver. So please re-phrase this
> part.
>
> This datasheet:
> https://www.phoenixdisplay.com/wp-content/uploads/2019/05/NV3052C-Dat
> asheet-V0.2.pdf tells that the driver supports additional resoltions.
> I guess the 320x240 resolution is limited to the leadtek panel.
The word "driver" is overloaded ;)
I guess "driver IC" would make it clearer.
Bye,
Maarten
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2020-07-27 19:25 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-07-27 16:46 [PATCH 0/6] DBI/DSI, panel drivers, and tinyDRM compat Paul Cercueil
2020-07-27 16:46 ` Paul Cercueil
2020-07-27 16:46 ` [PATCH 1/6] dt-bindings: display: Document NewVision NV3052C DT node Paul Cercueil
2020-07-27 16:46 ` Paul Cercueil
2020-07-27 19:10 ` Sam Ravnborg
2020-07-27 19:10 ` Sam Ravnborg
2020-07-27 19:24 ` Maarten ter Huurne [this message]
2020-07-27 19:24 ` Maarten ter Huurne
2020-07-29 17:10 ` Laurent Pinchart
2020-07-29 17:10 ` Laurent Pinchart
2020-07-27 16:46 ` [PATCH 2/6] drm: dsi: Let host and device specify supported bus Paul Cercueil
2020-07-27 16:46 ` Paul Cercueil
2020-07-27 17:02 ` Laurent Pinchart
2020-07-27 17:02 ` Laurent Pinchart
2020-07-27 17:59 ` Paul Cercueil
2020-07-27 17:59 ` Paul Cercueil
2020-07-27 20:03 ` Sam Ravnborg
2020-07-27 20:03 ` Sam Ravnborg
2020-07-27 16:46 ` [PATCH 3/6] drm/bridge: Add SPI DBI host driver Paul Cercueil
2020-07-27 16:46 ` Paul Cercueil
2020-07-27 17:06 ` Laurent Pinchart
2020-07-27 17:06 ` Laurent Pinchart
2020-07-27 17:54 ` Paul Cercueil
2020-07-27 17:54 ` Paul Cercueil
2020-07-27 20:31 ` Sam Ravnborg
2020-07-27 20:31 ` Sam Ravnborg
2020-07-27 21:10 ` Paul Cercueil
2020-07-27 21:10 ` Paul Cercueil
2020-07-27 16:46 ` [PATCH 4/6] drm/panel: Add panel driver for NewVision NV3052C based LCDs Paul Cercueil
2020-07-27 16:46 ` Paul Cercueil
2020-07-30 19:50 ` Sam Ravnborg
2020-07-30 19:50 ` Sam Ravnborg
2020-07-27 16:46 ` [PATCH 5/6] drm/tiny: Add TinyDRM for DSI/DBI panels Paul Cercueil
2020-07-27 16:46 ` Paul Cercueil
2020-07-27 16:46 ` [PATCH 6/6] drm/panel: Add Ilitek ILI9341 DBI panel driver Paul Cercueil
2020-07-27 16:46 ` Paul Cercueil
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=2191202.OZe27ScSiE@hyperion \
--to=maarten@treewalker.org \
--cc=Laurent.pinchart@ideasonboard.com \
--cc=a.hajda@samsung.com \
--cc=airlied@linux.ie \
--cc=daniel@ffwll.ch \
--cc=devicetree@vger.kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=jernej.skrabec@siol.net \
--cc=jonas@kwiboo.se \
--cc=linux-kernel@vger.kernel.org \
--cc=maarten.lankhorst@linux.intel.com \
--cc=mripard@kernel.org \
--cc=narmstrong@baylibre.com \
--cc=noralf@tronnes.org \
--cc=od@zcrc.me \
--cc=paul@crapouillou.net \
--cc=robh+dt@kernel.org \
--cc=sam@ravnborg.org \
--cc=thierry.reding@gmail.com \
--cc=tzimmermann@suse.de \
/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.