Message-ID: <1392164751.19535.15.camel@x220>
Subject: Re: [PATCH] staging: r8188eu: default to "y" in Kconfig
From: Paul Bolle <pebolle@tiscali.nl>
To: Martin Walch <walch.martin@web.de>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Richard Weinberger <richard@nod.at>, devel@driverdev.osuosl.org,
        linux-kernel@vger.kernel.org
Date: Wed, 12 Feb 2014 01:25:51 +0100
In-Reply-To: <10092086.3B4WUYQ8VT@tacticalops>
References: <1392121892.13064.9.camel@x220>
	 <10092086.3B4WUYQ8VT@tacticalops>
Content-Type: text/plain; charset="UTF-8"
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org

On Wed, 2014-02-12 at 01:09 +0100, Martin Walch wrote:
> On Tuesday 11 February 2014 13:31:32 Paul Bolle wrote:
> > This patch also deserves a
> >     Reported-by: Martin Walch <walch.martin@web.de>
> > 
> > line, but that apparently requires Martin's permission.
> 
> I would be happy.

It seems I interpreted the requirements for Reported-By: tags rather
strictly. We might change the documentation to discourage that
interpretation. (Greg, is it too late to add this tag in your queue?)

> And I am sorry for not sending patches myself. As I expect
> to find some more issues in Kconfig files soon, I wanted to gather all of
> them before sending any patches. However, I did not expect this to take me
> so long. Therefore I am glad you took the initiative and just created
> some patches.

My experience is that one can easily get stuck when trying to fix all
issues of a certain, broad category. Fixing all (say) all unused Kconfig
symbols is a big job. Fixing all that got added in v3.14-rc1 is doable
(depending on the amount of time one wants to spend).

Anyhow, I do hope you find time to send in patches yourself. Perhaps it
would help if we'd coordinate the stuff a bit. Richard seems keen to
help too.


Paul Bolle

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/