[PATCH v1 06/26] drm/panel: decouple connector from drm_panel

[Linus Walleij]

On Mon, Dec 2, 2019 at 8:33 PM Sam Ravnborg <sam at ravnborg.org> wrote:

> To facilitate that connector creation will be moved
> to display drivers, decouple the drm_connector from drm_panel.
> Add a connector argument to drm_panel_get_modes()
>
> All users of drm_panel_get_modes() already had the connector
> available, so updating users was trivial.
>
> With this patch drm_panel no longer keep a copy of the drm_connector.
>
> Signed-off-by: Sam Ravnborg <sam at ravnborg.org>
> Cc: Thierry Reding <thierry.reding at gmail.com>
> Cc: Laurent Pinchart <Laurent.pinchart at ideasonboard.com>
> Cc: Sam Ravnborg <sam at ravnborg.org>
> Cc: Andrzej Hajda <a.hajda at samsung.com>
> Cc: Neil Armstrong <narmstrong at baylibre.com>
> Cc: Jonas Karlman <jonas at kwiboo.se>
> Cc: Jernej Skrabec <jernej.skrabec at siol.net>
> Cc: Maarten Lankhorst <maarten.lankhorst at linux.intel.com>
> Cc: Maxime Ripard <mripard at kernel.org>
> Cc: David Airlie <airlied at linux.ie>
> Cc: Daniel Vetter <daniel at ffwll.ch>
> Cc: Inki Dae <inki.dae at samsung.com>
> Cc: Joonyoung Shim <jy0922.shim at samsung.com>
> Cc: Seung-Woo Kim <sw0312.kim at samsung.com>
> Cc: Kyungmin Park <kyungmin.park at samsung.com>
> Cc: Kukjin Kim <kgene at kernel.org>
> Cc: Krzysztof Kozlowski <krzk at kernel.org>
> Cc: Stefan Agner <stefan at agner.ch>
> Cc: Alison Wang <alison.wang at nxp.com>
> Cc: Philipp Zabel <p.zabel at pengutronix.de>
> Cc: Shawn Guo <shawnguo at kernel.org>
> Cc: Sascha Hauer <s.hauer at pengutronix.de>
> Cc: Pengutronix Kernel Team <kernel at pengutronix.de>
> Cc: Fabio Estevam <festevam at gmail.com>
> Cc: NXP Linux Team <linux-imx at nxp.com>
> Cc: CK Hu <ck.hu at mediatek.com>
> Cc: Matthias Brugger <matthias.bgg at gmail.com>
> Cc: Marek Vasut <marex at denx.de>
> Cc: Tomi Valkeinen <tomi.valkeinen at ti.com>
> Cc: Kieran Bingham <kieran.bingham+renesas at ideasonboard.com>
> Cc: Sandy Huang <hjc at rock-chips.com>
> Cc: "Heiko Stübner" <heiko at sntech.de>
> Cc: Benjamin Gaignard <benjamin.gaignard at linaro.org>
> Cc: Vincent Abriou <vincent.abriou at st.com>
> Cc: Chen-Yu Tsai <wens at csie.org>
> Cc: Jonathan Hunter <jonathanh at nvidia.com>
> Cc: Torsten Duwe <duwe at lst.de>
> Cc: Vasily Khoruzhick <anarsoul at gmail.com>
> Cc: Icenowy Zheng <icenowy at aosc.io>
> Cc: Sean Paul <seanpaul at chromium.org>
> Cc: Linus Walleij <linus.walleij at linaro.org>
> Cc: Boris Brezillon <boris.brezillon at collabora.com>
> Cc: Hariprasad Kelam <hariprasad.kelam at gmail.com>
> Cc: Alexios Zavras <alexios.zavras at intel.com>
> Cc: Brian Masney <masneyb at onstation.org>
> Cc: Rob Clark <robdclark at chromium.org>
> Cc: Thomas Gleixner <tglx at linutronix.de>
> Cc: Allison Randal <allison at lohutok.net>
> Cc: Shayenne Moura <shayenneluzmoura at gmail.com>
> Cc: Abhinav Kumar <abhinavk at codeaurora.org>
> Cc: linux-arm-kernel at lists.infradead.org
> Cc: linux-samsung-soc at vger.kernel.org
> Cc: linux-mediatek at lists.infradead.org
> Cc: linux-renesas-soc at vger.kernel.org
> Cc: linux-rockchip at lists.infradead.org
> Cc: linux-tegra at vger.kernel.org

This is a good refactoring. It makes it easier to understand
where things belong.

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

Yours,
Linus Walleij