Re: [PATCH 0/6] drm: tackle byteorder issues, take two

2017-04-24 Thread Michel Dänzer
On 24/04/17 03:25 PM, Gerd Hoffmann wrote: > Hi, > > Ok, different approach up for discussion. Given that userspace didn't > made the transition from ADDFB to ADDFB2 yet it seems we still can muck > with the fourcc codes without breaking everything, as long as we > maintain ADDFB and fbdev beha

Re: [PATCH 1/2] drm/amd/display: Fix race between vblank irq and pageflip irq.

2017-04-24 Thread Michel Dänzer
On 22/04/17 01:23 AM, Mario Kleiner wrote: > Since DC now uses CRTC_VERTICAL_INTERRUPT0 as VBLANK irq trigger > and vblank interrupts actually happen earliest at start of vblank, > instead of a bit before vblank, we no longer need some of the > fudging logic to deal with too early vblank irq handli

Re: [PATCH 2/2] drm/amd/display: Prevent premature pageflip when comitting in vblank.

2017-04-24 Thread Michel Dänzer
On 22/04/17 01:23 AM, Mario Kleiner wrote: > Make sure we do not program a hw pageflip inside vblank 'n' iff the > atomic flip is comitted while inside the same vblank 'n'. We must > defer such a flip by one refresh cycle to vblank 'n+1'. > > Without this, pageflips programmed via X11 GLX_OML_sync

Re: [PATCH 0/6] drm: tackle byteorder issues, take two

2017-04-24 Thread Gerd Hoffmann
Hi, > > Gerd Hoffmann (6): > > drm: fourcc byteorder: drop DRM_FORMAT_BIG_ENDIAN > > I don't see how it can be dropped. It's only optional for formats where > all components have 8 bits. Well, we can, because it is simply not used anywhere ... We indeed can't specify RGB565 or XRGB2101010 i

[Bug 100761] `radeon_ib_ring_tests` takes 0.5 s on resume

2017-04-24 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=100761 Christian König changed: What|Removed |Added Resolution|--- |NOTABUG Status|NEW

Re: [PATCH 1/1] drm/radeon: check return value of radeon_ring_lock

2017-04-24 Thread Christian König
Am 23.04.2017 um 15:36 schrieb Pan Bian: From: Pan Bian Function radeon_ring_lock() returns an errno on failure, and its return value should be validated. However, in functions r420_cp_errata_init() and r420_cp_errata_fini(), its return value is not checked. This patch adds the checks. Signed-

Re: [PATCH 1/1] drm/radeon: check return vlaue of radeon_fence_emit

2017-04-24 Thread Christian König
Am 23.04.2017 um 15:50 schrieb Pan Bian: From: Pan Bian Function radeon_fence_emit() returns -ENOMEM if there is no enough memory. And in this case, function radeon_ring_unlock_undo() rather than function radeon_ring_unlock_commit() should be called. However, in function radeon_test_create_and_

Re: [PATCH 0/6] drm: tackle byteorder issues, take two

2017-04-24 Thread Michel Dänzer
On 24/04/17 04:36 PM, Gerd Hoffmann wrote: > >>> drm: fourcc byteorder: add DRM_FORMAT_CPU_* >>> drm: fourcc byteorder: add bigendian support to >>> drm_mode_legacy_fb_format >> >> As I explained in my last followup in the "[PATCH] drm: fourcc >> byteorder: brings header file comments in l

Re: [PATCH v5 07/10] drm: bridge: dw-hdmi: Add support for custom PHY configuration

2017-04-24 Thread 郑阳
Hi, Laurent Pinchart: 在 2017年03月04日 01:20, Laurent Pinchart 写道: From: Kieran Bingham The DWC HDMI TX controller interfaces with a companion PHY. While Synopsys provides multiple standard PHYs, SoC vendors can also integrate a custom PHY. Modularize PHY configuration to support vendor PHYs thr

Re: [PATCH v5 02/11] clk: sunxi-ng: add support for DE2 CCU

2017-04-24 Thread Maxime Ripard
Hi, On Sun, Apr 23, 2017 at 06:37:45PM +0800, Icenowy Zheng wrote: > +static const struct of_device_id sunxi_de2_clk_ids[] = { > + { > + .compatible = "allwinner,sun8i-a83t-de2-clk", > + .data = &sun8i_a83t_de2_clk_desc, > + }, > + { > + .compatible

[PATCH] RFC: Design: DRM: Blending pipeline using DRM plane properties

2017-04-24 Thread Shashank Sharma
This patch proposes a RFC design to handle blending of various framebuffers with different color spaces, using the DRM color properties. Signed-off-by: Shashank Sharma --- drivers/gpu/drm/rfc-design-blending.txt | 52 + 1 file changed, 52 insertions(+) create mod

Re: [PATCH v5 05/11] drm/sun4i: abstract a engine type

2017-04-24 Thread Maxime Ripard
Hi, Thanks a lot for that work. On Sun, Apr 23, 2017 at 06:37:48PM +0800, Icenowy Zheng wrote: > As we are going to add support for the Allwinner DE2 engine in sun4i-drm > driver, we will finally have two types of display engines -- the DE1 > backend and the DE2 mixer. They both do some display b

Re: [PATCH v5 09/11] ARM: dts: sun8i: add DE2 nodes for V3s SoC

2017-04-24 Thread Maxime Ripard
On Sun, Apr 23, 2017 at 06:37:52PM +0800, Icenowy Zheng wrote: > + tcon0_out: port@1 { > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <1>; > +

Re: [PATCH 1/1] drm/radeon: check return value of radeon_fence_emit

2017-04-24 Thread Christian König
Am 24.04.2017 um 10:45 schrieb Pan Bian: From: Pan Bian Function radeon_fence_emit() returns -ENOMEM if there is no enough memory. And in this case, function radeon_ring_unlock_undo() rather than function radeon_ring_unlock_commit() should be called. However, in function radeon_test_create_and_

Re: [PATCH v2 5/5] drm: omapdrm: Add zpos property

2017-04-24 Thread Tomi Valkeinen
On 15/04/17 12:16, Laurent Pinchart wrote: > Create a standard zpos property for every plane as an alias to the > omapdrm-specific zorder property. Unlike the zorder property that has to > be instantiated for both planes and CRTCs due to backward compatibility, > the zpos property is only instant

Re: [PATCH v2 4/5] drm: omapdrm: Store the Z order in the plane state zpos field

2017-04-24 Thread Tomi Valkeinen
On 15/04/17 12:16, Laurent Pinchart wrote: > The DRM core implements a standard "zpos" property to control planes > ordering. The omapdrm driver implements a similar property named > "zorder". Although we can't switch to DRM core handling of the "zpos" > property for backward compatibility reasons,

Re: [PATCH 1/1] drm/radeon: check return value of radeon_ring_lock

2017-04-24 Thread Christian König
Am 24.04.2017 um 10:38 schrieb Pan Bian: From: Pan Bian Function radeon_ring_lock() returns an errno on failure, and its return value should be validated. However, in functions r420_cp_errata_init() and r420_cp_errata_fini(), its return value is not checked. This patch adds the checks. Signed-

[PATCH] [RFC] drm/radeon: clear WC flag when moving bo from vram to gtt

2017-04-24 Thread Julien Isorce
But re-add the flag is the bo is moved back to vram. This fixes "ring 0/3 stalled" issue which happens when the driver evicts bo from vram to gtt, at least on TAHITI and CAPVERDE. I do not know the exact reason among the following: - si_copy_dma from vram to gtt is slow if WC (only for

[PATCH 2/2] drm/amd/display: Prevent premature pageflip when comitting in vblank. (v2)

2017-04-24 Thread Mario Kleiner
Make sure we do not program a hw pageflip inside vblank 'n' iff the atomic flip is comitted while inside the same vblank 'n'. We must defer such a flip by one refresh cycle to vblank 'n+1'. Without this, pageflips programmed via X11 GLX_OML_sync_control extensions glXSwapBuffersMscOML(..., target_

[PATCH 1/2] drm/amd/display: Fix race between vblank irq and pageflip irq. (v2)

2017-04-24 Thread Mario Kleiner
Since DC now uses CRTC_VERTICAL_INTERRUPT0 as VBLANK irq trigger and vblank interrupts actually happen earliest at start of vblank, instead of a bit before vblank, we no longer need some of the fudging logic to deal with too early vblank irq handling (grep for lb_vblank_lead_lines). This itself fix

Re: [PATCH] [RFC] drm/radeon: clear WC flag when moving bo from vram to gtt

2017-04-24 Thread Christian König
Am 24.04.2017 um 11:42 schrieb Julien Isorce: But re-add the flag is the bo is moved back to vram. This fixes "ring 0/3 stalled" issue which happens when the driver evicts bo from vram to gtt, at least on TAHITI and CAPVERDE. Interesting find, but NAK on the approach for fixing it. If WC mapp

[Bug 78562] [gma500] wrong pixel clock on LVDS (half the correct frequency)

2017-04-24 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=78562 --- Comment #17 from Patrik Jakobsson --- I've posted a patch that is reported to fix this issue. If anyone still have the hardware available please give it a try: https://patchwork.freedesktop.org/patch/150985/ -- You are receiving this mail

Re: Applied "ASoC: Add Odroid sound DT bindings documentation" to the asoc tree

2017-04-24 Thread Mark Brown
On Fri, Apr 21, 2017 at 08:07:10PM +0200, Krzysztof Kozlowski wrote: > Mutt is complaining that your key used to sign the mails has expired: > gpg: Note: This key has expired! > Primary key fingerprint: 3F25 68AA C269 98F9 E813 A1C5 C3F4 36CA 30F5 > D8EB > Subkey fingerprint: A

Re: [PATCH v2 1/5] drm: omapdrm: Handle events when enabling/disabling CRTCs

2017-04-24 Thread Tomi Valkeinen
On 15/04/17 12:16, Laurent Pinchart wrote: > The driver currently handles vblank events only when updating planes on > an already enabled CRTC. The atomic update API however allows requesting > an event when enabling or disabling a CRTC. This currently leads to > event objects being leaked in the k

Re: [PATCH] [RFC] drm/radeon: clear WC flag when moving bo from vram to gtt

2017-04-24 Thread Julien Isorce
On 24 April 2017 at 10:51, Christian König wrote: > Am 24.04.2017 um 11:42 schrieb Julien Isorce: > >> But re-add the flag is the bo is moved back to vram. >> >> This fixes "ring 0/3 stalled" issue which happens when the driver >> evicts bo from vram to gtt, at least on TAHITI and CAPVERDE. >> >

Re: [PATCH v2.1 3/5] drm: omapdrm: Remove legacy buffer synchronization support

2017-04-24 Thread Tomi Valkeinen
On 20/04/17 13:20, Laurent Pinchart wrote: > The omapdrm driver uses a custom API to synchronize with the SGX GPU. > This is unusable as such in the mainline kernel as the API is only > partially implemented and requires additional out-of-tree patches. > Furthermore, as no SGX driver is available i

Re: [PATCH] [RFC] drm/radeon: clear WC flag when moving bo from vram to gtt

2017-04-24 Thread Michel Dänzer
On 24/04/17 06:51 PM, Christian König wrote: > Am 24.04.2017 um 11:42 schrieb Julien Isorce: >> But re-add the flag is the bo is moved back to vram. >> >> This fixes "ring 0/3 stalled" issue which happens when the driver >> evicts bo from vram to gtt, at least on TAHITI and CAPVERDE. > > Interesti

Re: [PATCH] [RFC] drm/radeon: clear WC flag when moving bo from vram to gtt

2017-04-24 Thread Michel Dänzer
On 24/04/17 07:01 PM, Julien Isorce wrote: > > On 24 April 2017 at 10:51, Christian König > wrote: > > Am 24.04.2017 um 11:42 schrieb Julien Isorce: > > But re-add the flag is the bo is moved back to vram. > > This fixes "ring 0/3 stalled" is

Re: [PATCH v2 01/10] drm: omapdrm: Remove remap argument to omap_gem_get_paddr()

2017-04-24 Thread Tomi Valkeinen
On 21/04/17 00:33, Laurent Pinchart wrote: > The function is always called with the remap argument set to true. > Hardcode that behaviour and remove it. > > Signed-off-by: Laurent Pinchart > --- > drivers/gpu/drm/omapdrm/omap_drv.h| 3 +-- > drivers/gpu/drm/omapdrm/omap_fb.c |

Re: [PATCH v2 02/10] drm: omapdrm: Rename occurrences of paddr to dma_addr

2017-04-24 Thread Tomi Valkeinen
On 21/04/17 00:33, Laurent Pinchart wrote: > The fields, variables and functions deal with DMA addresses, name them > accordingly. The omap_gem_get_paddr() and omap_gem_put_paddr() will be > addressed differently separately. > > Signed-off-by: Laurent Pinchart > --- > drivers/gpu/drm/omapdrm/oma

[Bug 98629] OpenGL applications warns "MESA-LOADER: failed to retrieve device information"

2017-04-24 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98629 Emil Velikov changed: What|Removed |Added Status|REOPENED|RESOLVED Resolution|---

Re: [PATCH v2 03/10] drm: omapdrm: Rename omap_gem_(get|put)_paddr() to omap_gem_(un)pin()

2017-04-24 Thread Tomi Valkeinen
On 21/04/17 00:33, Laurent Pinchart wrote: > The reflects the purpose of the function better. s/The/This/? Btw, I usually use "drm/omap" as the subject prefix, it's shorter. > Signed-off-by: Laurent Pinchart > --- > drivers/gpu/drm/omapdrm/omap_drv.h| 4 ++-- > drivers/gpu/drm/omapdrm

Re: [PATCH v2 04/10] drm: omapdrm: Lower indentation level in omap_gem_dma_sync_buffer()

2017-04-24 Thread Tomi Valkeinen
On 21/04/17 00:33, Laurent Pinchart wrote: > This makes the function more readable. > > Signed-off-by: Laurent Pinchart > --- > drivers/gpu/drm/omapdrm/omap_gem.c | 43 > +++--- > 1 file changed, 21 insertions(+), 22 deletions(-) Reviewed-by: Tomi Valkeinen T

Re: [PATCH v2 05/10] drm: omapdrm: Rename the omap_gem_object addrs field to dma_addrs

2017-04-24 Thread Tomi Valkeinen
On 21/04/17 00:33, Laurent Pinchart wrote: > The field contains DMA addresses, clarify that by renaming it. > > Signed-off-by: Laurent Pinchart > --- > drivers/gpu/drm/omapdrm/omap_gem.c | 22 +++--- > 1 file changed, 11 insertions(+), 11 deletions(-) Reviewed-by: Tomi Valkeinen

Re: [PATCH v2 06/10] drm: omapdrm: Rename GEM DMA sync functions

2017-04-24 Thread Tomi Valkeinen
On 21/04/17 00:33, Laurent Pinchart wrote: > The omap_gem_cpu_sync() function operates at a page level, while the > omap_gem_dma_sync() function operates at a buffer level. Rename them to > omap_gem_cpu_sync_page() and omap_gem_dma_sync_buffer() respectively to > avoid confusion. > > Signed-off-by

Re: [PATCH v2 07/10] drm: omapdrm: Fix incorrect usage of the term 'cache coherency'

2017-04-24 Thread Tomi Valkeinen
On 21/04/17 00:33, Laurent Pinchart wrote: > The is_cache_coherent() function currently returns true when the mapping > is not cache-coherent. This isn't a bug as such as the callers interpret > cache-coherent as meaning that the driver has to handle the coherency > manually, but it is nonetheless

Re: [PATCH] drm/etnaviv: add thermal dependency

2017-04-24 Thread Lucas Stach
Am Mittwoch, den 19.04.2017, 20:11 +0200 schrieb Arnd Bergmann: > When CONFIG_THERMAL is enabled as a loadable module, and etnaviv is > built-in, we get a link error: > > drivers/gpu/drm/etnaviv/etnaviv_gpu.o: In function `etnaviv_gpu_bind': > etnaviv_gpu.c:(.text.etnaviv_gpu_bind+0x34): undefined

[Bug 100712] ring 0 stalled after bytes_moved_threshold reached - CAPVERDE/HD7770 - TAHITI/W9000

2017-04-24 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=100712 --- Comment #8 from Julien Isorce --- Hack submitted here https://patchwork.kernel.org/patch/9695945/. It contains some info in the commit message and in the replies. For those who want to try, it is easier to just set R600_DEBUG=nowc as said in

Re: [PATCH RFC 1/7] clk: samsung: Add enable/disable operation for PLL36XX clocks

2017-04-24 Thread Sylwester Nawrocki
On 04/22/2017 04:51 AM, Stephen Boyd wrote: +static int samsung_pll3xxx_enable(struct clk_hw *hw) +{ + struct samsung_clk_pll *pll = to_clk_pll(hw); + u32 tmp; + + tmp = readl_relaxed(pll->con_reg); + tmp |= BIT(pll->enable_offs); + writel_relaxed(tmp, pll->con_reg);

Re: [PATCH RFC 1/7] clk: samsung: Add enable/disable operation for PLL36XX clocks

2017-04-24 Thread Sylwester Nawrocki
On 04/22/2017 05:22 PM, Krzysztof Kozlowski wrote: [...] +static void samsung_pll3xxx_disable(struct clk_hw *hw) +{ + struct samsung_clk_pll *pll = to_clk_pll(hw); + u32 tmp; + + tmp = readl_relaxed(pll->con_reg); + tmp |= BIT(pll->enable_offs); I think you meant here:

[Bug 100067] [OpenCL] const int in argument list crashes build

2017-04-24 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=100067 --- Comment #10 from Vedran Miletić --- Thank you for reporting back! -- You are receiving this mail because: You are the assignee for the bug.___ dri-devel mailing list dri-devel@lists.freedesktop.o

Re: [PATCH v4.1 01/9] drm/atomic: Handle aspect ratio and scaling mode in core, v2.

2017-04-24 Thread Maarten Lankhorst
On 19-04-17 17:43, Daniel Vetter wrote: > On Thu, Apr 13, 2017 at 11:15:37AM +0200, Maarten Lankhorst wrote: >> This is required to for i915 to convert connector properties to atomic. >> >> Changes since v1: >> - Add docbook info. (danvet) >> - Change picture_aspect_ratio to enum hdmi_picture_aspec

Re: [PATCH 1/2] drm: Make fbdev inherit the crtc's initial rotation

2017-04-24 Thread Ville Syrjälä
On Sun, Apr 23, 2017 at 06:11:04PM +0200, Hans de Goede wrote: > Hi All, > > So I recently bought a (second-hand) Bay Trail tablet which has its LCD > mounted upside-down. As such I've ported Ville Syrjala's patches to deal > with this to current mainline and I'm hereby posting them upstream > for

[Bug 194843] [amdgpu] oops [drm:gfx_v8_0_priv_reg_irq] *ERROR* Illegal register access in command stream

2017-04-24 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=194843 --- Comment #8 from Johannes Hirte (johannes.hi...@datenkhaos.de) --- (In reply to Michel Dänzer from comment #7) > I wonder if there might be memory corruption going on, in which case > enabling CONFIG_KASAN for the kernel build might give more c

Re: [PATCH libdrm v2] Header: Add rotation property fields

2017-04-24 Thread Emil Velikov
On 18 April 2017 at 23:16, Daniel Vetter wrote: > On Tue, Apr 18, 2017 at 8:33 PM, Kristian Høgsberg > wrote: As far as I understand the property mechanism, the numeric values aren't actually ABI. The string names of the enum values are the ABI and you're supposed to parse the enu

Re: [PATCH RFC 3/6] drm: Plane YCbCr to RGB conversion related properties

2017-04-24 Thread Brian Starkey
On Fri, Apr 21, 2017 at 07:49:04PM +0300, Ville Syrjälä wrote: On Fri, Apr 21, 2017 at 04:34:00PM +0100, Brian Starkey wrote: On Fri, Apr 21, 2017 at 06:21:48PM +0300, Ville Syrjälä wrote: >On Fri, Apr 21, 2017 at 03:53:42PM +0100, Brian Starkey wrote: >> Hi, >> >> Thanks for picking this up Jyr

Re: [PATCH] drm: fourcc byteorder: brings header file comments in line with reality.

2017-04-24 Thread Ville Syrjälä
On Mon, Apr 24, 2017 at 03:57:02PM +0900, Michel Dänzer wrote: > On 22/04/17 07:05 PM, Ville Syrjälä wrote: > > On Fri, Apr 21, 2017 at 06:14:31PM +0200, Gerd Hoffmann wrote: > >> Hi, > >> > My personal opinion is that formats in drm_fourcc.h should be > independent of the CPU byte ord

Re: [PATCH 2/2] drm/panel: simple: add support for NLT NL192108AC18-02D

2017-04-24 Thread Lucas Stach
Hi Rob, Am Mittwoch, den 19.04.2017, 16:15 -0500 schrieb Rob Herring: > On Wed, Apr 12, 2017 at 07:15:26PM +0200, Lucas Stach wrote: > > This adds support for the NLT Technologies NL192108AC18-02D > > 15.6" LVDS FullHD TFT LCD panel, which can be supported > > by the simple panel driver. > > > >

Re: [PATCH RFC 3/6] drm: Plane YCbCr to RGB conversion related properties

2017-04-24 Thread Jyri Sarha
On 04/21/17 16:52, Ville Syrjälä wrote: > On Fri, Apr 21, 2017 at 04:39:16PM +0300, Jyri Sarha wrote: >> On 04/21/17 14:17, Ville Syrjälä wrote: +static char *ycbcr_to_rgb_mode_name[] = { + [DRM_PLANE_YCBCR_BT601_LIM_TO_RGB_BT601_FULL] = + "YCbCr BT.601 limited range TO RGB

Re: [PATCH v2 5/5] drm: omapdrm: Add zpos property

2017-04-24 Thread Laurent Pinchart
Hi Tomi, On Monday 24 Apr 2017 12:37:24 Tomi Valkeinen wrote: > On 15/04/17 12:16, Laurent Pinchart wrote: > > Create a standard zpos property for every plane as an alias to the > > omapdrm-specific zorder property. Unlike the zorder property that has to > > be instantiated for both planes and CRT

Re: [PATCH v2 4/5] drm: omapdrm: Store the Z order in the plane state zpos field

2017-04-24 Thread Laurent Pinchart
Hi Tomi, On Monday 24 Apr 2017 12:40:10 Tomi Valkeinen wrote: > On 15/04/17 12:16, Laurent Pinchart wrote: > > The DRM core implements a standard "zpos" property to control planes > > ordering. The omapdrm driver implements a similar property named > > "zorder". Although we can't switch to DRM cor

Re: [PATCH v2 5/5] drm: omapdrm: Add zpos property

2017-04-24 Thread Laurent Pinchart
Hi Tomi, On Monday 24 Apr 2017 17:00:52 Laurent Pinchart wrote: > On Monday 24 Apr 2017 12:37:24 Tomi Valkeinen wrote: > > On 15/04/17 12:16, Laurent Pinchart wrote: > >> Create a standard zpos property for every plane as an alias to the > >> omapdrm-specific zorder property. Unlike the zorder pro

Re: [PATCH v2 03/10] drm: omapdrm: Rename omap_gem_(get|put)_paddr() to omap_gem_(un)pin()

2017-04-24 Thread Laurent Pinchart
Hi Tomi, On Monday 24 Apr 2017 13:16:18 Tomi Valkeinen wrote: > On 21/04/17 00:33, Laurent Pinchart wrote: > > The reflects the purpose of the function better. > > s/The/This/? Yes, my bad, sorry. I'll fix it in my branch. > Btw, I usually use "drm/omap" as the subject prefix, it's shorter. I

[Bug 194843] [amdgpu] oops [drm:gfx_v8_0_priv_reg_irq] *ERROR* Illegal register access in command stream

2017-04-24 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=194843 --- Comment #9 from Johannes Hirte (johannes.hi...@datenkhaos.de) --- (In reply to Johannes Hirte from comment #8) > (In reply to Michel Dänzer from comment #7) > > I wonder if there might be memory corruption going on, in which case > > enabling

Re: [PATCH v2 07/10] drm: omapdrm: Fix incorrect usage of the term 'cache coherency'

2017-04-24 Thread Laurent Pinchart
Hi Tomi, On Monday 24 Apr 2017 13:44:23 Tomi Valkeinen wrote: > On 21/04/17 00:33, Laurent Pinchart wrote: > > The is_cache_coherent() function currently returns true when the mapping > > is not cache-coherent. This isn't a bug as such as the callers interpret > > cache-coherent as meaning that th

Re: [PATCH 0/6] drm: tackle byteorder issues, take two

2017-04-24 Thread Ville Syrjälä
On Mon, Apr 24, 2017 at 04:54:25PM +0900, Michel Dänzer wrote: > On 24/04/17 04:36 PM, Gerd Hoffmann wrote: > > > >>> drm: fourcc byteorder: add DRM_FORMAT_CPU_* > >>> drm: fourcc byteorder: add bigendian support to > >>> drm_mode_legacy_fb_format > >> > >> As I explained in my last follow

Re: [PATCH 2/3] drm/vc4: Don't try to initialize FBDEV if we're only bound to V3D.

2017-04-24 Thread Alex Deucher
On Fri, Apr 21, 2017 at 6:53 PM, Eric Anholt wrote: > Daniel Vetter writes: > >> On Wed, Apr 19, 2017 at 7:55 PM, Eric Anholt wrote: >>> Daniel Vetter writes: On Tue, Apr 18, 2017 at 9:11 PM, Eric Anholt wrote: > The FBDEV initialization would throw an error in dmesg, when we just >>>

[GIT PULL] drm/mali-dp changes for v4.12

2017-04-24 Thread Liviu Dudau
Hi Dave, Latest updates on Mali DP, adding support for colour management, plane scaling and power management. Many thanks, Liviu The following changes since commit 6b1462700b4a6a1244c018cdd2fa97ded43090a0: Merge tag 'drm-misc-next-fixes-2017-04-20' of git://anongit.freedesktop.org/git/drm-mi

Re: [PATCH -next] gpu: host1x: Fix wrong pointer passed to PTR_ERR()

2017-04-24 Thread Jon Hunter
On 16/04/17 05:08, Wei Yongjun wrote: > From: Wei Yongjun > > PTR_ERR should access the value just tested by IS_ERR, otherwise > the wrong error code will be returned. > > Fixes: b386c6b73ac6 ("gpu: host1x: Support module reset") > Signed-off-by: Wei Yongjun > --- > drivers/gpu/host1x/dev.c |

Re: [PATCH RFC 3/6] drm: Plane YCbCr to RGB conversion related properties

2017-04-24 Thread Ville Syrjälä
On Mon, Apr 24, 2017 at 04:21:39PM +0300, Jyri Sarha wrote: > On 04/21/17 16:52, Ville Syrjälä wrote: > > On Fri, Apr 21, 2017 at 04:39:16PM +0300, Jyri Sarha wrote: > >> On 04/21/17 14:17, Ville Syrjälä wrote: > +static char *ycbcr_to_rgb_mode_name[] = { > +[DRM_PLANE_YCBCR_BT601

Re: [PATCH RFC 3/6] drm: Plane YCbCr to RGB conversion related properties

2017-04-24 Thread Jyri Sarha
On 04/24/17 18:13, Ville Syrjälä wrote: >> What about the uapi structs? In the patch there is an explicit struct >> naming each coefficient for what they are for in YCbCr to RGB >> conversion. Is this Ok, or should we go with a generic (CTM style) >> matrix, that could be used for RGB to YCbCr conv

[Bug 100726] [REGRESSION][BISECTED] Severe flickering with an R9 290

2017-04-24 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=100726 --- Comment #3 from hiwatari.se...@gmail.com --- This seems to have been fixed somewhere between 4.10.8 and 4.10.11. -- You are receiving this mail because: You are the assignee for the bug.___ dri-de

Re: [PATCH v7 0/4] Adding driver-private objects to atomic state

2017-04-24 Thread Harry Wentland
Patches 1-3: Reviewed-by: Harry Wentland Patch 4: Acked-by: Harry Wentland Harry On 2017-04-21 01:51 AM, Dhinakaran Pandiyan wrote: Changes in this version: Used connector->atomic_check() to release vcpi slots instead of the atomic_release() callback. This series introduces void * type drive

[PATCH 2/2] drm/amd/display: Prevent premature pageflip when comitting in vblank. (v3)

2017-04-24 Thread Mario Kleiner
Make sure we do not program a hw pageflip inside vblank 'n' iff the atomic flip is comitted while inside the same vblank 'n'. We must defer such a flip by one refresh cycle to vblank 'n+1', unless this is a DRM_MODE_PAGE_FLIP_ASYNC async pageflip, which must always execute as soon as possible. Wit

Re: [PATCH RFC 3/6] drm: Plane YCbCr to RGB conversion related properties

2017-04-24 Thread Ville Syrjälä
On Mon, Apr 24, 2017 at 06:49:04PM +0300, Jyri Sarha wrote: > On 04/24/17 18:13, Ville Syrjälä wrote: > >> What about the uapi structs? In the patch there is an explicit struct > >> naming each coefficient for what they are for in YCbCr to RGB > >> conversion. Is this Ok, or should we go with a gen

Re: [PATCH 0/6] drm: tackle byteorder issues, take two

2017-04-24 Thread Daniel Stone
Hi, On 24 April 2017 at 15:26, Ville Syrjälä wrote: > On Mon, Apr 24, 2017 at 04:54:25PM +0900, Michel Dänzer wrote: >> On 24/04/17 04:36 PM, Gerd Hoffmann wrote: >> > When running in opengl mode there will be a hardware-specific mesa >> > driver in userspace, which will either know what the gpu

Re: [PATCH 0/6] drm: tackle byteorder issues, take two

2017-04-24 Thread Ville Syrjälä
On Mon, Apr 24, 2017 at 05:26:03PM +0300, Ville Syrjälä wrote: > On Mon, Apr 24, 2017 at 04:54:25PM +0900, Michel Dänzer wrote: > > On 24/04/17 04:36 PM, Gerd Hoffmann wrote: > > > > > >>> drm: fourcc byteorder: add DRM_FORMAT_CPU_* > > >>> drm: fourcc byteorder: add bigendian support to > > >

Re: [PATCH] drm/etnaviv: add thermal dependency

2017-04-24 Thread Arnd Bergmann
On Mon, Apr 24, 2017 at 12:44 PM, Lucas Stach wrote: > Am Mittwoch, den 19.04.2017, 20:11 +0200 schrieb Arnd Bergmann: >> When CONFIG_THERMAL is enabled as a loadable module, and etnaviv is >> built-in, we get a link error: >> >> drivers/gpu/drm/etnaviv/etnaviv_gpu.o: In function `etnaviv_gpu_bind

[Bug 100726] [REGRESSION][BISECTED] Severe flickering with an R9 290

2017-04-24 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=100726 Alex Deucher changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

RE: [PATCH 2/2] drm/amd/display: Prevent premature pageflip when comitting in vblank. (v3)

2017-04-24 Thread Grodzovsky, Andrey
> -Original Message- > From: Mario Kleiner [mailto:mario.kleiner...@gmail.com] > Sent: Monday, April 24, 2017 12:54 PM > To: dri-devel@lists.freedesktop.org > Cc: amd-...@lists.freedesktop.org; Grodzovsky, Andrey; > mario.kleiner...@gmail.com; Wentland, Harry; Deucher, Alexander; > Daenze

[PATCH] drm/pl111: Register the clock divider and use it.

2017-04-24 Thread Eric Anholt
This is required for the panel to work on bcm911360, where CLCDCLK is the fixed 200Mhz AXI41 clock. The rate set is still passed up to the CLCDCLK, for platforms that have a settable rate on that one. Signed-off-by: Eric Anholt --- drivers/gpu/drm/pl111/pl111_display.c | 161 +++

[PATCH 1/3 v3] drm/vc4: Turn the V3D clock on at runtime.

2017-04-24 Thread Eric Anholt
For the Raspberry Pi's bindings, the power domain also implicitly turns on the clock and deasserts reset, but for the new Cygnus port we start representing the clock in the devicetree. v2: Document the clock-names property, check for -ENOENT for no clock in DT. v3: Drop NULL checks around clk

[Bug 100781] "Black Mesa" will not Launch

2017-04-24 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=100781 Bug ID: 100781 Summary: "Black Mesa" will not Launch Product: Mesa Version: 17.0 Hardware: x86-64 (AMD64) OS: Linux (All) Status: NEW Severity: normal

[Bug 100781] "Black Mesa" will not Launch

2017-04-24 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=100781 katof...@protonmail.com changed: What|Removed |Added CC||katof...@protonmail.com

[PATCH v2 08/29] drm/hisilicon: fix include notation and remove -Iinclude/drm flag

2017-04-24 Thread Masahiro Yamada
Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. Signed-off-by: Masahiro Yamada --- Changes in v2: None drivers/gpu/drm/hisilicon/hibmc/Makefile| 1 - drivers/gpu/drm/hisilicon/hibmc/hibmc_ttm.c | 2 +- 2 files changed, 1 insertion(+), 2 dele

[PATCH v2 11/29] drm/nouveau: fix include notation and remove -Iinclude/drm flag

2017-04-24 Thread Masahiro Yamada
Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. Signed-off-by: Masahiro Yamada --- Changes in v2: None drivers/gpu/drm/nouveau/Kbuild| 1 - drivers/gpu/drm/nouveau/nouveau_drm.c | 4 ++-- drivers/gpu/drm/nouveau/nouveau_drv.h | 2 +- dri

[PATCH v2 25/29] drm/sis: remove unneeded -Iinclude/drm compiler flag

2017-04-24 Thread Masahiro Yamada
With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- Changes in v2: None drivers/gpu/drm/sis/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/sis/Makefile b/drivers/gpu/drm/sis/Makefile index 441c061.

[PATCH v2 16/29] drm/virtio: fix include notation and remove -Iinclude/drm flag

2017-04-24 Thread Masahiro Yamada
Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. Signed-off-by: Masahiro Yamada --- Changes in v2: None drivers/gpu/drm/virtio/Makefile | 2 -- drivers/gpu/drm/virtio/virtgpu_debugfs.c | 2 +- drivers/gpu/drm/virtio/virtgpu_drv.c |

[PATCH v2 26/29] drm/tdfx: remove unneeded -Iinclude/drm compiler flag

2017-04-24 Thread Masahiro Yamada
With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- Changes in v2: None drivers/gpu/drm/tdfx/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/tdfx/Makefile b/drivers/gpu/drm/tdfx/Makefile index 0379f

[PATCH v2 29/29] drm/via: remove unneeded -Iinclude/drm compiler flag

2017-04-24 Thread Masahiro Yamada
With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- Changes in v2: None drivers/gpu/drm/via/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/via/Makefile b/drivers/gpu/drm/via/Makefile index d59e258.

[PATCH v2 24/29] drm/savage: remove unneeded -Iinclude/drm compiler flag

2017-04-24 Thread Masahiro Yamada
With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- Changes in v2: None drivers/gpu/drm/savage/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/savage/Makefile b/drivers/gpu/drm/savage/Makefile index

[PATCH v2 15/29] drm/vc4: fix include notation and remove -Iinclude/drm flag

2017-04-24 Thread Masahiro Yamada
Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. While we are here, use <...> instead of "..." for include/linux/*.h headers too. Signed-off-by: Masahiro Yamada --- Changes in v2: None drivers/gpu/drm/vc4/Makefile| 2 -- drivers/gpu/drm/vc4

[PATCH v2 10/29] drm/msm: fix include notation and remove -Iinclude/drm flag

2017-04-24 Thread Masahiro Yamada
Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. While we are here, sort the touched parts with public headers first. mdp4_kms.h must declare struct device_node to be self-contained. Signed-off-by: Masahiro Yamada --- Changes in v2: - Sort inclu

[PATCH v2 18/29] drm/gma500: remove unneeded -Iinclude/drm compiler flag

2017-04-24 Thread Masahiro Yamada
With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- Changes in v2: None drivers/gpu/drm/gma500/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/gma500/Makefile b/drivers/gpu/drm/gma500/Makefile index

Re: [PATCH v2 2/2] drm: dw-hdmi: gate audio clock from the I2S enablement callbacks

2017-04-24 Thread Romain Perier
Hello, Le 19/04/2017 à 06:51, Archit Taneja a écrit : > > > On 04/14/2017 02:01 PM, Romain Perier wrote: >> Currently, the audio sampler clock is enabled from dw_hdmi_setup() at >> step E. and is kept enabled for later use. This clock should be enabled >> and disabled along with the actual audio

[PATCH v2 22/29] drm/omap: remove unneeded -Iinclude/drm compiler flag

2017-04-24 Thread Masahiro Yamada
With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- Changes in v2: None drivers/gpu/drm/omapdrm/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/omapdrm/Makefile b/drivers/gpu/drm/omapdrm/Makefile in

Re: [PATCH 0/4] Fix DP busy wait and defer disabling overlay plane

2017-04-24 Thread Dan MacDonald
Hi Phillip Do you think there is any hope for us getting the GPU to work on my Element14 SABRE Lite or is it a lost cause, with it being such a rare board combined with my lack of of kernel dev skills and limited dev time for this? I'm sure I'll find a non-GPU dependent use for the SL if that is t

Re: [PATCH v2 2/2] drm: dw-hdmi: gate audio clock from the I2S enablement callbacks

2017-04-24 Thread Romain Perier
Hello, > Took the liberty of going ahead with the fixes. Queued both patches to > drm-misc-next. > > Thanks, > Archit You were right. Sorry for the delay, I am back from vacations :) Regards, Romain ___ dri-devel mailing list dri-devel@lists.freedeskto

Re: [PATCH] drm/etnaviv: add thermal dependency

2017-04-24 Thread Eduardo Valentin
dg On Mon, Apr 24, 2017 at 07:39:27PM +0200, Arnd Bergmann wrote: > On Mon, Apr 24, 2017 at 12:44 PM, Lucas Stach wrote: > > Am Mittwoch, den 19.04.2017, 20:11 +0200 schrieb Arnd Bergmann: > >> When CONFIG_THERMAL is enabled as a loadable module, and etnaviv is > >> built-in, we get a link error:

[PATCH v2 09/29] drm/mgag200: fix include notation and remove -Iinclude/drm flag

2017-04-24 Thread Masahiro Yamada
Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. Signed-off-by: Masahiro Yamada --- Changes in v2: - Sort the touched part alphabetically drivers/gpu/drm/mgag200/Makefile | 1 - drivers/gpu/drm/mgag200/mgag200_ttm.c | 3 ++- 2 files change

[PATCH v2 01/29] drm: make drm_panel.h self-contained

2017-04-24 Thread Masahiro Yamada
-ENOSYS and -EINVAL are referenced in some static inline functions. of_drm_find_pane() takes a pointer to struct device_node. Make this header self-contained to not depend on specific include order. Signed-off-by: Masahiro Yamada --- Changes in v2: None include/drm/drm_panel.h | 2 ++ 1 file

[PATCH v2 02/29] drm/ttm: fix include notation and remove -Iinclude/drm flag

2017-04-24 Thread Masahiro Yamada
For the C file, include instead of relative path from include/drm. For headers in include/drm/ttm, simplify the with "*.h". This allows us to remove the -Iinclude/drm compiler flag from drivers/gpu/drm/ttm/Makefile (and from other drivers' Makefiles). Signed-off-by: Masahiro Yamada --- Chang

[PATCH v2 20/29] drm/i2c: remove unneeded -Iinclude/drm compiler flag

2017-04-24 Thread Masahiro Yamada
With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- Changes in v2: None drivers/gpu/drm/i2c/Makefile | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/i2c/Makefile b/drivers/gpu/drm/i2c/Makefile index 43aa33

[PATCH v2 03/29] drm/amd: fix include notation and remove -Iinclude/drm flag

2017-04-24 Thread Masahiro Yamada
Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. Signed-off-by: Masahiro Yamada Reviewed-by: Michel Dänzer --- Changes in v2: None drivers/gpu/drm/amd/amdgpu/Makefile | 2 +- drivers/gpu/drm/amd/amdgpu/amdgpu.h | 10 +- driver

Re: [PATCH RFC 1/7] clk: samsung: Add enable/disable operation for PLL36XX clocks

2017-04-24 Thread Krzysztof Kozlowski
On Mon, Apr 24, 2017 at 1:12 PM, Sylwester Nawrocki wrote: > On 04/22/2017 05:22 PM, Krzysztof Kozlowski wrote: >>> @@ -354,10 +359,12 @@ static int samsung_pll36xx_set_rate(struct clk_hw >>> *hw, unsigned long drate, >>> writel_relaxed(pll_con1, pll->con_reg + 4); >>> >>> /* wait_

[PATCH v2 17/29] drm/vmwgfx: fix include notation and remove -Iinclude/drm flag

2017-04-24 Thread Masahiro Yamada
Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. Signed-off-by: Masahiro Yamada --- Changes in v2: None drivers/gpu/drm/vmwgfx/Makefile | 3 --- drivers/gpu/drm/vmwgfx/vmwgfx_cmdbuf.c | 3 ++- drivers/gpu/drm/vmwgfx/vmwgfx_context.c | 3

[PATCH v2 19/29] drm/i810: remove unneeded -Iinclude/drm compiler flag

2017-04-24 Thread Masahiro Yamada
With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- Changes in v2: None drivers/gpu/drm/i810/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/i810/Makefile b/drivers/gpu/drm/i810/Makefile index 43844

Re: [PATCH 0/4] Fix DP busy wait and defer disabling overlay plane

2017-04-24 Thread Dan MacDonald
Having another quick search now had revealed the Hummingboard Edge, which has 2GB RAM and M.2, although I've not found a UK reseller yet. That seems like it could be another good choice? On Mon, Apr 24, 2017 at 10:18 AM, Dan MacDonald wrote: > Hi Phillip > > Do you think there is any hope for us

[PATCH v2 23/29] drm/r128: remove unneeded -Iinclude/drm compiler flag

2017-04-24 Thread Masahiro Yamada
With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- Changes in v2: None drivers/gpu/drm/r128/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/r128/Makefile b/drivers/gpu/drm/r128/Makefile index 1cc72

[PATCH v2 04/29] drm/ast: fix include notation and remove -Iinclude/drm flag

2017-04-24 Thread Masahiro Yamada
Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. Signed-off-by: Masahiro Yamada --- Changes in v2: - Sort the touched part alphabetically drivers/gpu/drm/ast/Makefile | 2 -- drivers/gpu/drm/ast/ast_ttm.c | 3 ++- 2 files changed, 2 insertions

  1   2   >