public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Jesper Nilsson <jesper.nilsson@axis.com>
To: Paul Bolle <pebolle@tiscali.nl>
Cc: Michal Marek <mmarek@suse.cz>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Mikael Starvik <starvik@axis.com>,
	"Valdis.Kletnieks@vt.edu" <Valdis.Kletnieks@vt.edu>
Subject: Re: [PATCH 00/17] drop unused Kconfig symbols
Date: Tue, 1 Nov 2011 11:12:27 +0100	[thread overview]
Message-ID: <20111101101226.GI26898@axis.com> (raw)
In-Reply-To: <1320140764.14409.135.camel@x61.thuisdomein>

On Tue, Nov 01, 2011 at 10:46:04AM +0100, Paul Bolle wrote:
> On Tue, 2011-11-01 at 09:59 +0100, Jesper Nilsson wrote:
> > On Mon, Oct 31, 2011 at 11:55:16PM +0100, Paul Bolle wrote:
> > > Just today Jesper Nilsson basically dropped that NAK. I assume Jesper
> > > will try to carry the patch in (one of) their tree(s). Is that correct?
> > 
> > The patch that reintroduces the Kconfigs, yes, the one that removes them, no.
> 
> But yesterday you wrote:
> On Fri, Oct 14, 2011 at 09:11:01PM +0200, Valdis.Kletnieks@vt.edu wrote:
> [...]
> > > (Personally, I'm tending towards unless it's going to be in linux-next within
> > > the next month or so, pull the Kconfig variables out and the driver can re-add
> > > them when it actually lands)
> > 
> > Ok, let's do it the right way and remove them now, and I can re-add
> > them in the same tree.
> 
> So your plans have changed? It's OK with me if they change, of course,
> but now I'm unsure what you intend to do.

Sorry, I was unclear and not complete in my sentence above.
I was referring above to the tree that the drivers will be added in,
not the one where the Kconfigs will be removed.

I assumed that the removal patch was handled elsewhere, so I didn't
pick up the original patch which removed them.

So, just to be clear on what I intend to do (now that I know what you meant):

- Pick up the removal patch to the CRIS-tree.
- Create a new tree/branch with the missing drivers and the Kconfigs readded.

Does this seem allright for you?

> Paul Bolle

/^JN - Jesper Nilsson
-- 
               Jesper Nilsson -- jesper.nilsson@axis.com

  reply	other threads:[~2011-11-01 10:12 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1319456255.2409.30.camel@x61.thuisdomein>
2011-10-31 22:42 ` [PATCH 00/17] drop unused Kconfig symbols Michal Marek
2011-10-31 22:55   ` Paul Bolle
2011-11-01  8:59     ` Jesper Nilsson
2011-11-01  9:46       ` Paul Bolle
2011-11-01 10:12         ` Jesper Nilsson [this message]
2011-11-01 10:35           ` Paul Bolle

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=20111101101226.GI26898@axis.com \
    --to=jesper.nilsson@axis.com \
    --cc=Valdis.Kletnieks@vt.edu \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mmarek@suse.cz \
    --cc=pebolle@tiscali.nl \
    --cc=starvik@axis.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox