From: Maxime Ripard <maxime.ripard@free-electrons.com>
To: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Cc: dri-devel@lists.freedesktop.org,
linux-renesas-soc@vger.kernel.org,
Archit Taneja <architt@codeaurora.org>
Subject: Re: [PATCH v3 07/13] drm: bridge: vga-dac: Add adi,adv7123 compatible string
Date: Tue, 29 Nov 2016 10:50:34 +0100 [thread overview]
Message-ID: <20161129095034.slso5rellb4cj2za@lukather> (raw)
In-Reply-To: <1480410283-28698-8-git-send-email-laurent.pinchart+renesas@ideasonboard.com>
[-- Attachment #1: Type: text/plain, Size: 908 bytes --]
On Tue, Nov 29, 2016 at 11:04:37AM +0200, Laurent Pinchart wrote:
> The ADV7123 is a transparent VGA DAC. Unlike dumb VGA DACs it can be
> controlled through a power save pin, and requires a power supply.
> However, on most boards where the device is used neither the power save
> signal nor the power supply are controllable.
>
> To avoid developing a separate device-specific driver add an
> "adi,adv7123" compatible entry to the dumb-vga-dac driver. This will
> allow supporting most ADV7123-based boards easily, while allowing future
> development of an adv7123 driver when needed without breaking backward
> compatibility.
>
> Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Acked-by: Maxime Ripard <maxime.ripard@free-electrons.com>
Thanks!
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]
next prev parent reply other threads:[~2016-11-29 9:50 UTC|newest]
Thread overview: 58+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-11-29 9:04 [PATCH v3 00/13] R-Car DU: Use drm bridge API Laurent Pinchart
2016-11-29 9:04 ` [PATCH v3 01/13] drm: Don't include <drm/drm_encoder.h> in <drm/drm_crtc.h> Laurent Pinchart
2016-11-29 9:30 ` Daniel Vetter
2016-11-29 9:37 ` Laurent Pinchart
2016-12-02 21:21 ` Sinclair Yeh
2016-11-29 9:04 ` [PATCH v3 02/13] drm: Fix compilation warning caused by static inline forward declaration Laurent Pinchart
2016-11-29 9:31 ` Daniel Vetter
2016-11-29 9:04 ` [PATCH v3 03/13] drm: bridge: Link encoder and bridge in core code Laurent Pinchart
2016-11-29 9:35 ` Daniel Vetter
2016-11-29 9:43 ` Laurent Pinchart
2016-11-29 10:05 ` Daniel Vetter
2016-11-29 18:02 ` Laurent Pinchart
2016-11-29 18:51 ` Laurent Pinchart
2016-11-29 10:27 ` Archit Taneja
2016-11-29 17:57 ` Laurent Pinchart
2016-11-30 5:05 ` Archit Taneja
2016-11-30 10:23 ` Laurent Pinchart
2016-11-30 11:00 ` Archit Taneja
2016-11-30 11:05 ` Laurent Pinchart
2016-11-30 13:27 ` Archit Taneja
2016-11-29 17:01 ` Stefan Agner
2016-11-29 19:58 ` Boris Brezillon
2016-11-30 15:30 ` Vincent ABRIOU
2016-11-29 9:04 ` [PATCH v3 04/13] drm: bridge: Detach bridge from encoder at encoder cleanup time Laurent Pinchart
2016-11-29 9:48 ` Daniel Vetter
2016-11-29 19:00 ` Laurent Pinchart
2016-11-29 10:34 ` Archit Taneja
2016-11-29 18:56 ` Laurent Pinchart
2016-11-29 20:22 ` Daniel Vetter
2016-11-29 9:04 ` [PATCH v3 05/13] drm: bridge: Add LVDS encoder DT bindings Laurent Pinchart
2016-11-29 9:04 ` [PATCH v3 06/13] drm: bridge: Add LVDS encoder driver Laurent Pinchart
2016-11-29 9:54 ` Daniel Vetter
2016-11-29 20:57 ` Laurent Pinchart
2017-01-04 1:33 ` Laurent Pinchart
2017-01-04 8:18 ` Daniel Vetter
2017-01-04 13:08 ` Laurent Pinchart
2017-01-04 13:51 ` Daniel Vetter
2017-01-04 14:33 ` Laurent Pinchart
2017-01-04 14:58 ` Daniel Vetter
2017-01-04 15:13 ` Laurent Pinchart
2017-03-02 0:30 ` Laurent Pinchart
2017-03-02 7:05 ` Daniel Vetter
2016-11-29 9:04 ` [PATCH v3 07/13] drm: bridge: vga-dac: Add adi,adv7123 compatible string Laurent Pinchart
2016-11-29 9:50 ` Maxime Ripard [this message]
2016-11-29 9:04 ` [PATCH v3 08/13] drm: bridge: lvds-encoder: Add thine,thc63lvdm83d " Laurent Pinchart
2016-11-29 9:04 ` [PATCH v3 09/13] drm: Add encoder_type field to the drm_bridge structure Laurent Pinchart
2016-11-29 9:56 ` Daniel Vetter
2016-11-29 9:58 ` Laurent Pinchart
2016-11-29 10:27 ` Daniel Vetter
2016-11-29 17:49 ` Laurent Pinchart
2016-11-29 20:25 ` Daniel Vetter
2016-11-29 22:42 ` Laurent Pinchart
2016-11-29 9:04 ` [PATCH v3 10/13] drm: bridge: Set bridges' encoder type Laurent Pinchart
2016-11-29 9:04 ` [PATCH v3 11/13] drm: Set on-chip " Laurent Pinchart
2016-11-30 15:28 ` Vincent ABRIOU
2016-11-29 9:04 ` [PATCH v3 12/13] drm: rcar-du: Replace manual bridge implementation with DRM bridge Laurent Pinchart
2016-12-27 12:40 ` Geert Uytterhoeven
2016-11-29 9:04 ` [PATCH v3 13/13] drm: rcar-du: Initialize encoder's type based on the bridge's type Laurent Pinchart
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=20161129095034.slso5rellb4cj2za@lukather \
--to=maxime.ripard@free-electrons.com \
--cc=architt@codeaurora.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=laurent.pinchart+renesas@ideasonboard.com \
--cc=linux-renesas-soc@vger.kernel.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