dri-devel July 2012

dri-devel@lists.freedesktop.org

115 participants
688 discussions

[Bug 51870] r600_streamout=1 nothing change
by bugzilla-daemon＠freedesktop.org 09 Jul '12

09 Jul '12

https://bugs.freedesktop.org/show_bug.cgi?id=51870 --- Comment #2 from ErikAlesund <fallosistema(a)gmail.com> 2012-07-09 11:13:09 UTC --- (In reply to comment #1) > Those environment variables aren't required anymore. GLSL 1.3 and streamout are > enabled by default. The only thing missing is MSAA and there is no way to > enable it at the moment. Closing. If would be nice to avoid confusion remove this: 10 Requires env var R600_STREAMOUT=1 in the radeon's feature page: http://www.x.org/wiki/RadeonFeature/ Thanks -- Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug.

1 0

[PATCH, RFC] i.MX DRM support
by Sascha Hauer 09 Jul '12

09 Jul '12

Hi All, The following is the state-of-the-art i.MX IPU (Image Processing Unit) DRM support. This code is around for quite some time now and has been posted several times with different APIs, first with plain old framebuffer support, now DRM, first platform device binding, now devicetree. Unfortunately there's quite much code needed to get something useful out of the IPU, so these patches haven't received a lot of attention from people not involved in i.MX. I think we have now come to a point where this code needs more public exposure and where it's easier to talk in incremental changes instead of blobs. Therefore I request this to go to staging for some cycles. This would allow us to have something in mainline as a base for further discussion while still being able to change bigger amounts of the driver without breaking officially supported features. What do you think about this? The IPU driver generally works on i.MX51, i.MX53 and i.MX6. The latter has two IPUs, the driver is able to handle them both. This series only contains the IPU driver. There are some additional glue code patches necessary. Also necessary are two drm CMA/GEM helper patches currently waiting for review on dri-devel. For testing the driver please use this branch: git://git.pengutronix.de/git/imx/linux-2.6.git work/gpu/imx-drm-ipu-complete The branch contains board support for the i.MX51 babbage (both digital and analog DVI outputs are supported) and i.MX6q armadillo2 (HDMI). Compile with imx_v6_v7_defconfig. For discussion of the devicetree binding please refer to the seperate mail I sent. As usual, all comments/suggestions welcome Sascha The following changes since commit cfaf025112d3856637ff34a767ef785ef5cf2ca9: Linux 3.5-rc2 (2012-06-08 18:40:09 -0700) are available in the git repository at: git://git.pengutronix.de/git/imx/linux-2.6.git work/gpu/imx-drm-ipu for you to fetch changes up to 6c25e018285a93053daa69786faa8b3fe43597be: DRM i.MX: Add devicetree binding documentation (2012-06-14 14:30:52 +0200) ---------------------------------------------------------------- Philipp Zabel (1): DRM i.MX: Add devicetree binding documentation Sascha Hauer (5): DRM: Add i.MX drm core support DRM i.MX: Add parallel display support DRM i.MX: Add LCDC support DRM: add i.MX IPUv3 base driver DRM: Add i.MX IPUv3 crtc support .../devicetree/bindings/drm/fsl-imx-drm.txt | 126 +++ drivers/gpu/drm/Kconfig | 2 + drivers/gpu/drm/Makefile | 1 + drivers/gpu/drm/imx/Kconfig | 40 + drivers/gpu/drm/imx/Makefile | 10 + drivers/gpu/drm/imx/imx-drm-core.c | 891 ++++++++++++++++ drivers/gpu/drm/imx/imx-drm.h | 58 + drivers/gpu/drm/imx/imx-fb.c | 47 + drivers/gpu/drm/imx/imx-fbdev.c | 65 ++ drivers/gpu/drm/imx/imx-ipuv3-crtc.c | 579 ++++++++++ drivers/gpu/drm/imx/imx-lcdc-crtc.c | 523 +++++++++ drivers/gpu/drm/imx/imx-parallel-display.c | 255 +++++ drivers/gpu/drm/imx/ipu-v3/Makefile | 3 + drivers/gpu/drm/imx/ipu-v3/ipu-common.c | 1111 ++++++++++++++++++++ drivers/gpu/drm/imx/ipu-v3/ipu-dc.c | 384 +++++++ drivers/gpu/drm/imx/ipu-v3/ipu-di.c | 695 ++++++++++++ drivers/gpu/drm/imx/ipu-v3/ipu-dmfc.c | 410 ++++++++ drivers/gpu/drm/imx/ipu-v3/ipu-dp.c | 336 ++++++ drivers/gpu/drm/imx/ipu-v3/ipu-prv.h | 211 ++++ include/drm/imx-ipu-v3.h | 320 ++++++ 20 files changed, 6067 insertions(+) create mode 100644 Documentation/devicetree/bindings/drm/fsl-imx-drm.txt create mode 100644 drivers/gpu/drm/imx/Kconfig create mode 100644 drivers/gpu/drm/imx/Makefile create mode 100644 drivers/gpu/drm/imx/imx-drm-core.c create mode 100644 drivers/gpu/drm/imx/imx-drm.h create mode 100644 drivers/gpu/drm/imx/imx-fb.c create mode 100644 drivers/gpu/drm/imx/imx-fbdev.c create mode 100644 drivers/gpu/drm/imx/imx-ipuv3-crtc.c create mode 100644 drivers/gpu/drm/imx/imx-lcdc-crtc.c create mode 100644 drivers/gpu/drm/imx/imx-parallel-display.c create mode 100644 drivers/gpu/drm/imx/ipu-v3/Makefile create mode 100644 drivers/gpu/drm/imx/ipu-v3/ipu-common.c create mode 100644 drivers/gpu/drm/imx/ipu-v3/ipu-dc.c create mode 100644 drivers/gpu/drm/imx/ipu-v3/ipu-di.c create mode 100644 drivers/gpu/drm/imx/ipu-v3/ipu-dmfc.c create mode 100644 drivers/gpu/drm/imx/ipu-v3/ipu-dp.c create mode 100644 drivers/gpu/drm/imx/ipu-v3/ipu-prv.h create mode 100644 include/drm/imx-ipu-v3.h

6 20

[Bug 42984] scrabled video with nouveau drivers on mac mini (320m)
by bugzilla-daemon＠bugzilla.kernel.org 09 Jul '12

09 Jul '12

https://bugzilla.kernel.org/show_bug.cgi?id=42984 m.b.lankhorst(a)gmail.com <m.b.lankhorst(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |m.b.lankhorst(a)gmail.com --- Comment #15 from m.b.lankhorst(a)gmail.com <m.b.lankhorst(a)gmail.com> 2012-07-09 08:33:00 --- This bug is fixed with commit a6a17859f1b and can be closed. It's backported to 3.2 stable and 3.4 with confirmation on 3.2.27 -- Configure bugmail: https://bugzilla.kernel.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching the assignee of the bug.

1 0

[PATCH 0/2] fix memory deallocation and exception
by Inki Dae 09 Jul '12

09 Jul '12

this patch corrects to deallocate the pages allocated by alloc_page() and fixes exception to page allocation failure. Thanks. Inki Dae (2): drm/exynos: use __free_page() to deallocate memory drm/exynos: fixed exception to page allocation failure drivers/gpu/drm/exynos/exynos_drm_gem.c | 16 ++++------------ 1 files changed, 4 insertions(+), 12 deletions(-) -- 1.7.4.1

1 4

question about drivers/gpu/drm/gma500/oaktrail_lvds.c
by Julia Lawall 09 Jul '12

09 Jul '12

In the function oaktrail_lvds_mode_set, I don't think that the following code makes any sense: /* Find the connector we're trying to set up */ list_for_each_entry(connector, &mode_config->connector_list, head) { if (!connector->encoder || connector->encoder->crtc != crtc) continue; } if (!connector) { DRM_ERROR("Couldn't find connector when setting mode"); return; } drm_connector_property_get_value( connector, dev->mode_config.scaling_mode_property, &v); The initial loop is a no-op, because it always continues. The test !connector can never be true, because at the end of a list_for_each_entry connector points to the list head, and calling drm_connector_property_get_value on the list head probably does not make sense. julia

3 4

[git pull] drm/exynos: updated exynos-drm-fixes
by Inki Dae 09 Jul '12

09 Jul '12

Hi Dave, Please pull from git://git.infradead.org/users/kmpark/linux-samsung exynos-drm-fixes these patch sets below had already been posted several weeks ago for review and include some fixes and exceptions. thes patch sets are based on git repository below: git://people.freedesktop.org/~airlied/linux.git drm-fixes commit-id: 9f846a16d213523fbe6daea17e20df6b8ac5a1e5 Thanks. Cooper Yuan (1): drm/exynos: fix buffer pitch calculation Inki Dae (11): drm/exynos: removed unnecessary declaration. drm/exynos: set edid fake data only for test. drm/exynos: check if raw edid data is fake or not for test drm/exynos: fixed edid data setting at vidi connection request drm/exynos: fixed build warning. drm/exynos: use alloc_page() to allocate pages. drm/exynos: set buffer type from exporter. drm/exynos: do not release memory region from exporter. drm/exynos: removed unnecessary variable drm/exynos: fixed a comment to gem size. drm/exynos: add packed_size not aligned in page unit. Sachin Kamat (4): drm/exynos: Add missing static storage class specifier drm/exynos: Use devm_* functions in exynos_drm_fimd.c drm/exynos: Use devm_* functions in exynos_hdmi.c drm/exynos: Use devm_* functions in exynos_mixer.c Subash Patel (2): drm/exynos: return NULL if exynos_pages_to_sg fails drm/exynos: check for null in return value of dma_buf_map_attachment() drivers/gpu/drm/exynos/exynos_drm_connector.c | 3 +- drivers/gpu/drm/exynos/exynos_drm_core.c | 5 -- drivers/gpu/drm/exynos/exynos_drm_dmabuf.c | 33 ++++++++++++---- drivers/gpu/drm/exynos/exynos_drm_fimd.c | 40 ++++--------------- drivers/gpu/drm/exynos/exynos_drm_gem.c | 28 +++++++------ drivers/gpu/drm/exynos/exynos_drm_gem.h | 6 ++- drivers/gpu/drm/exynos/exynos_drm_vidi.c | 53 +++++++++++++++++++------ drivers/gpu/drm/exynos/exynos_hdmi.c | 36 +++------------- drivers/gpu/drm/exynos/exynos_mixer.c | 48 +++++++---------------- 9 files changed, 118 insertions(+), 134 deletions(-) -- 1.7.4.1

1 0

[PATCH 1/3] drivers/gpu/drm/gma500/mdfld_intel_display.c: remove invalid reference to list iterator variable
by Julia Lawall 08 Jul '12

08 Jul '12

1 2

[PATCH 4/7] drivers/gpu/drm: remove invalid reference to list iterator variable
by Julia Lawall 08 Jul '12

08 Jul '12

From: Julia Lawall <Julia.Lawall(a)lip6.fr> If list_for_each_entry, etc complete a traversal of the list, the iterator variable ends up pointing to an address at an offset from the list head, and not a meaningful structure. Thus this value should not be used after the end of the iterator. In the first two cases, a a break is added after the switch; the break in the switch would jump out of the switch, but not out of the complete iteration. In the first case, the null test on connector is removed, because the iterator variable of list_for_each_entry is never null. In the third case, entry->base.crtc.fb is not a meaningful value. What seems to be intended is crtc->fb, which gets the information from the last element of the list. These problems were found using Coccinelle (http://coccinelle.lip6.fr/). Signed-off-by: Julia Lawall <Julia.Lawall(a)lip6.fr> --- drivers/gpu/drm/gma500/mdfld_intel_display.c | 4 +--- drivers/gpu/drm/gma500/oaktrail_crtc.c | 1 + drivers/gpu/drm/vmwgfx/vmwgfx_ldu.c | 2 +- 3 files changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/gma500/mdfld_intel_display.c b/drivers/gpu/drm/gma500/mdfld_intel_display.c index 3f3cd61..fc1ced0 100644 --- a/drivers/gpu/drm/gma500/mdfld_intel_display.c +++ b/drivers/gpu/drm/gma500/mdfld_intel_display.c @@ -755,9 +755,6 @@ static int mdfld_crtc_mode_set(struct drm_crtc *crtc, sizeof(struct drm_display_mode)); list_for_each_entry(connector, &mode_config->connector_list, head) { - if (!connector) - continue; - encoder = connector->encoder; if (!encoder) @@ -779,6 +776,7 @@ static int mdfld_crtc_mode_set(struct drm_crtc *crtc, is_hdmi = true; break; } + break; } /* Disable the VGA plane that we never use */ diff --git a/drivers/gpu/drm/gma500/oaktrail_crtc.c b/drivers/gpu/drm/gma500/oaktrail_crtc.c index f821c83..4b2172e 100644 --- a/drivers/gpu/drm/gma500/oaktrail_crtc.c +++ b/drivers/gpu/drm/gma500/oaktrail_crtc.c @@ -329,6 +329,7 @@ static int oaktrail_crtc_mode_set(struct drm_crtc *crtc, is_mipi = true; break; } + break; } /* Disable the VGA plane that we never use */ diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_ldu.c b/drivers/gpu/drm/vmwgfx/vmwgfx_ldu.c index 070fb23..634611a 100644 --- a/drivers/gpu/drm/vmwgfx/vmwgfx_ldu.c +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_ldu.c @@ -93,7 +93,7 @@ static int vmw_ldu_commit_list(struct vmw_private *dev_priv) if (crtc == NULL) return 0; - fb = entry->base.crtc.fb; + fb = crtc->fb; return vmw_kms_write_svga(dev_priv, w, h, fb->pitches[0], fb->bits_per_pixel, fb->depth);

3 2

[Bug 42162] Display not enabled on resume (HP DV6)
by bugzilla-daemon＠freedesktop.org 08 Jul '12

08 Jul '12

https://bugs.freedesktop.org/show_bug.cgi?id=42162 --- Comment #7 from Johan Brannlund <freedesktop-bugs(a)nullinfinity.org> 2012-07-08 08:31:56 PDT --- This is still an issue with kernel 3.5-rc4. -- Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug.

1 0

Tegra DRM device tree bindings
by Thierry Reding 08 Jul '12

08 Jul '12

Hi, while I haven't got much time to work on the actual code right now, I think it might still be useful if we could get the device tree binding to a point where everybody is happy with it. That'll also save me some time once I get to writing the code because I won't have to redo it over again. =) So here's the current proposal: host1x { compatible = "nvidia,tegra20-host1x", "simple-bus"; reg = <0x50000000 0x00024000>; interrupts = <0 64 0x04 /* cop syncpt */ 0 65 0x04 /* mpcore syncpt */ 0 66 0x04 /* cop general */ 0 67 0x04>; /* mpcore general */ #address-cells = <1>; #size-cells = <1>; ranges = <0x54000000 0x54000000 0x04000000>; status = "disabled"; gart = <&gart>; /* video-encoding/decoding */ mpe { reg = <0x54040000 0x00040000>; interrupts = <0 68 0x04>; status = "disabled"; }; /* video input */ vi { reg = <0x54080000 0x00040000>; interrupts = <0 69 0x04>; status = "disabled"; }; /* EPP */ epp { reg = <0x540c0000 0x00040000>; interrupts = <0 70 0x04>; status = "disabled"; }; /* ISP */ isp { reg = <0x54100000 0x00040000>; interrupts = <0 71 0x04>; status = "disabled"; }; /* 2D engine */ gr2d { reg = <0x54140000 0x00040000>; interrupts = <0 72 0x04>; status = "disabled"; }; /* 3D engine */ gr3d { reg = <0x54180000 0x00040000>; status = "disabled"; }; /* display controllers */ dc1: dc@54200000 { compatible = "nvidia,tegra20-dc"; reg = <0x54200000 0x00040000>; interrupts = <0 73 0x04>; status = "disabled"; }; dc2: dc@54240000 { compatible = "nvidia,tegra20-dc"; reg = <0x54240000 0x00040000>; interrupts = <0 74 0x04>; status = "disabled"; }; /* outputs */ rgb { compatible = "nvidia,tegra20-rgb"; status = "disabled"; }; hdmi { compatible = "nvidia,tegra20-hdmi"; reg = <0x54280000 0x00040000>; interrupts = <0 75 0x04>; status = "disabled"; }; tvo { compatible = "nvidia,tegra20-tvo"; reg = <0x542c0000 0x00040000>; interrupts = <0 76 0x04>; status = "disabled"; }; dsi { compatible = "nvidia,tegra20-dsi"; reg = <0x54300000 0x00040000>; status = "disabled"; }; }; This really isn't anything new but merely brings the Tegra DRM binding in sync with other devices in tegra20.dtsi (disable devices by default, leave out unit addresses for unique nodes). The only actual change is that host1x clients are now children of the host1x node. The children are instantiated by the initial call to of_platform_populate() since the host1x node is also marked compatible with "simple-bus". An alternative would be to call of_platform_populate() from the host1x driver. This has the advantage that it could integrate better with the host1x bus implementation that Terje is working on, but it also needs additional code to tear down the devices when the host1x driver is unloaded because a module reload would try to create duplicate devices otherwise. The rgb node is something that I don't quite know how to handle yet. Since it is really part of the display controller and uses its register space, it isn't quite correct to represent it as a separate device. But we will need a separate node to make it available as a connector, which will become more obvious below. Perhaps the ranges property can also be used to remap the reg properties of the child nodes so that they can be specified as an offset into the host1x aperture instead of an address in the CPU address space. But that's just a minor issue because the OF code should be able to handle it transparently. Board DTS files could then extend this with board-specific requirements and connectors. The following describes the Medcom Wide: host1x { carveout = <0x0e000000 0x02000000>; display-controllers = <&dc1>; status = "okay"; dc@54200000 { status = "okay"; }; lvds: rgb { status = "okay"; }; }; connectors { #address-cells = <1>; #size-cells = <0>; connector@0 { reg = <0>; edid = /incbin/("tegra-medcom.edid"); output = <&lvds>; }; }; Basically this turns on the first display controller and the RGB/LVDS output and hooks up a static EDID block with the LVDS output. There is also a carveout property which might be a better replacement for the "crippled" GART on Tegra20. Alternatively the CMA might work just as well instead. The Plutux can be described like this: host1x { carveout = <0x0e000000 0x02000000>; display-controllers = <&dc1 &dc2>; status = "okay"; dc@54200000 { status = "okay"; }; hdmi: hdmi { vdd-supply = <&ldo7_reg>; pll-supply = <&ldo8_reg>; status = "okay"; }; }; connectors { #address-cells = <1>; #size-cells = <0>; connector@0 { reg = <0>; output = <&hdmi>; ddc = <&ddc>; hpd-gpio = <&gpio 111 0>; /* PN7 */ }; }; Which is similar to the Medcom Wide, only it uses the HDMI output and hooks it up with one of the Tegra's I2C controllers to do EDID probing via DDC. Additionally it can detect whether an HDMI cable is connected using the GPIO specified by the hpd-gpio property. Thierry

7 68

← Newer
1
...
62
63
64
65
66
67
68
69
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

dri-devel July 2012