On 26/04/17 19:46, Oscar Salvador wrote:
> This patch replaces the symbolic permissions with the numeric ones,
> and adds me to the authors too.
>
> Signed-off-by: Oscar Salvador
> ---
> drivers/gpu/drm/nouveau/nouveau_hwmon.c | 14 +++---
> 1 file changed, 7 insertions(+), 7 deletions
The name of the patch is misleading since you also add support for
pwm1_min/max.
Could you add change the title to "nouveau/hwmon: expose the auto_point
and pwm_min/max attrs"? And while you are at it, please change every
commit title to nouveau/hwmon instead of nouveau_hwmon.
On 26/04/17 19:46,
On 26/04/17 19:46, Oscar Salvador wrote:
> This patch removes old code related to the old api and transforms the
> functions for the new api. It also adds the .write and .read operations.
>
> Signed-off-by: Oscar Salvador
> ---
> drivers/gpu/drm/nouveau/nouveau_hwmon.c | 722
> +++--
On 26/04/17 19:46, Oscar Salvador wrote:
> This patch introduces the nouveau_hwmon_ops structure, sets up
> .is_visible and .read_string operations and adds all the functions
> for these operations.
> This is also a preparation for the next patches, where most of the
> work is being done.
> This co
https://bugs.freedesktop.org/show_bug.cgi?id=99784
--- Comment #12 from lazane...@gmail.com ---
Problem persists with Ubuntu 17.10 / DRM 2.49.0 / 4.11.0-041100rc8-lowlatency,
LLVM 4.0.0
Tested all resolution options for all three of the monitors, and has been able
to output either on HDMI and VGA
https://bugs.freedesktop.org/show_bug.cgi?id=100896
Michel Dänzer changed:
What|Removed |Added
Product|DRI |xorg
Assignee|dri-devel@lists
Resending for CI.
Pandiyan, Dhinakaran (4):
drm: Add driver-private objects to atomic state
drm/dp: Introduce MST topology state to track available link bandwidth
drm/dp: Add DP MST helpers to atomically find and release vcpi slots
drm/dp: Track MST link bandwidth
drivers/gpu/drm/drm_ato
From: "Pandiyan, Dhinakaran"
Link bandwidth is shared between multiple display streams in DP MST
configurations. The DP MST topology manager structure maintains the
shared link bandwidth for a primary link directly connected to the GPU. For
atomic modesetting drivers, checking if there is suffici
From: "Pandiyan, Dhinakaran"
drm_dp_atomic_find_vcpi_slots() should be called from ->atomic_check() to
check there are sufficient vcpi slots for a mode and to add that to the
state. This should be followed by a call to drm_dp_mst_allocate_vcpi()
in ->atomic_commit() to initialize a struct vcpi fo
From: "Pandiyan, Dhinakaran"
It is necessary to track states for objects other than connector, crtc
and plane for atomic modesets. But adding objects like DP MST link
bandwidth to drm_atomic_state would mean that a non-core object will be
modified by the core helper functions for swapping and cle
On Mon, 2017-05-01 at 10:24 +0200, Maarten Lankhorst wrote:
> Op 29-04-17 om 01:14 schreef Dhinakaran Pandiyan:
> > From: "Pandiyan, Dhinakaran"
> >
> > Use the added helpers to track MST link bandwidth for atomic modesets.
> > Link bw is acquired in the ->atomic_check() phase when CRTCs are being
From: "Pandiyan, Dhinakaran"
Use the added helpers to track MST link bandwidth for atomic modesets.
Link bw is acquired in the ->atomic_check() phase when CRTCs are being
enabled with drm_atomic_find_vcpi_slots(). Similarly, link bw is released
during ->atomic_check() with the connector helper ca
Hi Ben,
A kernel bug report was opened against Ubuntu [0]. After a kernel
bisect, it was found that reverting the following commit resolved this bug:
commit da2ba564a6dcf46df4f828624ff55531ff11d5b0
Author: Ben Skeggs
Date: Thu Apr 6 10:35:26 2017 +1000
drm/nouveau: initial support (displ
https://bugs.freedesktop.org/show_bug.cgi?id=98520
--- Comment #38 from mburns92...@yahoo.com ---
Oops! There is a regression in kernel 4.11 between rc8 git0.1 and git2.2.
--
You are receiving this mail because:
You are the assignee for the bug.___
dr
On Fri, Apr 28, 2017 at 5:42 PM, Eric Anholt wrote:
> For the Raspberry Pi's bindings, the power domain also implicitly
> turns on the clock and deasserts reset, but for the new Cygnus port we
> start representing the clock in the devicetree.
>
> v2: Document the clock-names property, check for -E
On Mon, May 01, 2017 at 03:37:53PM +0200, Maarten Lankhorst wrote:
> This is only used in i915, which had used its own non-taomic way to
> deal with the picture aspect ratio. Move selected aspect_ratio to
> atomic state and use the atomic state in the affected i915 connectors.
>
Please correct the
Thanks Dan!
Reviewed-by: Sinclair Yeh
On Thu, Apr 27, 2017 at 12:12:08PM +0300, Dan Carpenter wrote:
> If vmalloc() fails then we need to a bit of cleanup before returning.
>
> Fixes: fb1d9738ca05 ("drm/vmwgfx: Add DRM driver for VMware Virtual GPU")
> Signed-off-by: Dan Carpenter
>
> diff --
On Mon, 2017-05-01 at 18:46 +0200, SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Mon, 1 May 2017 17:08:56 +0200
>
> A few single characters (line breaks) should be put into a sequence.
> Thus use the corresponding function "seq_putc".
>
> This issue was detected by using the Coccinelle
From: Markus Elfring
Date: Mon, 1 May 2017 17:15:43 +0200
Some data were put into a sequence by two separate function calls.
Print the same data by a single function call instead.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring
---
drivers/gpu/drm/drm_d
From: Markus Elfring
Date: Mon, 1 May 2017 18:38:08 +0200
A few update suggestions were taken into account
from static source code analysis.
Markus Elfring (4):
Use seq_putc() in drm_dp_mst_dump_topology()
Combine two seq_printf() calls into one call in drm_dp_mst_dump_topology()
Replace s
From: Markus Elfring
Date: Mon, 1 May 2017 18:22:52 +0200
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
The script “checkpatch.pl” pointed information out like the following.
Comparison to NULL could be written …
Thus fix the affected source code plac
From: Markus Elfring
Date: Mon, 1 May 2017 17:23:24 +0200
Strings which did not contain data format specifications should be put
into a sequence. Thus use the corresponding function "seq_puts".
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring
---
drivers
From: Markus Elfring
Date: Mon, 1 May 2017 17:08:56 +0200
A few single characters (line breaks) should be put into a sequence.
Thus use the corresponding function "seq_putc".
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring
---
drivers/gpu/drm/drm_dp_mst
https://bugs.freedesktop.org/show_bug.cgi?id=100896
Bug ID: 100896
Summary: Can't set screen scaling modes for external monitors.
Product: DRI
Version: unspecified
Hardware: Other
OS: All
Status: NEW
Sever
SDVO was the last connector that's still using the legacy paths
for properties, and this is with a reason!
This connector implements a lot of properties dynamically,
and some of them shared with the digital connector state,
so sdvo_connector_state subclasses intel_digital_connector_state.
set_pro
intel_hdmi supports 3 properties, force_audio, broadcast rgb and
scaling mode. The last one is only created for eDP, so the is_eDP
in set_property is not required.
panel fitting and broadcast rgb are straightforward and only requires
changing compute_config.
force_audio is also used to force DVI
Signed-off-by: Maarten Lankhorst
Reviewed-by: Daniel Vetter
---
drivers/gpu/drm/i915/intel_dsi.c | 38 +-
1 file changed, 5 insertions(+), 33 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_dsi.c b/drivers/gpu/drm/i915/intel_dsi.c
index 8617d996abe5..40f
Signed-off-by: Maarten Lankhorst
Reviewed-by: Daniel Vetter
---
drivers/gpu/drm/i915/intel_lvds.c | 35 +--
1 file changed, 5 insertions(+), 30 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_lvds.c
b/drivers/gpu/drm/i915/intel_lvds.c
index a3f9216e32eb..1d
Always detect if audio is available during edid detection.
With less magic switching it's easier to convert the dp connector
properties to atomic.
Signed-off-by: Maarten Lankhorst
Reviewed-by: Daniel Vetter
---
drivers/gpu/drm/i915/intel_dp.c | 38 --
1 file
Do the same as other connectors, attempt to detect hdmi audio in
the detect() callback, and only use the force_audio property as
override. Compute has_audio in pipe_config, and use that value
instead of the probed value directly.
Signed-off-by: Maarten Lankhorst
Reviewed-by: Daniel Vetter
---
d
Some atomic properties are common between the various kinds of
connectors, for example a lot of them use panel fitting mode.
It makes sense to put a lot of it in a common place, so each
connector can use it while they're being converted.
Implement the properties required for the connectors:
- scal
Some connectors may not allow all scaling mode properties, this function will
allow
creating the scaling mode property with only the supported subset. It also
wires up
this state for atomic.
This will make it possible to convert i915 connectors to atomic.
Changes since v1:
- Add DRM_MODE_PROP_
intel_dp supports 3 properties, scaling mode, broadcast rgb and
force_audio. intel_digital_connector handles the plumbing,
so we only have to hook this up in compute_config and init.
Changes since v1:
- Remove limited_color_range too, unused. (danvet)
Signed-off-by: Maarten Lankhorst
Reviewed-by
This is required to allow setting i915 connector properties using the atomic
ioctl.
This version shouldn't cause unbisectable changes in between.
CHanges since last version:
- Split out picture_aspect_ratio and atomic scaling mode property to separate
commits.
- Make sure it bisects correctly,
The first step in converting connector properties to atomic is
wiring up the atomic state. We're still not completely supoprting
the scaling mode in the atomic case, but this is the first step
towards it.
Signed-off-by: Maarten Lankhorst
---
drivers/gpu/drm/i915/intel_dp.c | 13 +
None of the intel connectors can use all types of scaling modes,
so only try the ones that are possible. This is another preparation
for connectors towards conversion to atomic.
Signed-off-by: Maarten Lankhorst
---
drivers/gpu/drm/i915/intel_dp.c | 29 -
drivers/gpu
This is only used in i915, which had used its own non-taomic way to
deal with the picture aspect ratio. Move selected aspect_ratio to
atomic state and use the atomic state in the affected i915 connectors.
Signed-off-by: Maarten Lankhorst
---
drivers/gpu/drm/drm_atomic.c | 4
drivers/g
https://bugs.freedesktop.org/show_bug.cgi?id=100239
--- Comment #7 from Bruno Jacquet (Xaapyks) ---
Thank you very much Michel, indeed masking this extension produces a correct
trace that can be replayed to reproduce the scope bug.
So here is a new apitrace showing the bug : http://dl.free.fr/oD
https://bugs.freedesktop.org/show_bug.cgi?id=100534
Michel Dänzer changed:
What|Removed |Added
Product|DRI |xorg
Version|XOrg git
https://bugs.freedesktop.org/show_bug.cgi?id=100891
--- Comment #3 from Edward O'Callaghan ---
Created attachment 131165
--> https://bugs.freedesktop.org/attachment.cgi?id=131165&action=edit
kern4.10_dmesg
--
You are receiving this mail because:
You are the assignee for the bug.__
https://bugs.freedesktop.org/show_bug.cgi?id=100891
--- Comment #2 from Edward O'Callaghan ---
Same issue:
$ lspci -vvnn -s 05:00.0
05:00.0 Display controller [0380]: Advanced Micro Devices, Inc. [AMD/ATI] Topaz
XT [Radeon R7 M260/M265 / M340/M360 / M440/M445] [1002:6900]
Subsystem: Leno
Op 29-04-17 om 01:14 schreef Dhinakaran Pandiyan:
> From: "Pandiyan, Dhinakaran"
>
> Use the added helpers to track MST link bandwidth for atomic modesets.
> Link bw is acquired in the ->atomic_check() phase when CRTCs are being
> enabled with drm_atomic_find_vcpi_slots(). Similarly, link bw is re
42 matches
Mail list logo
