removed (C)
On Fri, Aug 8, 2008 at 2:11 PM, Uwe Hermann <[EMAIL PROTECTED]> wrote:
> On Fri, Aug 08, 2008 at 10:21:55PM +0200, Peter Stuge wrote:
>> On Fri, Aug 08, 2008 at 10:00:39AM -0700, Sean Nelson wrote:
>> > -static inline uint64_t unpack_lb64(struct lb_uint64 value)
>> > +static inline uint64_t unpack_lb64(struct cb_uint64 value)
>> > {
>>
>> You missed the function names, but the rest looks good.
>>
>> Can you send a new patch with that fixed?
>
> Also, I'd rather not add an additional (C) for this patch, simple
> "fix typo"- or "whitespace-fixes"-style patches such as this don't
> warrant adding a (C) to the code (IMHO). (the other parts of your
> flashrom changes do, of course, as they're non-cosmetic and
> non-trivial).
>
>
> Uwe.
> --
> http://www.hermann-uwe.de | http://www.holsham-traders.de
> http://www.crazy-hacks.org | http://www.unmaintained-free-software.org
>
> --
> coreboot mailing list
> coreboot@coreboot.org
> http://www.coreboot.org/mailman/listinfo/coreboot
>
lb-to-cb-flashrom3.patch
Description: Binary data
-- coreboot mailing list coreboot@coreboot.org http://www.coreboot.org/mailman/listinfo/coreboot
