From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1751213AbdARXEy (ORCPT <rfc822;w@1wt.eu>);
        Wed, 18 Jan 2017 18:04:54 -0500
Received: from galahad.ideasonboard.com ([185.26.127.97]:45592 "EHLO
        galahad.ideasonboard.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S1750992AbdARXEx (ORCPT
        <rfc822;linux-kernel@vger.kernel.org>);
        Wed, 18 Jan 2017 18:04:53 -0500
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: John Stultz <john.stultz@linaro.org>
Cc: lkml <linux-kernel@vger.kernel.org>, David Airlie <airlied@linux.ie>,
        Archit Taneja <architt@codeaurora.org>,
        Wolfram Sang <wsa+renesas@sang-engineering.com>,
        Lars-Peter Clausen <lars@metafoo.de>, dri-devel@lists.freedesktop.org
Subject: Re: [PATCH 0/6 v4] adv7511 EDID probing improvements
Date: Thu, 19 Jan 2017 01:04:02 +0200
Message-ID: <3859476.cEhDXg5Hko@avalon>
User-Agent: KMail/4.14.10 (Linux/4.8.6-gentoo; KDE/4.14.24; x86_64; ; )
In-Reply-To: <1484614372-15342-1-git-send-email-john.stultz@linaro.org>
References: <1484614372-15342-1-git-send-email-john.stultz@linaro.org>
MIME-Version: 1.0
Content-Transfer-Encoding: 7Bit
Content-Type: text/plain; charset="us-ascii"
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi John,

Thank you for the patches.

On Monday 16 Jan 2017 16:52:46 John Stultz wrote:
> Wanted to re-send out v4  of this patch set, integrating some
> changes suggested by Laurent, for consideration for merging for
> v4.11
> 
> The first three patches are fixups that are hopefully straight
> forward, integrating feedback I got from Laurant, with minimal
> change from the previous versions.
> 
> The last three patches try to clean up and rework the EDID
> probing code, to avoid issues seen on HiKey. I've reworked these
> more significantly since v3 to address feedback from Laurent.
> 
> Thoughts and feedback would be appreciated!

For the whole series,

Tested-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

> New in v4:
> * Tweaked connector.status assignment to avoid race, as
>   suggested by Laurent
> * Reworked the __adv7511_power_on helpers to avoid calling
>   regcache_sync in the EDID probe path
> * Added new patch to set EDID_I2C_ADDR register before doing
>   EDID read.
> 
> Cc: David Airlie <airlied@linux.ie>
> Cc: Archit Taneja <architt@codeaurora.org>
> Cc: Wolfram Sang <wsa+renesas@sang-engineering.com>
> Cc: Lars-Peter Clausen <lars@metafoo.de>
> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> Cc: dri-devel@lists.freedesktop.org
> 
> Archit Taneja (1):
>   drm/bridge: adv7511: Enable HPD interrupts to support hotplug and
>     improve monitor detection
> 
> John Stultz (5):
>   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
>   drm/bridge: adv7511: Re-write the i2c address before EDID probing
> 
>  drivers/gpu/drm/bridge/adv7511/adv7511.h     |  2 +
>  drivers/gpu/drm/bridge/adv7511/adv7511_drv.c | 67
> ++++++++++++++++++++-------- 2 files changed, 51 insertions(+), 18
> deletions(-)

-- 
Regards,

Laurent Pinchart