From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Jackson Subject: Re: [PATCH] drm: i2c: tda998x: Retry fetching the EDID if it fails first time. Date: Fri, 28 Nov 2014 09:02:39 +0000 Message-ID: <54783A2F.5010405@arm.com> References: <546B84C6.3090809@arm.com> <20141128091905.3e491c39@armhf> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20141128091905.3e491c39@armhf> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Jean-Francois Moine Cc: Russell King - ARM Linux , Liviu Dudau , "linux-kernel@vger.kernel.org" , "dri-devel@lists.freedesktop.org" , Dave Airlie , "linux-arm-kernel@lists.infradead.org" List-Id: dri-devel@lists.freedesktop.org On 11/28/14 08:19, Jean-Francois Moine wrote: > On Tue, 18 Nov 2014 17:41:26 +0000 > Andrew Jackson wrote: > >> Fetching the EDID from a connected monitor is an automated thing >> with NXP TDA19988. But on some boards the fetching fails for the >> first time silently without any indication that an error has occured. >> More than that, subsequent fetches of the EDID succeed until the >> monitor(s) are unplugged. >> >> Add a function to validate the read EDID and retry if the block >> retrieved is not valid. >> >> Signed-off-by: Andrew Jackson >> Signed-off-by: Liviu Dudau > > It seems that your patch is deprecated by Laurent Pinchart's > [PATCH] drm: tda998x: Use drm_do_get_edid() > http://lists.freedesktop.org/archives/dri-devel/2014-November/072906.html > Thank you for the heads-up, I'd not seen that. I'll consider how my patch might be modified to suit the new infrastructure. Andrew