Atomicity in KMS panic notifier

2014-05-05 Thread Takashi Iwai
At Mon, 5 May 2014 16:29:37 +0200, Daniel Vetter wrote: > > On Mon, May 5, 2014 at 3:02 PM, Takashi Iwai wrote: > > Hi, > > > > while debugging a few reported bugs, I noticed that > > drm_fb_helper_force_kernel_mode() that is called in the KMS panic > &g

Atomicity in KMS panic notifier

2014-05-05 Thread Takashi Iwai
Hi, while debugging a few reported bugs, I noticed that drm_fb_helper_force_kernel_mode() that is called in the KMS panic notifier isn't really atomic-safe. It invokes crtc's set_config(), and all implementations seem to involve with page allocations (kmalloc with GFP_KERNEL, via some ttm ops,

[PATCH] drm/mgag200: Implement basic PM support

2014-04-29 Thread Takashi Iwai
Add a basic PM support to mgag200 driver. As there is no hardware specific init codes for this device, the callbacks are written in a fairly simple way. Signed-off-by: Takashi Iwai --- drivers/gpu/drm/mgag200/mgag200_drv.c | 75 +++ drivers/gpu/drm/mgag200

[PATCH] drm: Prefer noninterlace cmdline mode unless explicitly specified

2014-04-01 Thread Takashi Iwai
At Wed, 19 Mar 2014 15:17:50 +0100, Daniel Vetter wrote: > > On Wed, Mar 19, 2014 at 02:53:13PM +0100, Takashi Iwai wrote: > > Currently drm_pick_cmdline_mode() doesn't care about the interlace > > when the given mode line has no "i" suffix. That is, when th

[PATCH] drm: Prefer noninterlace cmdline mode unless explicitly specified

2014-03-19 Thread Takashi Iwai
At Wed, 19 Mar 2014 15:17:50 +0100, Daniel Vetter wrote: > > On Wed, Mar 19, 2014 at 02:53:13PM +0100, Takashi Iwai wrote: > > Currently drm_pick_cmdline_mode() doesn't care about the interlace > > when the given mode line has no "i" suffix. That is, when th

[PATCH] drm: Prefer noninterlace cmdline mode unless explicitly specified

2014-03-19 Thread Takashi Iwai
to exclude it. This patch changes the logic for the mode selection, to prefer the noninterlace mode unless the interlace mode is explicitly given. When no matching mode is found, it still tries the interlace mode as fallback. Signed-off-by: Takashi Iwai --- drivers/gpu/drm/drm_fb_helper.c | 11 +

[alsa-devel] [PATCH 4/4] ASoC: tda998x: adjust the audio hw parameters from EDID

2014-01-28 Thread Takashi Iwai
At Tue, 28 Jan 2014 11:00:51 +, Mark Brown wrote: > > On Tue, Jan 28, 2014 at 10:23:57AM +0100, Takashi Iwai wrote: > > Mark Brown wrote: > > > On Mon, Jan 27, 2014 at 08:49:15PM +, Russell King - ARM Linux wrote: > > > > > Yes, preferably as a gen

[alsa-devel] [PATCH 4/4] ASoC: tda998x: adjust the audio hw parameters from EDID

2014-01-28 Thread Takashi Iwai
At Mon, 27 Jan 2014 20:54:37 +, Mark Brown wrote: > > On Mon, Jan 27, 2014 at 08:49:15PM +, Russell King - ARM Linux wrote: > > On Mon, Jan 27, 2014 at 08:44:41PM +, Mark Brown wrote: > > > > Can this parsing code be factored out - it (or large parts of it) should > > > be usable by

Need your advice: Add a new communication inteface between HD-Audio and Gfx drivers for hotplug notification/ELD update

2014-01-23 Thread Takashi Iwai
At Thu, 23 Jan 2014 06:35:12 +, Lin, Mengdong wrote: > > > -Original Message- > > From: Takashi Iwai [mailto:tiwai at suse.de] > > Sent: Thursday, January 23, 2014 1:19 AM > > To: Daniel Vetter > > Cc: Lin, Mengdong; Barnes, Jesse; Zanoni, Paulo R; &

[Intel-gfx] Need your advice: Add a new communication inteface between HD-Audio and Gfx drivers for hotplug notification/ELD update

2014-01-22 Thread Takashi Iwai
At Wed, 22 Jan 2014 10:45:26 -0500, Rob Clark wrote: > > On Wed, Jan 22, 2014 at 10:20 AM, Daniel Vetter wrote: > > On Wed, Jan 22, 2014 at 10:04:14AM -0500, Rob Clark wrote: > >> sorry to jump into this a bit late, so maybe this was covered already > >> earlier.. > > > > It just started, I've

Need your advice: Add a new communication inteface between HD-Audio and Gfx drivers for hotplug notification/ELD update

2014-01-22 Thread Takashi Iwai
gt; > Daniel Vetter > > > Sent: Tuesday, January 21, 2014 9:11 PM > > > To: Lin, Mengdong > > > Cc: Takashi Iwai (tiwai at suse.de); Barnes, Jesse; Zanoni, Paulo R; > > > alsa-devel at alsa-project.org; intel-gfx at lists.freedesktop.org > > > Subject: Re: Ne

[alsa-devel] [PATCH v2 28/28] drm/i2c: tda998x: add a CODEC interface

2014-01-10 Thread Takashi Iwai
At Thu, 9 Jan 2014 12:08:03 +0100, Jean-Francois Moine wrote: > > The TDA998x chips may get audio input from I2S or S/PDIF and this input > is defined by the audio subsystem. > > In the other way, the audio subsystem may need to know if audio output > may be done through the HDMI connector. > >

[PATCH] drm/cirrus: Fix cirrus drm driver for fbdev + qemu

2014-01-09 Thread Takashi Iwai
From: Martin Koegler <martin.koeg...@chello.at> Xorg fbdev driver requires smem_start/smem_len, otherwise it tries to map 0 bytes as video memory. Bugzilla: https://bugzilla.novell.com/show_bug.cgi?id=856760 Signed-off-by: Martin Koegler Signed-off-by: Takashi Iwai --- drivers/gpu/drm/

Re: [PATCH 6/8] ALSA: ppc: keywest: Don't use i2c_client-driver

2013-09-30 Thread Takashi Iwai
Acked-by: Takashi Iwai ti...@suse.de thanks, Takashi --- sound/ppc/keywest.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sound/ppc/keywest.c b/sound/ppc/keywest.c index 01aecc2..0d1c27e 100644 --- a/sound/ppc/keywest.c +++ b/sound/ppc/keywest.c @@ -65,7

trees for secondary powerdown

2013-08-28 Thread Takashi Iwai
At Wed, 28 Aug 2013 18:25:47 +0200, Takashi Iwai wrote: > > Hi Dave, > > At Wed, 28 Aug 2013 16:51:55 +1000, > Dave Airlie wrote: > > > > Hi Takashi, > > > > I've put two branches in my repo, > > > > http://cgit.freedesktop.org/~a

trees for secondary powerdown

2013-08-28 Thread Takashi Iwai
Hi Dave, At Wed, 28 Aug 2013 16:51:55 +1000, Dave Airlie wrote: > > Hi Takashi, > > I've put two branches in my repo, > > http://cgit.freedesktop.org/~airlied/linux/log/?h=drm-optimus-power-down > http://cgit.freedesktop.org/~airlied/linux/log/?h=drm-optimus-power-down-snd-merge > >

Re: trees for secondary powerdown

2013-08-28 Thread Takashi Iwai
Hi Dave, At Wed, 28 Aug 2013 16:51:55 +1000, Dave Airlie wrote: Hi Takashi, I've put two branches in my repo, http://cgit.freedesktop.org/~airlied/linux/log/?h=drm-optimus-power-down http://cgit.freedesktop.org/~airlied/linux/log/?h=drm-optimus-power-down-snd-merge

Re: trees for secondary powerdown

2013-08-28 Thread Takashi Iwai
At Wed, 28 Aug 2013 18:25:47 +0200, Takashi Iwai wrote: Hi Dave, At Wed, 28 Aug 2013 16:51:55 +1000, Dave Airlie wrote: Hi Takashi, I've put two branches in my repo, http://cgit.freedesktop.org/~airlied/linux/log/?h=drm-optimus-power-down http://cgit.freedesktop.org

[PATCH 4/4] snd/hda: add runtime suspend/resume on optimus support (v3)

2013-08-06 Thread Takashi Iwai
At Mon, 5 Aug 2013 12:56:04 +1000, Dave Airlie wrote: > > Add support for HDMI audio device on VGA cards that powerdown > to D3cold using non-standard ACPI/PCI infrastructure (optimus). > > This does a couple of things to make it work: > > a) add a set of power ops for the hdmi domain, and

Re: [PATCH 4/4] snd/hda: add runtime suspend/resume on optimus support (v3)

2013-08-06 Thread Takashi Iwai
At Mon, 5 Aug 2013 12:56:04 +1000, Dave Airlie wrote: Add support for HDMI audio device on VGA cards that powerdown to D3cold using non-standard ACPI/PCI infrastructure (optimus). This does a couple of things to make it work: a) add a set of power ops for the hdmi domain, and enables

[PATCH 4/4] snd/hda: add runtime suspend/resume on optimus support

2013-07-29 Thread Takashi Iwai
At Mon, 29 Jul 2013 16:06:59 +1000, Dave Airlie wrote: > > Add support for HDMI audio device on VGA cards that powerdown > to D3cold using non-standard ACPI/PCI infrastructure (optimus). > > This does a couple of things to make it work: > > a) add a set of power ops for the hdmi domain, and

Re: [PATCH 4/4] snd/hda: add runtime suspend/resume on optimus support

2013-07-29 Thread Takashi Iwai
At Mon, 29 Jul 2013 16:06:59 +1000, Dave Airlie wrote: Add support for HDMI audio device on VGA cards that powerdown to D3cold using non-standard ACPI/PCI infrastructure (optimus). This does a couple of things to make it work: a) add a set of power ops for the hdmi domain, and enables

sub function power management (gpu + audio device)

2013-03-12 Thread Takashi Iwai
At Tue, 12 Mar 2013 16:05:26 +1000, Dave Airlie wrote: > > Hey guys, > > I've been writing dynamic power management code for the secondary > GPUs, however a lot of these GPUs have audio codecs as a subfunction > PCI device. > > So we get 01:00.0 as the GPU and 01:00.1 as the HDMI audio device.

Re: sub function power management (gpu + audio device)

2013-03-12 Thread Takashi Iwai
At Tue, 12 Mar 2013 16:05:26 +1000, Dave Airlie wrote: Hey guys, I've been writing dynamic power management code for the secondary GPUs, however a lot of these GPUs have audio codecs as a subfunction PCI device. So we get 01:00.0 as the GPU and 01:00.1 as the HDMI audio device. Now

[PATCH] drm/cirrus: Add bpp option

2013-01-29 Thread Takashi Iwai
At Tue, 29 Jan 2013 10:53:50 +0100, Daniel Vetter wrote: > > On Tue, Jan 29, 2013 at 09:29:17AM +0100, Takashi Iwai wrote: > > Add a new option, bpp, to specify the default bpp value. > > > > Signed-off-by: Takashi Iwai > > --- > > > > This patch is

[PATCH] drm/cirrus: Add bpp option

2013-01-29 Thread Takashi Iwai
Add a new option, bpp, to specify the default bpp value. Signed-off-by: Takashi Iwai --- This patch is applied on the top of previous two patches. I couldn't find an easy way to specify the default bpp, so I cooked the driver quickly. If there is any other convenient way to achieve this, let

[PATCH 1/2] drm/cirrus: Correct register values for 16bpp

2013-01-29 Thread Takashi Iwai
Hi Dave, any chance to take a look at this problem? thanks, Takashi At Fri, 25 Jan 2013 17:21:54 +0100, Takashi Iwai wrote: > > When the mode is set with 16bpp on QEMU, the output gets totally > broken. The culprit is the bogus register values set for 16bpp, > which was likely

Re: [PATCH 1/2] drm/cirrus: Correct register values for 16bpp

2013-01-29 Thread Takashi Iwai
Hi Dave, any chance to take a look at this problem? thanks, Takashi At Fri, 25 Jan 2013 17:21:54 +0100, Takashi Iwai wrote: When the mode is set with 16bpp on QEMU, the output gets totally broken. The culprit is the bogus register values set for 16bpp, which was likely copied from from

[PATCH] drm/cirrus: Add bpp option

2013-01-29 Thread Takashi Iwai
Add a new option, bpp, to specify the default bpp value. Signed-off-by: Takashi Iwai ti...@suse.de --- This patch is applied on the top of previous two patches. I couldn't find an easy way to specify the default bpp, so I cooked the driver quickly. If there is any other convenient way

Re: [PATCH] drm/cirrus: Add bpp option

2013-01-29 Thread Takashi Iwai
At Tue, 29 Jan 2013 10:53:50 +0100, Daniel Vetter wrote: On Tue, Jan 29, 2013 at 09:29:17AM +0100, Takashi Iwai wrote: Add a new option, bpp, to specify the default bpp value. Signed-off-by: Takashi Iwai ti...@suse.de --- This patch is applied on the top of previous two patches

[PATCH 2/2] drm/cirrus: Use 16bpp as default

2013-01-25 Thread Takashi Iwai
Signed-off-by: Takashi Iwai --- drivers/gpu/drm/cirrus/cirrus_fbdev.c | 2 +- drivers/gpu/drm/cirrus/cirrus_mode.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/cirrus/cirrus_fbdev.c b/drivers/gpu/drm/cirrus/cirrus_fbdev.c index 6c6b4c8..bc081ea 100644

[PATCH 1/2] drm/cirrus: Correct register values for 16bpp

2013-01-25 Thread Takashi Iwai
When the mode is set with 16bpp on QEMU, the output gets totally broken. The culprit is the bogus register values set for 16bpp, which was likely copied from from a wrong place. Bugzilla: https://bugzilla.novell.com/show_bug.cgi?id=799216 Cc: Signed-off-by: Takashi Iwai --- drivers/gpu/drm

[PATCH 1/2] drm/cirrus: Correct register values for 16bpp

2013-01-25 Thread Takashi Iwai
When the mode is set with 16bpp on QEMU, the output gets totally broken. The culprit is the bogus register values set for 16bpp, which was likely copied from from a wrong place. Bugzilla: https://bugzilla.novell.com/show_bug.cgi?id=799216 Cc: sta...@vger.kernel.org Signed-off-by: Takashi Iwai

[PATCH 2/2] drm/cirrus: Use 16bpp as default

2013-01-25 Thread Takashi Iwai
Signed-off-by: Takashi Iwai ti...@suse.de --- drivers/gpu/drm/cirrus/cirrus_fbdev.c | 2 +- drivers/gpu/drm/cirrus/cirrus_mode.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/cirrus/cirrus_fbdev.c b/drivers/gpu/drm/cirrus/cirrus_fbdev.c index 6c6b4c8

[PATCH 1/2] fb: Rework locking to fix lock ordering on takeover

2013-01-23 Thread Takashi Iwai
At Wed, 23 Jan 2013 17:25:08 +0100, Daniel Vetter wrote: > > From: Alan Cox > > Adjust the console layer to allow a take over call where the caller already > holds the locks. Make the fb layer lock in order. > > This s partly a band aid, the fb layer is terminally confused about the > locking

Re: [PATCH 1/2] fb: Rework locking to fix lock ordering on takeover

2013-01-23 Thread Takashi Iwai
At Wed, 23 Jan 2013 17:25:08 +0100, Daniel Vetter wrote: From: Alan Cox a...@lxorguk.ukuu.org.uk Adjust the console layer to allow a take over call where the caller already holds the locks. Make the fb layer lock in order. This s partly a band aid, the fb layer is terminally confused

[PATCH 13/17] DRM/KMS/EDID: Allow for multiple Connectors when specifying 'firmware'-EDID Files.

2012-11-20 Thread Takashi Iwai
At Mon, 19 Nov 2012 15:23:14 -0500, "Egbert Eich " <"eich at novell.com> wrote: > > So far it was only possible to load an EDID for a single connector (unless > no connector was specified at all in which case the same EDID file was used > for all). > This patch extends the EDID loader so that

[PATCH 10/17] DRM/KMS/EDID: Cache EDID blobs with extensions.

2012-11-20 Thread Takashi Iwai
At Mon, 19 Nov 2012 15:23:11 -0500, "Egbert Eich " <"eich at novell.com> wrote: > > According the the VESA specs there can be up to 254 EEDID extension blocks. > Since we may read the EDID (including extensions) in 10 second intervals to > probe for display hotplugging (at least in cases where no

[PATCH 07/17] DRM/KMS/EDID: Move drm_edid_load.o to drm.ko

2012-11-20 Thread Takashi Iwai
At Mon, 19 Nov 2012 15:23:08 -0500, "Egbert Eich " <"eich at novell.com> wrote: > > EDIDs are an integral concept of connectors, connectors are a concept > of drm core also drm_edid.o is already part of this drm core. > Overridden, 'firmware-supplied' EDIDs should be treated exactly the > same as

[PATCH 08/17] DRM/KMS/EDID: Use Extension Block Fixup Code also for 'firmware' EDID.

2012-11-20 Thread Takashi Iwai
At Mon, 19 Nov 2012 15:23:09 -0500, "Egbert Eich " <"eich at novell.com> wrote: > > Signed-off-by: Egbert Eich > --- > drivers/gpu/drm/drm_edid.c | 77 > --- > drivers/gpu/drm/drm_edid_load.c | 54 ++- >

[PATCH 05/17] DRM/KMS/EDID: Fix up EEDID Map Blocks if Extension block count has changed.

2012-11-20 Thread Takashi Iwai
At Mon, 19 Nov 2012 15:23:06 -0500, "Egbert Eich " <"eich at novell.com> wrote: > > EEDID v1.3 mandates map blocks if more than one EDID extension block > is used while in v1.4 they are optional. > If the extension count has been changed (because some extension > blocks were not readable) those

Re: [PATCH 05/17] DRM/KMS/EDID: Fix up EEDID Map Blocks if Extension block count has changed.

2012-11-20 Thread Takashi Iwai
At Mon, 19 Nov 2012 15:23:06 -0500, Egbert Eich e...@novell.com wrote: EEDID v1.3 mandates map blocks if more than one EDID extension block is used while in v1.4 they are optional. If the extension count has been changed (because some extension blocks were not readable) those map blocks

Re: [PATCH 08/17] DRM/KMS/EDID: Use Extension Block Fixup Code also for 'firmware' EDID.

2012-11-20 Thread Takashi Iwai
At Mon, 19 Nov 2012 15:23:09 -0500, Egbert Eich e...@novell.com wrote: Signed-off-by: Egbert Eich e...@suse.de --- drivers/gpu/drm/drm_edid.c | 77 --- drivers/gpu/drm/drm_edid_load.c | 54 ++- include/drm/drm_edid.h

Re: [PATCH 07/17] DRM/KMS/EDID: Move drm_edid_load.o to drm.ko

2012-11-20 Thread Takashi Iwai
At Mon, 19 Nov 2012 15:23:08 -0500, Egbert Eich e...@novell.com wrote: EDIDs are an integral concept of connectors, connectors are a concept of drm core also drm_edid.o is already part of this drm core. Overridden, 'firmware-supplied' EDIDs should be treated exactly the same as device

Re: [PATCH 10/17] DRM/KMS/EDID: Cache EDID blobs with extensions.

2012-11-20 Thread Takashi Iwai
At Mon, 19 Nov 2012 15:23:11 -0500, Egbert Eich e...@novell.com wrote: According the the VESA specs there can be up to 254 EEDID extension blocks. Since we may read the EDID (including extensions) in 10 second intervals to probe for display hotplugging (at least in cases where no hardware

Re: [PATCH 13/17] DRM/KMS/EDID: Allow for multiple Connectors when specifying 'firmware'-EDID Files.

2012-11-20 Thread Takashi Iwai
At Mon, 19 Nov 2012 15:23:14 -0500, Egbert Eich e...@novell.com wrote: So far it was only possible to load an EDID for a single connector (unless no connector was specified at all in which case the same EDID file was used for all). This patch extends the EDID loader so that EDID files can

[RFC] drm dynamic power off support

2012-09-11 Thread Takashi Iwai
At Mon, 10 Sep 2012 18:50:04 +1000, Dave Airlie wrote: > > On Mon, Sep 10, 2012 at 6:47 PM, Takashi Iwai wrote: > > At Mon, 10 Sep 2012 15:04:02 +1000, > > Dave Airlie wrote: > >> > >> On Mon, Sep 10, 2012 at 2:31 PM, Dave Airlie wrote: > >> >

Re: [RFC] drm dynamic power off support

2012-09-11 Thread Takashi Iwai
At Mon, 10 Sep 2012 18:50:04 +1000, Dave Airlie wrote: On Mon, Sep 10, 2012 at 6:47 PM, Takashi Iwai ti...@suse.de wrote: At Mon, 10 Sep 2012 15:04:02 +1000, Dave Airlie wrote: On Mon, Sep 10, 2012 at 2:31 PM, Dave Airlie airl...@gmail.com wrote: For optimus and powerxpress setups

[RFC] drm dynamic power off support

2012-09-10 Thread Takashi Iwai
At Mon, 10 Sep 2012 15:04:02 +1000, Dave Airlie wrote: > > On Mon, Sep 10, 2012 at 2:31 PM, Dave Airlie wrote: > > For optimus and powerxpress setups where we can explicitly power off > > the GPU I'd like to do this under control of the driver. Now that I've > > added X server support for

Re: [RFC] drm dynamic power off support

2012-09-10 Thread Takashi Iwai
At Mon, 10 Sep 2012 15:04:02 +1000, Dave Airlie wrote: On Mon, Sep 10, 2012 at 2:31 PM, Dave Airlie airl...@gmail.com wrote: For optimus and powerxpress setups where we can explicitly power off the GPU I'd like to do this under control of the driver. Now that I've added X server support

[alsa-devel] [RFC] ASoC: snd_soc_jack for HDMI audio: does it make sense?

2012-08-24 Thread Takashi Iwai
At Thu, 23 Aug 2012 20:44:32 -0500, Ricardo Neri wrote: > > Hi Takashi, > > On 08/22/2012 02:55 AM, Takashi Iwai wrote: > > At Tue, 21 Aug 2012 19:58:02 -0500, > > Ricardo Neri wrote: > >> > >> > >> > >> On 08/21/2012 07:39 AM, C

Re: [alsa-devel] [RFC] ASoC: snd_soc_jack for HDMI audio: does it make sense?

2012-08-23 Thread Takashi Iwai
At Thu, 23 Aug 2012 20:44:32 -0500, Ricardo Neri wrote: Hi Takashi, On 08/22/2012 02:55 AM, Takashi Iwai wrote: At Tue, 21 Aug 2012 19:58:02 -0500, Ricardo Neri wrote: On 08/21/2012 07:39 AM, Clark, Rob wrote: On Tue, Aug 21, 2012 at 1:01 AM, Tomi Valkeinen tomi.valkei

[alsa-devel] [RFC] ASoC: snd_soc_jack for HDMI audio: does it make sense?

2012-08-22 Thread Takashi Iwai
At Tue, 21 Aug 2012 19:58:02 -0500, Ricardo Neri wrote: > > > > On 08/21/2012 07:39 AM, Clark, Rob wrote: > > On Tue, Aug 21, 2012 at 1:01 AM, Tomi Valkeinen > > wrote: > >> On Mon, 2012-08-20 at 20:47 -0500, Ricardo Neri wrote: > >>> Hello! > >>> > >>> I have been working on prototypes for

Re: [alsa-devel] [RFC] ASoC: snd_soc_jack for HDMI audio: does it make sense?

2012-08-22 Thread Takashi Iwai
At Tue, 21 Aug 2012 19:58:02 -0500, Ricardo Neri wrote: On 08/21/2012 07:39 AM, Clark, Rob wrote: On Tue, Aug 21, 2012 at 1:01 AM, Tomi Valkeinen tomi.valkei...@ti.com wrote: On Mon, 2012-08-20 at 20:47 -0500, Ricardo Neri wrote: Hello! I have been working on prototypes for the

[PATCH] drm: edid: Don't add inferred modes with higher resolution

2012-07-03 Thread Takashi Iwai
Signed-off-by: Takashi Iwai --- Dave, could you apply this as a regression fix for 3.5 kernel? drivers/gpu/drm/drm_edid.c | 27 --- 1 file changed, 24 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index 5873e48

Bogus video resolution in Linux 3.5-rc4

2012-07-03 Thread Takashi Iwai
At Mon, 02 Jul 2012 15:46:29 -0400, Adam Jackson wrote: > > On 6/26/12 3:21 AM, Takashi Iwai wrote: > > > From: Takashi Iwai > > Subject: [PATCH] drm: edid: Don't add inferred modes with higher resolution > > > > When a monitor EDID doesn't give

Re: Bogus video resolution in Linux 3.5-rc4

2012-07-03 Thread Takashi Iwai
At Mon, 02 Jul 2012 15:46:29 -0400, Adam Jackson wrote: On 6/26/12 3:21 AM, Takashi Iwai wrote: From: Takashi Iwai ti...@suse.de Subject: [PATCH] drm: edid: Don't add inferred modes with higher resolution When a monitor EDID doesn't give the preferred bit, driver assumes

[PATCH] drm: edid: Don't add inferred modes with higher resolution

2012-07-03 Thread Takashi Iwai
a...@redhat.com Signed-off-by: Takashi Iwai ti...@suse.de --- Dave, could you apply this as a regression fix for 3.5 kernel? drivers/gpu/drm/drm_edid.c | 27 --- 1 file changed, 24 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm

Bogus video resolution in Linux 3.5-rc4

2012-06-30 Thread Takashi Iwai
At Sat, 30 Jun 2012 13:46:54 -0500, Calvin Owens wrote: > > On 06/26/2012 02:21 AM, Takashi Iwai wrote: > > At Mon, 25 Jun 2012 21:38:56 +0200, > > Sven Joachim wrote: > >> > >> Am 25.06.2012 um 21:24 schrieb Takashi Iwai: > >> > >>>

Re: Bogus video resolution in Linux 3.5-rc4

2012-06-30 Thread Takashi Iwai
At Sat, 30 Jun 2012 13:46:54 -0500, Calvin Owens wrote: On 06/26/2012 02:21 AM, Takashi Iwai wrote: At Mon, 25 Jun 2012 21:38:56 +0200, Sven Joachim wrote: Am 25.06.2012 um 21:24 schrieb Takashi Iwai: And, does the patch below help? Somewhat: at least I get 1280x1024 again

Bogus video resolution in Linux 3.5-rc4

2012-06-26 Thread Takashi Iwai
At Mon, 25 Jun 2012 21:38:56 +0200, Sven Joachim wrote: > > Am 25.06.2012 um 21:24 schrieb Takashi Iwai: > > >> > And, does the patch below help? > >> > >> Somewhat: at least I get 1280x1024 again, but at 60 rather than 75 Hz. > > > >

Re: Bogus video resolution in Linux 3.5-rc4

2012-06-26 Thread Takashi Iwai
At Mon, 25 Jun 2012 21:38:56 +0200, Sven Joachim wrote: Am 25.06.2012 um 21:24 schrieb Takashi Iwai: And, does the patch below help? Somewhat: at least I get 1280x1024 again, but at 60 rather than 75 Hz. I guess it worked casually because 1280x1024@75 was the highest resolution

Bogus video resolution in Linux 3.5-rc4

2012-06-25 Thread Takashi Iwai
At Mon, 25 Jun 2012 19:40:48 +0200, Sven Joachim wrote: > > Am 25.06.2012 um 17:53 schrieb Takashi Iwai: > > > Looking at the EDID data, the problem is likely that your monitor > > doesn't give the proper preferred mode. > > What does xrandr output show? > >

Bogus video resolution in Linux 3.5-rc4

2012-06-25 Thread Takashi Iwai
At Mon, 25 Jun 2012 17:53:12 +0200, Takashi Iwai wrote: > > And, does the patch below help? BTW, the patch below contains the possible generic fix. It seems that EDID_QUIRK_FIRST_DETAILED_PREFERRED handling is missing from the beginning. So I wrote it just from what I can imagin

Bogus video resolution in Linux 3.5-rc4

2012-06-25 Thread Takashi Iwai
o the range descriptor itself. In any event, the range descriptor > can act as a filter on the DMT list without regard to a specific timing > formula. > > Signed-off-by: Adam Jackson > Tested-by: Takashi Iwai > Reviewed-by: Rodrigo Vivi > Signed-o

Re: Bogus video resolution in Linux 3.5-rc4

2012-06-25 Thread Takashi Iwai
on the DMT list without regard to a specific timing formula. Signed-off-by: Adam Jackson a...@redhat.com Tested-by: Takashi Iwai ti...@suse.de Reviewed-by: Rodrigo Vivi rodrigo.v...@intel.com Signed-off-by: Dave Airlie airl...@redhat.com diff --git a/drivers/gpu/drm

Re: Bogus video resolution in Linux 3.5-rc4

2012-06-25 Thread Takashi Iwai
At Mon, 25 Jun 2012 17:53:12 +0200, Takashi Iwai wrote: And, does the patch below help? BTW, the patch below contains the possible generic fix. It seems that EDID_QUIRK_FIRST_DETAILED_PREFERRED handling is missing from the beginning. So I wrote it just from what I can imagine from the comment

Re: Bogus video resolution in Linux 3.5-rc4

2012-06-25 Thread Takashi Iwai
At Mon, 25 Jun 2012 19:40:48 +0200, Sven Joachim wrote: Am 25.06.2012 um 17:53 schrieb Takashi Iwai: Looking at the EDID data, the problem is likely that your monitor doesn't give the proper preferred mode. What does xrandr output show? , | Screen 0: minimum 320 x 200, current

[alsa-devel] [PATCH 0/2] HD-audio HDMI regression fixes with VGA-switcheroo

2012-06-09 Thread Takashi Iwai
At Sat, 9 Jun 2012 08:57:20 +0100, Dave Airlie wrote: > > > Good to hear. > > Dave, is it OK to apply the patch below through sound tree? > > ack below, > > > > > > thanks, > > > > Takashi > > > > --- > > From: Takashi Iw

[PATCH 0/2] HD-audio HDMI regression fixes with VGA-switcheroo

2012-06-09 Thread Takashi Iwai
At Fri, 08 Jun 2012 22:17:29 +0200, J?rg-Volker Peetz wrote: > > Takashi Iwai wrote, on 06/08/12 17:52: > > At Fri, 08 Jun 2012 17:45:17 +0200, > > J?rg-Volker Peetz wrote: > >> > >> Hello Takashi, > >> > >> Takashi Iwai wrote, on 06/08/12 15

Re: [alsa-devel] [PATCH 0/2] HD-audio HDMI regression fixes with VGA-switcheroo

2012-06-09 Thread Takashi Iwai
At Sat, 9 Jun 2012 08:57:20 +0100, Dave Airlie wrote: Good to hear. Dave, is it OK to apply the patch below through sound tree? ack below, thanks, Takashi --- From: Takashi Iwai ti...@suse.de Subject: [PATCH] vga_switcheroo: Enable/disable audio clients at the right

[PATCH 0/2] HD-audio HDMI regression fixes with VGA-switcheroo

2012-06-08 Thread Takashi Iwai
At Fri, 08 Jun 2012 17:45:17 +0200, J?rg-Volker Peetz wrote: > > Hello Takashi, > > Takashi Iwai wrote, on 06/08/12 15:03: > > At Fri, 08 Jun 2012 13:26:57 +0200, > > J?rg-Volker Peetz wrote: > >> > >> Takashi Iwai wrote, on 06/07/12 12:15: > &g

[PATCH 0/2] HD-audio HDMI regression fixes with VGA-switcheroo

2012-06-08 Thread Takashi Iwai
At Fri, 08 Jun 2012 13:26:57 +0200, J?rg-Volker Peetz wrote: > > Takashi Iwai wrote, on 06/07/12 12:15: > > Hi, > > > > this is a series of patches to fix the regressions of HD-audio HDMI > > on D-GPUs in 3.5-rc1 due to the support of VGA-switcheroo audio clients.

[alsa-devel] [PATCH 1/2] vga_switcheroo: Add a helper function to get the client state

2012-06-08 Thread Takashi Iwai
At Fri, 8 Jun 2012 09:42:58 +0100, Dave Airlie wrote: > > On Thu, Jun 7, 2012 at 11:15 AM, Takashi Iwai wrote: > > Add vga_switcheroo_get_client_state() to get the current state of the > > client. ?This is necessary to determine the proper initial state of > > audio cl

Re: [alsa-devel] [PATCH 1/2] vga_switcheroo: Add a helper function to get the client state

2012-06-08 Thread Takashi Iwai
At Fri, 8 Jun 2012 09:42:58 +0100, Dave Airlie wrote: On Thu, Jun 7, 2012 at 11:15 AM, Takashi Iwai ti...@suse.de wrote: Add vga_switcheroo_get_client_state() to get the current state of the client.  This is necessary to determine the proper initial state of audio clients in HD-audio

Re: [PATCH 0/2] HD-audio HDMI regression fixes with VGA-switcheroo

2012-06-08 Thread Takashi Iwai
At Fri, 08 Jun 2012 13:26:57 +0200, Jörg-Volker Peetz wrote: Takashi Iwai wrote, on 06/07/12 12:15: Hi, this is a series of patches to fix the regressions of HD-audio HDMI on D-GPUs in 3.5-rc1 due to the support of VGA-switcheroo audio clients. The first patch adds a new helper

Re: [PATCH 0/2] HD-audio HDMI regression fixes with VGA-switcheroo

2012-06-08 Thread Takashi Iwai
At Fri, 08 Jun 2012 17:45:17 +0200, Jörg-Volker Peetz wrote: Hello Takashi, Takashi Iwai wrote, on 06/08/12 15:03: At Fri, 08 Jun 2012 13:26:57 +0200, Jörg-Volker Peetz wrote: Takashi Iwai wrote, on 06/07/12 12:15: Hi, this is a series of patches to fix the regressions of HD

[PATCH 2/2] ALSA: hda - Fix uninitialized HDMI controllers with VGA-switcheroo

2012-06-07 Thread Takashi Iwai
audio client state more correctly. Signed-off-by: Takashi Iwai --- sound/pci/hda/hda_intel.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c index 2b6392b..5f0375f 100644 --- a/sound/pci/hda/hda_intel.c +++ b/sound/pci

[PATCH 1/2] vga_switcheroo: Add a helper function to get the client state

2012-06-07 Thread Takashi Iwai
Add vga_switcheroo_get_client_state() to get the current state of the client. This is necessary to determine the proper initial state of audio clients in HD-audio driver. Signed-off-by: Takashi Iwai --- drivers/gpu/vga/vga_switcheroo.c | 13 + include/linux/vga_switcheroo.h

[PATCH 0/2] HD-audio HDMI regression fixes with VGA-switcheroo

2012-06-07 Thread Takashi Iwai
Hi, this is a series of patches to fix the regressions of HD-audio HDMI on D-GPUs in 3.5-rc1 due to the support of VGA-switcheroo audio clients. The first patch adds a new helper function to vga-switcheroo and the second just uses that instead of an open code. Dave, if the first patch is OK,

[PATCH 0/2] HD-audio HDMI regression fixes with VGA-switcheroo

2012-06-07 Thread Takashi Iwai
Hi, this is a series of patches to fix the regressions of HD-audio HDMI on D-GPUs in 3.5-rc1 due to the support of VGA-switcheroo audio clients. The first patch adds a new helper function to vga-switcheroo and the second just uses that instead of an open code. Dave, if the first patch is OK,

[PATCH 1/2] vga_switcheroo: Add a helper function to get the client state

2012-06-07 Thread Takashi Iwai
Add vga_switcheroo_get_client_state() to get the current state of the client. This is necessary to determine the proper initial state of audio clients in HD-audio driver. Signed-off-by: Takashi Iwai ti...@suse.de --- drivers/gpu/vga/vga_switcheroo.c | 13 + include/linux

[PATCH 2/2] ALSA: hda - Fix uninitialized HDMI controllers with VGA-switcheroo

2012-06-07 Thread Takashi Iwai
audio client state more correctly. Signed-off-by: Takashi Iwai ti...@suse.de --- sound/pci/hda/hda_intel.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c index 2b6392b..5f0375f 100644 --- a/sound/pci/hda/hda_intel.c

[PATCH] drm: Skip too big EDID extensions

2012-06-05 Thread Takashi Iwai
a hardware-specific problem, but it's anyway safer to avoid such almost unrealistic setups. Signed-off-by: Takashi Iwai --- drivers/gpu/drm/drm_edid.c |7 +++ 1 file changed, 7 insertions(+) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index eb92fe2..61e4cf1 100644 ---

[PATCH] drm: Skip too big EDID extensions

2012-06-05 Thread Takashi Iwai
-specific problem, but it's anyway safer to avoid such almost unrealistic setups. Signed-off-by: Takashi Iwai ti...@suse.de --- drivers/gpu/drm/drm_edid.c |7 +++ 1 file changed, 7 insertions(+) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index eb92fe2..61e4cf1 100644

[PATCH 3/3] vga_switcheroo: Add the support for audio clients

2012-05-11 Thread Takashi Iwai
-off-by: Takashi Iwai --- drivers/gpu/vga/vga_switcheroo.c | 74 ++ include/linux/vga_switcheroo.h |6 2 files changed, 66 insertions(+), 14 deletions(-) diff --git a/drivers/gpu/vga/vga_switcheroo.c b/drivers/gpu/vga/vga_switcheroo.c index 5f176b0

[PATCH 2/3] vga_switcheroo: Introduce struct vga_switcheroo_client_ops

2012-05-11 Thread Takashi Iwai
This changes the API as a clean-up. Instead of passing multiple function pointers at each time, introduce a new struct holding the whole callback functions and pass it to the registration. The same struct will be used for the upcoming audio client registration, too. Signed-off-by: Takashi Iwai

[PATCH 1/3] vga_switcheroo: Refactor using linked list

2012-05-11 Thread Takashi Iwai
Refactor the code base a bit for the further work to adapt more clients. Signed-off-by: Takashi Iwai --- drivers/gpu/vga/vga_switcheroo.c | 209 -- 1 file changed, 110 insertions(+), 99 deletions(-) diff --git a/drivers/gpu/vga/vga_switcheroo.c b/drivers

[PATCH 0/3] Audio client support for VGA-switcheroo (v2)

2012-05-11 Thread Takashi Iwai
Hi, here is the updated patches including the conversion to struct vga_switcheroo_client_ops. The patches can be pulled from git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git topic/vga-switcheroo [background info if anyone hasn't read v1 patch series: this is for allowing to

[PATCH 0/3] Audio client support for VGA-switcheroo (v2)

2012-05-11 Thread Takashi Iwai
Hi, here is the updated patches including the conversion to struct vga_switcheroo_client_ops. The patches can be pulled from git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git topic/vga-switcheroo [background info if anyone hasn't read v1 patch series: this is for allowing to

[PATCH 2/3] vga_switcheroo: Introduce struct vga_switcheroo_client_ops

2012-05-11 Thread Takashi Iwai
This changes the API as a clean-up. Instead of passing multiple function pointers at each time, introduce a new struct holding the whole callback functions and pass it to the registration. The same struct will be used for the upcoming audio client registration, too. Signed-off-by: Takashi Iwai

[PATCH 3/3] vga_switcheroo: Add the support for audio clients

2012-05-11 Thread Takashi Iwai
-off-by: Takashi Iwai ti...@suse.de --- drivers/gpu/vga/vga_switcheroo.c | 74 ++ include/linux/vga_switcheroo.h |6 2 files changed, 66 insertions(+), 14 deletions(-) diff --git a/drivers/gpu/vga/vga_switcheroo.c b/drivers/gpu/vga/vga_switcheroo.c

[PATCH 1/3] vga_switcheroo: Refactor using linked list

2012-05-11 Thread Takashi Iwai
Refactor the code base a bit for the further work to adapt more clients. Signed-off-by: Takashi Iwai ti...@suse.de --- drivers/gpu/vga/vga_switcheroo.c | 209 -- 1 file changed, 110 insertions(+), 99 deletions(-) diff --git a/drivers/gpu/vga/vga_switcheroo.c

[PATCH 2/2] vga_switcheroo: Add the support for audio clients

2012-05-10 Thread Takashi Iwai
At Thu, 10 May 2012 20:05:25 +0100, Dave Airlie wrote: > > On Thu, May 10, 2012 at 7:42 PM, Takashi Iwai wrote: > > At Thu, 10 May 2012 13:42:09 +0200, > > Takashi Iwai wrote: > >> > >> At Thu, 10 May 2012 12:20:05 +0100, > >> Alan Cox wrote: >

[PATCH 2/2] vga_switcheroo: Add the support for audio clients

2012-05-10 Thread Takashi Iwai
At Thu, 10 May 2012 13:42:09 +0200, Takashi Iwai wrote: > > At Thu, 10 May 2012 12:20:05 +0100, > Alan Cox wrote: > > > > On Thu, 10 May 2012 09:10:16 +0200 > > Takashi Iwai wrote: > > > > > Add the support for audio clients to VGA-switcheroo fo

[PATCH 2/2] vga_switcheroo: Add the support for audio clients

2012-05-10 Thread Takashi Iwai
At Thu, 10 May 2012 11:40:50 +0100, Dave Airlie wrote: > > On Thu, May 10, 2012 at 8:10 AM, Takashi Iwai wrote: > > Add the support for audio clients to VGA-switcheroo for handling the > > HDMI audio controller together with VGA switching. ?The id of the > > audio co

[PATCH 2/2] vga_switcheroo: Add the support for audio clients

2012-05-10 Thread Takashi Iwai
At Thu, 10 May 2012 12:20:05 +0100, Alan Cox wrote: > > On Thu, 10 May 2012 09:10:16 +0200 > Takashi Iwai wrote: > > > Add the support for audio clients to VGA-switcheroo for handling the > > HDMI audio controller together with VGA switching. The id of the > > au

[PATCH 0/2] Audio client support for VGA-switcheroo

2012-05-10 Thread Takashi Iwai
At Thu, 10 May 2012 09:08:38 +0200, Takashi Iwai wrote: > > Dave, > > this is a small patch series to add the support for audio clients > to VGA switcheroo. The background problem is that the HD-audio HDMI > controller of the discrete GPU is also disabled when the graphics

[PATCH 2/2] vga_switcheroo: Add the support for audio clients

2012-05-10 Thread Takashi Iwai
-off-by: Takashi Iwai --- drivers/gpu/vga/vga_switcheroo.c | 84 ++ include/linux/vga_switcheroo.h | 10 + 2 files changed, 78 insertions(+), 16 deletions(-) diff --git a/drivers/gpu/vga/vga_switcheroo.c b/drivers/gpu/vga/vga_switcheroo.c index c91573f

[PATCH 1/2] vga_switcheroo: Refactor using linked list

2012-05-10 Thread Takashi Iwai
Refactor the code base a bit for the further work to adapt more clients. Signed-off-by: Takashi Iwai --- drivers/gpu/vga/vga_switcheroo.c | 209 -- 1 file changed, 110 insertions(+), 99 deletions(-) diff --git a/drivers/gpu/vga/vga_switcheroo.c b/drivers

<    1   2   3   4   5   6   7   >