On Thu, Oct 4, 2018 at 4:38 PM Jani Nikula wrote:
>
> They don't need to be modified.
>
> Cc: Greg Kroah-Hartman
> Cc: "Rafael J. Wysocki"
> Signed-off-by: Jani Nikula
Reviewed-by: Rafael J. Wysocki
> ---
> drivers/gpu/drm/i915/i915_sysfs.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 del
On Thu, Oct 4, 2018 at 4:38 PM Jani Nikula wrote:
>
> Let the passed in array be const (and thus placed in rodata) instead of
> a mutable array of const pointers.
I'm not sure if the changes guarantee what you want. If I'm not
mistaken, they just mean that the function itself cannot modify eithe
== Series Details ==
Series: firmware/dmc/icl: Add missing MODULE_FIRMWARE() for Icelake. (rev3)
URL : https://patchwork.freedesktop.org/series/49678/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4932_full -> Patchwork_10367_full =
== Summary - WARNING ==
Minor unknown
== Series Details ==
Series: drm/i915/csr Added DC5 and DC6 counter register for ICL in debugfs
entry. (rev4)
URL : https://patchwork.freedesktop.org/series/49800/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4933 -> Patchwork_10372 =
== Summary - SUCCESS ==
No regress
DC5 and DC6 counter register tells about residency of DC5 and DC6.
These registers are same for SKL and ICL.
v2 : Remove csr_version check.
Added generic check regarding DC counters for Gen9 onwards. (Rodrigo)
v3 : Simplified gen checks. (Chris)
v4 : Simplified "if" ladder for multiple gens.
== Series Details ==
Series: drm/i915: Fix VIDEO_DIP_CTL bit shifts
URL : https://patchwork.freedesktop.org/series/50573/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4932_full -> Patchwork_10365_full =
== Summary - WARNING ==
Minor unknown changes coming with Patchwork
== Series Details ==
Series: series starting with [1/2] drm/i915/psr: Reduce PSR2 "frames before
selective update entry"
URL : https://patchwork.freedesktop.org/series/50593/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4933 -> Patchwork_10371 =
== Summary - SUCCESS ==
== Series Details ==
Series: series starting with [1/2] drm/i915/psr: Reduce PSR2 "frames before
selective update entry"
URL : https://patchwork.freedesktop.org/series/50593/
State : warning
== Summary ==
$ dim sparse origin/drm-tip
Commit: drm/i915/psr: Reduce PSR2 "frames before selective u
The hardware can start selective update following capture of a full frame
in the remote frame buffer, there is no need to wait any longer. Set
"Frames Before SU Entry" bitfield to the default value of 1.
Signed-off-by: Dhinakaran Pandiyan
---
drivers/gpu/drm/i915/intel_psr.c | 3 +--
1 file chan
PSR2 sinks that require Y coordinates for selective update also need the
Y coordinate Valid bit in VSC SDP.
Spec: eDP 1.4b VSC payload extension for PSR2 operation (Table 6-12)
Signed-off-by: Dhinakaran Pandiyan
---
drivers/gpu/drm/i915/intel_psr.c | 2 +-
1 file changed, 1 insertion(+), 1 delet
On Thu, 2018-10-04 at 17:27 -0700, Manasi Navare wrote:
> On Thu, Oct 04, 2018 at 05:00:06PM -0700, Dhinakaran Pandiyan wrote:
> > On Thu, 2018-10-04 at 16:28 -0700, Manasi Navare wrote:
> > > On Thu, Oct 04, 2018 at 04:13:26PM -0700, Dhinakaran Pandiyan
> > > wrote:
> > > > On Thu, 2018-10-04 at 1
== Series Details ==
Series: series starting with [v2,1/6] drm/i915/icl: Add WaEnable32PlaneMode
URL : https://patchwork.freedesktop.org/series/50569/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4932_full -> Patchwork_10364_full =
== Summary - SUCCESS ==
No regressions
== Series Details ==
Series: Fix legacy DPMS changes with MST (rev4)
URL : https://patchwork.freedesktop.org/series/49878/
State : failure
== Summary ==
Applying: drm/atomic_helper: Disallow new modesets on unregistered connectors
Applying: drm/nouveau: Fix nv50_mstc->best_encoder()
Applying:
On Thu, Oct 04, 2018 at 05:00:06PM -0700, Dhinakaran Pandiyan wrote:
> On Thu, 2018-10-04 at 16:28 -0700, Manasi Navare wrote:
> > On Thu, Oct 04, 2018 at 04:13:26PM -0700, Dhinakaran Pandiyan wrote:
> > > On Thu, 2018-10-04 at 16:03 -0700, Lucas De Marchi wrote:
> > > > On Thu, Oct 04, 2018 at 01:
Currently we set intel_connector->mst_port to NULL to signify that the
MST port has been removed from the system so that we can prevent further
action on the port such as connector probes, mode probing, etc.
However, we're going to need access to intel_connector->mst_port in
order to fixup ->best_e
Since we need to be able to allow DPMS on->off prop changes after an MST
port has disappeared from the system, we need to be able to make sure we
can compute a config for the resulting atomic commit. Currently this is
impossible when the port has disappeared, since the VCPI slot searching
we try to
Currently, i915 appears to rely on blocking modesets on
no-longer-present MSTB ports by simply returning NULL for
->best_encoder(), which in turn causes any new atomic commits that don't
disable the CRTC to fail. This is wrong however, since we still want to
allow userspace to disable CRTCs on no-l
As mentioned in the previous commit, we currently prevent new modesets
on recently-removed MST connectors by returning no encoder from our
->best_encoder() callback once the MST port has disappeared. This is
wrong however, because it prevents legacy modesetting users from being
able to disable CRTC
Next version of https://patchwork.freedesktop.org/series/49878/ . No
changes, except that these patches are against master so hopefully
intel's CI doesn't get confused this time.
Lyude Paul (5):
drm/atomic_helper: Disallow new modesets on unregistered connectors
drm/nouveau: Fix nv50_mstc->bes
With the exception of modesets which would switch the DPMS state of a
connector from on to off, we want to make sure that we disallow all
modesets which would result in enabling a new monitor or a new mode
configuration on a monitor if the connector for the display in question
is no longer register
On Thu, 2018-10-04 at 16:28 -0700, Manasi Navare wrote:
> On Thu, Oct 04, 2018 at 04:13:26PM -0700, Dhinakaran Pandiyan wrote:
> > On Thu, 2018-10-04 at 16:03 -0700, Lucas De Marchi wrote:
> > > On Thu, Oct 04, 2018 at 01:51:49PM -0700, Dhinakaran Pandiyan
> > > wrote:
> > > > The shifts for VSC_SE
== Series Details ==
Series: igt/gem_ctx_exec: Exercise I915_CONTEXT_PARAM_RECOVERABLE
URL : https://patchwork.freedesktop.org/series/50561/
State : success
== Summary ==
= CI Bug Log - changes from IGT_4667_full -> IGTPW_1909_full =
== Summary - WARNING ==
Minor unknown changes coming wit
== Series Details ==
Series: Watermarks small fixes/improvements
URL : https://patchwork.freedesktop.org/series/50579/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4933 -> Patchwork_10368 =
== Summary - SUCCESS ==
No regressions found.
External URL:
https://patchwor
== Series Details ==
Series: drm/i915/icl: MBUS B credit change
URL : https://patchwork.freedesktop.org/series/50560/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4931_full -> Patchwork_10363_full =
== Summary - SUCCESS ==
No regressions found.
== Known issues ==
== Series Details ==
Series: Fix legacy DPMS changes with MST (rev3)
URL : https://patchwork.freedesktop.org/series/49878/
State : failure
== Summary ==
Applying: drm/atomic_helper: Disallow new modesets on unregistered connectors
Applying: drm/nouveau: Fix nv50_mstc->best_encoder()
Applying:
== Series Details ==
Series: series starting with [1/2] sysfs: constify sysfs create/remove files
harder
URL : https://patchwork.freedesktop.org/series/50558/
State : failure
== Summary ==
= CI Bug Log - changes from CI_DRM_4931_full -> Patchwork_10362_full =
== Summary - FAILURE ==
Serio
== Series Details ==
Series: Watermarks small fixes/improvements
URL : https://patchwork.freedesktop.org/series/50579/
State : warning
== Summary ==
$ dim sparse origin/drm-tip
Commit: drm/i915: don't apply Display WAs 1125 and 1126 to GLK/CNL+
Okay!
Commit: drm/i915: fix the transition minim
On Thu, Oct 04, 2018 at 04:13:26PM -0700, Dhinakaran Pandiyan wrote:
> On Thu, 2018-10-04 at 16:03 -0700, Lucas De Marchi wrote:
> > On Thu, Oct 04, 2018 at 01:51:49PM -0700, Dhinakaran Pandiyan wrote:
> > > The shifts for VSC_SELECT bits are wrong, fix it. Good thing is the
> > > definitions are u
Currently, i915 appears to rely on blocking modesets on
no-longer-present MSTB ports by simply returning NULL for
->best_encoder(), which in turn causes any new atomic commits that don't
disable the CRTC to fail. This is wrong however, since we still want to
allow userspace to disable CRTCs on no-l
Since we need to be able to allow DPMS on->off prop changes after an MST
port has disappeared from the system, we need to be able to make sure we
can compute a config for the resulting atomic commit. Currently this is
impossible when the port has disappeared, since the VCPI slot searching
we try to
Currently we set intel_connector->mst_port to NULL to signify that the
MST port has been removed from the system so that we can prevent further
action on the port such as connector probes, mode probing, etc.
However, we're going to need access to intel_connector->mst_port in
order to fixup ->best_e
As mentioned in the previous commit, we currently prevent new modesets
on recently-removed MST connectors by returning no encoder from our
->best_encoder() callback once the MST port has disappeared. This is
wrong however, because it prevents legacy modesetting users from being
able to disable CRTC
Next version of https://patchwork.freedesktop.org/series/49877/
This fixes some rather silly bugs regarding DPMS On->Off changes failing
for connectors which were just recently destroyed.
Lyude Paul (5):
drm/atomic_helper: Disallow new modesets on unregistered connectors
drm/nouveau: Fix nv50
With the exception of modesets which would switch the DPMS state of a
connector from on to off, we want to make sure that we disallow all
modesets which would result in enabling a new monitor or a new mode
configuration on a monitor if the connector for the display in question
is no longer register
Em Qui, 2018-10-04 às 15:36 -0700, Rodrigo Vivi escreveu:
> From: Anusha Srivatsa
>
> Add missing MODULE_FIRMWARE while loading DMC ICL.
>
> v2: Add Fixes tag. (Rodrigo)
> v3: Rebase by Rodrigo after commit 7fe78985cd08 ("drm/i915/csr:
> restructure CSR firmware definition macros")
> v4: Rodrig
On Thu, Oct 04, 2018 at 04:03:05PM -0700, Lucas De Marchi wrote:
> On Thu, Oct 04, 2018 at 01:51:49PM -0700, Dhinakaran Pandiyan wrote:
> > The shifts for VSC_SELECT bits are wrong, fix it. Good thing is the
> > definitions are unused.
>
> If they are unused why are we fixing them instead of remov
We were writing to PLANE_BUF_CFG(pipe, plane_id) twice for every
platform, and we were even using different values on the gen10- planar
case. The first write is useless since it just gets replaced with the
next one, so kill it.
There's a lot to improve in the DDB code, but let's start by avoiding
The transition watermarks ask for Selected Result Blocks (the real
value), not Result Blocks (the integer value). Given how ceilings are
applied in both the non-transition and the transition watermarks
calculations, we can get away with assuming that Selected Result
Blocks is actually Result Blocks
I'm investigating ICL watermarks failures and these are some of the immediate
problems I was able to find in the watermarks code. I don't think they're enough
to fix the problems our CI is able to reproduce, but I do think these changes
are worth having.
Paulo Zanoni (6):
drm/i915: don't apply D
On these platforms we're supposed to unconditonally pick the method 2
result instead of the minimum.
Signed-off-by: Paulo Zanoni
---
drivers/gpu/drm/i915/intel_pm.c | 23 ---
1 file changed, 16 insertions(+), 7 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_pm.c b/driv
The transition minimum is 14 blocks for gens 9 and 10, and 4 blocks
for gen 11. This minimum value is supposed to be added to the
configurable trans_amount. This matches both BSpec and additional
information provided by our HW engineers.
Signed-off-by: Paulo Zanoni
---
drivers/gpu/drm/i915/intel
BSpec does not show these WAs as applicable to GLK, and for CNL it
only shows them applicable for a super early pre-production stepping
we shouldn't be caring about anymore. Remove these so we can avoid
them on ICL too.
Signed-off-by: Paulo Zanoni
---
drivers/gpu/drm/i915/intel_pm.c | 43 +++
This message is currently marked as DRM_DEBUG_ATOMIC. I would like it
to be DRM_DEBUG_KMS since it is more KMS than atomic, and this will
also make the message appear in the CI logs, which may or may not help
us with some FIFO underrun bugs.
Signed-off-by: Paulo Zanoni
---
drivers/gpu/drm/i915/i
On Thu, 2018-10-04 at 16:03 -0700, Lucas De Marchi wrote:
> On Thu, Oct 04, 2018 at 01:51:49PM -0700, Dhinakaran Pandiyan wrote:
> > The shifts for VSC_SELECT bits are wrong, fix it. Good thing is the
> > definitions are unused.
>
> If they are unused why are we fixing them instead of removing? Or
== Series Details ==
Series: firmware/dmc/icl: Add missing MODULE_FIRMWARE() for Icelake. (rev3)
URL : https://patchwork.freedesktop.org/series/49678/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4932 -> Patchwork_10367 =
== Summary - SUCCESS ==
No regressions found.
On Thu, Oct 04, 2018 at 01:51:49PM -0700, Dhinakaran Pandiyan wrote:
> The shifts for VSC_SELECT bits are wrong, fix it. Good thing is the
> definitions are unused.
If they are unused why are we fixing them instead of removing? Or better,
why did we add them?
Lucas De Marchi
>
> Cc: Manasi Nava
From: Anusha Srivatsa
Add missing MODULE_FIRMWARE while loading DMC ICL.
v2: Add Fixes tag. (Rodrigo)
v3: Rebase by Rodrigo after commit 7fe78985cd08 ("drm/i915/csr:
restructure CSR firmware definition macros")
v4: Rodrigo fixing his own mess on commit mentioning on v3
comment above.
Fixes
On Thu, Oct 04, 2018 at 10:14:17PM -, Patchwork wrote:
> == Series Details ==
>
> Series: firmware/dmc/icl: Add missing MODULE_FIRMWARE() for Icelake. (rev2)
> URL : https://patchwork.freedesktop.org/series/49678/
> State : warning
>
> == Summary ==
>
> $ dim checkpatch origin/drm-tip
> 75
== Series Details ==
Series: firmware/dmc/icl: Add missing MODULE_FIRMWARE() for Icelake. (rev2)
URL : https://patchwork.freedesktop.org/series/49678/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4932 -> Patchwork_10366 =
== Summary - SUCCESS ==
No regressions found.
== Series Details ==
Series: firmware/dmc/icl: Add missing MODULE_FIRMWARE() for Icelake. (rev2)
URL : https://patchwork.freedesktop.org/series/49678/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
751aeeb3a90f firmware/dmc/icl: Add missing MODULE_FIRMWARE() for Icelake.
-:9: ER
From: Anusha Srivatsa
Add missing MODULE_FIRMWARE while loading DMC ICL.
v2: Add Fixes tag. (Rodrigo)
v3: Rebase by Rodrigo after 7fe78985cd08 ("drm/i915/csr: restructure CSR
firmware definition macros")
Fixes: 4445930f1c4a ("firmware/dmc/icl: load v1.07 on icelake.")
Cc: Rodrigo Vivi
Cc: Paul
== Series Details ==
Series: drm/i915: Fix VIDEO_DIP_CTL bit shifts
URL : https://patchwork.freedesktop.org/series/50573/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4932 -> Patchwork_10365 =
== Summary - SUCCESS ==
No regressions found.
External URL:
https://patch
On Wed, Oct 03, 2018 at 12:52:03PM +0530, Mahesh Kumar wrote:
> DDI/TC clock-off bits are not equally distanced. TC1-3 bits are
> from offset 12 & TC4 is at offset 21.
> Create a function to choose correct clk-off bit.
>
> Signed-off-by: Mahesh Kumar
> Signed-off-by: Vandita Kulkarni
> Cc: Lucas
On Wed, Oct 03, 2018 at 12:52:00PM +0530, Mahesh Kumar wrote:
> From: Vandita Kulkarni
>
> This patch adds helper function for identifying
> whether the given PLL is combo PHY PLL or not.
> This helper function is used inside various ICL
> functions to make them scalable.
>
> Signed-off-by: Vand
On Wed, Oct 03, 2018 at 12:51:59PM +0530, Mahesh Kumar wrote:
> From: Vandita Kulkarni
>
> Use intel_port_is_tc and intel_port_is_combophy
> functions to replace the individual port checks
> from port C to F and port A to B respectively.
>
> Signed-off-by: Vandita Kulkarni
> Signed-off-by: Mahe
On Wed, Oct 03, 2018 at 12:51:58PM +0530, Mahesh Kumar wrote:
> From: Vandita Kulkarni
>
> Use the existing port-to-id helper function, to refactor
> hence making it scalable.
>
> Signed-off-by: Vandita Kulkarni
> Signed-off-by: Mahesh Kumar
> Cc: Lucas De Marchi
> Cc: Madhav Chauhan
Revie
On Wed, Oct 03, 2018 at 12:51:57PM +0530, Mahesh Kumar wrote:
> Instead of directly comparing HPD pins use intel_port_is_combophy/tc
> helper functions to distinguish between combophy/TC ports.
>
> Signed-off-by: Mahesh Kumar
> Cc: Manasi Navare
> ---
Reviewed-by: Lucas De Marchi
> drivers/g
The shifts for VSC_SELECT bits are wrong, fix it. Good thing is the
definitions are unused.
Cc: Manasi Navare
Cc: Anusha Srivatsa
Cc: Rodrigo Vivi
Fixes: 7af2be6d54d4 ("drm/i915/icl: Add VIDEO_DIP registers")
Signed-off-by: Dhinakaran Pandiyan
---
drivers/gpu/drm/i915/i915_reg.h | 18
On Wed, Oct 03, 2018 at 11:18:26AM +0200, Daniel Vetter wrote:
> It's for legacy drivers only (atomic ones should use
> drm_atomic_helper_check_plane_state() instead), and there's no users
> left except the one in the primary plane helpers.
>
> Signed-off-by: Daniel Vetter
Reviewed-by: Ville Syr
On Tue, Oct 02, 2018 at 04:36:31PM +, Thomas Hellstrom wrote:
> On 10/02/2018 05:15 PM, Ville Syrjälä wrote:
> > On Tue, Oct 02, 2018 at 03:35:12PM +0200, Daniel Vetter wrote:
> >> The core _does_ the call to drm_atomic_commit for you. That's pretty
> >> much the entire point of having the fanc
On Thu, Oct 04, 2018 at 02:20:43PM +0530, Mahesh Kumar wrote:
> This patch creates a function/wrapper to check if port is combophy port
> instead of explicitly comparing ports.
>
> Changes since V1:
> - keep all intel_port_is_* helper together (Lucas)
>
> Signed-off-by: Mahesh Kumar
> Cc: Madha
On Thu, Oct 04, 2018 at 02:33:24PM -0400, Sean Paul wrote:
> On Tue, Oct 02, 2018 at 03:35:10PM +0200, Daniel Vetter wrote:
> > It's the default. The exported version was kinda a transition state,
> > before we made this the default.
> >
> > To stop new atomic drivers from using it (instead of jus
== Series Details ==
Series: series starting with [v2,1/6] drm/i915/icl: Add WaEnable32PlaneMode
URL : https://patchwork.freedesktop.org/series/50569/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4932 -> Patchwork_10364 =
== Summary - SUCCESS ==
No regressions found.
== Series Details ==
Series: drm/i915: Engine discovery query (rev6)
URL : https://patchwork.freedesktop.org/series/39958/
State : failure
== Summary ==
= CI Bug Log - changes from CI_DRM_4931_full -> Patchwork_10358_full =
== Summary - FAILURE ==
Serious unknown changes coming with Patchw
On Tue, Oct 02, 2018 at 03:35:10PM +0200, Daniel Vetter wrote:
> It's the default. The exported version was kinda a transition state,
> before we made this the default.
>
> To stop new atomic drivers from using it (instead of just relying on
> the default) let's unexport it.
>
> Signed-off-by: Da
Shader feature to prefetch binding tables does not support 16:6 18:8 BTP
formats. Enabling fault handling could result in hangs with faults.
Disabling demand prefetch would disable binding table prefetch.
V2: Fix the stepping rivision to B0(Mika)
References: HSDES#1406609255, HSDES#1406573985
Cc:
From: Anuj Phogat
Incorrect TDL's SSP address shift in SARB for 16:6 & 18:8 modes.
Disable the Sampler state prefetch functionality in the SARB by
programming 0xB000[30] to '1'. This is to be done at boot time
and the feature must remain disabled permanently.
Fixes flaky tex-mip-level-selection*
Display WA_1405510057 asks to not enable YUV 420 HDMI
10bpc when horizontal blank size mod 8 reminder is 2.
Cc: James Ausmus
Cc: Paulo Zanoni
Cc: Rodrigo Vivi
Cc: Ville Syrjälä
Signed-off-by: Radhakrishna Sripada
---
drivers/gpu/drm/i915/intel_hdmi.c | 10 +-
1 file changed, 9 insert
From: Oscar Mateo
Required for Bindless samplers.
Cc: Mika Kuoppala
Signed-off-by: Oscar Mateo
Signed-off-by: Radhakrishna Sripada
---
drivers/gpu/drm/i915/i915_reg.h | 2 ++
drivers/gpu/drm/i915/intel_workarounds.c | 3 +++
2 files changed, 5 insertions(+)
diff --git a/drivers/gpu
From: Oscar Mateo
Required to dinamically set 'Trilinear Filter Quality Mode'
Cc: Mika Kuoppala
Signed-off-by: Oscar Mateo
Signed-off-by: Radhakrishna Sripada
---
drivers/gpu/drm/i915/intel_workarounds.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/gpu/drm/i915/intel_workaro
Gen11 Display suports 32 planes in total. Enable the new format in context
status to be used and expanded to 32 planes.
V2: Move the WA to display WA's(Chris)
Cc: Chris Wilson
Cc: Michel Thierry
Cc: James Ausmus
Cc: Anusha Srivatsa
Signed-off-by: Radhakrishna Sripada
---
drivers/gpu/drm/i91
On Thu, 2018-10-04 at 08:54 -0700, Rodrigo Vivi wrote:
> On Thu, Oct 04, 2018 at 01:59:52PM +0300, Ville Syrjälä wrote:
> > On Thu, Oct 04, 2018 at 10:03:39AM +0300, Jani Nikula wrote:
> > > On Wed, 03 Oct 2018, Dhinakaran Pandiyan <
> > > dhinakaran.pandi...@intel.com> wrote:
> > > > MST is enable
On Wed, Oct 03, 2018 at 06:12:42PM +0200, Daniel Vetter wrote:
> On Wed, Oct 03, 2018 at 05:50:17PM +0300, Ville Syrjala wrote:
> > From: Ville Syrjälä
> >
> > When we decide that a plane is attached to the wrong pipe we try
> > to turn off said plane. However we are passing around the crtc we
>
Hi Daniel,
On Wednesday, 3 October 2018 12:08:38 EEST Daniel Vetter wrote:
> On Tue, Oct 02, 2018 at 04:53:12PM +0300, Laurent Pinchart wrote:
> > On Tuesday, 2 October 2018 16:35:10 EEST Daniel Vetter wrote:
> > > It's the default. The exported version was kinda a transition state,
> > > before w
== Series Details ==
Series: Refactor and Add helper function for combophy/tc ports (rev2)
URL : https://patchwork.freedesktop.org/series/50484/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4930_full -> Patchwork_10356_full =
== Summary - SUCCESS ==
No regressions found
== Series Details ==
Series: Refactor and Add helper function for combophy/tc ports (rev2)
URL : https://patchwork.freedesktop.org/series/50484/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4930_full -> Patchwork_10355_full =
== Summary - SUCCESS ==
No regressions found
Quoting Tvrtko Ursulin (2018-10-04 11:51:18)
> From: Tvrtko Ursulin
>
> Engine discovery query allows userspace to enumerate engines, probe their
> configuration features, all without needing to maintain the internal PCI
> ID based database.
>
> A new query for the generic i915 query ioctl is ad
== Series Details ==
Series: igt/gem_ctx_exec: Exercise I915_CONTEXT_PARAM_RECOVERABLE
URL : https://patchwork.freedesktop.org/series/50561/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4931 -> IGTPW_1909 =
== Summary - SUCCESS ==
No regressions found.
External URL:
On 03/10/18 23:29, Chris Wilson wrote:
Quoting Daniele Ceraolo Spurio (2018-10-03 23:45:02)
On 03/10/18 08:24, Chris Wilson wrote:
Quoting Daniele Ceraolo Spurio (2018-10-03 01:12:57)
On 02/10/18 15:39, Patchwork wrote:
== Series Details ==
Series: series starting with [v2,1/3] drm/i91
== Series Details ==
Series: drm/i915/icl: MBUS B credit change
URL : https://patchwork.freedesktop.org/series/50560/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4931 -> Patchwork_10363 =
== Summary - SUCCESS ==
No regressions found.
External URL:
https://patchwork
== Series Details ==
Series: drm/i915: Only reset seqno if actually idle
URL : https://patchwork.freedesktop.org/series/50537/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4930_full -> Patchwork_10354_full =
== Summary - WARNING ==
Minor unknown changes coming with Patc
On Thu, Oct 04, 2018 at 01:59:52PM +0300, Ville Syrjälä wrote:
> On Thu, Oct 04, 2018 at 10:03:39AM +0300, Jani Nikula wrote:
> > On Wed, 03 Oct 2018, Dhinakaran Pandiyan
> > wrote:
> > > MST is enabled by default on all platforms that support it. I don't think
> > > we should be providing a swit
Quoting Stanislav Lisovskiy (2018-10-04 13:49:41)
> diff --git a/src/sna/sna_render.h b/src/sna/sna_render.h
> index 6669af9d..ef88d1f9 100644
> --- a/src/sna/sna_render.h
> +++ b/src/sna/sna_render.h
> @@ -139,20 +139,25 @@ struct sna_composite_op {
>
> struct {
>
When RECOVERABLE is set, the kernel will attempt to automatically recover
a context after a hang. But if it is unset, the kernel will ban the
guilty context on a hang, preventing subsequent execution.
v2: Create a has_recoverable_param()
Signed-off-by: Chris Wilson
Cc: Mika Kuoppala
---
tests/
On Thu, Oct 04, 2018 at 03:25:42PM +0200, Maarten Lankhorst wrote:
> Op 04-10-18 om 14:57 schreef Ville Syrjälä:
> > On Thu, Oct 04, 2018 at 11:45:59AM +0200, Maarten Lankhorst wrote:
> >> Do not rely on crtc->config any more. Remove the assertion from
> >> ibx_pch_dpll_disable, because we the dpll
Quoting Mika Kuoppala (2018-10-04 14:26:16)
> Chris Wilson writes:
>
> > When RECOVERABLE is set, the kernel will attempt to automatically recover
> > a context after a hang. But if it is unset, the kernel will ban the
> > guilty context on a hang, preventing subsequent execution.
> >
> > Signed-
No functional change. But just a minor change to keep
up with Spec, since it has changed since commit c3cc39c539d4
("drm/i915/icl: program mbus during pipe enable")
The instructions previously said to program pipe's
B credit = 24 / number of pipes, which is 8 for ICL.
Now the spec gives us direct
== Series Details ==
Series: series starting with [1/2] sysfs: constify sysfs create/remove files
harder
URL : https://patchwork.freedesktop.org/series/50558/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4931 -> Patchwork_10362 =
== Summary - SUCCESS ==
No regressions
== Series Details ==
Series: series starting with [1/2] sysfs: constify sysfs create/remove files
harder
URL : https://patchwork.freedesktop.org/series/50558/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
fd1ccbdbc465 sysfs: constify sysfs create/remove files harder
-:53: WARN
They don't need to be modified.
Cc: Greg Kroah-Hartman
Cc: "Rafael J. Wysocki"
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/i915_sysfs.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_sysfs.c
b/drivers/gpu/drm/i915/i915_sysfs.c
index e
Let the passed in array be const (and thus placed in rodata) instead of
a mutable array of const pointers.
Cc: Greg Kroah-Hartman
Cc: "Rafael J. Wysocki"
Signed-off-by: Jani Nikula
---
fs/sysfs/file.c | 4 ++--
include/linux/sysfs.h | 8
2 files changed, 6 insertions(+), 6 delet
Some comments below, mostly related to trying to keep the uapi header
nice and tidy.
Quoting Tvrtko Ursulin (2018-10-04 14:32:48)
> @@ -1747,6 +1748,52 @@ struct drm_i915_query_topology_info {
> __u8 data[];
> };
>
> +/**
> + * struct drm_i915_engine_info
> + *
> + * Describes one engin
== Series Details ==
Series: drm/i915: First cleanup pass to get rid of more crtc->config users.
(rev3)
URL : https://patchwork.freedesktop.org/series/50506/
State : failure
== Summary ==
= CI Bug Log - changes from CI_DRM_4931 -> Patchwork_10361 =
== Summary - FAILURE ==
Serious unknown
== Series Details ==
Series: series starting with [1/2] drm/i915: Report the number of closed vma
held by each context in debugfs
URL : https://patchwork.freedesktop.org/series/50536/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4929_full -> Patchwork_10353_full =
== Summ
== Series Details ==
Series: drm/i915: First cleanup pass to get rid of more crtc->config users.
(rev3)
URL : https://patchwork.freedesktop.org/series/50506/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
743f67a7f663 drm/i915: Remove dereferences of crtc->config in set_pipecon
== Series Details ==
Series: Add XYUV format support (rev8)
URL : https://patchwork.freedesktop.org/series/48007/
State : failure
== Summary ==
= CI Bug Log - changes from CI_DRM_4931 -> Patchwork_10360 =
== Summary - FAILURE ==
Serious unknown changes coming with Patchwork_10360 absolutel
Hi Dave,
Here comes -fixes for drm-next.
One compiler warning fix and adding back a removed max stride
check, nothing end user visible.
Regards, Joonas
PS. Travelling next week, so I'll skip PR unless there's
something big.
---
drm-intel-next-fixes-2018-10-04:
Compiler warning fix and adding
Do not rely on crtc->config any more. Remove the assertion from
ibx_pch_dpll_disable, because we the dpll state tracking should
already handle this case correctly.
Changes since v1:
- Fixup accidental early return in intel_prepare_shared_dpll, oops!
Signed-off-by: Maarten Lankhorst
Reviewed-by:
Chris Wilson writes:
> When RECOVERABLE is set, the kernel will attempt to automatically recover
> a context after a hang. But if it is unset, the kernel will ban the
> guilty context on a hang, preventing subsequent execution.
>
> Signed-off-by: Chris Wilson
> ---
> tests/gem_ctx_exec.c | 38 +
Op 04-10-18 om 14:57 schreef Ville Syrjälä:
> On Thu, Oct 04, 2018 at 11:45:59AM +0200, Maarten Lankhorst wrote:
>> Do not rely on crtc->config any more. Remove the assertion from
>> ibx_pch_dpll_disable, because we the dpll state tracking should
>> already handle this case correctly.
>>
>> Signed-
1 - 100 of 164 matches
Mail list logo
