From: Mathieu Poirier <mathieu.poirier-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> To: Jon Hunter <jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> Cc: Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>, Thierry Reding <thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>, Alexandre Courbot <gnurou-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>, "Rafael J. Wysocki" <rjw-LthD3rsA81gm4RdzfppkhA@public.gmane.org>, Kevin Hilman <khilman-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>, Ulf Hansson <ulf.hansson-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>, Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>, Pawel Moll <pawel.moll-5wv7dgnIgG8@public.gmane.org>, Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>, Ian Campbell <ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org>, Kumar Gala <galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>, linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-pm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" <devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>, "linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" <linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org> Subject: Re: [PATCH V5 05/14] soc: tegra: pmc: Wait for powergate state to change Date: Wed, 3 Feb 2016 08:58:58 -0700 [thread overview] Message-ID: <CANLsYkyPj+ExhBFffokvtMcZ-OWocMc0Hx3_nfVgRu=uUVvJfg@mail.gmail.com> (raw) In-Reply-To: <56B1C647.4060504-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> On 3 February 2016 at 02:20, Jon Hunter <jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> wrote: > > On 01/02/16 13:44, Jon Hunter wrote: >> >> On 29/01/16 16:58, Mathieu Poirier wrote: >>> On 28 January 2016 at 09:33, Jon Hunter <jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> wrote: >>>> Currently, the function tegra_powergate_set() simply sets the desired >>>> powergate state but does not wait for the state to change. In most cases >>>> we should wait for the state to change before proceeding. Currently, there >>>> is a case for tegra114 and tegra124 devices where we do not wait when >>>> starting the secondary CPU as this is not necessary. However, this is only >>>> done at boot time and so waiting here will only have a small impact on >>>> boot time. Therefore, update tegra_powergate_set() to wait when setting >>>> the powergate. >>>> >>>> By adding this feature, we can also eliminate the polling loop from >>>> tegra30_boot_secondary(). >>>> >>>> A function has been added for checking the status of the powergate and >>>> so update the tegra_powergate_is_powered() to use this macro as well. >>>> >>>> Signed-off-by: Jon Hunter <jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> >>>> --- >>>> arch/arm/mach-tegra/platsmp.c | 16 +++------------- >>>> drivers/soc/tegra/pmc.c | 9 ++++++++- >>>> 2 files changed, 11 insertions(+), 14 deletions(-) >>>> >>>> diff --git a/arch/arm/mach-tegra/platsmp.c b/arch/arm/mach-tegra/platsmp.c >>>> index f3f61dbbda97..75620ae73913 100644 >>>> --- a/arch/arm/mach-tegra/platsmp.c >>>> +++ b/arch/arm/mach-tegra/platsmp.c >>>> @@ -108,19 +108,9 @@ static int tegra30_boot_secondary(unsigned int cpu, struct task_struct *idle) >>>> * be un-gated by un-toggling the power gate register >>>> * manually. >>>> */ >>>> - if (!tegra_pmc_cpu_is_powered(cpu)) { >>>> - ret = tegra_pmc_cpu_power_on(cpu); >>>> - if (ret) >>>> - return ret; >>>> - >>>> - /* Wait for the power to come up. */ >>>> - timeout = jiffies + msecs_to_jiffies(100); >>>> - while (!tegra_pmc_cpu_is_powered(cpu)) { >>>> - if (time_after(jiffies, timeout)) >>>> - return -ETIMEDOUT; >>>> - udelay(10); >>>> - } >>>> - } >>>> + ret = tegra_pmc_cpu_power_on(cpu); >>>> + if (ret) >>>> + return ret; >>>> >>>> remove_clamps: >>>> /* CPU partition is powered. Enable the CPU clock. */ >>>> diff --git a/drivers/soc/tegra/pmc.c b/drivers/soc/tegra/pmc.c >>>> index 99cb2fdd29e1..35ee60fd17be 100644 >>>> --- a/drivers/soc/tegra/pmc.c >>>> +++ b/drivers/soc/tegra/pmc.c >>>> @@ -28,6 +28,7 @@ >>>> #include <linux/export.h> >>>> #include <linux/init.h> >>>> #include <linux/io.h> >>>> +#include <linux/iopoll.h> >>>> #include <linux/of.h> >>>> #include <linux/of_address.h> >>>> #include <linux/platform_device.h> >>>> @@ -186,6 +187,9 @@ static inline bool tegra_powergate_state(int id) >>>> */ >>>> static int tegra_powergate_set(unsigned int id, bool new_state) >>>> { >>>> + bool status; >>>> + int err; >>>> + >>>> mutex_lock(&pmc->powergates_lock); >>>> >>>> if (tegra_powergate_state(id) == new_state) { >>>> @@ -195,9 +199,12 @@ static int tegra_powergate_set(unsigned int id, bool new_state) >>>> >>>> tegra_pmc_writel(PWRGATE_TOGGLE_START | id, PWRGATE_TOGGLE); >>>> >>>> + err = readx_poll_timeout(tegra_powergate_state, id, status, >>>> + status == new_state, 10, 100000); >>>> + >>> >>> I understand (and agree) with the goal of this patch but I wonder (and >>> I don't know much about the system/context) if holding a mutex while >>> sleeping won't incur adverse effect on other parts of the system that >>> weren't use to see this wait. One way to fix this might be to use >>> "mutex_trylock()" and let callers retry as they see fit if an >>> operation is already in progress. >> >> I could unlock the mutex after writing the TOGGLE_START register. I >> don't believe that we need to waiting for a powergate to change before >> writing again. > > I was not thinking clearly here, the whole reason for the locking is to > protect the base address as we swap it during the probe. > > In general, I prefer to keep it the way we have it in this patch. By > using mutex_trylock() would mean that now clients would have to handle > retrying if they cannot get the lock and make it more complex. Yes, the complexity is indeed pushed to the caller but that way we know exactly what is going on. Another way to implement this without any locks at all would be through RCUs. I'll let Stephen and Thierry make the call here. Thanks, Mahtieu > Powergates should not be turned on and off that frequently and so > hopefully a delay here should be tolerable. For CPUs, this should only > be used on cold boot and not during CPU hotplug events. > > Cheers > Jon
WARNING: multiple messages have this Message-ID (diff)
From: mathieu.poirier@linaro.org (Mathieu Poirier) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH V5 05/14] soc: tegra: pmc: Wait for powergate state to change Date: Wed, 3 Feb 2016 08:58:58 -0700 [thread overview] Message-ID: <CANLsYkyPj+ExhBFffokvtMcZ-OWocMc0Hx3_nfVgRu=uUVvJfg@mail.gmail.com> (raw) In-Reply-To: <56B1C647.4060504@nvidia.com> On 3 February 2016 at 02:20, Jon Hunter <jonathanh@nvidia.com> wrote: > > On 01/02/16 13:44, Jon Hunter wrote: >> >> On 29/01/16 16:58, Mathieu Poirier wrote: >>> On 28 January 2016 at 09:33, Jon Hunter <jonathanh@nvidia.com> wrote: >>>> Currently, the function tegra_powergate_set() simply sets the desired >>>> powergate state but does not wait for the state to change. In most cases >>>> we should wait for the state to change before proceeding. Currently, there >>>> is a case for tegra114 and tegra124 devices where we do not wait when >>>> starting the secondary CPU as this is not necessary. However, this is only >>>> done at boot time and so waiting here will only have a small impact on >>>> boot time. Therefore, update tegra_powergate_set() to wait when setting >>>> the powergate. >>>> >>>> By adding this feature, we can also eliminate the polling loop from >>>> tegra30_boot_secondary(). >>>> >>>> A function has been added for checking the status of the powergate and >>>> so update the tegra_powergate_is_powered() to use this macro as well. >>>> >>>> Signed-off-by: Jon Hunter <jonathanh@nvidia.com> >>>> --- >>>> arch/arm/mach-tegra/platsmp.c | 16 +++------------- >>>> drivers/soc/tegra/pmc.c | 9 ++++++++- >>>> 2 files changed, 11 insertions(+), 14 deletions(-) >>>> >>>> diff --git a/arch/arm/mach-tegra/platsmp.c b/arch/arm/mach-tegra/platsmp.c >>>> index f3f61dbbda97..75620ae73913 100644 >>>> --- a/arch/arm/mach-tegra/platsmp.c >>>> +++ b/arch/arm/mach-tegra/platsmp.c >>>> @@ -108,19 +108,9 @@ static int tegra30_boot_secondary(unsigned int cpu, struct task_struct *idle) >>>> * be un-gated by un-toggling the power gate register >>>> * manually. >>>> */ >>>> - if (!tegra_pmc_cpu_is_powered(cpu)) { >>>> - ret = tegra_pmc_cpu_power_on(cpu); >>>> - if (ret) >>>> - return ret; >>>> - >>>> - /* Wait for the power to come up. */ >>>> - timeout = jiffies + msecs_to_jiffies(100); >>>> - while (!tegra_pmc_cpu_is_powered(cpu)) { >>>> - if (time_after(jiffies, timeout)) >>>> - return -ETIMEDOUT; >>>> - udelay(10); >>>> - } >>>> - } >>>> + ret = tegra_pmc_cpu_power_on(cpu); >>>> + if (ret) >>>> + return ret; >>>> >>>> remove_clamps: >>>> /* CPU partition is powered. Enable the CPU clock. */ >>>> diff --git a/drivers/soc/tegra/pmc.c b/drivers/soc/tegra/pmc.c >>>> index 99cb2fdd29e1..35ee60fd17be 100644 >>>> --- a/drivers/soc/tegra/pmc.c >>>> +++ b/drivers/soc/tegra/pmc.c >>>> @@ -28,6 +28,7 @@ >>>> #include <linux/export.h> >>>> #include <linux/init.h> >>>> #include <linux/io.h> >>>> +#include <linux/iopoll.h> >>>> #include <linux/of.h> >>>> #include <linux/of_address.h> >>>> #include <linux/platform_device.h> >>>> @@ -186,6 +187,9 @@ static inline bool tegra_powergate_state(int id) >>>> */ >>>> static int tegra_powergate_set(unsigned int id, bool new_state) >>>> { >>>> + bool status; >>>> + int err; >>>> + >>>> mutex_lock(&pmc->powergates_lock); >>>> >>>> if (tegra_powergate_state(id) == new_state) { >>>> @@ -195,9 +199,12 @@ static int tegra_powergate_set(unsigned int id, bool new_state) >>>> >>>> tegra_pmc_writel(PWRGATE_TOGGLE_START | id, PWRGATE_TOGGLE); >>>> >>>> + err = readx_poll_timeout(tegra_powergate_state, id, status, >>>> + status == new_state, 10, 100000); >>>> + >>> >>> I understand (and agree) with the goal of this patch but I wonder (and >>> I don't know much about the system/context) if holding a mutex while >>> sleeping won't incur adverse effect on other parts of the system that >>> weren't use to see this wait. One way to fix this might be to use >>> "mutex_trylock()" and let callers retry as they see fit if an >>> operation is already in progress. >> >> I could unlock the mutex after writing the TOGGLE_START register. I >> don't believe that we need to waiting for a powergate to change before >> writing again. > > I was not thinking clearly here, the whole reason for the locking is to > protect the base address as we swap it during the probe. > > In general, I prefer to keep it the way we have it in this patch. By > using mutex_trylock() would mean that now clients would have to handle > retrying if they cannot get the lock and make it more complex. Yes, the complexity is indeed pushed to the caller but that way we know exactly what is going on. Another way to implement this without any locks at all would be through RCUs. I'll let Stephen and Thierry make the call here. Thanks, Mahtieu > Powergates should not be turned on and off that frequently and so > hopefully a delay here should be tolerable. For CPUs, this should only > be used on cold boot and not during CPU hotplug events. > > Cheers > Jon
next prev parent reply other threads:[~2016-02-03 15:58 UTC|newest] Thread overview: 98+ messages / expand[flat|nested] mbox.gz Atom feed top 2016-01-28 16:33 [PATCH V5 00/14] Add generic PM domain support for Tegra Jon Hunter 2016-01-28 16:33 ` Jon Hunter 2016-01-28 16:33 ` [PATCH V5 02/14] soc: tegra: pmc: Protect public functions from potential race conditions Jon Hunter 2016-01-28 16:33 ` Jon Hunter 2016-01-29 16:20 ` Mathieu Poirier 2016-01-29 16:20 ` Mathieu Poirier 2016-02-01 13:42 ` Jon Hunter 2016-02-01 13:42 ` Jon Hunter 2016-01-28 16:33 ` [PATCH V5 03/14] soc: tegra: pmc: Change powergate and rail IDs to be an unsigned type Jon Hunter 2016-01-28 16:33 ` Jon Hunter 2016-01-28 16:33 ` [PATCH V5 04/14] soc: tegra: pmc: Fix testing of powergate state Jon Hunter 2016-01-28 16:33 ` Jon Hunter 2016-01-28 16:33 ` [PATCH V5 07/14] soc: tegra: pmc: Ensure partitions can be toggled on/off by PMC Jon Hunter 2016-01-28 16:33 ` Jon Hunter 2016-01-28 16:33 ` [PATCH V5 08/14] PM / Domains: Add function to remove a pm-domain Jon Hunter 2016-01-28 16:33 ` Jon Hunter [not found] ` <1453998832-27383-9-git-send-email-jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2016-02-02 15:35 ` Ulf Hansson 2016-02-02 15:35 ` Ulf Hansson [not found] ` <CAPDyKFqJLdoee4a9819XukXTmYyd3pue452K_zbiV6XhfA=fTw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2016-02-03 10:51 ` Jon Hunter 2016-02-03 10:51 ` Jon Hunter [not found] ` <1453998832-27383-1-git-send-email-jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2016-01-28 16:33 ` [PATCH V5 01/14] soc: tegra: pmc: Restore base address on probe failure Jon Hunter 2016-01-28 16:33 ` Jon Hunter 2016-01-28 16:33 ` [PATCH V5 05/14] soc: tegra: pmc: Wait for powergate state to change Jon Hunter 2016-01-28 16:33 ` Jon Hunter 2016-01-29 16:58 ` Mathieu Poirier 2016-01-29 16:58 ` Mathieu Poirier [not found] ` <CANLsYkycbEo+wyMX8RJ9H-S5kDTjQR4nnDZc5gvf2kShOZAv9w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2016-02-01 13:44 ` Jon Hunter 2016-02-01 13:44 ` Jon Hunter [not found] ` <56AF613A.1000909-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2016-02-03 9:20 ` Jon Hunter 2016-02-03 9:20 ` Jon Hunter [not found] ` <56B1C647.4060504-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2016-02-03 15:58 ` Mathieu Poirier [this message] 2016-02-03 15:58 ` Mathieu Poirier 2016-01-28 16:33 ` [PATCH V5 06/14] soc: tegra: pmc: Fix checking of valid partitions Jon Hunter 2016-01-28 16:33 ` Jon Hunter [not found] ` <1453998832-27383-7-git-send-email-jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2016-01-29 17:08 ` Mathieu Poirier 2016-01-29 17:08 ` Mathieu Poirier [not found] ` <CANLsYkxY5P2wQxGev0veN39nD-1cTVkZCVpX9jca7da39JJpWg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2016-02-01 13:45 ` Jon Hunter 2016-02-01 13:45 ` Jon Hunter 2016-01-28 16:33 ` [PATCH V5 09/14] Documentation: DT: bindings: Update NVIDIA PMC for Tegra Jon Hunter 2016-01-28 16:33 ` Jon Hunter 2016-01-29 16:08 ` Rob Herring 2016-01-29 16:08 ` Rob Herring 2016-01-28 16:33 ` [PATCH V5 10/14] Documentation: DT: bindings: Add power domain info for NVIDIA PMC Jon Hunter 2016-01-28 16:33 ` Jon Hunter 2016-01-29 16:06 ` Rob Herring 2016-01-29 16:06 ` Rob Herring 2016-02-03 11:02 ` Jon Hunter 2016-02-03 11:02 ` Jon Hunter [not found] ` <56B1DE40.7080403-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2016-02-03 15:48 ` Rob Herring 2016-02-03 15:48 ` Rob Herring [not found] ` <CAL_JsqLcoKW2znNNvM=sYLmZ6O6ZWqn7+aXspkXoONw6-O1ygg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2016-02-10 10:57 ` Jon Hunter 2016-02-10 10:57 ` Jon Hunter [not found] ` <56BB1787.4050801-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2016-02-10 14:06 ` Rob Herring 2016-02-10 14:06 ` Rob Herring 2016-01-28 16:33 ` [PATCH V5 11/14] soc: tegra: pmc: Add generic PM domain support Jon Hunter 2016-01-28 16:33 ` Jon Hunter 2016-02-04 15:44 ` Ulf Hansson 2016-02-04 15:44 ` Ulf Hansson 2016-02-10 18:01 ` Jon Hunter 2016-02-10 18:01 ` Jon Hunter [not found] ` <56BB7AF4.8040708-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2016-02-10 18:25 ` Ulf Hansson 2016-02-10 18:25 ` Ulf Hansson [not found] ` <CAPDyKFrZ6tWBsQC0tyWWeChiZja3h_zcbaiX25ak-Zyp4MzqVw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2016-02-11 9:13 ` Jon Hunter 2016-02-11 9:13 ` Jon Hunter 2016-02-11 9:57 ` Ulf Hansson 2016-02-11 9:57 ` Ulf Hansson [not found] ` <CAPDyKFrdmufsMqNL0U7q5gPEUqsg3SrkrNChcziQjEOjvd30Ng-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2016-02-11 10:13 ` Jon Hunter 2016-02-11 10:13 ` Jon Hunter [not found] ` <56BC5EE0.2040804-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2016-02-11 10:26 ` Jon Hunter 2016-02-11 10:26 ` Jon Hunter 2016-02-11 10:37 ` Ulf Hansson 2016-02-11 10:37 ` Ulf Hansson 2016-02-11 10:52 ` Jon Hunter 2016-02-11 10:52 ` Jon Hunter 2016-02-11 10:28 ` Ulf Hansson 2016-02-11 10:28 ` Ulf Hansson [not found] ` <CAPDyKFq_0t4tcvkgMBW8p8ubJDALWMjdhgGM+_Z6auRxEkSPdA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2016-02-11 16:38 ` Jon Hunter 2016-02-11 16:38 ` Jon Hunter [not found] ` <56BCB90C.8000302-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2016-02-18 15:06 ` Ulf Hansson 2016-02-18 15:06 ` Ulf Hansson 2016-02-12 23:14 ` Kevin Hilman 2016-02-12 23:14 ` Kevin Hilman [not found] ` <7hh9hdzflv.fsf-rdvid1DuHRBWk0Htik3J/w@public.gmane.org> 2016-02-15 11:27 ` Jon Hunter 2016-02-15 11:27 ` Jon Hunter [not found] ` <56C1B62B.5060708-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2016-02-18 16:00 ` Ulf Hansson 2016-02-18 16:00 ` Ulf Hansson [not found] ` <CAPDyKFoPrFoMOFxC37zXX4L3VdLKknaw_LUTw7ycr9mfa_=7_A-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2016-02-18 16:31 ` Jon Hunter 2016-02-18 16:31 ` Jon Hunter 2016-02-24 0:03 ` Kevin Hilman 2016-02-24 0:03 ` Kevin Hilman 2016-01-28 16:33 ` [PATCH V5 12/14] clk: tegra210: Add the APB2APE audio clock Jon Hunter 2016-01-28 16:33 ` Jon Hunter 2016-02-02 14:37 ` Thierry Reding 2016-02-02 14:37 ` Thierry Reding 2016-01-28 16:33 ` [PATCH V5 13/14] ARM64: tegra: Add audio PM domain device node for Tegra210 Jon Hunter 2016-01-28 16:33 ` Jon Hunter 2016-01-28 16:33 ` [PATCH V5 14/14] ARM64: tegra: select PM_GENERIC_DOMAINS Jon Hunter 2016-01-28 16:33 ` Jon Hunter
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='CANLsYkyPj+ExhBFffokvtMcZ-OWocMc0Hx3_nfVgRu=uUVvJfg@mail.gmail.com' \ --to=mathieu.poirier-qsej5fyqhm4dnm+yrofe0a@public.gmane.org \ --cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \ --cc=galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org \ --cc=gnurou-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \ --cc=ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org \ --cc=jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org \ --cc=khilman-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \ --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \ --cc=linux-pm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \ --cc=linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \ --cc=mark.rutland-5wv7dgnIgG8@public.gmane.org \ --cc=pawel.moll-5wv7dgnIgG8@public.gmane.org \ --cc=rjw-LthD3rsA81gm4RdzfppkhA@public.gmane.org \ --cc=robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \ --cc=swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org \ --cc=thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \ --cc=ulf.hansson-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \ /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.