From: Jean Delvare <jdelvare@suse.de> To: Laszlo Papp <lpapp@kde.org> Cc: Lee Jones <lee.jones@linaro.org>, LKML <linux-kernel@vger.kernel.org>, lm-sensors@lm-sensors.org Subject: Re: [lm-sensors] [RFC PATCH] hwmon: (max6650) Convert to be a platform driver Date: Thu, 13 Feb 2014 13:57:31 +0100 [thread overview] Message-ID: <20140213135731.139b23aa@endymion.delvare> (raw) In-Reply-To: <CAOMwXhMNHqXnzR0YHF9rx710JbaL3YFn=3cLf_WdNkCx7hbZug@mail.gmail.com> Hi Laszlo, On Thu, 13 Feb 2014 12:27:28 +0000, Laszlo Papp wrote: > On Thu, Feb 13, 2014 at 11:33 AM, Lee Jones <lee.jones@linaro.org> wrote: > > Right, I've had enough. I'm removing your patch from the MFD tree. > > > > I've asked too many people to give you a second chance and asked you > > privately to behave yourself and treat others with respect. So far I > > haven't seen an ounce of self control or depomacy from you. > > > > This is how it's going to work from now on: > > > > - You submit a patch > > - It gets reviewed <----\ > > - You fix up the review comments as requested -----/ > > - Non-compliance or arguments with the _experts_ results in: > > `$INTEREST > /dev/null || \ > > grep "From: Laszio Papp" ~/.mail | xargs rm -rf` > > http://comments.gmane.org/gmane.linux.kernel/1645251 > > Step 2 did not happen. I did not get any review for my change. I > literally submitted that within a couple of hours after the request. Yes, twice even, and broken each time. And without a changelog on v2 (despite Documentation/SubmittingPatches explaining this is a good practice - see section 15.) > Could you please tell me what was wrong with that change, and why I > did not get any respect not to "xargs rm -rf" my work in that area? I > believe I was ignored instead of improving the change, and someone > else tried to address the same thing. There was no argument in that > thread. It was a technical change. I personally do not feel happy > about it. The change itself was so wrong that I don't even know where to start. But the main problem really was you. You had pissed me (and I suspect, everybody else) off so much that day that I really didn't want to deal with your rants or code any longer. As you can imagine, I have more than enough on my plate, so I just moved on to another task. Then by the time I may have been willing to give you another chance and review your code, Guenter wrote a more complete, better patch set. So I thought I'd just review that one. And it was good, and it took me less time to review and test it than to (attempt to) teach you how to behave. Working with Guenter is a pleasure. Working with you is a pain, really. And guess what, I get to choose who I'm working with. If people no longer want to work with you, well, blame it on yourself. -- Jean Delvare Suse L3 Support
WARNING: multiple messages have this Message-ID (diff)
From: Jean Delvare <jdelvare@suse.de> To: Laszlo Papp <lpapp@kde.org> Cc: Lee Jones <lee.jones@linaro.org>, LKML <linux-kernel@vger.kernel.org>, lm-sensors@lm-sensors.org Subject: Re: [lm-sensors] [RFC PATCH] hwmon: (max6650) Convert to be a platform driver Date: Thu, 13 Feb 2014 12:57:31 +0000 [thread overview] Message-ID: <20140213135731.139b23aa@endymion.delvare> (raw) In-Reply-To: <CAOMwXhMNHqXnzR0YHF9rx710JbaL3YFn=3cLf_WdNkCx7hbZug@mail.gmail.com> Hi Laszlo, On Thu, 13 Feb 2014 12:27:28 +0000, Laszlo Papp wrote: > On Thu, Feb 13, 2014 at 11:33 AM, Lee Jones <lee.jones@linaro.org> wrote: > > Right, I've had enough. I'm removing your patch from the MFD tree. > > > > I've asked too many people to give you a second chance and asked you > > privately to behave yourself and treat others with respect. So far I > > haven't seen an ounce of self control or depomacy from you. > > > > This is how it's going to work from now on: > > > > - You submit a patch > > - It gets reviewed <----\ > > - You fix up the review comments as requested -----/ > > - Non-compliance or arguments with the _experts_ results in: > > `$INTEREST > /dev/null || \ > > grep "From: Laszio Papp" ~/.mail | xargs rm -rf` > > http://comments.gmane.org/gmane.linux.kernel/1645251 > > Step 2 did not happen. I did not get any review for my change. I > literally submitted that within a couple of hours after the request. Yes, twice even, and broken each time. And without a changelog on v2 (despite Documentation/SubmittingPatches explaining this is a good practice - see section 15.) > Could you please tell me what was wrong with that change, and why I > did not get any respect not to "xargs rm -rf" my work in that area? I > believe I was ignored instead of improving the change, and someone > else tried to address the same thing. There was no argument in that > thread. It was a technical change. I personally do not feel happy > about it. The change itself was so wrong that I don't even know where to start. But the main problem really was you. You had pissed me (and I suspect, everybody else) off so much that day that I really didn't want to deal with your rants or code any longer. As you can imagine, I have more than enough on my plate, so I just moved on to another task. Then by the time I may have been willing to give you another chance and review your code, Guenter wrote a more complete, better patch set. So I thought I'd just review that one. And it was good, and it took me less time to review and test it than to (attempt to) teach you how to behave. Working with Guenter is a pleasure. Working with you is a pain, really. And guess what, I get to choose who I'm working with. If people no longer want to work with you, well, blame it on yourself. -- Jean Delvare Suse L3 Support _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
next prev parent reply other threads:[~2014-02-13 12:57 UTC|newest] Thread overview: 46+ messages / expand[flat|nested] mbox.gz Atom feed top 2014-02-13 8:50 [RFC PATCH] hwmon: (max6650) Convert to be a platform driver Laszlo Papp 2014-02-13 8:50 ` [lm-sensors] " Laszlo Papp 2014-02-13 9:58 ` Lee Jones 2014-02-13 9:58 ` [lm-sensors] " Lee Jones 2014-02-13 10:15 ` Jean Delvare 2014-02-13 10:15 ` Jean Delvare 2014-02-13 10:38 ` Laszlo Papp 2014-02-13 10:38 ` Laszlo Papp 2014-02-13 10:46 ` Laszlo Papp 2014-02-13 10:46 ` Laszlo Papp 2014-02-13 11:07 ` Jean Delvare 2014-02-13 11:07 ` Jean Delvare 2014-02-13 11:29 ` Laszlo Papp 2014-02-13 11:29 ` Laszlo Papp 2014-02-13 11:33 ` Lee Jones 2014-02-13 11:33 ` Lee Jones 2014-02-13 12:27 ` Laszlo Papp 2014-02-13 12:27 ` Laszlo Papp 2014-02-13 12:40 ` Lee Jones 2014-02-13 12:40 ` Lee Jones 2014-02-14 7:03 ` Laszlo Papp 2014-02-14 7:03 ` Laszlo Papp 2014-02-14 9:02 ` Lee Jones 2014-02-14 9:02 ` Lee Jones 2014-02-14 9:20 ` Laszlo Papp 2014-02-14 9:20 ` Laszlo Papp 2014-02-14 10:17 ` Lee Jones 2014-02-14 10:17 ` Lee Jones 2014-02-13 12:57 ` Jean Delvare [this message] 2014-02-13 12:57 ` Jean Delvare 2014-02-13 13:19 ` Laszlo Papp 2014-02-13 13:19 ` Laszlo Papp 2014-02-13 16:16 ` Guenter Roeck 2014-02-13 16:16 ` Guenter Roeck 2014-02-13 16:53 ` Laszlo Papp 2014-02-13 16:53 ` Laszlo Papp 2014-02-14 9:13 ` Lee Jones 2014-02-14 9:13 ` Lee Jones 2014-02-13 11:16 ` Lee Jones 2014-02-13 11:16 ` Lee Jones 2014-02-13 11:58 ` Jean Delvare 2014-02-13 11:58 ` Jean Delvare 2014-02-13 16:29 ` Guenter Roeck 2014-02-13 16:29 ` Guenter Roeck 2014-02-13 10:55 ` Laszlo Papp 2014-02-13 10:55 ` [lm-sensors] " Laszlo Papp
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=20140213135731.139b23aa@endymion.delvare \ --to=jdelvare@suse.de \ --cc=lee.jones@linaro.org \ --cc=linux-kernel@vger.kernel.org \ --cc=lm-sensors@lm-sensors.org \ --cc=lpapp@kde.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.