Re: ESP32 toolchain

[Greg Troxel <gdt%lexort.com@localhost>]

Frédéric Fauberteau <triaxx%netbsd.org@localhost> writes:

> (cross/xtensa-esp-elf-{binutils, gcc}).
> Actually, the produced binary is a ELF 32-bit MSB executable. But it seems that ESP32 chips are little endian and the cross GCC should produce a LSB executable.
>
> Therefore I have packaged in wip another cross toolchain: wip/xtensa-esp32-elf-{binutils, gcc} that uses the Espressif overlays. It is based on cross/xtensa-esp-elf-{binutils, gcc}: thank you tnn@ for your initial packaging.

I am theoretically interested but have not gotten to this yet.

The larger situation is seeming messy, at least in my head.  I don't
think it's ok to add a package with a similar name and different
behavior without trying to resolve the confusion.

ESP is a vague name, covering at least esp8266 and esp32 as I hear it.
Do you believe that cross/xtensa-esp-elf-gcc is really for esp8266 only,
and that the esp8266 family really is big endian?  Are you proposing to
change the DESCR for that port?  To rename it to esp8266 or to explain
that 'esp' means 'esp8266'?

Has anyone used xtensa-esp-elf-gcc to do anything successfully?
Could it just be that it's buggy?

These are nits, but I wonder if the cross/ packages should be upgraded
to 15, or if yours should match.  It would be nice to edit in variables
in both, in a NFCI way for cross/*, so that the diffs between them are
minimal, because it seems likely future maintenance should be in
parallel.  Perhaps even Makefile.common someday.

In wip/*gcc, this seems odd, or rather what's in cross seems odd, as the
commented-out lines reference a variable that's not set.

-#      ${CP} -r ${WRKDIR}/crosstool-NG-esp-13.2.0_20240530-src/overlays/xtensa_${ESP32_TYPE}/gcc/. ${WRKSRC}/.
-#      ${CP} -r ${WRKDIR}/crosstool-NG-esp-13.2.0_20240530-src/overlays/xtensa_${ESP32_TYPE}/newlib/. ${WRKSRC}/.
+       ${CP} -r ${WRKDIR}/${CROSSTOOLSRC}/overlays/xtensa_${ESP32_TYPE}/gcc/. ${WRKSRC}/.
+       ${CP} -r ${WRKDIR}/${CROSSTOOLSRC}/overlays/xtensa_${ESP32_TYPE}/newlib/. ${WRKSRC}/.

Looks like variable is needed for sysroot prefix

-CONFIGURE_ARGS+=       --with-local-prefix=${GNU_CONFIGURE_PREFIX}/xtensa-esp-elf
-CONFIGURE_ARGS+=       --with-sysroot=${GNU_CONFIGURE_PREFIX}/xtensa-esp-elf
+CONFIGURE_ARGS+=       --with-local-prefix=${GNU_CONFIGURE_PREFIX}/xtensa-esp32-elf
+CONFIGURE_ARGS+=       --with-sysroot=${GNU_CONFIGURE_PREFIX}/xtensa-esp32-elf

but also that this is a double subdir:

-GNU_CONFIGURE_PREFIX=  ${PREFIX}/xtensa-esp-elf
+GNU_CONFIGURE_PREFIX=  ${PREFIX}/xtensa-esp32-elf