[coreboot] [libpayload] patch: add cbtables code to libpayload

Stefan Reinauer stepan at coresystems.de
Tue Aug 12 15:16:13 CEST 2008


Uwe Hermann wrote:
> On Mon, Aug 11, 2008 at 11:51:13PM -0700, Sean Nelson wrote:
>   
>> Patrick Georgi wrote:
>>     
>>> Sean Nelson schrieb:
>>>       
>>>> patch to move coreboot table code into libpayload. Allows payloads
>>>> using libpayload to access coreboot tables.
>>>>
>>>> Signed-Off-By: Sean Nelson <snelson at nmt.edu>
>>>>   
>>>>         
>>> What about licensing? libpayload is BSD-licensed, the cbtable code  
>>> seems to be GPL.
>>>
>>>
>>> Regards,
>>> Patrick Georgi
>>>
>>>       
>> Laziness got the best of me, this patch fixes the licensing.
>>     
>
>
>
>   
>> Index: drivers/cbtables.c
>> ===================================================================
>> --- drivers/cbtables.c	(revision 0)
>> +++ drivers/cbtables.c	(revision 0)
>> @@ -0,0 +1,161 @@
>> +/*
>> + * This file is part of the libpayload project.
>> + *
>> + * Copyright (C) 2008 Sean Nelson <snelson at nmt.edu>
>> + *
>> + * Redistribution and use in source and binary forms, with or without
>> + * modification, are permitted provided that the following conditions
>> + * are met:
>> + * 1. Redistributions of source code must retain the above copyright
>> + *    notice, this list of conditions and the following disclaimer.
>> + * 2. Redistributions in binary form must reproduce the above copyright
>> + *    notice, this list of conditions and the following disclaimer in the
>> + *    documentation and/or other materials provided with the distribution.
>> + * 3. The name of the author may not be used to endorse or promote products
>> + *    derived from this software without specific prior written permission.
>> + *
>> + * THIS SOFTWARE IS PROVIDED BY THE AUTHOR AND CONTRIBUTORS ``AS IS'' AND
>> + * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
>> + * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
>> + * ARE DISCLAIMED.  IN NO EVENT SHALL THE AUTHOR OR CONTRIBUTORS BE LIABLE
>> + * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
>> + * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
>> + * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
>> + * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
>> + * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
>> + * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
>> + * SUCH DAMAGE.
>> + */
>>     
>
> Eh, sorry, this won't work. The code was written by Jordan and is (C) AMD,
> you cannot just change the copyright owner or license at will.
> Jordan can do if he wants, though.

Does that also apply for cosmetic changes, btw?

-- 
coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br.
      Tel.: +49 761 7668825 • Fax: +49 761 7664613
Email: info at coresystems.dehttp://www.coresystems.de/
Registergericht: Amtsgericht Freiburg • HRB 7656
Geschäftsführer: Stefan Reinauer • Ust-IdNr.: DE245674866





More information about the coreboot mailing list