[PATCH] drm/dp_mst: Avoid NULL pointer dereference

[Why] Found kernel NULL pointer dereference under the below situation: src — HDMI_Monitor src — HDMI_Monitor e.g.: \=> MSTB — MSTB (unplug) MSTB — MSTB When display 1 HDMI and 2 DP daisy chain monitors, unplugging the dp cable connected to source

[PATCH 1/2] drm/panel: Add DT bindings for Novatek NT35510-based panels

This adds device tree bindings for the Novatek NT35510-based family of panels. Since several such panels are in existence we define bindings common for all, and define the compatible string for one certain panel (Hydis HVA40WV1). As other panels are discovered and investigated, we can add more

[PATCH 2/2] drm/panel: Add driver for Novatek NT35510-based panels

This adds a driver for panels based on the Novatek NT35510 display driver IC, such as the Hydis HVA40WV1 panel found in the Samsung GT-S7710. The NT35510 can be used with both internal and external backlight (such as GPIO backlight) so we support both: if no external backlight is found, we

[PATCH v3 32/35] drm/mediatek: hdmi: Use drm_bridge_init()

No functional change. v3: - drop driver_private argument (Laurent) Acked-by: CK Hu Reviewed-by: Laurent Pinchart Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/mediatek/mtk_hdmi.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH v2 11/11] arm64: dts: rockchip: Add PX30 LVDS

Describe LVDS IP. Add the CRTC and LVDS relevant endpoints so they can be linked together. Signed-off-by: Miquel Raynal --- arch/arm64/boot/dts/rockchip/px30.dtsi | 37 ++ 1 file changed, 37 insertions(+) diff --git a/arch/arm64/boot/dts/rockchip/px30.dtsi

[PATCH v3 21/35] drm/bridge: sil_sii8620: Use drm_bridge_init()

No functional change. v3: - drop driver_private argument (Laurent) Reviewed-by: Laurent Pinchart Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/bridge/sil-sii8620.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/bridge/sil-sii8620.c

[PATCH v3 08/35] drm/bridge: Drop driver_private from struct drm_bridge

It's no longer used, all drivers have been converted to an embedded drm_bridge struct. Signed-off-by: Mihail Atanassov --- include/drm/drm_bridge.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/include/drm/drm_bridge.h b/include/drm/drm_bridge.h index 695d0898df38..ee175a2f95e6 100644

[PATCH v2 01/11] dt-bindings: display: rockchip-lvds: Declare PX30 compatible

Document the PX30 LVDS compatible. Signed-off-by: Miquel Raynal Acked-by: Rob Herring --- .../devicetree/bindings/display/rockchip/rockchip-lvds.txt | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bindings/display/rockchip/rockchip-lvds.txt

[PATCH v3 26/35] drm/bridge: thc63: Use drm_bridge_init()

No functional change. v3: - drop driver_private argument (Laurent) Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/bridge/thc63lvd1024.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/bridge/thc63lvd1024.c b/drivers/gpu/drm/bridge/thc63lvd1024.c

[PATCH v3 33/35] drm: rcar-du: lvds: Use drm_bridge_init()

No functional change. v3: - drop driver_private argument (Laurent) Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/rcar-du/rcar_lvds.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/rcar-du/rcar_lvds.c b/drivers/gpu/drm/rcar-du/rcar_lvds.c index

[PATCH v3 28/35] drm/bridge: ti-tfp410: Use drm_bridge_init()

No functional change. v3: - drop driver_private argument (Laurent) Reviewed-by: Laurent Pinchart Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/bridge/ti-tfp410.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/gpu/drm/bridge/ti-tfp410.c

[PATCH v3 25/35] drm/bridge: tc358767: Use drm_bridge_init()

No functional change. v3: - drop driver_private argument (Laurent) Reviewed-by: Laurent Pinchart Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/bridge/tc358767.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/bridge/tc358767.c

[PATCH v3 35/35] drm/msm: Use drm_bridge_init()

No functional change: drm_bridge_init() sets bridge->of_node, but that's not used by msm anywhere, and the bridges aren't published with drm_bridge_add() for it to matter. v3: - drop driver_private argument (Laurent) Reviewed-by: Laurent Pinchart Signed-off-by: Mihail Atanassov ---

[PATCH v3 17/35] drm/bridge: panel: Use drm_bridge_init()

No functional change. v3: - drop driver_private argument (Laurent) Reviewed-by: Laurent Pinchart Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/bridge/panel.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/bridge/panel.c

[PATCH v3 20/35] drm/bridge: sii9234: Use drm_bridge_init()

No functional change. v3: - drop driver_private argument (Laurent) - update commit title prefix (Neil) Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/bridge/sii9234.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/bridge/sii9234.c

[PATCH v2 03/11] drm/rockchip: lvds: Fix indentation of a #define

Fix a #define indentation before adding more lines. Fixes: 34cc0aa25456 ("drm/rockchip: Add support for Rockchip Soc LVDS") Signed-off-by: Miquel Raynal --- drivers/gpu/drm/rockchip/rockchip_lvds.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH v3 12/35] drm/bridge: cdns: Use drm_bridge_init()

No functional change. v3: - drop driver_private argument (Laurent) Reviewed-by: Laurent Pinchart Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/bridge/cdns-dsi.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/bridge/cdns-dsi.c

[PATCH v3 01/35] drm: Remove drm_bridge->dev

As suggested in [1], the 'dev' field is a bit repetitive, since it 1:1 follows the setting and NULLing of the 'encoder' field. Therefore, use drm_bridge->encoder->dev in place of drm_bridge->dev. [1] https://patchwork.freedesktop.org/patch/343824/ v3: - resend as part of drm_bridge_init()

[PATCH v3 29/35] drm/exynos: mic: Use drm_bridge_init()

No functional change. v3: - drop driver_private argument (Laurent) Cc: Inki Dae Cc: Laurent Pinchart Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/exynos/exynos_drm_mic.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/gpu/drm/exynos/exynos_drm_mic.c

[PATCH v3 16/35] drm/bridge: nxp-ptn3460: Use drm_bridge_init()

No functional change. v3: - drop driver_private argument (Laurent) Reviewed-by: Laurent Pinchart Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/bridge/nxp-ptn3460.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/bridge/nxp-ptn3460.c

[PATCH v3 09/35] drm: Introduce drm_bridge_init()

A simple convenience function to initialize the struct drm_bridge. The goal is to standardize initialization for any bridge registered with drm_bridge_add() so that we can later add device links for consumers of those bridges. v3: - drop driver_private parameter (Laurent) - spelling & style

[PATCH v2 07/11] drm/rockchip: lvds: Helpers should return decent values

Return errors instead of returning void from internal helpers. When these helpers are called, check the returned value and print an error message in this case. Signed-off-by: Miquel Raynal --- drivers/gpu/drm/rockchip/rockchip_lvds.c | 31 ++-- 1 file changed, 23

[PATCH v3 22/35] drm/bridge: dw-hdmi: Use drm_bridge_init()

No functional change. v3: - drop driver_private argument (Laurent) Reviewed-by: Laurent Pinchart Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/bridge/synopsys/dw-hdmi.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/gpu/drm/bridge/synopsys/dw-hdmi.c

[PATCH v2 00/11] Add PX30 LVDS support

Hello, This series aims at supporting LVDS on PX30. A first couple of patches update the documentation with the new compatible and the presence of a PHY. Then, the existing Rockchip driver is cleaned and extended to support PX30 specificities. Finally, the PX30 DTSI is updated with CRTC routes,

[PATCH v3 06/35] drm: rcar-du: lvds: Don't set drm_bridge->driver_private

It's unused. Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/rcar-du/rcar_lvds.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/rcar-du/rcar_lvds.c b/drivers/gpu/drm/rcar-du/rcar_lvds.c index 83538125a722..1d6dfae8a200 100644 --- a/drivers/gpu/drm/rcar-du/rcar_lvds.c

[PATCH v3 23/35] drm/bridge/synopsys: dsi: Use drm_bridge_init()

No functional change. v3: - drop driver_private argument (Laurent) Reviewed-by: Laurent Pinchart Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git

[PATCH v3 05/35] drm/exynos: mic: Stop using drm_bridge->driver_private

Use a container_of wrapper instead. Note that no logic depends on driver_private being NULL or not, so the cleanup in exynos_mic_bind is safe to apply. Cc: Inki Dae Signed-off-by: Mihail Atanassov --- Reviewer note: the driver_private cleanup used to be part of the relevant drm_bridge_init()

[PATCH v3 11/35] drm/bridge/analogix: Use drm_bridge_init()

No functional change. The setting of bridge->of_node by drm_bridge_init() in analogix_dp_core.c is safe, since ->of_node isn't used directly and the bridge isn't published with drm_bridge_add(). v3: - drop driver_private argument (Laurent) - pass correct struct device pointer to

[PATCH v3 02/35] drm/bridge: analogix_dp: Stop using drm_bridge->driver_private

Instead, embed the drm_bridge structure into analogix_dp_core and use a container_of wrapper to access the latter. Signed-off-by: Mihail Atanassov --- .../drm/bridge/analogix/analogix_dp_core.c| 40 +++ .../drm/bridge/analogix/analogix_dp_core.h| 8 +++- 2 files

[PATCH v3 13/35] drm/bridge: dumb-vga-dac: Use drm_bridge_init()

No functional change. v3: - drop driver_private argument (Laurent) Reviewed-by: Laurent Pinchart Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/bridge/dumb-vga-dac.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/bridge/dumb-vga-dac.c

Re: KASAN: use-after-free Read in fb_mode_is_equal

syzbot has found a reproducer for the following crash on: HEAD commit:46cf053e Linux 5.5-rc3 git tree: upstream console output: https://syzkaller.appspot.com/x/log.txt?x=15cf54e1e0 kernel config: https://syzkaller.appspot.com/x/.config?x=ed9d672709340e35 dashboard link:

[PATCH v3 00/35] drm/bridge: Consolidate initialization

Hi all, Version 3 of my drm_bridge_init() series here. The biggest change from the previous version is the removal of drm_bridge->dev (1/35) and the removal of ->driver_private (2-7/35). The rest of the patches are mostly the same, save for dropping the driver_private argument from

[PATCH v3 10/35] drm/bridge: adv7511: Use drm_bridge_init()

No functional change. v3: - drop driver_private argument (Laurent) Reviewed-by: Laurent Pinchart Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/bridge/adv7511/adv7511_drv.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git

[PATCH v3 27/35] drm/bridge: ti-sn65dsi86: Use drm_bridge_init()

No functional change. v3: - drop driver_private argument (Laurent) Reviewed-by: Laurent Pinchart Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/bridge/ti-sn65dsi86.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/bridge/ti-sn65dsi86.c

[PATCH v3 31/35] drm/mcde: dsi: Use drm_bridge_init()

No functional change. v3: - drop driver_private argument (Laurent) Acked-by: Linus Walleij Reviewed-by: Laurent Pinchart Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/mcde/mcde_dsi.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/mcde/mcde_dsi.c

[PATCH v3 RESEND 01/35] drm: Remove drm_bridge->dev

As suggested in [1], the 'dev' field is a bit repetitive, since it 1:1 follows the setting and NULLing of the 'encoder' field. Therefore, use drm_bridge->encoder->dev in place of drm_bridge->dev. [1] https://patchwork.freedesktop.org/patch/343824/ v3: - resend as part of drm_bridge_init()

[PATCH v3 19/35] drm/bridge: sii902x: Use drm_bridge_init()

No functional change. v3: - drop driver_private argument (Laurent) Reviewed-by: Laurent Pinchart Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/bridge/sii902x.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/bridge/sii902x.c

[PATCH v2 10/11] arm64: dts: rockchip: Add PX30 DSI DPHY

Add the PHY which outputs MIPI DSI and LVDS. Signed-off-by: Miquel Raynal --- arch/arm64/boot/dts/rockchip/px30.dtsi | 11 +++ 1 file changed, 11 insertions(+) diff --git a/arch/arm64/boot/dts/rockchip/px30.dtsi b/arch/arm64/boot/dts/rockchip/px30.dtsi index b2af0f02ecbe..672a3a2f42b9

[PATCH v2 05/11] drm/rockchip: lvds: Change platform data

Prepare the introduction of PX30 support by using drm_encoder_helper_funcs as platform data instead of multiple register names which are specific to rk3288 and not generic to all Rockchip IPs. This way adding support for a new flavor of a similar IP will be a matter of adding the relevant helper

[PATCH v2 02/11] dt-bindings: display: rockchip-lvds: Document PX30 PHY

PX30 SoCs use a single PHY shared by two display pipelines: MIPI DSI and LVDS. In the case of the LVDS IP, document the possibility to fill a PHY handle. Signed-off-by: Miquel Raynal --- .../devicetree/bindings/display/rockchip/rockchip-lvds.txt | 3 +++ 1 file changed, 3 insertions(+)

[PATCH v2 04/11] drm/rockchip: lvds: Harmonize function names

Prepare the introduction of PX30 support by clarifying the function prefixes. We continue to prefix with 'rockchip_lvds_' generic functions that are not specific to a single hardware. Functions implying hardware modifications are now prefixed with 'rk3288_lvds_'. Signed-off-by: Miquel Raynal

[PATCH v3 24/35] drm/bridge: tc358764: Use drm_bridge_init()

No functional change. v3: - drop driver_private argument (Laurent) Reviewed-by: Laurent Pinchart Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/bridge/tc358764.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/gpu/drm/bridge/tc358764.c

[PATCH v3 14/35] drm/bridge: lvds-codec: Use drm_bridge_init()

No functional change. v3: - drop driver_private argument (Laurent) Reviewed-by: Laurent Pinchart Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/bridge/lvds-codec.c | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/bridge/lvds-codec.c

[PATCH v3 07/35] drm/sti: Stop using drm_bridge->driver_private

Instead, embed the drm_bridge structure in the originally-pointed-to struct and use a container_of wrapper to access it. Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/sti/sti_dvo.c | 20 drivers/gpu/drm/sti/sti_hda.c | 17 -

[PATCH v2 06/11] drm/rockchip: lvds: Create an RK3288 specific probe function

The probe function is highly tighten to the RK3288 specificities, move all specific bits into an "rk3288_probe" function, also part of the platform data. The goal is to ease the addition of new flavors of Rockchip LVDS IPs. Signed-off-by: Miquel Raynal ---

Re: [PATCH 1/2] Return from mtk_ovl_layer_config after mtk_ovl_layer_off

Thanks for the update! On Tue, Dec 24, 2019, 9:25 PM CK Hu wrote: > Hi, Mark: > > On Wed, 2019-12-11 at 10:49 -0500, Mark Yacoub wrote: > > drm/mediatek: return if plane pending state is disabled. > > > > If the plane pending state is disabled, call mtk_ovl_layer_off then > > return. > > This

[PATCH v3 18/35] drm/bridge: ps8622: Use drm_bridge_init()

No functional change. v3: - drop driver_private argument (Laurent) Reviewed-by: Laurent Pinchart Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/bridge/parade-ps8622.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/bridge/parade-ps8622.c

[PATCH v3 04/35] drm/bridge: thc63lvd1024: Don't set drm_bridge->driver_private

It's unused. Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/bridge/thc63lvd1024.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/bridge/thc63lvd1024.c b/drivers/gpu/drm/bridge/thc63lvd1024.c index 3d74129b2995..784f4e4eea05 100644 ---

[PATCH v3 30/35] drm/i2c: tda998x: Use drm_bridge_init()

No functional change. v3: - drop driver_private argument (Laurent) Reviewed-by: Laurent Pinchart Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/i2c/tda998x_drv.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c

[PATCH v3 03/35] drm/bridge/synopsys: Stop using drm_bridge->driver_private

dw_hdmi: The drm_bridge struct is already embedded, so use a container_of wrapper to access it. dw-mipi-dsi: The field is unused, remove it. Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/bridge/synopsys/dw-hdmi.c | 15 --- drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c |

[PATCH v3 34/35] drm/sti: Use drm_bridge_init()

No functional change. v3: - drop driver_private argument (Laurent) v2: - Also apply drm_bridge_init() in sti_hdmi.c and sti_hda.c (Sam, Benjamin) Reviewed-by: Benjamin Gaignard Reviewed-by: Laurent Pinchart Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/sti/sti_dvo.c | 3 +--

[PATCH v3 15/35] drm/bridge: megachips-stdpxxxx-ge-b850v3-fw: Use drm_bridge_init()

No functional change. v3: - drop driver_private argument (Laurent) Reviewed-by: Laurent Pinchart Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/bridge/megachips-stdp-ge-b850v3-fw.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH v2 09/11] drm/rockchip: lvds: Add PX30 support

Introduce PX30 LVDS support. This means adding the relevant helper functions, a specific probe and also the initialization of a specific PHY. Signed-off-by: Miquel Raynal --- drivers/gpu/drm/rockchip/rockchip_lvds.c | 143 +++ drivers/gpu/drm/rockchip/rockchip_lvds.h | 14

[PATCH v2 08/11] drm/rockchip: lvds: Pack functions together

Reorganize a bit the functions order to clarify the driver. This change only moves functions around, there is no functional change. Signed-off-by: Miquel Raynal --- drivers/gpu/drm/rockchip/rockchip_lvds.c | 90 1 file changed, 45 insertions(+), 45 deletions(-) diff

Re: [PATCH v2 3/3] drm/panel: add panel driver for Leadtek LTK500HD1829

Hi Heiko, I love your patch! Perhaps something to improve: [auto build test WARNING on robh/for-next] [cannot apply to drm-exynos/exynos-drm-next drm-intel/for-linux-next tegra-drm/drm/tegra/for-next drm-tip/drm-tip linus/master drm/drm-next v5.5-rc3 next-20191220] [if your patch is applied to