[PATCH v7 06/10] drm/vc4: crtc: Add some logging

2021-08-19 Thread Maxime Ripard
The encoder retrieval code has been a source of bugs and glitches in the past and the crtc <-> encoder association been wrong in a number of different ways. Add some logging to quickly spot issues if they occur. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_crtc.c | 6 ++

[PATCH v7 05/10] drm/vc4: crtc: Rework the encoder retrieval code (again)

2021-08-19 Thread Maxime Ripard
thus our encoder retrieval code will not match on anything. We can however simplify the code by using drm_for_each_encoder_mask, the drm_crtc_state storing the encoders a given CRTC is connected to directly and without relying on any other state. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4

[PATCH v7 09/10] drm/vc4: hdmi: Enable the scrambler on reconnection

2021-08-19 Thread Maxime Ripard
s the scrambler to be enabled. Fixes: c85695a2016e ("drm/vc4: hdmi: Enable the scrambler") Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/vc4/vc4_hdmi.c b/drivers/gpu

[PATCH v7 07/10] drm/vc4: Leverage the load tracker on the BCM2711

2021-08-19 Thread Maxime Ripard
against them, and prevent the debugfs file to enable it from being created. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_debugfs.c | 7 +-- drivers/gpu/drm/vc4/vc4_drv.h | 3 --- drivers/gpu/drm/vc4/vc4_kms.c | 16 +--- drivers/gpu/drm/vc4/vc4_plane.c | 5 -

[PATCH v7 08/10] drm/vc4: hdmi: Raise the maximum clock rate

2021-08-19 Thread Maxime Ripard
Reviewed-by: Dave Stevenson Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/vc4/vc4_hdmi.c b/drivers/gpu/drm/vc4/vc4_hdmi.c index 251dfecf1d4c..d2a720e05ddd 100644 --- a/drivers/gpu/drm/vc4

[PATCH v7 04/10] drm/vc4: crtc: Add encoder to vc4_crtc_config_pv prototype

2021-08-19 Thread Maxime Ripard
vc4_crtc_config_pv() retrieves the encoder again, even though its only caller, vc4_crtc_atomic_enable(), already did. Pass the encoder pointer as an argument instead of going through all the connectors to retrieve it again. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_crtc.c | 7

[PATCH v7 10/10] drm/vc4: Increase the core clock based on HVS load

2021-08-19 Thread Maxime Ripard
Depending on a given HVS output (HVS to PixelValves) and input (planes attached to a channel) load, the HVS needs for the core clock to be raised above its boot time default. Failing to do so will result in a vblank timeout and a stalled display pipeline. Signed-off-by: Maxime Ripard

[PATCH 2/2] drm/vc4: hdmi: Actually check for the connector status in hotplug

2021-08-19 Thread Maxime Ripard
) if it was enabled it has a side effect, and an application such as Kodi or modetest doesn't deal with hotplug events. This resulted with a black screen when Kodi or modetest was running when a screen was disconnected and then reconnected, or switched off and on. Signed-off-by: Maxime Ri

[PATCH 1/2] drm/probe-helper: Create a HPD IRQ event helper for a single connector

2021-08-19 Thread Maxime Ripard
ation for each connector or use that helper and iterate over all connectors all the time. Since both are suboptimal, let's create a helper that will only perform the status detection on a single connector. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/drm_probe_help

[PATCH v3 0/6] drm/vc4: hdmi: Fix CEC access while disabled

2021-08-19 Thread Maxime Ripard
is disabled. - Fixed the pre_crtc_configure error path Maxime Ripard (6): drm/vc4: select PM drm/vc4: hdmi: Make sure the controller is powered up during bind drm/vc4: hdmi: Rework the pre_crtc_configure error handling drm/vc4: hdmi: Split the CEC disable / enable functions in two drm/vc4: h

[PATCH v3 1/6] drm/vc4: select PM

2021-08-19 Thread Maxime Ripard
We already depend on runtime PM to get the power domains and clocks for most of the devices supported by the vc4 driver, so let's just select it to make sure it's there, and remove the ifdef. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/Kconfig| 1 + drivers/gpu/drm/vc4/

[PATCH v3 2/6] drm/vc4: hdmi: Make sure the controller is powered up during bind

2021-08-19 Thread Maxime Ripard
hdmi: Move the HSM clock enable to runtime_pm") Reviewed-by: Dave Stevenson Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c | 17 +++-- 1 file changed, 15 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/vc4/vc4_hdmi.c b/drivers/gpu/drm/vc4/vc4_hd

[PATCH v3 3/6] drm/vc4: hdmi: Rework the pre_crtc_configure error handling

2021-08-19 Thread Maxime Ripard
goto-based implementation to have an easier consitency. Fixes: 4f6e3d66ac52 ("drm/vc4: Add runtime PM support to the HDMI encoder driver") Reviewed-by: Dave Stevenson Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c | 21 ++--- 1 file changed, 14 i

[PATCH v3 4/6] drm/vc4: hdmi: Split the CEC disable / enable functions in two

2021-08-19 Thread Maxime Ripard
In order to ease further additions to the CEC enable and disable, let's split the function into two functions, one to enable and the other to disable. Reviewed-by: Dave Stevenson Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c | 73 -- 1

[PATCH v3 5/6] drm/vc4: hdmi: Make sure the device is powered with CEC

2021-08-19 Thread Maxime Ripard
Let's make sure we run the runtime_pm hooks when the CEC adapter is opened and closed by the userspace to avoid that issue. Fixes: 15b4511a4af6 ("drm/vc4: add HDMI CEC support") Reviewed-by: Dave Stevenson Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c | 10 +

[PATCH v3 6/6] drm/vc4: hdmi: Warn if we access the controller while disabled

2021-08-19 Thread Maxime Ripard
We've had many silent hangs where the kernel would look like it just stalled due to the access to one of the HDMI registers while the controller was disabled. Add a warning if we're about to do that so that it's at least not silent anymore. Reviewed-by: Dave Stevenson Signed

[PATCH] drm/vc4: hdmi: Remove unused struct

2021-08-19 Thread Maxime Ripard
t") Reported-by: kernel test robot Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c | 8 1 file changed, 8 deletions(-) diff --git a/drivers/gpu/drm/vc4/vc4_hdmi.c b/drivers/gpu/drm/vc4/vc4_hdmi.c index b7dc32a0c9bb..1e2d976e8736 100644 --- a/drivers/gpu/drm/vc4/vc4_hdmi

Re: [PATCH v2 0/8] drm/bridge: Make panel and bridge probe order consistent

2021-08-20 Thread Maxime Ripard
Hi Andrzej, On Wed, Aug 04, 2021 at 04:09:38PM +0200, a.hajda wrote: > Hi Maxime, > > I have been busy with other tasks, and I did not follow the list last > time, so sorry for my late response. > > On 28.07.2021 15:32, Maxime Ripard wrote: > > Hi, > > > >

[PATCH v3 0/8] drm/bridge: Make panel and bridge probe order consistent

2021-08-23 Thread Maxime Ripard
ttach and _detach succeed when no callback is there Maxime Ripard (8): drm/bridge: Add documentation sections drm/bridge: Document the probe issue with MIPI-DSI bridges drm/mipi-dsi: Create devm device registration drm/mipi-dsi: Create devm device attachment drm/bridge: ps8640: Switch

[PATCH v3 1/8] drm/bridge: Add documentation sections

2021-08-23 Thread Maxime Ripard
The bridge documentation overview is quite packed already, and we'll add some more documentation that isn't part of an overview at all. Let's add some sections to the documentation to separate each bits. Reviewed-by: Sam Ravnborg Signed-off-by: Maxime Ripard --- Documentat

[PATCH v3 2/8] drm/bridge: Document the probe issue with MIPI-DSI bridges

2021-08-23 Thread Maxime Ripard
Interactions between bridges, panels, MIPI-DSI host and the component framework are not trivial and can lead to probing issues when implementing a display driver. Let's document the various cases we need too consider, and the solution to support all the cases. Signed-off-by: Maxime R

[PATCH v3 5/8] drm/bridge: ps8640: Switch to devm MIPI-DSI helpers

2021-08-23 Thread Maxime Ripard
Let's switch to the new devm MIPI-DSI function to register and attach our secondary device. This also avoids leaking the device on removal. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/bridge/parade-ps8640.c | 10 +++--- 1 file changed, 3 insertions(+), 7 deletions(-) diff --

[PATCH v3 7/8] drm/bridge: sn65dsi83: Switch to devm MIPI-DSI helpers

2021-08-23 Thread Maxime Ripard
Let's switch to the new devm MIPI-DSI function to register and attach our secondary device. This also avoids leaking the device when we detach the bridge but don't remove its driver. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/bridge/ti-sn65dsi83.c | 12 +++- 1 file

[PATCH v3 4/8] drm/mipi-dsi: Create devm device attachment

2021-08-23 Thread Maxime Ripard
Signed-off-by: Maxime Ripard --- drivers/gpu/drm/drm_mipi_dsi.c | 35 ++ include/drm/drm_mipi_dsi.h | 1 + 2 files changed, 36 insertions(+) diff --git a/drivers/gpu/drm/drm_mipi_dsi.c b/drivers/gpu/drm/drm_mipi_dsi.c index ddf67463eaa1..18cef04df2f2 100644

[PATCH v3 3/8] drm/mipi-dsi: Create devm device registration

2021-08-23 Thread Maxime Ripard
call to mipi_dsi_device_unregister(). Let's create a device-managed variant of the registration function that will automatically unregister the device at unbind. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/drm_mipi_dsi.c | 46 ++ include/drm/drm_mipi_

[PATCH v3 6/8] drm/bridge: ps8640: Register and attach our DSI device at probe

2021-08-23 Thread Maxime Ripard
In order to avoid any probe ordering issue, the best practice is to move the secondary MIPI-DSI device registration and attachment to the MIPI-DSI host at probe time. Let's do this. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/bridge/parade-ps8640.c | 93 ++ 1

[PATCH v3 8/8] drm/bridge: sn65dsi83: Register and attach our DSI device at probe

2021-08-23 Thread Maxime Ripard
In order to avoid any probe ordering issue, the best practice is to move the secondary MIPI-DSI device registration and attachment to the MIPI-DSI host at probe time. Let's do this. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/bridge/ti-sn65dsi83.c | 76 +++ 1

Re: [PATCH 10/54] dt-bindings: display: panel-lvds: Document panel compatibles

2021-08-23 Thread Maxime Ripard
Hi, On Wed, Aug 18, 2021 at 08:48:46AM -0500, Rob Herring wrote: > On Wed, Aug 18, 2021 at 7:43 AM Maxime Ripard wrote: > > > > Hi Rob, Sam, > > > > On Wed, Jul 21, 2021 at 08:29:47PM -0600, Rob Herring wrote: > > > On Wed, Jul 21, 2021 at 04:03:40PM +02

Re: [GIT PULL] drm-misc + drm-intel: Add support for out-of-band hotplug notification

2021-08-26 Thread Maxime Ripard
On Wed, Aug 25, 2021 at 04:03:43PM +, Vivi, Rodrigo wrote: > On Tue, 2021-08-24 at 18:48 +0200, Hans de Goede wrote: > > Hi, > > > > On 8/24/21 10:45 AM, Jani Nikula wrote: > > > On Fri, 20 Aug 2021, Hans de Goede wrote: > > > > Hello drm-misc and drm-intel maintainers, > > > > > > > > My "A

Re: [PATCH v3 2/8] drm/bridge: Document the probe issue with MIPI-DSI bridges

2021-08-26 Thread Maxime Ripard
Hi Andrzej, On Mon, Aug 23, 2021 at 06:32:11PM +0200, Andrzej Hajda wrote: > Hi Maxime, > > On 23.08.2021 10:47, Maxime Ripard wrote: > > > Interactions between bridges, panels, MIPI-DSI host and the component > > framework are not trivial and can lead to probing issues

[PULL] drm-misc-next

2021-03-16 Thread Maxime Ripard
Hi, It's been a fairly quiet week, but here are the changes for drm-misc-next Thanks! Maxime drm-misc-next-2021-03-16: drm-misc-next for 5.13: UAPI Changes: - Documentation for the DRM_CAP_* defines Cross-subsystem Changes: Core Changes: - dma-fence: More documentation for the recoverable

Re: Need -fixes commit in drm-misc-next

2021-03-16 Thread Maxime Ripard
Hi! On Tue, Mar 09, 2021 at 06:08:49PM +0100, Noralf Trønnes wrote: > Hi drm-misc maintainers, > > I have this series: > > GUD USB Display driver > https://patchwork.freedesktop.org/series/87044/#rev3 > > That depends on this drm-misc-fixes commit: > > 3a3fe21242a3 ("drm: Use USB controller's

Re: [PATCH 1/8] clk: Add range accessors

2021-03-17 Thread Maxime Ripard
On Tue, Mar 16, 2021 at 06:06:40PM -0700, Stephen Boyd wrote: > Quoting Maxime Ripard (2021-03-03 00:45:27) > > Hi Stephen, > > > > On Tue, Mar 02, 2021 at 03:18:58PM -0800, Stephen Boyd wrote: > > > Quoting Maxime Ripard (2021-02-25 07:59:02) > > > > Som

[PATCH 04/18] drm/vc4: txp: Properly set the possible_crtcs mask

2021-03-17 Thread Maxime Ripard
The current code does a binary or on the possible_crtcs variable of the TXP encoder, while we want to set it to that value instead. Fixes: 39fcb2808376 ("drm/vc4: txp: Turn the TXP into a CRTC of its own") Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_txp.c | 2 +- 1 file

[PATCH 00/18] drm/vc4: hdmi: Add Support for the YUV output

2021-03-17 Thread Maxime Ripard
know what you think, Maxime Maxime Ripard (18): drm: Introduce new HDMI helpers drm/bridge: Add HDMI output fmt helper drm/bridge: dw-hdmi: Use helpers drm/vc4: txp: Properly set the possible_crtcs mask drm/vc4: crtc: Skip the TXP drm/vc4: Rework the encoder retrieval code drm/vc4: hdmi

[PATCH 06/18] drm/vc4: Rework the encoder retrieval code

2021-03-17 Thread Maxime Ripard
d, we can iterate over all the encoders to find their associated CRTC and set the pointer. This is similar to what we're doing to set the possible_crtcs field. Fixes: 875a4d536842 ("drm/vc4: drv: Disable the CRTC at boot time") Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4

[PATCH 03/18] drm/bridge: dw-hdmi: Use helpers

2021-03-17 Thread Maxime Ripard
Signed-off-by: Maxime Ripard --- drivers/gpu/drm/bridge/synopsys/dw-hdmi.c | 141 +- 1 file changed, 28 insertions(+), 113 deletions(-) diff --git a/drivers/gpu/drm/bridge/synopsys/dw-hdmi.c b/drivers/gpu/drm/bridge/synopsys/dw-hdmi.c index d010c9c525d9..39b380453183 100644

[PATCH 02/18] drm/bridge: Add HDMI output fmt helper

2021-03-17 Thread Maxime Ripard
already has that code done in a fairly generic manner, so let's turn that code into an helper for all the HDMI controllers to reuse. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/bridge/synopsys/dw-hdmi.c | 127 -- drivers/gpu/drm/drm_bridge.c

[PATCH 01/18] drm: Introduce new HDMI helpers

2021-03-17 Thread Maxime Ripard
The new bridge rework to support the input and output formats introduced some boilerplate code that will need to be shared across drivers. Since dw-hdmi is the only driver so far, let's introduce those helpers based on that code. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/Mak

[PATCH 08/18] drm/vc4: hdmi: Use full range helper in csc functions

2021-03-17 Thread Maxime Ripard
er to tell whether the full or limited range RGB should be used. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c | 14 -- drivers/gpu/drm/vc4/vc4_hdmi.h | 3 ++- 2 files changed, 10 insertions(+), 7 deletions(-) diff --git a/drivers/gpu/drm/vc4/vc4_hdmi.c b/drivers/g

[PATCH 10/18] drm/vc4: hdmi: Convert to bridge

2021-03-17 Thread Maxime Ripard
Converting the HDMI controller to a bridge seems like the preferred way to support an YUV output, so let's do this. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_crtc.c | 37 ++- drivers/gpu/drm/vc4/vc4_drv.c | 15 +++-- drivers/gpu/drm/vc4/vc4_drv.h

[PATCH 09/18] drm/vc4: hdmi: Remove limited_rgb_range

2021-03-17 Thread Maxime Ripard
The limited_rgb_range field in the vc4_hdmi_encoder structure is used to tell whether we're supposed to output with a full or limited RGB range. This is redundant with the new helper we introduced, so let's convert to that helper and drop that field. Signed-off-by: Maxime Ripard --

[PATCH 05/18] drm/vc4: crtc: Skip the TXP

2021-03-17 Thread Maxime Ripard
08376 ("drm/vc4: txp: Turn the TXP into a CRTC of its own") Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_crtc.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/gpu/drm/vc4/vc4_crtc.c b/drivers/gpu/drm/vc4/vc4_crtc.c index 269390bc586e..f1f2e8cbce79 100644

[PATCH 07/18] drm/vc4: hdmi: Add full range RGB helper

2021-03-17 Thread Maxime Ripard
We're going to need to tell whether we want to run with a full or limited range RGB output in multiple places in the code, so let's create a helper that will return whether we need with full range or not. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c | 12 ++

[PATCH 11/18] drm/vc4: hdmi: Move XBAR setup to csc_setup

2021-03-17 Thread Maxime Ripard
On the BCM2711, the HDMI_VEC_INTERFACE_XBAR register configuration depends on whether we're using an RGB or YUV output. Let's move that configuration to the CSC setup. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletio

[PATCH 14/18] drm/vc4: hdmi: Change CSC callback prototype

2021-03-17 Thread Maxime Ripard
In order to support the YUV output, we'll need the atomic state to know what is the state of the associated property in the CSC setup callback. Let's change the prototype of that callback to allow us to access it. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_

[PATCH 12/18] drm/vc4: hdmi: Replace CSC_CTL hardcoded value by defines

2021-03-17 Thread Maxime Ripard
On BCM2711, the HDMI_CSC_CTL register value has been hardcoded to an opaque value. Let's replace it with properly defined values. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c | 5 ++--- drivers/gpu/drm/vc4/vc4_regs.h | 3 +++ 2 files changed, 5 insertions(+), 3 dele

[PATCH 13/18] drm/vc4: hdmi: Define colorspace matrices

2021-03-17 Thread Maxime Ripard
hould switch to something a bit more flexible that takes the matrix as an argument and programs the CSC accordingly. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c | 77 +- 1 file changed, 48 insertions(+), 29 deletions(-) diff --git a/drivers/gpu/dr

[PATCH 16/18] drm/vc4: hdmi: Support HDMI YUV output

2021-03-17 Thread Maxime Ripard
The HDMI controllers in the BCM2711 support YUV444 and YUV420 outputs, let's add support for it. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c | 73 +++-- drivers/gpu/drm/vc4/vc4_hdmi_regs.h | 6 +++ drivers/gpu/drm/vc4/vc4_regs.h

[PATCH 15/18] drm/vc4: hdmi: Rework the infoframe prototype

2021-03-17 Thread Maxime Ripard
ed the vc4_hdmi structure, let's pass it instead of the drm_encoder. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c | 38 -- 1 file changed, 18 insertions(+), 20 deletions(-) diff --git a/drivers/gpu/drm/vc4/vc4_hdmi.c b/drivers/gpu/drm/

[PATCH 18/18] drm/vc4: hdmi: Force YUV422 if the rate is too high

2021-03-17 Thread Maxime Ripard
ff-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c | 37 +- 1 file changed, 36 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/vc4/vc4_hdmi.c b/drivers/gpu/drm/vc4/vc4_hdmi.c index c4f91d39d91c..12eda1e76338 100644 --- a/drivers/gpu/drm/vc4/vc4_hdmi.c

[PATCH 17/18] drm/vc4: hdmi: Move the pixel rate calculation to a helper

2021-03-17 Thread Maxime Ripard
In order to implement a fallback mechanism to YUV422 when the pixel rate is too high, let's move the pixel rate computation to a function of its own that will be shared across two functions. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c

[PATCH v2 0/5] drm/vc4: hdmi: Support the 4k @ 60Hz modes

2021-03-18 Thread Maxime Ripard
enable_hdmi_4kp60 only if there's some modes that can't be reached - Rework the BVB clock computation Maxime Ripard (5): drm/vc4: hvs: Make the HVS bind first drm/vc4: hdmi: Properly compute the BVB clock rate drm/vc4: hdmi: Check and warn if we can't reach 4kp60 freque

[PATCH v2 1/5] drm/vc4: hvs: Make the HVS bind first

2021-03-18 Thread Maxime Ripard
We'll need to have the HVS binding before the HDMI controllers so that we can check whether the firmware allows to run in 4kp60. Reorder a bit the component list, and document the current constraints we're aware of. Acked-by: Dave Stevenson Signed-off-by: Maxime Ripard --- drivers/g

[PATCH v2 4/5] drm/vc4: hdmi: Enable the scrambler

2021-03-18 Thread Maxime Ripard
The HDMI controller on the BCM2711 includes a scrambler in order to reach the modes that require it. Let's add the support for it. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c | 56 + drivers/gpu/drm/vc4/vc4_hdmi_regs.h | 3 ++ 2

[PATCH v2 5/5] drm/vc4: hdmi: Raise the maximum clock rate

2021-03-18 Thread Maxime Ripard
Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/vc4/vc4_hdmi.c b/drivers/gpu/drm/vc4/vc4_hdmi.c index 530c83097b1a..a31771dd09e6 100644 --- a/drivers/gpu/drm/vc4/vc4_hdmi.c +++ b/drivers/gpu/drm/vc4

[PATCH v2 3/5] drm/vc4: hdmi: Check and warn if we can't reach 4kp60 frequencies

2021-03-18 Thread Maxime Ripard
ned-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c | 26 ++ drivers/gpu/drm/vc4/vc4_hdmi.h | 8 2 files changed, 34 insertions(+) diff --git a/drivers/gpu/drm/vc4/vc4_hdmi.c b/drivers/gpu/drm/vc4/vc4_hdmi.c index eaee853bb404..0924a1b9e186 100644 --- a/d

[PATCH v2 2/5] drm/vc4: hdmi: Properly compute the BVB clock rate

2021-03-18 Thread Maxime Ripard
The BVB clock rate computation doesn't take into account a mode clock of 594MHz that we're going to need to support 4k60. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c | 17 + 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/drivers/g

[PATCH 2/3] drm/vc4: plane: Remove redundant assignment

2021-03-18 Thread Maxime Ripard
The vc4_plane_atomic_async_update function assigns twice in a row the src_h field in the drm_plane_state structure to the same value. Remove the second one. Reviewed-by: Dave Stevenson Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_plane.c | 1 - 1 file changed, 1 deletion(-) diff

[PATCH 3/3] drm/vc4: crtc: Reduce PV fifo threshold on hvs4

2021-03-18 Thread Maxime Ripard
he PV FIFO is full" Fixes: c8b75bca92cb ("drm/vc4: Add KMS support for Raspberry Pi.") Signed-off-by: Dom Cobley Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_crtc.c | 17 + 1 file changed, 17 insertions(+) diff --git a/drivers/gpu/drm/vc4/vc4_crtc.c b

[PATCH 1/3] drm/vc4: plane: Fix typo in scaler width and height

2021-03-18 Thread Maxime Ripard
) Reviewed-by: Dave Stevenson Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_plane.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/vc4/vc4_plane.c b/drivers/gpu/drm/vc4/vc4_plane.c index eb37d7f8a118..3fdc32244b59 100644 --- a/drivers/g

Re: [PATCH 02/18] drm/bridge: Add HDMI output fmt helper

2021-03-19 Thread Maxime Ripard
On Fri, Mar 19, 2021 at 10:44:56AM +0100, Neil Armstrong wrote: > On 18/03/2021 19:31, Jernej Škrabec wrote: > > Dne sreda, 17. marec 2021 ob 17:08:07 CET je Neil Armstrong napisal(a): > >> On 17/03/2021 16:43, Maxime Ripard wrote: > >>> The atomic_get_output_bus_fm

Re: [PATCH 00/18] drm/vc4: hdmi: Add Support for the YUV output

2021-03-19 Thread Maxime Ripard
Hi Jernej, On Thu, Mar 18, 2021 at 07:16:33PM +0100, Jernej Škrabec wrote: > Dne sreda, 17. marec 2021 ob 16:43:34 CET je Maxime Ripard napisal(a): > > Hi, > > > > Here's an attempt at support the HDMI YUV output on the BCM2711 SoC found on > > the RaspberryPi4. &

[PATCH 3/3] drm/vc4: Add HDR metadata property to the VC5 HDMI connectors

2021-03-19 Thread Maxime Ripard
From: Dave Stevenson Now that we can export deeper colour depths, add in the signalling for HDR metadata. Signed-off-by: Dave Stevenson Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c | 53 ++ drivers/gpu/drm/vc4/vc4_hdmi.h | 3 ++ 2 files

[PATCH 1/3] drm/connector: Create a helper to attach the hdr_output_metadata property

2021-03-19 Thread Maxime Ripard
All the drivers that implement HDR output call pretty much the same function to initialise the hdr_output_metadata property, and while the creation of that property is in a helper, every driver uses the same code to attach it. Provide a helper for it as well Signed-off-by: Maxime Ripard

[PATCH 2/3] drm/connector: Add helper to compare HDR metadata

2021-03-19 Thread Maxime Ripard
those drivers. Signed-off-by: Maxime Ripard --- .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 21 +- drivers/gpu/drm/bridge/synopsys/dw-hdmi.c | 17 +-- drivers/gpu/drm/drm_connector.c | 28 +++ drivers/gpu/drm/i915/display/intel_atomic.c

[PATCH] drm/rockchip: Remove unused variable

2021-03-19 Thread Maxime Ripard
Remove it. Fixes: 977697e20b3d ("drm/atomic: Pass the full state to planes atomic disable and update") Reported-by: Stephen Rothwell Signed-off-by: Maxime Ripard --- drivers/gpu/drm/rockchip/rockchip_drm_vop.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/rockchip/r

Re: [PATCH] drm/rockchip: Remove unused variable

2021-03-24 Thread Maxime Ripard
On Sat, Mar 20, 2021 at 02:20:56AM +0200, Laurent Pinchart wrote: > Hi Maxime, > > Thank you for the patch. > > On Fri, Mar 19, 2021 at 04:29:20PM +0100, Maxime Ripard wrote: > > Commit 977697e20b3d ("drm/atomic: Pass the full state to planes atomic > > disable

Re: [PATCH v4 1/4] drm: sun4i: dsi: Use drm_of_find_panel_or_bridge

2021-03-24 Thread Maxime Ripard
On Wed, Mar 24, 2021 at 11:55:35AM +0200, Laurent Pinchart wrote: > Hi Jagan, > > On Wed, Mar 24, 2021 at 03:19:10PM +0530, Jagan Teki wrote: > > On Wed, Mar 24, 2021 at 3:09 PM Laurent Pinchart wrote: > > > On Wed, Mar 24, 2021 at 02:44:57PM +0530, Jagan Teki wrote: > > > > On Wed, Mar 24, 2021 a

Re: [PATCH v4 2/6] drm/sprd: add Unisoc's drm kms master

2021-03-24 Thread Maxime Ripard
Hi On Mon, Feb 22, 2021 at 09:28:18PM +0800, Kevin Tang wrote: > Adds drm support for the Unisoc's display subsystem. > > This is drm kms driver, this driver provides support for the > application framework in Android, Yocto and more. > > Application framework can access Unisoc's display interne

Re: [PATCH v4 4/6] drm/sprd: add Unisoc's drm display controller driver

2021-03-24 Thread Maxime Ripard
Hi, On Mon, Feb 22, 2021 at 09:28:20PM +0800, Kevin Tang wrote: > Adds DPU(Display Processor Unit) support for the Unisoc's display subsystem. > It's support multi planes, scaler, rotation, PQ(Picture Quality) and more. > > Cc: Orson Zhai > Cc: Chunyan Zhang > Signed-off-by: Kevin Tang > > v2

Re: [PATCH v4 5/6] dt-bindings: display: add Unisoc's mipi dsi controller bindings

2021-03-24 Thread Maxime Ripard
On Mon, Feb 22, 2021 at 09:28:21PM +0800, Kevin Tang wrote: > From: Kevin Tang > > Adds MIPI DSI Controller > support for Unisoc's display subsystem. > > Cc: Orson Zhai > Cc: Chunyan Zhang > Signed-off-by: Kevin Tang > Reviewed-by: Rob Herring > --- > .../display/sprd/sprd,sharkl3-dsi-host.

Re: [PATCH v4 6/6] drm/sprd: add Unisoc's drm mipi dsi&dphy driver

2021-03-24 Thread Maxime Ripard
On Mon, Feb 22, 2021 at 09:28:22PM +0800, Kevin Tang wrote: > Adds dsi host controller support for the Unisoc's display subsystem. > Adds dsi phy support for the Unisoc's display subsystem. > Only MIPI DSI Displays supported, DP/TV/HMDI will be support > in the feature. > > v1: > - Remove dphy a

[PULL] drm-misc-next

2021-03-25 Thread Maxime Ripard
idge/analogix/anx6345: Cleanup on errors in anx6345_bridge_attach() drm/bridge/analogix/dp_core: Unregister DP AUX channel on error in analogix_dp_probe() Matthew Wilcox (Oracle) (1): fb_defio: Remove custom address_space_operations Maxime Ripard (2): Merge drm/drm-next into

Re: [PATCH v5 09/10] drm: rcar-du: Perform group setup from the atomic tail handler

2021-03-26 Thread Maxime Ripard
Hi Kieran, On Mon, Mar 22, 2021 at 04:35:34PM +, Kieran Bingham wrote: > Create rcar_du_group_atomic_check() and rcar_du_group_atomic_setup() > functions to track and apply group state through the DRM atomic state. > The use_count field is moved from the rcar_du_group structure to an > enabled

Re: [Intel-gfx] [PATCH v5 1/2] drm/dp_helper: Define options for FRL training for HDMI2.1 PCON

2021-03-26 Thread Maxime Ripard
Hi, On Fri, Mar 26, 2021 at 11:47:58AM +0200, Jani Nikula wrote: > On Tue, 23 Mar 2021, Ankit Nautiyal wrote: > > Currently the FRL training mode (Concurrent, Sequential) and > > training type (Normal, Extended) are not defined properly and > > are passed as bool values in drm_helpers for pcon >

Re: [PATCH] drm: DON'T require each CRTC to have a unique primary plane

2021-03-29 Thread Maxime Ripard
On Sat, Mar 27, 2021 at 11:22:14AM +, Paul Cercueil wrote: > The ingenic-drm driver has two mutually exclusive primary planes > already; so the fact that a CRTC must have one and only one primary > plane is an invalid assumption. I mean, no? It's been documented for a while that a CRTC should

Re: [PATCH] drm: DON'T require each CRTC to have a unique primary plane

2021-03-29 Thread Maxime Ripard
On Mon, Mar 29, 2021 at 04:15:28PM +0100, Paul Cercueil wrote: > Hi Maxime, > > Le lun. 29 mars 2021 à 16:07, Maxime Ripard a écrit : > > On Sat, Mar 27, 2021 at 11:22:14AM +, Paul Cercueil wrote: > > > The ingenic-drm driver has two mutually exclusive primary plane

Re: [PATCH] dt-bindings: bcm2711-hdmi: Fix broken schema

2021-03-30 Thread Maxime Ripard
Hi Stephen, On Sat, Mar 27, 2021 at 10:00:26AM +1100, Stephen Rothwell wrote: > Hi Rob, > > On Fri, 26 Mar 2021 15:01:34 -0600 Rob Herring wrote: > > > > On Tue, Feb 23, 2021 at 2:26 PM Rob Herring wrote: > > > > > > On Thu, 18 Feb 2021 16:28:37 +0100,

Re: [PATCH v3 10/11] drm: Use state helper instead of the plane state pointer

2021-03-30 Thread Maxime Ripard
Hi Stephen, On Mon, Mar 29, 2021 at 06:52:01PM -0700, Stephen Boyd wrote: > Trimming Cc list way down, sorry if that's too much. > > Quoting Maxime Ripard (2021-02-19 04:00:30) > > Many drivers reference the plane->state pointer in order to get the > > current plane

[PULL] drm-misc-next

2021-04-01 Thread Maxime Ripard
analogix/dp_core: Unregister DP AUX channel on error in analogix_dp_probe() Matthew Wilcox (Oracle) (1): fb_defio: Remove custom address_space_operations Maxime Ripard (2): Merge drm/drm-next into drm-misc-next drm/rockchip: Remove unused variable Noralf Trønnes (3): drm/uap

Re: [PATCH RFC] drm/vc4: hdmi: Avoid ASoC error messages on startup

2021-01-14 Thread Maxime Ripard
Hi Stefan, On Sat, Jan 09, 2021 at 12:41:57PM +0100, Stefan Wahren wrote: > Hi Maxime, > > Am 29.12.20 um 16:36 schrieb Stefan Wahren: > > During startup of Raspberry Pi 4 there seems to be a race between > > VC4 probing and Pulseaudio trying to open its PCM device: > > > > ASoC: error at snd

Re: [PATCH V3] dt-bindings: gpu: Convert v3d to json-schema

2021-01-14 Thread Maxime Ripard
Hi, On Sat, Jan 09, 2021 at 11:50:32AM +0100, Stefan Wahren wrote: > This converts the v3d bindings to yaml format. > > Signed-off-by: Stefan Wahren > --- > > Changes in V3: > - drop redundant maxItems in case we already have items defined > - fix order of reg-names enum > - tag required items

Re: [PATCH 0/3] drm/sun4i: de2/de3: CSC improvements

2021-01-14 Thread Maxime Ripard
On Sun, Jan 10, 2021 at 09:19:44PM +0100, Jernej Skrabec wrote: > This short series reworks CSC handling to remove duplicated constants > (patch 1 and 2) and adds BT2020 encoding to DE3 (patch 3). > > Please take a look. Applied, thanks Maxime signature.asc Description: PGP signature __

Re: [PATCH v3] drm/sun4i: de2: Reimplement plane z position setting logic

2021-01-14 Thread Maxime Ripard
Hi, On Wed, Jan 06, 2021 at 09:46:30PM +0100, Jernej Skrabec wrote: > From: Roman Stratiienko > > To set blending channel order register software needs to know state and > position of each channel, which impossible at plane commit stage. > > Move this procedure to atomic_flush stage, where all

Re: [PATCH V3] dt-bindings: gpu: Convert v3d to json-schema

2021-01-14 Thread Maxime Ripard
On Wed, Jan 13, 2021 at 01:53:38PM +0100, Stefan Wahren wrote: > Hi Maxime, > > Am 13.01.21 um 10:15 schrieb Maxime Ripard: > > Hi, > > > > On Sat, Jan 09, 2021 at 11:50:32AM +0100, Stefan Wahren wrote: > >> This converts the v3d bindings to yaml format. &g

Re: [PATCH RFT 0/2] drm/vc4: Clear the display list on startup

2021-01-14 Thread Maxime Ripard
Hi Ryutaroh, Lucas, On Wed, Jan 13, 2021 at 09:51:59AM +0900, Ryutaroh Matsumoto wrote: > Hi Lucas, > > > week-end, so I cannot test before early next week. However I'm Ccing > > Ryutaroh Matsumoto who could also reproduce it. Maybe he is in a better > > position to test this (@Ryutaroh: I bounce

Re: [PATCH v3] drm/sun4i: tcon: fix inverted DCLK polarity

2021-01-14 Thread Maxime Ripard
ther signals polarity. > > Fixes: 88bc4178568b ("drm: Use new > DRM_BUS_FLAG_*_(DRIVE|SAMPLE)_(POS|NEG)EDGE flags") > Suggested-by: Maxime Ripard > Signed-off-by: Giulio Benetti > --- > drivers/gpu/drm/sun4i/sun4i_tcon.c | 20 +--- > drivers/gpu/drm/sun4i/su

Re: [PATCH V4] dt-bindings: gpu: Convert v3d to json-schema

2021-01-15 Thread Maxime Ripard
On Wed, Jan 13, 2021 at 08:08:37PM +0100, Stefan Wahren wrote: > This converts the v3d bindings to yaml format. > > Signed-off-by: Stefan Wahren Acked-by: Maxime Ripard Thanks! Maxime signature.asc Description: PGP signature ___ dri-deve

Re: [PATCH v6] drm/sun4i: tcon: fix inverted DCLK polarity

2021-01-15 Thread Maxime Ripard
ther signals polarity. > > Fixes: 88bc4178568b ("drm: Use new > DRM_BUS_FLAG_*_(DRIVE|SAMPLE)_(POS|NEG)EDGE flags") > Suggested-by: Maxime Ripard > Signed-off-by: Giulio Benetti > --- > V2->V3: > - squash 2 patches into 1 > V3->V4: > - add SUN4I_TCON0_IO_POL_DCLK_

Re: [PATCH] drm/vc4: Initialize vc4_drm_driver with CMA helper defaults

2021-01-15 Thread Maxime Ripard
initializer > macro. > > Signed-off-by: Thomas Zimmermann Acked-by: Maxime Ripard Thanks! Maxime signature.asc Description: PGP signature ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel

[PATCH 07/10] drm: Store new plane state in a variable for atomic_update and disable

2021-01-16 Thread Maxime Ripard
old_s; @@ func(struct drm_plane *plane, struct drm_plane_state *old_s) { + struct drm_plane_state *new_s = plane->state; <+... - plane->state + new_s ...+> } Signed-off-by: Maxime Ripard --- drivers/gpu/drm/arc/arcpgu_crtc.c | 7 ++-

[PATCH 05/10] drm: Use the state pointer directly in planes atomic_check

2021-01-16 Thread Maxime Ripard
e *plane_state = drm_atomic_get_new_plane_state(state, plane); <... - plane_state->state + state ...> } Signed-off-by: Maxime Ripard --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 2 +- drivers/gpu/drm/arm/display/komeda/komeda_plane.c | 2 +- drivers/gpu/drm/arm/hdlcd_crtc.c

[PATCH 01/10] drm/atomic: Pass the full state to planes async atomic check and update

2021-01-16 Thread Maxime Ripard
); <+... - plane_state->state + state ...+> } Signed-off-by: Maxime Ripard --- .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 8 ++- drivers/gpu/drm/drm_atomic_helper.c | 4 +- drivers/gpu/drm/mediatek/mtk_drm_plane.c | 26 + drivers/gpu/drm/msm/di

[PATCH 09/10] drm/atomic: Pass the full state to planes atomic disable and update

2021-01-16 Thread Maxime Ripard
lt;+... - plane_state->state + state ...+> } Signed-off-by: Maxime Ripard --- drivers/gpu/drm/arc/arcpgu_crtc.c | 2 +- .../gpu/drm/arm/display/komeda/komeda_plane.c | 2 +- drivers/gpu/drm/arm/hdlcd_crtc.c | 2 +- drivers/gpu/drm/arm/malidp_planes.c

[PATCH 06/10] drm: Use state helper instead of plane state pointer in atomic_check

2021-01-16 Thread Maxime Ripard
plane->state + old_plane_state ... } @ include depends on adds_old_state || replaces_old_state @ @@ #include @ no_include depends on !include && (adds_old_state || replaces_old_state) @ @@ + #include #include Signed-off-by: Maxime Ripard --- drivers/gpu/drm/imx/ipuv3-plane.c

Re: [PATCH 02/10] drm: Rename plane atomic_check state names

2021-01-16 Thread Maxime Ripard
Hi, On Fri, Jan 15, 2021 at 02:46:36PM +0100, Thomas Zimmermann wrote: > Hi > > Am 15.01.21 um 13:56 schrieb Maxime Ripard: > > diff --git a/drivers/gpu/drm/imx/ipuv3-plane.c > > b/drivers/gpu/drm/imx/ipuv3-plane.c > > index 8a4235d9d9f1..2cb09e9d9306 100644 > >

[PATCH 08/10] drm: Rename plane->state variables in atomic update and disable

2021-01-16 Thread Maxime Ripard
func(struct drm_plane *plane, struct drm_plane_state *old_pstate) { <... - state + new_pstate ...> } Signed-off-by: Maxime Ripard --- drivers/gpu/drm/arm/malidp_planes.c | 34 +++--- drivers/gpu/drm/armada/armada_overlay.c | 104 +

[PATCH 10/10] drm: Use state helper instead of the plane state pointer

2021-01-16 Thread Maxime Ripard
Signed-off-by: Maxime Ripard --- drivers/gpu/drm/arc/arcpgu_crtc.c | 4 +++- drivers/gpu/drm/arm/hdlcd_crtc.c| 3 ++- drivers/gpu/drm/arm/malidp_planes.c | 3 ++- drivers/gpu/drm/armada/armada_overlay.c | 3 ++- drivers/gpu/drm/armada/armada_plane.c

[PATCH 04/10] drm/atomic: Pass the full state to planes atomic_check

2021-01-16 Thread Maxime Ripard
s on !include && adds_new_state @ @@ + #include #include Signed-off-by: Maxime Ripard --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 4 +++- drivers/gpu/drm/arm/display/komeda/komeda_plane.c | 4 +++- drivers/gpu/drm/arm/hdlcd_crtc.c | 4 +++- drivers/gpu/dr

<    1   2   3   4   5   6   7   8   9   10   >