From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: John Stultz <john.stultz@linaro.org>
Cc: lkml <linux-kernel@vger.kernel.org>,
dri-devel@lists.freedesktop.org,
Wolfram Sang <wsa+renesas@sang-engineering.com>
Subject: Re: [PATCH 0/6 v4] adv7511 EDID probing improvements
Date: Thu, 19 Jan 2017 01:04:02 +0200 [thread overview]
Message-ID: <3859476.cEhDXg5Hko@avalon> (raw)
In-Reply-To: <1484614372-15342-1-git-send-email-john.stultz@linaro.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
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
WARNING: multiple messages have this Message-ID (diff)
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 [thread overview]
Message-ID: <3859476.cEhDXg5Hko@avalon> (raw)
In-Reply-To: <1484614372-15342-1-git-send-email-john.stultz@linaro.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
next prev parent reply other threads:[~2017-01-18 23:03 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-01-17 0:52 [PATCH 0/6 v4] adv7511 EDID probing improvements John Stultz
2017-01-17 0:52 ` John Stultz
2017-01-17 0:52 ` [PATCH 1/6] drm/bridge: adv7511: Use work_struct to defer hotplug handing to out of irq context John Stultz
2017-01-17 0:52 ` John Stultz
2017-01-17 0:52 ` [PATCH 2/6] drm/bridge: adv7511: Switch to using drm_kms_helper_hotplug_event() John Stultz
2017-01-17 0:52 ` John Stultz
2017-01-17 0:52 ` [PATCH 3/6] drm/bridge: adv7511: Enable HPD interrupts to support hotplug and improve monitor detection John Stultz
2017-01-17 0:52 ` John Stultz
2017-01-17 0:52 ` [PATCH 4/6] drm/bridge: adv7511: Rework adv7511_power_on/off() so they can be reused internally John Stultz
2017-01-17 0:52 ` John Stultz
2017-01-18 23:04 ` Laurent Pinchart
2017-01-18 23:04 ` Laurent Pinchart
2017-01-17 0:52 ` [PATCH 5/6] drm/bridge: adv7511: Reuse __adv7511_power_on/off() when probing EDID John Stultz
2017-01-17 0:52 ` John Stultz
2017-01-18 23:05 ` Laurent Pinchart
2017-01-18 23:05 ` Laurent Pinchart
2017-01-17 0:52 ` [PATCH 6/6] drm/bridge: adv7511: Re-write the i2c address before EDID probing John Stultz
2017-01-17 0:52 ` John Stultz
2017-01-18 23:06 ` Laurent Pinchart
2017-01-18 23:06 ` Laurent Pinchart
2017-01-18 23:04 ` Laurent Pinchart [this message]
2017-01-18 23:04 ` [PATCH 0/6 v4] adv7511 EDID probing improvements Laurent Pinchart
2017-01-19 5:22 ` Archit Taneja
2017-01-19 5:22 ` Archit Taneja
2017-01-19 7:37 ` John Stultz
2017-01-19 7:37 ` John Stultz
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=3859476.cEhDXg5Hko@avalon \
--to=laurent.pinchart@ideasonboard.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=john.stultz@linaro.org \
--cc=linux-kernel@vger.kernel.org \
--cc=wsa+renesas@sang-engineering.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.