The VGA arbiter selects a default VGA device that is enabled and
reachable via the legacy VGA resources (mem 0xa-0xb, io
0x3b0-0x3bb, io 0x3c0-0x3df, etc).
(As a special case for x86 and IA64, this can be overridden by
EFI.)
If there is no such device, e.g., because there's no enabled VGA
We have refactored and extended this - document it.
Signed-off-by: Daniel Axtens
---
Documentation/gpu/default_display.rst | 93 +++
Documentation/gpu/index.rst | 1 +
2 files changed, 94 insertions(+)
create mode 100644 Documentation/gpu/default_displ
On Wed, Aug 30, 2017, at 10:57 AM, Adam Borowski wrote:
> On Tue, Aug 29, 2017 at 08:56:15PM -0400, Jerome Glisse wrote:
> > I will wait for people to test and for result of my own test before
> > reposting if need be, otherwise i will post as separate patch.
> >
> > > But from a _very_ quick read-
This patch set:
- splits the default display handling out from VGA arbiter, into its
own file and behind its own Kconfig option (and gives the functions
better names).
- adds extra detection of default devices. To be nominated, the vga
arbiter and platform hooks must not have nominated
taskboxtes...@gmail.com liked your message with Boxer for Android.
On Sep 1, 2017 10:48 AM, Jeff Cook wrote:
On Wed, Aug 30, 2017, at 10:57 AM, Adam Borowski wrote:
> On Tue, Aug 29, 2017 at 08:56:15PM -0400, Jerome Glisse wrote:
> > I will wait for people to test and for result of my own test
Split the small bit of code that does default VGA handling out from
the arbiter. Add a Kconfig option to allow the kernel to be built
with just the default handling, or the arbiter and default handling.
While doing this, rename the functions from vga_(set_)default_device
to pci_(set_)default_displ
Hi Ard,
> If we are all in agreement that fixing X is not an option, I think
> this is a reasonable approach
This did come up in discussion at some earlier point in one of the many
spins we've done of this - I don't remember if you brought it up or
someone else did - but my concern was this: If i
On 1 September 2017 at 08:27, Daniel Axtens wrote:
> This patch set:
>
> - splits the default display handling out from VGA arbiter, into its
>own file and behind its own Kconfig option (and gives the functions
>better names).
>
> - adds extra detection of default devices. To be nominate
v2:
- add nv138 and drop nv13b chipsets (Ilia Mirkin)
- refactor out status variable and instead mask tsensor (Ilia Mirkin)
- switch SHADOWed state message away from nvkm_error() (Ilia Mirkin)
- rename internal temperature variable (Karol Herbst)
Signed-off-by: Rhys Kidd
---
.../gpu/drm/nouv
On Fri, Sep 1, 2017 at 3:47 AM, Hoegeun Kwon wrote:
> The gscaler has hardware rotation limits that need to be imported from
> dts. Parse them and add them to the property list.
>
> The rotation hardware limits are related to the cropped source size.
> When swap occurs, use rot_max size instead of
https://bugs.freedesktop.org/show_bug.cgi?id=102511
--- Comment #5 from Rob MacKinnon ---
Comments inline...
(In reply to Alex Deucher from comment #2)
> (In reply to Rob MacKinnon from comment #0)
> > * Actual Result:
> > - DPMS change level changes as expected.
> > - Dmesg:
> > [ 3302.814889]
https://bugs.freedesktop.org/show_bug.cgi?id=102511
--- Comment #4 from Rob MacKinnon ---
Created attachment 133934
--> https://bugs.freedesktop.org/attachment.cgi?id=133934&action=edit
dmesg.log
--
You are receiving this mail because:
You are the assignee for the bug.
https://bugs.freedesktop.org/show_bug.cgi?id=101769
--- Comment #2 from Matias N. Goldberg ---
Bah, nevermind. Either my bug is not related or it gets fixed the same way.
Updating kernel from 4.11.6 to 4.12.10 seems to have solved all the weird
performance issues.
--
You are receiving this mai
https://bugs.freedesktop.org/show_bug.cgi?id=102511
--- Comment #3 from Rob MacKinnon ---
Created attachment 133933
--> https://bugs.freedesktop.org/attachment.cgi?id=133933&action=edit
Xorg.0.log
--
You are receiving this mail because:
You are the assignee for the bug.___
https://bugs.freedesktop.org/show_bug.cgi?id=102500
Dieter Nützel changed:
What|Removed |Added
CC||deathsim...@vodafone.de
--- Comment #5
https://bugs.freedesktop.org/show_bug.cgi?id=101769
--- Comment #1 from Matias N. Goldberg ---
I was going to submit a new bug report, but I suspect this is the same issue.
I recently purchased an i7 7700 + Radeon RX 560 2GB to replace my older i5 4650
+ Radeon HD 7770.
But I noticed performance
https://bugs.freedesktop.org/show_bug.cgi?id=50655
Bryan Quigley changed:
What|Removed |Added
Resolution|--- |FIXED
Status|NEW
https://bugs.freedesktop.org/show_bug.cgi?id=102511
--- Comment #2 from Alex Deucher ---
(In reply to Rob MacKinnon from comment #0)
> * Actual Result:
> - DPMS change level changes as expected.
> - Dmesg:
> [ 3302.814889] [drm:btc_dpm_set_power_state [radeon]] *ERROR*
> rv770_restrict_performanc
https://bugs.freedesktop.org/show_bug.cgi?id=102511
--- Comment #1 from Alex Deucher ---
Please attach your xorg log and dmesg output with radeon.dpm=1 specified on
kernel command line in grub.
--
You are receiving this mail because:
You are the assignee for the bug.
https://bugs.freedesktop.org/show_bug.cgi?id=102511
Bug ID: 102511
Summary: RV770 error on change dpm balanced<->battery
Product: DRI
Version: unspecified
Hardware: x86-64 (AMD64)
OS: Linux (All)
Status: NEW
Fixes crash when/usr/share/libdrm/amdgpu.ids contains ASIC_ID_TABLE_NUM_ENTRIES
+ 1 entries.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=102432
Signed-off-by: Jan Vesely
---
Compile tested only.
amdgpu/amdgpu_asic_id.c | 15 ---
1 file changed, 8 insertions(+), 7 deletio
On Fri, Sep 01, 2017 at 07:51:41PM +0300, Ville Syrjälä wrote:
> On Fri, Sep 01, 2017 at 04:40:41PM +0200, Thierry Reding wrote:
> > From: Thierry Reding
> >
> > The callback is named .atomic_check, not .atomc_check.
> >
> > Signed-off-by: Thierry Reding
>
> For the series
> Reviewed-by: Ville
On Fri, Sep 01, 2017 at 07:00:52PM +0300, Ville Syrjälä wrote:
> On Fri, Sep 01, 2017 at 04:44:28PM +0200, Thierry Reding wrote:
> > From: Thierry Reding
> >
> > It's unusual to separate kerneldoc comments from the functions that they
> > describe by a blank line. Remove them.
> >
> > Signed-off
Linus Walleij writes:
> This replaces the custom connector in the TVE200 with the
> panel bridge helper. As long as we're just using panels
> and no other bridges, this works just fine.
Yeah, we definitely need a way to get the bus_flags/bus_formats from the
next link in the bridge chain in gene
Den 01.09.2017 09.28, skrev Laurent Pinchart:
Hi Noralf,
On Thursday, 31 August 2017 20:16:42 EEST Noralf Trønnes wrote:
Den 31.08.2017 12.18, skrev Laurent Pinchart:
On Monday, 28 August 2017 20:17:46 EEST Noralf Trønnes wrote:
Might as well embed drm_device since tinydrm_device (embeds pip
https://bugs.freedesktop.org/show_bug.cgi?id=102432
--- Comment #4 from Jan Vesely ---
(In reply to Michel Dänzer from comment #3)
> (In reply to Jan Vesely from comment #2)
> > The parsing mechanism is flimsy and adding an entry to
> > /usr/share/libdrm/amdgpu.ids crashes libdrm
>
> Can you sha
Boris Brezillon writes:
> Hi Eric,
>
> On Thu, 31 Aug 2017 10:03:23 -0700
> Eric Anholt wrote:
>
>> Boris Brezillon writes:
>> > +VID_DATATYPE(MIPI_DSI_PACKED_PIXEL_STREAM_16);
>> > + break;
>> > +
>> > + default:
>> > + dev_err
Linus Walleij writes:
> The silcon and components around the PL111 may require some
"silicon"
Other than that, while I'm still a little concerned about exposing too
many DRM formats on these boards, I think this is all worth landing at
this point. Series is:
Reviewed-by: Eric Anholt
Thanks!
On 01/09/17 01:48 PM, Thierry Reding wrote:
On Fri, Sep 01, 2017 at 12:14:23PM -0400, Tom St Denis wrote:
Signed-off-by: Tom St Denis
(v2): Drop Makefile change too.
---
drivers/gpu/drm/ttm/Makefile| 2 +-
drivers/gpu/drm/ttm/ttm_trace.h | 2 +-
2 files changed, 2 insertions(+), 2 dele
On Fri, Sep 01, 2017 at 12:14:23PM -0400, Tom St Denis wrote:
> Signed-off-by: Tom St Denis
>
> (v2): Drop Makefile change too.
> ---
> drivers/gpu/drm/ttm/Makefile| 2 +-
> drivers/gpu/drm/ttm/ttm_trace.h | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
Reviewed-by: Thierry Redin
On Fri, Sep 01, 2017 at 07:53:28PM +0300, ville.syrj...@linux.intel.com wrote:
> From: Ville Syrjälä
>
> Make __drm_object_property_get_value() static to fix the following
> sparse warning:
> drm_mode_object.c:250:5: warning: symbol '__drm_object_property_get_value'
> was not declared. Should it
On Fri, Sep 01, 2017 at 07:53:27PM +0300, ville.syrj...@linux.intel.com wrote:
> From: Ville Syrjälä
>
> drm_get_link_status_name() isn't used so kill it.
>
> Fixes the following sparse warning:
> drm_connector.c:618:1: warning: symbol 'drm_get_link_status_name' was not
> declared. Should it be
On Fri, Sep 01, 2017 at 07:53:26PM +0300, ville.syrj...@linux.intel.com wrote:
> From: Ville Syrjälä
>
> 'user_handles' needs a __user annotation for fix the following sparse
> warning:
> drm_syncobj.c:813:37: warning: incorrect type in argument 2 (different
> address spaces)
> drm_syncobj.c:813
On Fri, Sep 01, 2017 at 07:53:25PM +0300, ville.syrj...@linux.intel.com wrote:
> From: Ville Syrjälä
>
> Fix the following sparse warnings by making functions static:
> drm_syncobj.c:420:5: warning: symbol 'drm_syncobj_import_sync_file_fence' was
> not declared. Should it be static?
> drm_syncob
Signed-off-by: Tom St Denis
---
drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h
b/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h
index b1f97417241d..cef1a26deb2d 100644
--- a/drivers/gpu/drm/amd/am
Signed-off-by: Tom St Denis
---
drivers/gpu/drm/ttm/ttm_trace.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/ttm/ttm_trace.h b/drivers/gpu/drm/ttm/ttm_trace.h
index 23279b9b8e64..715ce68b7b33 100644
--- a/drivers/gpu/drm/ttm/ttm_trace.h
+++ b/drivers/gpu/drm
Am 01.09.2017 um 18:15 schrieb Deucher, Alexander:
-Original Message-
From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf
Of Tom St Denis
Sent: Friday, September 01, 2017 12:14 PM
To: amd-...@lists.freedesktop.org
Cc: StDenis, Tom; dri-devel@lists.freedesktop.org
Subjec
From: Ville Syrjälä
'user_handles' needs a __user annotation for fix the following sparse
warning:
drm_syncobj.c:813:37: warning: incorrect type in argument 2 (different address
spaces)
drm_syncobj.c:813:37:expected void const [noderef] *from
drm_syncobj.c:813:37:got void *user_handles
d
From: Ville Syrjälä
Make __drm_object_property_get_value() static to fix the following
sparse warning:
drm_mode_object.c:250:5: warning: symbol '__drm_object_property_get_value' was
not declared. Should it be static?
Cc: Daniel Vetter
Fixes: 4a97a3da420b ("drm: Don't update property values for
From: Ville Syrjälä
drm_get_link_status_name() isn't used so kill it.
Fixes the following sparse warning:
drm_connector.c:618:1: warning: symbol 'drm_get_link_status_name' was not
declared. Should it be static?
Cc: Manasi Navare
Fixes: 40ee6fbef75f ("drm: Add a new connector atomic property f
From: Ville Syrjälä
Fix the following sparse warnings by making functions static:
drm_syncobj.c:420:5: warning: symbol 'drm_syncobj_import_sync_file_fence' was
not declared. Should it be static?
drm_syncobj.c:441:5: warning: symbol 'drm_syncobj_export_sync_file' was not
declared. Should it be s
On Fri, Sep 01, 2017 at 04:40:41PM +0200, Thierry Reding wrote:
> From: Thierry Reding
>
> The callback is named .atomic_check, not .atomc_check.
>
> Signed-off-by: Thierry Reding
For the series
Reviewed-by: Ville Syrjälä
> ---
> include/drm/drm_modeset_helper_vtables.h | 2 +-
> 1 file cha
On Fri, Sep 1, 2017 at 11:02 AM, Christian König
wrote:
> Am 01.09.2017 um 16:49 schrieb Thierry Reding:
>>
>> From: Thierry Reding
>>
>> The header comment in include/trace/define_trace.h specifies that the
>> TRACE_INCLUDE_PATH needs to be relative to the define_trace.h header
>> rather than th
On Fri, Sep 01, 2017 at 03:57:06PM +0100, Chris Wilson wrote:
> We take advantage of that syncobj->fence is an RCU-protected pointer, and
> so sparse complains that it is lacking annotation.
>
> Signed-off-by: Chris Wilson
> ---
> include/drm/drm_syncobj.h | 2 +-
> 1 file changed, 1 insertion(+
https://bugs.freedesktop.org/show_bug.cgi?id=99195
johnrory.odw...@gmail.com changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|-
> -Original Message-
> From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf
> Of Tom St Denis
> Sent: Friday, September 01, 2017 12:14 PM
> To: amd-...@lists.freedesktop.org
> Cc: StDenis, Tom; dri-devel@lists.freedesktop.org
> Subject: [PATCH] drm/ttm: Fix trace include p
Signed-off-by: Tom St Denis
(v2): Drop Makefile change too.
---
drivers/gpu/drm/ttm/Makefile| 2 +-
drivers/gpu/drm/ttm/ttm_trace.h | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/ttm/Makefile b/drivers/gpu/drm/ttm/Makefile
index fd3da00c0bf2..f0549eab7
On 01/09/17 12:12 PM, Deucher, Alexander wrote:
-Original Message-
From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf
Of Tom St Denis
Sent: Friday, September 01, 2017 12:11 PM
To: amd-...@lists.freedesktop.org
Cc: StDenis, Tom; dri-devel@lists.freedesktop.org
Subject:
> -Original Message-
> From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf
> Of Tom St Denis
> Sent: Friday, September 01, 2017 12:11 PM
> To: amd-...@lists.freedesktop.org
> Cc: StDenis, Tom; dri-devel@lists.freedesktop.org
> Subject: [PATCH 2/2] drm/ttm: Fix trace inclu
On 01/09/17 11:02 AM, Christian König wrote:
Am 01.09.2017 um 16:49 schrieb Thierry Reding:
From: Thierry Reding
The header comment in include/trace/define_trace.h specifies that the
TRACE_INCLUDE_PATH needs to be relative to the define_trace.h header
rather than the trace file including it. M
On Fri, Sep 01, 2017 at 04:44:28PM +0200, Thierry Reding wrote:
> From: Thierry Reding
>
> It's unusual to separate kerneldoc comments from the functions that they
> describe by a blank line. Remove them.
>
> Signed-off-by: Thierry Reding
Series lgtm
Reviewed-by: Ville Syrjälä
> ---
> drive
https://bugs.freedesktop.org/show_bug.cgi?id=102507
--- Comment #4 from Dieter Nützel ---
This one is related, too:
https://lists.freedesktop.org/archives/amd-gfx/2017-August/012558.html
--
You are receiving this mail because:
You are the assignee for the bug.__
https://bugs.freedesktop.org/show_bug.cgi?id=102507
--- Comment #3 from Dieter Nützel ---
(In reply to Vedran Miletić from comment #2)
> Found it:
> https://lists.freedesktop.org/archives/amd-gfx/2017-August/012283.html
>
> Guess I should subscribe to one more list. :-)
Maybe or I _should_ make
On Thu, Jul 6, 2017 at 3:17 AM, Dave Airlie wrote:
> From: Dave Airlie
>
> This adds kernel semaphore support to the command submission
> interface in what should be a backwards compatible manner,
> it adds a new command submission API.
>
> Signed-off-by: Dave Airlie
> ---
> amdgpu/amdgpu.h
https://bugs.freedesktop.org/show_bug.cgi?id=102500
--- Comment #4 from Dieter Nützel ---
(In reply to Alex Deucher from comment #1)
> can you bisect?
Hello all,
I get the same on 'amd-staging-drm-next' since 1. of Sep (kernel build time: 1.
Sep 02:14 CEST) update, too. Will go to bisect in the
https://bugs.freedesktop.org/show_bug.cgi?id=102507
Vedran Miletić changed:
What|Removed |Added
URL||https://lists.freedesktop.o
https://bugs.freedesktop.org/show_bug.cgi?id=102507
--- Comment #1 from Dieter Nützel ---
I've reported this 2 times to amd-gfx-devel (Alex, Harry et al.) in the last
several days. Was '[amd-staging-drm-next] kernel crash with amdgpu on RX580 in
'drm_object_property_get_value'.
Should be 'drm_ob
Am 01.09.2017 um 16:49 schrieb Thierry Reding:
From: Thierry Reding
The header comment in include/trace/define_trace.h specifies that the
TRACE_INCLUDE_PATH needs to be relative to the define_trace.h header
rather than the trace file including it. Most instances get that wrong
and work around i
In order to silent sparse in dma_fence_get_rcu_safe(), we need to mark
the incoming fence object as being RCU protected and not the pointer to
the object.
Signed-off-by: Chris Wilson
---
include/linux/dma-fence.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/linux/d
We take advantage of that syncobj->fence is an RCU-protected pointer, and
so sparse complains that it is lacking annotation.
Signed-off-by: Chris Wilson
---
include/drm/drm_syncobj.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/drm/drm_syncobj.h b/include/drm/drm_s
From: Thierry Reding
The header comment in include/trace/define_trace.h specifies that the
TRACE_INCLUDE_PATH needs to be relative to the define_trace.h header
rather than the trace file including it. Most instances get that wrong
and work around it by adding the $(src) directory to the include p
From: Thierry Reding
The header comment in include/trace/define_trace.h specifies that the
TRACE_INCLUDE_PATH needs to be relative to the define_trace.h header
rather than the trace file including it. Most instances get that wrong
and work around it by adding the $(src) directory to the include p
From: Thierry Reding
The header comment in include/trace/define_trace.h specifies that the
TRACE_INCLUDE_PATH needs to be relative to the define_trace.h header
rather than the trace file including it. Most instances get that wrong
and work around it by adding the $(src) directory to the include p
From: Thierry Reding
The header comment in include/trace/define_trace.h specifies that the
TRACE_INCLUDE_PATH needs to be relative to the define_trace.h header
rather than the trace file including it. Most instances get that wrong
and work around it by adding the $(src) directory to the include p
From: Thierry Reding
The header comment in include/trace/define_trace.h specifies that the
TRACE_INCLUDE_PATH needs to be relative to the define_trace.h header
rather than the trace file including it. Most instances get that wrong
and work around it by adding the $(src) directory to the include p
From: Thierry Reding
The header comment in include/trace/define_trace.h specifies that the
TRACE_INCLUDE_PATH needs to be relative to the define_trace.h header
rather than the trace file including it. Most instances get that wrong
and work around it by adding the $(src) directory to the include p
From: Thierry Reding
The file uses inconsistent capitalization for TMDS. Since it is an
abbreviation, all uppercase is correct.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/drm_scdc_helper.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/drm_scdc_h
From: Thierry Reding
The error messages generated by the SCDC helpers are somewhat
inconsistent with other DRM errors and even with other errors in the
same file. Fix them all up to use a common format.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/drm_scdc_helper.c | 10 +-
1 file
From: Thierry Reding
It's unusual to separate kerneldoc comments from the functions that they
describe by a blank line. Remove them.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/drm_scdc_helper.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/gpu/drm/drm_scdc_helper.c
b/dri
From: Thierry Reding
The callback is named .atomic_check, not .atomc_check.
Signed-off-by: Thierry Reding
---
include/drm/drm_modeset_helper_vtables.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/drm/drm_modeset_helper_vtables.h
b/include/drm/drm_modeset_helper_
From: Thierry Reding
The for_each_crtc_in_state() is used to iterate over CRTCs rather than
connectors.
Signed-off-by: Thierry Reding
---
include/drm/drm_atomic.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/drm/drm_atomic.h b/include/drm/drm_atomic.h
index 8a580
On 01.08.2017 15:23, Philippe CORNU wrote:
> Based on patch "Convert drivers to explicit reset API" from Philipp Zabel
>
> Commit a53e35db70d1 ("reset: Ensure drivers are explicit when requesting
> reset lines") started to transition the reset control request API calls
> to explicitly state whether
On Fri, Sep 01, 2017 at 04:07:16PM +0900, Hoegeun Kwon wrote:
> If scanout started, we should reduce etime by delta_ns. But delta_ns
> is negative if scanout has not started. If delta_ns is negative,
> subtraction of delta_ns from etime increases etime. This is wrong, the
> etime should not be incr
On 01.08.2017 15:23, Philippe CORNU wrote:
> This patch cleans up the Synopsys mipi dsi register list:
> - rename registers according to the Synopsys documentation
> (1.30 & 1.31)
> - fix typos
> - re-order registers for a better coherency
>
> Signed-off-by: Philippe CORNU
> Reviewed-by: Laurent
On 01.08.2017 15:23, Philippe CORNU wrote:
> Constify dw_mipi_dsi_bridge_funcs as these functions are not supposed
> to change at runtime.
>
> Signed-off-by: Philippe CORNU
> Reviewed-by: Laurent Pinchart
> ---
> drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c | 2 +-
> 1 file changed, 1 insertion
Hi Peter,
Thank you for the patch.
On Tuesday, 29 August 2017 10:32:16 EEST Peter Ujfalusi wrote:
> Sort the dssdev array based on DT aliases.
>
> With this change we can remove the panel ordering from dss/display.c and
> have all sorting related to dssdevs in one place.
>
> Signed-off-by: Pet
Hi Peter,
On Tuesday, 29 August 2017 10:32:11 EEST Peter Ujfalusi wrote:
> Hi
>
> The series adds support for changing the order of the displays defined by DT
> display aliases.
>
> The motivation to do such a thing is that for example the fb emulation is
> treating the first display/crtc as the
Hi Peter,
Thank you for the patch.
On Tuesday, 29 August 2017 10:32:14 EEST Peter Ujfalusi wrote:
> Instead of reaching back to DSS to iterate through the dss_devices every
> time, use an internal array where we store the available and usable
> dss_devices.
>
> Signed-off-by: Peter Ujfalusi
> -
Hi Peter,
Thank you for the patch.
On Tuesday, 29 August 2017 10:32:13 EEST Peter Ujfalusi wrote:
> If we allocate the drm_device earlier we can just return the error code
> without the need to use goto.
> Do the unref of the drm_device as a last step when cleaning up. This will
> make the drm_de
Hi Peter,
Thank you for the patch.
On Tuesday, 29 August 2017 10:32:12 EEST Peter Ujfalusi wrote:
> It makes the cleanup paths a bit cleaner.
But it also goes against a proper implementation of object lifetime management
:-( In DRM driver private structures are accessible from file operations,
On Fri, Sep 1, 2017 at 3:13 AM, Laurent Pinchart
wrote:
> Hi Nicolas,
>
> On Thursday, 31 August 2017 19:12:58 EEST Nicolas Dufresne wrote:
>> Le jeudi 31 août 2017 à 17:28 +0300, Laurent Pinchart a écrit :
>> >> e.g. if I have two devices which support MODIFIER_FOO, I could attempt
>> >> to share
Hi Archit,
Gentle reminder :-)
Do not hesitate to send me any comments so I can update these patches.
Note that the "Constify funcs structures" patch is now obsolete as a
similar ones from Bhumika Goyal has been posted then integrated recently.
Many thanks for your support,
Philippe :-)
On 08
Hi Noralf,
Many thanks for your patchset (and sorry for this late reply due to
summer holidays :-),
On 08/13/2017 03:32 PM, Noralf Trønnes wrote:
> drm_fb_cma_create() is just a wrapper around drm_gem_fb_create() now,
> so use the function directly.
>
> Cc: Yannick Fertre
> Cc: Philippe Cornu
On 8/31/2017 5:08 PM, Mark Brown wrote:
> On Fri, Aug 18, 2017 at 02:10:30PM -0400, Alex Deucher wrote:
>
>> +++ b/sound/soc/amd/Kconfig
>> @@ -2,3 +2,10 @@ config SND_SOC_AMD_ACP
>> tristate "AMD Audio Coprocessor support"
>> help
>> This option enables ACP DMA support on AMD pl
https://bugs.freedesktop.org/show_bug.cgi?id=102242
Jari Tahvanainen changed:
What|Removed |Added
QA Contact|intel-gfx-bugs@lists.freede |
|sktop.org
https://bugs.freedesktop.org/show_bug.cgi?id=102500
--- Comment #3 from Arek Ruśniak ---
Created attachment 133923
--> https://bugs.freedesktop.org/attachment.cgi?id=133923&action=edit
dmesg for first bad commit
autostart throu gdm to gnome3-session failed so I booted into multi-user (no vm
fa
https://bugs.freedesktop.org/show_bug.cgi?id=102500
--- Comment #2 from Arek Ruśniak ---
Ok, sometimes gnome session refuse to work or even crashes on it (leds blinking
on my kb)
1753d85bc82849deeb68cb5d7883207f0acbddc4 is the first bad commit
commit 1753d85bc82849deeb68cb5d7883207f0acbddc4
Au
We need to be able to get the connector out of a dumb VGA
for the PL111 driver. This is because we are using the
connector in the driver when initializing the display
pipe.
Signed-off-by: Linus Walleij
---
This is a bit ugly but not uglier than how we are reaching
into the connector of the panels
If we cannot find a panel, assume that the output from the
PL111 is connected directly to a "dumb" VGA connector,
so look up the connector from that bridge.
Signed-off-by: Linus Walleij
---
This is how the new API is used in the PL111.
---
drivers/gpu/drm/pl111/Kconfig | 1 +
drivers/gpu/drm
This adds device tree bindings for the Texas Instruments
THS8134A and THS8134B VGA DACs by extending and renaming the
existing bindings for THS8135.
These DACs are used for the VGA outputs on the ARM reference
designs such as Integrator, Versatile and RealView.
Cc: devicet...@vger.kernel.org
Cc:
This extends the dumb VGA DAC bridge to handle the THS8134A
and THS8134B VGA DACs in addition to those already handled.
The THS8134A, THS8134B and as it turns out also THS8135 need to
have data clocked out at the negative edge of the clock pulse,
since they clock it into the DAC at the positive ed
The ARM reference designs in the Versatile family: Integrator,
Versatile and RealView can make use of the new DRM driver as well.
We just need to create a bit of platform-specific code for them
that we isolate to its own file.
Signed-off-by: Linus Walleij
---
ChangeLog v1->v2:
- Rebase on the oth
The header file contains prototypes for two nonexisting
functions. Get rid of them.
Reviewed-by: Eric Anholt
Signed-off-by: Linus Walleij
---
ChangeLog v1->v2:
- Collect Eric's ACK.
---
drivers/gpu/drm/pl111/pl111_drm.h | 4
1 file changed, 4 deletions(-)
diff --git a/drivers/gpu/drm/pl11
The old codebase has a delay between enabling and powering up the
PL11x.
According to the manual for PL110, ARM DDI 0161E page 1-5 and
the PL111 manual ARM DDI 0293C page 1-6, the power sequence should
be such that once Vdd is stable (which we assume it is at boot)
LCDEN is enabled first and then
The silcon and components around the PL111 may require some
variants to perform special set-up of the display. Add two
callbacks to manage this.
Signed-off-by: Linus Walleij
---
ChangeLog v1->v2:
- Rebase on other display driver changes.
---
drivers/gpu/drm/pl111/pl111_display.c | 6 ++
driv
This is the v2 patch set fixing the review comments (hopefully).
Mainly I had to drop the cleverness of wrapping power enablement
inside the panel prepare/enable pair and fall back to a 20ms
delay as in the framebuffer driver. I hope everyone survives
this.
Linus Walleij (7):
drm/pl111: Cleanup
This replaces the custom connector in the PL111 with the
panel bridge helper.
This works nicely for all standard panels, but since there
are several PL11x-based systems that will need to use the dumb
VGA connector bridge we use drm_of_find_panel_or_bridge()
and make some headroom for dealing with
This adds all the main control registers to the debugfs
register file. This was helpful for my debugging so it will
likely help others as well.
Signed-off-by: Linus Walleij
---
ChangeLog v1->v2:
- Collect Eric's ACK.
---
drivers/gpu/drm/pl111/pl111_debugfs.c | 6 ++
1 file changed, 6 inserti
We detect and enable the use of the PL110 variant, an earlier
incarnation of PL111. The only real difference is that the
control and interrupt enable registers have swapped place.
The Versatile AB and Versatile PB have a variant inbetween
PL110 and PL111, it is PL110 but they have already swapped t
On Thu, Aug 31, 2017 at 7:58 PM, Eric Anholt wrote:
> Linus Walleij writes:
>
>> The ARM reference designs in the Versatile family: Integrator,
>> Versatile and RealView can make use of the new DRM driver as well.
>> We just need to create a bit of platform-specific code for them
>> that we isola
1 - 100 of 116 matches
Mail list logo
