From: Chris Wilson
Proxy fences allow you to create a place-holder fence which you can
later assign to the "real" fence at which point the two look
indistinguishable to anyone other than the driver which created the
real fence. These should be used with care as it is the responsibility
of proxy
From: Chris Wilson
This is an illegal scenario, to free the fence whilst there are pending
callbacks. Currently, we emit a WARN and then cast aside the callbacks
leaving them dangling. Alternatively, we could set an error on the fence
and then signal fence so that any dependency chains from the f
This just resets the dma_fence to NULL so it looks like it's never been
signaled. This will be useful once we add the new wait API for allowing
wait on "submit and signal" behavior.
Signed-off-by: Jason Ekstrand
---
drivers/gpu/drm/drm_internal.h | 2 ++
drivers/gpu/drm/drm_ioctl.c| 2 ++
Vulkan VkFence semantics require that the application be able to perform
a CPU wait on work which may not yet have been submitted. This is
perfectly safe because the CPU wait has a timeout which will get
triggered eventually if no work is ever submitted. This behavior is
advantageous for multi-th
This series does the same thing as my earlier series in that it adds a sync
object wait interface complete with WAIT_FOR_SUBMIT flag. While the uapi
remains unchanged, the guts look a bit different. Instead of adding a
callback mechanism to drm_syncobj that fired whenever replace_fence was
called
The atomic exchange operation in drm_syncobj_replace_fence is sufficient
for the case where it races with itself. However, if you have a race
between a replace_fence and dma_fence_get(syncobj->fence), you may end
up with the entire replace_fence happening between the point in time
where the one th
The function has far more in common with drm_syncobj_find than with
any in the get/put functions.
Signed-off-by: Jason Ekstrand
---
drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 2 +-
drivers/gpu/drm/drm_syncobj.c | 10 +-
include/drm/drm_syncobj.h | 6 +++---
3 files
Hi Daniel,
On Tuesday 25 Jul 2017 10:01:16 Daniel Vetter wrote:
> Atomic drivers only use the property value store for immutable (i.e.
> can't be set by userspace, but the kernel can still adjust it)
> properties. The only tricky part is the removal of the update in
> drm_atomic_helper_update_lega
If the kmalloc() fails then we dereference "state" when we set
"state->zpos".
Fixes: 3dfeb631a15d ("drm/omap: Rework the rotation-on-crtc hack")
Signed-off-by: Dan Carpenter
diff --git a/drivers/gpu/drm/omapdrm/omap_crtc.c
b/drivers/gpu/drm/omapdrm/omap_crtc.c
index 400d0d2f6790..cc85c16cbc2a 1
"gpu->aspace" can be either an error pointer or NULL so we need to check
for both.
Fixes: 0dbd7327f4d5 ("drm/msm: Move memptrs to msm_gpu")
Signed-off-by: Dan Carpenter
diff --git a/drivers/gpu/drm/msm/msm_gpu.c b/drivers/gpu/drm/msm/msm_gpu.c
index 66af5ac334ba..156ad7d5c68a 100644
--- a/driver
On Fri, Aug 11, 2017 at 11:32 AM, Matthias Kaehlcke wrote:
> El Mon, Jul 17, 2017 at 12:58:54PM -0700 Matthias Kaehlcke ha dit:
>
>> For 0.85V cnl_get_buf_trans_edp() returns the DP table, instead of EDP.
>> Use the correct table.
>>
>> The error was pointed out by this clang warning:
>>
>> driver
El Mon, Jul 17, 2017 at 12:58:54PM -0700 Matthias Kaehlcke ha dit:
> For 0.85V cnl_get_buf_trans_edp() returns the DP table, instead of EDP.
> Use the correct table.
>
> The error was pointed out by this clang warning:
>
> drivers/gpu/drm/i915/intel_ddi.c:392:39: warning: variable
> 'cnl_ddi_t
From: Dmitry Osipenko
Since DRM IOCTL's are lockless, there is a chance that BOs could be
released while a job submission is in progress. To avoid that, keep the
GEM reference until the job has been pinned, part of which will be to
take another reference.
Signed-off-by: Dmitry Osipenko
Signed-o
On Fri, Aug 11, 2017 at 12:11 PM, Hans de Goede wrote:
> Hi,
>
> On 11-08-17 18:04, Sean Paul wrote:
>>
>> On Fri, Aug 11, 2017 at 03:26:45PM +0200, Hans de Goede wrote:
>>>
>>> Hi,
>>>
>>> On 11-08-17 14:33, Cihangir Akturk wrote:
Use drm_*_get() and drm_*_put() helpers instead of drm_*
Den 09.08.2017 19.59, skrev Noralf Trønnes:
Den 07.08.2017 19.39, skrev David Lechner:
LEGO MINDSTORMS EV3 has an LCD with a ST7586 controller. This adds a new
module for the ST7586 controller with parameters for the LEGO MINDSTORMS
EV3 LCD display.
Signed-off-by: David Lechner
---
This lo
> -Original Message-
> From: Sumit Semwal [mailto:sumit.sem...@linaro.org]
> Sent: Friday, August 11, 2017 11:52 AM
> To: Alex Deucher
> Cc: amd-gfx list; DRI mailing list; Koenig, Christian; Deucher, Alexander
> Subject: Re: [PATCH 0/3] dma-buf changes for ttm and amdgpu
>
> Hi Alex,
>
>
On Fri, Aug 11, 2017 at 03:32:57PM +0300, Cihangir Akturk wrote:
> Use drm_*_get() and drm_*_put() helpers instead of drm_*_reference()
> and drm_*_unreference() helpers.
>
> drm_*_reference() and drm_*_unreference() functions are just
> compatibility alias for drm_*_get() and drm_*_put() and shou
On Fri, Aug 11, 2017 at 03:33:06PM +0300, Cihangir Akturk wrote:
> Use drm_*_get() and drm_*_put() helpers instead of drm_*_reference()
> and drm_*_unreference() helpers.
>
> drm_*_reference() and drm_*_unreference() functions are just
> compatibility alias for drm_*_get() and drm_*_put() and shou
On Fri, Aug 11, 2017 at 03:33:00PM +0300, Cihangir Akturk wrote:
> Use drm_*_get() and drm_*_put() helpers instead of drm_*_reference()
> and drm_*_unreference() helpers.
>
> drm_*_reference() and drm_*_unreference() functions are just
> compatibility alias for drm_*_get() and drm_*_put() and shou
On Fri, Aug 11, 2017 at 03:32:50PM +0300, Cihangir Akturk wrote:
> Use drm_*_get() and drm_*_put() helpers instead of drm_*_reference()
> and drm_*_unreference() helpers.
>
> drm_*_reference() and drm_*_unreference() functions are just
> compatibility alias for drm_*_get() and drm_*_put() and shou
https://bugzilla.kernel.org/show_bug.cgi?id=196379
Jani Nikula (jani.nik...@intel.com) changed:
What|Removed |Added
Component|Video(DRI - non Intel) |Other
Ass
https://bugs.freedesktop.org/show_bug.cgi?id=100443
--- Comment #30 from Andy Furniss ---
(In reply to Andy Furniss from comment #22)
> I've always seen
>
> [powerplay] Can't find requested voltage id in vdd_dep_on_sclk table!
>
> on R9 285 Tonga, but powerplay works OK and there are no other
On Fri, Aug 11, 2017 at 10:15:16AM +0800, Sandy Huang wrote:
>
>
> 在 2017/8/11 2:05, Sean Paul 写道:
> > On Thu, Aug 10, 2017 at 05:35:52PM +0800, Sandy Huang wrote:
> > > Hi Sean Paul,
> > > Thanks for your review.
> > >
> > > 在 2017/8/10 3:58, Sean Paul 写道:
> > > > On Wed, Aug 09, 2017 at 0
> -Original Message-
> From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf
> Of Cihangir Akturk
> Sent: Friday, August 11, 2017 8:33 AM
> Cc: de...@driverdev.osuosl.org; linux-arm-...@vger.kernel.org; intel-
> g...@lists.freedesktop.org; linux-ker...@vger.kernel.org; dri-
On Friday, August 11, 2017 04:49:04 PM Laurent Pinchart wrote:
> The HDMI audio driver only needs to know which generation of HDMI
> transmitter it deals with, not the detailed SoC model. Pass the version
> number as an integer to prepare for removal of the OMAP SoC version from
> the omapdrm drive
Hi Tomi,
Thank you for the patch.
On Monday 08 May 2017 11:51:22 Tomi Valkeinen wrote:
> On SoCs with TILER, we have to ways to allocate buffers: normal
s/to ways/two ways/
> dma_alloc or via TILER (which basically functions as an IOMMU). TILER
> can map 128MB at a time, and we only map the TIL
https://bugzilla.kernel.org/show_bug.cgi?id=196379
--- Comment #3 from leviathansol...@gmail.com (leviathansol...@gmail.com) ---
This is the reddit link:
https://www.reddit.com/r/linuxquestions/comments/6ex4e0/cpu_doesnt_speed_up_when_gaming/
Coolsense is a HP driver that uses the accelerometer t
Hi Tomi,
Thank you for the patch.
On Monday 08 May 2017 11:51:21 Tomi Valkeinen wrote:
> Add omap_gem_put_paddr_locked() which is a version of
> omap_gem_put_paddr() that expects the caller to hold the struct_mutex.
>
> Signed-off-by: Tomi Valkeinen
> ---
> drivers/gpu/drm/omapdrm/omap_gem.c |
The OMAP4 and OMAP5 HDMI PHYs have different properties that require
specific handling in the HDMI PHY driver. This needs knowledge of the
PHY version, which is currently inferred from the DSS version. As part
of the effort to remove usage of the DSS version, use the HDMI
controller version instead
From: Colin Ian King
The structure intel_sprite_plane_funcs is local to the source
and does not need to be in global scope, so make it static.
Cleans up sparse warning:
symbol 'intel_sprite_plane_funcs' was not declared. Should it be static?
Signed-off-by: Colin Ian King
---
drivers/gpu/drm/i
The omapdrm driver doesn't need the omapdss device anymore. Although it
can't be removed completely as the fbdev driver still requires it, we
can condition its registration to the usage of the omapfb driver.
Signed-off-by: Laurent Pinchart
Acked-by: Tony Lindgren
Reviewed-by: Tomi Valkeinen
---
The omapdrm platform data are not used anymore, remove them.
Signed-off-by: Laurent Pinchart
Reviewed-by: Tomi Valkeinen
---
include/linux/platform_data/omap_drm.h | 53 --
1 file changed, 53 deletions(-)
delete mode 100644 include/linux/platform_data/omap_drm.h
The omapdrm platform device is unused, as a replacement is now
registered in the omapdss driver. Remove it.
Signed-off-by: Laurent Pinchart
Acked-by: Tony Lindgren
Reviewed-by: Tomi Valkeinen
---
arch/arm/mach-omap2/Makefile | 2 +-
arch/arm/mach-omap2/display.c | 7 --
arch/arm/mach-om
The omapdrm platform device is a virtual device created for the sole
purpose of handling the omapdss/omapdrm driver split. It should
eventually be removed. As a first step to ease refactoring move its
registration from platform code to driver code.
The omapdrm driver name must be changed internall
The dsi_pll_ops structure and dsi_init_pll_data() function incorrectly
use a dsi_ prefix, likely due to copy & paste. Fix it by using the
correct hdmi_ prefix.
Signed-off-by: Laurent Pinchart
---
drivers/gpu/drm/omapdrm/dss/hdmi_pll.c | 9 +
1 file changed, 5 insertions(+), 4 deletions(-
There's no need to allocate memory dynamically to duplicate the contents
of a const structure, only to store the memory pointer in a const
pointer field. Just use the original structures directly.
Signed-off-by: Laurent Pinchart
---
drivers/gpu/drm/omapdrm/dss/hdmi_phy.c | 32 ---
The omapdss driver (not to be confused with the omapdss_dss driver) is
now a dummy driver with empty probe and remove functions. Remove it.
Signed-off-by: Laurent Pinchart
Reviewed-by: Tomi Valkeinen
---
drivers/gpu/drm/omapdrm/dss/core.c| 47 ---
drivers/gpu
The OMAP4 and OMAP5 PLLs have different properties that require specific
handling in the HDMI PLL driver. This needs knowledge of the PLL
version, which is currently inferred from the DSS version. AS part of
the effort to remove usage of the DSS version, use the HDMI controller
version instead.
Si
The HDMI audio driver only needs to know which generation of HDMI
transmitter it deals with, not the detailed SoC model. Pass the version
number as an integer to prepare for removal of the OMAP SoC version from
the omapdrm driver.
Signed-off-by: Laurent Pinchart
Acked-by: Mark Brown
---
Changes
Hello,
This patch series is a third attempt at removing the omapdrm and omapdss
devices from platform code. Most of the patches from the previous version have
been picked up already, this version contains all the leftovers.
The omapdss/omapdrm initialization code is quite a mess. The physical dev
The HDMI wrapper code only needs to differentiate between major OMAP
revisions, which can be obtained from the HDMI transmitter compatible
string. Replace the OMAP SoC model checks to prepare for removal of the
OMAP SoC version platform data.
Signed-off-by: Laurent Pinchart
---
drivers/gpu/drm/o
Hi,
On 11-08-17 14:33, Cihangir Akturk wrote:
Use drm_*_get() and drm_*_put() helpers instead of drm_*_reference()
and drm_*_unreference() helpers.
drm_*_reference() and drm_*_unreference() functions are just
compatibility alias for drm_*_get() and drm_*_put() and should not be
used by new code
On Fri, 2017-08-11 at 15:32 +0300, Cihangir Akturk wrote:
> Use drm_*_get() and drm_*_put() helpers instead of drm_*_reference()
> and drm_*_unreference() helpers.
>
> drm_*_reference() and drm_*_unreference() functions are just
> compatibility alias for drm_*_get() and drm_*_put() and should not
On Fri, 2017-08-11 at 15:32 +0300, Cihangir Akturk wrote:
> Use drm_*_get() and drm_*_put() helpers instead of drm_*_reference()
> and drm_*_unreference() helpers.
>
> drm_*_reference() and drm_*_unreference() functions are just
> compatibility alias for drm_*_get() and drm_*_put() and should not
Hi Laurent,
On 2017-08-11 11:40, Laurent Pinchart wrote:
Hi Marek,
On Friday 11 Aug 2017 08:39:31 Marek Szyprowski wrote:
On 2017-08-10 15:39, Laurent Pinchart wrote:
On Thursday 10 Aug 2017 15:25:29 Maciej Purski wrote:
The driver should be switched on if an external connector is plugged an
Inki Dae wrote:
>
>
> 2017년 08월 09일 20:48에 Tobias Jakobi 이(가) 쓴 글:
>> We always translate the dma address such that the offsets of
>> the source image are zero. Hence we can remove manipulation of
>> the MXR_GRAPHIC_SXY(win) register.
>>
>> We leave the register defines (in regs_mixer.h) in place
Inki Dae wrote:
>
>
> 2017년 08월 09일 20:48에 Tobias Jakobi 이(가) 쓴 글:
>> In some of drivers we compute something like 'pitch / cpp' at some
>> point, silently assuming that the pitch (which is in bytes) is
>> divisible by the buffer's cpp. This is not always true, in particular
>> DRM core does not
Inki Dae wrote:
>
>
> 2017년 08월 09일 20:48에 Tobias Jakobi 이(가) 쓴 글:
>> DRM core already checks the validity of the pixelformats, so we
>> can simplify the checks here. The same applies to the FB modifier,
>> which is now checked in common Exynos plane code.
>>
>> Also rename the booleans to reflec
Hi Hans,
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
On 02/08/17 11:53, Hans Verkuil wrote:
> From: Hans Verkuil
>
> This patch series adds CEC support for the omap4. It is based on
> the 4.13-rc2 kernel with
Hi Marek,
On Friday 11 Aug 2017 08:39:31 Marek Szyprowski wrote:
> On 2017-08-10 15:39, Laurent Pinchart wrote:
> > On Thursday 10 Aug 2017 15:25:29 Maciej Purski wrote:
> >> The driver should be switched on if an external connector is plugged and
> >> switched off if it is unplugged. Extcon is op
Hi Marek,
(now CC'ing Mark Brown with his correct e-mail address)
On Friday 11 Aug 2017 09:00:00 Marek Szyprowski wrote:
> On 2017-08-10 16:51, Laurent Pinchart wrote:
> > On Friday 04 Aug 2017 08:55:55 Marek Szyprowski wrote:
> >> Hi Laurent,
> >>
> >> Thanks for your detailed comments. Maciej
Hi Marek,
(CC'ing Mark Brown)
On Friday 11 Aug 2017 09:00:00 Marek Szyprowski wrote:
> On 2017-08-10 16:51, Laurent Pinchart wrote:
> > On Friday 04 Aug 2017 08:55:55 Marek Szyprowski wrote:
> >> Hi Laurent,
> >>
> >> Thanks for your detailed comments. Maciej resurrected some orphaned code,
> >>
On Thu, Aug 03, 2017 at 01:52:55PM -0400, Alex Deucher wrote:
> On Thu, Aug 3, 2017 at 9:54 AM, Daniel Vetter wrote:
> > On Thu, Aug 3, 2017 at 1:17 AM, Daniel Vetter
> > wrote:
> >> On Wed, Aug 2, 2017 at 10:50 PM, Alex Deucher
> >> wrote:
> >>> On Wed, Aug 2, 2017 at 7:56 AM, Daniel Vetter
2017년 08월 09일 20:48에 Tobias Jakobi 이(가) 쓴 글:
> In some of drivers we compute something like 'pitch / cpp' at some
> point, silently assuming that the pitch (which is in bytes) is
> divisible by the buffer's cpp. This is not always true, in particular
> DRM core does not check for pitch alignment
2017년 08월 09일 20:48에 Tobias Jakobi 이(가) 쓴 글:
> We always translate the dma address such that the offsets of
> the source image are zero. Hence we can remove manipulation of
> the MXR_GRAPHIC_SXY(win) register.
>
> We leave the register defines (in regs_mixer.h) in place, since
> they document th
2017년 08월 09일 20:48에 Tobias Jakobi 이(가) 쓴 글:
> DRM core already checks the validity of the pixelformats, so we
> can simplify the checks here. The same applies to the FB modifier,
> which is now checked in common Exynos plane code.
>
> Also rename the booleans to reflect what true/false actually
On Thu, Aug 10, 2017 at 09:04:34PM +0200, Noralf Trønnes wrote:
>
> Den 09.08.2017 22.07, skrev Daniel Vetter:
> > On Wed, Aug 09, 2017 at 12:11:05PM +0200, Noralf Trønnes wrote:
> > > Use the new drm_gem_framebuffer_helper who's code was copied
> > > from this helper.
> > >
> > > Signed-off-by:
On Thu, Aug 10, 2017 at 08:55:59PM +0200, Noralf Trønnes wrote:
>
> Den 09.08.2017 22.04, skrev Daniel Vetter:
> > On Wed, Aug 09, 2017 at 12:11:04PM +0200, Noralf Trønnes wrote:
> > > This library provides helpers for drivers that don't subclass
> > > drm_framebuffer and are backed by drm_gem_obj
Am 10.08.2017 um 21:43 schrieb Dave Airlie:
On 10 August 2017 at 23:56, Emil Velikov wrote:
Hi Dave,
On 18 July 2017 at 04:52, Dave Airlie wrote:
+int amdgpu_cs_submit_raw(amdgpu_device_handle dev,
+amdgpu_context_handle context,
+amdgpu_bo_li
Hi Laurent,
On 2017-08-10 16:51, Laurent Pinchart wrote:
Hi Marek,
On Friday 04 Aug 2017 08:55:55 Marek Szyprowski wrote:
Hi Laurent,
Thanks for your detailed comments. Maciej resurrected some orphaned code,
which is still useful today (Tomasz has left Samsung a few years ago).
I'm not sure w
60 matches
Mail list logo
