From: Thierry Reding <thierry.reding@gmail.com>
To: "Uwe Kleine-König" <uwe@kleine-koenig.org>
Cc: "Lee Jones" <lee.jones@linaro.org>,
"Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>,
"Ajit Pal Singh" <ajitpal.singh@st.com>,
linux-pwm@vger.kernel.org
Subject: Re: [PATCH] pwm: sti: fix error handling
Date: Wed, 11 Nov 2020 20:52:04 +0100 [thread overview]
Message-ID: <20201111195204.GE6125@ulmo> (raw)
In-Reply-To: <0d9adfa9-9015-c8c5-c21a-f4ad9b1ef7a9@kleine-koenig.org>
[-- Attachment #1: Type: text/plain, Size: 895 bytes --]
On Wed, Nov 11, 2020 at 08:43:18PM +0100, Uwe Kleine-König wrote:
> Hello Thierry,
>
> On 11/11/20 8:28 PM, Thierry Reding wrote:
> > I agree with Lee on this one, so I've applied the patch and touched up
> > the label names while at it.
>
> Have you tried to follow my reasoning?
Yes I did, and I happen to disagree.
> I consider it irritating that you
> overrule the preference I justified in such detail by applying a patch with
> me as author and the error handling variant I opposed to.
Well, it was either this or have you respin the patch. I didn't think
you'd mind, so I did that work myself but still wanted to credit you for
fixing this in the first place (the names of the labels really don't
mean much for authorship in my opinion).
If you prefer I can just remove the patch and you can do this yourself.
Thierry
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
next prev parent reply other threads:[~2020-11-11 19:52 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-10-13 8:15 [PATCH] pwm: sti: fix error handling Uwe Kleine-König
2020-11-06 8:29 ` Lee Jones
2020-11-06 9:34 ` Uwe Kleine-König
2020-11-06 10:29 ` Lee Jones
2020-11-11 19:28 ` Thierry Reding
2020-11-11 19:43 ` Uwe Kleine-König
2020-11-11 19:52 ` Thierry Reding [this message]
2020-11-11 20:39 ` Uwe Kleine-König
2020-11-11 21:16 ` Thierry Reding
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=20201111195204.GE6125@ulmo \
--to=thierry.reding@gmail.com \
--cc=ajitpal.singh@st.com \
--cc=lee.jones@linaro.org \
--cc=linux-pwm@vger.kernel.org \
--cc=u.kleine-koenig@pengutronix.de \
--cc=uwe@kleine-koenig.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: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox