Hi,

On Wed, 15 Jan 2025 12:04:02 -0500
Bill <[email protected]> wrote:

> i added a hack to allow arnv7h to build despite a -Werror trap - it is
> likely something that affects all users with the newest compiler-set,
> so maybe should be handled upstream, if someone wants to report the
> bug to gnuboot
> 
> > cc -O2 -g -Wall -Wextra -Wmissing-prototypes -I. -DCMOS_HAL=1   -c
> > -o accessors/cmos-hw-unix.o accessors/cmos-hw-unix.c  
> > >accessors/cmos-hw-unix.c: In function ‘cmos_hal_read’:  
> > accessors/cmos-hw-unix.c:179:9: error: ‘return’ with no value, in
> > function returning non-void [-Wreturn-mismatch]  
> 
> +# armv7h error: ‘return’ with no value, in function returning
> non-void +if [ "${CARCH}" = 'armv7h' ] ; then
> +  export CPPFLAGS+=" -Wno-return-mismatch"
> +fi
For the record, we discussed that on IRC and this is the best way to
workaround the issue for now. Coreboot has fixed the issue so when we
update Coreboot, this would be fixed, but until then we need to
workaround in Parabola.

Also we would need more tests by users to be confident enough to update
the Coreboot revision, so it probably won't happen soon, especially
because the work in progress is more about documentation at the moment
than with updating the revisions of the software we reuse. Also
updating GRUB will probably happen way earlier than Coreboot as it
is less risky to do.

> is there any reason to keep both This and libretools-utils for some
> time?
I don't think this would be very useful as the newer Libreboot
releases would be risky to use as there is no requirements anymore not
to include nonfree software, and GNU Boot should have the same versions
of Coreboot/GRUB/SeaBIOS/etc than the latest free release of Libreboot.

And I also see no advantages of using mksource() just to keep the
libreboot-utils package around. If something different than GNU Boot is
desirable, for instance to have newer revisions, there is also
Canoeboot that can probably be used instead without mksource(),
probably as drop-in replacement for GNU Boot in the case of Parabola.

Canoeboot and GNU Boot also diverged in the tools that they provide:
for instance GNU Boot also provides utilities for installing
GNU Boot on I945 ThinkPads[1] that recent Canoeboot probably don't
provide anymore, and Canoeboot probably provides utilities for
installing Canoeboot on computers not yet supported by GNU Boot
(because the code is experimental and/or we didn't have the time yet to
look into it). 

In all these cases the utilities can also and should be packaged
separately as they are not from Coreboot.

References:
-----------
[1] While these tools are well known (basically bucts and a modified
    flashrom with patches), we only tested them with GNU Boot and not
    with other boot software distributions, so it might be best not to
    blindly mix and match installation instructions and images as
    otherwise there are cases where the installation could fail.

    In any case it should be safe to package them, but it is wise to
    also test them to see if they work fine or not and/or to warn users
    to only follow tutorials that mention specific images.

    More details are to be found in the
    585f4d359accf68f3ce438e4d1a27105c6ddd62e commit ("coreboot/i945
    Thinkpads: replace dd commands with INTEL_ADD_TOP_SWAP_BOOTBLOCK")
    inside GNU Boot.
   
    Also note that in our tests, bucts fails to reset the top swap to
    the way it was before, but thanks to this commit, things are OK.

Denis.

Attachment: pgp8M5W61nybK.pgp
Description: OpenPGP digital signature

_______________________________________________
Dev mailing list
[email protected]
https://lists.parabola.nu/mailman/listinfo/dev

Reply via email to