On 5 September 2016 at 14:16, Peter Griffin wrote:
> ST_SLIM_REMOTEPROC must select REMOTEPROC, which exposes the following
> recursive dependency.
>
>From a humble skim through remoteproc, drm and a few other subsystems
I think the above is wrong. All the drivers (outside of remoteproc),
that I'v
HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/6cf9165c/attachment.html>
https://bugzilla.kernel.org/show_bug.cgi?id=68571
Weber K. changed:
What|Removed |Added
CC||weberkai at yahoo.com.br
--- Comment #78 from
https://bugzilla.kernel.org/show_bug.cgi?id=71891
Weber K. changed:
What|Removed |Added
CC||weberkai at yahoo.com.br
--- Comment #49 from
receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/89f84af7/attachment.html>
On 2016å¹´09æ18æ¥ 12:01, Tomasz Figa wrote:
> Hi Mark,
>
> On Sun, Sep 18, 2016 at 10:50 AM, Mark yao wrote:
>> On 2016å¹´09æ14æ¥ 20:54, Tomasz Figa wrote:
>>> Current code implements prepare_fb and cleanup_fb callbacks only to
>>> grab/release fb references, which is already done by atomic
https://bugzilla.kernel.org/show_bug.cgi?id=68571
--- Comment #79 from Weber K. ---
Forgot to mention: And relatime,lazytime,commit=60 in fstab
I believe maybe dpm need some fs information to work well.
--
You are receiving this mail because:
You are watching the assignee of the bug.
https://bugzilla.kernel.org/show_bug.cgi?id=71891
--- Comment #50 from Weber K. ---
Forgot to mention: And relatime,lazytime,commit=60 in fstab
I believe maybe dpm need some fs information to work well.
--
You are receiving this mail because:
You are watching the assignee of the bug.
next part --
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/522bddc9/attachment.html>
Hi Sean,
On 09/12/2016 09:51 PM, Sean Paul wrote:
> On Fri, Sep 9, 2016 at 5:44 AM, Yakir Yang wrote:
>> Signed-off-by: Yakir Yang
>> ---
>> Changes in v3:
>> - Suggested by Sean
>>
>> Changes in v2: None
>>
>> drivers/gpu/drm/bridge/analogix/analogix_dp_core.h | 3 ++-
>> drivers/gpu/drm/b
Hi Sean,
On 09/12/2016 09:52 PM, Sean Paul wrote:
> On Fri, Sep 9, 2016 at 5:45 AM, Yakir Yang wrote:
>> Make sure the request PSR state takes effect in analogix_dp_send_psr_spd()
>> function, or print the sink PSR error state if we failed to apply the
>> requested PSR setting.
>>
>> Signed-off-b
On 09/20/2016 03:12 AM, Fabio Estevam wrote:
> On Mon, Sep 19, 2016 at 4:02 PM, Randy Li wrote:
>
>> + vcc_sys_lcd: sys-lcd {
>> + compatible = "regulator-fixed";
>> + regulator-name = "vcc_5v";
>> + regulator-min-microvolt = <500>;
>> +
The timings issue is still here, this version is just some modifications
in dts file.
Randy Li (2):
ARM: dts: samsung: add rga-lvds panel in itop elite
drm/panel: Add support for Chunghwa CLAA070WP03XG panel
.../display/panel/chunghwa,claa070wp03xg.txt | 7 +++
arch/arm/boot/dts/exyno
The Chunghwa CLAA070WP03XG is a 7" 1280x800 panel, which can be
supported by the simple panel driver.
Signed-off-by: Randy Li
---
.../display/panel/chunghwa,claa070wp03xg.txt | 7 ++
drivers/gpu/drm/panel/panel-simple.c | 27 ++
2 files changed, 34 in
It is actually a lvds panel connected through a rga-lvds bridge.
The touchscreen is communicated with i2c bus but the driver is not
support now.
Signed-off-by: Randy Li
---
arch/arm/boot/dts/exynos4412-itop-elite.dts | 54 +++--
1 file changed, 52 insertions(+), 2 deletio
On 17/09/16 09:25 PM, Mario Kleiner wrote:
> Pre DCE4 hw doesn't have reliable pageflip completion
> interrupts, so instead polling for flip completion is
> used from within the vblank irq handler to complete
> page flips.
>
> This causes a race if pageflip ioctl is called close to
> vblank:
>
>
On 17/09/16 09:25 PM, Mario Kleiner wrote:
> Pre DCE4 hardware doesn't have (reliable) pageflip completion
> irqs, therefore we have to use the old polling method for flip
> completion handling in vblank irq.
>
> As vblank irqs fire a bit before start of vblank (when the
> linebuffer fifo read pos
On 17/09/16 09:41 PM, Mario Kleiner wrote:
> Hi Michel,
>
> all your patches, both the already merged kernel bits in radeon/amdgpu
> and also all the userspace bits in libdrm/ati-ddx/amdgpu-ddx are all
>
> Reviewed-and-tested-by: Mario Kleiner
>
> I successfully tested with old/current userspac
https://bugzilla.kernel.org/show_bug.cgi?id=68571
--- Comment #80 from Alex Deucher ---
dpm has nothing to do with the filesystem.
--
You are receiving this mail because:
You are watching the assignee of the bug.
.
-- next part --
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/803654ff/attachment.html>
On Sun, Sep 18, 2016 at 9:37 AM, Baoyou Xie wrote:
> In amdgpu_gfx.h, the declaration of amdgpu_gfx_parse_disable_cu()
> is incorrect.
>
> Signed-off-by: Baoyou Xie
> ---
> drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c | 4 +++-
> drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.h | 5 -
> 2 files changed, 7
On Mon, Sep 19, 2016 at 6:02 AM, Christian König
wrote:
> Am 18.09.2016 um 16:13 schrieb Baoyou Xie:
>>
>> We get 2 warnings when building kernel with W=1:
>> drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.c:146:5: warning: no previous
>> prototype for 'pool_to_domain' [-Wmissing-prototypes]
>> drivers
On Mon, Sep 19, 2016 at 6:01 AM, Christian König
wrote:
> Am 18.09.2016 um 16:09 schrieb Baoyou Xie:
>>
>> We get 7 warnings when building kernel with W=1:
>> drivers/gpu/drm/amd/amdgpu/amdgpu_device.c:1990:5: warning: no previous
>> prototype for 'amdgpu_pre_soft_reset' [-Wmissing-prototypes]
>>
https://bugzilla.kernel.org/show_bug.cgi?id=68571
--- Comment #81 from Weber K. ---
I think so as well...
Anyway before, I had to put this in kernel parameters, or else, black screen
and hang up during boot:
radeon.dpm=0
And since I put this in grub.cfg and fstab, never had a black screen aga
https://bugzilla.kernel.org/show_bug.cgi?id=68571
--- Comment #82 from Weber K. ---
Ooops...
Forgot to mention... I appreciate your good work supporting kernel!
Thank all of you very much!
--
You are receiving this mail because:
You are watching the assignee of the bug.
ceiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/94274066/attachment.html>
ause:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/6da7c42c/attachment.html>
.
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/7a070355/attachment.html>
https://bugzilla.kernel.org/show_bug.cgi?id=68571
--- Comment #83 from Michel Dänzer ---
(In reply to Weber K. from comment #81)
> And since I put this in grub.cfg and fstab, never had a black screen again:
> rootflags=relatime,lazytime,commit=60 iommu=noagp radeon.dpm=1
It could be iommu=noa
Hi Ville,
Thank you for the patch.
On Monday 19 Sep 2016 16:33:43 ville.syrjala at linux.intel.com wrote:
> From: Ville Syrjälä
>
> drm/drm_blend.c:207:5: warning: symbol 'drm_atomic_normalize_zpos' was not
> declared. Should it be static?
>
> Cc: Marek Szyprowski
> Cc: Benjamin Gaignard
>
Am Montag, den 19.09.2016, 22:48 +0200 schrieb Daniel Vetter:
> On Thu, Aug 11, 2016 at 11:18 AM, Lucas Stach
> wrote:
> > This allows the atomic helper to wait on them, instead of open-coding
> > the same in the imx-drm driver.
> >
> > Signed-off-by: Lucas Stach
> > ---
> > drivers/gpu/drm/imx
-- next part --
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/65ca951e/attachment.html>
sktop.org/archives/dri-devel/attachments/20160920/70146c43/attachment.html>
stih415 and stih416 platform are obsolete and no more supported.
Only stih407 and stih410 platform are maintained.
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/Kconfig| 6 +-
drivers/gpu/drm/sti/sti_compositor.c | 19 --
drivers/gpu/drm/sti/sti_hda.c
On Mon, Sep 19, 2016 at 04:33:45PM +0300, ville.syrjala at linux.intel.com
wrote:
> From: Ville Syrjälä
>
> drm/arm/malidp_planes.c:49:24: warning: symbol 'malidp_duplicate_plane_state'
> was not declared. Should it be static?
> drm/arm/malidp_planes.c:66:6: warning: symbol 'malidp_destroy_pl
On Mon, 19 Sep 2016, James Bottomley
wrote:
> On Mon, 2016-09-19 at 08:09 -0700, James Bottomley wrote:
>> On Sun, 2016-09-18 at 13:35 +0200, Thorsten Leemhuis wrote:
>> > Hi! James & Paulo: What's the current status of this?
>>
>> No, the only interaction has been the suggestion below for a rev
In the err_free_vram and err_release_fbi error paths in astfb_create(), we
attempt to free afbdev->sysram. The only jumps to these error paths occur
before we assign afbdev->sysram = sysram. Free sysram instead.
Signed-off-by: Andrew Donnellan
---
Found by Coverity Scan. Compile tested only.
R
On 20/09/16 11:56, Andrew Donnellan wrote:
> In the err_free_vram and err_release_fbi error paths in astfb_create(), we
> attempt to free afbdev->sysram. The only jumps to these error paths occur
> before we assign afbdev->sysram = sysram. Free sysram instead.
>
> Signed-off-by: Andrew Donnellan
>
On 18.09.2016 15:58, Baoyou Xie wrote:
> We get 1 warning when building kernel with W=1:
> drivers/gpu/drm/exynos/exynos_dp.c:46:5: warning: no previous prototype for
> 'exynos_dp_crtc_clock_enable' [-Wmissing-prototypes]
>
> In fact, this function is only used in the file in which it is
> declare
drm_atomic_state has a complicated single owner model that tracks the
single reference from allocation through to destruction on another
thread - or perhaps on a local error path. We can simplify this tracking
by using reference counting (at a cost of a few more atomics). This is
even more benefici
On 19.09.2016 15:53, Tobias Jakobi wrote:
> The mixer context struct already has a 'flags' field, so
> we can use it to store the 'interlace', 'vp_enabled' and
> 'has_sclk' booleans.
> We use the non-atomic helper functions to access these bits.
>
> Signed-off-by: Tobias Jakobi
Reviewed-by: Andrz
From: Markus Elfring
Date: Tue, 20 Sep 2016 10:48:04 +0200
A few update suggestions were taken into account
from static source code analysis.
Markus Elfring (6):
Use kmalloc_array() in mid_get_vbt_data_r10()
Rename a jump label in mid_get_vbt_data_r10()
Move a variable assignment in mid_ge
From: Markus Elfring
Date: Tue, 20 Sep 2016 08:54:07 +0200
A multiplication for the size determination of a memory allocation
indicated that an array data structure should be processed.
Thus use the corresponding function "kmalloc_array".
This issue was detected by using the Coccinelle software.
From: Markus Elfring
Date: Tue, 20 Sep 2016 09:09:10 +0200
Adjust a jump label according to the current Linux coding style convention.
Signed-off-by: Markus Elfring
---
drivers/gpu/drm/gma500/mid_bios.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/gma
From: Markus Elfring
Date: Tue, 20 Sep 2016 10:32:12 +0200
One local variable was set to an error code before a concrete
error situation was detected. Thus move the corresponding assignment into
an if branch to indicate a software failure there.
Signed-off-by: Markus Elfring
---
drivers/gpu/dr
From: Markus Elfring
Date: Tue, 20 Sep 2016 10:34:28 +0200
Adjust the indentation for a single function call parameter here.
Signed-off-by: Markus Elfring
---
drivers/gpu/drm/gma500/mid_bios.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/gma500/mid_bios.c
From: Markus Elfring
Date: Tue, 20 Sep 2016 10:36:19 +0200
A single error message should be sufficient to inform about
the detection of an unknown GCT revision at the end.
Thus return after the logging call in this case directly.
Signed-off-by: Markus Elfring
---
drivers/gpu/drm/gma500/mid_bio
From: Markus Elfring
Date: Tue, 20 Sep 2016 10:40:22 +0200
* Adjust a jump target.
* Delete the explicit initialisation for the local variable "ret"
which became unnecessary with this refactoring.
Signed-off-by: Markus Elfring
---
drivers/gpu/drm/gma500/mid_bios.c | 11 +--
1 file c
On 09/14/2016 07:07 AM, Kristian H. Kristensen wrote:
> The only current user of this open codes the ioctl. Let's add an entry
> point for this to libdrm.
Tested-by: Alexandre Courbot
Replaced one custom (staging) ioctl on Tegra with FB modifiers, noticed
this was missing!
>
> Signed-off-by: K
From: Michel Dänzer
It was only added in xserver 1.15. Fixes build against older xserver.
Reported-by: Pali Rohár
Signed-off-by: Michel Dänzer
---
configure.ac | 6 ++
src/radeon.h | 19 +++
2 files changed, 25 insertions(+)
diff --git a/configure.ac b/configure.ac
in
Hi,
I am experiencing display issues on HP Probook 4540s ( i5 based
laptop). Reading /var/log/kern.log I found the cause is probably
rooted in:
[drm] Wrong MCH_SSKPD value: 0x16040307
Just after the kernel takes control of the machine, I cannot read the
boot text. Successive pixel lines are shif
Hi Emil,
On Tue, 20 Sep 2016, Emil Velikov wrote:
> On 5 September 2016 at 14:16, Peter Griffin
> wrote:
> > ST_SLIM_REMOTEPROC must select REMOTEPROC, which exposes the following
> > recursive dependency.
> >
> From a humble skim through remoteproc, drm and a few other subsystems
> I think t
Hi
On Mon, Sep 5, 2016 at 10:30 AM, Dmitry Vyukov wrote:
> On Fri, Aug 19, 2016 at 7:10 PM, Alexander Potapenko
> wrote:
>> Hello,
>>
>> the program below triggers a NULL deref in DRM code when ran on QEMU:
>>
>> ===
>> BUG: unable to handle kerne
On Tue, 20 Sep 2016, Edward Bartolo wrote:
> I am experiencing display issues on HP Probook 4540s ( i5 based
> laptop). Reading /var/log/kern.log I found the cause is probably
> rooted in:
>
> [drm] Wrong MCH_SSKPD value: 0x16040307
Maybe, maybe not.
Please file a bug at [1], attaching the dmesg
stih415 and stih416 platform are obsolete and no more supported.
Only stih407 and stih410 platform are maintained.
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/Kconfig| 6 +-
drivers/gpu/drm/sti/Makefile | 1 -
drivers/gpu/drm/sti/sti_compositor.c
On Tue, 20 Sep 2016, Peter Griffin wrote:
> Hi Emil,
>
> On Tue, 20 Sep 2016, Emil Velikov wrote:
>
>> On 5 September 2016 at 14:16, Peter Griffin
>> wrote:
>> > ST_SLIM_REMOTEPROC must select REMOTEPROC, which exposes the following
>> > recursive dependency.
>
>
>> >
>> From a humble skim throu
On 19.09.2016 16:16, Tobias Jakobi wrote:
> Only manipulate the MXR_CFG and MXR_LAYER_CFG registers once
> in mixer_cfg_layer().
> Trigger this via atomic flush.
>
> Signed-off-by: Tobias Jakobi
> ---
> drivers/gpu/drm/exynos/exynos_mixer.c | 104
> ++
> drivers/g
On Tue, 20 Sep 2016, SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Tue, 20 Sep 2016 09:09:10 +0200
>
> Adjust a jump label according to the current Linux coding style convention.
Generally, please don't send patches to fix checkpatch issues in
existing code. Moreover, this particular "
On Tue, 20 Sep 2016, SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Tue, 20 Sep 2016 08:54:07 +0200
>
> A multiplication for the size determination of a memory allocation
> indicated that an array data structure should be processed.
> Thus use the corresponding function "kmalloc_array".
On Tue, 20 Sep 2016, SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Tue, 20 Sep 2016 10:36:19 +0200
>
> A single error message should be sufficient to inform about
> the detection of an unknown GCT revision at the end.
> Thus return after the logging call in this case directly.
Did you
On Tue, 20 Sep 2016, SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Tue, 20 Sep 2016 10:40:22 +0200
>
> * Adjust a jump target.
Please don't.
BR,
Jani.
>
> * Delete the explicit initialisation for the local variable "ret"
> which became unnecessary with this refactoring.
>
> Signed-
On 09/17/16 18:53, Wei Yongjun wrote:
> From: Wei Yongjun
>
> Fix the retrn value check which testing the wrong variable
> in dsi_bind().
Reviewed-by: Peter Ujfalusi
>
> Signed-off-by: Wei Yongjun
> ---
> drivers/gpu/drm/omapdrm/dss/dsi.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 delet
On Tue, Sep 20, 2016 at 11:21 AM, David Herrmann
wrote:
> Hi
>
> On Mon, Sep 5, 2016 at 10:30 AM, Dmitry Vyukov wrote:
>> On Fri, Aug 19, 2016 at 7:10 PM, Alexander Potapenko
>> wrote:
>>> Hello,
>>>
>>> the program below triggers a NULL deref in DRM code when ran on QEMU:
>>>
>>>
AFAICS the only drm driver built with "make defconfig" is i915.
CONFIG_DRM=y
CONFIG_DRM_MIPI_DSI=y
CONFIG_DRM_KMS_HELPER=y
CONFIG_DRM_KMS_FB_HELPER=y
CONFIG_DRM_FBDEV_EMULATION=y
CONFIG_DRM_I915=y
CONFIG_DRM_I915_USERPTR=y
CONFIG_DRM_PANEL=y
CONFIG_DRM_BRIDGE=y
Guenter
On Tue, Sep 20, 2016 at 2:
>> A multiplication for the size determination of a memory allocation
>> indicated that an array data structure should be processed.
>> Thus use the corresponding function "kmalloc_array".
>>
>> This issue was detected by using the Coccinelle software.
>
> Did you test this running on the hardware
>> A single error message should be sufficient to inform about
>> the detection of an unknown GCT revision at the end.
>> Thus return after the logging call in this case directly.
>
> Did you test this?
What is your software development opinion for this update suggestion?
Regards,
Markus
On Tue, Sep 20, 2016 at 01:03:06PM +0200, SF Markus Elfring wrote:
> Are you looking for any special information to improve
> a corresponding discussion?
If you restricted yourself to only sending bug fixes and not sending
any more cleanups that would be good.
Please stop sending clean up patches
On Tue, Sep 20, 2016 at 01:07:35PM +0300, Jani Nikula wrote:
> On Tue, 20 Sep 2016, SF Markus Elfring
> wrote:
> > From: Markus Elfring
> > Date: Tue, 20 Sep 2016 10:36:19 +0200
> >
> > A single error message should be sufficient to inform about
> > the detection of an unknown GCT revision at th
On 20.09.2016 13:23, Tobias Jakobi wrote:
> Hello Andrzej,
>
> first of all, I've noticed an error myself. mixer_disable() calls
> mixer_disable_plane(), so it should also be modified. I'll send a v2 later.
>
> Now to your points...
>
>
> Andrzej Hajda wrote:
>> On 19.09.2016 16:16, Tobias Jakobi w
s
Embedded Linux and Kernel engineering
http://free-electrons.com
-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/201
From: Colin Ian King
The check for an out of bound index into array interrupt_status_offsets
is off-by-one. Fix this and also don't compared to a hard coded array
size but use adev->mode_info.num_hpd instead.
Signed-off-by: Colin Ian King
---
drivers/gpu/drm/amd/amdgpu/dce_v6_0.c | 2 +-
1 fil
eneric (and not specific to a UV plane).
Tomi
-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/3b233bda/attachment-0001.sig>
Hi Emilio,
On 19 September 2016 at 17:40, Gustavo Padovan
wrote:
> Hi Emilio,
>
> 2016-09-19 Emilio López :
>
>> The ioctl name and description on the documentation block don't
>> match the ioctl being defined. This was probably overlooked while
>> renaming the ioctls during the sync file destag
ng this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/16594dcc/attachment-0001.html>
Hi Vincent,
On Tue, 20 Sep 2016, Vincent Abriou wrote:
> stih415 and stih416 platform are obsolete and no more supported.
> Only stih407 and stih410 platform are maintained.
>
> Signed-off-by: Vincent Abriou
Subject should have a v2, but apart from that: -
Acked-by: Peter Griffin
Peter.
right?
Tomi
-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/fb515fbd/attachment-0001.sig>
On 20.09.2016 14:34, Andrzej Hajda wrote:
> On 20.09.2016 13:23, Tobias Jakobi wrote:
>> Hello Andrzej,
>>
>> first of all, I've noticed an error myself. mixer_disable() calls
>> mixer_disable_plane(), so it should also be modified. I'll send a v2 later.
>>
>> Now to your points...
>>
>>
>> Andrzej
On Tue, 20 Sep 2016, Dan Carpenter wrote:
> Don't be a dummy... This is easy to review an it fixes a bug.
>
> I'm fine with you NAKing all these patches based on who they are from.
> I mostly just delete these without responding because the guy has
> history of introducing bugs and never listens
Hi
I've bisected back to this commit in the drm-intel-nightly branch
05a76d3d6ad1ee9f9814f88949cc9305fc165460 is the first bad commit
commit 05a76d3d6ad1ee9f9814f88949cc9305fc165460
Author: Lyude
Date: Wed Aug 17 15:55:57 2016 -0400
drm/i915/skl: Ensure pipes with changed wms get added to
On Tue, Sep 20, 2016 at 11:49 AM, Colin King
wrote:
> From: Colin Ian King
>
> The check for an out of bound index into array interrupt_status_offsets
> is off-by-one. Fix this and also don't compared to a hard coded array
> size but use adev->mode_info.num_hpd instead.
>
> Signed-off-by: Colin
the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/dada5c3f/attachment-0001.html>
On Tue, Sep 20, 2016 at 01:08:12PM +0300, Jani Nikula wrote:
> On Tue, 20 Sep 2016, SF Markus Elfring
> wrote:
> > From: Markus Elfring
> > Date: Tue, 20 Sep 2016 10:40:22 +0200
> >
> > * Adjust a jump target.
>
> Please don't.
>
Also there is nothing in CodingStyle that prohibits out: labels
from the encoder enable/disable and suspend/resume
handlers. */
which doesn't seem to hold true...
Tomi
-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/473f9457/attachment-0001.sig>
On Tue, Sep 20, 2016 at 2:08 PM, Jani Nikula
wrote:
> On Tue, 20 Sep 2016, Dan Carpenter wrote:
>> Don't be a dummy... This is easy to review an it fixes a bug.
In this particular case it might not be clear that an unknown GCT
version causes a complete GCT failure so both messages are useful.
lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
>
-- next part --
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/1ee00b89/attachment-0001.html>
Hello Andrzej,
first of all, I've noticed an error myself. mixer_disable() calls
mixer_disable_plane(), so it should also be modified. I'll send a v2 later.
Now to your points...
Andrzej Hajda wrote:
> On 19.09.2016 16:16, Tobias Jakobi wrote:
>> Only manipulate the MXR_CFG and MXR_LAYER_CFG re
Hi Baoyou,
On 20 September 2016 at 16:43, Gustavo Padovan wrote:
> 2016-09-18 Baoyou Xie :
>
>> We get 1 warning when building kernel with W=1:
>> drivers/dma-buf/sw_sync.c:87:23: warning: no previous prototype for
>> 'sync_timeline_create' [-Wmissing-prototypes]
>>
>> In fact, this function is
>>> A single error message should be sufficient to inform about
>>> the detection of an unknown GCT revision at the end.
>>> Thus return after the logging call in this case directly.
>>
>> Did you test this?
>>
>
> Don't be a dummy... This is easy to review an it fixes a bug.
Thanks for this kin
e bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/c35eac93/attachment-0001.html>
her rate than the ideal rate.
>
> Including the divider makes it compare the actual dot clock frequency
> if a given divider was used.
>
> Does this makes sense? Explaining this kind of makes my head spin...
Yes, sorry, I didn't remember rounded was actually the rounded parent
> If you restricted yourself to only sending bug fixes and not sending
> any more cleanups that would be good.
Thanks for another bit of constructive feedback.
> Please stop sending clean up patches.
This will not happen for a while.
I am in the process of informing various developers about so
Hi Rafael,
On 16 September 2016 at 00:44, Rafael Antognolli
wrote:
> When we merge several fences, if all of them are signaled already, we
> still keep one of them. So instead of using add_fence(), which will not
> increase the refcount of signaled fences, we should explicitly call
> fence_get()
pc, and have
a similar func to dispc_mgr_get_sync_lost_irq().
Tomi
-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/7411a729/attachment-0001.sig>
https://bugzilla.kernel.org/show_bug.cgi?id=172381
Bug ID: 172381
Summary: nouveau kernel module leaks memory on hibernate.
Product: Drivers
Version: 2.5
Kernel Version: 4.8rc6
Hardware: All
OS: Linux
Tree: Ma
From: Colin Ian King
The check for an out of bound index into array interrupt_status_offsets
is off-by-one. Fix this and also don't compared to a hard coded array
size but use ARRAY_SIZE instead.
Signed-off-by: Colin Ian King
---
drivers/gpu/drm/amd/amdgpu/dce_v6_0.c | 2 +-
1 file changed, 1
.
-- next part --
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/70d06f45/attachment-0001.html>
44fc000:mono pattern dw1
0x0158: 0x: MI_NOOP
--
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/c820904e/
--
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/dri-devel/attachments/20160920/4ccad3e7/attachment-0001.html>
Hi Andrzej,
Andrzej Hajda wrote:
> On 20.09.2016 14:34, Andrzej Hajda wrote:
>> On 20.09.2016 13:23, Tobias Jakobi wrote:
>>> Hello Andrzej,
>>>
>>> first of all, I've noticed an error myself. mixer_disable() calls
>>> mixer_disable_plane(), so it should also be modified. I'll send a v2 later.
>>
https://bugzilla.kernel.org/show_bug.cgi?id=141741
--- Comment #14 from Jose Barros ---
Created attachment 239331
--> https://bugzilla.kernel.org/attachment.cgi?id=239331&action=edit
dmesg | egrep 'drm|radeon' on 4.6.0-kali1-amd64
--
You are receiving this mail because:
You are watching the a
1 - 100 of 113 matches
Mail list logo
