Date: Thu, 17 Mar 2011 11:22:18 -0700
From: Andres Salomon <dilinger@queued.net>
To: Ed W <lists@wildgooses.com>
Cc: Grant Likely <grant.likely@secretlab.ca>, rpurdie@rpsys.net,
        linux-geode@lists.infradead.org, const@mimas.ru,
        linux-kernel@vger.kernel.org
Subject: Re: Feedback please: [PATCH] leds: New PCEngines Alix LED driver
 using gpio interface
Message-ID: <20110317112218.563a5f21@queued.net>
In-Reply-To: <4D824BEA.70502@wildgooses.com>
References: <4D81D7FD.1040602@wildgooses.com>
	<20110317084328.41831b4c@debxo>
	<AANLkTikga=zrNgfsuN4=8tCfZz0n7vkk6i0xW-zj6zSD@mail.gmail.com>
	<4D8243E7.7030309@wildgooses.com>
	<20110317105210.420e55c4@queued.net>
	<4D824BEA.70502@wildgooses.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1549
Lines: 36

On Thu, 17 Mar 2011 17:59:06 +0000
Ed W <lists@wildgooses.com> wrote:

> On 17/03/2011 17:52, Andres Salomon wrote:
> > OLPC stuff lives in arch/x86/platform/olpc; if there was more
> > alix-specific stuff, I'd suggest moving it into something similar.
> > However, I didn't find any.  Maybe an arch/x86/platform/geode as a
> > place to collect platform drivers for the various geode-based
> > machines out there (alix, soekris, etc)?  Though honestly, I'm not
> > that interested in doing the work to migrate stuff over to there.
> 
> 
> As this is my first stab at this, I think you are agreeing that the
> current change is ok as a first step at least?  (I can have a go at a
> larger code re-org later...)

Yes.


> 
> Can I get your final opinion on what should be in the code Copyright
> statement before I resubmit with the module_init change? To recap I
> don't want to alienate anyone, but the code is now based on a)
> leds-5501 and b) a patch from someone not currently credited in the
> file copyright statement?  I have stated this origin in the commit
> log and noted the derivation in the code itself. What to do..?


alix_present came from commit
ec9a943ce9f6d6a8ea09587b49d29a020c418c76 , which was from Constantin.
As such, it's probably not a good idea to remove the copyright notice.
--
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/