Hi,

We are doing some quick prototyping and noticing some issues. Look at the latest changeset ( http://git.denx.de/?p=u-boot.git;a=commit;h=026f30ec3e846edb85b5df8265d8cad0... ) that changed arch/arm/cpu/armv8/cpu-dt.c

We can find the diff here: http://git.denx.de/?p=u-boot.git;a=blobdiff;f=arch/arm/cpu/armv8/cpu-dt.c;h=...

For some reason #ifdef CONFIG_ARMV8_SEC_FIRMWARE_SUPPORT

Was moved to the line before the start of the definition of psci_update_dt(void *fdt). Our prototyping board uses PSCI, but we currently do not make use of CONFIG_ARMV8_SEC_FIRMWARE_SUPPORT.

From the changeset's commit message:

PSCI can be used on both multiple and single core socs. Current implementation only allows PSCI to work on multiple core socs. This patch removes this restriction so that PSCI can work on single core socs as well.

Can somebody explain to me why the function definition of psci_update_dt is removed when CONFIG_ARMV8_SEC_FIRMWARE_SUPPORT is not defined?

According to me, the only thing that should have been removed (based on the commit message) is #ifdef CONFIG_MP .

Kind regards, Bruno Chevalier