[PATCH] firmware-utils: fix compilation with macOS

Rosen Penev rosenp at gmail.com
Mon Feb 28 00:51:40 PST 2022


On Mon, Feb 28, 2022 at 12:43 AM Paul Spooren <mail at aparcar.org> wrote:
>
>
>
> > On 28. Feb 2022, at 08:34, Rosen Penev <rosenp at gmail.com> wrote:
> >
> > __bswap_32 is a GNU extension.
> >
> > Signed-off-by: Rosen Penev <rosenp at gmail.com>
> > —
>
> Seems to fix the compile issue on my Mac.
>
> Acked-by: Paul Spooren <mail at aparcar.org>
>
> If people change things in tools/, please use GitHub PRs. I added a CI to compile check changes on Ubuntu/macOS since breakage is a recurring issue.
The issue is that it would have to be implemented as a patch file,
which is bad form to be merged.
>
> > src/avm-wasp-checksum.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/src/avm-wasp-checksum.c b/src/avm-wasp-checksum.c
> > index 8c112f3..41a425e 100644
> > --- a/src/avm-wasp-checksum.c
> > +++ b/src/avm-wasp-checksum.c
> > @@ -156,7 +156,7 @@ int main(int argc, char *argv[])
> >               }
> >       }
> >       if (model == MODEL_X490)
> > -             crc = __bswap_32(crc);
> > +             crc = bswap_32(crc);
> >       fwrite(&crc, sizeof(uint32_t), 1, out_fp);
> >       if (ferror(out_fp)) {
> >               fprintf(stderr, "Error writing checksum to output file: %s\n", outfile);
> > --
> > 2.35.1
> >
> >
> > _______________________________________________
> > openwrt-devel mailing list
> > openwrt-devel at lists.openwrt.org
> > https://lists.openwrt.org/mailman/listinfo/openwrt-devel
>



More information about the openwrt-devel mailing list