From: Maxime Ripard <maxime.ripard@free-electrons.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [linux-sunxi] Re: [RFC PATCH 0/3] sunxi: video: Add support for HDMI output on H3
Date: Tue, 20 Dec 2016 11:47:21 +0100 [thread overview]
Message-ID: <20161220104721.skz3v23id4nvmprl@lukather> (raw)
In-Reply-To: <A60DF1B9-FA37-4318-B033-1E9264987D66@suse.de>
Hi,
On Wed, Dec 14, 2016 at 11:37:38AM +0200, Alexander Graf wrote:
>
>
> > Am 14.12.2016 um 11:25 schrieb Maxime Ripard <maxime.ripard@free-electrons.com>:
> >
> >> On Wed, Dec 14, 2016 at 11:02:33AM +0800, Chen-Yu Tsai wrote:
> >>> On Wed, Dec 14, 2016 at 5:16 AM, Icenowy Zheng <icenowy@aosc.xyz> wrote:
> >>>
> >>>
> >>> 14.12.2016, 04:29, "Simon Glass" <sjg@chromium.org>:
> >>>> Hi,
> >>>>
> >>>>> On 12 December 2016 at 19:36, Jernej Skrabec <jernej.skrabec@siol.net> wrote:
> >>>>> This patch series add support for HDMI output. Support for other,
> >>>>> newer, SoCs, which also uses DE2 and same or similar HDMI controller
> >>>>> and PHY can be easily added later (A83T/A64/H5/R40). Current driver
> >>>>> can also be easily extended with TV out support, just like video
> >>>>> driver for older Allwinner SoCs.
> >>>>>
> >>>>> While driver works, I would like to get few opinions first.
> >>>>> - From what I understand, drivers which supports DT are preferred.
> >>>>> Would it be better to rewrite this driver to support device tree?
> >>>>
> >>>> Yes I think so, and in fact it should use driver model also.
> >>>>
> >>>> The rockchip driver provides a reasonable example of how to split the
> >>>> driver up as you suggest below. The VIDEO driver provides the
> >>>> top-level video interface, DISPLAY drivers provide display output for
> >>>> the video, and you have PANEL as well for receiving the display
> >>>> output. VIDCONSOLE works automatically to display text.
> >>>>
> >>>> I actually took a bit of a look at this a few weeks ago so am happy to
> >>>> help with review or discussions.
> >>>
> >>> Yes, I think go to Driver Model is also valuable, as Allwinner's display parts
> >>> are really in reusable components.
> >>>
> >>> In addition, Driver Model provides the possibility to use EFI GOP, which can
> >>> be a more standardized replacement of SimpleFB, especially for OSes other
> >>> than Linux. (For example, Windows, if we can really get it running ;-) )
> >>
> >> We're going to use EFI now? Doesn't EFI mean we have to leave another bit
> >> of firmware in memory running under the OS? What about PSCI and ATF on
> >> ARMv8?
> >
> > Until EFI on A64 on U-boot is a real thing, I don't really care to be
> > honest.
>
> How is it not a real thing today?
Sorry, I missed it...
But still, the point remains. If it needs the DT and the device model
to be enabled and working properly, since that's not a feature that
worked before (on the previous SoC), I don't see why we should hold it
off. If and when someone is interested, and if and when we have a DT
bindings in Linux, then we will take care of this.
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161220/88d5449f/attachment.sig>
next prev parent reply other threads:[~2016-12-20 10:47 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-12-13 0:36 [U-Boot] [RFC PATCH 0/3] sunxi: video: Add support for HDMI output on H3 Jernej Skrabec
2016-12-13 0:36 ` [U-Boot] [RFC PATCH 1/3] sunxi: Add clocks for DE2/HDMI/TCON Jernej Skrabec
2016-12-13 15:18 ` Maxime Ripard
2016-12-13 19:20 ` Jernej Škrabec
2016-12-14 2:39 ` Chen-Yu Tsai
2016-12-13 0:36 ` [U-Boot] [RFC PATCH 2/3] sunxi: video: Add video driver for H3 SoC Jernej Skrabec
2016-12-13 15:40 ` Maxime Ripard
2016-12-13 16:33 ` Chen-Yu Tsai
2016-12-13 20:13 ` Jernej Škrabec
2016-12-14 10:28 ` Maxime Ripard
2016-12-14 22:11 ` Jernej Škrabec
2016-12-19 13:28 ` Maxime Ripard
2017-02-05 22:23 ` Rask Ingemann Lambertsen
2016-12-13 0:36 ` [U-Boot] [RFC PATCH 3/3] sunxi: Enable H3 video driver in defconfigs Jernej Skrabec
2016-12-13 20:28 ` [U-Boot] [RFC PATCH 0/3] sunxi: video: Add support for HDMI output on H3 Simon Glass
2016-12-13 21:16 ` [U-Boot] [linux-sunxi] " Icenowy Zheng
2016-12-14 3:02 ` Chen-Yu Tsai
2016-12-14 9:25 ` Maxime Ripard
2016-12-14 9:37 ` Alexander Graf
2016-12-14 9:51 ` Emmanuel Vadot
2016-12-20 10:47 ` Maxime Ripard [this message]
2016-12-14 8:37 ` [U-Boot] " Hans de Goede
2016-12-14 9:24 ` Maxime Ripard
2016-12-17 22:47 ` Simon Glass
2017-02-05 22:14 ` Rask Ingemann Lambertsen
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=20161220104721.skz3v23id4nvmprl@lukather \
--to=maxime.ripard@free-electrons.com \
--cc=u-boot@lists.denx.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox