AE-591: Created patch to remove the call to "enableEthPowerSupply" fr…

[bogdanarduino]

Created patch to remove the call to "enableEthPowerSupply" from "HAL_ETH_MspInit" when building the Portenta H7 variant. And introduced new bootloader binary in "STM32H747_manageBootloader.ino".

[pennam]

@bogdanarduino will ethernet work using bootloader 25 (or even previous versions) and removing enableEthPowerSupply ?

I would wait to officialy release the bootloader 26 before adding the bootloader dump in the core, to make sure we are using the exact same binary.

[bogdanarduino]

@bogdanarduino will ethernet work using bootloader 25 (or even previous versions) and removing enableEthPowerSupply ?

I would wait to officialy release the bootloader 26 before adding the bootloader dump in the core, to make sure we are using the exact same binary.

Yes, Ethernet functionality should be unaffected by these changes, since the bootloader only contains a fix for the nominal voltages being output by the board's PMIC DC-DC stage, but the rails are nevertheless still being turned on automatically during start-up, by the config of the OTP registers of the PMIC. What we did here is to remove the call to this, basically redundant function, which made absolutely sure that we have power supplied to the PHY before going to low power (stand-by) mode. We now moved the call to "turnOffEthernet" in the function which sets all the peripherals to low power "setAllPeripheralsPower(false).

[bogdanarduino]

@bogdanarduino will ethernet work using bootloader 25 (or even previous versions) and removing enableEthPowerSupply ?

I would wait to officialy release the bootloader 26 before adding the bootloader dump in the core, to make sure we are using the exact same binary.

Should I then remove the new bootloader form the header? Thank you very much for the comments!

[pennam]

Should I then remove the new bootloader form the header?

Yes please

[pennam]

Adding references to other PRs linked to this:

• AE-591: "turnOffEthernet" relocation to be executed before turning rails off. arduino-libraries/Arduino_PowerManagement#20
• Ae 591 configure correctly the sw1 and sw3 pmic output rails mcuboot-arduino-stm32h7#43

[sebromero]

I would wait to officialy release the bootloader 26

@pennam Have we already scheduled a release?

[pennam]

@sebromero no, I guess this is the only change we need to include. If so we can merge it as soon as is ready and create a pre-release of the binary to allow testing

[sebromero]

LGTM