Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932172AbcLLIkx (ORCPT ); Mon, 12 Dec 2016 03:40:53 -0500 Received: from smtp.codeaurora.org ([198.145.29.96]:33772 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751068AbcLLIkv (ORCPT ); Mon, 12 Dec 2016 03:40:51 -0500 DMARC-Filter: OpenDMARC Filter v1.3.1 smtp.codeaurora.org 391CC612E7 Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=pass smtp.mailfrom=architt@codeaurora.org Subject: Re: [RFC][PATCH 0/5 v2] adv7511 EDID probing improvements To: John Stultz , lkml References: <1480395884-5471-1-git-send-email-john.stultz@linaro.org> Cc: David Airlie , Wolfram Sang , Lars-Peter Clausen , Laurent Pinchart , dri-devel@lists.freedesktop.org From: Archit Taneja Message-ID: <10bc366d-9955-1f03-7136-9fb05eee943f@codeaurora.org> Date: Mon, 12 Dec 2016 14:10:44 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 MIME-Version: 1.0 In-Reply-To: <1480395884-5471-1-git-send-email-john.stultz@linaro.org> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2028 Lines: 57 Hi, On 11/29/2016 10:34 AM, John Stultz wrote: > Wanted to send out v2 of this patch set improving the EDID > probing on the adv7511 used on HiKey. > > The first three patches are fixups that are hopefully straight > forward, integrating feedback I got from Laurant. > > One of the previous patches that Laurant had concerns about, I > broke into two patches, which are the last two in this series. > The core issue seems to be something going wrong with the regmap > cache state for the ADV7511_REG_EDID_I2C_ADDR (0x43) register > which results in i2c_transfer errors if we don't do the > regcache_sync/_mark_dirty() calls. I suspect there might be a > better solution there, so suggestions will be very welcome. > > Thoughts and feedback would be appreciated! I tested this for ADV7533 on DB410c and it works well. We can pull this if it works fine for ADV7511 too. Laurent, could we get an Ack for the series from you? Thanks, Archit > > thanks > -john > > Cc: David Airlie > Cc: Archit Taneja > Cc: Wolfram Sang > Cc: Lars-Peter Clausen > Cc: Laurent Pinchart > Cc: dri-devel@lists.freedesktop.org > > Archit Taneja (1): > drm/bridge: adv7511: Enable HPD interrupts to support hotplug and > improve monitor detection > > John Stultz (4): > drm/bridge: adv7511: Use work_struct to defer hotplug handing to out > of irq context > drm/bridge: adv7511: Switch to using drm_kms_helper_hotplug_event() > drm/bridge: adv7511: Rework adv7511_power_on/off() so they can be > reused internally > drm/bridge: adv7511: Reuse __adv7511_power_on/off() when probing EDID > > drivers/gpu/drm/bridge/adv7511/adv7511.h | 2 ++ > drivers/gpu/drm/bridge/adv7511/adv7511_drv.c | 49 ++++++++++++++++------------ > 2 files changed, 31 insertions(+), 20 deletions(-) > -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project