From: Sean Paul <seanpaul@chromium.org> To: Andrzej Hajda <a.hajda@samsung.com> Cc: zain wang <wzz@rock-chips.com>, Lin Huang <hl@rock-chips.com>, Tomeu Vizoso <tomeu.vizoso@collabora.com>, Douglas Anderson <dianders@chromium.org>, dri-devel@lists.freedesktop.org, Tomasz Figa <tfiga@chromium.org>, linux-rockchip@lists.infradead.org, Krzysztof Kozlowski <krzk@kernel.org>, Yakir Yang <ykk@rock-chips.com>, linux-arm-kernel@lists.infradead.org, Caesar Wang <wxt@rock-chips.com> Subject: Re: [PATCH 10/41] drm/bridge: analogix_dp: Don't change psr while bridge is disabled Date: Tue, 21 Mar 2017 15:58:18 -0400 [thread overview] Message-ID: <20170321195818.GE19389@art_vandelay> (raw) In-Reply-To: <e7b7ef5b-7935-a74c-c046-27c56ae16774@samsung.com> On Thu, Mar 16, 2017 at 02:40:21PM +0100, Andrzej Hajda wrote: > On 10.03.2017 05:32, Sean Paul wrote: > > From: zain wang <wzz@rock-chips.com> > > > > There is a race between AUX CH bring-up and enabling bridge which will > > cause link training to fail. To avoid hitting it, don't change psr state > > while enabling the bridge. > > > > Cc: Tomeu Vizoso <tomeu.vizoso@collabora.com> > > Cc: Sean Paul <seanpaul@chromium.org> > > Signed-off-by: zain wang <wzz@rock-chips.com> > > Signed-off-by: Caesar Wang <wxt@rock-chips.com> > > [seanpaul fixed up the commit message a bit and renamed *_supported to *_enabled] > > Signed-off-by: Sean Paul <seanpaul@chromium.org> > > Hmm, beside resetting psr_enable in analogix_dp_bridge_disable I do not > see functional change, or am I blind? The patch also adds a check of psr_enable in analogix_dp_commit. This will avoid trying to enable psr when the bridge is disabled (that's why psr_enable is reset). Sean > > Regards > Andrzej > > > --- > > drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 15 ++++++++------- > > drivers/gpu/drm/bridge/analogix/analogix_dp_core.h | 2 +- > > drivers/gpu/drm/rockchip/analogix_dp-rockchip.c | 2 +- > > include/drm/bridge/analogix_dp.h | 2 +- > > 4 files changed, 11 insertions(+), 10 deletions(-) > > > > diff --git a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c b/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c > > index 8a8f05fe9da3..64d94a34874d 100644 > > --- a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c > > +++ b/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c > > @@ -98,20 +98,20 @@ static int analogix_dp_detect_hpd(struct analogix_dp_device *dp) > > return 0; > > } > > > > -int analogix_dp_psr_supported(struct device *dev) > > +int analogix_dp_psr_enabled(struct device *dev) > > { > > struct analogix_dp_device *dp = dev_get_drvdata(dev); > > > > - return dp->psr_support; > > + return dp->psr_enable; > > } > > -EXPORT_SYMBOL_GPL(analogix_dp_psr_supported); > > +EXPORT_SYMBOL_GPL(analogix_dp_psr_enabled); > > > > int analogix_dp_enable_psr(struct device *dev) > > { > > struct analogix_dp_device *dp = dev_get_drvdata(dev); > > struct edp_vsc_psr psr_vsc; > > > > - if (!dp->psr_support) > > + if (!dp->psr_enable) > > return 0; > > > > /* Prepare VSC packet as per EDP 1.4 spec, Table 6.9 */ > > @@ -134,7 +134,7 @@ int analogix_dp_disable_psr(struct device *dev) > > struct edp_vsc_psr psr_vsc; > > int ret; > > > > - if (!dp->psr_support) > > + if (!dp->psr_enable) > > return 0; > > > > /* Prepare VSC packet as per EDP 1.4 spec, Table 6.9 */ > > @@ -878,8 +878,8 @@ static void analogix_dp_commit(struct analogix_dp_device *dp) > > /* Enable video */ > > analogix_dp_start_video(dp); > > > > - dp->psr_support = analogix_dp_detect_sink_psr(dp); > > - if (dp->psr_support) > > + dp->psr_enable = analogix_dp_detect_sink_psr(dp); > > + if (dp->psr_enable) > > analogix_dp_enable_sink_psr(dp); > > } > > > > @@ -1120,6 +1120,7 @@ static void analogix_dp_bridge_disable(struct drm_bridge *bridge) > > if (ret) > > DRM_ERROR("failed to setup the panel ret = %d\n", ret); > > > > + dp->psr_enable = false; > > dp->dpms_mode = DRM_MODE_DPMS_OFF; > > } > > > > diff --git a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.h b/drivers/gpu/drm/bridge/analogix/analogix_dp_core.h > > index b039b28d8fcc..e135a42cb19e 100644 > > --- a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.h > > +++ b/drivers/gpu/drm/bridge/analogix/analogix_dp_core.h > > @@ -170,7 +170,7 @@ struct analogix_dp_device { > > int dpms_mode; > > int hpd_gpio; > > bool force_hpd; > > - bool psr_support; > > + bool psr_enable; > > > > struct mutex panel_lock; > > bool panel_is_modeset; > > diff --git a/drivers/gpu/drm/rockchip/analogix_dp-rockchip.c b/drivers/gpu/drm/rockchip/analogix_dp-rockchip.c > > index 64e7e2c0bc58..f44756029478 100644 > > --- a/drivers/gpu/drm/rockchip/analogix_dp-rockchip.c > > +++ b/drivers/gpu/drm/rockchip/analogix_dp-rockchip.c > > @@ -83,7 +83,7 @@ static void analogix_dp_psr_set(struct drm_encoder *encoder, bool enabled) > > int vact_end; > > int ret; > > > > - if (!analogix_dp_psr_supported(dp->dev)) > > + if (!analogix_dp_psr_enabled(dp->dev)) > > return; > > > > dev_dbg(dp->dev, "%s PSR...\n", enabled ? "enable" : "disable"); > > diff --git a/include/drm/bridge/analogix_dp.h b/include/drm/bridge/analogix_dp.h > > index c99d6eaef1ac..4fc0165ed3f5 100644 > > --- a/include/drm/bridge/analogix_dp.h > > +++ b/include/drm/bridge/analogix_dp.h > > @@ -38,7 +38,7 @@ struct analogix_dp_plat_data { > > struct drm_connector *); > > }; > > > > -int analogix_dp_psr_supported(struct device *dev); > > +int analogix_dp_psr_enabled(struct device *dev); > > int analogix_dp_enable_psr(struct device *dev); > > int analogix_dp_disable_psr(struct device *dev); > > > -- Sean Paul, Software Engineer, Google / Chromium OS _______________________________________________ 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: seanpaul@chromium.org (Sean Paul) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH 10/41] drm/bridge: analogix_dp: Don't change psr while bridge is disabled Date: Tue, 21 Mar 2017 15:58:18 -0400 [thread overview] Message-ID: <20170321195818.GE19389@art_vandelay> (raw) In-Reply-To: <e7b7ef5b-7935-a74c-c046-27c56ae16774@samsung.com> On Thu, Mar 16, 2017 at 02:40:21PM +0100, Andrzej Hajda wrote: > On 10.03.2017 05:32, Sean Paul wrote: > > From: zain wang <wzz@rock-chips.com> > > > > There is a race between AUX CH bring-up and enabling bridge which will > > cause link training to fail. To avoid hitting it, don't change psr state > > while enabling the bridge. > > > > Cc: Tomeu Vizoso <tomeu.vizoso@collabora.com> > > Cc: Sean Paul <seanpaul@chromium.org> > > Signed-off-by: zain wang <wzz@rock-chips.com> > > Signed-off-by: Caesar Wang <wxt@rock-chips.com> > > [seanpaul fixed up the commit message a bit and renamed *_supported to *_enabled] > > Signed-off-by: Sean Paul <seanpaul@chromium.org> > > Hmm, beside resetting psr_enable in analogix_dp_bridge_disable I do not > see functional change, or am I blind? The patch also adds a check of psr_enable in analogix_dp_commit. This will avoid trying to enable psr when the bridge is disabled (that's why psr_enable is reset). Sean > > Regards > Andrzej > > > --- > > drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 15 ++++++++------- > > drivers/gpu/drm/bridge/analogix/analogix_dp_core.h | 2 +- > > drivers/gpu/drm/rockchip/analogix_dp-rockchip.c | 2 +- > > include/drm/bridge/analogix_dp.h | 2 +- > > 4 files changed, 11 insertions(+), 10 deletions(-) > > > > diff --git a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c b/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c > > index 8a8f05fe9da3..64d94a34874d 100644 > > --- a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c > > +++ b/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c > > @@ -98,20 +98,20 @@ static int analogix_dp_detect_hpd(struct analogix_dp_device *dp) > > return 0; > > } > > > > -int analogix_dp_psr_supported(struct device *dev) > > +int analogix_dp_psr_enabled(struct device *dev) > > { > > struct analogix_dp_device *dp = dev_get_drvdata(dev); > > > > - return dp->psr_support; > > + return dp->psr_enable; > > } > > -EXPORT_SYMBOL_GPL(analogix_dp_psr_supported); > > +EXPORT_SYMBOL_GPL(analogix_dp_psr_enabled); > > > > int analogix_dp_enable_psr(struct device *dev) > > { > > struct analogix_dp_device *dp = dev_get_drvdata(dev); > > struct edp_vsc_psr psr_vsc; > > > > - if (!dp->psr_support) > > + if (!dp->psr_enable) > > return 0; > > > > /* Prepare VSC packet as per EDP 1.4 spec, Table 6.9 */ > > @@ -134,7 +134,7 @@ int analogix_dp_disable_psr(struct device *dev) > > struct edp_vsc_psr psr_vsc; > > int ret; > > > > - if (!dp->psr_support) > > + if (!dp->psr_enable) > > return 0; > > > > /* Prepare VSC packet as per EDP 1.4 spec, Table 6.9 */ > > @@ -878,8 +878,8 @@ static void analogix_dp_commit(struct analogix_dp_device *dp) > > /* Enable video */ > > analogix_dp_start_video(dp); > > > > - dp->psr_support = analogix_dp_detect_sink_psr(dp); > > - if (dp->psr_support) > > + dp->psr_enable = analogix_dp_detect_sink_psr(dp); > > + if (dp->psr_enable) > > analogix_dp_enable_sink_psr(dp); > > } > > > > @@ -1120,6 +1120,7 @@ static void analogix_dp_bridge_disable(struct drm_bridge *bridge) > > if (ret) > > DRM_ERROR("failed to setup the panel ret = %d\n", ret); > > > > + dp->psr_enable = false; > > dp->dpms_mode = DRM_MODE_DPMS_OFF; > > } > > > > diff --git a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.h b/drivers/gpu/drm/bridge/analogix/analogix_dp_core.h > > index b039b28d8fcc..e135a42cb19e 100644 > > --- a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.h > > +++ b/drivers/gpu/drm/bridge/analogix/analogix_dp_core.h > > @@ -170,7 +170,7 @@ struct analogix_dp_device { > > int dpms_mode; > > int hpd_gpio; > > bool force_hpd; > > - bool psr_support; > > + bool psr_enable; > > > > struct mutex panel_lock; > > bool panel_is_modeset; > > diff --git a/drivers/gpu/drm/rockchip/analogix_dp-rockchip.c b/drivers/gpu/drm/rockchip/analogix_dp-rockchip.c > > index 64e7e2c0bc58..f44756029478 100644 > > --- a/drivers/gpu/drm/rockchip/analogix_dp-rockchip.c > > +++ b/drivers/gpu/drm/rockchip/analogix_dp-rockchip.c > > @@ -83,7 +83,7 @@ static void analogix_dp_psr_set(struct drm_encoder *encoder, bool enabled) > > int vact_end; > > int ret; > > > > - if (!analogix_dp_psr_supported(dp->dev)) > > + if (!analogix_dp_psr_enabled(dp->dev)) > > return; > > > > dev_dbg(dp->dev, "%s PSR...\n", enabled ? "enable" : "disable"); > > diff --git a/include/drm/bridge/analogix_dp.h b/include/drm/bridge/analogix_dp.h > > index c99d6eaef1ac..4fc0165ed3f5 100644 > > --- a/include/drm/bridge/analogix_dp.h > > +++ b/include/drm/bridge/analogix_dp.h > > @@ -38,7 +38,7 @@ struct analogix_dp_plat_data { > > struct drm_connector *); > > }; > > > > -int analogix_dp_psr_supported(struct device *dev); > > +int analogix_dp_psr_enabled(struct device *dev); > > int analogix_dp_enable_psr(struct device *dev); > > int analogix_dp_disable_psr(struct device *dev); > > > -- Sean Paul, Software Engineer, Google / Chromium OS
next prev parent reply other threads:[~2017-03-21 19:58 UTC|newest] Thread overview: 110+ messages / expand[flat|nested] mbox.gz Atom feed top 2017-03-10 4:32 [PATCH 00/41] Chromebook Plus (aka kevin) kernel patches Sean Paul 2017-03-10 4:32 ` [PATCH 01/41] drm/panel: simple: Change mode for Sharp lq123p1jx31 Sean Paul 2017-03-20 13:59 ` Thierry Reding 2017-03-20 16:37 ` Doug Anderson 2017-03-20 20:01 ` Stéphane Marchesin 2017-03-20 20:05 ` Doug Anderson 2017-03-10 4:32 ` [PATCH 03/41] drm/rockchip: support prime import sg table Sean Paul 2017-03-10 4:32 ` Sean Paul 2017-12-12 12:32 ` Heiko Stuebner 2017-12-12 12:32 ` Heiko Stuebner 2017-03-10 4:32 ` [PATCH 04/41] drm/rockchip: Respect page offset for PRIME mmap calls Sean Paul 2017-03-10 4:32 ` Sean Paul 2017-12-12 16:58 ` Heiko Stuebner 2017-12-12 16:58 ` Heiko Stuebner 2017-03-10 4:32 ` [PATCH 05/41] drm/bridge: analogix_dp: set psr activate/deactivate when enable/disable bridge Sean Paul 2017-03-10 4:32 ` Sean Paul [not found] ` <20170310043305.17216-1-seanpaul-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org> 2017-03-10 4:32 ` [PATCH 02/41] drm/rockchip: Get rid of some unnecessary code Sean Paul 2017-03-10 4:32 ` Sean Paul 2017-12-12 12:25 ` Heiko Stuebner 2017-12-12 12:25 ` Heiko Stuebner 2017-03-10 4:32 ` [PATCH 06/41] drm/bridge: analogix_dp: Don't power bridge in analogix_dp_bind Sean Paul 2017-03-16 12:31 ` Andrzej Hajda 2017-03-10 4:32 ` [PATCH 08/41] drm/bridge: analogix_dp: detect Sink PSR state after configuring the PSR Sean Paul 2017-03-16 13:28 ` Andrzej Hajda 2017-03-10 4:32 ` [PATCH 12/41] drm/bridge: analogix_dp: add fast link train for eDP Sean Paul 2017-03-16 14:14 ` Andrzej Hajda 2017-03-21 20:37 ` Sean Paul 2017-03-22 8:07 ` Andrzej Hajda 2017-03-10 4:32 ` [PATCH 15/41] drm/bridge: analogix_dp: Move enable video into config_video() Sean Paul 2017-03-16 14:26 ` Andrzej Hajda 2017-03-10 4:32 ` [PATCH 16/41] drm/bridge: analogix_dp: Check AUX_EN status when doing AUX transfer Sean Paul 2017-03-16 14:28 ` Andrzej Hajda 2017-03-10 4:32 ` [PATCH 17/41] drm/bridge: analogix_dp: Don't use fast link training when panel just powered up Sean Paul 2017-03-16 14:34 ` Andrzej Hajda 2017-03-10 4:32 ` [PATCH 18/41] drm/bridge: analogix_dp: Retry bridge enable when it failed Sean Paul 2017-03-16 14:45 ` Andrzej Hajda 2017-03-10 4:32 ` [PATCH 19/41] drm/bridge: analogix_dp: Wait for HPD signal before configuring link Sean Paul 2017-03-16 14:51 ` Andrzej Hajda 2017-03-10 4:32 ` [PATCH 21/41] drm/bridge: analogix_dp: Ensure edp is disabled when shutting down the panel Sean Paul 2017-03-22 8:29 ` Andrzej Hajda 2017-03-10 4:32 ` [PATCH 22/41] drm/bridge: analogix_dp: Extend hpd check time to 100ms Sean Paul 2017-03-22 8:32 ` Andrzej Hajda 2017-03-10 4:32 ` [PATCH 23/41] drm/bridge: analogix_dp: Fix incorrect usage of enhanced mode Sean Paul 2017-03-22 8:46 ` Andrzej Hajda 2017-03-10 4:32 ` [PATCH 24/41] drm/bridge: analogix_dp: Check dpcd write/read status Sean Paul 2017-03-22 9:00 ` Andrzej Hajda 2017-03-10 4:32 ` [PATCH 25/41] drm/bridge: analogix_dp: Fix AUX_PD bit for Rockchip Sean Paul 2017-03-22 9:09 ` Andrzej Hajda 2017-03-10 4:32 ` [PATCH 26/41] drm/bridge: analogix_dp: Reset aux channel if an error occurred Sean Paul 2017-03-22 9:14 ` Andrzej Hajda 2017-03-10 4:32 ` [PATCH 35/41] drm/rockchip: analogix_dp: Fix invalid implementation of unbind Sean Paul 2017-03-10 4:32 ` Sean Paul 2017-03-10 4:32 ` [PATCH 36/41] drm/bridge: analogix_dp: Add analogix_dp_shutdown Sean Paul 2017-03-10 4:32 ` [PATCH 37/41] drm/rockchip: analogix_dp: Wire the shutdown callback to disable PSR Sean Paul 2017-03-10 4:32 ` Sean Paul 2017-03-10 4:32 ` [PATCH 38/41] drm/bridge: analogix_dp: Reorder plat_data->power_off to happen sooner Sean Paul 2017-03-22 10:34 ` Andrzej Hajda 2017-03-10 4:32 ` [PATCH 41/41] drm/bridge: analogix_dp: Properly disable aux chan retries on rockchip Sean Paul 2017-03-22 10:57 ` Andrzej Hajda 2017-03-22 15:59 ` Doug Anderson 2017-03-28 15:40 ` Javier Martinez Canillas 2017-03-10 4:32 ` [PATCH 07/41] drm/rockchip: Don't use atomic constructs for psr Sean Paul 2017-03-10 4:32 ` Sean Paul 2017-03-10 4:32 ` [PATCH 09/41] drm/rockchip: Remove analogix psr worker Sean Paul 2017-03-10 4:32 ` Sean Paul 2017-03-10 4:32 ` [PATCH 10/41] drm/bridge: analogix_dp: Don't change psr while bridge is disabled Sean Paul 2017-03-10 4:32 ` Sean Paul 2017-03-16 13:40 ` Andrzej Hajda 2017-03-16 13:40 ` Andrzej Hajda 2017-03-21 19:58 ` Sean Paul [this message] 2017-03-21 19:58 ` Sean Paul 2017-03-22 8:36 ` Andrzej Hajda 2017-03-22 8:36 ` Andrzej Hajda 2017-03-22 15:19 ` Sean Paul 2017-03-22 15:19 ` Sean Paul 2017-03-23 9:04 ` Andrzej Hajda 2017-03-23 9:04 ` Andrzej Hajda 2017-03-10 4:32 ` [PATCH 11/41] drm/rockchip: add mutex vop lock Sean Paul 2017-03-10 4:32 ` Sean Paul 2017-03-10 4:32 ` [PATCH 13/41] drm/rockchip: pre dither down when output bpc is 8bit Sean Paul 2017-03-10 4:32 ` Sean Paul 2017-03-10 4:32 ` [PATCH 14/41] drm/rockchip: Only wait for panel ACK on PSR entry Sean Paul 2017-03-10 4:32 ` [PATCH 20/41] drm/bridge: analogix_dp: Set PD_INC_BG first when powering up edp phy Sean Paul 2017-03-16 14:54 ` Andrzej Hajda 2017-03-16 14:54 ` Andrzej Hajda 2017-03-10 4:32 ` [PATCH 27/41] drm/rockchip: Restore psr->state when enable/disable psr failed Sean Paul 2017-03-10 4:32 ` Sean Paul 2017-03-10 4:32 ` [PATCH 28/41] drm/bridge: analogix_dp: Don't use ANALOGIX_DP_PLL_CTL to control pll Sean Paul 2017-03-22 9:17 ` Andrzej Hajda 2017-03-10 4:32 ` [PATCH 29/41] drm/bridge: analogix_dp: Fix timeout of video streamclk config Sean Paul 2017-03-22 9:24 ` Andrzej Hajda 2017-03-10 4:32 ` [PATCH 30/41] drm/bridge: analogix_dp: Fix incorrect operations with register ANALOGIX_DP_FUNC_EN_1 Sean Paul 2017-03-22 9:29 ` Andrzej Hajda 2017-03-10 4:32 ` [PATCH 31/41] drm/bridge: analogix_dp: Move fast link training detect to set_bridge Sean Paul 2017-03-22 10:25 ` Andrzej Hajda 2017-03-10 4:32 ` [PATCH 32/41] drm/rockchip: Flush PSR before committing modeset disables/enables Sean Paul 2017-03-10 4:32 ` Sean Paul 2017-03-10 4:32 ` [PATCH 33/41] drm/rockchip: Disable VOP windows when PSR is active Sean Paul 2017-03-10 4:32 ` Sean Paul 2017-03-10 4:32 ` [PATCH 34/41] drm/bridge: analogix_dp: Allow master driver to cleanup in unbind Sean Paul 2017-03-10 7:09 ` Tomasz Figa 2017-03-10 14:24 ` Sean Paul 2017-03-10 4:32 ` [PATCH 39/41] drm/bridge: analogix_dp: Split the platform-specific poweron in two parts Sean Paul 2017-03-10 4:32 ` Sean Paul 2017-03-22 10:42 ` Andrzej Hajda 2017-03-22 10:42 ` Andrzej Hajda 2017-03-10 4:32 ` [PATCH 40/41] drm/bridge: analogix_dp: Properly log AUX CH errors Sean Paul 2017-03-22 10:47 ` Andrzej Hajda 2017-03-14 20:43 ` [PATCH 00/41] Chromebook Plus (aka kevin) kernel patches Sean Paul 2017-03-16 16:45 ` Enric Balletbo Serra
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=20170321195818.GE19389@art_vandelay \ --to=seanpaul@chromium.org \ --cc=a.hajda@samsung.com \ --cc=dianders@chromium.org \ --cc=dri-devel@lists.freedesktop.org \ --cc=hl@rock-chips.com \ --cc=krzk@kernel.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-rockchip@lists.infradead.org \ --cc=tfiga@chromium.org \ --cc=tomeu.vizoso@collabora.com \ --cc=wxt@rock-chips.com \ --cc=wzz@rock-chips.com \ --cc=ykk@rock-chips.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: linkBe 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.