Hi Keith,
If I read the patch correctly, the plane has been interpreted as the same as
connector, and the stackIndex is the index of connector of current device.
Is it by intentional?
If the hardware don't have underlay/overlay supported, is it better to always
report plane 0 rather than pretend
On Sun, Mar 25, 2018 at 12:59:56PM +0200, Christian König wrote:
> Add a peer2peer flag noting that the importer can deal with device
> resources which are not backed by pages.
>
> Signed-off-by: Christian König
Um strictly speaking they all should, but ttm never bothered to use the
real interfa
Hi Kieran,
On Wednesday, 28 March 2018 22:04:49 EEST Kieran Bingham wrote:
> On 28/03/18 13:27, Kieran Bingham wrote:
> > On 26/02/18 21:45, Laurent Pinchart wrote:
> >> The entities in the pipeline are all started when the LIF is setup.
> >> Remove the outdated comment that state otherwise.
> >>
https://bugs.freedesktop.org/show_bug.cgi?id=105797
Timothy Arceri changed:
What|Removed |Added
Blocks||101747, 101384, 103915,
https://bugs.freedesktop.org/show_bug.cgi?id=103915
Timothy Arceri changed:
What|Removed |Added
Depends on||105797
Referenced Bugs:
https://bugs
https://bugs.freedesktop.org/show_bug.cgi?id=101384
Timothy Arceri changed:
What|Removed |Added
Depends on||105797
Referenced Bugs:
https://bugs
https://bugs.freedesktop.org/show_bug.cgi?id=101675
Timothy Arceri changed:
What|Removed |Added
Depends on||105797
Referenced Bugs:
https://bugs
Hi Linus,
Nothing serious, two amdkfd and two tegra fixes.
I'll be submitting -next early, in the next couple of hours.
Dave.
The following changes since commit 3eb2ce825ea1ad89d20f7a3b5780df850e4be274:
Linux 4.16-rc7 (2018-03-25 12:44:30 -1000)
are available in the Git repository at:
gi
https://bugs.freedesktop.org/show_bug.cgi?id=104520
--- Comment #19 from Gennady ---
Linux p50-debian 4.15.0-1-amd64 #1 SMP Debian 4.15.4-1 (2018-02-18) x86_64
GNU/Linux
ii libgl1-mesa-dri:amd64 17.3.7-1
amd64free implementation of the
https://bugs.freedesktop.org/show_bug.cgi?id=105425
--- Comment #9 from MirceaKitsune ---
And we have a verdict. The influential factor is by far the anti-aliasing, at
least in the case of Xonotic. The other cvars I previously mentioned have
absolutely no effect on the frequency of this freeze.
https://bugs.freedesktop.org/show_bug.cgi?id=104520
--- Comment #18 from Gennady ---
GPU HANG: ecode 9:0:0x85db, in Xorg [1035], reason: Hang on rcs0, action:
reset
Kernel: 4.15.0-1-amd64
Time: 1522274173 s 645209 us
Boottime: 262 s 812221 us
Uptime: 260 s 247539 us
Active process (on ring re
https://bugs.freedesktop.org/show_bug.cgi?id=104520
--- Comment #17 from gennady.kup...@gmail.com ---
I can not attach error log, please let me know if it is necessary.
Problem is reproducible, if I run certain qt5 app xorg hangs.
If I wait for a minute, it restarts.
--
You are receiving this
https://bugs.freedesktop.org/show_bug.cgi?id=104520
--- Comment #16 from gennady.kup...@gmail.com ---
I have same or very similar problem on Skylake.
--
You are receiving this mail because:
You are the assignee for the bug.___
dri-devel mailing list
dr
Cosmetic change.
Reviewed-by: Rodrigo Vivi
Signed-off-by: José Roberto de Souza
---
v3: rebased
drivers/gpu/drm/i915/i915_reg.h | 3 ++-
drivers/gpu/drm/i915/intel_psr.c | 2 +-
2 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i
IGT tests could be improved with sink status, knowing for sure that
hardware have activate or exit PSR.
Cc: Dhinakaran Pandiyan
Cc: Rodrigo Vivi
Signed-off-by: José Roberto de Souza
---
v3: rebased
drivers/gpu/drm/i915/i915_debugfs.c | 29 +
1 file changed, 29 ins
For Geminilake and Cannonlake+ the Y-coordinate support must be
enabled in PSR2_CTL too.
Spec: 7713 and 7720
Cc: Dhinakaran Pandiyan
Reviewed-by: Rodrigo Vivi
Signed-off-by: José Roberto de Souza
---
v3: rebased
drivers/gpu/drm/i915/i915_reg.h | 3 +++
drivers/gpu/drm/i915/intel_psr.c | 1
This value do not change overtime so better cache it than
fetch it every PSR enable.
Cc: Dhinakaran Pandiyan
Reviewed-by: Rodrigo Vivi
Signed-off-by: José Roberto de Souza
---
v3: rebased
drivers/gpu/drm/i915/i915_drv.h | 1 +
drivers/gpu/drm/i915/intel_psr.c | 28 -
Sink can support our PSR2 requirements but userspace can request
a resolution that PSR2 hardware do not support, in this case it
was overwritten the PSR2 sink support.
Adding another flag here, this way if requested resolution changed
to a value that PSR2 hardware can handle, PSR2 can be enabled.
Although i915 don't implement aux sync frame through tests was
findout that pannels can do selective update when the y-coordinate
is also included in SDP, that is why it is required to run PSR2 in
i915.
So moving to only one place the sink requirements that the actual
driver needs to enable PSR2.
eDP spec states that aux frame is required to do PSR2 selective
update but i915 don't fully implement it. It sends the aux frame
sync messages but the value is always zero as the GTC is not enabled
in driver.
Through tests was findout that pannels can do selective update when
the y-coordinate is a
This is a register to help debug what is in the last SDP VSC
packet revived by sink.
Signed-off-by: José Roberto de Souza
Reviewed-by: Rodrigo Vivi
---
v3: rebased
include/drm/drm_dp_helper.h | 9 +
1 file changed, 9 insertions(+)
diff --git a/include/drm/drm_dp_helper.h b/include/dr
In the 2 eDP1.4a pannels tested set or not set bit have no effect
but is better set it and comply with specification.
Signed-off-by: José Roberto de Souza
Cc: Dhinakaran Pandiyan
Reviewed-by: Rodrigo Vivi
---
v3: rebased
drivers/gpu/drm/i915/intel_psr.c | 11 ++-
1 file changed, 6 in
To comply with eDP1.4a this bit should be set when enabling PSR2.
Signed-off-by: José Roberto de Souza
Reviewed-by: Rodrigo Vivi
---
v3: rebased
include/drm/drm_dp_helper.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/include/drm/drm_dp_helper.h b/include/drm/drm_dp_helper.h
index 6290
https://bugs.freedesktop.org/show_bug.cgi?id=98974
Timothy Arceri changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://bugs.freedesktop.org/show_bug.cgi?id=103234
--- Comment #18 from Marek Olšák ---
Possible fix:
https://patchwork.freedesktop.org/patch/211876/
--
You are receiving this mail because:
You are the assignee for the bug.___
dri-devel mailing list
https://bugs.freedesktop.org/show_bug.cgi?id=105463
--- Comment #9 from erhar...@mailbox.org ---
Created attachment 138402
--> https://bugs.freedesktop.org/attachment.cgi?id=138402&action=edit
html summary from 'pigllit run all' (18.0.0)
--
You are receiving this mail because:
You are the assi
tree: git://people.freedesktop.org/~agd5f/linux.git amd-staging-drm-next
head: 6761ccd944db730dca6f9033781b4645a6eecfd4
commit: 8e62ef009a4bb61d25d391d0384f3314cc6d2059 [1046/1062] drm/amd/pp: Use
atombios api directly in powerplay
config: i386-randconfig-s1-201812 (attached as .config)
compil
https://bugs.freedesktop.org/show_bug.cgi?id=105463
erhar...@mailbox.org changed:
What|Removed |Added
Attachment #138084|0 |1
is obsolete|
https://bugs.freedesktop.org/show_bug.cgi?id=105463
erhar...@mailbox.org changed:
What|Removed |Added
Summary|[r600g] 4210 piglit |[r600g] 3047 piglit
tree: git://people.freedesktop.org/~agd5f/linux.git amd-staging-drm-next
head: 6761ccd944db730dca6f9033781b4645a6eecfd4
commit: 8e62ef009a4bb61d25d391d0384f3314cc6d2059 [1046/1062] drm/amd/pp: Use
atombios api directly in powerplay
config: x86_64-randconfig-s5-03290316 (attached as .config)
co
On Mon, Mar 26, 2018 at 3:22 PM, Samuel Li wrote:
> To reduce some warnings.
>
> Signed-off-by: Samuel Li
Reviewed-by: Alex Deucher
And pushed.
Alex
> ---
> drivers/gpu/drm/drm_prime.c | 13 +
> 1 file changed, 13 insertions(+)
>
> diff --git a/drivers/gpu/drm/drm_prime.c b/driv
Am 28.03.2018 um 20:57 schrieb Logan Gunthorpe:
On 28/03/18 12:28 PM, Christian König wrote:
I'm just using amdgpu as blueprint because I'm the co-maintainer of it
and know it mostly inside out.
Ah, I see.
The resource addresses are translated using dma_map_resource(). As far
as I know that
Hi Dave,
Here's a lonely fix for -next, please pull at your convenience.
drm-misc-next-fixes-2018-03-28:
UABI:
- Mask mode type garbage from userspace (Ville)
Cc: Ville Syrjälä
Cheers, Sean
The following changes since commit b4eec0fa537165efc3265cdbb4bac06e6bdaf596:
Merge tag 'drm-intel-
Now that all of the msm-specific goo is tucked safely away we can switch
over to using the atomic helper commit directly. \o/
Changes in v2:
- None
Cc: Abhinav Kumar
Signed-off-by: Sean Paul
---
drivers/gpu/drm/msm/msm_atomic.c | 120 +--
drivers/gpu/drm/msm/msm_drv
Moving further towards switching fully to the the atomic helpers, this
patch removes the hand-rolled worker nonblock commit code and uses the
atomic helpers commit_work model.
Changes in v2:
- Remove commit_destroy()
- Shuffle order of commit_tail calls to further serialize commits
- Use stall in
Ensure that any queued events are issued when disabling the crtc. This
avoids timeouts when we come back and wait for dependencies (like the
previous frame's flip_done).
Changes in v2:
- None
Signed-off-by: Sean Paul
---
drivers/gpu/drm/msm/disp/mdp5/mdp5_crtc.c | 9 +
1 file changed, 9
Don't leave the event != NULL once it's consumed, this is used a signal
to the atomic helpers that the event will be handled by the driver.
Changes in v2:
- None
Cc: Jeykumar Sankaran
Signed-off-by: Sean Paul
---
drivers/gpu/drm/msm/disp/mdp4/mdp4_crtc.c | 1 +
drivers/gpu/drm/msm/disp/mdp5/md
Factor out the commit_tail() portions of complete_commit() into a
separate function to facilitate moving to the atomic helpers in future
patches.
Changes in v2:
- None
Cc: Jeykumar Sankaran
Signed-off-by: Sean Paul
---
drivers/gpu/drm/msm/msm_atomic.c | 25 -
1 file cha
Now that we have private state handled by the core, we can use those
instead of rolling our own swap_state for private data.
Originally posted here: https://patchwork.freedesktop.org/patch/211361/
Changes in v2:
- Use state->state in disp duplicate_state callback (Jeykumar)
Changes in v3:
- Upd
I noticed a couple of rough edges yesterday as I was running some tests,
so here's v2 which fixes them. Most notably, I was experiencing issues when
the crtc was being disabled with abandoned events and vblank timeouts.
Note that I've bundled the private_obj change into this set since it
shares th
On 28/03/18 13:27, Kieran Bingham wrote:
> Hi Laurent,
>
> Thank you for the patch.
>
> On 26/02/18 21:45, Laurent Pinchart wrote:
>> The entities in the pipeline are all started when the LIF is setup.
>> Remove the outdated comment that state otherwise.
>>
>> Signed-off-by: Laurent Pinchart
>
Hi Laurent,
On 26/02/18 21:45, Laurent Pinchart wrote:
> To implement fully dynamic plane assignment to pipelines, we need to
> reassign the BRU and BRS to the DRM pipelines in the atomic commit
> handler. In preparation for this setup factor out the BRU source pad
> code and call it both at LIF s
From: Colin Ian King
Trivial fix to spelling mistake in warning message text
Signed-off-by: Colin Ian King
---
drivers/gpu/drm/nouveau/nvkm/subdev/bios/init.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/bios/init.c
b/drivers/gpu/drm/
Am 28.03.2018 um 18:25 schrieb Logan Gunthorpe:
On 28/03/18 10:02 AM, Christian König wrote:
Yeah, that looks very similar to what I picked up from the older
patches, going to read up on that after my vacation.
Yeah, I was just reading through your patchset and there are a lot of
similarities.
https://bugs.freedesktop.org/show_bug.cgi?id=101442
Emil Velikov changed:
What|Removed |Added
Resolution|--- |FIXED
Status|NEW
From: Emil Velikov
Currently the error pointer returned by msm_alloc_stolen_fb gets passed
to drm_framebuffer_remove. The latter handles only NULL pointers, thus
a nasty crash will occur.
Drop the unnecessary fail label and the associated checks - both err and
fb will be set at this stage.
Cc:
On Fri, Mar 23, 2018 at 05:28:03PM +0100, Noralf Trønnes wrote:
>
> Den 23.03.2018 16.35, skrev Ville Syrjala:
> > From: Ville Syrjälä
> >
> > mipi_dbi_enable_flush() wants to call the fb->dirty() hook from the
> > bowels of the .atomic_enable() hook. That prevents us from taking the
> > plane mu
We have seen a case of a bad reference count for vblanks with the
Rockchip VOP:
[ cut here ]
WARNING: CPU: 1 PID: 383 at drivers/gpu/drm/drm_irq.c:1198
drm_vblank_put+0x40/0xcc
Modules linked in: brcmfmac brcmutil
CPU: 1 PID: 383 Comm: kwork
On 27 March 2018 at 00:27, John Stultz wrote:
> On Tue, Mar 20, 2018 at 10:36 AM, Emil Velikov
> wrote:
>> From: Tomasz Figa
>>
>> This patch makes the code not rely anymore on libpciaccess when compiled
>> for Android to eliminate ioperm() and iopl() syscalls required by that
>> library. As a
https://bugs.freedesktop.org/show_bug.cgi?id=105788
Daniel Vetter changed:
What|Removed |Added
Assignee|dri-devel@lists.freedesktop |sitewranglers@lists.freedes
https://bugs.freedesktop.org/show_bug.cgi?id=105788
--- Comment #3 from Daniel Vetter ---
Ack. Also acked by Gustavo and Sean on irc.
--
You are receiving this mail because:
You are the assignee for the bug.___
dri-devel mailing list
dri-devel@lists.f
Am 28.03.2018 um 17:53 schrieb Arnd Bergmann:
Building amdkfd without MMU notifiers is broken:
In file included from drivers/gpu/drm/amd/amdkfd/kfd_mqd_manager_vi.c:28:
drivers/gpu/drm/amd/amdkfd/kfd_priv.h:584:22: error: field 'mmu_notifier' has
incomplete type
This adds the missing 'select M
Am 28.03.2018 um 17:47 schrieb Logan Gunthorpe:
On 28/03/18 09:07 AM, Christian König wrote:
Am 28.03.2018 um 14:38 schrieb Christoph Hellwig:
On Sun, Mar 25, 2018 at 12:59:54PM +0200, Christian König wrote:
From: "wda...@nvidia.com"
Add an interface to find the first device which is upstre
From: Emil Velikov
Currently the error pointer returned by msm_alloc_stolen_fb gets passed
to drm_framebuffer_remove. The latter handles only NULL pointers, thus
a nasty crash will occur.
Drop the unnecessary fail label and the associated checks - both err and
fb will be set at this stage.
Cc:
Building amdkfd without MMU notifiers is broken:
In file included from drivers/gpu/drm/amd/amdkfd/kfd_mqd_manager_vi.c:28:
drivers/gpu/drm/amd/amdkfd/kfd_priv.h:584:22: error: field 'mmu_notifier' has
incomplete type
This adds the missing 'select MMU_NOTIFIER' line to make it build
cleanly all t
https://bugs.freedesktop.org/show_bug.cgi?id=105788
--- Comment #2 from Oleksandr Andrushchenko ---
Created attachment 138394
--> https://bugs.freedesktop.org/attachment.cgi?id=138394&action=edit
GPG key
--
You are receiving this mail because:
You are the assignee for the bug.
https://bugs.freedesktop.org/show_bug.cgi?id=105788
--- Comment #1 from Oleksandr Andrushchenko ---
Created attachment 138393
--> https://bugs.freedesktop.org/attachment.cgi?id=138393&action=edit
SSH public key
--
You are receiving this mail because:
You are the assignee for the bug._
https://bugs.freedesktop.org/show_bug.cgi?id=105788
Bug ID: 105788
Summary: Account request for drm-misc
Product: DRI
Version: DRI git
Hardware: Other
OS: All
Status: NEW
Severity: normal
Priorit
On Wed, Mar 28, 2018 at 10:19 AM, Rob Herring wrote:
> On Sat, Mar 17, 2018 at 2:33 PM, Stefan Schake wrote:
>> Hey John,
>>
>> On Fri, Mar 16, 2018 at 10:48 PM, John Stultz wrote:
>>> In trying to integrate the new gralloc_handle.h with the
>>> drm_hwcomposer, I started seeing the following com
Hi Dave,
The following changes since commit 192b4af6cd28cdad9b42fd79c21a90a2aeb0bec7:
drm/tegra: Shutdown on driver unbind (2018-03-19 09:57:32 +0100)
are available in the Git repository at:
git://anongit.freedesktop.org/tegra/linux tags/drm/tegra/for-4.16-fixes
for you to fetch changes up
https://bugs.freedesktop.org/show_bug.cgi?id=105733
--- Comment #5 from Emil Velikov ---
Hi Allan, just sharing some ideas - I'm not working on the AMD drivers
Make sure you're not using libdrm* 2.4.90 - it has some nasty bugs.
Afterwords, try to track down exactly what's causing the problem a
On Sat, Mar 17, 2018 at 2:33 PM, Stefan Schake wrote:
> Hey John,
>
> On Fri, Mar 16, 2018 at 10:48 PM, John Stultz wrote:
>> In trying to integrate the new gralloc_handle.h with the
>> drm_hwcomposer, I started seeing the following compilation
>> errors:
>>
>> In file included from external/drm_
https://bugs.freedesktop.org/show_bug.cgi?id=105786
Bug ID: 105786
Summary: System freeze, amdgpu 4.17-WIP,
4.16.0-rc1-576e538e5fe6
Product: DRI
Version: DRI git
Hardware: Other
OS: All
Status: N
On 28 March 2018 at 15:49, Chris Wilson wrote:
> Quoting Emil Velikov (2018-03-28 02:24:48)
>> From: Deepak Sharma
>>
>> Modify vgem_init to take platform dev as parent in drm_dev_init.
>> This will make drm device available at "/sys/devices/platform/vgem"
>> in x86 chromebook.
>>
>> Cc: Daniel V
On Wed, Mar 28, 2018 at 10:38:35AM +0300, Oleksandr Andrushchenko wrote:
> From: Noralf Trønnes
>
> Use srcu to protect drm_device.unplugged in a race free manner.
> Drivers can use drm_dev_enter()/drm_dev_exit() to protect and mark
> sections preventing access to device resources that are not av
On Tue, Mar 27, 2018 at 05:25:21PM +0200, Greg Kroah-Hartman wrote:
> On Tue, Mar 27, 2018 at 11:18:03AM +0200, Daniel Vetter wrote:
> > On Tue, Mar 27, 2018 at 10:34:07AM +0200, Greg Kroah-Hartman wrote:
> > > On Tue, Mar 27, 2018 at 10:23:52AM +0200, Daniel Vetter wrote:
> > > > vboxvideo doesn't
Am 28.03.2018 um 14:38 schrieb Christoph Hellwig:
On Sun, Mar 25, 2018 at 12:59:54PM +0200, Christian König wrote:
From: "wda...@nvidia.com"
Add an interface to find the first device which is upstream of both
devices.
Please work with Logan and base this on top of the outstanding peer
to peer
Hi Laurent,
On 26/02/18 21:45, Laurent Pinchart wrote:
> The vsp1_du_setup_lif() function setups the DRM pipeline input manually.
s/ setups the / sets up the /
> This duplicates the code from the vsp1_du_pipeline_setup_input()
> function. Replace the manual implementation by a call to the functi
Quoting Emil Velikov (2018-03-28 02:24:48)
> From: Deepak Sharma
>
> Modify vgem_init to take platform dev as parent in drm_dev_init.
> This will make drm device available at "/sys/devices/platform/vgem"
> in x86 chromebook.
>
> Cc: Daniel Vetter
> Signed-off-by: Deepak Sharma
> Reviewed-by: S
On 28 March 2018 at 08:16, Daniel Vetter wrote:
> On Wed, Mar 28, 2018 at 02:24:48AM +0100, Emil Velikov wrote:
>> From: Deepak Sharma
>>
>> Modify vgem_init to take platform dev as parent in drm_dev_init.
>> This will make drm device available at "/sys/devices/platform/vgem"
>> in x86 chromebook
Hi Laurent,
On 26/02/18 21:45, Laurent Pinchart wrote:
> The DRM pipeline setup code used at atomic commit time is similar to the
> setup code used when enabling the pipeline. Move it to a separate
> function in order to share it.
>
> Signed-off-by: Laurent Pinchart
Assuming no hidden secret co
Hi Laszlo,
On 28 March 2018 at 11:27, Laszlo Ersek wrote:
> On 03/28/18 03:24, Emil Velikov wrote:
>> From: Emil Velikov
>>
>> Ealier commit a325725633c26aa66ab940f762a6b0778edf76c0 did not attribute
>> that virtio can be either PCI or a platform device and removed the
>> .set_busid hook. Wherea
On Wednesday, February 14, 2018 04:56:53 PM Jingoo Han wrote:
> On Wednesday, February 14, 2018 7:01 AM wrote:
> >
> > Exynos5, Exynos4 and S5PV210 platforms have been converted to
> > use Device Tree and Exynos DRM driver long time ago. Remove
> > dead platform code for these platforms and updat
On Monday, November 27, 2017 07:01:14 PM SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Mon, 27 Nov 2017 18:40:23 +0100
>
> The local variable "fbdev" will be reassigned by a following statement.
> Thus omit the explicit initialisation at the beginning.
>
> Signed-off-by: Markus Elfring
On Monday, November 27, 2017 06:59:57 PM SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Mon, 27 Nov 2017 18:18:34 +0100
>
> Replace the specification of a data structure by a pointer dereference
> as the parameter for the operator "sizeof" to make the corresponding size
> determination a
On Monday, November 27, 2017 06:58:22 PM SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Mon, 27 Nov 2017 18:14:41 +0100
>
> Omit an extra message for a memory allocation failure in this function.
>
> This issue was detected by using the Coccinelle software.
>
> Signed-off-by: Markus El
On 26/02/18 21:45, Laurent Pinchart wrote:
> Move the duplicated DRM pipeline configuration code to a function and
> call it from vsp1_du_setup_lif() and vsp1_du_atomic_flush().
>
> Signed-off-by: Laurent Pinchart
LGTM.
I thought I had a bit of deja-vu on this patch ... but I can't seem to fi
On Tuesday, December 05, 2017 07:00:22 PM SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Tue, 5 Dec 2017 18:45:42 +0100
>
> Omit an extra message for a memory allocation failure in this function.
>
> This issue was detected by using the Coccinelle software.
>
> Signed-off-by: Markus El
On Tuesday, December 05, 2017 06:13:33 PM SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Tue, 5 Dec 2017 17:53:19 +0100
>
> Replace the specification of a data structure by a pointer dereference
> as the parameter for the operator "sizeof" to make the corresponding size
> determination a
On Tuesday, December 05, 2017 06:12:30 PM SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Tue, 5 Dec 2017 17:50:11 +0100
>
> Omit an extra message for a memory allocation failure in this function.
>
> This issue was detected by using the Coccinelle software.
>
> Signed-off-by: Markus El
https://bugs.freedesktop.org/show_bug.cgi?id=104064
--- Comment #52 from Bjorn ---
(In reply to taijian from comment #51)
>
> OK, I'm not seeing that at all, and I also have a Kaby Lake chip. What
> version of Linux-Firmware are you using?
1.169.3.
I tried going back to an older kernel, and th
Hi Laurent,
On 26/02/18 21:45, Laurent Pinchart wrote:
> The DRM pipeline handling code uses the entity's pipe list head to check
> whether the entity is already included in a pipeline. This method is a
> bit fragile in the sense that it uses list_empty() on a list_head that
> is a list member. Re
On Sunday, November 26, 2017 10:32:06 PM SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Sun, 26 Nov 2017 22:13:55 +0100
>
> Omit an extra message for a memory allocation failure in this function.
>
> This issue was detected by using the Coccinelle software.
>
> Signed-off-by: Markus El
On Monday, November 27, 2017 01:07:15 PM Jingoo Han wrote:
> On Sunday, November 26, 2017 9:18 AM, SF Markus Elfring wrote:
> >
> > From: Markus Elfring
> > Date: Sun, 26 Nov 2017 15:03:03 +0100
> >
> > Replace the specification of a data structure by a pointer dereference
> > as the parameter f
On Monday, November 27, 2017 01:06:30 PM Jingoo Han wrote:
> On Sunday, November 26, 2017 9:17 AM, SF Markus Elfring wrote:
> >
> > From: Markus Elfring
> > Date: Sun, 26 Nov 2017 15:00:16 +0100
> >
> > Omit an extra message for a memory allocation failure in this function.
> >
> > This issue w
On Tue, Mar 27, 2018 at 10:23:53AM +0200, Daniel Vetter wrote:
> Just want to clean out all grep hits. gem_free_object is deprecated.
>
> Signed-off-by: Daniel Vetter
Reviewed-by: Sean Paul
> Cc: Sandy Huang
> Cc: "Heiko Stübner"
> Cc: linux-arm-ker...@lists.infradead.org
> Cc: linux-rockc..
On Monday, November 27, 2017 11:31:10 AM Timur Tabi wrote:
> On 11/27/17 3:00 AM, SF Markus Elfring wrote:
> > From: Markus Elfring
> > Date: Mon, 27 Nov 2017 09:56:09 +0100
> >
> > Omit an extra message for a memory allocation failure in this function.
> >
> > This issue was detected by using th
On Sunday, January 07, 2018 07:34:44 PM SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Sun, 7 Jan 2018 19:27:28 +0100
>
> Replace the specification of a data structure by a pointer dereference
> as the parameter for the operator "sizeof" to make the corresponding size
> determination a b
On Sunday, January 07, 2018 05:06:13 PM SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Sun, 7 Jan 2018 16:12:40 +0100
>
> Omit an extra message for a memory allocation failure in this function.
>
> This issue was detected by using the Coccinelle software.
>
> Signed-off-by: Markus Elfr
On Sunday, January 07, 2018 05:04:04 PM SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Sun, 7 Jan 2018 16:07:11 +0100
>
> * Return an error code without storing it in an intermediate variable.
>
> * Delete the label "error" and local variable "retval"
> which became unnecessary with t
On Sunday, January 07, 2018 05:02:21 PM SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Sun, 7 Jan 2018 15:56:11 +0100
>
> Up to four checks could be repeated by the ufx_usb_probe() function
> during error handling even if the relevant properties can be determined
> for the involved varia
Hi Laurent,
Thank you for the patch.
On 26/02/18 21:45, Laurent Pinchart wrote:
> Various types of objects subclassing vsp1_entity currently store a
> pointer to the pipeline. Move the pointer to vsp1_entity to simplify the
> code and avoid storing the pipeline in more entity subclasses later.
T
On Sunday, January 07, 2018 02:34:51 PM SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Sun, 7 Jan 2018 14:02:36 +0100
>
> * Return an error code without storing it in an intermediate variable.
>
> * Delete the label "error" and local variable "retval"
> which became unnecessary with t
On Sunday, January 07, 2018 12:58:47 PM SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Sun, 7 Jan 2018 12:34:22 +0100
>
> Omit an extra message for a memory allocation failure in this function.
>
> This issue was detected by using the Coccinelle software.
>
> Signed-off-by: Markus Elfr
On Sunday, January 07, 2018 12:56:45 PM SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Sun, 7 Jan 2018 11:33:59 +0100
>
> Replace an error code for the indication of a memory allocation failure
> in this function.
>
> Fixes: 1da177e4c3f41524e886b7f1b8a0c1fc7321cac ("Linux-2.6.12-rc2: In
On Thursday, March 22, 2018 03:13:32 PM Gustavo A. R. Silva wrote:
> Assign true or false to boolean variables instead of an integer value.
>
> This issue was detected with the help of Coccinelle.
>
> Signed-off-by: Gustavo A. R. Silva
Patch queued for 4.17, thanks.
Best regards,
--
Bartlomiej
On Thursday, March 15, 2018 06:16:24 PM Colin King wrote:
> From: Colin Ian King
>
> There is a missing indentation following an if statement, fix this.
>
> Detected by Coccinelle:
> drivers/video/fbdev/aty/mach64_ct.c:183:2-15: code aligned with
> following code on line 184
>
> Signed-off-by:
On Sun, Mar 25, 2018 at 12:59:54PM +0200, Christian König wrote:
> From: "wda...@nvidia.com"
>
> Add an interface to find the first device which is upstream of both
> devices.
Please work with Logan and base this on top of the outstanding peer
to peer patchset.
__
On Sun, Mar 25, 2018 at 12:59:53PM +0200, Christian König wrote:
> Use this function to set an sg entry to point to device resources mapped
> using dma_map_resource(). The page pointer is set to NULL and only the DMA
> address, length and offset values are valid.
NAK. Please provide a higher leve
Hi Laurent,
Thanks for the patch,
On 26/02/18 21:45, Laurent Pinchart wrote:
> The vsp1_drm_pipeline enabled field is set but never used. Remove it.
>
Indeed - not used.
> Signed-off-by: Laurent Pinchart
Reviewed-by: Kieran Bingham
> ---
> drivers/media/platform/vsp1/vsp1_drm.c | 4
>
1 - 100 of 128 matches
Mail list logo
