Re: [PATCH] staging: drm/omap: remove reclaim_buffers callback

2012-08-01 Thread Paul Menzel
= { > .irq_postinstall = dev_irq_postinstall, > .irq_uninstall = dev_irq_uninstall, > .irq_handler = dev_irq_handler, > - .reclaim_buffers = drm_core_reclaim_buffers, > #ifdef CONFIG_DEBUG_FS > .debugfs_init = omap_debugfs_in

Re: [PATCH 1/3] Replace i2f() in r600_blit.c with an optimized version.

2012-08-06 Thread Paul Menzel
Dear Steven, thank you for your patches. Am Montag, den 06.08.2012, 11:14 -0700 schrieb Steven Fuerst: > Replace i2f() in r600_blit.c with an optimized version. > > We use __fls() to find the most significant bit. Using that, the > loop can be avoided. A second trick is to use the mod(32) > b

Re: [PATCH 1/2] radeon: force 2D tiling for MSAA surfaces

2012-08-09 Thread Paul Menzel
Dear Marek, Am Donnerstag, den 09.08.2012, 16:37 +0200 schrieb Marek Olšák: > --- > radeon/radeon_surface.c | 24 ++-- > 1 file changed, 22 insertions(+), 2 deletions(-) […] do you have some numbers indicating an improvement with this change? On what hardware did you test

Re: [PATCH] drm: EDID quirk improvements

2012-08-11 Thread Paul Menzel
Not all users have access to the Linux source tree, so maybe a small overview is still needed? Or even an URL? > > module_param_named(debug, drm_debug, int, 0600); > module_param_named(vblankoffdelay, drm_vblank_offdelay, int, 0600); > module_param_named(timestamp_precision_usec, drm_

Kconfig: warning: (DRM_RADEON_KMS && DRM_I915 && …) selects BACKLIGHT_CLASS_DEVICE which has unmet direct dependencies (HAS_IOMEM && BACKLIGHT_LCD_SUPPORT)

2012-08-11 Thread Paul Menzel
Dear Linux folks, where should I report the following warning warning: (DRM_RADEON_KMS && DRM_I915 && STUB_POULSBO && FB_BACKLIGHT && USB_APPLEDISPLAY && FB_OLPC_DCON && ASUS_LAPTOP && SONY_LAPTOP && THINKPAD_ACPI && EEEPC_LAPTOP && ACPI_CMPC && SAMSUNG_Q10 && APPLE_GMUX) selects BACK

Re: [PATCH v4 1/1] video: drm: exynos: Adds display-timing node parsing using video helper function

2013-02-05 Thread Paul Menzel
Dear Vikas, thank you for the patch. Please send a fifth iteration with the following changes to the commit message. Am Dienstag, den 05.02.2013, 11:02 +0530 schrieb Vikas Sajjan: The summary should not implicitly assume »patch« written before it. So do not add third person s to »Add«.

Re: 答复: [PATCH]Siliconmotion initial patch

2013-02-06 Thread Paul Menzel
Dear Aaron, thank you for your patch. It is much appreciated. Here are some more comments about formal stuff in addition to what Daniel replied. Am Dienstag, den 05.02.2013, 15:51 +0800 schrieb Aaron.Chen 陈俊杰: […] > diff --git a/drivers/video/lynxfb/lynx_drv.c b/drivers/video/lynxfb/lynx_drv.c

Re: [PATCH v5 1/1] video: drm: exynos: Add display-timing node parsing using video helper function

2013-02-06 Thread Paul Menzel
Am Mittwoch, den 06.02.2013, 09:54 +0530 schrieb Vikas Sajjan: > Add support for parsing the display-timing node using video helper > function. > > The DT node parsing and pinctrl selection is done only if 'dev.of_node' > exists and the NON-DT logic is still maintained under the 'else' part. Last

Re: [PATCH] drm/mgag200: Cleanup: Remove pointless call to drm_fb_get_bpp_depth

2013-02-26 Thread Paul Menzel
Dear Christopher, thank you for your patches. Not sure if you should CC some maintainer (or if you did already). Am Dienstag, den 26.02.2013, 10:53 -0500 schrieb Christopher Harvey: 1. I guess you can remove Cleanup from the commit summary. 2. Why is it »pointless«? I guess a compiler warning

Re: [PATCH] drm/mgag200: Cleanup: Pass driver specific mga_device in driver functions

2013-02-26 Thread Paul Menzel
Am Dienstag, den 26.02.2013, 10:54 -0500 schrieb Christopher Harvey: > Signed-off-by: Christopher Harvey > --- > drivers/gpu/drm/mgag200/mgag200_mode.c | 7 +++ > 1 file changed, 3 insertions(+), 4 deletions(-) […] Acked-by: Paul Menzel Thanks, Paul signature.asc Descript

Re: [PATCH] drm/mgag200: Cleanup: Remove extra variable assigns

2013-02-26 Thread Paul Menzel
= (void *)&mga_mode_funcs; > - dev->mode_config.min_width = 0; > - dev->mode_config.min_height = 0; > dev->mode_config.preferred_depth = 24; > dev->mode_config.prefer_shadow = 1; Acked-by: Paul Menzel Thanks, Paul signature.asc Description: Th

Re: [PATCH] drm/mgag200: Reject modes that are too big for VRAM

2013-02-26 Thread Paul Menzel
Am Dienstag, den 26.02.2013, 10:55 -0500 schrieb Christopher Harvey: > A monitor or a user could request a resolution greater than the > available VRAM for the backing framebuffer. This change checks the > required framebuffer size against the max VRAM size and rejects modes > if they are too big.

Re: [PATCH V2] drm/edid: kernel-doc minimal cleanup

2013-03-02 Thread Paul Menzel
Am Freitag, den 01.03.2013, 08:00 -0600 schrieb Nishanth Menon: > Some basic cleanups for kernel-doc errors or missing documentation > parameters. Nishanth, thanks for doing that! > Warnings generated by ./scripts/kernel-doc drivers/gpu/drm/drm_edid.c >Kerr > > Warning(drivers/gpu/drm/drm_edid.c

Re: [PATCH] drm/radeon: don't check mipmap alignment if MIP_ADDRESS is FMASK

2013-03-04 Thread Paul Menzel
Am Montag, den 04.03.2013, 11:13 -0500 schrieb Alex Deucher: > On Fri, Mar 1, 2013 at 7:40 AM, Marek Olšák wrote: > > Signed-off-by: Marek Olšák > > Added to my -fixes queue. Too few information in my opinion as to why this change was made. Please be strict with that. Thanks, Paul signatur

Re: [PATCH V2] drm/exynos: fimd: calculate the correct address offset

2013-03-06 Thread Paul Menzel
Dear Leela, thank you for your patch. Am Mittwoch, den 06.03.2013, 00:20 -0500 schrieb Leela Krishna Amudala: > Calculate the correct address offset values for alpha and color key > control registers and clear size control register for window 0 1. The *and* implies this should be split up into

Re: Wrong vsync offset calculation in drm_edid.c

2013-03-08 Thread Paul Menzel
Dear Peter, thank you for your patch. It has some formal issues though. Am Donnerstag, den 07.03.2013, 16:41 +0100 schrieb Peter Blum: 1. Your subject line does not contain the tag [PATCH]. 2. You should CC the maintainers. > This patch is a bug fix for the file drm_edid.c of the kernel 3.8.

Re: [PATCH] DRI2: don't advertise GLX_INTEL_swap_event if it can't

2013-03-13 Thread Paul Menzel
Dear Rob, Am Dienstag, den 12.03.2013, 19:44 -0400 schrieb Rob Clark: »it« sounds strange in commit summary. > If ddx does not support swap, don't advertise it. So how is `dri2BindExtensions` changed. Some things passed beforehand are already available in `struct dri2_screen *psc`? Are bugs

Re: [PATCH 1/2] drm/radeon/benchmark: make sure bo blit copy exists before using it

2013-03-14 Thread Paul Menzel
Dear Alex, Am Mittwoch, den 13.03.2013, 12:38 -0400 schrieb alexdeuc...@gmail.com: > From: Alex Deucher > > Fixes a segfault on asics without a blit callback. so as a result this is not benchmarked for these ASICS, right? > Fixes: > https://bugs.freedesktop.org/show_bug.cgi?id=62239 > > Sign

Re: [PATCH 2/2] drm/radeon/benchmark: allow same domains for dma copy

2013-03-14 Thread Paul Menzel
Dear Alex, Am Mittwoch, den 13.03.2013, 12:38 -0400 schrieb alexdeuc...@gmail.com: > From: Alex Deucher > > Remove old comment and allow benchmarking moves within the > same memory domain for both dma and blit methods. > > Signed-off-by: Alex Deucher > --- > drivers/gpu/drm/radeon/radeon_benc

Re: [PATCH] drm/radeon: UVD support for RV710-SI

2013-04-04 Thread Paul Menzel
Dear Christian, dear AMD developers, Am Mittwoch, den 03.04.2013, 01:18 +0200 schrieb Christian König: > the following patchset implements the kernel side of UVD support for the > radeon hardware generations RV710-SI. thank you very much for getting these patches out! > The R6xx and RS780/RS8

Re: [PATCH 1/1] drm: Add 1600x1200 (UXGA) screen resolution to the built-in EDIDs

2013-04-06 Thread Paul Menzel
drivers/gpu/drm/drm_edid_load.c | 21 ++- > 3 files changed, 70 insertions(+), 7 deletions(-) Acked-by: Paul Menzel Thanks, Paul signature.asc Description: This is a digitally signed message part ___ dri-devel mailing list dri-devel@lists.freedeskto

Re: [PATCH] drivers/gpu/drm/nouveau: remove erroneous semicolon

2013-04-07 Thread Paul Menzel
a); > + flip->chan->data) > return true; > usleep_range(1, 2); > return false; Acked-by: Paul Menzel Thanks, Paul signature.asc Description: This is a digitally signed message part ___

Re: [RFC][PATCH] drm: add helper extracting SADs from EDID

2013-04-07 Thread Paul Menzel
Am Sonntag, den 07.04.2013, 12:52 +0200 schrieb Rafał Miłecki: > Some devices (ATI/AMD cards) don't want passing ELD struct to the want to pass > hardware but just require filling specific registers and then at end: »then *the* hardware« (I think) > hardware/firmware does the rest. In such a c

Re: [RFC][PATCH] drm: add helper extracting SADs from EDID

2013-04-07 Thread Paul Menzel
Am Sonntag, den 07.04.2013, 14:11 +0200 schrieb Rafał Miłecki: > Thanks for comments! > > 2013/4/7 Paul Menzel : > > Am Sonntag, den 07.04.2013, 12:52 +0200 schrieb Rafał Miłecki: > >> +struct cea_sad *drm_edid_to_sad(struct edid *edid) > >> +{ > >> +

Re: [PATCH 1/2] radeon: update radeon_drm.h to kernel last API additions

2013-04-10 Thread Paul Menzel
Am Montag, den 08.04.2013, 13:37 -0400 schrieb j.gli...@gmail.com: > From: Jerome Glisse Could you add the commit adding these definitions to the commit message, please? > Signed-off-by: Jerome Glisse > --- > include/drm/radeon_drm.h | 61 > >

OT: GSoC project idea: Replacement of Video BIOS for native graphics initialization in coreboot

2013-04-11 Thread Paul Menzel
(Sent to dri-devel and intel-gfx lists.) Dear Linux graphics folks, coreboot has been accepted to participate in Google Summer of Code 2013 [1]. coreboot is a Free Software project aimed at replacing the proprietary BIOS (firmware) found in most computers. Please find more information about GS

Re: [PATCH 2/6] drm/radeon: remove HDMI interrupts on Evergreen

2013-04-14 Thread Paul Menzel
Am Sonntag, den 14.04.2013, 01:26 +0200 schrieb Rafał Miłecki: > We need interrupts on format change for R6xx only, where hardware seems > to be somehow bugged and requires setting audio info manually. How should this be tested? > Signed-off-by: Rafał Miłecki > --- > drivers/gpu/drm/radeon/ever

Re: [PATCH 3/6] drm/radeon: add some HDMI comments

2013-04-14 Thread Paul Menzel
Am Sonntag, den 14.04.2013, 01:26 +0200 schrieb Rafał Miłecki: Maybe for a more descriptive summary: drm/radeon: Add some HDMI (audio) comments about fglrx’ reg reads > Signed-off-by: Rafał Miłecki > --- > drivers/gpu/drm/radeon/evergreen_hdmi.c | 14 ++ > drivers/gpu/drm/radeon/

Re: [PATCH 4/6] drm/radeon/evergreen: setup HDMI before enabling it

2013-04-14 Thread Paul Menzel
Am Sonntag, den 14.04.2013, 01:26 +0200 schrieb Rafał Miłecki: > Closed source driver fglrx seems to enable infoframes and audio packets > at the end, which makes sense, do the same. Any functionality change? Does not sound like it, but being unambiguous is better. > Signed-off-by: Rafał Miłecki

Re: [PATCH 5/6] drm/radeon/evergreen: reorder HDMI setup

2013-04-14 Thread Paul Menzel
Am Sonntag, den 14.04.2013, 01:26 +0200 schrieb Rafał Miłecki: Maybe use the following for a more descriptive summary: drm/radeon/evergreen: Set up audio and ACR packets after basic init > Driver fglrx setups audio and ACR packets after basic initialization, > which sounds sane, do the same. Co

Re: [PATCH] mgag200 code cleanup patches

2013-05-06 Thread Paul Menzel
Dear Christopher, Am Montag, den 29.04.2013, 14:37 -0400 schrieb Christopher Harvey: > I submitted these a while ago, but I think they got lost in the > mailing list. Just wanted to make sure they get a shot at the merge > window. as you resend the patches you could have easily amended the commi

Re: [PATCH] drm: refactor call to request_module

2013-05-08 Thread Paul Menzel
or are there any problems with stack sizes on some systems? (This patch would be good alone for decreasing the number of code lines. ;-)) > Signed-off-by: Kees Cook > --- > drivers/gpu/drm/drm_encoder_slave.c |6 +- > 1 file changed, 1 insertion(+), 5 deletions(-) Ack

Re: [PATCH] drm/radeon: fix UPLL_REF_DIV_MASK definition

2013-05-08 Thread Paul Menzel
Dear Christian, Am Montag, den 29.04.2013, 10:20 +0200 schrieb Christian König: > From: Christian König > > Stupid copy & paste error over all generations. does this fix an error or was this found just by reading the code? Could please you add such information to commit message in the future?

Re: Black monitor with external AMD card and coreboot: `radeon_bo_create:132 alloc size 0M bigger than 0Mb limit`

2013-05-13 Thread Paul Menzel
Am Montag, den 13.05.2013, 09:51 -0400 schrieb Jerome Glisse: > On Mon, May 13, 2013 at 8:15 AM, Paul Menzel wrote: > > using the ASRock E350M1 with coreboot [1] and plugging in an external > > AMD graphics card, > > > > 01:00.0 VGA compatible controller: Advan

Re: [PATCH] drm: fix a use-after-free when GPU acceleration disabled

2013-05-21 Thread Paul Menzel
t; Cc: > --- > drivers/gpu/drm/drm_irq.c |6 +- > 1 files changed, 5 insertions(+), 1 deletions(-) […] Acked-by: Paul Menzel Thanks, Paul signature.asc Description: This is a digitally signed message part ___ dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH] drm/mgag200: Don't do full cleanup if mgag200_device_init fails

2013-06-12 Thread Paul Menzel
+- > 1 file changed, 1 insertion(+), 1 deletion(-) […] Acked-by: Paul Menzel You should always CC the maintainer when sending in patches. Thanks, Paul signature.asc Description: This is a digitally signed message part ___ dri-devel mailin

No common commits between agd5f’s and Linus’ Git trees (was: [pull] radeon drm-next-3.11)

2013-07-06 Thread Paul Menzel
Dear Alex, Am Freitag, den 05.07.2013, 18:24 -0400 schrieb alexdeuc...@gmail.com: […] > The following changes since commit 338a95a95508537e23c82d59a2d87be6fde4b6ff: > > drm/radeon/sumo: implement support for disable_gfx_power_gating_in_uvd flag > (2013-07-03 17:37:31 -0400) > > are availab

Re: [solved] No common commits between agd5f’s and Linus’ Git trees

2013-07-06 Thread Paul Menzel
Dear Alex, Am Samstag, den 06.07.2013, 11:48 +0200 schrieb Paul Menzel: > Am Freitag, den 05.07.2013, 18:24 -0400 schrieb alexdeuc...@gmail.com: > > […] > > > The following changes since commit 338a95a95508537e23c82d59a2d87be6fde4b6ff: > > > > drm/rade

Radeon DPM branch successfully tested on Radeon HD 6310 (AMD Wrestler) (was: [pull] radeon drm-next-3.11)

2013-07-06 Thread Paul Menzel
Dear Radeon folks, Am Freitag, den 05.07.2013, 18:24 -0400 schrieb alexdeuc...@gmail.com: > From: Alex Deucher […] > The following changes since commit 338a95a95508537e23c82d59a2d87be6fde4b6ff: > > drm/radeon/sumo: implement support for disable_gfx_power_gating_in_uvd flag > (2013-07-03 17

Re: [PATCH] [RFC] drm/nouveau: bring back hdmi audio device after switcheroo power down

2013-07-24 Thread Paul Menzel
Am Mittwoch, den 24.07.2013, 17:13 +1000 schrieb Dave Airlie: > After a full device powerdown via the optimus power switch, we seem > to lose the HDMI device completely on power on, this keep track of keep*s* > whether we had a hdmi audio sub function device at power on, and > pokes a magic regis

Re: [PATCH] nouveau: Do not leak in nv20_graph_create

2011-07-03 Thread Paul Menzel
Dear Jesper, Am Sonntag, den 03.07.2011, 12:25 +0200 schrieb Jesper Juhl: > If we return due to an unknown chipset in > drivers/gpu/drm/nouveau/nv20_graph.c:nv20_graph_create() we'll leak the > memory allocated to 'pgraph'. > > This patch should fix the leak and it also disambiguates the "PGRA

Re: [PATCHv2] drm/radeon: Move pageflip request from vblank IRQ to ioctl

2011-07-06 Thread Paul Menzel
Am Mittwoch, den 06.07.2011, 19:04 +0100 schrieb Simon Farnsworth: > The radeon pageflip ioctl handler delayed submitting the pageflip to > hardware until the vblank IRQ handler. On AMD Fusion (PALM GPU, G-T56N > CPU), when using a reduced blanking CVT mode, a pageflip submitted to > hardware in th

Re: radeon, Linux 2.6.39: long one minute S3 resume because of missing firmware

2011-07-08 Thread Paul Menzel
Am Freitag, den 08.07.2011, 18:21 -0400 schrieb Alex Deucher: > On Fri, Jul 8, 2011 at 6:16 PM, Paul Menzel wrote: > > > > So there seems to be a long one minute delay before the following > > message? > > > >[ 1895.904100] r600_cp: Failed to load fi

Re: [Intel-gfx] [PATCH v5] drm/i915: pass ELD to HDMI/DP audio driver

2011-09-04 Thread Paul Menzel
Dear Wu, I hope that is your first name. Am Sonntag, den 04.09.2011, 05:15 +0800 schrieb Wu Fengguang: > Changes from v4: remove a debug call to dump_stack(). > Thanks to Bossart for catching this! His first name is Pierre-Louis. I do not know how you address people at Intel though. > --- I t

OT: sending patches to the list (was: [PATCH] kernel/drm: vblank wait on crtc > 1)

2011-03-21 Thread Paul Menzel
Am Sonntag, den 20.03.2011, 18:47 -0500 schrieb Ilija Hadzic: > sorry about that, I use pine and thought that's as plain as it gets. I > guess next time I'll try just 'mail' from command line. Or `git send-email`¹. Thanks, Paul ¹ manual: `git help send-email` signature.asc Description: Thi

Re: [PATCH] drm/modes: Fix CVT-R modeline generation

2010-08-28 Thread Paul Menzel
Am Montag, den 23.08.2010, 10:19 -0400 schrieb Adam Jackson: > Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=16651 > > Signed-off-by: Adam Jackson > --- > drivers/gpu/drm/drm_modes.c |5 - > 1 files changed, 4 insertions(+), 1 deletions(-) > > diff --git a/drivers/gpu/drm/drm_mo

Oops with Linux 3.0.0 and radeon driver: shmem_truncate_range

2011-10-05 Thread Paul Menzel
Dear DRI folks, the Linux kernel showed an Oops with the following call trace, which I copied from a picture taken from the screen. Pid: 1503, comm: Xorg Tainted: G C 3.0.0-1-amd64 #1 LENOVO INVALID/Bali [?] Process Xorg (pid: 1503, threadinfo 88010c20800

Oops with Linux 3.0.0 and radeon driver: shmem_truncate_range

2011-10-06 Thread Paul Menzel
Am Mittwoch, den 05.10.2011, 11:21 +0200 schrieb Michel D?nzer: > On Mit, 2011-10-05 at 10:54 +0200, Paul Menzel wrote: > > > > the Linux kernel showed an Oops with the following call trace, which I > > copied from a picture taken from the screen. > > > >

[PATCH 2/3] drm/radeon: no need to check all relocs for dublicates

2011-10-26 Thread Paul Menzel
Am Dienstag, den 25.10.2011, 17:04 +0200 schrieb Christian K?nig: > Only check the previusly checked relocs for previ*o*usly > dublicates. du*p*licates (also in commit summary) > Also leaving the handle uninitialized isn't such a good idea. > > Signed-off-by: Christian K?nig > --- > drivers/

[Intel-gfx] [PATCH v5] drm/i915: pass ELD to HDMI/DP audio driver

2011-09-04 Thread Paul Menzel
Dear Wu, I hope that is your first name. Am Sonntag, den 04.09.2011, 05:15 +0800 schrieb Wu Fengguang: > Changes from v4: remove a debug call to dump_stack(). > Thanks to Bossart for catching this! His first name is Pierre-Louis. I do not know how you address people at Intel though. > --- I t

[PATCH] staging: drm/omap: remove reclaim_buffers callback

2012-08-01 Thread Paul Menzel
ebugfs_init, > .debugfs_cleanup = omap_debugfs_cleanup, Acked-by: Paul Menzel Thanks, Paul -- next part -- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 198 bytes Desc: This is a digitally signed messag

[PATCH 1/3] Replace i2f() in r600_blit.c with an optimized version.

2012-08-06 Thread Paul Menzel
Dear Steven, thank you for your patches. Am Montag, den 06.08.2012, 11:14 -0700 schrieb Steven Fuerst: > Replace i2f() in r600_blit.c with an optimized version. > > We use __fls() to find the most significant bit. Using that, the > loop can be avoided. A second trick is to use the mod(32) > b

[PATCH 1/2] radeon: force 2D tiling for MSAA surfaces

2012-08-09 Thread Paul Menzel
Dear Marek, Am Donnerstag, den 09.08.2012, 16:37 +0200 schrieb Marek Ol??k: > --- > radeon/radeon_surface.c | 24 ++-- > 1 file changed, 22 insertions(+), 2 deletions(-) [?] do you have some numbers indicating an improvement with this change? On what hardware did you test

[PATCH] drm: EDID quirk improvements

2012-08-11 Thread Paul Menzel
s have access to the Linux source tree, so maybe a small overview is still needed? Or even an URL? > > module_param_named(debug, drm_debug, int, 0600); > module_param_named(vblankoffdelay, drm_vblank_offdelay, int, 0600); > module_param_named(timestamp_precision_usec, drm_timestamp_

Linux 3.2: After resume screen only turned on after switching terminals

2012-08-11 Thread Paul Menzel
Am Freitag, den 06.07.2012, 17:39 +0200 schrieb Paul Menzel: > Am Freitag, den 06.07.2012, 17:34 +0200 schrieb Paul Menzel: > > Am Donnerstag, den 05.07.2012, 08:58 -0400 schrieb Alex Deucher: > > > On Wed, Jul 4, 2012 at 10:11 AM, Paul Menzel wrote: > > > > Am M

Kconfig: warning: (DRM_RADEON_KMS && DRM_I915 && …) selects BACKLIGHT_CLASS_DEVICE which has unmet direct dependencies (HAS_IOMEM && BACKLIGHT_LCD_SUPPORT)

2012-08-11 Thread Paul Menzel
Dear Linux folks, where should I report the following warning warning: (DRM_RADEON_KMS && DRM_I915 && STUB_POULSBO && FB_BACKLIGHT && USB_APPLEDISPLAY && FB_OLPC_DCON && ASUS_LAPTOP && SONY_LAPTOP && THINKPAD_ACPI && EEEPC_LAPTOP && ACPI_CMPC && SAMSUNG_Q10 && APPLE_GMUX) selects BACK

[PATCH] drm: EDID quirk improvements

2012-08-11 Thread Paul Menzel
Dear Ian, Am Freitag, den 10.08.2012, 13:44 -0500 schrieb Ian Pilcher: [?] > diff --git a/Documentation/EDID/edid_quirks.txt > b/Documentation/EDID/edid_quirks.txt > new file mode 100644 > index 000..256ded0 > --- /dev/null > +++ b/Documentation/EDID/edid_quirks.txt [?] > +Overview > +==

[PATCH] drm: EDID quirk improvements

2012-08-11 Thread Paul Menzel
Dear Ian, Am Samstag, den 11.08.2012, 10:38 -0500 schrieb Ian Pilcher: > On 08/11/2012 03:31 AM, Paul Menzel wrote: > > As a side note, could you also mention the patch iteration in the tag, > > that means [PATCH vN] so that I know what is the latest version. That > > wou

[PATCH v3 0/2] Enhanced EDID quirk functionality

2012-08-12 Thread Paul Menzel
Dear Ian, thank you a million for doing that. Am Samstag, den 11.08.2012, 23:30 -0500 schrieb Ian Pilcher: > Updated patch set, based on Paul's feedback. > > * Separate user-defined quirks stuff from new HDMI-related quirks > * (Hopefully) improve documentation Yeah, maybe somebody else can c

How to find out modeline without attaching actual device?

2012-08-12 Thread Paul Menzel
Dear Linux folks, regarding modelines problems with some TVs [1] and EDID quirks, Ian sent nice patches for [2], is there a way to test what modeline the DRM subsystem would choose without actually attaching the device? The problem is, the TV is at a different place and I rarely have access to i

[PATCH v3 1/2] drm: Add user-defined EDID quirks capability

2012-08-12 Thread Paul Menzel
Dear Ian, Am Samstag, den 11.08.2012, 23:30 -0500 schrieb Ian Pilcher: > Add the ability for users to define their own EDID quirks via a > module parameter or sysfs attribute. > > Signed-off-by: Ian Pilcher > Acked-by: Paul Menzel please also add Cc: as documented in [1] s

[PATCH v3 2/2] drm: Add EDID quirks to disable HDMI audio and InfoFrames

2012-08-12 Thread Paul Menzel
easily. So it would be great if you could split this patch up too although it is quite small too. Could you also mention in the commit message of the patch adding the LG monitor on what system you tested this just for the record. > Signed-off-by: Ian Pilcher > Acked-by: Paul Menzel >

[PATCH v4 0/3] Enhanced EDID quirk functionality

2012-08-12 Thread Paul Menzel
Dear Ian, thank you very much for your quick iterations. Hopefully the developers will review this on Monday. Am Sonntag, den 12.08.2012, 15:07 -0500 schrieb Ian Pilcher: > Another rev. I figured out how to use git reset --hard and --soft to > make regenerating the patch series a bit easier. (

[PATCH] drm/radeon: Include swiotlb.h if SWIOTLB configured.

2012-08-13 Thread Paul Menzel
Dear Chen, thanks for your patch. Firstly, is Chen your first or last name? If it is your first name, your

[PATCH] drm/radeon: Include swiotlb.h if SWIOTLB configured.

2012-08-13 Thread Paul Menzel
Dear Huacai, Am Montag, den 13.08.2012, 15:16 +0800 schrieb Huacai Chen: > On Mon, Aug 13, 2012 at 3:00 PM, Paul Menzel wrote: > > thanks for your patch. > > > > Firstly, is Chen your first or last name? If it is your first name, your > > From address should be switch

[PATCH] drm/radeon: Include swiotlb.h if SWIOTLB configured.

2012-08-13 Thread Paul Menzel
Am Montag, den 13.08.2012, 16:04 +0800 schrieb Huacai Chen: > On Mon, Aug 13, 2012 at 3:50 PM, Paul Menzel wrote: > > Am Montag, den 13.08.2012, 15:16 +0800 schrieb Huacai Chen: > >> On Mon, Aug 13, 2012 at 3:00 PM, Paul Menzel wrote: [?] > >> > Am Montag, den 1

[PATCH v4 1/3] drm: Add user-defined EDID quirks capability

2012-08-14 Thread Paul Menzel
Am Sonntag, den 12.08.2012, 15:07 -0500 schrieb Ian Pilcher: > Add the ability for users to define their own EDID quirks via a > module parameter or sysfs attribute. > > Signed-off-by: Ian Pilcher > Acked-by: Paul Menzel > --- > Documentation/EDID/edid_quirks.txt | 126

[PATCH v4 1/3] drm: Add user-defined EDID quirks capability

2012-08-15 Thread Paul Menzel
Am Dienstag, den 14.08.2012, 10:45 -0500 schrieb Ian Pilcher: > On 08/14/2012 10:13 AM, Paul Menzel wrote: > > Testing your patch, I would vote that a normal user is allowed to read > > the quirk list. > > I thought about that, but I decided to leave it at 0600, because

[PATCH] drm: Add EDID_QUIRK_FORCE_REDUCED_BLANKING for Philips 32PFL5404H

2012-08-15 Thread Paul Menzel
ugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=26294 Tested-by: Paul Menzel (ASUS Eee PC 701 4G with Debian Sid/unstable connected over VGA) Signed-off-by: Paul Menzel Cc: Cc: Adam Jackson Cc: Ian Pilcher Cc: --- Ian, I did not base this patch on your series, to make it easier to get

[PATCH] drm: Add EDID_QUIRK_FORCE_REDUCED_BLANKING for LG SL80

2012-08-15 Thread Paul Menzel
.jsp [2] http://git.kernel.org/?p=linux/kernel/git/torvalds/linux.git;a=commit;h=bc42aabc6a01b92b0f961d65671564e0e1cd7592 Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=53544 Signed-off-by: Paul Menzel Cc: Cc: Adam Jackson Cc: Ian Pilcher Cc: --- Same as in previous patch: Ian, I did

[PATCH] drm: Add EDID_QUIRK_FORCE_REDUCED_BLANKING for ASUS VW222S

2012-08-15 Thread Paul Menzel
/VW222S/ [2] http://git.kernel.org/?p=linux/kernel/git/torvalds/linux.git;a=commit;h=bc42aabc6a01b92b0f961d65671564e0e1cd7592 Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=17629 Signed-off-by: Paul Menzel Cc: Cc: Adam Jackson Cc: Ian Pilcher Cc: --- Same comment as in previous patch

[PATCH] drm/radeon: fix checking of MSAA renderbuffers on r600-r700

2012-08-20 Thread Paul Menzel
Dear Marek, thank you for all your work on MSAA. Am Sonntag, den 19.08.2012, 21:23 +0200 schrieb Marek Ol??k: Unfortunately you do not provide any commit message. What is the problem and what are the symptoms? When was it introduced? How is it solved in your patch? > Signed-off-by: Marek Ol??

[PATCH] drm: edid: add support for E-DDC

2012-08-21 Thread Paul Menzel
Am Dienstag, den 21.08.2012, 12:40 +0530 schrieb Shirish S: > The current logic for probing ddc is limited to > 2 blocks (256 bytes), this patch adds support > for the 4 block (512) data. > > To do this, a single 8-bit segment index is > passed to the display via the I2C address 30h. > Data from t

[PATCH] drm/exynos: fixed a issue that plane isn't disabled when released

2012-08-24 Thread Paul Menzel
Dear Inki, Am Freitag, den 24.08.2012, 18:27 +0900 schrieb Inki Dae: You can shorten the commit summary by leaving out the word issue, which is redundant. Maybe one of the following? I do not understand the process of releasing so it might be wrong. drm/exynos: Disable plane after relea

[PATCH] drm/radeon/atom: powergating fixes for DCE6

2012-08-28 Thread Paul Menzel
Alex Deucher > Cc: stable at vger.kernel.org > > --- > drivers/gpu/drm/radeon/atombios_crtc.c | 22 ++ > 1 files changed, 2 insertions(+), 20 deletions(-) [?] Acked-by: Paul Menzel Thanks, Paul -- next part -- A non-text attachmen

[PATCH] drm: Add EDID_QUIRK_FORCE_REDUCED_BLANKING for ASUS VW222S

2012-08-29 Thread Paul Menzel
://bugs.freedesktop.org/show_bug.cgi?id=17629 Signed-off-by: Paul Menzel Cc: Cc: Adam Jackson Cc: Ian Pilcher Cc: --- v2: ? Prepend model number with 0x to fix compilation. ? Update commit message to mention that there is only a VGA connector. Same comment as in previous patch. Ian, I did not base this

[PATCH] drm/i915: add a tracepoint for gpu frequency changes

2012-08-30 Thread Paul Menzel
915/intel_pm.c > @@ -2343,6 +2343,8 @@ void gen6_set_rps(struct drm_device *dev, u8 val) > I915_WRITE(GEN6_RP_INTERRUPT_LIMITS, limits); > > dev_priv->rps.cur_delay = val; > + > + trace_intel_gpu_freq_change(val * 50); > } > > static void gen6_disable_rps(struct

[PATCH] drm: Add EDID_QUIRK_FORCE_REDUCED_BLANKING for ASUS VW222S

2012-12-18 Thread Paul Menzel
Dear Florian, Am Dienstag, den 18.12.2012, 21:03 +0100 schrieb Florian Mickler: > On Wed, 15 Aug 2012 17:40:40 +0200 Paul Menzel wrote: > > > Date: Wed, 8 Aug 2012 23:12:19 +0200 > > > > Connecting an ASUS VW222S [1] over VGA a garbled screen is shown with > > ve

[PATCH] drm/ttm: fix delayed ttm_bo_cleanup_refs_and_unlock delayed handling

2012-12-19 Thread Paul Menzel
Am Mittwoch, den 19.12.2012, 15:18 +0100 schrieb Maarten Lankhorst: > Fix regression introduced by 85b144f860176 Thanks for the catch and patch. Also please add the commit summary to make the commit message self contained? The problem description would also be nice. > Signed-off-by: Maarten Lan

[PATCH] drm/ttm: fix delayed ttm_bo_cleanup_refs_and_unlock delayed handling

2012-12-19 Thread Paul Menzel
* at this point the buffer should be dead, so >* no new sync objects can be attached. >*/ > - sync_obj = driver->sync_obj_ref(&bo->sync_obj); > + sync_obj = driver->sync_obj_ref(bo->sync_obj); >

[PATCH] drm: Add EDID_QUIRK_FORCE_REDUCED_BLANKING for ASUS VW222S

2012-12-31 Thread Paul Menzel
Dear Florian, Am Freitag, den 21.12.2012, 14:52 +0100 schrieb Florian Mickler: > On Tue, 18 Dec 2012 23:33:20 +0100 > Paul Menzel wrote: > > Am Dienstag, den 18.12.2012, 21:03 +0100 schrieb Florian Mickler: > > > > > On Wed, 15 Aug 2012 17:40:40 +0200 Paul Menzel

[PATCH] drm: Add EDID_QUIRK_FORCE_REDUCED_BLANKING for ASUS VW222S

2013-01-01 Thread Paul Menzel
Dear Florian, first, I wish everyone a Happy New Year! Am Montag, den 31.12.2012, 18:22 +0100 schrieb Florian Mickler: > do you have a quick pointer to your decoded EDID? yes, everything is in the bug report the patch was supposed to fix [1]. > Maybe we can tell the VW222S and the VW222U apa

[PATCH] drm/radeon/kms/blit: fix blit copy for very large buffers

2012-02-02 Thread Paul Menzel
Dear Ilija, Am Mittwoch, den 01.02.2012, 17:07 -0500 schrieb Ilija Hadzic: [?] > +#define I2F_MAX_BITS 15 > +#define I2F_MAX_INPUT ((2 << I2F_MAX_BITS) - 1) > +#define I2F_SHIFT (24 - I2F_MAX_BITS) > + > +/* > + * converts unsigned integer into 32-bit IEEE floating point representation; > + *

`radeontool light off` does not turn off backlight

2012-02-10 Thread Paul Menzel
Dear Radeon folks, using radeontool 1.6.2-1.1 from Debian Wheezy/testing [1] $ sudo radeontool --debug light off Found card 1002:9555 (3) Radeon found. Base control address is 7f0746826000; base framebuffer address is 7f0735e73000. reading RADEON_LVDS_

`radeontool light off` does not turn off backlight

2012-02-10 Thread Paul Menzel
Am Freitag, den 10.02.2012, 05:58 -0500 schrieb David Airlie: > > using radeontool 1.6.2-1.1 from Debian Wheezy/testing [1] > > > > $ sudo radeontool --debug light off > > Found card 1002:9555 (3) > > Radeon found. Base control address is 7f0746826000; base > >

[PATCH] Fix spelling in comments.

2012-02-13 Thread Paul Menzel
Dear Kristof, Am Samstag, den 11.02.2012, 15:04 +0100 schrieb Kristof Ralovich: > Dear All, > > attached is the patch. please only send inlined patches. You can use `git send-email` for that. > From 87b2ba50ca33b85fae58e85cd51ba7515bceb81b Mon Sep 17 00:00:00 2001 > From: =?UTF-8?q?"RALOVICH,=

[PATCH] drm/i915: error_buffer->ring should be signed

2012-02-16 Thread Paul Menzel
f the report is the following. http://lists.freedesktop.org/archives/dri-devel/2012-February/019183.html > Signed-off-by: Daniel Vetter Acked-by: Paul Menzel > --- > drivers/gpu/drm/i915/i915_drv.h |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --gi

Flickering with page-flipping on Acer Iconia W500 (AMD C-50 APU)

2012-02-22 Thread Paul Menzel
Hi Felix! Am Dienstag, den 21.02.2012, 15:07 -0500 schrieb Alex Deucher: > On Wed, Feb 1, 2012 at 5:43 PM, Felix Kuehling > wrote: > > Following up on my message from Jan 19, now with a lot more hard data and a > > less intrusive modification. Still a prototype though. CC-ing DRI-devel and > >

[PATCH] drm/radeon/kms: fix radeon_dp_get_modes for LVDS bridges

2012-02-28 Thread Paul Menzel
Dear Alex, Am Montag, den 27.02.2012, 19:31 -0500 schrieb alexdeucher at gmail.com: > From: Alex Deucher > > Need to call ExternalEncoderControl to set up DDC before > trying to get an EDID for all DP bridge chipsi (including Should that be `s/chipsi/chips/`? > DP to LVDS). > > Also remove r

regression(?) 3.3-rc4 -> 3.3-rc5: drm intel hangs

2012-02-28 Thread Paul Menzel
Dear Norbert, Am Dienstag, den 28.02.2012, 13:03 +0900 schrieb Norbert Preining: > Dear all, > > (please Cc) > > since upgrade to 3.3-rc5 I see the following behaviour repeatedly: > > Feb 28 11:42:47 mithrandir kernel: [15627.756071] > [drm:i915_hangcheck_elapsed] *ERROR* Hangcheck timer elap

Flickering with page-flipping on Acer Iconia W500 (AMD C-50 APU)

2012-02-29 Thread Paul Menzel
Am Mittwoch, den 29.02.2012, 10:36 + schrieb Dave Airlie: > On Tue, Feb 28, 2012 at 3:06 PM, Felix Kuehling > wrote: > > On 12-02-28 09:40 AM, Mario Kleiner wrote: > >> > >> Ok, you had a larger test set of machines and more specific test cases > >> for this problem, i'm convinced. Thanks for

[PATCH] i915: Add option to bypass vbt table.

2012-02-29 Thread Paul Menzel
> tables.\n"); > + return; > + } > + > if (index == -1) { > struct bdb_sdvo_lvds_options *sdvo_lvds_options; Otherwise this patch looks fine, so you can directly add Reviewed-by: Paul Menzel to the commit message of the patch

Linux 3.2: After resume screen only turned on after switching terminals

2012-07-04 Thread Paul Menzel
Am Samstag, den 30.06.2012, 21:00 +0200 schrieb Paul Menzel: > Am Dienstag, den 26.06.2012, 07:42 +0200 schrieb Michel D?nzer: > > On Mon, 2012-06-25 at 21:14 +0200, Paul Menzel wrote: > > > > > > resuming from suspend to RAM the monitor was indicating by a blinkin

Linux 3.2: After resume screen only turned on after switching terminals

2012-07-04 Thread Paul Menzel
Am Mittwoch, den 04.07.2012, 09:55 -0400 schrieb Alex Deucher: > On Wed, Jul 4, 2012 at 9:41 AM, Paul Menzel wrote: > > Am Samstag, den 30.06.2012, 21:00 +0200 schrieb Paul Menzel: > >> Am Dienstag, den 26.06.2012, 07:42 +0200 schrieb Michel D?nzer: > >> > On Mon, 2

Linux 3.2: After resume screen only turned on after switching terminals

2012-07-06 Thread Paul Menzel
Am Donnerstag, den 05.07.2012, 08:58 -0400 schrieb Alex Deucher: > On Wed, Jul 4, 2012 at 10:11 AM, Paul Menzel wrote: > > Am Mittwoch, den 04.07.2012, 09:55 -0400 schrieb Alex Deucher: > >> On Wed, Jul 4, 2012 at 9:41 AM, Paul Menzel wrote: > >> > Am Samstag, den 3

Linux 3.2: After resume screen only turned on after switching terminals

2012-07-06 Thread Paul Menzel
Am Freitag, den 06.07.2012, 17:34 +0200 schrieb Paul Menzel: > Am Donnerstag, den 05.07.2012, 08:58 -0400 schrieb Alex Deucher: > > On Wed, Jul 4, 2012 at 10:11 AM, Paul Menzel wrote: > > > Am Mittwoch, den 04.07.2012, 09:55 -0400 schrieb Alex Deucher: > > >> On Wed

[PATCH 4/7] drivers/gpu/drm: remove invalid reference to list iterator variable

2012-07-08 Thread Paul Menzel
; 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); Acked-b

OT: sending patches to the list (was: [PATCH] kernel/drm: vblank wait on crtc > 1)

2011-03-21 Thread Paul Menzel
Am Sonntag, den 20.03.2011, 18:47 -0500 schrieb Ilija Hadzic: > sorry about that, I use pine and thought that's as plain as it gets. I > guess next time I'll try just 'mail' from command line. Or `git send-email`?. Thanks, Paul ? manual: `git help send-email` -- next part ---

[PATCH] drm_edid: support CEA video modes

2011-11-14 Thread Paul Menzel
Am Sonntag, den 13.11.2011, 23:16 +0100 schrieb alanwww1: > This is awesome ! A LOT of HTPC fans are waiting for this! Is there a > chance that this lands in 3.2 ? Have you tested this? Adding a Tested-by line would surely help getting it included. Thanks, Paul -- next part

[PATCH] drm/modes: Fix CVT-R modeline generation

2010-08-28 Thread Paul Menzel
Am Montag, den 23.08.2010, 10:19 -0400 schrieb Adam Jackson: > Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=16651 > > Signed-off-by: Adam Jackson > --- > drivers/gpu/drm/drm_modes.c |5 - > 1 files changed, 4 insertions(+), 1 deletions(-) > > diff --git a/drivers/gpu/drm/drm_mo

<    1   2   3   4   >