From: Pavel Machek <pavel@ucw.cz> To: Javier Martinez Canillas <martinez.javier@gmail.com> Cc: "Pali Rohár" <pali.rohar@gmail.com>, "Felipe Balbi" <balbi@ti.com>, "Tony Lindgren" <tony@atomide.com>, "Anton Vorontsov" <anton@enomsg.org>, "Russell King" <linux@arm.linux.org.uk>, "David Woodhouse" <dwmw2@infradead.org>, "Greg Kroah-Hartman" <gregkh@linuxfoundation.org>, freemangordon@abv.bg, "Aaro Koskinen" <aaro.koskinen@iki.fi>, "linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>, "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org>, "Linux Kernel" <linux-kernel@vger.kernel.org>, linux-usb@vger.kernel.org Subject: Re: [PATCH 1/4] usb: musb: Call atomic_notifier_call_chain when status is changed Date: Wed, 18 Sep 2013 16:22:03 +0200 [thread overview] Message-ID: <20130918142203.GD3748@amd.pavel.ucw.cz> (raw) In-Reply-To: <CAAwP0s0nGthkm9Bp1ug23Rtz6-sbR60FFTSfd-Cj9wKMU0RBEA@mail.gmail.com> Hi! > >> >> > So will you do that? Or it is needed to resend this one line > >> >> > hunk again in new email again? > >> >> > >> >> new patch, new email > >> > > >> > Guys, WHY ARE YOU SO STUPID AND ARROGANT? > >> > > >> > Sorry but, need to copy full isolated patch/hunk from one mail to > >> > another is hassling. So what you want from me? Do all those non > >> > sense working only because yesterday you had bad day? Or what? ... > > Actually, there is need to be rude, because Felipe fails to act as > > maintainer. Instead of fixing bugs in his code, he bounces bugfix > > patches, points people to random READMEs and wastes everyones time. > > I don't know what are you talking about (if that happened in another > thread then I need more context). Felipe is not bouncing any bugfix Take a look here: https://lkml.org/lkml/2013/9/17/286 I clearly state that patch can not be tested as required for "proper" submission, but offer patch anyway. I get irrelevant boilerplate on patch format. > but just asked to split the patch in two since the patch was solving > two separate issues so is way better to have it in two separate > patches for the reasons I explained before. > > So, as far as I can tell Felipe did exactly what I would expect from a > maintainer. He took the time to review the patches sent to him and I'd expect maintainer to, well, maintain code. It means actually fixing bugs in his code, when he's pointed at them. > gave feedback. If the sender doesn't want to take his feedback into > account and prefer to send pretty insulting emails instead that is his > choice but I would say that is this not the greatest approach to get > your code merged (to say the least). Clearly not. But Pali found bug in code Felipe should maintain. Instead of "thank you for bug report, I applied this one line of your code to fix it", Pali got "new patch, new email" for his efforts. That is how you train dogs, not how you should treat kernel contributors. Now, it is possible that Felipe just has problems with english, as he called me piece of wood in https://lkml.org/lkml/2013/9/17/476 , but he appears more arogant than usual over email. Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
WARNING: multiple messages have this Message-ID (diff)
From: pavel@ucw.cz (Pavel Machek) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH 1/4] usb: musb: Call atomic_notifier_call_chain when status is changed Date: Wed, 18 Sep 2013 16:22:03 +0200 [thread overview] Message-ID: <20130918142203.GD3748@amd.pavel.ucw.cz> (raw) In-Reply-To: <CAAwP0s0nGthkm9Bp1ug23Rtz6-sbR60FFTSfd-Cj9wKMU0RBEA@mail.gmail.com> Hi! > >> >> > So will you do that? Or it is needed to resend this one line > >> >> > hunk again in new email again? > >> >> > >> >> new patch, new email > >> > > >> > Guys, WHY ARE YOU SO STUPID AND ARROGANT? > >> > > >> > Sorry but, need to copy full isolated patch/hunk from one mail to > >> > another is hassling. So what you want from me? Do all those non > >> > sense working only because yesterday you had bad day? Or what? ... > > Actually, there is need to be rude, because Felipe fails to act as > > maintainer. Instead of fixing bugs in his code, he bounces bugfix > > patches, points people to random READMEs and wastes everyones time. > > I don't know what are you talking about (if that happened in another > thread then I need more context). Felipe is not bouncing any bugfix Take a look here: https://lkml.org/lkml/2013/9/17/286 I clearly state that patch can not be tested as required for "proper" submission, but offer patch anyway. I get irrelevant boilerplate on patch format. > but just asked to split the patch in two since the patch was solving > two separate issues so is way better to have it in two separate > patches for the reasons I explained before. > > So, as far as I can tell Felipe did exactly what I would expect from a > maintainer. He took the time to review the patches sent to him and I'd expect maintainer to, well, maintain code. It means actually fixing bugs in his code, when he's pointed at them. > gave feedback. If the sender doesn't want to take his feedback into > account and prefer to send pretty insulting emails instead that is his > choice but I would say that is this not the greatest approach to get > your code merged (to say the least). Clearly not. But Pali found bug in code Felipe should maintain. Instead of "thank you for bug report, I applied this one line of your code to fix it", Pali got "new patch, new email" for his efforts. That is how you train dogs, not how you should treat kernel contributors. Now, it is possible that Felipe just has problems with english, as he called me piece of wood in https://lkml.org/lkml/2013/9/17/476 , but he appears more arogant than usual over email. Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
next prev parent reply other threads:[~2013-09-18 14:22 UTC|newest] Thread overview: 148+ messages / expand[flat|nested] mbox.gz Atom feed top 2013-09-08 8:50 [PATCH 0/4] Add support for charging battery in Nokia RX-51 Pali Rohár 2013-09-08 8:50 ` Pali Rohár 2013-09-08 8:50 ` [PATCH 1/4] usb: musb: Call atomic_notifier_call_chain when status is changed Pali Rohár 2013-09-08 8:50 ` Pali Rohár 2013-09-17 15:48 ` Felipe Balbi 2013-09-17 15:48 ` Felipe Balbi 2013-09-17 15:48 ` Felipe Balbi 2013-09-17 15:49 ` Felipe Balbi 2013-09-17 15:49 ` Felipe Balbi 2013-09-17 15:49 ` Felipe Balbi 2013-09-17 16:05 ` Pali Rohár 2013-09-17 16:05 ` Pali Rohár 2013-09-17 16:05 ` Pali Rohár 2013-09-17 16:08 ` Felipe Balbi 2013-09-17 16:08 ` Felipe Balbi 2013-09-17 16:08 ` Felipe Balbi 2013-09-17 19:28 ` Pali Rohár 2013-09-17 19:28 ` Pali Rohár 2013-09-18 1:49 ` Felipe Balbi 2013-09-18 1:49 ` Felipe Balbi 2013-09-18 1:49 ` Felipe Balbi 2013-09-18 8:20 ` Pali Rohár 2013-09-18 8:20 ` Pali Rohár 2013-09-18 8:20 ` Pali Rohár 2013-09-18 9:04 ` Javier Martinez Canillas 2013-09-18 9:04 ` Javier Martinez Canillas 2013-09-18 9:04 ` Javier Martinez Canillas 2013-09-18 13:30 ` Pavel Machek 2013-09-18 13:30 ` Pavel Machek 2013-09-18 13:57 ` Javier Martinez Canillas 2013-09-18 13:57 ` Javier Martinez Canillas 2013-09-18 14:22 ` Pavel Machek [this message] 2013-09-18 14:22 ` Pavel Machek 2013-09-18 14:35 ` Pavel Machek 2013-09-18 14:35 ` Pavel Machek 2013-09-18 16:25 ` Felipe Balbi 2013-09-18 16:25 ` Felipe Balbi 2013-09-18 16:25 ` Felipe Balbi 2013-09-18 14:53 ` Javier Martinez Canillas 2013-09-18 14:53 ` Javier Martinez Canillas 2013-09-18 15:56 ` Pali Rohár 2013-09-18 15:56 ` Pali Rohár 2013-09-18 15:56 ` Pali Rohár 2013-09-18 16:36 ` Felipe Balbi 2013-09-18 16:36 ` Felipe Balbi 2013-09-18 16:36 ` Felipe Balbi 2013-09-18 16:43 ` Pali Rohár 2013-09-18 16:43 ` Pali Rohár 2013-09-18 16:43 ` Pali Rohár 2013-09-18 16:48 ` Felipe Balbi 2013-09-18 16:48 ` Felipe Balbi 2013-09-18 16:48 ` Felipe Balbi 2013-09-18 17:03 ` [PATCH usb 1/2] usb: musb: Add missing ATOMIC_INIT_NOTIFIER_HEAD Pali Rohár 2013-09-18 17:03 ` Pali Rohár 2013-09-18 17:03 ` Pali Rohár 2013-09-25 8:17 ` Pali Rohár 2013-09-25 8:17 ` Pali Rohár 2013-09-25 8:17 ` Pali Rohár 2013-09-25 20:33 ` Felipe Balbi 2013-09-25 20:33 ` Felipe Balbi 2013-09-25 20:33 ` Felipe Balbi 2013-09-26 0:00 ` Pavel Machek 2013-09-26 0:00 ` Pavel Machek 2013-10-01 14:22 ` Felipe Balbi 2013-10-01 14:22 ` Felipe Balbi 2013-10-01 14:22 ` Felipe Balbi 2013-09-18 17:03 ` [PATCH usb 2/2] usb: musb: Call atomic_notifier_call_chain when status is changed Pali Rohár 2013-09-18 17:03 ` Pali Rohár 2013-09-18 17:03 ` Pali Rohár 2013-09-08 8:50 ` [PATCH 2/4] power: isp1704_charger: Fix driver to work with changes introduced in v3.5 Pali Rohár 2013-09-08 8:50 ` Pali Rohár 2013-10-22 21:03 ` Anton Vorontsov 2013-10-22 21:03 ` Anton Vorontsov 2013-09-08 8:50 ` [PATCH 3/4] power: isp1704_charger: Add callback function set_current Pali Rohár 2013-09-08 8:50 ` Pali Rohár 2013-09-08 8:50 ` [PATCH 4/4] RX-51: Add platform function and data for bq24150a charger Pali Rohár 2013-09-08 8:50 ` Pali Rohár 2013-09-09 13:39 ` Sebastian Reichel 2013-09-09 13:39 ` Sebastian Reichel 2013-09-20 19:22 ` Pali Rohár 2013-09-20 19:22 ` Pali Rohár 2013-09-23 18:03 ` Tony Lindgren 2013-09-23 18:03 ` Tony Lindgren 2013-09-23 19:16 ` Pali Rohár 2013-09-23 19:16 ` Pali Rohár 2013-09-23 20:00 ` Sebastian Reichel 2013-09-23 20:00 ` Sebastian Reichel 2013-09-23 20:06 ` Pali Rohár 2013-09-23 20:06 ` Pali Rohár 2013-09-23 20:47 ` Sebastian Reichel 2013-09-23 20:47 ` Sebastian Reichel 2013-09-23 23:11 ` Tony Lindgren 2013-09-23 23:11 ` Tony Lindgren 2013-09-24 0:05 ` Pavel Machek 2013-09-24 0:05 ` Pavel Machek 2013-09-24 17:05 ` Pali Rohár 2013-09-24 17:05 ` Pali Rohár 2013-09-24 20:50 ` Sebastian Reichel 2013-09-24 20:50 ` Sebastian Reichel 2013-09-14 9:38 ` [PATCH 0/4] Add support for charging battery in Nokia RX-51 Pali Rohár 2013-09-14 9:38 ` Pali Rohár 2013-09-21 13:42 ` Pali Rohár 2013-09-21 13:42 ` Pali Rohár 2013-11-19 10:18 ` [PATCH v2 0/3] " Pali Rohár 2013-11-19 10:18 ` [PATCH v2 1/3] power_supply: Add power_supply notifier Pali Rohár 2013-11-19 13:10 ` Pavel Machek 2013-11-19 13:19 ` Pali Rohár 2013-11-24 17:02 ` Sebastian Reichel 2013-11-27 17:23 ` Tc, Jenny 2013-11-27 17:23 ` Tc, Jenny 2013-12-01 22:20 ` Anton Vorontsov 2013-12-01 22:23 ` Anton Vorontsov 2013-12-01 22:23 ` Anton Vorontsov 2014-01-03 11:09 ` Tc, Jenny 2014-01-03 11:09 ` Tc, Jenny 2014-01-03 16:02 ` Anton Vorontsov 2013-11-19 10:18 ` [PATCH v2 2/3] bq2415x_charger: Use power_supply notifier for automode Pali Rohár 2013-11-19 13:24 ` Pavel Machek 2013-12-24 1:58 ` Anton Vorontsov 2013-11-24 17:00 ` Sebastian Reichel 2013-11-24 19:01 ` Pali Rohár 2013-11-24 19:26 ` Sebastian Reichel 2013-11-24 19:41 ` Pali Rohár 2013-11-24 21:00 ` Sebastian Reichel 2013-11-25 14:01 ` Pavel Machek 2013-11-25 14:10 ` Pali Rohár 2013-11-25 15:18 ` Pavel Machek 2013-11-25 18:52 ` Pali Rohár 2013-11-24 17:18 ` Michael Trimarchi 2013-11-24 17:18 ` Michael Trimarchi 2013-11-24 19:01 ` Pali Rohár 2013-11-25 19:16 ` Michael Trimarchi 2013-11-25 19:16 ` Michael Trimarchi 2013-12-01 22:37 ` Anton Vorontsov 2013-12-02 0:02 ` Michael Trimarchi 2013-12-02 0:24 ` Anton Vorontsov 2013-12-02 1:45 ` Michael Trimarchi 2014-01-19 20:54 ` Sebastian Reichel 2014-01-19 20:56 ` Michael Trimarchi 2014-01-20 10:21 ` Russell King - ARM Linux 2014-01-20 11:24 ` Sebastian Reichel 2013-12-06 13:32 ` Pali Rohár 2013-11-19 10:18 ` [PATCH v2 3/3] RX-51: Add platform data for bq24150a charger Pali Rohár 2013-11-19 13:32 ` Pavel Machek 2013-11-24 10:28 ` Pali Rohár 2013-11-19 14:44 ` [PATCH v2 0/3] Add support for charging battery in Nokia RX-51 Sebastian Reichel 2013-11-19 15:40 ` Tony Lindgren 2013-11-19 21:38 ` Pali Rohár
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=20130918142203.GD3748@amd.pavel.ucw.cz \ --to=pavel@ucw.cz \ --cc=aaro.koskinen@iki.fi \ --cc=anton@enomsg.org \ --cc=balbi@ti.com \ --cc=dwmw2@infradead.org \ --cc=freemangordon@abv.bg \ --cc=gregkh@linuxfoundation.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-omap@vger.kernel.org \ --cc=linux-usb@vger.kernel.org \ --cc=linux@arm.linux.org.uk \ --cc=martinez.javier@gmail.com \ --cc=pali.rohar@gmail.com \ --cc=tony@atomide.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.