From: Guru Das Srinagesh <gurus@codeaurora.org>
To: Daniel Thompson <daniel.thompson@linaro.org>
Cc: "Kate Stewart" <kstewart@linuxfoundation.org>,
"Daniel Vetter" <daniel@ffwll.ch>,
linux-fbdev@vger.kernel.org, "David Airlie" <airlied@linux.ie>,
"Wesley W. Terpstra" <wesley@sifive.com>,
"Michael Turquette" <mturquette@baylibre.com>,
"Joonas Lahtinen" <joonas.lahtinen@linux.intel.com>,
"Kamil Debski" <kamil@wypas.org>,
dri-devel@lists.freedesktop.org,
"Liam Girdwood" <lgirdwood@gmail.com>,
"Atish Patra" <atish.patra@wdc.com>,
"Thierry Reding" <thierry.reding@gmail.com>,
"Benjamin Gaignard" <benjamin.gaignard@linaro.org>,
linux-riscv@lists.infradead.org,
"Lee Jones" <lee.jones@linaro.org>,
linux-clk@vger.kernel.org,
"Ville Syrjälä" <ville.syrjala@linux.intel.com>,
"Alexandre Torgue" <alexandre.torgue@st.com>,
"Mauro Carvalho Chehab" <mchehab@kernel.org>,
"Axel Lin" <axel.lin@ingics.com>, "Arnd Bergmann" <arnd@arndb.de>,
"Alexander Shiyan" <shc_work@mail.ru>,
"Fabio Estevam" <festevam@gma>
Subject: Re: [PATCH v10 00/12] Convert PWM period and duty cycle to u64
Date: Mon, 30 Mar 2020 14:00:12 -0700 [thread overview]
Message-ID: <20200330210012.GA27611@codeaurora.org> (raw)
In-Reply-To: <20200330202636.njjo4savgzf3g6yx@holly.lan>
On Mon, Mar 30, 2020 at 09:26:36PM +0100, Daniel Thompson wrote:
> On Mon, Mar 30, 2020 at 12:15:07PM -0700, Guru Das Srinagesh wrote:
> > On Sat, Mar 21, 2020 at 02:47:03PM +0300, Dan Carpenter wrote:
> > > This is a giant CC list.
> >
> > Yes, this is because I received feedback [1] on an earlier patchset
> > directing me to add the reviewers of patches to the cover letter as
> > well so that they get some context for the patch.
> > ...
> > [1] https://www.spinics.net/lists/linux-pwm/msg11735.html
>
> Strictly speaking I only asked for backlight maintainers to be Cc:ed.
> I was fairly careful to be specific since I'm aware there are a variety
> of differing habits when putting together the Cc: list for covering
> letters.
>
> With the original patch header the purpose of the patch I was Cc:ed on
> was impossible to determine without the covering letter.
I suspect this might be the case for all the other reviewers as well -
that they also would appreciate context for the specific patch they are
being added to review.
I wasn't entirely sure what the convention was, so I applied your
suggestion to all the files. How do you suggest I handle this in my next
patchset? I fully agree that such a large CC list does look really
ungainly.
Thank you.
Guru Das.
WARNING: multiple messages have this Message-ID (diff)
From: Guru Das Srinagesh <gurus@codeaurora.org>
To: Daniel Thompson <daniel.thompson@linaro.org>
Cc: "Kate Stewart" <kstewart@linuxfoundation.org>,
"Daniel Vetter" <daniel@ffwll.ch>,
linux-fbdev@vger.kernel.org, "David Airlie" <airlied@linux.ie>,
"Wesley W. Terpstra" <wesley@sifive.com>,
"Michael Turquette" <mturquette@baylibre.com>,
"Joonas Lahtinen" <joonas.lahtinen@linux.intel.com>,
"Kamil Debski" <kamil@wypas.org>,
dri-devel@lists.freedesktop.org,
"Liam Girdwood" <lgirdwood@gmail.com>,
"Atish Patra" <atish.patra@wdc.com>,
"Thierry Reding" <thierry.reding@gmail.com>,
"Benjamin Gaignard" <benjamin.gaignard@linaro.org>,
linux-riscv@lists.infradead.org,
"Lee Jones" <lee.jones@linaro.org>,
linux-clk@vger.kernel.org,
"Ville Syrjälä" <ville.syrjala@linux.intel.com>,
"Alexandre Torgue" <alexandre.torgue@st.com>,
"Mauro Carvalho Chehab" <mchehab@kernel.org>,
"Axel Lin" <axel.lin@ingics.com>, "Arnd Bergmann" <arnd@arndb.de>,
"Alexander Shiyan" <shc_work@mail.ru>,
"Fabio Estevam" <festevam@gmail.com>,
"Chen-Yu Tsai" <wens@csie.org>,
"NXP Linux Team" <linux-imx@nxp.com>,
"Mukesh Ojha" <mojha@codeaurora.org>,
"Gerald Baeza" <gerald.baeza@st.com>,
intel-gfx@lists.freedesktop.org,
"Dan Carpenter" <dan.carpenter@oracle.com>,
linux-media@vger.kernel.org, linux-pwm@vger.kernel.org,
"Jean Delvare" <jdelvare@suse.com>,
"Philipp Zabel" <p.zabel@pengutronix.de>,
"Uwe Kleine-König" <uwe@kleine-koenig.org>,
"Bartlomiej Zolnierkiewicz" <b.zolnierkie@samsung.com>,
"Sascha Hauer" <s.hauer@pengutronix.de>,
"Maarten Lankhorst" <maarten.lankhorst@linux.intel.com>,
"Maxime Ripard" <mripard@kernel.org>,
"Rodrigo Vivi" <rodrigo.vivi@intel.com>,
"Mark Brown" <broonie@kernel.org>,
"Jani Nikula" <jani.nikula@linux.intel.com>,
"Paul Walmsley" <paul.walmsley@sifive.com>,
"Subbaraman Narayanamurthy" <subbaram@codeaurora.org>,
"Thomas Gleixner" <tglx@linutronix.de>,
"Fabrice Gasnier" <fabrice.gasnier@st.com>,
"Ding Xiang" <dingxiang@cmss.chinamobile.com>,
"Pengutronix Kernel Team" <kernel@pengutronix.de>,
"Allison Randal" <allison@lohutok.net>,
linux-hwmon@vger.kernel.org,
"Chris Wilson" <chris@chris-wilson.co.uk>,
"Anson Huang" <Anson.Huang@nxp.com>,
"Richard Fontana" <rfontana@redhat.com>,
"Stephen Boyd" <sboyd@kernel.org>,
"Jingoo Han" <jingoohan1@gmail.com>,
linux-kernel@vger.kernel.org, "Yash Shah" <yash.shah@sifive.com>,
"Palmer Dabbelt" <palmer@dabbelt.com>,
"Guenter Roeck" <linux@roeck-us.net>,
"Maxime Coquelin" <mcoquelin.stm32@gmail.com>,
"Shawn Guo" <shawnguo@kernel.org>
Subject: Re: [PATCH v10 00/12] Convert PWM period and duty cycle to u64
Date: Mon, 30 Mar 2020 14:00:12 -0700 [thread overview]
Message-ID: <20200330210012.GA27611@codeaurora.org> (raw)
In-Reply-To: <20200330202636.njjo4savgzf3g6yx@holly.lan>
On Mon, Mar 30, 2020 at 09:26:36PM +0100, Daniel Thompson wrote:
> On Mon, Mar 30, 2020 at 12:15:07PM -0700, Guru Das Srinagesh wrote:
> > On Sat, Mar 21, 2020 at 02:47:03PM +0300, Dan Carpenter wrote:
> > > This is a giant CC list.
> >
> > Yes, this is because I received feedback [1] on an earlier patchset
> > directing me to add the reviewers of patches to the cover letter as
> > well so that they get some context for the patch.
> > ...
> > [1] https://www.spinics.net/lists/linux-pwm/msg11735.html
>
> Strictly speaking I only asked for backlight maintainers to be Cc:ed.
> I was fairly careful to be specific since I'm aware there are a variety
> of differing habits when putting together the Cc: list for covering
> letters.
>
> With the original patch header the purpose of the patch I was Cc:ed on
> was impossible to determine without the covering letter.
I suspect this might be the case for all the other reviewers as well -
that they also would appreciate context for the specific patch they are
being added to review.
I wasn't entirely sure what the convention was, so I applied your
suggestion to all the files. How do you suggest I handle this in my next
patchset? I fully agree that such a large CC list does look really
ungainly.
Thank you.
Guru Das.
WARNING: multiple messages have this Message-ID (diff)
From: Guru Das Srinagesh <gurus@codeaurora.org>
To: Daniel Thompson <daniel.thompson@linaro.org>
Cc: "Kate Stewart" <kstewart@linuxfoundation.org>,
linux-fbdev@vger.kernel.org, "David Airlie" <airlied@linux.ie>,
"Wesley W. Terpstra" <wesley@sifive.com>,
"Michael Turquette" <mturquette@baylibre.com>,
"Kamil Debski" <kamil@wypas.org>,
dri-devel@lists.freedesktop.org,
"Liam Girdwood" <lgirdwood@gmail.com>,
"Atish Patra" <atish.patra@wdc.com>,
"Thierry Reding" <thierry.reding@gmail.com>,
linux-riscv@lists.infradead.org,
"Lee Jones" <lee.jones@linaro.org>,
linux-clk@vger.kernel.org,
"Alexandre Torgue" <alexandre.torgue@st.com>,
"Mauro Carvalho Chehab" <mchehab@kernel.org>,
"Axel Lin" <axel.lin@ingics.com>, "Arnd Bergmann" <arnd@arndb.de>,
"Alexander Shiyan" <shc_work@mail.ru>,
"Chen-Yu Tsai" <wens@csie.org>,
"NXP Linux Team" <linux-imx@nxp.com>,
"Mukesh Ojha" <mojha@codeaurora.org>,
"Gerald Baeza" <gerald.baeza@st.com>,
intel-gfx@lists.freedesktop.org,
"Dan Carpenter" <dan.carpenter@oracle.com>,
linux-media@vger.kernel.org, linux-pwm@vger.kernel.org,
"Jean Delvare" <jdelvare@suse.com>,
"Uwe Kleine-König" <uwe@kleine-koenig.org>,
"Bartlomiej Zolnierkiewicz" <b.zolnierkie@samsung.com>,
"Sascha Hauer" <s.hauer@pengutronix.de>,
"Rodrigo Vivi" <rodrigo.vivi@intel.com>,
"Mark Brown" <broonie@kernel.org>,
"Paul Walmsley" <paul.walmsley@sifive.com>,
"Subbaraman Narayanamurthy" <subbaram@codeaurora.org>,
"Thomas Gleixner" <tglx@linutronix.de>,
"Fabrice Gasnier" <fabrice.gasnier@st.com>,
"Ding Xiang" <dingxiang@cmss.chinamobile.com>,
"Pengutronix Kernel Team" <kernel@pengutronix.de>,
"Allison Randal" <allison@lohutok.net>,
linux-hwmon@vger.kernel.org,
"Chris Wilson" <chris@chris-wilson.co.uk>,
"Anson Huang" <Anson.Huang@nxp.com>,
"Richard Fontana" <rfontana@redhat.com>,
"Stephen Boyd" <sboyd@kernel.org>,
"Jingoo Han" <jingoohan1@gmail.com>,
linux-kernel@vger.kernel.org, "Yash Shah" <yash.shah@sifive.com>,
"Palmer Dabbelt" <palmer@dabbelt.com>,
"Guenter Roeck" <linux@roeck-us.net>,
"Maxime Coquelin" <mcoquelin.stm32@gmail.com>,
"Shawn Guo" <shawnguo@kernel.org>
Subject: Re: [PATCH v10 00/12] Convert PWM period and duty cycle to u64
Date: Mon, 30 Mar 2020 14:00:12 -0700 [thread overview]
Message-ID: <20200330210012.GA27611@codeaurora.org> (raw)
In-Reply-To: <20200330202636.njjo4savgzf3g6yx@holly.lan>
On Mon, Mar 30, 2020 at 09:26:36PM +0100, Daniel Thompson wrote:
> On Mon, Mar 30, 2020 at 12:15:07PM -0700, Guru Das Srinagesh wrote:
> > On Sat, Mar 21, 2020 at 02:47:03PM +0300, Dan Carpenter wrote:
> > > This is a giant CC list.
> >
> > Yes, this is because I received feedback [1] on an earlier patchset
> > directing me to add the reviewers of patches to the cover letter as
> > well so that they get some context for the patch.
> > ...
> > [1] https://www.spinics.net/lists/linux-pwm/msg11735.html
>
> Strictly speaking I only asked for backlight maintainers to be Cc:ed.
> I was fairly careful to be specific since I'm aware there are a variety
> of differing habits when putting together the Cc: list for covering
> letters.
>
> With the original patch header the purpose of the patch I was Cc:ed on
> was impossible to determine without the covering letter.
I suspect this might be the case for all the other reviewers as well -
that they also would appreciate context for the specific patch they are
being added to review.
I wasn't entirely sure what the convention was, so I applied your
suggestion to all the files. How do you suggest I handle this in my next
patchset? I fully agree that such a large CC list does look really
ungainly.
Thank you.
Guru Das.
_______________________________________________
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: Guru Das Srinagesh <gurus@codeaurora.org>
To: Daniel Thompson <daniel.thompson@linaro.org>
Cc: "Kate Stewart" <kstewart@linuxfoundation.org>,
linux-fbdev@vger.kernel.org, "David Airlie" <airlied@linux.ie>,
"Wesley W. Terpstra" <wesley@sifive.com>,
"Michael Turquette" <mturquette@baylibre.com>,
"Kamil Debski" <kamil@wypas.org>,
dri-devel@lists.freedesktop.org,
"Liam Girdwood" <lgirdwood@gmail.com>,
"Atish Patra" <atish.patra@wdc.com>,
"Benjamin Gaignard" <benjamin.gaignard@linaro.org>,
linux-riscv@lists.infradead.org,
"Lee Jones" <lee.jones@linaro.org>,
linux-clk@vger.kernel.org,
"Alexandre Torgue" <alexandre.torgue@st.com>,
"Mauro Carvalho Chehab" <mchehab@kernel.org>,
"Axel Lin" <axel.lin@ingics.com>, "Arnd Bergmann" <arnd@arndb.de>,
"Alexander Shiyan" <shc_work@mail.ru>,
"Fabio Estevam" <festevam@gmail.com>,
"Chen-Yu Tsai" <wens@csie.org>,
"NXP Linux Team" <linux-imx@nxp.com>,
"Mukesh Ojha" <mojha@codeaurora.org>,
"Gerald Baeza" <gerald.baeza@st.com>,
intel-gfx@lists.freedesktop.org,
"Dan Carpenter" <dan.carpenter@oracle.com>,
linux-media@vger.kernel.org, linux-pwm@vger.kernel.org,
"Jean Delvare" <jdelvare@suse.com>,
"Philipp Zabel" <p.zabel@pengutronix.de>,
"Uwe Kleine-König" <uwe@kleine-koenig.org>,
"Bartlomiej Zolnierkiewicz" <b.zolnierkie@samsung.com>,
"Sascha Hauer" <s.hauer@pengutronix.de>,
"Maxime Ripard" <mripard@kernel.org>,
"Mark Brown" <broonie@kernel.org>,
"Paul Walmsley" <paul.walmsley@sifive.com>,
"Subbaraman Narayanamurthy" <subbaram@codeaurora.org>,
"Thomas Gleixner" <tglx@linutronix.de>,
"Fabrice Gasnier" <fabrice.gasnier@st.com>,
"Ding Xiang" <dingxiang@cmss.chinamobile.com>,
"Pengutronix Kernel Team" <kernel@pengutronix.de>,
"Allison Randal" <allison@lohutok.net>,
linux-hwmon@vger.kernel.org,
"Chris Wilson" <chris@chris-wilson.co.uk>,
"Anson Huang" <Anson.Huang@nxp.com>,
"Richard Fontana" <rfontana@redhat.com>,
"Stephen Boyd" <sboyd@kernel.org>,
"Jingoo Han" <jingoohan1@gmail.com>,
linux-kernel@vger.kernel.org, "Yash Shah" <yash.shah@sifive.com>,
"Palmer Dabbelt" <palmer@dabbelt.com>,
"Guenter Roeck" <linux@roeck-us.net>,
"Maxime Coquelin" <mcoquelin.stm32@gmail.com>,
"Shawn Guo" <shawnguo@kernel.org>
Subject: Re: [Intel-gfx] [PATCH v10 00/12] Convert PWM period and duty cycle to u64
Date: Mon, 30 Mar 2020 14:00:12 -0700 [thread overview]
Message-ID: <20200330210012.GA27611@codeaurora.org> (raw)
In-Reply-To: <20200330202636.njjo4savgzf3g6yx@holly.lan>
On Mon, Mar 30, 2020 at 09:26:36PM +0100, Daniel Thompson wrote:
> On Mon, Mar 30, 2020 at 12:15:07PM -0700, Guru Das Srinagesh wrote:
> > On Sat, Mar 21, 2020 at 02:47:03PM +0300, Dan Carpenter wrote:
> > > This is a giant CC list.
> >
> > Yes, this is because I received feedback [1] on an earlier patchset
> > directing me to add the reviewers of patches to the cover letter as
> > well so that they get some context for the patch.
> > ...
> > [1] https://www.spinics.net/lists/linux-pwm/msg11735.html
>
> Strictly speaking I only asked for backlight maintainers to be Cc:ed.
> I was fairly careful to be specific since I'm aware there are a variety
> of differing habits when putting together the Cc: list for covering
> letters.
>
> With the original patch header the purpose of the patch I was Cc:ed on
> was impossible to determine without the covering letter.
I suspect this might be the case for all the other reviewers as well -
that they also would appreciate context for the specific patch they are
being added to review.
I wasn't entirely sure what the convention was, so I applied your
suggestion to all the files. How do you suggest I handle this in my next
patchset? I fully agree that such a large CC list does look really
ungainly.
Thank you.
Guru Das.
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
next prev parent reply other threads:[~2020-03-30 21:00 UTC|newest]
Thread overview: 47+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-03-19 20:50 [PATCH v10 00/12] Convert PWM period and duty cycle to u64 Guru Das Srinagesh
2020-03-19 20:50 ` [Intel-gfx] " Guru Das Srinagesh
2020-03-19 20:50 ` Guru Das Srinagesh
2020-03-19 20:50 ` Guru Das Srinagesh
2020-03-19 20:50 ` [PATCH v10 01/12] drm/i915: Use 64-bit division macro Guru Das Srinagesh
2020-03-19 20:50 ` [Intel-gfx] " Guru Das Srinagesh
2020-03-19 20:50 ` Guru Das Srinagesh
2020-03-19 20:50 ` [PATCH v10 02/12] hwmon: pwm-fan: " Guru Das Srinagesh
2020-03-19 21:12 ` Guenter Roeck
2020-03-20 1:13 ` Guru Das Srinagesh
2020-03-19 20:50 ` [PATCH v10 03/12] ir-rx51: " Guru Das Srinagesh
2020-03-19 20:50 ` [PATCH v10 04/12] pwm: clps711x: Cast period to u32 before use as divisor Guru Das Srinagesh
2020-03-19 20:50 ` [PATCH v10 05/12] pwm: pwm-imx-tpm: Use 64-bit division macro Guru Das Srinagesh
2020-03-19 20:50 ` [PATCH v10 06/12] pwm: imx27: Use 64-bit division macro and function Guru Das Srinagesh
2020-03-19 20:50 ` [PATCH v10 07/12] pwm: sifive: Use 64-bit division macro Guru Das Srinagesh
2020-03-19 20:50 ` Guru Das Srinagesh
2020-03-19 20:50 ` [PATCH v10 08/12] pwm: stm32-lp: Use %llu format specifier for period Guru Das Srinagesh
2020-03-19 20:50 ` [PATCH v10 09/12] pwm: sun4i: Use 64-bit division function Guru Das Srinagesh
2020-03-19 20:50 ` [PATCH v10 10/12] backlight: pwm_bl: " Guru Das Srinagesh
2020-03-19 20:50 ` Guru Das Srinagesh
2020-03-19 20:50 ` Guru Das Srinagesh
2020-03-19 20:50 ` [PATCH v10 11/12] clk: pwm: Assign u64 divisor to unsigned int before use Guru Das Srinagesh
2020-03-19 20:50 ` [PATCH v10 12/12] pwm: core: Convert period and duty cycle to u64 Guru Das Srinagesh
2020-03-21 11:47 ` [PATCH v10 00/12] Convert PWM " Dan Carpenter
2020-03-21 11:47 ` [Intel-gfx] " Dan Carpenter
2020-03-21 11:47 ` Dan Carpenter
2020-03-21 11:47 ` Dan Carpenter
2020-03-30 19:15 ` Guru Das Srinagesh
2020-03-30 19:15 ` [Intel-gfx] " Guru Das Srinagesh
2020-03-30 19:15 ` Guru Das Srinagesh
2020-03-30 19:15 ` Guru Das Srinagesh
2020-03-30 20:26 ` Daniel Thompson
2020-03-30 20:26 ` [Intel-gfx] " Daniel Thompson
2020-03-30 20:26 ` Daniel Thompson
2020-03-30 20:26 ` Daniel Thompson
2020-03-30 21:00 ` Guru Das Srinagesh [this message]
2020-03-30 21:00 ` [Intel-gfx] " Guru Das Srinagesh
2020-03-30 21:00 ` Guru Das Srinagesh
2020-03-30 21:00 ` Guru Das Srinagesh
2020-03-31 13:48 ` Daniel Thompson
2020-03-31 13:48 ` [Intel-gfx] " Daniel Thompson
2020-03-31 13:48 ` Daniel Thompson
2020-03-31 13:48 ` Daniel Thompson
2020-03-31 19:59 ` Guru Das Srinagesh
2020-03-31 19:59 ` [Intel-gfx] " Guru Das Srinagesh
2020-03-31 19:59 ` Guru Das Srinagesh
2020-03-31 19:59 ` Guru Das Srinagesh
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=20200330210012.GA27611@codeaurora.org \
--to=gurus@codeaurora.org \
--cc=airlied@linux.ie \
--cc=alexandre.torgue@st.com \
--cc=arnd@arndb.de \
--cc=atish.patra@wdc.com \
--cc=axel.lin@ingics.com \
--cc=benjamin.gaignard@linaro.org \
--cc=daniel.thompson@linaro.org \
--cc=daniel@ffwll.ch \
--cc=dri-devel@lists.freedesktop.org \
--cc=festevam@gma \
--cc=joonas.lahtinen@linux.intel.com \
--cc=kamil@wypas.org \
--cc=kstewart@linuxfoundation.org \
--cc=lee.jones@linaro.org \
--cc=lgirdwood@gmail.com \
--cc=linux-clk@vger.kernel.org \
--cc=linux-fbdev@vger.kernel.org \
--cc=linux-riscv@lists.infradead.org \
--cc=mchehab@kernel.org \
--cc=mturquette@baylibre.com \
--cc=shc_work@mail.ru \
--cc=thierry.reding@gmail.com \
--cc=ville.syrjala@linux.intel.com \
--cc=wesley@sifive.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.