[RFC][PATCH 0/5 v3] adv7511 EDID probing improvements

Path	csiph.com!eternal-september.org!feeder.eternal-september.org!aioe.org!bofh.it!news.nic.it!robomod
From	John Stultz <john.stultz@linaro.org>
Newsgroups	linux.kernel
Subject	[RFC][PATCH 0/5 v3] adv7511 EDID probing improvements
Date	Mon, 12 Dec 2016 22:10:02 +0100
Message-ID	<sNG1b-R8-5@gated-at.bofh.it> (permalink)
Dkim-Signature	v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id; bh=gcbCjmbK18/tdcXkBbvnHLnUiL04CGbOEQxHCjKn7/Y=; b=LGrKfIK4UITq15kxZS1eJt+7o7RgEGxzZtHfnnvWKUR1qC3l+n1HM5mlsGIFHRLlH9 mxO57aLJV/3UckhpwbRW5JnTJlOrg6gbsBcF5b3qi0KdwBgtK2MLlQfW9NNRODTm/79k /OF9MeibjzNi5z3gs2QxD8qdqlwJ4UMZFt/yw=
X-Google-Dkim-Signature	v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=gcbCjmbK18/tdcXkBbvnHLnUiL04CGbOEQxHCjKn7/Y=; b=fCz+x49aVkJXWBSbFE2/stv+waBW+TFGIezlH8MAwJsMafnXf2hDdLMNvrJkxsCW7j DulAOaTiHnnWOqzAqP4kDYPGOh3v2Sg5jLegto8rbL+DaiuMBCnC1lZU+uwvqX0PTtbW IBaPyfkM4h7IKmESsJg8/OsTwb8RHHAOTmp3yC08HoFr2Ljt2Myr4LQJFDUWazuWMcRz YHggaaFXPsjgM61ZW5NSIbZy6ystWyBmfk1cOKlU30gGomznTmcYmZirqGCTYgdQiEoE 4U/h71WGgJXRVAq5TZKOTrnouC49evaLUpgJDNqi5567oOSx6Cy6l0oDBu+x7CP+iW// YAUQ==
X-Gm-Message-State	AKaTC00d7E9PxStnb+j8OKY7ViwJAZ8Nt/Wg2bYSYzegTA4/2OXFbLCByjj5S2OmXQV4TG/M
X-Received	by 10.99.192.69 with SMTP id z5mr171092669pgi.169.1481576445606; Mon, 12 Dec 2016 13:00:45 -0800 (PST)
X-Mailer	git-send-email 2.7.4
Sender	robomod@news.nic.it
List-ID	<linux-kernel.vger.kernel.org>
X-Mailing-List	linux-kernel@vger.kernel.org
Approved	robomod@news.nic.it
Lines	47
Organization	linux.* mail to news gateway
X-Original-Cc	John Stultz <john.stultz@linaro.org>, David Airlie <airlied@linux.ie>, Archit Taneja <architt@codeaurora.org>, Wolfram Sang <wsa+renesas@sang-engineering.com>, Lars-Peter Clausen <lars@metafoo.de>, Laurent Pinchart <laurent.pinchart@ideasonboard.com>, dri-devel@lists.freedesktop.org
X-Original-Date	Mon, 12 Dec 2016 13:00:36 -0800
X-Original-Message-ID	<1481576441-23529-1-git-send-email-john.stultz@linaro.org>
X-Original-Sender	linux-kernel-owner@vger.kernel.org
Xref	csiph.com linux.kernel:1540663

Show key headers only | View raw

Wanted to send out v3 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.

The last two patches try to clean up and resue code to avoid
an issue I'm seeing where something is 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!

thanks
-john

New in v3:
* Addressed naming improvements and drm_kms_helper_hotplug_event
  usage corrections as suggested by Laurent.

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 (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 | 62 +++++++++++++++++++---------
 2 files changed, 44 insertions(+), 20 deletions(-)

-- 
2.7.4

Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread

Thread

[RFC][PATCH 0/5 v3] adv7511 EDID probing improvements John Stultz <john.stultz@linaro.org> - 2016-12-12 22:10 +0100
  [RFC][PATCH 1/5 v3] drm/bridge: adv7511: Use work_struct to defer hotplug handing to out of irq context John Stultz <john.stultz@linaro.org> - 2016-12-12 22:10 +0100

csiph-web