[PATCH v2 2/4] drm: rcar-du: dsi: Improve DSI shutdown

2022-08-22 Thread Tomi Valkeinen
From: Tomi Valkeinen Improve the DSI shutdown procedure by clearing various bits that were set while enabling the DSI output. There has been no clear issues caused by these, but it's safer to ensure that the features are disabled at the start of the next DSI enable. Signed-off-by:

[PATCH v2 4/4] drm: rcar-du: lvds: Rename pclk enable/disable functions

2022-08-22 Thread Tomi Valkeinen
From: Tomi Valkeinen Rename LVDS pclk enable/disable functions to match what we use for DSI. Signed-off-by: Tomi Valkeinen --- drivers/gpu/drm/rcar-du/rcar_du_crtc.c | 4 ++-- drivers/gpu/drm/rcar-du/rcar_lvds.c| 4 ++-- drivers/gpu/drm/rcar-du/rcar_lvds.h| 10 +- 3 files

Re: [PATCH v2 2/4] drm: rcar-du: dsi: Improve DSI shutdown

2022-08-22 Thread Tomi Valkeinen
On 22/08/2022 16:20, Laurent Pinchart wrote: Hi Tomi, Thank you for the patch. On Mon, Aug 22, 2022 at 04:05:10PM +0300, Tomi Valkeinen wrote: From: Tomi Valkeinen Improve the DSI shutdown procedure by clearing various bits that were set while enabling the DSI output. There has been no

Re: [PATCH v2 4/4] drm: rcar-du: lvds: Rename pclk enable/disable functions

2022-08-22 Thread Tomi Valkeinen
On 22/08/2022 16:52, Laurent Pinchart wrote: Hi Tomi, Thank you for the patch. On Mon, Aug 22, 2022 at 04:05:12PM +0300, Tomi Valkeinen wrote: From: Tomi Valkeinen Rename LVDS pclk enable/disable functions to match what we use for DSI. I'd explain here the rationale for the new

Re: [PATCH v2 1/4] drm: rcar-du: dsi: Properly stop video mode TX

2022-08-22 Thread Tomi Valkeinen
Hi, On 22/08/2022 16:25, Biju Das wrote: Hi Tomi, Thanks for the patch. Subject: [PATCH v2 1/4] drm: rcar-du: dsi: Properly stop video mode TX From: Tomi Valkeinen The driver does not explicitly stop the video mode transmission when disabling the output. While this doesn't seem

Re: [PATCH v2 2/4] drm: rcar-du: dsi: Improve DSI shutdown

2022-08-22 Thread Tomi Valkeinen
On 22/08/2022 17:05, Laurent Pinchart wrote: On Mon, Aug 22, 2022 at 04:49:02PM +0300, Tomi Valkeinen wrote: On 22/08/2022 16:20, Laurent Pinchart wrote: Hi Tomi, Thank you for the patch. On Mon, Aug 22, 2022 at 04:05:10PM +0300, Tomi Valkeinen wrote: From: Tomi Valkeinen Improve the DSI

[PATCH v3 0/5] drm: rcar-du: DSI fixes

2022-08-22 Thread Tomi Valkeinen
Changes to v2: - Move the LVDS patch to the beginning and improve the patch desc. - Write 0 to registers in rcar_mipi_dsi_shutdown instead of just clearing a bit. - Fix function names in "fix DSI enable & disable sequence" patch desc. - Add "drm: rcar-du: dsi: Fix VCLKSET

[PATCH v3 2/5] drm: rcar-du: dsi: Properly stop video mode TX

2022-08-22 Thread Tomi Valkeinen
From: Tomi Valkeinen The driver does not explicitly stop the video mode transmission when disabling the output. While this doesn't seem to be causing any issues, lets follow the steps described in the documentation and add a rcar_mipi_dsi_stop_video() which stop the video mode transmission.

[PATCH v3 3/5] drm: rcar-du: dsi: Improve DSI shutdown

2022-08-22 Thread Tomi Valkeinen
From: Tomi Valkeinen Improve the DSI shutdown procedure by clearing various bits that were set while enabling the DSI output. There has been no clear issues caused by these, but it's safer to ensure that the features are disabled at the start of the next DSI enable. Signed-off-by:

[PATCH v3 5/5] drm: rcar-du: dsi: Fix VCLKSET write

2022-08-22 Thread Tomi Valkeinen
From: Tomi Valkeinen rcar_mipi_dsi_startup() writes correct values to VCLKSET, but as it uses or-operation to add the new values to the current value in the register, it should first make sure the fields are cleared. Do this by using rcar_mipi_dsi_write() to write the VCLKSET_CKEN bit to

[PATCH v3 1/5] drm: rcar-du: lvds: Rename pclk enable/disable functions

2022-08-22 Thread Tomi Valkeinen
From: Tomi Valkeinen The DU driver uses the rcar_lvds_clk_enable() and rcar_lvds_clk_disable() functions enable or disable the pixel clock generated by the LVDS encoder, as it requires that clock for proper DU operation. Rename the functions by replacing "clk" with "pclk" to

Re: [PATCH v3 5/5] drm: rcar-du: dsi: Fix VCLKSET write

2022-08-23 Thread Tomi Valkeinen
On 22/08/2022 17:45, Laurent Pinchart wrote: Hi Tomi, Thank you for the patch. On Mon, Aug 22, 2022 at 05:34:01PM +0300, Tomi Valkeinen wrote: From: Tomi Valkeinen rcar_mipi_dsi_startup() writes correct values to VCLKSET, but as it uses or-operation to add the new values to the current

[PATCH v3.1 5/5] drm: rcar-du: dsi: Fix VCLKSET write

2022-08-23 Thread Tomi Valkeinen
From: Tomi Valkeinen rcar_mipi_dsi_startup() writes correct values to VCLKSET, but as it uses or-operation to add the new values to the current value in the register, it should first make sure the fields are cleared. Do this by using rcar_mipi_dsi_write() to write the VCLKSET register with a

Re: [PATCH v3 4/5] drm: rcar-du: fix DSI enable & disable sequence

2022-08-24 Thread Tomi Valkeinen
On 22/08/2022 17:34, Tomi Valkeinen wrote: +struct drm_atomic_state; +struct drm_bridge; + +#if IS_ENABLED(CONFIG_DRM_RCAR_MIPI_DSI) +void rcar_mipi_dsi_pclk_enable(struct drm_bridge *bridge, + struct drm_atomic_state *state); +void rcar_mipi_dsi_pclk_disable(struct

[PATCH v4 0/5] drm: rcar-du: DSI fixes

2022-08-24 Thread Tomi Valkeinen
Changes to v3: - Fix LVDS function renames wrt. export symbol. - Fix missing static inline - Use rcar_mipi_dsi_write for VCLKSET writes Tomi Tomi Valkeinen (5): drm: rcar-du: lvds: Rename pclk enable/disable functions drm: rcar-du: dsi: Properly stop video mode TX drm: rcar-du: dsi

[PATCH v4 2/5] drm: rcar-du: dsi: Properly stop video mode TX

2022-08-24 Thread Tomi Valkeinen
From: Tomi Valkeinen The driver does not explicitly stop the video mode transmission when disabling the output. While this doesn't seem to be causing any issues, lets follow the steps described in the documentation and add a rcar_mipi_dsi_stop_video() which stop the video mode transmission.

[PATCH v4 5/5] drm: rcar-du: dsi: Fix VCLKSET write

2022-08-24 Thread Tomi Valkeinen
From: Tomi Valkeinen rcar_mipi_dsi_startup() writes correct values to VCLKSET, but as it uses or-operation to add the new values to the current value in the register, it should first make sure the fields are cleared. Do this by using rcar_mipi_dsi_write() to write the VCLKSET register with a

[PATCH v4 3/5] drm: rcar-du: dsi: Improve DSI shutdown

2022-08-24 Thread Tomi Valkeinen
From: Tomi Valkeinen Improve the DSI shutdown procedure by clearing various bits that were set while enabling the DSI output. There has been no clear issues caused by these, but it's safer to ensure that the features are disabled at the start of the next DSI enable. Signed-off-by:

[PATCH v4 1/5] drm: rcar-du: lvds: Rename pclk enable/disable functions

2022-08-24 Thread Tomi Valkeinen
From: Tomi Valkeinen The DU driver uses the rcar_lvds_clk_enable() and rcar_lvds_clk_disable() functions enable or disable the pixel clock generated by the LVDS encoder, as it requires that clock for proper DU operation. Rename the functions by replacing "clk" with "pclk" to

[PATCH v4 4/5] drm: rcar-du: fix DSI enable & disable sequence

2022-08-24 Thread Tomi Valkeinen
From: Tomi Valkeinen The rcar crtc depends on the clock provided from the rcar DSI bridge. When the DSI bridge is disabled, the clock is stopped, which causes the crtc disable to timeout. Also, while I have no issue with the enable, the documentation suggests to enable the DSI before the crtc

[PATCH v5 0/4] drm/bridge: ti-sn65dsi86: Basic DP support

2022-08-24 Thread Tomi Valkeinen
ort (non-eDP) mode drm/bridge: ti-sn65dsi86: Implement bridge connector operations Tomi Valkeinen (2): drm/bridge: ti-sn65dsi86: check AUX errors better drm/bridge: ti-sn65dsi86: Reject modes with too large blanking drivers/gpu/drm/bridge/ti-sn65dsi86.c | 77 +--

[PATCH v5 1/4] drm/bridge: ti-sn65dsi86: check AUX errors better

2022-08-24 Thread Tomi Valkeinen
From: Tomi Valkeinen The driver does not check AUX_IRQ_STATUS_NAT_I2C_FAIL bit at all when sending AUX transfers, which leads to the driver not returning an error. Add the check. Signed-off-by: Tomi Valkeinen --- drivers/gpu/drm/bridge/ti-sn65dsi86.c | 5 + 1 file changed, 5 insertions

[PATCH v5 3/4] drm/bridge: ti-sn65dsi86: Support DisplayPort (non-eDP) mode

2022-08-24 Thread Tomi Valkeinen
: Laurent Pinchart Reworked to set bridge type based on the next bridge/connector. Signed-off-by: Kieran Bingham Reviewed-by: Laurent Pinchart Reviewed-by: Douglas Anderson Signed-off-by: Tomi Valkeinen -- Changes since v1/RFC: - Rebased on top of "drm/bridge: ti-sn65dsi86: swit

[PATCH v5 4/4] drm/bridge: ti-sn65dsi86: Implement bridge connector operations

2022-08-24 Thread Tomi Valkeinen
the comments, they've had issues with eDP panels and HPD. So better be safe and only enable them for DP for now. Signed-off-by: Laurent Pinchart Signed-off-by: Kieran Bingham Signed-off-by: Tomi Valkeinen --- Changes since v1: - The connector .get_modes() operation doesn't rely on ED

[PATCH v5 2/4] drm/bridge: ti-sn65dsi86: Reject modes with too large blanking

2022-08-24 Thread Tomi Valkeinen
From: Tomi Valkeinen The blanking related registers are 8 bits, so reject any modes with larger blanking periods. Signed-off-by: Tomi Valkeinen --- drivers/gpu/drm/bridge/ti-sn65dsi86.c | 23 +++ 1 file changed, 23 insertions(+) diff --git a/drivers/gpu/drm/bridge/ti

[PATCH 3/3] drm: rcar-du: fix DSI enable & disable sequence

2022-08-24 Thread Tomi Valkeinen
From: Tomi Valkeinen The rcar crtc depends on the clock provided from the rcar DSI bridge. When the DSI bridge is disabled, the clock is stopped, which causes the crtc disable to timeout. Also, while I have no issue with the enable, the documentation suggests to enable the DSI before the crtc

[PATCH v3 4/5] drm: rcar-du: fix DSI enable & disable sequence

2022-08-24 Thread Tomi Valkeinen
From: Tomi Valkeinen The rcar crtc depends on the clock provided from the rcar DSI bridge. When the DSI bridge is disabled, the clock is stopped, which causes the crtc disable to timeout. Also, while I have no issue with the enable, the documentation suggests to enable the DSI before the crtc

Re: [PATCH] drm/tidss: Soft Reset DISPC on startup

2022-04-12 Thread Tomi Valkeinen
not valid, this patch is ok and: Reviewed-by: Tomi Valkeinen Tomi Signed-off-by: Devarsh Thakkar --- drivers/gpu/drm/tidss/tidss_dispc.c | 18 ++ 1 file changed, 18 insertions(+) diff --git a/drivers/gpu/drm/tidss/tidss_dispc.c b/drivers/gpu/drm/tidss/tidss_dispc.c index 60

Re: [RESEND][PATCH] omapdrm: fix missing check on list iterator

2022-04-14 Thread Tomi Valkeinen
Hi, On 14/04/2022 09:14, Xiaomeng Tong wrote: The bug is here: bus_flags = connector->display_info.bus_flags; The list iterator 'connector-' will point to a bogus position containing HEAD if the list is empty or no element is found. This case must be checked before any use of the iterat

Re: [PATCH 0/2] Update register & interrupt info in am65x DSS

2022-04-19 Thread Tomi Valkeinen
ch/arm64/boot/dts/ti/k3-am65-main.dtsi | 6 -- 2 files changed, 11 insertions(+), 5 deletions(-) Reviewed-by: Tomi Valkeinen How are you planning to use the common1 area? Tomi

Re: [PATCH] drm/tidss: Soft Reset DISPC on startup

2022-04-19 Thread Tomi Valkeinen
On 14/03/2022 13:37, Devarsh Thakkar wrote: Soft reset the display subsystem controller on startup and wait for the reset to complete. This helps the scenario where display was already in use by some other core before the linux was booted. Signed-off-by: Devarsh Thakkar --- drivers/gpu/drm/ti

Re: [PATCH v5 2/4] drm/bridge: ti-sn65dsi86: Reject modes with too large blanking

2022-08-30 Thread Tomi Valkeinen
Hi Doug, On 29/08/2022 20:23, Doug Anderson wrote: Hi, On Wed, Aug 24, 2022 at 6:00 AM Tomi Valkeinen wrote: From: Tomi Valkeinen The blanking related registers are 8 bits, so reject any modes with larger blanking periods. Signed-off-by: Tomi Valkeinen --- drivers/gpu/drm/bridge/ti

Re: [PATCH v5 4/4] drm/bridge: ti-sn65dsi86: Implement bridge connector operations

2022-08-30 Thread Tomi Valkeinen
Hi, On 29/08/2022 20:38, Doug Anderson wrote: Hi, On Wed, Aug 24, 2022 at 6:00 AM Tomi Valkeinen wrote: From: Laurent Pinchart Implement the bridge connector-related .get_edid() and .detect() operations, and report the related bridge capabilities and type. These ops are only added for DP

Re: [PATCH v5 1/4] drm/bridge: ti-sn65dsi86: check AUX errors better

2022-08-30 Thread Tomi Valkeinen
Hi, On 29/08/2022 20:15, Doug Anderson wrote: Hi, On Wed, Aug 24, 2022 at 6:01 AM Tomi Valkeinen wrote: From: Tomi Valkeinen The driver does not check AUX_IRQ_STATUS_NAT_I2C_FAIL bit at all when sending AUX transfers, It doesn't? What about a few lines down from where your

Re: [PATCH v5 4/4] drm/bridge: ti-sn65dsi86: Implement bridge connector operations

2022-08-30 Thread Tomi Valkeinen
Hi, On 30/08/2022 17:55, Doug Anderson wrote: Hi, On Tue, Aug 30, 2022 at 2:00 AM Tomi Valkeinen wrote: Hi, On 29/08/2022 20:38, Doug Anderson wrote: Hi, On Wed, Aug 24, 2022 at 6:00 AM Tomi Valkeinen wrote: From: Laurent Pinchart Implement the bridge connector-related .get_edid

Re: [PATCH 3/3] drm: omapdrm: Do no allocate non-scanout GEMs through DMM/TILER

2022-08-30 Thread Tomi Valkeinen
On 30/08/2022 21:08, Ivaylo Dimitrov wrote: flags &= ~OMAP_BO_TILED_MASK; flags |= 0x0008; flags |= OMAP_BO_WC; bo = omap_bo_new(dev, size, flags); As you can see we use 0x0008 (OMAP_BO_MEM_CONTIG) unconditionally. This was a hack added since even no

[PATCH v6 1/3] drm/bridge: ti-sn65dsi86: Reject modes with too large blanking

2022-08-31 Thread Tomi Valkeinen
From: Tomi Valkeinen The front and back porch registers are 8 bits, and pulse width registers are 15 bits, so reject any modes with larger periods. Signed-off-by: Tomi Valkeinen --- drivers/gpu/drm/bridge/ti-sn65dsi86.c | 23 +++ 1 file changed, 23 insertions(+) diff

[PATCH v6 2/3] drm/bridge: ti-sn65dsi86: Support DisplayPort (non-eDP) mode

2022-08-31 Thread Tomi Valkeinen
: Laurent Pinchart Reworked to set bridge type based on the next bridge/connector. Signed-off-by: Kieran Bingham Reviewed-by: Laurent Pinchart Reviewed-by: Douglas Anderson Signed-off-by: Tomi Valkeinen -- Changes since v1/RFC: - Rebased on top of "drm/bridge: ti-sn65dsi86: swit

[PATCH v6 3/3] drm/bridge: ti-sn65dsi86: Implement bridge connector operations for DP

2022-08-31 Thread Tomi Valkeinen
From: Laurent Pinchart Implement the bridge connector-related .get_edid() and .detect() operations for full DP mode, and report the related bridge capabilities and type. Signed-off-by: Laurent Pinchart Signed-off-by: Kieran Bingham Signed-off-by: Tomi Valkeinen Reviewed-by: Douglas Anderson

[PATCH v6 0/3] drm/bridge: ti-sn65dsi86: Basic DP support

2022-08-31 Thread Tomi Valkeinen
bout eDP, detect and get_edid from the desc of "Implement bridge connector operations" Tomi Laurent Pinchart (2): drm/bridge: ti-sn65dsi86: Support DisplayPort (non-eDP) mode drm/bridge: ti-sn65dsi86: Implement bridge connector operations for DP Tomi Valkeinen (1): drm/bridge: t

Re: [PATCH] drm/bridge_connector: enable HPD by default if supported

2022-08-31 Thread Tomi Valkeinen
Hi, On 23/02/2022 19:02, Kieran Bingham wrote: Quoting Laurent Pinchart (2022-02-23 16:25:28) Hello, On Wed, Feb 23, 2022 at 04:17:22PM +, Kieran Bingham wrote: Quoting Laurent Pinchart (2021-12-29 23:44:29) On Sat, Dec 25, 2021 at 09:31:51AM +0300, Nikita Yushchenko wrote: Hotplug even

Re: [PATCH] drm/bridge_connector: enable HPD by default if supported

2022-09-04 Thread Tomi Valkeinen
On 31/08/2022 16:02, Tomi Valkeinen wrote: Hi, On 23/02/2022 19:02, Kieran Bingham wrote: Quoting Laurent Pinchart (2022-02-23 16:25:28) Hello, On Wed, Feb 23, 2022 at 04:17:22PM +, Kieran Bingham wrote: Quoting Laurent Pinchart (2021-12-29 23:44:29) On Sat, Dec 25, 2021 at 09:31:51AM

Re: [PATCH] drm: omapdrm: dss: replace ternary operator with max()

2022-09-09 Thread Tomi Valkeinen
On 17/05/2022 08:02, Guo Zhengkui wrote: Fix the following coccicheck warning: drivers/gpu/drm/omapdrm/dss/dispc.c:2454:21-22: WARNING opportunity for max() Signed-off-by: Guo Zhengkui --- drivers/gpu/drm/omapdrm/dss/dispc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [PATCH] drm/omap: dss: Fix refcount leak bugs

2022-09-09 Thread Tomi Valkeinen
On 22/07/2022 17:43, Liang He wrote: In dss_init_ports() and __dss_uninit_ports(), we should call of_node_put() for the reference returned by of_graph_get_port_by_id() in fail path or when it is not used anymore. Fixes: 09bffa6e5192 ("drm: omap: use common OF graph helpers") Signed-off-by: Liang

Re: [PATCH] drm/tidss: fix repeated words in comments

2022-09-09 Thread Tomi Valkeinen
On 24/08/2022 16:04, Jilin Yuan wrote: Delete the redundant word 'to'. Signed-off-by: Jilin Yuan --- drivers/gpu/drm/tidss/tidss_kms.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/tidss/tidss_kms.c b/drivers/gpu/drm/tidss/tidss_kms.c index 666e527a0a

Re: [PATCH v2] drm: omapdrm: Improve check for contiguous buffers

2022-09-09 Thread Tomi Valkeinen
On 25/08/2022 19:26, Andrew Davis wrote: While a scatter-gather table having only 1 entry does imply it is contiguous, it is a logic error to assume the inverse. Tables can have more than 1 entry and still be contiguous. Use a proper check here. Signed-off-by: Andrew Davis --- Changes from v1:

Re: [PATCH v1 0/7] drm/bridge_connector: perform HPD enablement automatically

2022-09-12 Thread Tomi Valkeinen
Hi, On 29/04/2022 21:51, Dmitry Baryshkov wrote: From all the drivers using drm_bridge_connector only iMX/dcss and OMAP DRM driver do a proper work of calling drm_bridge_connector_en/disable_hpd() in right places. Rather than teaching each and every driver how to properly handle drm_bridge_conn

Re: [PATCH v1 7/7] drm/bridge_connector: drop drm_bridge_connector_en/disable_hpd()

2022-09-12 Thread Tomi Valkeinen
On 29/04/2022 21:51, Dmitry Baryshkov wrote: Now as all drivers stopped calling drm_bridge_connector_enable_hpd() and drm_bridge_connector_disable_hpd() it is safe to remove them complelely. Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/drm_bridge_connector.c | 25 --

[PATCH 1/2] drm/omap: dsi: Fix excessive stack usage

2022-09-16 Thread Tomi Valkeinen
, let's allocate the struct with kmalloc instead. Compile-tested only. Signed-off-by: Tomi Valkeinen Reported-by: kernel test robot --- drivers/gpu/drm/omapdrm/dss/dsi.c | 26 -- 1 file changed, 16 insertions(+), 10 deletions(-) diff --git a/drivers/gpu/drm/omapdr

[PATCH 2/2] drm/omap: Fix kernel docs

2022-09-16 Thread Tomi Valkeinen
Fix doc related warnings seen with W=1: the function names have changed but the docs have not been changed accordingly. Signed-off-by: Tomi Valkeinen --- drivers/gpu/drm/omapdrm/omap_gem.c | 2 +- drivers/gpu/drm/omapdrm/omap_irq.c | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions

Re: drivers/gpu/drm/omapdrm/dss/dsi.c:1126:1: warning: the frame size of 1060 bytes is larger than 1024 bytes

2022-09-16 Thread Tomi Valkeinen
On 15/09/2022 21:49, Arnd Bergmann wrote: On Thu, Sep 15, 2022, at 8:17 PM, kernel test robot wrote: Hi Arnd, tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 3245cb65fd91cd514801bf91f5a3066d562f0ac4 commit: 3d427228f7370894680580fcd0381c0349624fa1 ARM: ixp

Re: [PATCH 8/8] drm/tidss: Enable Dual and Duplicate Modes for OLDI

2022-07-27 Thread Tomi Valkeinen
Hi, On 19/07/2022 11:08, Aradhya Bhatia wrote: The AM625 DSS peripheral supports 2 OLDI TXes which can work to enable 2 duplicated displays of smaller resolutions or enable a single Dual-Link display with a higher resolution (1920x1200). Configure the necessary register to enable the different

Re: [PATCH 1/8] dt-bindings: display: ti,am65x-dss: Add port properties for DSS

2022-07-27 Thread Tomi Valkeinen
On 22/07/2022 19:16, Nishanth Menon wrote: On 17:28-20220720, Rob Herring wrote: On the bridge side R0->R2, G0->G1, B0->B2 would be tied to ground. The bridge sees 24bits of data, but the lsb's are always zero. Unless the bridge ignores the LSBs, that's not the right way to do 16 to 24 bit. T

Re: [PATCH 8/8] drm/tidss: Enable Dual and Duplicate Modes for OLDI

2022-07-27 Thread Tomi Valkeinen
On 27/07/2022 16:22, Tomi Valkeinen wrote: Hi, On 19/07/2022 11:08, Aradhya Bhatia wrote: The AM625 DSS peripheral supports 2 OLDI TXes which can work to enable 2 duplicated displays of smaller resolutions or enable a single Dual-Link display with a higher resolution (1920x1200). Configure

Re: [PATCH 7/8] drm/tidss: Fix clock request value for OLDI videoports

2022-07-28 Thread Tomi Valkeinen
On 19/07/2022 11:08, Aradhya Bhatia wrote: The OLDI TX(es) require a serial clock which is 7 times the pixel clock of the display panel. When the OLDI is enabled in DSS, the pixel clock input of the corresponding videoport gets a divided-by 7 value of the requested clock. For the am625-dss, the

Re: [PATCH 4/8] drm/tidss: Add support for Dual Link LVDS Bus Format

2022-07-28 Thread Tomi Valkeinen
On 19/07/2022 11:08, Aradhya Bhatia wrote: The 2 OLDI TXes in the AM625 SoC can be synced together to output a 2K resolution video. Add support in the driver for the discovery of such a dual mode connection on the OLDI video port, using the values of "ti,oldi-mode" property. Signed-off-by: Arad

Re: [PATCH 1/8] dt-bindings: display: ti,am65x-dss: Add port properties for DSS

2022-07-28 Thread Tomi Valkeinen
On 19/07/2022 11:08, Aradhya Bhatia wrote: Add "ti,oldi-mode" property to indicate the tidss driver the OLDI output mode. The 2 OLDI TXes on am625-dss allow a 3 different types of panel connections with the board. 1. Single Link / Single Mode on OLDI TX 0 OR 1. 2. Single Link / Duplicate Mode

Re: [PATCH 8/8] drm/tidss: Enable Dual and Duplicate Modes for OLDI

2022-07-28 Thread Tomi Valkeinen
On 28/07/2022 11:49, Aradhya Bhatia wrote: Hi Tomi, On 28-Jul-22 12:16, Tomi Valkeinen wrote: On 27/07/2022 16:22, Tomi Valkeinen wrote: Hi, On 19/07/2022 11:08, Aradhya Bhatia wrote: The AM625 DSS peripheral supports 2 OLDI TXes which can work to enable 2 duplicated displays of smaller

Re: [PATCH 4/8] drm/tidss: Add support for Dual Link LVDS Bus Format

2022-07-28 Thread Tomi Valkeinen
On 28/07/2022 14:03, Tomi Valkeinen wrote: On 19/07/2022 11:08, Aradhya Bhatia wrote: The 2 OLDI TXes in the AM625 SoC can be synced together to output a 2K resolution video. Add support in the driver for the discovery of such a dual mode connection on the OLDI video port, using the values of

Re: [PATCH v3 2/2] drm/tidss: Add support for AM625 DSS

2022-07-28 Thread Tomi Valkeinen
e similar DSS functionality as we have for AM65x. So, I think these two patches could be merged, or we could wait a bit until the OLDI situation becomes more clear. Up to you. In any case, for both patches: Reviewed-by: Tomi Valkeinen Tomi

Re: [PATCH 3/8] drm/tidss: Add support for DSS port properties

2022-07-28 Thread Tomi Valkeinen
On 19/07/2022 11:08, Aradhya Bhatia wrote: Add support to enable and read the dss port properties. The "ti,oldi-mode" property indicates the tidss driver how many OLDI TXes are enabled as well as which mode do they need to be connected in. The "ti,rgb565_to_888" is a special property that force

Re: [PATCH 7/8] drm/tidss: Fix clock request value for OLDI videoports

2022-07-29 Thread Tomi Valkeinen
On 29/07/2022 06:56, Aradhya Bhatia wrote: On 28-Jul-22 15:35, Tomi Valkeinen wrote: On 19/07/2022 11:08, Aradhya Bhatia wrote: The OLDI TX(es) require a serial clock which is 7 times the pixel clock of the display panel. When the OLDI is enabled in DSS, the pixel clock input of the

Re: [PATCH v5 1/8] drm/omap: use refcount API to track the number of users of dma_addr

2019-10-28 Thread Tomi Valkeinen
Hi JJ, On 10/10/2019 14:59, Jean-Jacques Hiblot wrote: This would give us a WARN_ON() if the pin/unpin calls are unbalanced. Proposed-by: Laurent Pinchart Signed-off-by: Jean-Jacques Hiblot Reviewed-by: Tomi Valkeinen --- drivers/gpu/drm/omapdrm/omap_gem.c | 44

Re: [PATCH] drm/bridge: tc358767: fix max_tu_symbol value

2019-11-06 Thread Tomi Valkeinen
On 10/10/2019 12:25, Tomi Valkeinen wrote: Hi Andrzej, On 10/10/2019 12:19, Andrzej Hajda wrote: On 24.09.2019 15:17, Tomi Valkeinen wrote: max_tu_symbol was programmed to TU_SIZE_RECOMMENDED - 1, which is not what the spec says. The spec says: roundup ((input active video bandwidth in bytes

[PATCH] drm/omap: fix dma_addr refcounting

2019-11-14 Thread Tomi Valkeinen
em_unpin if the buffer being unpinned is contiguous. Signed-off-by: Tomi Valkeinen --- drivers/gpu/drm/omapdrm/omap_gem.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/gpu/drm/omapdrm/omap_gem.c b/drivers/gpu/drm/omapdrm/omap_gem.c index e518d93ca6df..d08ae95ecc0a 100644 --- a/

[PATCH 1/3] ARM: dts: am437x-gp/epos-evm: fix panel compatible

2019-11-14 Thread Tomi Valkeinen
The LCD panel on AM4 GP EVMs and ePOS boards seems to be osd070t1718-19ts. The current dts files say osd057T0559-34ts. Possibly the panel has changed since the early EVMs, or there has been a mistake with the panel type. Update the DT files accordingly. Signed-off-by: Tomi Valkeinen --- arch

[PATCH 0/3] drm/omap: fix am4 evm lcd

2019-11-14 Thread Tomi Valkeinen
roller driver should be able to deduce the sync drive edge from the pixdata drive edge. Are they usually the same? I have no idea... Tomi Tomi Valkeinen (3): ARM: dts: am437x-gp/epos-evm: fix panel compatible ARM: dts: am437x-gp/epos-evm: drop unused panel timings drm/panel: simple: fix o

[PATCH 3/3] drm/panel: simple: fix osd070t1718_19ts sync drive edge

2019-11-14 Thread Tomi Valkeinen
fixes this issue. Signed-off-by: Tomi Valkeinen --- drivers/gpu/drm/panel/panel-simple.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c index 5d487686d25c..0784536ae6af 100644 --- a/drivers/gpu/drm

[PATCH 2/3] ARM: dts: am437x-gp/epos-evm: drop unused panel timings

2019-11-14 Thread Tomi Valkeinen
panel-simple now handled panel osd070t1718-19ts, and we no longer need the panel timings in the DT file. So remove them. Signed-off-by: Tomi Valkeinen --- arch/arm/boot/dts/am437x-gp-evm.dts | 16 arch/arm/boot/dts/am43x-epos-evm.dts | 16 2 files changed, 32

Re: [PATCH 0/4] drm/omap: dss: Use PTR_ERR_OR_ZERO() to simplify code

2019-11-18 Thread Tomi Valkeinen
Hi zhengbin, On 18/11/2019 14:07, zhengbin wrote: zhengbin (4): drm/omap: dss: Use PTR_ERR_OR_ZERO() to simplify code in hdmi_audio_register drm/omap: dss: Use PTR_ERR_OR_ZERO() to simplify code in hdmi4_core_init drm/omap: dss: Use PTR_ERR_OR_ZERO() to simplify code in h

Re: [PATCH 07/15] drm/omapdrm: Drop dma_buf->k(un)map

2019-11-18 Thread Tomi Valkeinen
gt; impossible to buy a chip with tegra host1x and ompadrm on the same > SoC, there's no problem here. > > Signed-off-by: Daniel Vetter > Cc: Tomi Valkeinen > --- > drivers/gpu/drm/omapdrm/omap_gem_dmabuf.c | 21 - > 1 file changed, 21 deletions(-

Re: [RFCv1 11/42] ARM: dts: omap: add channel to DSI panels

2019-11-18 Thread Tomi Valkeinen
On 17/11/2019 04:39, Sebastian Reichel wrote: The standard binding for DSI requires, that the channel number of the panel is encoded in DT. This adds the channel number in all OMAP3-5 boards, in preparation for using common infrastructure. Signed-off-by: Sebastian Reichel --- .../devicetree/b

Re: [RFCv1 33/42] drm/omap: dsi: use atomic helper for dirtyfb

2019-11-18 Thread Tomi Valkeinen
On 19/11/2019 01:05, Tony Lindgren wrote: * Sebastian Reichel [191117 07:11]: We can simply use the atomic helper for handling the dirtyfb callback. ... --- a/drivers/gpu/drm/omapdrm/omap_crtc.c +++ b/drivers/gpu/drm/omapdrm/omap_crtc.c -void omap_crtc_flush(struct drm_crtc *crtc) +static voi

Re: [PATCH 2/3] ARM: dts: am437x-gp/epos-evm: drop unused panel timings

2019-12-12 Thread Tomi Valkeinen
On 11/12/2019 18:53, Tony Lindgren wrote: * Laurent Pinchart [191202 13:05]: Hi Tomi, Thank you for the patch. On Thu, Nov 14, 2019 at 11:39:49AM +0200, Tomi Valkeinen wrote: panel-simple now handled panel osd070t1718-19ts, and we no longer need the panel timings in the DT file. So remove

Re: [PATCH 1/4] ARM: dts: am437x-gp-evm: add HDMI support

2019-12-13 Thread Tomi Valkeinen
On 13/12/2019 12:42, Laurent Pinchart wrote: I think the correct way would be to have DRM framework understand that we have two displays, which are mutually exclusive, and the display pipeline drivers would have the means to switch the gpio. And that the display setup could be communicated pro

Re: [PATCH v3 5/5] MAINTAINERS: add entry for tidss

2019-12-13 Thread Tomi Valkeinen
Hi Daniel, On 13/12/2019 12:30, Daniel Vetter wrote: +DRM DRIVERS FOR TI KEYSTONE +M: Jyri Sarha +M: Tomi Valkeinen +L: dri-devel@lists.freedesktop.org +S: Maintained +F: drivers/gpu/drm/tidss/ +F: Documentation/devicetree/bindings/display/ti/ti,k2g-dss.yaml +F

Re: [PATCH 1/4] ARM: dts: am437x-gp-evm: add HDMI support

2019-12-13 Thread Tomi Valkeinen
On 13/12/2019 13:42, Laurent Pinchart wrote: Hi Tomi, On Fri, Dec 13, 2019 at 12:56:31PM +0200, Tomi Valkeinen wrote: On 13/12/2019 12:42, Laurent Pinchart wrote: I think the correct way would be to have DRM framework understand that we have two displays, which are mutually exclusive, and

Re: [PATCH 1/4] ARM: dts: am437x-gp-evm: add HDMI support

2019-12-13 Thread Tomi Valkeinen
On 12/12/2019 19:31, Tony Lindgren wrote: * Tony Lindgren [191212 17:21]: Hi, * Tomi Valkeinen [191125 05:11]: Add HDMI support for AM437x GP EVM. The HDMI uses SiI9022 HDMI encoder, and is mutually exclusive with the LCD. The choice between LCD and HDMI is made by booting either with

Re: [PATCH 1/4] ARM: dts: am437x-gp-evm: add HDMI support

2019-12-13 Thread Tomi Valkeinen
On 13/12/2019 14:28, Laurent Pinchart wrote: So... In the DT file, we would have multiple endpoints in the same output port in DSS, one going to the panel, one to the SiI9022? omapdrm could then create two encoders, one abstracting the DPI output and the connection to the panel, one abstractin

Re: [PATCH 1/4] ARM: dts: am437x-gp-evm: add HDMI support

2019-12-13 Thread Tomi Valkeinen
On 12/12/2019 19:31, Tony Lindgren wrote: Anyways, I'm applying the changes to dra76-evm am57xx-idk-common into omap-for-v5.6/dt as they have no GPIO pin limitation. Thanks! I'd like to hear comments from folks on the first two though. Just to summarize the discussion, let's drop the first

Re: [PATCH v3 12/50] drm/bridge: Add driver for the TI TPD12S015 HDMI level shifter

2019-12-16 Thread Tomi Valkeinen
++ 3 files changed, 220 insertions(+) create mode 100644 drivers/gpu/drm/bridge/ti-tpd12s015.c Reviewed-by: Tomi Valkeinen Tomi -- Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

Re: [PATCH v3 17/50] drm/omap: Fix possible object reference leak

2019-12-16 Thread Tomi Valkeinen
insertions(+), 1 deletion(-) Reviewed-by: Tomi Valkeinen Tomi -- Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki ___ dri-devel mailing list dri-devel@lists.freedesktop.org https

Re: [PATCH v3 18/50] drm/omap: dss: Cleanup DSS ports on initialisation failure

2019-12-16 Thread Tomi Valkeinen
chart --- drivers/gpu/drm/omapdrm/dss/dss.c | 43 +++ 1 file changed, 26 insertions(+), 17 deletions(-) Reviewed-by: Tomi Valkeinen Tomi -- Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Hel

Re: [PATCH v3 11/50] drm/bridge: Add bridge driver for display connectors

2019-12-17 Thread Tomi Valkeinen
On 11/12/2019 00:57, Laurent Pinchart wrote: Display connectors are modelled in DT as a device node, but have so far been handled manually in several bridge drivers. This resulted in duplicate code in several bridge drivers, with slightly different (and thus confusing) logics. In order to fix th

Re: [PATCH v3 05/50] drm/bridge: Extend bridge API to disable connector creation

2019-12-17 Thread Tomi Valkeinen
On 11/12/2019 00:57, Laurent Pinchart wrote: Most bridge drivers create a DRM connector to model the connector at the output of the bridge. This model is historical and has worked pretty well so far, but causes several issues: - It prevents supporting more complex display pipelines where DRM con

Re: [PATCH v3 26/50] drm/omap: hdmi4: Rework EDID read to isolate data read

2019-12-17 Thread Tomi Valkeinen
/drm/omapdrm/dss/hdmi4_core.c | 59 +++ drivers/gpu/drm/omapdrm/dss/hdmi4_core.h | 4 +- 3 files changed, 73 insertions(+), 84 deletions(-) Reviewed-by: Tomi Valkeinen Tomi -- Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4

Re: [PATCH v3 27/50] drm/omap: hdmi5: Rework EDID read to isolate data read

2019-12-17 Thread Tomi Valkeinen
/drm/omapdrm/dss/hdmi5_core.c | 48 +++-- drivers/gpu/drm/omapdrm/dss/hdmi5_core.h | 5 +- 3 files changed, 65 insertions(+), 77 deletions(-) Reviewed-by: Tomi Valkeinen Tomi -- Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4

Re: [PATCH v3 28/50] drm/omap: hdmi4: Register a drm_bridge for EDID read

2019-12-17 Thread Tomi Valkeinen
: - Unregister bridge if output initialisation fails --- drivers/gpu/drm/omapdrm/dss/hdmi.h | 3 ++ drivers/gpu/drm/omapdrm/dss/hdmi4.c | 78 ++--- 2 files changed, 75 insertions(+), 6 deletions(-) Reviewed-by: Tomi Valkeinen Tomi -- Texas Instruments Finland Oy

Re: [PATCH v3 29/50] drm/omap: hdmi5: Register a drm_bridge for EDID read

2019-12-17 Thread Tomi Valkeinen
: - Unregister bridge if output initialisation fails --- drivers/gpu/drm/omapdrm/dss/hdmi5.c | 79 ++--- 1 file changed, 73 insertions(+), 6 deletions(-) Reviewed-by: Tomi Valkeinen Tomi -- Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus

Re: [PATCH v3 11/50] drm/bridge: Add bridge driver for display connectors

2019-12-17 Thread Tomi Valkeinen
On 18/12/2019 03:45, Laurent Pinchart wrote: + dev_info(&pdev->dev, +"Found %s display connector '%s' %s DDC bus and %s HPD GPIO (ops 0x%x)\n", +drm_get_connector_type_name(conn->bridge.type), +conn->label ? conn->label : "", +

Re: [PATCH v3 10/50] drm/bridge: simple-bridge: Add support for the TI OP362

2019-12-17 Thread Tomi Valkeinen
On 11/12/2019 00:57, Laurent Pinchart wrote: The TI OP362 is an analog video amplifier controlled through a GPIO. Add support for it to the simple-bridge driver. Noticed just now, you have "OP" instead of "OPA" above and in the subject. Tomi -- Texas Instruments Finland Oy, Porkkalankatu 22,

Re: [PATCH v3 00/50] drm/omap: Replace custom display drivers with drm_bridge and drm_panel

2019-12-17 Thread Tomi Valkeinen
On 18/12/2019 04:03, Laurent Pinchart wrote: Hopefully we can soon have this series landed so we can start working on top of the new bridge/connector handling. I assume it will be applied direct to drm-misc-next, so we do not have to wait extra time to get it. Tomi, how would you like to proc

Re: [PATCH v3 30/50] drm/omap: hdmi4: Move mode set, enable and disable operations to bridge

2019-12-18 Thread Tomi Valkeinen
.set_infoframe() and .set_hdmi_mode() handling in the commit message --- drivers/gpu/drm/omapdrm/dss/hdmi4.c | 201 +++- 1 file changed, 106 insertions(+), 95 deletions(-) Reviewed-by: Tomi Valkeinen Tomi -- Texas Instruments Finland Oy, Porkkalankatu 22, 00180

Re: [PATCH v3 31/50] drm/omap: hdmi5: Move mode set, enable and disable operations to bridge

2019-12-18 Thread Tomi Valkeinen
.set_infoframe() and .set_hdmi_mode() handling in the commit message --- drivers/gpu/drm/omapdrm/dss/hdmi5.c | 204 +++- 1 file changed, 106 insertions(+), 98 deletions(-) Reviewed-by: Tomi Valkeinen Tomi -- Texas Instruments Finland Oy, Porkkalankatu 22, 00180

Re: [PATCH v3 32/50] drm/omap: hdmi4: Implement drm_bridge .hpd_notify() operation

2019-12-18 Thread Tomi Valkeinen
On 11/12/2019 00:57, Laurent Pinchart wrote: The HDMI4 encoder is transitioning to the drm_bridge API, implement the last missing operation. Signed-off-by: Laurent Pinchart --- drivers/gpu/drm/omapdrm/dss/hdmi4.c | 10 ++ 1 file changed, 10 insertions(+) Reviewed-by: Tomi

Re: [PATCH v3 33/50] drm/omap: dss: Remove .set_hdmi_mode() and .set_infoframe() operations

2019-12-18 Thread Tomi Valkeinen
-- 2 files changed, 29 deletions(-) Reviewed-by: Tomi Valkeinen Tomi -- Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki ___ dri-devel mailing list

Re: [PATCH v3 34/50] drm/omap: venc: Register a drm_bridge

2019-12-18 Thread Tomi Valkeinen
--- Changes since v2: - Unregister bridge if output initialisation fails --- drivers/gpu/drm/omapdrm/dss/venc.c | 247 +++-- 1 file changed, 163 insertions(+), 84 deletions(-) Reviewed-by: Tomi Valkeinen Tomi -- Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki

Re: [PATCH v3 35/50] drm/omap: Create connector for bridges

2019-12-18 Thread Tomi Valkeinen
deletions(-) Reviewed-by: Tomi Valkeinen Tomi -- Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freed

Re: [PATCH v3 36/50] drm/omap: Switch the HDMI and VENC outputs to drm_bridge

2019-12-18 Thread Tomi Valkeinen
everything working. Other than that: Reviewed-by: Tomi Valkeinen Tomi -- Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki ___ dri-devel mailing list dri-devel

Re: [PATCH v3 00/50] drm/omap: Replace custom display drivers with drm_bridge and drm_panel

2019-12-18 Thread Tomi Valkeinen
On 18/12/2019 16:46, Laurent Pinchart wrote: Hi Tomi, On Wed, Dec 18, 2019 at 09:07:52AM +0200, Tomi Valkeinen wrote: On 18/12/2019 04:03, Laurent Pinchart wrote: Hopefully we can soon have this series landed so we can start working on top of the new bridge/connector handling. I assume it

<    3   4   5   6   7   8   9   10   11   12   >