[PATCH v4 4/7] drm/tilcdc: Add DRM_TILCDC_SLAVE_COMPAT for ti, tilcdc, slave binding support

2015-04-01 Thread Jyri Sarha
Adds a CONFIG_DRM_TILCDC_SLAVE_COMPAT module for "ti,tilcdc,slave" node conversion. The implementation is in tilcdc_slave_compat.c and it uses tilcdc_slave_compat.dts as a basis for creating a DTS overlay. The DTS overlay adds an external tda998x encoder to tilcdc that corresponds to the old tda998

[PATCH v4 7/7] drm/tilcdc: Decrement refcount of ep-node from of_graph_get_next_endpoint

2015-04-01 Thread Jyri Sarha
This patch should be dropped/reverterd if/after "of: Decrement refcount of previous endpoint in of_graph_get_next_endpoint" patch has been merged. Signed-off-by: Jyri Sarha --- drivers/gpu/drm/tilcdc/tilcdc_external.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/tilcdc/til

[PATCH v4 3/7] drm/tilcdc: Add support for external tda998x encoder

2015-04-01 Thread Jyri Sarha
Add support for an external compontised DRM encoder. The external encoder can be connected to tilcdc trough device tree graph binding. The binding document for tilcdc has been updated. The current implementation supports only tda998x encoder. To be able to filter out the unsupported video modes th

[PATCH v4 5/7] drm/tilcdc: Force building of DRM_TILCDC_SLAVE_COMPAT

2015-04-01 Thread Jyri Sarha
If I read Documentation/kbuild/makefiles.txt section 3.6 right, this patch should not be needed. However, without this patch the objects needed for DRM_TILCDC_SLAVE_COMPAT are not linked, if DRM_TILCDC is built as module. Signed-off-by: Jyri Sarha --- drivers/gpu/drm/Makefile | 2 +- 1 file chan

[PATCH v4 6/7] ARM: dts: am335x-boneblack: Use new binding for HDMI

2015-04-01 Thread Jyri Sarha
Use new binding for the external tda19988 HDMI encoder. Signed-off-by: Jyri Sarha --- arch/arm/boot/dts/am335x-boneblack.dts | 20 +++- 1 file changed, 15 insertions(+), 5 deletions(-) diff --git a/arch/arm/boot/dts/am335x-boneblack.dts b/arch/arm/boot/dts/am335x-boneblack.dts

[PATCH v4 0/7] Use DRM component API in tilcdc to connect to tda998x

2015-04-01 Thread Jyri Sarha
Ok, let's do one more full review round. The mode filtering issue was the main reason for this new patch series version. However, I found couple other things to fix too after scrutinizing the patches once more. Changes since v3 version of the patch-set: * drm/tilcdc: Add support for external tda99

[PATCH v4 2/7] drm/tilcdc: Remove tilcdc slave support for tda998x driver

2015-04-01 Thread Jyri Sarha
Remove tilcdc slave support for tda998x driver. The tilcdc slave support would conflicts with componentized use of tda998x. Signed-off-by: Jyri Sarha --- .../devicetree/bindings/drm/tilcdc/slave.txt | 18 - drivers/gpu/drm/tilcdc/Makefile| 1 - drivers/gpu/drm/tilcdc

[PATCH v4 1/7] drm/tilcdc: Fix module unloading

2015-04-01 Thread Jyri Sarha
Force crtc dpms off before destroying the crtc instead of just checking the dpms state. This fixes warning message and frozen picture after tilcdc module unloading. Signed-off-by: Jyri Sarha --- drivers/gpu/drm/tilcdc/tilcdc_crtc.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --

[PATCH RFC v2 06/12] dt-bindings: Add documentation for rockchip lvds

2015-04-01 Thread Heiko Stuebner
From: Mark Yao Add binding documentation for Rockchip SoC LVDS driver. Signed-off-by: Mark Yao Signed-off-by: Heiko Stuebner --- .../devicetree/bindings/video/rockchip-lvds.txt| 74 ++ 1 file changed, 74 insertions(+) create mode 100644 Documentation/devicetree/bindin

[PATCH RFC v2 04/12] drm/components: add generic vga encoder driver

2015-04-01 Thread Heiko Stuebner
This adds a driver for generic vga encoders like the Analog Devices adv7123 and similar ics. These chips do not have any special configuration options except a powersafe gpio. An exception is added for the rcar-du driver which also implements support for the adv7123 internally but is not yet conve

[PATCH RFC v2 03/12] drm: add components subdirectory and infrastructure

2015-04-01 Thread Heiko Stuebner
Until there are only the specialized bridge and i2c directories available to hold generic support code like external components. But as there are also things like external encoders that do not use i2c, those do not match all cases. Therefore introduce a new subdirectories to hold generic components

[PATCH RFC v2 08/12] drm/rockchip: lvds: register a bridge when no panel is set

2015-04-01 Thread Heiko Stuebner
On socs using the lvds components it also controls the use of the general rgb outputs and must thus be configured for things like external encoders. Therefore register a drm_bridge in this case and try to find the encoder in the output port. Signed-off-by: Heiko Stuebner --- drivers/gpu/drm/roc

[PATCH RFC v2 07/12] drm/rockchip: Add support for Rockchip Soc LVDS

2015-04-01 Thread Heiko Stuebner
From: Mark Yao This adds support for Rockchip soc lvds found on rk3288 Signed-off-by: Mark Yao Signed-off-by: Heiko Stuebner --- This still needs to address Laurent's comment about trying to get the lvds settings from the panel/connected device instead of encoding them as separate properties i

[PATCH RFC v2 01/12] drm/encoder: add functionality to register encoders to a global list

2015-04-01 Thread Heiko Stuebner
This allows standalone encoders to be registered and found again through their devicetree node when going through their connection graph. Setting the of_node property is of course still optional, as it is not necessary in most cases to lookup encoders that are part of a bigger component. Signed-o

[PATCH RFC v2 05/12] drm/components: add generic vga connector driver

2015-04-01 Thread Heiko Stuebner
This adds a driver for generic vga connectors using a system i2c-bus for ddc. An exception is included for rcar-du which implements the vga-connector binding interally already and is not yet converted to the component framework. Signed-off-by: Heiko Stuebner --- drivers/gpu/drm/components/Kconf

[PATCH RFC v2 00/12] drm/rockchip: add support for lvds controller and external encoders

2015-04-01 Thread Heiko Stuebner
changes since v1: - add separate components subdirectory - implement the already existing bindings for adv7123 and vga-connector instead of defining a new one - use component graph for subsequent lvds-panel or connected external encoders instead of defining special properties. This series sti

[PATCH RFC v2 02/12] drm/connector: add functionality to register connectors to a global list

2015-04-01 Thread Heiko Stuebner
This allows standalone connectors to be registered and found again through their devicetree node when going through their connection graph. Setting the of_node property is of course still optional, as it is not necessary in most cases to lookup connectors that are part of a bigger component. Sign

[PATCH RFC v2 09/12] drm/rockchip: enable rgb output of vops for all other connectors

2015-04-01 Thread Heiko Stuebner
The socs itself contains encoders for a lot of different outputs. But every unsupported connector will be routed through the lvds, as it controls the pins in question. Therefore enable the lvds output for all of those. Signed-off-by: Heiko Stuebner --- drivers/gpu/drm/rockchip/rockchip_drm_vop.c

[PATCH RFC v2 04/12] drm/components: add generic vga encoder driver

2015-04-01 Thread Russell King - ARM Linux
On Wed, Apr 01, 2015 at 12:09:38PM +0200, Heiko Stuebner wrote: > This adds a driver for generic vga encoders like the Analog Devices adv7123 > and similar ics. These chips do not have any special configuration options > except a powersafe gpio. > > An exception is added for the rcar-du driver whi

[PATCH RFC v2 11/12] ARM: dts: rockchip: add rk3288 lvds node

2015-04-01 Thread Heiko Stuebner
Add the basic node for the lvds controller of rk3288 and hook it into the display-subsystem hirarchy. Signed-off-by: Heiko Stuebner --- arch/arm/boot/dts/rk3288.dtsi | 42 ++ 1 file changed, 42 insertions(+) diff --git a/arch/arm/boot/dts/rk3288.dtsi b/ar

[PATCH RFC v2 10/12] ARM: dts: rockchip: add rk3288 lcdc0 pinmux settings

2015-04-01 Thread Heiko Stuebner
Add pinctrl settings for the configurable lcdc0 signals dclk, den, hsync and vsync. The lcdc0 data pin configuration is not software controlable. Signed-off-by: Heiko Stuebner --- arch/arm/boot/dts/rk3288.dtsi | 9 + 1 file changed, 9 insertions(+) diff --git a/arch/arm/boot/dts/rk3288.

[PATCH RFC v2 12/12] ARM: dts: rockchip: add vga encoder and enable lvds on rk3288-firefly

2015-04-01 Thread Heiko Stuebner
Add the sda7123 simple vga encoder, connect it to the vop outputs and enable the lvds controller with the correct settings. Signed-off-by: Heiko Stuebner --- arch/arm/boot/dts/rk3288-firefly.dtsi | 68 +++ 1 file changed, 68 insertions(+) diff --git a/arch/arm/bo

[PATCH RFC v2 04/12] drm/components: add generic vga encoder driver

2015-04-01 Thread Heiko Stübner
Hi Russell, Am Mittwoch, 1. April 2015, 11:27:51 schrieb Russell King - ARM Linux: > On Wed, Apr 01, 2015 at 12:09:38PM +0200, Heiko Stuebner wrote: > > This adds a driver for generic vga encoders like the Analog Devices > > adv7123 > > and similar ics. These chips do not have any special configur

[PATCH RFC 07/11] drm/edid: add function to help find SADs

2015-04-01 Thread Jani Nikula
On Mon, 30 Mar 2015, Russell King wrote: > Add a function to find the start of the SADs in the ELD. This > complements the helper to retrieve the SAD count. > > Signed-off-by: Russell King I guess version 31 is specific enough to warrant a #define of its own, but meh. Reviewed-by: Jani Nikula

[Bug 87278] Packet0 not allowed and GPU fault detected errors with Serious Engine games

2015-04-01 Thread bugzilla-dae...@freedesktop.org
s the binary garbage at the beginning. -- You are receiving this mail because: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150401/7091ef0d/attachment.html>

[Bug 87278] Packet0 not allowed and GPU fault detected errors with Serious Engine games

2015-04-01 Thread bugzilla-dae...@freedesktop.org
--- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150401/048d98ce/attachment.html>

[PATCH RFC 07/11] drm/edid: add function to help find SADs

2015-04-01 Thread Russell King - ARM Linux
On Wed, Apr 01, 2015 at 02:47:32PM +0300, Jani Nikula wrote: > On Mon, 30 Mar 2015, Russell King wrote: > > Add a function to find the start of the SADs in the ELD. This > > complements the helper to retrieve the SAD count. > > > > Signed-off-by: Russell King > > I guess version 31 is specific

[PATCH v2 0/4] Add virtio gpu driver.

2015-04-01 Thread Gerd Hoffmann
Hi, Next version of the virtio-gpu driver. Turned into a little patch series now. Most review comments should be addressed. Changes in v2: * add support for universal plane * add support for atomic modesetting * make non-failing funcs return void, kill pointless error checking * allocate

[PATCH v2 1/4] break kconfig dependency loop

2015-04-01 Thread Gerd Hoffmann
After adding virtio-gpu I get this funky kconfig dependency loop. scripts/kconfig/conf --oldconfig Kconfig drivers/video/fbdev/Kconfig:5:error: recursive dependency detected! drivers/video/fbdev/Kconfig:5: symbol FB is selected by DRM_KMS_FB_HELPER drivers/gpu/drm/Kconfig:34: symbol DRM_KMS_F

[PATCH v2 2/4] drm_vblank_get: don't WARN_ON in case vblanks are not initialized

2015-04-01 Thread Gerd Hoffmann
Signed-off-by: Gerd Hoffmann --- drivers/gpu/drm/drm_irq.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/gpu/drm/drm_irq.c b/drivers/gpu/drm/drm_irq.c index 10574a0..c631c1a 100644 --- a/drivers/gpu/drm/drm_irq.c +++ b/drivers/gpu/drm/drm_irq.c @@ -1026,6 +1026,9 @@ int drm_vblan

[PATCH v2 4/4] Add virtio-vga bits.

2015-04-01 Thread Gerd Hoffmann
--- drivers/gpu/drm/virtio/virtgpu_drm_bus.c | 32 ++-- drivers/virtio/virtio_pci_common.c | 7 ++- 2 files changed, 36 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/virtio/virtgpu_drm_bus.c b/drivers/gpu/drm/virtio/virtgpu_drm_bus.c index 56bd

[PATCH v2 3/4] Add virtio gpu driver.

2015-04-01 Thread Gerd Hoffmann
From: Dave Airlie This patch adds a kms driver for the virtio gpu. The xorg modesetting driver can handle the device just fine, the framebuffer for fbcon is there too. Qemu patches for the host side are under review currently. The pci version of the device comes in two variants: with and witho

[PATCH v2 4/4] Add virtio-vga bits.

2015-04-01 Thread Michael S. Tsirkin
On Wed, Apr 01, 2015 at 03:15:30PM +0200, Gerd Hoffmann wrote: > +static void virtio_pci_kick_out_firmware_fb(struct pci_dev *pci_dev) > +{ > + struct apertures_struct *ap; > + bool primary; > + > + ap = alloc_apertures(1); > + if (!ap) > + return; > + > + ap->ranges

[PATCH 1/3] drm/exynos: fix typos in hdmi and mixer

2015-04-01 Thread Tobias Jakobi
Use the correct spelling for 'progressive'. Signed-off-by: Tobias Jakobi --- drivers/gpu/drm/exynos/exynos_hdmi.c | 2 +- drivers/gpu/drm/exynos/exynos_mixer.c | 2 +- drivers/gpu/drm/exynos/regs-mixer.h | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/ex

[PATCH 2/3] drm/exynos: remove superfluous error messages

2015-04-01 Thread Tobias Jakobi
The messages are redundant since 'check_fb_gem_memory_type' already prints out exactly the same string when it fails. Signed-off-by: Tobias Jakobi --- drivers/gpu/drm/exynos/exynos_drm_fb.c | 10 +++--- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/drivers/gpu/drm/exynos/exyn

[PATCH 3/3] drm/exynos: mixer: add 2x scaling to mixer_graph_buffer

2015-04-01 Thread Tobias Jakobi
While the VP (video processor) supports arbitrary scaling of its input, the mixer just supports a simple 2x (line doubling) scaling. Expose this functionality and exit early when an unsupported scaling configuration is encountered. This was tested with modetest's DRM plane test (from the libdrm te

[PATCH v2 3/4] Add virtio gpu driver.

2015-04-01 Thread Michael S. Tsirkin
On Wed, Apr 01, 2015 at 03:15:29PM +0200, Gerd Hoffmann wrote: > +int virtio_gpu_driver_unload(struct drm_device *dev) > +{ > + struct virtio_gpu_device *vgdev = dev->dev_private; > + > + vgdev->vqs_ready = false; Probably ok since flush below will sync all cpus, but this needs a comment I

[PATCH 1/3] drm/exynos: mixer: add 2x scaling to mixer_graph_buffer

2015-04-01 Thread Tobias Jakobi
Hello Gustavo, On 2015-03-27 14:11, Gustavo Padovan wrote: > If you can rebase this in on top of my series this would be really > good. like I said in my other mail, the series doesn't apply cleanly anymore, so I had to rebase your series. >> Then it would just be: >> static int mixer_setup_sc

[PATCH v2 4/4] Add virtio-vga bits.

2015-04-01 Thread Gerd Hoffmann
On Mi, 2015-04-01 at 15:26 +0200, Michael S. Tsirkin wrote: > On Wed, Apr 01, 2015 at 03:15:30PM +0200, Gerd Hoffmann wrote: > > +static void virtio_pci_kick_out_firmware_fb(struct pci_dev *pci_dev) > > +{ > > + struct apertures_struct *ap; > > + bool primary; > > + > > + ap = alloc_apertures

[PATCH v2 1/4] break kconfig dependency loop

2015-04-01 Thread Jani Nikula
On Wed, 01 Apr 2015, Gerd Hoffmann wrote: > After adding virtio-gpu I get this funky kconfig dependency loop. > > scripts/kconfig/conf --oldconfig Kconfig > drivers/video/fbdev/Kconfig:5:error: recursive dependency detected! > drivers/video/fbdev/Kconfig:5: symbol FB is selected by DRM_KMS_FB_HEL

Solaris & [PATCH libdrm 1/2] configure.ac: split -fvisibility and __attribute__((visibility)) checks

2015-04-01 Thread ran...@sibernet.com
Sorry, went to drafts and not to send... >>> - The struct drm_map/drmMapBufs/drmRmMap is part of the legacy drm >>> cruft for which, I would like to think, there are no more users. >>> >>> Obviously the latter can be confirmed by Randy and friends. >> >> I'm somewhat confused by this statement,

[PATCH v2 1/4] break kconfig dependency loop

2015-04-01 Thread John Hunter
regards Junwang Zhao Microprocessor Research and Develop Center Department of Computer Science &Technology Peking University Beijing, 100871, PRC -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150401/2294ad8f/attachment.html>

[PATCH v2 1/4] break kconfig dependency loop

2015-04-01 Thread Michael S. Tsirkin
On Wed, Apr 01, 2015 at 10:55:01PM +0800, John Hunter wrote: > Hi Gerd, > I've read the patches about the virtio-gpu, it's a nice design. > As far as I know, there are two other drivers used by qemu, CIRRUS and BOCHS. > I have a question about the relationship of these three drivers, is that the >

[PATCH libdrm v2 1/8] tests/hash: extract test out of xf86drmHash.c

2015-04-01 Thread Jan Vesely
to any person obtaining a > + * copy of this software and associated documentation files (the "Software"), > + * to deal in the Software without restriction, including without limitation > + * the rights to use, copy, modify, merge, publish, distribute, sublicense, > + * and/or sell copies of the Software, and to permit persons to whom the > + * Software is furnished to do so, subject to the following conditions: > + * > + * The above copyright notice and this permission notice (including the next > + * paragraph) shall be included in all copies or substantial portions of the > + * Software. > + * > + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR > + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, > + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL > + * PRECISION INSIGHT AND/OR ITS SUPPLIERS BE LIABLE FOR ANY CLAIM, DAMAGES OR > + * OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, > + * ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR > OTHER > + * DEALINGS IN THE SOFTWARE. > + * > + * Authors: Rickard E. (Rik) Faith > + */ > + > +#define HASH_SIZE 512 /* Good for about 100 entries */ > + /* If you change this value, you probably > + have to change the HashHash hashing > + function! */ > + > +typedef struct HashBucket { > +unsigned long key; > +void *value; > +struct HashBucket *next; > +} HashBucket, *HashBucketPtr; > + > +typedef struct HashTable { > +unsigned longmagic; > +unsigned longentries; > +unsigned longhits; /* At top of linked list */ > +unsigned longpartials; /* Not at top of linked list */ > +unsigned longmisses; /* Not in table */ > +HashBucketPtrbuckets[HASH_SIZE]; > +int p0; > +HashBucketPtrp1; > +} HashTable, *HashTablePtr; For the series: Reviewed-by: Jan Vesely -- Jan Vesely -- next part -- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: This is a digitally signed message part URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150401/0b96/attachment-0001.sig>

[PATCH] drm/exynos: fimd: check whether exynos_drm_crtc_create succeed or not

2015-04-01 Thread Gustavo Padovan
Hi Hyungwon, 2015-04-01 Hyungwon Hwang : > From the commit "drm/exynos: fix the execution order in FIMD > initialization" (598285bfdce46d7c47632a2ba4b980f60be4a677), the error > checking code is removed improperly. This patch fix the regression. > > Signed-off-by: Hyungwon Hwang > --- > driver

[PATCH 1/3] drm/exynos: fix typos in hdmi and mixer

2015-04-01 Thread Gustavo Padovan
Hi Tobias, 2015-04-01 Tobias Jakobi : > Use the correct spelling for 'progressive'. > > Signed-off-by: Tobias Jakobi > --- > drivers/gpu/drm/exynos/exynos_hdmi.c | 2 +- > drivers/gpu/drm/exynos/exynos_mixer.c | 2 +- > drivers/gpu/drm/exynos/regs-mixer.h | 2 +- > 3 files changed, 3 insert

[PATCH 2/3] drm/exynos: remove superfluous error messages

2015-04-01 Thread Gustavo Padovan
Hi Tobias, 2015-04-01 Tobias Jakobi : > The messages are redundant since 'check_fb_gem_memory_type' > already prints out exactly the same string when it fails. > > Signed-off-by: Tobias Jakobi > --- > drivers/gpu/drm/exynos/exynos_drm_fb.c | 10 +++--- > 1 file changed, 3 insertions(+), 7

[PATCH 3/3] drm/exynos: mixer: add 2x scaling to mixer_graph_buffer

2015-04-01 Thread Gustavo Padovan
Hi Tobias, 2015-04-01 Tobias Jakobi : > While the VP (video processor) supports arbitrary scaling > of its input, the mixer just supports a simple 2x (line > doubling) scaling. Expose this functionality and exit > early when an unsupported scaling configuration is > encountered. > > This was tes

[PATCH v2 1/4] break kconfig dependency loop

2015-04-01 Thread Gerd Hoffmann
On Mi, 2015-04-01 at 22:55 +0800, John Hunter wrote: > Hi Gerd, > I've read the patches about the virtio-gpu, it's a nice design. > As far as I know, there are two other drivers used by qemu, CIRRUS and > BOCHS. > I have a question about the relationship of these three drivers, is > that the virtio

[PATCH libdrm] freedreno: link against CLOCK_LIB

2015-04-01 Thread Emil Velikov
Required by clock_gettime() Cc: Rob Clark Signed-off-by: Emil Velikov --- freedreno/Makefile.am | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/freedreno/Makefile.am b/freedreno/Makefile.am index 0720867..9b7ec7d 100644 --- a/freedreno/Makefile.am +++ b/freedreno/Makefil

[PATCH libdrm] android: set the HAVE_VISIBILITY define

2015-04-01 Thread Emil Velikov
... in order to limit the exported symbols only to the required ones. Both compilers used with Android (GCC and LLVM) support this, so set it unconditionally. Cc: Chih-Wei Huang Signed-off-by: Emil Velikov --- Android.mk | 1 + 1 file changed, 1 insertion(+) diff --git a/Android.mk b/Android.m

[PATCH -v4 0/8] drm/exynos: clean up patches (preparing for atomic)

2015-04-01 Thread Gustavo Padovan
From: Gustavo Padovan Hi, Here goes some clean ups to the exynos drivers. The main clean ups is the presetting and zpos making the property immutable and the removal of *_win_data structures. v2 contains a extra patch to fix alpha setting for planes in fimd, so now fimd works fine even after th

[PATCH -v4 1/8] drm/exynos: fimd: fix alpha setting for XR24 pixel format

2015-04-01 Thread Gustavo Padovan
From: Gustavo Padovan XR24 planes were not shown properly, so now set the right registers to correctly enable displaying these planes. It also moves the alpha register settings to fimd_win_set_pixfmt() to keep all pixel format stuff together. v2: remove leftover var alpha Signed-off-by: Gustav

[PATCH -v4 2/8] drm/exynos: remove unused exynos_crtc->win_enable() callback

2015-04-01 Thread Gustavo Padovan
From: Gustavo Padovan None of the exynos crtc drivers implements win_enable() so remove it for better clarity of the code. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/exynos/exynos_drm_drv.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/exynos/exynos_drm_drv.h b/

[PATCH -v4 3/8] drm/exynos: remove struct *_win_data abstraction on planes

2015-04-01 Thread Gustavo Padovan
From: Gustavo Padovan struct {fimd,mixer,vidi}_win_data was just keeping the same data as struct exynos_drm_plane thus get ride of it and use exynos_drm_plane directly. It changes how planes are created and remove .win_mode_set() callback that was only filling all *_win_data structs. v2: check

[PATCH -v4 4/8] drm/exynos: preset zpos value for overlay planes

2015-04-01 Thread Gustavo Padovan
From: Gustavo Padovan Usually userspace don't want to have two overlay planes on the same zpos so this change assign a different zpos for each plane. Before this change a zpos of value zero was created for all planes so the userspace had to set up the zpos of every plane it wanted to use. Also a

[PATCH -v4 5/8] drm/exynos: make zpos property immutable

2015-04-01 Thread Gustavo Padovan
From: Gustavo Padovan We already set each plane zpos at init, after that changes to zpos are not expected. This patch turns zpos into a read-only property so now it is impossible to set zpos. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/exynos/exynos_drm_plane.c | 21 ++--

[PATCH -v4 6/8] drm/exynos: remove exynos_plane_destroy()

2015-04-01 Thread Gustavo Padovan
From: Gustavo Padovan The .destroy() callback for exynos can be replaced by drm_plane_cleanup(). The only extra operation on exynos_plane_destroy() was a call to exynos_plane_disable() but the plane is already disabled by a earlier call to drm_framebuffer_remove(). Signed-off-by: Gustavo Padovan

[PATCH -v4 7/8] drm/exynos: remove leftover functions declarations

2015-04-01 Thread Gustavo Padovan
From: Gustavo Padovan These functions were already removed by previous cleanup work, but these ones were left behind. Signed-off-by: Gustavo Padovan Acked-by: Joonyoung Shim --- drivers/gpu/drm/exynos/exynos_drm_crtc.h | 6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/gpu/drm/e

[PATCH -v4 8/8] drm/exynos: track vblank events on a per crtc basis

2015-04-01 Thread Gustavo Padovan
From: Mandeep Singh Baines The goal of the change is to make sure we send the vblank event on the current vblank. My hope is to fix any races that might be causing flicker. After this change I only see a flicker in the transition plymouth and X11. Simplified the code by tracking vblank events on

[PATCH 3/8] drm/exynos: remove struct *_win_data abstraction on planes

2015-04-01 Thread Gustavo Padovan
Hi Tobias, 2015-03-31 Tobias Jakobi : > Hello, > > I just wanted to point out that this doesn't apply to exynos-drm-fixes, which > was probably caused by Daniel's pitch patch here: > https://git.kernel.org/cgit/linux/kernel/git/daeinki/drm-exynos.git/commit/?h=exynos-drm-fixes&id=57aaaf373ede95

[PATCH libdrm 00/24] Annotate private symbols, drop drm_public macro

2015-04-01 Thread Emil Velikov
Hi all, As mentioned by Alan, some compilers do not like it when the symbol declaration and definition differ wrt their visibility attribute. Afaict there are three ways to handle this; - Add the drm_public macro into the public headers, and annotate the declarations. - Use version script to

[PATCH libdrm 01/24] configure.ac: split -fvisibility and __attribute__((visibility)) checks

2015-04-01 Thread Emil Velikov
The former does not imply the latter and vice-versa. One such example is the Sun compiler. v2: Add missing closing brakets. (Alan) Cc: Alan Coopersmith Cc: Thierry Reding Signed-off-by: Emil Velikov --- configure.ac | 13 + 1 file changed, 9 insertions(+), 4 deletions(-) diff --g

[PATCH libdrm 02/24] radeon: remove empty function declarations

2015-04-01 Thread Emil Velikov
Missing definition and unused since their introduction. Cc: Jerome Glisse Signed-off-by: Emil Velikov --- radeon/bof.h | 4 1 file changed, 4 deletions(-) diff --git a/radeon/bof.h b/radeon/bof.h index 014affb..cb829a1 100644 --- a/radeon/bof.h +++ b/radeon/bof.h @@ -51,10 +51,6 @@ typede

[PATCH libdrm 03/24] radeon: remove unused functions

2015-04-01 Thread Emil Velikov
Namely bof_load_file, bof_print, bof_object_get. Unused since their introduction according to git log. Cc: Jerome Glisse Signed-off-by: Emil Velikov --- radeon/bof.c | 53 - radeon/bof.h | 3 --- 2 files changed, 56 deletions(-) diff --git a

[PATCH libdrm 04/24] radeon: annotate bof_incref, bof_string functions as static

2015-04-01 Thread Emil Velikov
Used locally in bof.c. Cc: Jerome Glisse Signed-off-by: Emil Velikov --- radeon/bof.c | 96 ++-- radeon/bof.h | 3 -- 2 files changed, 48 insertions(+), 51 deletions(-) diff --git a/radeon/bof.c b/radeon/bof.c index 6f3760a..20901a0 1006

[PATCH libdrm 05/24] radeon: remove more unused functions

2015-04-01 Thread Emil Velikov
bof_array_{get,size} and bof_blob_{size,value}. All of which unused since their introduction with commit 78de69713d7(drm/radeon: add new cs command stream dumping facilities) Cc: Jerome Glisse Signed-off-by: Emil Velikov --- radeon/bof.c | 28 radeon/bof.h | 4

[PATCH libdrm 06/24] radeon: remove no-longer used static functions

2015-04-01 Thread Emil Velikov
The users of bof_print_rec and bof_read(bof_print and bof_load_file respectively) were removed with earlier commit. With the former two gone, two more functions become unused - bof_print_bof and bof_indent. Remove those as well. Signed-off-by: Emil Velikov --- radeon/bof.c | 110

[PATCH libdrm 07/24] radeon: annotate the private symbols

2015-04-01 Thread Emil Velikov
They are less and easier to track than the public ones. The macro drm_public will be going away by the end of the series. Cc: Maarten Lankhorst Cc: Michel Dänzer Signed-off-by: Emil Velikov --- radeon/bof.c | 22 +- radeon/bof.h | 23 ++- 2 files change

[PATCH libdrm 08/24] radeon: add symbols test

2015-04-01 Thread Emil Velikov
Will allow us to catch when the library exports more symbols than the ones in the public headers. Signed-off-by: Emil Velikov --- radeon/Makefile.am | 3 ++- radeon/radeon-symbol-check | 61 ++ 2 files changed, 63 insertions(+), 1 deletion(-)

[PATCH libdrm 09/24] freedreno: annotate the private symbols

2015-04-01 Thread Emil Velikov
They are less and easier to track than the public ones. The macro drm_public will be going away by the end of the series. Cc: Rob Clark Signed-off-by: Emil Velikov --- freedreno/freedreno_bo.c | 2 +- freedreno/freedreno_device.c | 2 +- freedreno/freedreno_priv.h | 4 ++--

[PATCH libdrm 10/24] freedreno: add symbols test

2015-04-01 Thread Emil Velikov
Signed-off-by: Emil Velikov --- freedreno/Makefile.am| 3 ++- freedreno/freedreno-symbol-check | 52 2 files changed, 54 insertions(+), 1 deletion(-) create mode 100755 freedreno/freedreno-symbol-check diff --git a/freedreno/Makefile.am b/fr

[PATCH libdrm 11/24] intel: remove the drm_mm* symbol workarounds

2015-04-01 Thread Emil Velikov
Added with commit 57b4c4c32d3(Move the renaming of mm.c symbols to symbol duplication/collision with ones that are available elsewhere. As the public/private symbols of libdrm are properly annotated neither one of the symbols will end up in the global name-space, thus should no longer be required.

[PATCH libdrm 12/24] intel: remove unused mmFindBlock

2015-04-01 Thread Emil Velikov
The function was never part of the public API and a release or so back was hidden from the global name-space (list of exported symbols). According to git log this function was never used internally. Signed-off-by: Emil Velikov --- intel/mm.c | 12 intel/mm.h | 7 --- 2 files c

[PATCH libdrm 13/24] intel: annotate the private symbols

2015-04-01 Thread Emil Velikov
They are less and easier to track than the public ones. The macro drm_public will be going away by the end of the series. Cc: Damien Lespiau Cc: Daniel Vetter Signed-off-by: Emil Velikov --- intel/mm.c | 17 +++-- intel/mm.h | 19 +-- 2 files changed, 24 insertions(

[PATCH libdrm 14/24] intel: add symbols test

2015-04-01 Thread Emil Velikov
Signed-off-by: Emil Velikov --- intel/Makefile.am| 6 ++-- intel/intel-symbol-check | 88 2 files changed, 92 insertions(+), 2 deletions(-) create mode 100755 intel/intel-symbol-check diff --git a/intel/Makefile.am b/intel/Makefile.am in

[PATCH libdrm 15/24] nouveau: annotate the private symbols

2015-04-01 Thread Emil Velikov
They are less and easier to track than the public ones. The macro drm_public will be going away by the end of the series. Cc: Maarten Lankhorst Cc: Ben Skeggs Signed-off-by: Emil Velikov --- nouveau/abi16.c | 14 +++--- nouveau/nouveau.c | 2 +- nouveau/private.h | 18 +-

[PATCH libdrm 16/24] nouveau: add symbols test

2015-04-01 Thread Emil Velikov
Signed-off-by: Emil Velikov --- nouveau/Makefile.am | 3 ++- nouveau/nouveau-symbol-check | 52 2 files changed, 54 insertions(+), 1 deletion(-) create mode 100755 nouveau/nouveau-symbol-check diff --git a/nouveau/Makefile.am b/nouveau/Make

[PATCH libdrm 17/24] libkms: annotate private symbols

2015-04-01 Thread Emil Velikov
Signed-off-by: Emil Velikov --- libkms/api.c | 3 +++ libkms/dumb.c | 2 +- libkms/exynos.c | 2 +- libkms/intel.c| 2 +- libkms/internal.h | 19 --- libkms/linux.c| 4 ++-- libkms/nouveau.c | 2 +- libkms/radeon.c | 2 +- libkms/vmwgfx.c | 2 +- 9

[PATCH libdrm 18/24] libkms: add symbols test

2015-04-01 Thread Emil Velikov
Signed-off-by: Emil Velikov --- libkms/Makefile.am | 3 ++- libkms/kms-symbol-check | 25 + 2 files changed, 27 insertions(+), 1 deletion(-) create mode 100755 libkms/kms-symbol-check diff --git a/libkms/Makefile.am b/libkms/Makefile.am index 4baf4fc..6c0ab7a 10064

[PATCH libdrm 19/24] exynos: add symbols test

2015-04-01 Thread Emil Velikov
Signed-off-by: Emil Velikov --- exynos/Makefile.am | 3 +++ exynos/exynos-symbol-check | 37 + 2 files changed, 40 insertions(+) create mode 100755 exynos/exynos-symbol-check diff --git a/exynos/Makefile.am b/exynos/Makefile.am index a9da0ff..0136a61

[PATCH libdrm 20/24] omap: add symbols test

2015-04-01 Thread Emil Velikov
Signed-off-by: Emil Velikov --- omap/Makefile.am | 3 +++ omap/omap-symbol-check | 35 +++ 2 files changed, 38 insertions(+) create mode 100755 omap/omap-symbol-check diff --git a/omap/Makefile.am b/omap/Makefile.am index d6f5298..b34fba6 100644 --- a/omap

[PATCH libdrm 21/24] tegra: add symbols test

2015-04-01 Thread Emil Velikov
Signed-off-by: Emil Velikov --- tegra/Makefile.am| 3 +++ tegra/tegra-symbol-check | 30 ++ 2 files changed, 33 insertions(+) create mode 100755 tegra/tegra-symbol-check diff --git a/tegra/Makefile.am b/tegra/Makefile.am index a647487..efa14f9 100644 --- a/t

[PATCH libdrm 22/24] drm: rename libdrm{,_macros}.h

2015-04-01 Thread Emil Velikov
Provide a more meaningful name, considering what it does. Signed-off-by: Emil Velikov --- Makefile.sources | 2 +- exynos/exynos_drm.c| 2 +- exynos/exynos_fimg2d.c | 2 +- freedreno/freedreno_priv.h | 2 +- intel/intel_bufmgr.c | 2 +- intel/intel_bufmgr_fake.c

[PATCH libdrm 23/24] drm: remove no longer needed VISIBILITY_CFLAGS

2015-04-01 Thread Emil Velikov
With earlier commits we've annotated the private symbols, thus we no longer require the -fvisibility=hidden CFLAGS. Cc: Ben Skeggs Cc: Damien Lespiau Cc: Maarten Lankhorst Cc: Michel Dänzer Cc: Rob Clark Cc: Thierry Reding Signed-off-by: Emil Velikov --- configure.ac | 15 ---

[PATCH libdrm 24/24] drm: remove drm_public macro

2015-04-01 Thread Emil Velikov
Some compilers (like the Oracle Studio), require that the function declaration must be annotated with the same visibility attribute as the definition. As annotating functions with drm_public is no longer required just remove the macro. Cc: Ben Skeggs Cc: Damien Lespiau Cc: Maarten Lankhorst Cc:

[PATCH] drm/udl: implement usb_driver suspend/resume.

2015-04-01 Thread Haixia Shi
The usb_driver suspend and resume function pointers must be populated to prevent forced unbinding of USB interface driver. See usb/core/driver.c: unbind_no_pm_drivers_interfaces(). Restore mode and damage the entire frame buffer upon resume. Signed-off-by: Haixia Shi Reviewed-by: Stéphane March

[PATCH libdrm 02/24] radeon: remove empty function declarations

2015-04-01 Thread Jerome Glisse
On Wed, Apr 01, 2015 at 05:15:13PM +0100, Emil Velikov wrote: > Missing definition and unused since their introduction. > > Cc: Jerome Glisse > Signed-off-by: Emil Velikov NAK I use all this in tools to debug lockup. Best course of action is to exclude bof.h from being distributed. My tools st

[PATCH libdrm 03/24] radeon: remove unused functions

2015-04-01 Thread Jerome Glisse
On Wed, Apr 01, 2015 at 05:15:14PM +0100, Emil Velikov wrote: > Namely bof_load_file, bof_print, bof_object_get. Unused since their > introduction according to git log. > > Cc: Jerome Glisse > Signed-off-by: Emil Velikov NAK I use all this in tools to debug lockup. Best course of action is to

[PATCH libdrm 04/24] radeon: annotate bof_incref, bof_string functions as static

2015-04-01 Thread Jerome Glisse
On Wed, Apr 01, 2015 at 05:15:15PM +0100, Emil Velikov wrote: > Used locally in bof.c. > > Cc: Jerome Glisse > Signed-off-by: Emil Velikov NAK I use all this in tools to debug lockup. Best course of action is to exclude bof.h from being distributed. My tools static link and i just point them t

[PATCH libdrm 04/24] radeon: annotate bof_incref, bof_string functions as static

2015-04-01 Thread Jerome Glisse
On Wed, Apr 01, 2015 at 05:15:15PM +0100, Emil Velikov wrote: > Used locally in bof.c. > > Cc: Jerome Glisse > Signed-off-by: Emil Velikov NAK I use all this in tools to debug lockup. Best course of action is to exclude bof.h from being distributed. My tools static link and i just point them

[PATCH libdrm 05/24] radeon: remove more unused functions

2015-04-01 Thread Jerome Glisse
On Wed, Apr 01, 2015 at 05:15:16PM +0100, Emil Velikov wrote: > bof_array_{get,size} and bof_blob_{size,value}. All of which unused > since their introduction with commit 78de69713d7(drm/radeon: add new cs > command stream dumping facilities) > > Cc: Jerome Glisse > Signed-off-by: Emil Velikov

[PATCH 1/5] drm/i915: Fix for DP CTS test 4.2.2.5 - I2C DEFER handling

2015-04-01 Thread Todd Previte
For test 4.2.2.5 to pass per the Link CTS Core 1.2 rev1.1 spec, the source device must attempt at least 7 times to read the EDID when it receives an I2C defer. The normal DRM code makes only 7 retries, regardless of whether or not the response is a native defer or an I2C defer. Test 4.2.2.5 fails s

[PATCH 1/5] drm/i915: Fix for DP CTS test 4.2.2.5 - I2C DEFER handling

2015-04-01 Thread Ville Syrjälä
On Wed, Apr 01, 2015 at 10:52:58AM -0700, Todd Previte wrote: > For test 4.2.2.5 to pass per the Link CTS Core 1.2 rev1.1 spec, the source > device must attempt at least 7 times to read the EDID when it receives an > I2C defer. The normal DRM code makes only 7 retries, regardless of whether > or no

[Bug 89734] GL_AMD_pinned_memory extension causing a kernel hardlock

2015-04-01 Thread bugzilla-dae...@freedesktop.org
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150401/9a749615/attachment.html>

[Bug 89734] GL_AMD_pinned_memory extension causing a kernel hardlock

2015-04-01 Thread bugzilla-dae...@freedesktop.org
is mail because: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150401/9a3865ca/attachment.html>

[Bug 89829] [bisected] radeonsi, commit 2cf48c creates artefacts in some applications (worst being Flash animations that are garbage)

2015-04-01 Thread bugzilla-dae...@freedesktop.org
are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150401/2757cfed/attachment.html>

[PULL] vmwgfx-next

2015-04-01 Thread Thomas Hellstrom
Dave, I see you haven't pulled this one yet. Please don't do that as there is a problem with fbdev and screen targets. I'll resend a pull request when the code is fixed. Thanks, Thomas On 03/11/2015 08:02 PM, Thomas Hellstrom wrote: > Dave, > > The first pull request for 4.1. Mainly Sinclair's

[Bug 89785] GPU Fault 147 and Ring Stalls and Tests Fail in Pillars of Eternity

2015-04-01 Thread bugzilla-dae...@freedesktop.org
attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150401/2fbef53d/attachment.html>

[PATCH libdrm 02/24] radeon: remove empty function declarations

2015-04-01 Thread Emil Velikov
On 1 April 2015 at 18:30, Jerome Glisse wrote: > On Wed, Apr 01, 2015 at 05:15:13PM +0100, Emil Velikov wrote: >> Missing definition and unused since their introduction. >> >> Cc: Jerome Glisse >> Signed-off-by: Emil Velikov > > NAK > > I use all this in tools to debug lockup. Best course of act

  1   2   >