RE: [Linux-stm32] [PATCH] drm/stm: ltdc: Use simple encoder

gan Teki; Yannick FERTRE; Philippe CORNU; Benjamin Gaignard; Vincent ABRIOU Cc : dri-de...@lists.freedesktop.org; linux-amar...@amarulasolutions.com; linux-kernel@vger.kernel.org; linux-arm-ker...@lists.infradead.org; linux-st...@st-md-mailman.stormreply.com Objet : Re: [Linux-stm32] [PATCH] drm/s

RE: panel: panel-raydium68200 driver fails to write MIPI DSI init commands

; David Airlie ; Daniel Vetter ; Sam Ravnborg ; Philippe CORNU ; Antonio BORNEO ; Subject: drm: panel: panel-raydium68200 driver fails to write MIPI DSI init commands I need to bring up my MIPI DSI 1280x720 EDT ETML0500F3DHA panel on a RockPro64 V2.1 board. There is no completely suitable in-tr

Re: [PATCH] dt-bindings: display: Add dsi-controller.yaml in DSI controller schemas

lienne > Cc: Florian Fainelli > Cc: Ray Jui > Cc: Scott Branden > Cc: bcm-kernel-feedback-l...@broadcom.com > Cc: Maxime Coquelin > Cc: Alexandre Torgue > Cc: "Guido Gúnther" > Cc: Robert Chiras > Cc: Philippe Cornu Hi Rob, and many thanks for the patch. For the stm32 part, Reviewed-by: Philippe Cornu Philippe :-)

Re: [PATCH v2] drm/stm: dsi: Use dev_ based logging

nst struct > drm_display_mode *mode, > > /* Select the color coding */ > dsi_update_bits(dsi, DSI_WCFGR, WCFGR_COLMUX, > - dsi_color_from_mipi(format) << 1); > + dsi_color_from_mipi(dsi, format) << 1); > &

Re: [PATCH] drm/panel: rm68200: fix mode to 50fps

80 + 12 + 5, > + .vtotal = 1280 + 12 + 5 + 12, > .flags = 0, > .width_mm = 68, > .height_mm = 122, > Hi Yannick, Tested-by: Philippe Cornu Thank you, Philippe

Re: [PATCH] drm/bridge: dw-mipi-dsi: permit configuring the escape clock rate

et_timing)(void *priv_data, unsigned int lane_mbps, > struct dw_mipi_dsi_dphy_timing *timing); > + int (*get_esc_clk_rate)(void *priv_data, unsigned int *esc_clk_rate); > }; > > struct dw_mipi_dsi_host_ops { > Hi Neil, Thank you for the patch Reviewed-by: Philippe Cornu Philippe :-)

Re: [PATCH] drm/bridge/synopsys: dsi: allow sending longer LP commands

in high speed is enable (PHY_TXREQUESTCLKHS) > - */ > - dsi_write(dsi, DSI_DPI_LP_CMD_TIM, OUTVACT_LPCMD_TIME(4) > - | INVACT_LPCMD_TIME(4)); > } > > static void dw_mipi_dsi_packet_handler_config(struct dw_mipi_dsi *dsi) > (+ Antonio) Hi Yannick &a

Re: [PATCH] drm/bridge/synopsys: dsi: add support for non-continuous HS clock

dsi_write(dsi, DSI_MODE_CFG, ENABLE_CMD_MODE); > } > > + val = PHY_TXREQUESTCLKHS; > + if (dsi->mode_flags & MIPI_DSI_CLOCK_NON_CONTINUOUS) > + val |= AUTO_CLKLANE_CTRL; > + dsi_write(dsi, DSI_LPCLK_CTRL, val); > + > dsi_write(

Re: [PATCH v2] drm/bridge/synopsys: dsi: allow LP commands in video mode

VID_MODE_CFG, val); > } > > static int dw_mipi_dsi_gen_pkt_hdr_write(struct dw_mipi_dsi *dsi, u32 > hdr_val) > (+ Antonio) Hi Yannick & Antonio, Reviewed-by: Philippe Cornu Tested-by: Philippe Cornu (Tested with the 3 patches named drm/bridge/synopsys: dsi: allow LP command

Re: [PATCH v2] drm/bridge: dw-mipi-dsi.c: Add VPG runtime config through debugfs

On 7/8/20 7:08 PM, Angelo Ribeiro wrote: > Hi, > > Is this patch good to go? > @dan...@ffwll.ch, @Philippe CORNU > > Was already tested by @Yannick FERTRE > and @Adrian Pop > on https://lkml.org/lkml/2020/4/6/691 . > > Thanks, > Angelo > > From: Yann

Re: [PATCH] drm/stm: ltdc: remove call of pm-runtime functions

t; - if (ret) { > - DRM_ERROR("Failed to fixup mode, cannot get sync\n"); > - return false; > - } > - } > - > DRM_DEBUG_DRIVER("requested clock %dkHz, adjusted clock %dkHz\n", >mode->clock, adjusted_mode->clock); > > Hi Yannick, Many thanks for your patch, Acked-by: Philippe Cornu Philippe :-)

Re: [Linux-stm32] [PATCH v8 08/10] drm: stm: dw-mipi-dsi: let the bridge handle the HW version check

Hi Adrian, and thank you very much for the patchset. Thank you also for having tested it on STM32F769 and STM32MP1. Sorry for the late response, Yannick and I will review it as soon as possible and we will keep you posted. Note: Do not hesitate to put us in copy for the next version

Re: [PATCH] drm/stm: ltdc: add pinctrl for DPI encoder mode

drm_encoder_init(ddev, encoder, _encoder_funcs, >DRM_MODE_ENCODER_DPI, NULL); > > + drm_encoder_helper_add(encoder, _encoder_helper_funcs); > + > ret = drm_bridge_attach(encoder, bridge, NULL); > if (ret) { >

Re: [PATCH] ARM: dts: stm32: move ltdc pinctrl on stm32mp157a dk1 board

; - pinctrl-1 = <_pins_sleep_a>; > status = "okay"; > > ports { > @@ -276,6 +273,9 @@ > }; > >{ > + pinctrl-names = "default", "sleep"; > + pinctrl-0 = <_pins_a>; > + pinctrl-1 = <_pins_sleep_a>; Reviewed-by: Philippe Cornu Thanks Philippe :) > status = "okay"; > > port { >

Re: [PATCH] drm/bridge: sii902x: add audio graph card support

Hi Olivier, and many thanks for your patch. Good to have the audio graph card support, looks ok. Reviewed-by: Philippe Cornu Philippe :-) On 7/3/19 10:04 AM, Olivier Moysan wrote: > Implement get_dai_id callback of audio HDMI codec > to support ASoC audio graph card. > HDMI audio o

Re: [PATCH 2/3] dt-bindings: display: sii902x: Change audio mclk binding

input is not required." So, even if #sound-dai-cells is there, this is not mandatory to have MCLK. Reviewed-by: Philippe Cornu Philippe :-) On 7/2/19 5:47 PM, Olivier Moysan wrote: > As stated in SiL9022/24 datasheet, master clock is not required for I2S. > Make mclk property optional in DT bind

Re: [PATCH v2] drm/stm: ltdc: remove clk_round_rate comment

Dear Yannick, Acked-by: Philippe Cornu Thank you, Philippe :-) On 5/13/19 3:15 PM, Yannick Fertré wrote: > Clk_round_rate returns rounded clock without changing > the hardware in any way. > This function couldn't replace set_rate/get_rate calls. > Todo comment has been removed

Re: [PATCH] drm/bridge: checkpatch strict minor updates

37:36PM +0200, Philippe Cornu wrote: >> Minor fixes detected with "scripts/checkpatch.pl --strict" >> >> Signed-off-by: Philippe Cornu <philippe.co...@st.com> >> --- >> Detected when merging "drm: clarify adjusted_mode documentation for bridges"

Re: [PATCH] drm/bridge: checkpatch strict minor updates

37:36PM +0200, Philippe Cornu wrote: >> Minor fixes detected with "scripts/checkpatch.pl --strict" >> >> Signed-off-by: Philippe Cornu >> --- >> Detected when merging "drm: clarify adjusted_mode documentation for bridges" > > Acked-by: Daniel

[PATCH] drm/bridge: checkpatch strict minor updates

Minor fixes detected with "scripts/checkpatch.pl --strict" Signed-off-by: Philippe Cornu <philippe.co...@st.com> --- Detected when merging "drm: clarify adjusted_mode documentation for bridges" include/drm/drm_bridge.h | 10 +- 1 file changed, 5 insertions(+),

[PATCH] drm/bridge: checkpatch strict minor updates

Minor fixes detected with "scripts/checkpatch.pl --strict" Signed-off-by: Philippe Cornu --- Detected when merging "drm: clarify adjusted_mode documentation for bridges" include/drm/drm_bridge.h | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff

Re: [PATCH] drm: clarify adjusted_mode documentation for bridges

Hi, Applied on drm-misc-next. Many thanks, Philippe :-) On 04/19/2018 07:00 PM, Archit Taneja wrote: > > > On Thursday 19 April 2018 09:20 PM, Philippe CORNU wrote: >> Hi Archit & Andrzej, >> >> May I ask you please a short review of this documentation upd

Re: [PATCH] drm: clarify adjusted_mode documentation for bridges

Hi, Applied on drm-misc-next. Many thanks, Philippe :-) On 04/19/2018 07:00 PM, Archit Taneja wrote: > > > On Thursday 19 April 2018 09:20 PM, Philippe CORNU wrote: >> Hi Archit & Andrzej, >> >> May I ask you please a short review of this documentation upd

Re: [PATCH v2 2/2] drm/bridge: sii902x: add optional power supplies

Hi Andrzej, On 05/14/2018 12:33 PM, Andrzej Hajda wrote: > On 14.05.2018 11:38, Philippe CORNU wrote: >> Hi Laurent, Archit, Andrzej & Yannick, >> >> Do you have any comments on this v2 driver part? >> (more details regarding v1/v2 differences in the cover letter &

Re: [PATCH v2 2/2] drm/bridge: sii902x: add optional power supplies

Hi Andrzej, On 05/14/2018 12:33 PM, Andrzej Hajda wrote: > On 14.05.2018 11:38, Philippe CORNU wrote: >> Hi Laurent, Archit, Andrzej & Yannick, >> >> Do you have any comments on this v2 driver part? >> (more details regarding v1/v2 differences in the cover letter &

Re: [PATCH v2 2/2] drm/bridge: sii902x: add optional power supplies

Hi Laurent, Archit, Andrzej & Yannick, Do you have any comments on this v2 driver part? (more details regarding v1/v2 differences in the cover letter https://www.spinics.net/lists/dri-devel/msg174137.html) Thank you, Philippe :-) On 04/25/2018 09:53 AM, Philippe Cornu wrote: &

Re: [PATCH v2 2/2] drm/bridge: sii902x: add optional power supplies

Hi Laurent, Archit, Andrzej & Yannick, Do you have any comments on this v2 driver part? (more details regarding v1/v2 differences in the cover letter https://www.spinics.net/lists/dri-devel/msg174137.html) Thank you, Philippe :-) On 04/25/2018 09:53 AM, Philippe Cornu wrote: &

Re: [PATCH v2 1/2] dt-bindings/display/bridge: sii902x: add optional power supplies

Hi Rob & Laurent :) On 04/26/2018 12:05 AM, Laurent Pinchart wrote: > Hi Rob, > > On Wednesday, 25 April 2018 20:11:23 EEST Rob Herring wrote: >> On Wed, Apr 25, 2018 at 04:17:25PM +0300, Laurent Pinchart wrote: >>> On Wednesday, 25 April 2018 15:20:04 EEST Philipp

Re: [PATCH v2 1/2] dt-bindings/display/bridge: sii902x: add optional power supplies

Hi Rob & Laurent :) On 04/26/2018 12:05 AM, Laurent Pinchart wrote: > Hi Rob, > > On Wednesday, 25 April 2018 20:11:23 EEST Rob Herring wrote: >> On Wed, Apr 25, 2018 at 04:17:25PM +0300, Laurent Pinchart wrote: >>> On Wednesday, 25 April 2018 15:20:04 EEST Philipp

Re: [PATCH] drm/stm: ltdc: fix warnings in ltdc_plane_create()

On 04/25/2018 09:13 AM, Yannick FERTRE wrote: > Hi Philippe, > > Reviewed-by: Yannick Fertré <yannick.fer...@st.com> > Applied on drm-misc-next. Many thanks, Philippe :-) > > On 04/19/2018 03:28 PM, Philippe Cornu wrote: >> "make C=1" returns 2 wa

Re: [PATCH] drm/stm: ltdc: fix warnings in ltdc_plane_create()

On 04/25/2018 09:13 AM, Yannick FERTRE wrote: > Hi Philippe, > > Reviewed-by: Yannick Fertré > Applied on drm-misc-next. Many thanks, Philippe :-) > > On 04/19/2018 03:28 PM, Philippe Cornu wrote: >> "make C=1" returns 2 warnings in ltdc_plane_create(

Re: [PATCH] drm/stm: ltdc: add mode_valid()

On 04/25/2018 09:12 AM, Yannick FERTRE wrote: > Hi Philippe, > > Reviewed-by: Yannick Fertré <yannick.fer...@st.com> > Applied on drm-misc-next. Many thanks, Philippe :-) > On 04/17/2018 01:40 PM, Philippe Cornu wrote: >> Add mode_valid() function to filter m

Re: [PATCH] drm/stm: ltdc: add mode_valid()

On 04/25/2018 09:12 AM, Yannick FERTRE wrote: > Hi Philippe, > > Reviewed-by: Yannick Fertré > Applied on drm-misc-next. Many thanks, Philippe :-) > On 04/17/2018 01:40 PM, Philippe Cornu wrote: >> Add mode_valid() function to filter modes according to availabl

Re: [PATCH] drm/stm: ltdc: fix deferred endpoint management

On 04/25/2018 09:12 AM, Yannick FERTRE wrote: > Hi Philippe, > > Reviewed-by: Yannick Fertré <yannick.fer...@st.com> Applied on drm-misc-next. Many thanks, Philippe :-) > > > On 04/17/2018 01:34 PM, Philippe Cornu wrote: >> When a driver related to one of th

Re: [PATCH] drm/stm: ltdc: fix deferred endpoint management

On 04/25/2018 09:12 AM, Yannick FERTRE wrote: > Hi Philippe, > > Reviewed-by: Yannick Fertré Applied on drm-misc-next. Many thanks, Philippe :-) > > > On 04/17/2018 01:34 PM, Philippe Cornu wrote: >> When a driver related to one of the endpoints is deferred >

Re: [PATCH v2 1/2] dt-bindings/display/bridge: sii902x: add optional power supplies

Hi Laurent & Rob :-) On 04/25/2018 11:01 AM, Laurent Pinchart wrote: > Hi Philippe, > > Thank you for the patch. > > On Wednesday, 25 April 2018 10:53:13 EEST Philippe Cornu wrote: >> Add optional power supplies using the description found in >> "SiI9022A/

Re: [PATCH v2 1/2] dt-bindings/display/bridge: sii902x: add optional power supplies

Hi Laurent & Rob :-) On 04/25/2018 11:01 AM, Laurent Pinchart wrote: > Hi Philippe, > > Thank you for the patch. > > On Wednesday, 25 April 2018 10:53:13 EEST Philippe Cornu wrote: >> Add optional power supplies using the description found in >> "SiI9022A/

[PATCH v2 2/2] drm/bridge: sii902x: add optional power supplies

Add the optional power supplies using the description found in "SiI9022A/SiI9024A HDMI Transmitter Data Sheet (August 2016)". The sii902x input IOs are not "io safe" so it is important to enable/disable voltage regulators during probe/remove phases to avoid damages. Signed-of

[PATCH v2 2/2] drm/bridge: sii902x: add optional power supplies

Add the optional power supplies using the description found in "SiI9022A/SiI9024A HDMI Transmitter Data Sheet (August 2016)". The sii902x input IOs are not "io safe" so it is important to enable/disable voltage regulators during probe/remove phases to avoid damages. Signed-of

[PATCH v2 1/2] dt-bindings/display/bridge: sii902x: add optional power supplies

cc12 and avcc12 can be derived from the same power source" Signed-off-by: Philippe Cornu <philippe.co...@st.com> --- Documentation/devicetree/bindings/display/bridge/sii902x.txt | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Documentation/devicetree/bindings/display/bridge/si

[PATCH v2 1/2] dt-bindings/display/bridge: sii902x: add optional power supplies

cc12 and avcc12 can be derived from the same power source" Signed-off-by: Philippe Cornu --- Documentation/devicetree/bindings/display/bridge/sii902x.txt | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Documentation/devicetree/bindings/display/bridge/sii902x.txt b/Documentation

[PATCH v2 0/2] drm/bridge: sii902x: add optional power supplies

ual platform is not an handheld device so power consumption is not critical. But in the future, it could be nice to add the standby mode, paying attention to wake-up events (hdmi cable plug)... Version 1: - Initial commit Philippe Cornu (2): dt-bindings/display/bridge: sii902x: add optional

[PATCH v2 0/2] drm/bridge: sii902x: add optional power supplies

ual platform is not an handheld device so power consumption is not critical. But in the future, it could be nice to add the standby mode, paying attention to wake-up events (hdmi cable plug)... Version 1: - Initial commit Philippe Cornu (2): dt-bindings/display/bridge: sii902x: add optional

[PATCH 0/4] drm/panel: otm8009a: backlight fixes & improvements

This patch serie fixes 2 backlight issues and adds the new backlight api support. Philippe Cornu (4): drm/panel: otm8009a: fix backlight updates drm/panel: otm8009a: fix glitches by moving backlight enable to otm8009a_enable() drm/panel: otm8009a: no message if probe success drm/panel

[PATCH 0/4] drm/panel: otm8009a: backlight fixes & improvements

This patch serie fixes 2 backlight issues and adds the new backlight api support. Philippe Cornu (4): drm/panel: otm8009a: fix backlight updates drm/panel: otm8009a: fix glitches by moving backlight enable to otm8009a_enable() drm/panel: otm8009a: no message if probe success drm/panel

[PATCH 1/4] drm/panel: otm8009a: fix backlight updates

, sleep in/out... dsi commands work in lp mode. Signed-off-by: Philippe Cornu <philippe.co...@st.com> --- drivers/gpu/drm/panel/panel-orisetech-otm8009a.c | 18 -- 1 file changed, 16 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/panel/panel-orisetech-otm8009

[PATCH 3/4] drm/panel: otm8009a: no message if probe success

Remove the message in case of probe success. This comes from a suggestion followed in the recent integration of the raydium rm68200 panel. Signed-off-by: Philippe Cornu <philippe.co...@st.com> --- drivers/gpu/drm/panel/panel-orisetech-otm8009a.c | 11 ++- 1 file changed, 2 inse

[PATCH 3/4] drm/panel: otm8009a: no message if probe success

Remove the message in case of probe success. This comes from a suggestion followed in the recent integration of the raydium rm68200 panel. Signed-off-by: Philippe Cornu --- drivers/gpu/drm/panel/panel-orisetech-otm8009a.c | 11 ++- 1 file changed, 2 insertions(+), 9 deletions(-) diff

[PATCH 1/4] drm/panel: otm8009a: fix backlight updates

, sleep in/out... dsi commands work in lp mode. Signed-off-by: Philippe Cornu --- drivers/gpu/drm/panel/panel-orisetech-otm8009a.c | 18 -- 1 file changed, 16 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/panel/panel-orisetech-otm8009a.c b/drivers/gpu/drm/panel/panel

[PATCH 4/4] drm/panel: otm8009a: use new backlight api

Use the new backlight api. Signed-off-by: Philippe Cornu <philippe.co...@st.com> --- drivers/gpu/drm/panel/panel-orisetech-otm8009a.c | 26 1 file changed, 9 insertions(+), 17 deletions(-) diff --git a/drivers/gpu/drm/panel/panel-orisetech-otm8009a.c b/drivers/g

[PATCH 2/4] drm/panel: otm8009a: fix glitches by moving backlight enable to otm8009a_enable()

, avoiding also image glitches visible on some "slow" devices. Signed-off-by: Philippe Cornu <philippe.co...@st.com> --- drivers/gpu/drm/panel/panel-orisetech-otm8009a.c | 17 ++--- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/drivers/gpu/drm/panel/panel-oris

[PATCH 4/4] drm/panel: otm8009a: use new backlight api

Use the new backlight api. Signed-off-by: Philippe Cornu --- drivers/gpu/drm/panel/panel-orisetech-otm8009a.c | 26 1 file changed, 9 insertions(+), 17 deletions(-) diff --git a/drivers/gpu/drm/panel/panel-orisetech-otm8009a.c b/drivers/gpu/drm/panel/panel-orisetech

[PATCH 2/4] drm/panel: otm8009a: fix glitches by moving backlight enable to otm8009a_enable()

, avoiding also image glitches visible on some "slow" devices. Signed-off-by: Philippe Cornu --- drivers/gpu/drm/panel/panel-orisetech-otm8009a.c | 17 ++--- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/drivers/gpu/drm/panel/panel-orisetech-otm8009a.c b/drive

Re: [PATCH] drm: clarify adjusted_mode documentation for bridges

Hi Archit & Andrzej, May I ask you please a short review of this documentation update. Many thanks Philippe :-) On 04/09/2018 05:24 PM, Philippe Cornu wrote: > This patch clarifies the adjusted_mode documentation > for bridges. > > Signed-off-by: Philippe Cornu <philippe.co.

Re: [PATCH] drm: clarify adjusted_mode documentation for bridges

Hi Archit & Andrzej, May I ask you please a short review of this documentation update. Many thanks Philippe :-) On 04/09/2018 05:24 PM, Philippe Cornu wrote: > This patch clarifies the adjusted_mode documentation > for bridges. > > Signed-off-by: Philippe Cornu > Sig

[PATCH] drm/stm: ltdc: fix warnings in ltdc_plane_create()

"make C=1" returns 2 warnings in ltdc_plane_create() ("Using plain integer as NULL pointer"). This patch fixes them. Signed-off-by: Philippe Cornu <philippe.co...@st.com> --- drivers/gpu/drm/stm/ltdc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff -

[PATCH] drm/stm: ltdc: fix warnings in ltdc_plane_create()

"make C=1" returns 2 warnings in ltdc_plane_create() ("Using plain integer as NULL pointer"). This patch fixes them. Signed-off-by: Philippe Cornu --- drivers/gpu/drm/stm/ltdc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/stm/ltdc

Re: [PATCH 1/2] dt-bindings/display/bridge: sii902x: add optional power supplies

Hi Laurent, On 04/19/2018 01:09 PM, Laurent Pinchart wrote: > Hi Philippe, > > On Thursday, 19 April 2018 12:31:15 EEST Philippe CORNU wrote: >> On 04/19/2018 10:11 AM, Laurent Pinchart wrote: >>> On Tuesday, 10 April 2018 08:19:26 EEST Philippe Cornu wrote: >>&

Re: [PATCH 1/2] dt-bindings/display/bridge: sii902x: add optional power supplies

Hi Laurent, On 04/19/2018 01:09 PM, Laurent Pinchart wrote: > Hi Philippe, > > On Thursday, 19 April 2018 12:31:15 EEST Philippe CORNU wrote: >> On 04/19/2018 10:11 AM, Laurent Pinchart wrote: >>> On Tuesday, 10 April 2018 08:19:26 EEST Philippe Cornu wrote: >>&

Re: [PATCH v2] drm/bridge/synopsys: dsi: Adopt SPDX identifiers

Applied on drm-misc-next. Many thanks, Philippe :-) On 02/08/2018 05:46 PM, Laurent Pinchart wrote: > Hi Philippe, > > Thank you for the patch. > > On Thursday, 8 February 2018 16:58:05 EET Philippe Cornu wrote: >> Add SPDX identifiers to the Synopsys DesignWare MIPI D

Re: [PATCH v2] drm/bridge/synopsys: dsi: Adopt SPDX identifiers

Applied on drm-misc-next. Many thanks, Philippe :-) On 02/08/2018 05:46 PM, Laurent Pinchart wrote: > Hi Philippe, > > Thank you for the patch. > > On Thursday, 8 February 2018 16:58:05 EET Philippe Cornu wrote: >> Add SPDX identifiers to the Synopsys DesignWare MIPI D

Re: [PATCH] drm/stm: ltdc: fix warning in ltdc_crtc_update_clut()

Applied on drm-misc-next. Many thanks, Philippe :-) On 04/16/2018 11:18 AM, Yannick FERTRE wrote: > Reviewed-by: yannick fertre <yannick.fer...@st.com> > > > On 04/10/2018 03:53 PM, Philippe Cornu wrote: >> Fix the warning >> "warn: variable dereference

Re: [PATCH] drm/stm: ltdc: fix warning in ltdc_crtc_update_clut()

Applied on drm-misc-next. Many thanks, Philippe :-) On 04/16/2018 11:18 AM, Yannick FERTRE wrote: > Reviewed-by: yannick fertre > > > On 04/10/2018 03:53 PM, Philippe Cornu wrote: >> Fix the warning >> "warn: variable dereferenced before check 'crtc' (see line 390

Re: [PATCH] drm/stm: move enable/disable_vblank to crtc

Applied on drm-misc-next. Many thanks, Philippe :-) On 04/16/2018 11:07 AM, Vincent ABRIOU wrote: > Hi Philippe, > > Patch looks good to me. > > Reviewed-by: Vincent Abriou <vincent.abr...@st.com> > > On 04/07/2018 11:29 PM, Philippe Cornu wrote: >>

Re: [PATCH] drm/stm: ltdc: add user update info in plane print state

Applied on drm-misc-next. Many thanks, Philippe :-) On 04/16/2018 11:14 AM, Vincent ABRIOU wrote: > Hi Philippe, > > Reviewed-by: Vincent Abriou <vincent.abr...@st.com> > > On 04/07/2018 11:35 PM, Philippe Cornu wrote: >> This patch adds the user update informat

Re: [PATCH] drm/stm: move enable/disable_vblank to crtc

Applied on drm-misc-next. Many thanks, Philippe :-) On 04/16/2018 11:07 AM, Vincent ABRIOU wrote: > Hi Philippe, > > Patch looks good to me. > > Reviewed-by: Vincent Abriou > > On 04/07/2018 11:29 PM, Philippe Cornu wrote: >> enable/disable_vblank() fu

Re: [PATCH] drm/stm: ltdc: add user update info in plane print state

Applied on drm-misc-next. Many thanks, Philippe :-) On 04/16/2018 11:14 AM, Vincent ABRIOU wrote: > Hi Philippe, > > Reviewed-by: Vincent Abriou > > On 04/07/2018 11:35 PM, Philippe Cornu wrote: >> This patch adds the user update information in >> frames-per-second

Re: [PATCH 2/2] drm/bridge: sii902x: add optional power supplies

Hi Laurent :-) On 04/19/2018 10:20 AM, Laurent Pinchart wrote: > Hi Philippe, > > Thank you for the patch. > > On Tuesday, 10 April 2018 08:19:27 EEST Philippe Cornu wrote: >> Add the 3 optional power supplies using the exact description >> found in the document named

Re: [PATCH 2/2] drm/bridge: sii902x: add optional power supplies

Hi Laurent :-) On 04/19/2018 10:20 AM, Laurent Pinchart wrote: > Hi Philippe, > > Thank you for the patch. > > On Tuesday, 10 April 2018 08:19:27 EEST Philippe Cornu wrote: >> Add the 3 optional power supplies using the exact description >> found in the document named

Re: [PATCH 1/2] dt-bindings/display/bridge: sii902x: add optional power supplies

Hi Laurent : ) On 04/19/2018 10:11 AM, Laurent Pinchart wrote: > Hi Philippe, > > Thank you for the patch. > > On Tuesday, 10 April 2018 08:19:26 EEST Philippe Cornu wrote: >> Add the 3 optional power supplies using the exact description >> found in the document named

Re: [PATCH 1/2] dt-bindings/display/bridge: sii902x: add optional power supplies

Hi Laurent : ) On 04/19/2018 10:11 AM, Laurent Pinchart wrote: > Hi Philippe, > > Thank you for the patch. > > On Tuesday, 10 April 2018 08:19:26 EEST Philippe Cornu wrote: >> Add the 3 optional power supplies using the exact description >> found in the document named

[PATCH] drm/stm: ltdc: add mode_valid()

nel clock tolerances are bigger than hdmi ones and there is no reason to not accept them (the fps may vary a little but it is not a problem). - the hdmi preferred mode will be accepted too, but userland will be able to use others hdmi "valid" modes if necessary. Signed-off-by: Philipp

[PATCH] drm/stm: ltdc: add mode_valid()

nel clock tolerances are bigger than hdmi ones and there is no reason to not accept them (the fps may vary a little but it is not a problem). - the hdmi preferred mode will be accepted too, but userland will be able to use others hdmi "valid" modes if necessary. Signed-off-by: Philippe C

[PATCH] drm/stm: ltdc: fix deferred endpoint management

-off-by: Philippe Cornu <philippe.co...@st.com> --- drivers/gpu/drm/stm/ltdc.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm/stm/ltdc.c index e3121d9e4230..014cef8cef37 100644 --- a/drivers/gpu/drm/stm/ltdc.c +++ b/d

[PATCH] drm/stm: ltdc: fix deferred endpoint management

-off-by: Philippe Cornu --- drivers/gpu/drm/stm/ltdc.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm/stm/ltdc.c index e3121d9e4230..014cef8cef37 100644 --- a/drivers/gpu/drm/stm/ltdc.c +++ b/drivers/gpu/drm/stm/ltdc.c

[PATCH] drm/stm: ltdc: fix warning in ltdc_crtc_update_clut()

report https://lists.freedesktop.org/archives/dri-devel/2018-February/166918.html Signed-off-by: Philippe Cornu <philippe.co...@st.com> Reported-by: Dan Carpenter <dan.carpen...@oracle.com> --- drivers/gpu/drm/stm/ltdc.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/gpu/drm/stm/ltd

[PATCH] drm/stm: ltdc: fix warning in ltdc_crtc_update_clut()

report https://lists.freedesktop.org/archives/dri-devel/2018-February/166918.html Signed-off-by: Philippe Cornu Reported-by: Dan Carpenter --- drivers/gpu/drm/stm/ltdc.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm/stm/ltdc.c index 061d2b6e5157..e3121d

[PATCH 2/2] drm/bridge: sii902x: add optional power supplies

Add the 3 optional power supplies using the exact description found in the document named "SiI9022A/SiI9024A HDMI Transmitter Data Sheet (August 2016)". Signed-off-by: Philippe Cornu <philippe.co...@st.com> --- drivers/gpu/drm/bridge/sii902x.c | 39 +++

[PATCH 2/2] drm/bridge: sii902x: add optional power supplies

Add the 3 optional power supplies using the exact description found in the document named "SiI9022A/SiI9024A HDMI Transmitter Data Sheet (August 2016)". Signed-off-by: Philippe Cornu --- drivers/gpu/drm/bridge/sii902x.c | 39 +++ 1 file changed, 35

[PATCH 0/2] drm/bridge: sii902x: add optional power supplies

This patchset adds the 3 optional power supplies to the sii902x drm bridge driver. Philippe Cornu (2): dt-bindings/display/bridge: sii902x: add optional power supplies drm/bridge: sii902x: add optional power supplies .../devicetree/bindings/display/bridge/sii902x.txt | 3 ++ drivers/gpu

[PATCH 0/2] drm/bridge: sii902x: add optional power supplies

This patchset adds the 3 optional power supplies to the sii902x drm bridge driver. Philippe Cornu (2): dt-bindings/display/bridge: sii902x: add optional power supplies drm/bridge: sii902x: add optional power supplies .../devicetree/bindings/display/bridge/sii902x.txt | 3 ++ drivers/gpu

[PATCH 1/2] dt-bindings/display/bridge: sii902x: add optional power supplies

Add the 3 optional power supplies using the exact description found in the document named "SiI9022A/SiI9024A HDMI Transmitter Data Sheet (August 2016)". Signed-off-by: Philippe Cornu <philippe.co...@st.com> --- Documentation/devicetree/bindings/display/bridge/sii902x.txt | 3 +++

[PATCH 1/2] dt-bindings/display/bridge: sii902x: add optional power supplies

Add the 3 optional power supplies using the exact description found in the document named "SiI9022A/SiI9024A HDMI Transmitter Data Sheet (August 2016)". Signed-off-by: Philippe Cornu --- Documentation/devicetree/bindings/display/bridge/sii902x.txt | 3 +++ 1 file changed, 3 insertion

[PATCH] drm: clarify adjusted_mode documentation for bridges

This patch clarifies the adjusted_mode documentation for bridges. Signed-off-by: Philippe Cornu <philippe.co...@st.com> Signed-off-by: Laurent Pinchart <laurent.pinchart+rene...@ideasonboard.com> --- This patch follows discussions in: - "drm: clarify adjusted_mode for a bridge co

[PATCH] drm: clarify adjusted_mode documentation for bridges

This patch clarifies the adjusted_mode documentation for bridges. Signed-off-by: Philippe Cornu Signed-off-by: Laurent Pinchart --- This patch follows discussions in: - "drm: clarify adjusted_mode for a bridge connected to a crtc" https://patchwork.freedesktop.org/patch/206801/ - &q

[PATCH] drm/stm: ltdc: add user update info in plane print state

This patch adds the user update information in frames-per-second into the drm debugfs plane state. Signed-off-by: Philippe Cornu <philippe.co...@st.com> --- drivers/gpu/drm/stm/ltdc.c | 22 ++ drivers/gpu/drm/stm/ltdc.h | 8 2 files changed, 30 insertions(+)

[PATCH] drm/stm: ltdc: add user update info in plane print state

This patch adds the user update information in frames-per-second into the drm debugfs plane state. Signed-off-by: Philippe Cornu --- drivers/gpu/drm/stm/ltdc.c | 22 ++ drivers/gpu/drm/stm/ltdc.h | 8 2 files changed, 30 insertions(+) diff --git a/drivers/gpu/drm

[PATCH] drm/stm: move enable/disable_vblank to crtc

enable/disable_vblank() functions at drm_driver level are deprecated. Move them to the ltdc drm_crtc_funcs structure. Signed-off-by: Philippe Cornu <philippe.co...@st.com> --- drivers/gpu/drm/stm/drv.c | 2 -- drivers/gpu/drm/stm/ltdc.c | 10 ++ drivers/gpu/drm/stm/ltdc.h | 2

[PATCH] drm/stm: move enable/disable_vblank to crtc

enable/disable_vblank() functions at drm_driver level are deprecated. Move them to the ltdc drm_crtc_funcs structure. Signed-off-by: Philippe Cornu --- drivers/gpu/drm/stm/drv.c | 2 -- drivers/gpu/drm/stm/ltdc.c | 10 ++ drivers/gpu/drm/stm/ltdc.h | 2 -- 3 files changed, 6

Re: [PATCH] drm: clarify adjusted_mode for a bridge connected to a crtc

Hi Laurent, On 04/06/2018 04:53 PM, Laurent Pinchart wrote: > Hi Philippe, > > Thank you for the patch. > > On Monday, 26 February 2018 14:16:04 EEST Philippe Cornu wrote: >> This patch clarifies the adjusted_mode documentation >> for a bridge directly connected

Re: [PATCH] drm: clarify adjusted_mode for a bridge connected to a crtc

Hi Laurent, On 04/06/2018 04:53 PM, Laurent Pinchart wrote: > Hi Philippe, > > Thank you for the patch. > > On Monday, 26 February 2018 14:16:04 EEST Philippe Cornu wrote: >> This patch clarifies the adjusted_mode documentation >> for a bridge directly connected

Re: [PATCH] drm: clarify adjusted_mode for a bridge connected to a crtc

On 03/29/2018 09:39 AM, Daniel Vetter wrote: > On Thu, Mar 29, 2018 at 9:35 AM, Philippe CORNU <philippe.co...@st.com> wrote: >> Hi Daniel, >> >> On 03/27/2018 05:51 PM, Daniel Vetter wrote: >>> On Mon, Feb 26, 2018 at 01:16:04PM +0100, Philippe Co

Re: [PATCH] drm: clarify adjusted_mode for a bridge connected to a crtc

On 03/29/2018 09:39 AM, Daniel Vetter wrote: > On Thu, Mar 29, 2018 at 9:35 AM, Philippe CORNU wrote: >> Hi Daniel, >> >> On 03/27/2018 05:51 PM, Daniel Vetter wrote: >>> On Mon, Feb 26, 2018 at 01:16:04PM +0100, Philippe Cornu wrote: >>>> This patch

Re: [PATCH] drm/sti: Depend on OF rather than selecting it

On 04/05/2018 01:05 PM, Benjamin Gaignard wrote: > 2018-04-03 7:34 GMT+02:00 Oliver O'Halloran : >> Commit cc6b741c6f63 ("drm: sti: remove useless fields from vtg >> structure") reworked some code inside of this driver and made it select >> CONFIG_OF. This results in the entire

Re: [PATCH] drm/sti: Depend on OF rather than selecting it

On 04/05/2018 01:05 PM, Benjamin Gaignard wrote: > 2018-04-03 7:34 GMT+02:00 Oliver O'Halloran : >> Commit cc6b741c6f63 ("drm: sti: remove useless fields from vtg >> structure") reworked some code inside of this driver and made it select >> CONFIG_OF. This results in the entire OF layer being

Re: [PATCH] drm: clarify adjusted_mode for a bridge connected to a crtc

Hi Daniel, On 03/27/2018 05:51 PM, Daniel Vetter wrote: > On Mon, Feb 26, 2018 at 01:16:04PM +0100, Philippe Cornu wrote: >> This patch clarifies the adjusted_mode documentation >> for a bridge directly connected to a crtc. >> >> Signed-off-by: Philippe C

Re: [PATCH] drm: clarify adjusted_mode for a bridge connected to a crtc

Hi Daniel, On 03/27/2018 05:51 PM, Daniel Vetter wrote: > On Mon, Feb 26, 2018 at 01:16:04PM +0100, Philippe Cornu wrote: >> This patch clarifies the adjusted_mode documentation >> for a bridge directly connected to a crtc. >> >> Signed-off-by: Philippe Cornu >

Re: [PATCH v2 0/2] drm/panel: Add support for Raydium RM68200 panel

Hi Thierry, On 03/12/2018 09:04 AM, Thierry Reding wrote: > On Fri, Mar 02, 2018 at 04:32:20PM +0100, Philippe Cornu wrote: >> The Raydium Semiconductor Corporation RM68200 is a 5.5" 720x1280 >> TFT LCD panel connected using a MIPI-DSI video interface. >> >>

Re: [PATCH v2 0/2] drm/panel: Add support for Raydium RM68200 panel

Hi Thierry, On 03/12/2018 09:04 AM, Thierry Reding wrote: > On Fri, Mar 02, 2018 at 04:32:20PM +0100, Philippe Cornu wrote: >> The Raydium Semiconductor Corporation RM68200 is a 5.5" 720x1280 >> TFT LCD panel connected using a MIPI-DSI video interface. >> >>

Re: [PATCH v1 2/2] drm/panel: Add support for Raydium rm68200 panel driver

Hi Thierry, A big thank you for your code review and comments. It helped me to improve the driver and to send a v2. Philippe :-) On 02/28/2018 08:16 PM, Thierry Reding wrote: > On Thu, Feb 08, 2018 at 03:30:26PM +0100, Philippe Cornu wrote: >> This patch adds Raydium Semiconductor Co

  1   2   3   4   5   >