On Wed, Jun 24, 2015 at 04:10:24PM -0300, Danilo Cesar Lemes de Paula wrote:
> Functions, Structs and Parameters definitions on kernel documentation
> are pure cosmetic, it only highlights the element.
>
> To ease the navigation in the documentation we should use inside
> those tags so readers ca
On Wed, Jun 24, 2015 at 10:06 PM, Alex Deucher wrote:
> On Fri, Jun 12, 2015 at 4:08 AM, Oded Gabbay wrote:
>> From: Ben Goz
>>
>> Signed-off-by: Ben Goz
>> Acked-by: Oded Gabbay
>
> This patch effectively reverts a fix for a hw bug IIRC.
>
> Alex
Hmm, let's see what Ben says about this. To wh
From: Ben Goz
v2: add missing MTYPE_NONCACHED enum
Signed-off-by: Ben Goz
Acked-by: Oded Gabbay
Reviewed-by: Alex Deucher
---
drivers/gpu/drm/amd/amdgpu/cikd.h | 6 +
drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c | 42 ++
drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
On Wed, Jun 24, 2015 at 10:31 PM, Alex Deucher wrote:
> On Wed, Jun 24, 2015 at 3:28 PM, Oded Gabbay wrote:
>> On Wed, Jun 24, 2015 at 10:25 PM, Alex Deucher
>> wrote:
>>> On Fri, Jun 12, 2015 at 4:08 AM, Oded Gabbay
>>> wrote:
From: Ben Goz
Signed-off-by: Ben Goz
Acked-
On Wed, Jun 24, 2015 at 10:25 PM, Alex Deucher wrote:
> On Fri, Jun 12, 2015 at 4:08 AM, Oded Gabbay wrote:
>> From: Ben Goz
>>
>> Signed-off-by: Ben Goz
>> Acked-by: Oded Gabbay
>> Reviewed-by: Alex Deucher
>
> Does this depend on some other patch? I'm getting a build failure:
> drivers/gpu
Thank you guys!
Annie Matheson
Intel Corporation
Phone: (503) 712-0586
Email: annie.j.matheson at intel.com
-Original Message-
From: Malladi, Kausal
Sent: Wednesday, June 24, 2015 8:41 AM
To: Roper, Matthew D; Sharma, Shashank
Cc: Matheson, Annie J; R, Dhanya p; dri-devel at lists.freed
On Saturday 20 June 2015 04:20 AM, Matt Roper wrote:
> On Mon, Jun 08, 2015 at 05:58:23PM -0700, Matt Roper wrote:
>> On Sat, Jun 06, 2015 at 05:34:45PM +0530, Sharma, Shashank wrote:
>>> Regards
>>> Shashank
>>>
>>> On 6/6/2015 6:31 AM, Matt Roper wrote:
On Thu, Jun 04, 2015 at 07:12:37PM +05
The maximum output width of one pipeline depends on the LayerMixer's
capability. It may be different on each target. Also, MDP5 doesn't
have vertical limitation in one frame, as long as the pixel clock
can be supported.
This change obtains the maximum LM resolution from configuration
table and tre
was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150624/eec5471b/attachment.html>
https://bugzilla.kernel.org/show_bug.cgi?id=100351
--- Comment #3 from yaroslav.sapozhnik at gmail.com ---
Created attachment 180871
--> https://bugzilla.kernel.org/attachment.cgi?id=180871&action=edit
Xorg log
--
You are receiving this mail because:
You are watching the assignee of the bug.
https://bugzilla.kernel.org/show_bug.cgi?id=100351
--- Comment #2 from yaroslav.sapozhnik at gmail.com ---
Created attachment 180861
--> https://bugzilla.kernel.org/attachment.cgi?id=180861&action=edit
dmesg
--
You are receiving this mail because:
You are watching the assignee of the bug.
https://bugzilla.kernel.org/show_bug.cgi?id=100351
Alex Deucher changed:
What|Removed |Added
CC||alexdeucher at gmail.com
--- Comment #1 f
--
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150624/ca1aeacc/attachment.html>
Hello Dave Airlie,
The patch dc5698e80cf7: "Add virtio gpu driver." from Sep 9, 2013,
leads to the following static checker warning:
drivers/gpu/drm/virtio/virtgpu_vq.c:119 virtio_gpu_free_vbufs()
warn: inconsistent returns 'spin_lock:&vgdev->free_vbufs_lock'.
drivers/gpu/drm/vir
sender is dereferrenced before NULL check
struct drm_device *dev = sender->dev;
and due to there is warning during static analysis:
warn: variable dereferenced before check 'sender'
__read_panel_data Function is called by mdfld_dsi_read_mcs
and there is a same check, Thus removing the check from
Functions, Structs and Parameters definitions on kernel documentation
are pure cosmetic, it only highlights the element.
To ease the navigation in the documentation we should use inside
those tags so readers can easily jump between methods directly.
This was discussed in 2014[1] and is implement
On Wed, Jun 24, 2015 at 3:46 PM, Oded Gabbay wrote:
> On Wed, Jun 24, 2015 at 10:06 PM, Alex Deucher
> wrote:
>> On Fri, Jun 12, 2015 at 4:08 AM, Oded Gabbay
>> wrote:
>>> From: Ben Goz
>>>
>>> Signed-off-by: Ben Goz
>>> Acked-by: Oded Gabbay
>>
>> This patch effectively reverts a fix for a
On Wed, Jun 24, 2015 at 3:39 PM, Oded Gabbay wrote:
> From: Ben Goz
>
> v2: add missing MTYPE_NONCACHED enum
>
> Signed-off-by: Ben Goz
> Acked-by: Oded Gabbay
> Reviewed-by: Alex Deucher
Applied. Thanks!
Alex
> ---
> drivers/gpu/drm/amd/amdgpu/cikd.h | 6 +
> drivers/gpu/drm/am
On Wed, Jun 24, 2015 at 3:28 PM, Oded Gabbay wrote:
> On Wed, Jun 24, 2015 at 10:25 PM, Alex Deucher
> wrote:
>> On Fri, Jun 12, 2015 at 4:08 AM, Oded Gabbay
>> wrote:
>>> From: Ben Goz
>>>
>>> Signed-off-by: Ben Goz
>>> Acked-by: Oded Gabbay
>>> Reviewed-by: Alex Deucher
>>
>> Does this d
On Fri, Jun 12, 2015 at 4:08 AM, Oded Gabbay wrote:
> From: Ben Goz
>
> Signed-off-by: Ben Goz
> Acked-by: Oded Gabbay
> Reviewed-by: Alex Deucher
Does this depend on some other patch? I'm getting a build failure:
drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c: In function
âgmc_v7_0_init_compute_v
On Fri, Jun 12, 2015 at 4:08 AM, Oded Gabbay wrote:
> From: Ben Goz
>
> Signed-off-by: Ben Goz
> Acked-by: Oded Gabbay
This patch effectively reverts a fix for a hw bug IIRC.
Alex
> ---
> drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff
On Fri, Jun 19, 2015 at 3:26 AM, Christian König
wrote:
> On 19.06.2015 06:05, Maninder Singh wrote:
>>
>> Use kzalloc for allocating one thing rather than
>> kcalloc(1...
>>
>> The semantic patch that makes this change is as follows:
>>
>> //
>> @@
>> @@
>>
>> - kcalloc(1,
>> + kzalloc(
>>
On Wed, Jun 24, 2015 at 1:56 PM, Maninder Singh
wrote:
> sender is dereferrenced before NULL check
> struct drm_device *dev = sender->dev;
>
> and due to there is warning during static analysis:
> warn: variable dereferenced before check 'sender'
>
> __read_panel_data Function is called by mdfld_
On Wed, Jun 24, 2015 at 10:54 AM, Maninder Singh
wrote:
> Hi,
>
>>> - if (!sender || !data_out || !len_out) {
>>> - DRM_ERROR("Invalid parameters\n");
>>> - return -EINVAL;
>>> - }
>>> -
>>
>>I would prefer to have these kind of checks where it actually matt
On Fri, Jun 19, 2015 at 11:36 AM, Jilai Wang wrote:
> Add plane properties "premultiplied/zpos/alpha" used in msm MDP
> driver to perform proper blending operation.
>
> Signed-off-by: Jilai Wang
> ---
> Documentation/DocBook/drm.tmpl | 23 +++
> 1 file changed, 23 insertions(
sender is dereferrenced before NULL check
struct drm_device *dev = sender->dev;
and due to this we get warning during static analysis:
warn: variable dereferenced before check 'sender'
__read_panel_data Function is called by mdfld_dsi_read_mcs
and there is a same check, Thus removing the check fr
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150624/ddb560c3/attachment-0001.html>
the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150624/07628e28/attachment.html>
for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150624/68a7b7d5/attachment.html>
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150624/a8e8c29a/attachment.html>
because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150624/56735ee3/attachment.html>
Hi Gustavo,
On 06/23/2015 11:35 PM, Gustavo Padovan wrote:
> From: Gustavo Padovan
>
> DP is required if either FIMD or DECON are selected so select it
> automatically.
It is not true, there are even SoCs with FIMD/DECON and without DP.
Regards
Andrzej
>
> Signed-off-by: Gustavo Padovan
> -
Hi Andrzej,
2015-06-24 Andrzej Hajda :
> Hi Gustavo,
>
> On 06/23/2015 11:35 PM, Gustavo Padovan wrote:
> > From: Gustavo Padovan
> >
> > DP is required if either FIMD or DECON are selected so select it
> > automatically.
>
> It is not true, there are even SoCs with FIMD/DECON and without DP.
On Wed, Jun 24, 2015 at 08:57:23AM +0100, Chris Wilson wrote:
> On Tue, Jun 23, 2015 at 10:46:00PM +0200, Daniel Vetter wrote:
> > Now that dp mst hotplug takes all locks we can amend the locking rules
> > for the iterators. This is needed before we can roll these out in the
> > atomic code to avoi
On Wed, Jun 24, 2015 at 08:44:47AM +0100, Chris Wilson wrote:
> On Tue, Jun 23, 2015 at 10:45:51PM +0200, Daniel Vetter wrote:
> > Hi all,
> >
> > Dave&I have been discussing connector hotplug and unplugging around DP MST
> > and
> > if there's one thing that's clear it's that we don't even reall
On Wed, Jun 24, 2015 at 9:10 AM, Maninder Singh
wrote:
> sender is dereferrenced before NULL check
> struct drm_device *dev = sender->dev;
>
> and due to this we get warning during static analysis:
> warn: variable dereferenced before check 'sender'
>
> __read_panel_data Function is called by mdf
To support mdp5 blending for mdp5 v1.5 and later
V1: Initial change
V2: After the stage number is increased to 7, BLENDx registers are
not continuous now. Using the offset for each BLEND stage
to fix it.
Signed-off-by: Jilai Wang
---
drivers/gpu/drm/msm/mdp/mdp5/mdp5.xml.h | 95
MDP1.5 can support 7 stages. Update xml file accordingly.
V1: Initial change
V2: After the stage number is increased to 7, BLENDx registers are
not continuous now. Using the offset for each BLEND stage
to fix it.
Signed-off-by: Jilai Wang
---
rnndb/mdp/mdp5.xml | 38 +++
It's probably allowed to leave old_fb set to garbage when unlocking,
but to prevent undefined behavior unset it just in case.
Also crtc_state->event could be NULL on memory allocation failure,
in which case event_space is increased for no reason.
Signed-off-by: Maarten Lankhorst
---
drivers/gpu
This change updates the old_fb pointer only after acquiring the plane lock,
if there are no properties the fb cannot have been changed either, so
this works out correctly.
Found in a discussion with Rob Clark.
Cc: Rob Clark
Signed-off-by: Maarten Lankhorst
---
drivers/gpu/drm/drm_atomic.c | 12
On Tue, Jun 23, 2015 at 10:46:00PM +0200, Daniel Vetter wrote:
> Now that dp mst hotplug takes all locks we can amend the locking rules
> for the iterators. This is needed before we can roll these out in the
> atomic code to avoid getting burried in WARNINGs.
>
> Signed-off-by: Daniel Vetter
> --
Hi,
>> - if (!sender || !data_out || !len_out) {
>> - DRM_ERROR("Invalid parameters\n");
>> - return -EINVAL;
>> - }
>> -
>
>I would prefer to have these kind of checks where it actually matters
>(ie. in __read_panel_data()). The saner thing would be to move
On Tue, Jun 23, 2015 at 10:45:51PM +0200, Daniel Vetter wrote:
> Hi all,
>
> Dave&I have been discussing connector hotplug and unplugging around DP MST and
> if there's one thing that's clear it's that we don't even really know where
> all
> the problems are. Hence first step is to figure that ou
next part --
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150624/f04960e7/attachment.sig>
lication/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150624/f649e289/attachment.sig>
This function keeps causing the kernel build to whine at me about
its 1200 byte stack frame.
(TPC_MAC = 8 * GPC_MAX = 8 * 32 = 256)
The following generates the same nv_wr32() operations in the same
order as the original, but without the 256-word dist[] array.
(I'm not sure if 4-bit fields inside
46 matches
Mail list logo
