Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752443AbaAMRqm (ORCPT ); Mon, 13 Jan 2014 12:46:42 -0500 Received: from gw-1.arm.linux.org.uk ([78.32.30.217]:35827 "EHLO pandora.arm.linux.org.uk" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751316AbaAMRqk (ORCPT ); Mon, 13 Jan 2014 12:46:40 -0500 Date: Mon, 13 Jan 2014 17:46:31 +0000 From: Russell King - ARM Linux To: Jean-Francois Moine Cc: Rob Clark , dri-devel@lists.freedesktop.org, Dave Airlie , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v2 15/28] drm/i2c: tda998x: use the tda998x video format when cea mode Message-ID: <20140113174631.GC15937@n2100.arm.linux.org.uk> References: <20140109120448.24d89eba@armhf> <20140111181832.GD15937@n2100.arm.linux.org.uk> <20140113170707.10b908dc@armhf> <20140113161333.GA15937@n2100.arm.linux.org.uk> <20140113172220.521917ce@armhf> <20140113163501.GB15937@n2100.arm.linux.org.uk> <20140113182455.1f8c3f40@armhf> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20140113182455.1f8c3f40@armhf> User-Agent: Mutt/1.5.19 (2009-01-05) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Jan 13, 2014 at 06:24:55PM +0100, Jean-Francois Moine wrote: > On Mon, 13 Jan 2014 16:35:01 +0000 > Russell King - ARM Linux wrote: > > > Ah, I see. It's a thread about your DRM driver, and about the lack of > > audio on 1080 resolutions. Taking it in context, the reported issue > > appears to be lack of video when 1080p is selected, but the display > > doesn't support 1080p. That's hardly surprising. > > > > Then rabeeh posts his message about lack of audio at 1920x1080. > > > > Well, I can say that with the tda998x as it stands, the 1080* modes > > work with audio with armada-drm, so there's no need to change this. > > My DRM driver also has audio with and without the cea mode for all > resolutions. At the time of Rabeeh's message, I just had forgot to > switch the audio pins... > > So, TDA998x CEA mode or not? Firstly, why change something which isn't broken? Secondly, you need to justify this change by explaining why it is beneficial to use the internally programmed modes rather than programming the timings to match the adjusted settings from DRM. Thirdly, one of your previous patches changed the programming to use the adjusted mode settings, which may be the CEA mode with slight tweaks. Have you considered the implication of the hardware generating the video being programmed with those tweaks vs the encoder using a fixed set of unadjusted timings? I'm not convinced that we need this patch at the present time, and I'm not convinced that we need to introduce a load of tables for no measurable gain. -- FTTC broadband for 0.8mile line: 5.8Mbps down 500kbps up. Estimation in database were 13.1 to 19Mbit for a good line, about 7.5+ for a bad. Estimate before purchase was "up to 13.2Mbit". -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/