[coreboot] [openvsa] patch: consistent application of LGPL

Marc Jones Marc.Jones at amd.com
Tue Feb 12 01:48:38 CET 2008


Uwe Hermann wrote:
> On Sat, Feb 09, 2008 at 01:19:58PM -0800, Chris Kilgour wrote:
>   
>> [openvsa] 
>>   AMD released the VSA code base under LGPL v2.1 in 2006, so
>>     
>
> LGPL-v2.1-or-later to be precise.
>
>
>   
>>   Also add a copy of the LGPL v2.1 to the OpenVSA tree
>>     
>
> Yep, good idea.
>
>
>   
>> Index: sysmgr/sysmgr.h.S
>> ===================================================================
>> --- sysmgr/sysmgr.h.S	(revision 2)
>> +++ sysmgr/sysmgr.h.S	(working copy)
>> @@ -1,4 +1,22 @@
>> +/*
>> +        Copyright (c) 2006 Advanced Micro Devices,Inc. ("AMD").
>>  
>> +        This library is free software; you can redistribute it and/or
>> +        modify it under the terms of the GNU Lesser General Public
>> +        License as published by the Free Software Foundation; either
>> +        version 2.1 of the License, or (at your option) any later
>> +        version.
>> + 
>> +        This code is distributed in the hope that it will be useful,
>> +        but WITHOUT ANY WARRANTY; without even the implied warranty of
>> +        MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
>> +        GNU Lesser General Public License for more details.
>> + 
>> +        You should have received a copy of the GNU Lesser General
>> +        Public License along with this library; if not, write to the
>> +        Free Software Foundation, Inc., 59 Temple Place, Suite 330,
>> +        Boston, MA 02111-1307 USA
>> +*/
>>  
>>  // Build flags for various optional features
>>  #define HISTORY				 0	// History support (value determines size of history buffer; 0=disabled)
>>     
>
> Patch looks good in general, but I'm not sure about the above change.
> You're adding a license to a file which didn't have one before and where
> you are not the copyright owner. I'm pretty sure the intention of AMD
> was to release _all_ files under LGPL, but I'd still like to get an
> ACK from someone at AMD, just to be sure.
>
> Otherwise this is:
> Acked-by: Uwe Hermann <uwe at hermann-uwe.de>
>
>
> Uwe.
>   
Acked-by: Marc Jones <marc.jones at amd.com>

All the files were released under the LGPL. If there is a header missing 
it is an over site by AMD and should be corrected.

I will try to get to building and trying the code tomorrow.

Thanks!
Marc

-- 
Marc Jones
Senior Firmware Engineer
(970) 226-9684 Office
mailto:Marc.Jones at amd.com
http://www.amd.com/embeddedprocessors 







More information about the coreboot mailing list