[PATCH] fbdev/efifb: Release PCI device's runtime PM ref during FB destroy

2021-08-02 Thread Imre Deak
ed. Fixes: a6c0fd3d5a8b ("efifb: Ensure graphics device for efifb stays at PCI D0") Cc: Kai-Heng Feng Signed-off-by: Imre Deak --- drivers/video/fbdev/efifb.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/drivers/video/fbdev/efifb.c b/drivers/video/fbdev/efifb.c

Re: linux-next: build warning after merge of the drm-intel-fixes tree

2021-04-08 Thread Imre Deak
On Thu, Apr 08, 2021 at 12:38:52PM +0200, Daniel Vetter wrote: > On Mon, Mar 29, 2021 at 09:23:35PM +0300, Imre Deak wrote: > > Hi Stephen, > > > > thanks for the report. > > > > On Mon, Mar 29, 2021 at 09:01:17AM +1100, Stephen Rothwell wrote: > > > H

Re: [Intel-gfx] [PATCH] fbdev/efifb: Release PCI device's runtime PM ref during FB destroy

2021-08-07 Thread Imre Deak
On Thu, Aug 05, 2021 at 12:23:21AM +0200, Daniel Vetter wrote: > On Mon, Aug 02, 2021 at 04:35:51PM +0300, Imre Deak wrote: > > Atm the EFI FB driver gets a runtime PM reference for the associated GFX > > PCI device during driver probing and releases it only when removing

[PATCH v2] fbdev/efifb: Release PCI device's runtime PM ref during FB destroy

2021-08-09 Thread Imre Deak
't fail. - Clarify commit message wrt. platform/PCI device/driver and driver removal vs. device unbinding. Fixes: a6c0fd3d5a8b ("efifb: Ensure graphics device for efifb stays at PCI D0") Cc: Kai-Heng Feng Cc: Daniel Vetter Reviewed-by: Daniel Vetter (v1) Signed-off-by: Im

Re: [Intel-gfx] [PATCH v2] fbdev/efifb: Release PCI device's runtime PM ref during FB destroy

2021-08-10 Thread Imre Deak
Hi Kai-Heng, Alex, could you add your ack if the fix looks ok and you're ok if I push it to the i915 tree? Thanks, Imre On Mon, Aug 09, 2021 at 04:31:46PM +0300, Imre Deak wrote: > Atm the EFI FB platform driver gets a runtime PM reference for the > associated GFX PCI device during

Re: [Intel-gfx] [PATCH v2] fbdev/efifb: Release PCI device's runtime PM ref during FB destroy

2021-08-13 Thread Imre Deak
On Mon, Aug 09, 2021 at 04:31:46PM +0300, Imre Deak wrote: > Atm the EFI FB platform driver gets a runtime PM reference for the > associated GFX PCI device during probing the EFI FB platform device and > releases it only when the platform device gets unbound. > > When fbcon swi

Re: linux-next: build warning after merge of the drm-intel-fixes tree

2021-03-29 Thread Imre Deak
Hi Stephen, thanks for the report. On Mon, Mar 29, 2021 at 09:01:17AM +1100, Stephen Rothwell wrote: > Hi all, > > On Fri, 26 Mar 2021 19:58:38 +1100 Stephen Rothwell > wrote: > > > > After merging the drm-intel-fixes tree, today's linux-next build > > (htmldocs) produced this warning: > > >

[PATCH 1/2] drm/dp/mst: Export drm_dp_get_vc_payload_bw()

2021-01-25 Thread Imre Deak
: Cc: dri-devel@lists.freedesktop.org Signed-off-by: Imre Deak --- drivers/gpu/drm/drm_dp_mst_topology.c | 24 ++-- include/drm/drm_dp_mst_helper.h | 1 + 2 files changed, 19 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/drm_dp_mst_topology.c b/drivers/gpu

Re: ✓ Fi.CI.IGT: success for series starting with [1/2] drm/dp/mst: Export drm_dp_get_vc_payload_bw()

2021-01-28 Thread Imre Deak
On Tue, Jan 26, 2021 at 01:28:09AM +, Patchwork wrote: > == Series Details == > > Series: series starting with [1/2] drm/dp/mst: Export > drm_dp_get_vc_payload_bw() > URL : https://patchwork.freedesktop.org/series/86267/ > State : success Patchset pushed to -din with the docbook fix, thank

[PATCH 1/4] drm/dp_mst: Don't report ports connected if nothing is attached to them

2021-02-01 Thread Imre Deak
ts.freedesktop.org Signed-off-by: Imre Deak --- drivers/gpu/drm/drm_dp_mst_topology.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/drm_dp_mst_topology.c b/drivers/gpu/drm/drm_dp_mst_topology.c index e82b596d646c..deb7995f42fa 100644 --- a/drivers/gpu/drm/drm_dp_mst_topolog

[PATCH 2/4] drm/dp_mst: Don't cache EDIDs for physical ports

2021-02-01 Thread Imre Deak
Caching EDIDs for physical ports prevents updating the EDID if a port gets reconnected via a Connection Status Notification message, fix this. Fixes: db1a07956968 ("drm/dp_mst: Handle SST-only branch device case") Cc: Wayne Lin Cc: Lyude Paul Signed-off-by: Imre Deak --- drive

[PATCH 4/4] drm/dp_mst: Use DP_MST_LOGICAL_PORT_0 instead of magic number

2021-02-01 Thread Imre Deak
Use the macro defined for the first logical port instead of the corresponding magic number. Cc: Lyude Paul Signed-off-by: Imre Deak --- drivers/gpu/drm/drm_dp_mst_topology.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/drm_dp_mst_topology.c b/drivers

[PATCH 3/4] drm/dp_mst: Remove redundant tile property setting

2021-02-01 Thread Imre Deak
drm_get_edid() already updates the tile property since commit 2de3a078497b ("drm/dp: Set the connector's TILE property even for DP SST connectors") so no need to update it after calling this function. Cc: Lyude Paul Signed-off-by: Imre Deak --- drivers/gpu/drm/drm_dp_mst

Re: [PATCH 2/4] drm/dp_mst: Don't cache EDIDs for physical ports

2021-02-02 Thread Imre Deak
On Tue, Feb 02, 2021 at 03:38:16AM +, Lin, Wayne wrote: > [AMD Public Use] > > > -Original Message- > > From: Imre Deak > > Sent: Monday, February 1, 2021 8:02 PM > > To: dri-devel@lists.freedesktop.org > > Cc: Lin, Wayne ; Lyude Paul > > Su

Re: [Intel-gfx] [PATCH 1/4] drm/dp_mst: Don't report ports connected if nothing is attached to them

2021-02-03 Thread Imre Deak
On Mon, Feb 01, 2021 at 02:01:42PM +0200, Imre Deak wrote: > Reporting a port as connected if nothing is attached to them leads to > any i2c transactions on this port trying to use an uninitialized i2c > adapter, fix this. > > Let's account for this case even if branch devices

Re: [PATCH 2/4] drm/dp_mst: Don't cache EDIDs for physical ports

2021-02-04 Thread Imre Deak
On Mon, Feb 01, 2021 at 02:01:43PM +0200, Imre Deak wrote: > Caching EDIDs for physical ports prevents updating the EDID if a port > gets reconnected via a Connection Status Notification message, fix this. > > Fixes: db1a07956968 ("drm/dp_mst: Handle SST-only branch device case&

Re: [PATCH 5.10 078/120] drm/dp/mst: Export drm_dp_get_vc_payload_bw()

2021-02-10 Thread Imre Deak
linux-next. What is the plan here? the export is used by the upstream commit 882554042d138dbc6fb1a43017d0b9c3b38ee5f5 Author: Imre Deak Date: Mon Jan 25 19:36:36 2021 +0200 drm/i915: Fix the MST PBN divider calculation which I can also see now applied to 5.10.15: commit 0fe98e455784a6c11e

[PATCH] drm/dp_mst: Tune down the WARN modesetting a port with full_pbn=0

2021-02-16 Thread Imre Deak
ces: https://gitlab.freedesktop.org/drm/intel/-/issues/1917 Cc: Lyude Paul Signed-off-by: Imre Deak --- drivers/gpu/drm/drm_dp_mst_topology.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/drm_dp_mst_topology.c b/drivers/gpu/drm/drm_dp_mst_topology.c index 0a5

Re: [Intel-gfx] [PATCH v2] drm/i915/gen9bc: Handle TGP PCH during suspend/resume

2021-02-16 Thread Imre Deak
Hi, thanks for respinning this patchset, some comments below. On Fri, Feb 12, 2021 at 01:50:53PM -0500, Lyude Paul wrote: > From: Tejas Upadhyay > > For Legacy S3 suspend/resume GEN9 BC needs to enable and > setup TGP PCH. > > v2: > * Move Wa_14010685332 into it's own function - vsyrjala > * A

Re: [Intel-gfx] [PATCH v2] drm/i915/gen9bc: Handle TGP PCH during suspend/resume

2021-02-16 Thread Imre Deak
On Tue, Feb 16, 2021 at 09:36:01PM -0500, Lyude Paul wrote: > On Tue, 2021-02-16 at 20:08 +0200, Imre Deak wrote: > > Hi, > > > > thanks for respinning this patchset, some comments below. > > > > On Fri, Feb 12, 2021 at 01:50:53PM -0500, Lyude Paul

Re: [PATCH v3 1/2] drm/i915/gen9bc: Handle TGP PCH during suspend/resume

2021-02-17 Thread Imre Deak
On Tue, Feb 16, 2021 at 09:53:36PM -0500, Lyude Paul wrote: > From: Tejas Upadhyay > > For Legacy S3 suspend/resume GEN9 BC needs to enable and > setup TGP PCH. > > v2: > * Move Wa_14010685332 into it's own function - vsyrjala > * Add TODO comment about figuring out if we can move this workaroun

Re: [PATCH] drm/dp_mst: Tune down the WARN modesetting a port with full_pbn=0

2021-02-17 Thread Imre Deak
On Wed, Feb 17, 2021 at 01:07:11PM -0500, Lyude Paul wrote: > On Tue, 2021-02-16 at 14:34 +0200, Imre Deak wrote: > > It's possible to modeset a connector/mst port that has a 0 full_pbn > > value: if the sink on the port deasserts its HPD and a branch device > > repor

Re: [PATCH v4] drm/i915/gen9bc: Handle TGP PCH during suspend/resume

2021-02-17 Thread Imre Deak
ch. > v4: > * Revert Wa_14010685332 system list in comments to how it was before > * Add back HAS_PCH_SPLIT() check before calling ibx_irq_reset() > > Cc: Matt Roper > Signed-off-by: Tejas Upadhyay > Signed-off-by: Lyude Paul Thanks, looks ok to me: Reviewed-by: Imre Deak n

Re: [PATCH 4/9] drm/connector: Add support for out-of-band hotplug notification

2021-05-04 Thread Imre Deak
On Mon, May 03, 2021 at 11:00:20AM +0300, Heikki Krogerus wrote: > Hi Hans, > > On Wed, Apr 28, 2021 at 11:52:52PM +0200, Hans de Goede wrote: > > +/** > > + * struct drm_connector_oob_hotplug_event_data: OOB hotplug event data > > + * > > + * Contains data about out-of-band hotplug events, signal

Re: [Intel-gfx] [PATCH 1/2] drm/framebuffer: Format modifier for Intel Gen 12 render compression with Clear Color

2020-11-27 Thread Imre Deak
Hi Daniel, Jani, is it ok to merge this patch along with 2/2 via the i915 tree? --Imre On Mon, Nov 23, 2020 at 08:26:30PM +0200, Imre Deak wrote: > From: Radhakrishna Sripada > > Gen12 display can decompress surfaces compressed by render engine with > Clear Color, add a new mod

Re: [Intel-gfx] [PATCH 1/2] drm/framebuffer: Format modifier for Intel Gen 12 render compression with Clear Color

2020-11-27 Thread Imre Deak
On Fri, Nov 27, 2020 at 04:19:20PM +0100, Daniel Vetter wrote: > On Fri, Nov 27, 2020 at 04:31:00PM +0200, Imre Deak wrote: > > Hi Daniel, Jani, > > > > is it ok to merge this patch along with 2/2 via the i915 tree? > > Ack from mesa (userspace in general, but

Re: [Intel-gfx] [PATCH 1/2] drm/framebuffer: Format modifier for Intel Gen 12 render compression with Clear Color

2020-12-01 Thread Imre Deak
Hi Nanley, thanks for the review. +Ville, Chris. On Tue, Dec 01, 2020 at 02:18:26AM +0200, Chery, Nanley G wrote: > Hi Imre, > > I have a question and a couple comments: > > Is the map of the clear color address creating a new synchronization > point between the GPU and CPU? If so, I wonder h

Re: [Intel-gfx] [PATCH v5 15/19] drm/i915/dg2: Add DG2 unified compression

2022-03-18 Thread Imre Deak
On Thu, Feb 17, 2022 at 05:15:15PM +, Chery, Nanley G wrote: > > >> [...] > > >> --- a/include/uapi/drm/drm_fourcc.h > > >> +++ b/include/uapi/drm/drm_fourcc.h > > >> @@ -583,6 +583,28 @@ extern "C" { > > >>*/ > > >> #define I915_FORMAT_MOD_4_TILED fourcc_mod_code(INTEL, 9) > > >>

Re: [Intel-gfx] [PATCH v5 16/19] uapi/drm/dg2: Introduce format modifier for DG2 clear color

2022-03-21 Thread Imre Deak
Hi Nanley, JP, On Tue, Feb 15, 2022 at 09:34:22PM +0200, Juha-Pekka Heikkila wrote: > [...] > > > > > > > > > diff --git a/include/uapi/drm/drm_fourcc.h > > > > > > > > > b/include/uapi/drm/drm_fourcc.h index > > > > > > > > > b8fb7b44c03c..697614ea4b84 100644 > > > > > > > > > --- a/include/uapi

[PATCH v2] drm/i915: Add a DP1.2 compatible way to read LTTPR capabilities

2022-03-22 Thread Imre Deak
d, so re-read the DPCD capabilities after the LTTPR common and PHY caps were read out. v2: - Use for instead of a while loop. (Ville) - Add to code comment the monitor model with the problem. Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/4531 Cc: Ville Syrjälä Signed-off-by: Im

Re: [Intel-gfx] [PATCH v5 15/19] drm/i915/dg2: Add DG2 unified compression

2022-03-24 Thread Imre Deak
On Thu, Mar 24, 2022 at 01:40:37AM +0200, Chery, Nanley G wrote: > > [...] > > Capturing all the above would you be ok with the following?: > > > > Intel color control surfaces (CCS) for DG2 render compression. > > > > The main surface is Tile 4 and at plane index 0. The CCS data is stored > > ou

Re: [Intel-gfx] [PATCH v5 16/19] uapi/drm/dg2: Introduce format modifier for DG2 clear color

2022-03-24 Thread Imre Deak
On Thu, Mar 24, 2022 at 01:42:33AM +0200, Chery, Nanley G wrote: > > -Original Message- > > From: Deak, Imre > > Sent: Monday, March 21, 2022 6:20 AM > > To: Chery, Nanley G ; Juha-Pekka Heikkila > > > > Cc: Nanley Chery ; C, Ramalingam > > ; intel-gfx ; > > Auld, Matthew ; dri-devel >

Re: [Intel-gfx] [PATCH v5 17/19] drm/i915/dg2: Flat CCS Support

2022-03-24 Thread Imre Deak
ilä > Signed-off-by: Ramalingam C Reviewed-by: Imre Deak > --- > drivers/gpu/drm/i915/display/intel_display.c | 4 ++- > drivers/gpu/drm/i915/display/intel_fb.c | 32 +-- > .../drm/i915/display/skl_universal_plane.c| 16 ++ > 3 files chang

[PATCH] drm/i915: Add a DP1.2 compatible way to read LTTPR capabilities

2022-02-28 Thread Imre Deak
d, so re-read the DPCD capabilities after the LTTPR common and PHY caps were read out. Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/4531 Signed-off-by: Imre Deak --- drivers/gpu/drm/dp/drm_dp.c | 58 --- .../drm/i915/display/intel_dp_link_t

Re: [PATCH] drm/i915: Add a DP1.2 compatible way to read LTTPR capabilities

2022-03-01 Thread Imre Deak
On Tue, Mar 01, 2022 at 04:14:24PM +0200, Ville Syrjälä wrote: > On Mon, Feb 28, 2022 at 10:12:34PM +0200, Imre Deak wrote: > > At least some DELL monitors (P2715Q) with DPCD_REV 1.2 return corrupted > > DPCD register values when reading from the 0xF- LTTPR range with an >

[PATCH v2 6/6] drm/fourcc: Add the ADL-P specific pitch requirements of CCS modifiers

2021-09-06 Thread Imre Deak
@lists.freedesktop.org Signed-off-by: Imre Deak --- include/uapi/drm/drm_fourcc.h | 24 ++-- 1 file changed, 18 insertions(+), 6 deletions(-) diff --git a/include/uapi/drm/drm_fourcc.h b/include/uapi/drm/drm_fourcc.h index 45a914850be0d..b63b7fa8bbac6 100644 --- a/include/uapi/drm

Re: [RFC PATCH] component: do not leave master devres group open after bind

2021-09-21 Thread Imre Deak
ers/base: fix devres handling for master device"). > > BugLink: https://gitlab.freedesktop.org/drm/intel/-/issues/4136 > Signed-off-by: Kai Vehmanen This makes sense to me and also seems to match the devres logic for component binding (component_bind()), where the only resources fre

Re: WARNING: CPU: 1 PID: 722 at drivers/gpu/drm/i915/display/intel_tc.c:761

2021-12-07 Thread Imre Deak
Hi Ammar, On Tue, Dec 07, 2021 at 10:54:59AM +0700, Ammar Faizi wrote: > Hello, > > I found warnings in the stable tree. > > Commit: a2547651bc896f95a3680a6a0a27401e7c7a1080 ("Linux 5.15.6") > > There are two unique warn locations: > > ammarfaizi2@integral2:~$ sudo dmesg -Sr | grep -oiE 'WAR

Re: [RFC] drm/msm/dp: Allow attaching a drm_panel

2021-12-07 Thread Imre Deak
On Tue, Dec 07, 2021 at 02:26:05PM +0200, Heikki Krogerus wrote: > +Hans and Imre > [...] > > Originally I wanted an API that we could use to pass all the details > that we have in the USB Type-C drivers (that would be the > configuration and status) to the GPU drivers, but Hans was against > that

Re: [PATCH 1/2] drm/i915: Introduce new Tile 4 format

2021-12-10 Thread Imre Deak
or this from Jani or Danvet. The modifier will be exposed to userspace only after the second one, so I don't see a problem with that approach. Either way the patchset looks ok to me: Reviewed-by: Imre Deak > With that fixed, this patch is: > > Acked-by: Nanley Chery &g

Re: [Intel-gfx] [PATCH v4 11/16] drm/i915/dg2: Add DG2 unified compression

2021-12-10 Thread Imre Deak
On Thu, Dec 09, 2021 at 09:15:28PM +0530, Ramalingam C wrote: > From: Matt Roper > > DG2 unifies render compression and media compression into a single > format for the first time. The programming and buffer layout is > supposed to match compression on older gen12 platforms, but the actual > com

Re: [Intel-gfx] [PATCH 1/2] drm/framebuffer: Format modifier for Intel Gen 12 render compression with Clear Color

2020-12-14 Thread Imre Deak
On Fri, Dec 11, 2020 at 09:04:02AM +0200, Chery, Nanley G wrote: > [...] > > > We probably don't have to update the header, but we noticed in our > > > testing that the clear color prefers an alignment greater than 64B. > > > Unfortunately, I can't find any bspec note about this. As long as the > >

Re: [Bug Report] Desktop monitor sleep regression

2021-12-17 Thread Imre Deak
aphs some of you might have seen already.] > > > > Hi, this is your Linux kernel regression tracker speaking. > > /me again > > What's up here? We are getting close to rc6, but there afaics wasn't any > reply of substance since the report ten days ago. Hence:

[PATCH] drm: Allow modeset on unregisted connectors unconditionally

2019-05-20 Thread Imre Deak
ff-by: Imre Deak --- drivers/gpu/drm/drm_atomic_helper.c | 29 ++--- 1 file changed, 2 insertions(+), 27 deletions(-) diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_atomic_helper.c index 2e0cb4246cbd..e94e69483498 100644 --- a/drivers/gp

Re: [PATCH] drm: Allow modeset on unregisted connectors unconditionally

2019-05-20 Thread Imre Deak
On Mon, May 20, 2019 at 08:37:46PM +0200, Daniel Vetter wrote: > On Mon, May 20, 2019 at 08:41:09PM +0300, Imre Deak wrote: > > We allowed modesetting an unregistered connector only in the case the > > mode is getting disabled on the connector. > > > > The reason for

Re: [PATCH] drm: Allow modeset on unregisted connectors unconditionally

2019-05-20 Thread Imre Deak
On Mon, May 20, 2019 at 09:23:00PM +0200, Daniel Vetter wrote: > On Mon, May 20, 2019 at 10:09:24PM +0300, Imre Deak wrote: > > On Mon, May 20, 2019 at 08:37:46PM +0200, Daniel Vetter wrote: > > > On Mon, May 20, 2019 at 08:41:09PM +0300, Imre Deak wrote: > > >

Re: [PATCH] drm: Allow modeset on unregisted connectors unconditionally

2019-05-20 Thread Imre Deak
On Mon, May 20, 2019 at 10:15:20PM +0200, Daniel Vetter wrote: > On Mon, May 20, 2019 at 10:06 PM Imre Deak wrote: > > On Mon, May 20, 2019 at 09:23:00PM +0200, Daniel Vetter wrote: > > > On Mon, May 20, 2019 at 10:09:24PM +0300, Imre Deak wrote: > > > > On Mon, Ma

Re: [PATCH] drm: Allow modeset on unregisted connectors unconditionally

2019-05-20 Thread Imre Deak
On Mon, May 20, 2019 at 10:59:35PM +0200, Daniel Vetter wrote: > On Mon, May 20, 2019 at 10:51 PM Imre Deak wrote: > > > > On Mon, May 20, 2019 at 10:15:20PM +0200, Daniel Vetter wrote: > > > On Mon, May 20, 2019 at 10:06 PM Imre Deak wrote: > > > > On M

Re: [Intel-gfx] [PATCH] drm: Allow modeset on unregisted connectors unconditionally

2019-05-20 Thread Imre Deak
On Mon, May 20, 2019 at 08:41:09PM +0300, Imre Deak wrote: > We allowed modesetting an unregistered connector only in the case the > mode is getting disabled on the connector. > > The reason for this check was the lack of proper refcounting for the > backing memory objects. That p

[PATCH] drm/mst: Fix MST sideband up-reply failure handling

2019-05-23 Thread Imre Deak
ude Paul Cc: Dave Airlie Signed-off-by: Imre Deak --- drivers/gpu/drm/drm_dp_mst_topology.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/drm_dp_mst_topology.c b/drivers/gpu/drm/drm_dp_mst_topology.c index da1abca1b9e9..24c325f4a616 100644 --- a/dri

Re: [PATCH] drm/mst: Fix MST sideband up-reply failure handling

2019-05-23 Thread Imre Deak
On Thu, May 23, 2019 at 06:09:56PM -0400, Lyude Paul wrote: > Patch mostly looks good to me, one comment below > > On Fri, 2019-05-24 at 00:24 +0300, Imre Deak wrote: > > Fix the breakage resulting in the stacktrace below, due to tx queue > > being full when trying to se

Re: [PATCH v3 02/15] drm/i915: Don't try to use the hardware frame counter with i965gm TV output

2019-01-22 Thread Imre Deak
at zero > v3: max_vblank_count not populated yet in intel_enable_pipe() > use intel_crtc_max_vblank_count() instead > > Cc: sta...@vger.kernel.org > Cc: Daniel Vetter > Fixes: 51e31d49c890 ("drm/i915: Use generic vblank wait") > Bugzilla: https://bugs.freedesktop.or

Re: [Intel-gfx] [PATCH 03/15] drm/i915/tv: Fix interlaced ysize calculation

2019-01-22 Thread Imre Deak
On Mon, Nov 12, 2018 at 06:59:47PM +0200, Ville Syrjala wrote: > From: Ville Syrjälä > > Fix the calculation of the vertical active period for interlaced > TV modes. > > Signed-off-by: Ville Syrjälä Matches the spec: Reviewed-by: Imre Deak > --- > drivers/gpu

Re: [PATCH 04/15] drm/i915/tv: Fix tv mode clocks

2019-01-22 Thread Imre Deak
On Mon, Nov 12, 2018 at 06:59:48PM +0200, Ville Syrjala wrote: > From: Ville Syrjälä > > The oversample clock is always supposed to be either 108 MHz > or 148.5 MHz. Make it so. > > Signed-off-by: Ville Syrjälä Matches the spec: Reviewed-by: Imre Deak > --- &g

Re: [Intel-gfx] [PATCH 05/15] drm/i915/tv: Store the TV oversampling factor in the TV mode

2019-01-22 Thread Imre Deak
ille Syrjälä Reviewed-by: Imre Deak > --- > drivers/gpu/drm/i915/intel_tv.c | 42 ++--- > 1 file changed, 28 insertions(+), 14 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_tv.c b/drivers/gpu/drm/i915/intel_tv.c > index dea24ef88763..9625

Re: [PATCH 06/15] drm/i915/tv: Use bools where appropriate

2019-01-22 Thread Imre Deak
On Mon, Nov 12, 2018 at 06:59:50PM +0200, Ville Syrjala wrote: > From: Ville Syrjälä > > 'component_only' is a bool. Initialize it like a bool. > > Signed-off-by: Ville Syrjälä Reviewed-by: Imre Deak > --- > drivers/gpu/drm/i915/intel_tv.c | 24 +++

Re: [Intel-gfx] [PATCH 07/15] drm/i915/tv: Nuke silly 0 initialzation of xpos/ypos

2019-01-22 Thread Imre Deak
On Mon, Nov 12, 2018 at 06:59:51PM +0200, Ville Syrjala wrote: > From: Ville Syrjälä > > Just assign the margin values directly to xpos/ypos instead > of first initializing to zero and then adding the values. > > Signed-off-by: Ville Syrjälä Reviewed-by: Imre Deak > -

Re: [Intel-gfx] [PATCH 08/15] drm/i915/tv: Deobfuscate preferred mode selection

2019-01-22 Thread Imre Deak
e prefer 1080 line modes. > > Signed-off-by: Ville Syrjälä Reviewed-by: Imre Deak > --- > drivers/gpu/drm/i915/intel_tv.c | 50 - > 1 file changed, 30 insertions(+), 20 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_tv.c b/

Re: [PATCH 09/15] drm/i915/tv: Use drm_mode_set_name() to name TV modes

2019-01-22 Thread Imre Deak
On Mon, Nov 12, 2018 at 06:59:54PM +0200, Ville Syrjala wrote: > From: Ville Syrjälä > > No point in storing the mode names in the array. drm_mode_set_name() > will give us the same names without wasting space for these string > constants. > > Signed-off-by: Ville Syrjäl

Re: [PATCH 10/15] drm/i915/tv: Make TV mode autoselection actually useable

2019-01-22 Thread Imre Deak
de unless the new cable type actually disagrees with it. > > Signed-off-by: Ville Syrjälä Reviewed-by: Imre Deak > --- > drivers/gpu/drm/i915/intel_tv.c | 12 +++- > 1 file changed, 7 insertions(+), 5 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_tv.c b/dri

Re: [PATCH 11/15] drm/i915/tv: Nuke reported_modes[]

2019-01-22 Thread Imre Deak
we use for load detection. > We don't need it even for that, and instead we can just rely on > the fallback mode in intel_get_load_detect_pipe(). > > Signed-off-by: Ville Syrjälä Reviewed-by: Imre Deak > --- > drivers/gpu/drm/i915/intel_tv.c | 22 +-

Re: [Intel-gfx] [PATCH 12/15] drm/i915/tv: Add 1080p30/50/60 TV modes

2019-01-22 Thread Imre Deak
On Mon, Nov 12, 2018 at 06:59:57PM +0200, Ville Syrjala wrote: > From: Ville Syrjälä > > Add the missing 1080p TV modes. On gen4 all of them work just fine, > whereas on gen3 only the 30Hz mode actually works correctly. > > Signed-off-by: Ville Syrjälä Matches the spec: Revi

Re: [PATCH 13/15] drm/i915/tv: Generate better pipe timings for TV encoder

2019-01-22 Thread Imre Deak
* > + * When the TV encoder is used the pipe wants to run faster > + * than expected rate. During the active portion the TV > + * encoder stalls the pipe every few lines to keep it in > + * check. When the TV encoder reaches the bottom margin the > + * pipe si

Re: [PATCH 13/15] drm/i915/tv: Generate better pipe timings for TV encoder

2019-01-23 Thread Imre Deak
On Tue, Jan 22, 2019 at 07:34:55PM +0200, Ville Syrjälä wrote: > On Tue, Jan 22, 2019 at 07:22:24PM +0200, Imre Deak wrote: > > On Mon, Nov 12, 2018 at 06:59:58PM +0200, Ville Syrjala wrote: > > > From: Ville Syrjälä > > > > > > To make vblank timestamps w

Re: [Intel-gfx] [PATCH 14/15] drm/i915/tv: Fix >1024 modes on gen3

2019-01-23 Thread Imre Deak
nnector_state *state; > + > + state = kmemdup(connector->state, sizeof(*state), GFP_KERNEL); > + if (!state) > + return NULL; > + > + __drm_atomic_helper_connector_duplicate_state(connector, &state->base); > + return &state->ba

Re: [Intel-gfx] [PATCH 15/15] drm/i915/tv: Filter out >1024 wide modes that would need vertical scaling on gen3

2019-01-23 Thread Imre Deak
t work. > > Signed-off-by: Ville Syrjälä Reviewed-by: Imre Deak > --- > drivers/gpu/drm/i915/intel_tv.c | 6 ++ > 1 file changed, 6 insertions(+) > > diff --git a/drivers/gpu/drm/i915/intel_tv.c b/drivers/gpu/drm/i915/intel_tv.c > index 7099d837e31a..89c5378392

Re: [Intel-gfx] [PATCH 14/15] drm/i915/tv: Fix >1024 modes on gen3

2019-01-24 Thread Imre Deak
On Wed, Jan 23, 2019 at 06:38:17PM +0200, Ville Syrjälä wrote: > On Wed, Jan 23, 2019 at 03:49:02PM +0200, Imre Deak wrote: > > On Mon, Nov 12, 2018 at 06:59:59PM +0200, Ville Syrjala wrote: > > > From: Ville Syrjälä > > > > > > On gen3 we must disable the

Re: [Intel-gfx] [PATCH] drm/i915: Don't send MST hotplugs until after resume

2019-01-28 Thread Imre Deak
d-off-by: Lyude Paul > Fixes: 0e32b39ceed6 ("drm/i915: add DP 1.2 MST support (v0.7)") > Cc: Todd Previte > Cc: Dave Airlie > Cc: Jani Nikula > Cc: Joonas Lahtinen > Cc: Rodrigo Vivi > Cc: intel-...@lists.freedesktop.org > Cc: # v3.17+ Not knowing enough abou

Re: [PATCH] drm/mst: Fix MST sideband up-reply failure handling

2019-05-30 Thread Imre Deak
On Thu, May 23, 2019 at 06:31:15PM -0400, Lyude Paul wrote: > On Fri, 2019-05-24 at 01:28 +0300, Imre Deak wrote: > > On Thu, May 23, 2019 at 06:09:56PM -0400, Lyude Paul wrote: > > > Patch mostly looks good to me, one comment below > > > > > > On Fri, 2019-0

[PATCH] drm/dp/mst: Read the extended DPCD capabilities during system resume

2022-06-14 Thread Imre Deak
detection. While at it also fix up the same call in drm_dp_mst_dump_topology(). Cc: Lyude Paul Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/5292 Signed-off-by: Imre Deak --- drivers/gpu/drm/display/drm_dp_mst_topology.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff

Re: [PATCH] drm/dp/mst: Read the extended DPCD capabilities during system resume

2022-06-14 Thread Imre Deak
On Tue, Jun 14, 2022 at 03:32:59PM +0300, Jani Nikula wrote: > On Tue, 14 Jun 2022, Imre Deak wrote: > > The WD22TB4 Thunderbolt dock at least will revert its DP_MAX_LINK_RATE > > from HBR3 to HBR2 after system suspend/resume if the DP_DP13_DPCD_REV > > registers are not read

Re: ✗ Fi.CI.IGT: failure for drm/dp/mst: Read the extended DPCD capabilities during system resume

2022-06-16 Thread Imre Deak
On Wed, Jun 15, 2022 at 04:25:34AM +, Patchwork wrote: > == Series Details == > > Series: drm/dp/mst: Read the extended DPCD capabilities during system resume > URL : https://patchwork.freedesktop.org/series/105102/ > State : failure Thanks for the reviews, pushed the patch to drm-misc-next

Re: ✗ Fi.CI.IGT: failure for drm/dp/mst: Read the extended DPCD capabilities during system resume

2022-06-16 Thread Imre Deak
On Thu, Jun 16, 2022 at 09:57:43PM +0300, Imre Deak wrote: > On Wed, Jun 15, 2022 at 04:25:34AM +, Patchwork wrote: > > == Series Details == > > > > Series: drm/dp/mst: Read the extended DPCD capabilities during system resume > > URL : https://patchwork.fr

Re: [PATCH 1/2] drm: Add HPD state to drm_connector_oob_hotplug_event()

2022-02-14 Thread Imre Deak
On Mon, Feb 07, 2022 at 08:43:27PM -0800, Bjorn Andersson wrote: > In some implementations, such as the Qualcomm platforms, the display > driver has no way to query the current HPD state and as such it's > impossible to distinguish between disconnect and attention events. > > Add a parameter to dr

[PATCH 2/3] drm/amd/display: Fix 'no previous prototype' compiler warns in amdgpu_dm_plane.c

2022-08-01 Thread Imre Deak
at_info’ [-Wmissing-prototypes] const struct drm_format_info *amd_get_format_info(const struct drm_mode_fb_cmd2 *cmd) Fixes: 5d945cbcd4b1 ("drm/amd/display: Create a file dedicated to planes") Cc: Harry Wentland Cc: Alan Liu Cc: Rodrigo Siqueira Signed-off-by: Imre Deak --- drivers

[PATCH 1/3] drm/amd/display: Fix merge conflict resolution in amdgpu_dm_plane.c

2022-08-01 Thread Imre Deak
("Merge remote-tracking branch 'drm-misc/drm-misc-next' into drm-tip") Cc: Simon Ser Cc: Thomas Zimmermann Acked-by: Thomas Zimmermann Signed-off-by: Imre Deak --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_plane.c | 2 +- 1 file changed, 1 insertion(+), 1 del

[PATCH 3/3] drm/amd/display: Fix static declaration follows non-static declaration compiler warn

2022-08-01 Thread Imre Deak
*, int *)’ 39 | void get_min_max_dc_plane_scaling(struct drm_device *dev, complier warning. Fixes: 5d945cbcd4b1 ("drm/amd/display: Create a file dedicated to planes") Cc: Rodrigo Siqueira Cc: Harry Wentland Cc: Alan Liu Signed-off-by: Imre Deak --- drivers/gpu/drm/amd/display

Re: [PATCH 2/3] drm/amd/display: Fix 'no previous prototype' compiler warns in amdgpu_dm_plane.c

2022-08-03 Thread Imre Deak
On Tue, Aug 02, 2022 at 12:57:24PM -0400, Rodrigo Siqueira Jordao wrote: > > > On 2022-08-01 09:52, Imre Deak wrote: > > Fix compiler warnings like the following triggered by > > '-Wmissing-prototypes': > > > >CC [M] drivers/gpu/drm/amd/am

Re: [PATCH v6 1/4] drm/i915/hpd: postpone HPD cancel work after last user suspension

2022-08-22 Thread Imre Deak
On Fri, Jul 22, 2022 at 02:51:40PM +0200, Andrzej Hajda wrote: > i915->hotplug.dig_port_work can be queued from intel_hpd_irq_handler > called by IRQ handler or by intel_hpd_trigger_irq called from dp_mst. > Since dp_mst is suspended after irq handler uninstall, a cleaner approach > is to cancel hp

Re: [Intel-gfx] [PATCH v6 2/4] drm/i915/fbdev: suspend HPD before fbdev unregistration

2022-08-22 Thread Imre Deak
gt;drm, FBINFO_STATE_SUSPENDED, true); > + > if (!current_is_async()) > intel_fbdev_sync(ifbdev); > > @@ -618,7 +619,7 @@ void intel_fbdev_set_suspend(struct drm_device *dev, int > state, bool synchronous > struct fb_info *info; > >

Re: [Intel-gfx] [PATCH v6 3/4] drm/i915/display: add hotplug.suspended flag

2022-08-22 Thread Imre Deak
On Fri, Jul 22, 2022 at 02:51:42PM +0200, Andrzej Hajda wrote: > HPD events during driver removal can be generated by hardware and > software frameworks - drm_dp_mst, the former we can avoid by disabling > interrupts, the latter can be triggered by any drm_dp_mst transaction, > and this is too late

Re: [PATCH v6 4/4] drm/i915/fbdev: do not create fbdev if HPD is suspended

2022-08-22 Thread Imre Deak
On Fri, Jul 22, 2022 at 02:51:43PM +0200, Andrzej Hajda wrote: > In case of deferred FB setup core can try to create new > framebuffer. Disallow it if hpd_suspended flag is set. > > Signed-off-by: Andrzej Hajda Reviewed-by: Imre Deak > --- > drivers/gpu/drm/i915/display

Re: [PATCH v6 1/4] drm/i915/hpd: postpone HPD cancel work after last user suspension

2022-08-23 Thread Imre Deak
On Tue, Aug 23, 2022 at 10:41:22AM +0300, Jani Nikula wrote: > On Mon, 22 Aug 2022, Imre Deak wrote: > > On Fri, Jul 22, 2022 at 02:51:40PM +0200, Andrzej Hajda wrote: > >> i915->hotplug.dig_port_work can be queued from intel_hpd_irq_handler > >> called by IRQ handl

Re: [Intel-gfx] [PATCH v6 3/4] drm/i915/display: add hotplug.suspended flag

2022-08-24 Thread Imre Deak
On Tue, Aug 23, 2022 at 11:48:01PM +0200, Andrzej Hajda wrote: > > > On 22.08.2022 19:27, Imre Deak wrote: > > On Fri, Jul 22, 2022 at 02:51:42PM +0200, Andrzej Hajda wrote: > > > HPD events during driver removal can be generated by hardware and > > > software fr

[PATCH] drm/fourcc: Document the Intel CCS modifiers' CC plane expected pitch

2022-06-23 Thread Imre Deak
The driver expects the pitch of the Intel CCS CC color planes to be 64 bytes aligned, adjust the modifier descriptions accordingly. Cc: Nanley Chery Signed-off-by: Imre Deak --- include/uapi/drm/drm_fourcc.h | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/include

Re: [PATCH] drm/fourcc: Document the Intel CCS modifiers' CC plane expected pitch

2022-06-28 Thread Imre Deak
Intel CCS CC color planes to be > > 64 bytes aligned, adjust the modifier descriptions accordingly. > > > > Cc: Nanley Chery > > Signed-off-by: Imre Deak > > --- > > include/uapi/drm/drm_fourcc.h | 8 > > 1 file changed, 4 insertions(+), 4 deletio

[PATCH 1/4] drm/fourcc: Introduce format modifiers for DG2 render and media compression

2022-04-04 Thread Imre Deak
decompress either a render or media compressed surface; as such, we need new RC and MC framebuffer modifiers to represent buffers in this format. v2: Clarify modifier layout description. Cc: dri-devel@lists.freedesktop.org Signed-off-by: Matt Roper Signed-off-by: Imre Deak Acked-by: Nanley

[PATCH 3/4] drm/fourcc: Introduce format modifier for DG2 clear color

2022-04-04 Thread Imre Deak
fier addition to a separate patch. Clarify the modifier layout description. Cc: dri-devel@lists.freedesktop.org Signed-off-by: Mika Kahola cc: Anshuman Gupta Signed-off-by: Juha-Pekka Heikkilä Signed-off-by: Ramalingam C Signed-off-by: Imre Deak Acked-by: Nanley Chery --- include/uapi/drm/drm_four

[PATCH 1/2] drm/dp: Export drm_dp_dpcd_access()

2022-04-07 Thread Imre Deak
The next patch needs a way to read a DPCD register without the preceding wake-up read in drm_dp_dpcd_read(). Export drm_dp_dpcd_access() to allow this. Cc: dri-devel@lists.freedesktop.org Signed-off-by: Imre Deak --- drivers/gpu/drm/dp/drm_dp.c| 19 +-- include/drm/dp

Re: [PATCH 1/2] drm/dp: Export drm_dp_dpcd_access()

2022-04-08 Thread Imre Deak
On Thu, Apr 07, 2022 at 10:32:11PM +0300, Jani Nikula wrote: > On Thu, 07 Apr 2022, Imre Deak wrote: > > The next patch needs a way to read a DPCD register without the preceding > > wake-up read in drm_dp_dpcd_read(). Export drm_dp_dpcd_access() to allow > > this. >

[PATCH v2 1/2] drm/dp: Factor out a function to probe a DPCD address

2022-04-08 Thread Imre Deak
drm_dp_dpcd_access(). (Jani) Cc: Jani Nikula Cc: dri-devel@lists.freedesktop.org Signed-off-by: Imre Deak --- drivers/gpu/drm/dp/drm_dp.c| 28 +--- include/drm/dp/drm_dp_helper.h | 1 + 2 files changed, 26 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/dp

Re: [v2,1/2] drm/dp: Factor out a function to probe a DPCD address

2022-04-08 Thread Imre Deak
On Sat, Apr 09, 2022 at 01:47:21AM +0300, Almahallawy, Khaled wrote: > On Fri, 2022-04-08 at 20:21 +0300, Imre Deak wrote: > > Factor out from drm_dp_dpcd_read() a function to probe a DPCD address > > with a 1-byte read access. This will be needed by the next patch > > doi

[PATCH v3 1/2] drm/dp: Factor out a function to probe a DPCD address

2022-04-11 Thread Imre Deak
if the read was successful. v2: Add a probe function instead of exporting drm_dp_dpcd_access(). (Jani) v3: Add tracing for the 1-byte read even if the read was successful. (Khaled) Cc: Jani Nikula Cc: Khaled Almahallawy Cc: dri-devel@lists.freedesktop.org Signed-off-by: Imre Deak --- drivers

[PATCH v2 1/4] drm/fourcc: Introduce format modifiers for DG2 render and media compression

2022-04-11 Thread Imre Deak
decompress either a render or media compressed surface; as such, we need new RC and MC framebuffer modifiers to represent buffers in this format. v2: Clarify modifier layout description. Cc: dri-devel@lists.freedesktop.org Signed-off-by: Matt Roper Signed-off-by: Imre Deak Acked-by: Nanley

[PATCH v2 3/4] drm/fourcc: Introduce format modifier for DG2 clear color

2022-04-11 Thread Imre Deak
fier addition to a separate patch. Clarify the modifier layout description. Cc: dri-devel@lists.freedesktop.org Signed-off-by: Mika Kahola cc: Anshuman Gupta Signed-off-by: Juha-Pekka Heikkilä Signed-off-by: Ramalingam C Signed-off-by: Imre Deak Acked-by: Nanley Chery Reviewed-by: Juha-Pekka Heik

Re: [Intel-gfx] [PATCH v6 3/4] drm/i915/display: add hotplug.suspended flag

2022-08-25 Thread Imre Deak
On Thu, Aug 25, 2022 at 01:24:04PM +0200, Andrzej Hajda wrote: > On 24.08.2022 13:22, Imre Deak wrote: > > On Tue, Aug 23, 2022 at 11:48:01PM +0200, Andrzej Hajda wrote: > > > > > > > > > On 22.08.2022 19:27, Imre Deak wrote: > > > > On Fri, Jul 2

Re: [PATCH v7 1/3] drm/i915/hpd: suspend MST at the end of intel_modeset_driver_remove

2022-09-02 Thread Imre Deak
able+0x70/0x140 > [283.406313] i915_digport_work_func+0xba/0x150 [i915] > > Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/4586 > Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/5558 > Signed-off-by: Andrzej Hajda > Reviewed-by: Arun R Murthy Reviewed-by: Imre

[PATCH] drm/dp_mst: Avoid deleting payloads for connectors staying enabled

2022-09-07 Thread Imre Deak
6386 ("drm/display/dp_mst: Fix modeset tracking in drm_dp_atomic_release_vcpi_slots()") Testcase: igt@testdisplay Cc: Lyude Paul Signed-off-by: Imre Deak --- drivers/gpu/drm/display/drm_dp_mst_topology.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/gpu/drm/display/d

Re: [PATCH] drm/dp_mst: Avoid deleting payloads for connectors staying enabled

2022-09-08 Thread Imre Deak
and the problem is visible: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12093/bat-rpls-2/igt@kms_flip@basic-flip-vs-d...@b-dp5.html > thanks for the catch! > > Reviewed-by: Lyude Paul Thanks. > On Wed, 2022-09-07 at 17:25 +0300, Imre Deak wrote: > > When an MST connector stays

[PATCH 01/15] drm/framebuffer: Format modifier for Intel Gen-12 render compression

2019-12-20 Thread Imre Deak
Signed-off-by: Imre Deak Reviewed-by: Mika Kahola --- include/uapi/drm/drm_fourcc.h | 11 +++ 1 file changed, 11 insertions(+) diff --git a/include/uapi/drm/drm_fourcc.h b/include/uapi/drm/drm_fourcc.h index 8caaaf7ff91b..5ba481f49931 100644 --- a/include/uapi/drm/drm_fourcc.h +++ b

  1   2   3   4   5   6   7   8   >