From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Subject: Re: [BISECTED, -next] drm/i915: blurred HDMI output Date: Mon, 18 Oct 2010 22:30:17 +0200 Message-ID: <201010182230.17669.arnd@arndb.de> References: <201010171937.13642.arnd@arndb.de> <201010182157.05339.arnd@arndb.de> <20101018131523.19f71ca3@jbarnes-desktop> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from moutng.kundenserver.de (moutng.kundenserver.de [212.227.126.171]) by gabe.freedesktop.org (Postfix) with ESMTP id 432219E774 for ; Mon, 18 Oct 2010 13:30:21 -0700 (PDT) In-Reply-To: <20101018131523.19f71ca3@jbarnes-desktop> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org Errors-To: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org To: Jesse Barnes Cc: Jason Wessel , dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org On Monday 18 October 2010 22:15:23 Jesse Barnes wrote: > On Mon, 18 Oct 2010 21:57:05 +0200 Arnd Bergmann wrote: > > i915 is a module with kms enabled, the blur seems to show up on the > > text console the moment that the module gets loaded during boot. > > Before that, the console is in VGA text mode, which is always > > slightly blurred because the screen has a different native resolution. > > Arg why would the BIOS set the panel fitting bit at all in this > config? Any attached outputs (VGA, HDMI) will have their own scalers... I didn't think the BIOS was setting it. Text mode appears to be blurred differently, but that's hard to tell... > Obviously we need to clear it somewhere, but we should be able to avoid > clearing it on every mode set. We should probably shut down all panel > related bits in intel_lvds.c if we fail to detect an LVDS output; that > should save a little power and avoid problems like this. > > Maybe the below patch which does that will help? No luck here either. I don't think the code path you patch here actually gets used, since intel_lvds_init gets called by intel_setup_outputs only for mobile devices. Arnd