[LinuxBIOS] FYI flashrom included in Fedora Linux since yesterday.

[Peter Lemenkov]

Hello All!
Now not only a debian users can install it w/o any effords, but Fedora
Linux users too.

$ sudo yum install flashrom

Although this handy tool still in pending state for pushing into
Fedora 7 updates testing repository (just formal procedure but it
takes some time) users of Fedora Core 6 and Fedora Development may
install it as soon as mirrors will be synchronized with main Fedora
source.

Another one think I want to say - we have wishlist with small amount of entries:

==================================================
---------- Forwarded message ----------
From: [e-mail] 
Date: 06.09.2007 19:50
Subject: [Bug 250924] Review Request: flashrom - Simple program for
reading/writing BIOS chips content
To: [e-mail]

[sorry, skipped]

Optional:
Can you ask upstream to add an license header to udelay.c and include a copy of
their license in the repository?

There are some build warnings that should be reported upstream:
layout.c: In function 'read_romlayout':

layout.c:114: warning: ignoring return value of 'fscanf', declared with
attribute warn_unused_result
flashrom.c: In function 'main':

flashrom.c:402: warning: ignoring return value of 'fwrite', declared with
attribute warn_unused_result
  flashrom.c:421: warning: ignoring return value of 'fread', declared with
attribute warn_unused_result

And another one note - flashrom can't be built at PowerPC arch due to
lack of <sys/io.h>. Does anybody also tried to build it at this arch?
How powerpc-users of LinuxBIOS (if any) flash their machines?

[LinuxBIOS] FYI flashrom included in Fedora Linux since yesterday.

[Uwe Hermann]

On Sat, Sep 08, 2007 at 05:05:02PM +0400, Peter Lemenkov wrote:
> Now not only a debian users can install it w/o any effords, but Fedora
> Linux users too.
>
> $ sudo yum install flashrom

Great, thanks! I documented this in the wiki, too, see
http://linuxbios.org/Flashrom
> Optional:
> Can you ask upstream to add an license header to udelay.c and include a copy of

Done.
> their license in the repository?

Done.
> There are some build warnings that should be reported upstream:
> layout.c: In function 'read_romlayout':
>
> layout.c:114: warning: ignoring return value of 'fscanf', declared with
> attribute warn_unused_result
> flashrom.c: In function 'main':
>
> flashrom.c:402: warning: ignoring return value of 'fwrite', declared with
> attribute warn_unused_result
>   flashrom.c:421: warning: ignoring return value of 'fread', declared with
> attribute warn_unused_result

Which compiler options did you use here? I don't see any warnings using
the stock svn code and Makefile. The warnings don't look critical to me, btw.
> And another one note - flashrom can't be built at PowerPC arch due to
> lack of <sys/io.h>. Does anybody also tried to build it at this arch?
> How powerpc-users of LinuxBIOS (if any) flash their machines?

Yep, flashrom doesn't build on all platforms, yet. On some it might not
even make sense to have flashrom at all (maybe S/390, a mainframe which
probably doesn't even have a "BIOS", or at least not in the traditional
sense; I'm just guessing, though).

See http://buildd.debian.org/pkg.cgi?pkg=flashrom for the current
architecture build status of the Debian package.

Any help with porting flashrom is highly welcome!

Uwe.

[LinuxBIOS] FYI flashrom included in Fedora Linux since yesterday.

[Peter Stuge]

On Sun, Sep 09, 2007 at 10:09:03PM +0200, Uwe Hermann wrote:
> > flashrom.c:402: warning: ignoring return value of 'fwrite',
> > declared with attribute warn_unused_result
> > flashrom.c:421: warning: ignoring return value of 'fread',
> > declared with attribute warn_unused_result
>
> The warnings don't look critical to me, btw.

I guess glibc wants to emphasize that it's good form to check the
return value and handle errors.

//Peter