From: David Brown <davidb@codeaurora.org>
To: Russell King - ARM Linux <linux@arm.linux.org.uk>
Cc: David Brown <davidb@codeaurora.org>,
Daniel Walker <dwalker@fifo99.com>,
Bryan Huntsman <bryanh@codeaurora.org>,
linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org,
linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 1/3] ARM: msm: Remove MSM7x00 support
Date: Thu, 1 Dec 2011 12:26:32 -0800 [thread overview]
Message-ID: <20111201202632.GA25247@huya.qualcomm.com> (raw)
In-Reply-To: <20111201201950.GE9581@n2100.arm.linux.org.uk>
On Thu, Dec 01, 2011 at 08:19:50PM +0000, Russell King - ARM Linux wrote:
> On Thu, Dec 01, 2011 at 11:43:53AM -0800, David Brown wrote:
> > On Thu, Dec 01, 2011 at 11:33:12AM -0800, Daniel Walker wrote:
> > > On Thu, 2011-12-01 at 11:27 -0800, David Brown wrote:
> > > > On Thu, Dec 01, 2011 at 10:39:05AM -0800, Daniel Walker wrote:
> > > >
> > > > > > If someone is willing to step up and make it work, then we can keep it
> > > > > > around. My only G1 is fried, so I have no way to do anything other
> > > > > > than keep it compiling.
> > > > >
> > > > > Unfortunately, You can't remove it regardless of whether or not you have
> > > > > one to test on.. Your job is to do the best you can to keep it working
> > > > > and maintained. If you don't have one then you can't boot test but you
> > > > > still have to your best to keep it as close to working as possible.
> > > >
> > > > I was trying to get an idea if anyone uses the target. If people
> > > > want the target around, we can just ignore my patches to remove it.
> > >
> > > Please don't ever do this again.
> >
> > That's awfully harsh. I sent an email a few weeks back asking about
> > removing the target, and I only got one positive response. Sometimes
> > sending a patch is the best way to get a reaction.
>
> That's pretty normal for just 'asking' about doing something.
>
> You've done the right thing - if you ask, people treat your message as
> low priority and don't bother replying in the hope that nothing will
> happen.
>
> However, if you send a patch to delete a platform, people treat it as
> most urgent because if they care they need to speak up to actually stop
> it happening. (Even if you intend to only actually submit it in 6
> months time or so - but don't mention that in the initial posting!)
>
> Now, the obvious question to ask now is this: as you sent your question,
> and Daniel obviously objects, was Daniel one of your respondants? If
> not, then he carries some of the blame for this patch being created
> in the first place by having missed the email/not replied/etc.
Yeah, my mistake was forgetting to CC Daniel on the original question
about removing the target.
And I've learned what I needed to now: someone does still care about
the target.
David
--
Sent by an employee of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.
WARNING: multiple messages have this Message-ID (diff)
From: davidb@codeaurora.org (David Brown)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/3] ARM: msm: Remove MSM7x00 support
Date: Thu, 1 Dec 2011 12:26:32 -0800 [thread overview]
Message-ID: <20111201202632.GA25247@huya.qualcomm.com> (raw)
In-Reply-To: <20111201201950.GE9581@n2100.arm.linux.org.uk>
On Thu, Dec 01, 2011 at 08:19:50PM +0000, Russell King - ARM Linux wrote:
> On Thu, Dec 01, 2011 at 11:43:53AM -0800, David Brown wrote:
> > On Thu, Dec 01, 2011 at 11:33:12AM -0800, Daniel Walker wrote:
> > > On Thu, 2011-12-01 at 11:27 -0800, David Brown wrote:
> > > > On Thu, Dec 01, 2011 at 10:39:05AM -0800, Daniel Walker wrote:
> > > >
> > > > > > If someone is willing to step up and make it work, then we can keep it
> > > > > > around. My only G1 is fried, so I have no way to do anything other
> > > > > > than keep it compiling.
> > > > >
> > > > > Unfortunately, You can't remove it regardless of whether or not you have
> > > > > one to test on.. Your job is to do the best you can to keep it working
> > > > > and maintained. If you don't have one then you can't boot test but you
> > > > > still have to your best to keep it as close to working as possible.
> > > >
> > > > I was trying to get an idea if anyone uses the target. If people
> > > > want the target around, we can just ignore my patches to remove it.
> > >
> > > Please don't ever do this again.
> >
> > That's awfully harsh. I sent an email a few weeks back asking about
> > removing the target, and I only got one positive response. Sometimes
> > sending a patch is the best way to get a reaction.
>
> That's pretty normal for just 'asking' about doing something.
>
> You've done the right thing - if you ask, people treat your message as
> low priority and don't bother replying in the hope that nothing will
> happen.
>
> However, if you send a patch to delete a platform, people treat it as
> most urgent because if they care they need to speak up to actually stop
> it happening. (Even if you intend to only actually submit it in 6
> months time or so - but don't mention that in the initial posting!)
>
> Now, the obvious question to ask now is this: as you sent your question,
> and Daniel obviously objects, was Daniel one of your respondants? If
> not, then he carries some of the blame for this patch being created
> in the first place by having missed the email/not replied/etc.
Yeah, my mistake was forgetting to CC Daniel on the original question
about removing the target.
And I've learned what I needed to now: someone does still care about
the target.
David
--
Sent by an employee of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.
next prev parent reply other threads:[~2011-12-01 20:26 UTC|newest]
Thread overview: 43+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-12-01 0:29 [PATCH 1/3] ARM: msm: Remove MSM7x00 support David Brown
2011-12-01 0:29 ` David Brown
2011-12-01 0:29 ` [PATCH 2/3] gpio_msm: remove support for MSM7x01 David Brown
2011-12-01 0:29 ` David Brown
2011-12-01 0:29 ` [PATCH 3/3] ARM: msm: Remove cpu_is test for msm7201 soc David Brown
2011-12-01 0:29 ` David Brown
2011-12-01 15:17 ` [PATCH 1/3] ARM: msm: Remove MSM7x00 support Daniel Walker
2011-12-01 15:17 ` Daniel Walker
2011-12-01 15:17 ` Daniel Walker
2011-12-01 18:17 ` David Brown
2011-12-01 18:17 ` David Brown
2011-12-01 18:39 ` Daniel Walker
2011-12-01 18:39 ` Daniel Walker
2011-12-01 19:27 ` David Brown
2011-12-01 19:27 ` David Brown
2011-12-01 19:33 ` Daniel Walker
2011-12-01 19:33 ` Daniel Walker
2011-12-01 19:43 ` David Brown
2011-12-01 19:43 ` David Brown
2011-12-01 19:53 ` Daniel Walker
2011-12-01 19:53 ` Daniel Walker
2011-12-01 20:17 ` David Brown
2011-12-01 20:17 ` David Brown
2011-12-01 20:19 ` Russell King - ARM Linux
2011-12-01 20:19 ` Russell King - ARM Linux
2011-12-01 20:25 ` Daniel Walker
2011-12-01 20:25 ` Daniel Walker
2011-12-01 20:35 ` Russell King - ARM Linux
2011-12-01 20:35 ` Russell King - ARM Linux
2011-12-01 20:49 ` Daniel Walker
2011-12-01 20:49 ` Daniel Walker
2011-12-01 21:03 ` Russell King - ARM Linux
2011-12-01 21:03 ` Russell King - ARM Linux
2011-12-01 21:34 ` Daniel Walker
2011-12-01 21:34 ` Daniel Walker
2011-12-01 20:26 ` David Brown [this message]
2011-12-01 20:26 ` David Brown
2011-12-01 20:32 ` Daniel Walker
2011-12-01 20:32 ` Daniel Walker
2011-12-02 13:17 ` Denis 'GNUtoo' Carikli
2011-12-02 13:17 ` Denis 'GNUtoo' Carikli
2011-12-02 15:26 ` David Brown
2011-12-02 15:26 ` David Brown
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=20111201202632.GA25247@huya.qualcomm.com \
--to=davidb@codeaurora.org \
--cc=bryanh@codeaurora.org \
--cc=dwalker@fifo99.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
/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.