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

[Linus Walleij]

On Tue, Feb 27, 2018 at 1:56 PM, Ville Syrjala
<ville.syrjala at linux.intel.com> 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>
> Signed-off-by: Ville Syrjälä <ville.syrjala at linux.intel.com>

Acked-by: Linus Walleij <linus.walleij at linaro.org>

Yours,
Linus Walleij