[RFC][PATCH 04/11] drm: Split the display info into static and dynamic parts

Maxime Ripard maxime.ripard at bootlin.com
Tue Feb 27 13:08:58 UTC 2018


On Tue, Feb 27, 2018 at 02:56:53PM +0200, Ville Syrjala wrote:
> From: Ville Syrjälä <ville.syrjala at linux.intel.com>
> 
> Currently we have a mix of static and dynamic information stored in
> the display info structure. That makes it rather difficult to repopulate
> the dynamic parts when a new EDID appears. Let's make life easier by
> splitting the structure up into static and dynamic parts.
> 
> The static part will consist of subpixel_order, panel_orientation,
> and bus_formats.
> 
> Actually I'm not sure where bus_formats & co. fit in all this. For some
> drivers those seem to be static, even though they might fill them out
> from .get_modes(). For other drivers this stuff even gets frobbed at
> runtime, making it more some kind of a bastard encoder/connector state.
> I'll just stick it into the static side so that the behaviour doesn't
> change when I start clear out the entire dynamic state with memset().
> 
> Cc: Keith Packard <keithp at keithp.com>
> Cc: Daniel Vetter <daniel.vetter at ffwll.ch>
> Cc: Hans de Goede <hdegoede at redhat.com>
> Cc: Shashank Sharma <shashank.sharma at intel.com>
> Cc: Stefan Agner <stefan at agner.ch>
> Cc: Thierry Reding <thierry.reding at gmail.com>
> Cc: Boris Brezillon <boris.brezillon at bootlin.com>
> Cc: Philipp Zabel <p.zabel at pengutronix.de>
> Cc: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> Cc: Manfred Schlaegl <manfred.schlaegl at gmx.at>
> Cc: Marek Vasut <marex at denx.de>
> Cc: Archit Taneja <architt at codeaurora.org>
> Cc: Andrzej Hajda <a.hajda at samsung.com>
> Cc: Alison Wang <alison.wang at freescale.com>
> Cc: Eric Anholt <eric at anholt.net>
> Cc: Linus Walleij <linus.walleij at linaro.org>
> Cc: linux-renesas-soc at vger.kernel.org
> Cc: Maxime Ripard <maxime.ripard at bootlin.com>

For sun4i,
Acked-by: Maxime Ripard <maxime.ripard at bootlin.com>

Thanks!
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: <https://lists.freedesktop.org/archives/dri-devel/attachments/20180227/6c648992/attachment.sig>


More information about the dri-devel mailing list