Hi Masahiro, On 12 April 2017 at 20:39, Masahiro Yamada <yamada.masah...@socionext.com> wrote: > If the system is running PSCI firmware, the System Reset function > (func ID: 0x80000009) is supposed to be handled by PSCI, that is, > the SoC/board specific reset implementation should be moved to PSCI. > U-Boot should call the PSCI service according to the arm-smccc > manner. > > The arm-smccc is supported on ARMv7 or later. Especially, ARMv8 > generation SoCs are likely to run ARM Trusted Firmware BL31. In > this case, U-Boot is a non-secure world boot loader, so it should > not be able to reset the system directly. > > Signed-off-by: Masahiro Yamada <yamada.masah...@socionext.com> > --- > > Changes in v2: None > > arch/arm/Kconfig | 1 + > drivers/Kconfig | 2 + > drivers/Makefile | 5 +- > drivers/firmware/Kconfig | 6 +++ > drivers/firmware/Makefile | 2 + > drivers/firmware/firmware-uclass.c | 11 +++++ > drivers/firmware/psci.c | 94 > ++++++++++++++++++++++++++++++++++++++ > drivers/sysreset/Kconfig | 10 ++++ > drivers/sysreset/Makefile | 1 + > drivers/sysreset/sysreset_psci.c | 41 +++++++++++++++++ > include/dm/uclass-id.h | 1 + > include/linux/psci.h | 13 ++++++ > 12 files changed, 185 insertions(+), 2 deletions(-) > create mode 100644 drivers/firmware/Kconfig > create mode 100644 drivers/firmware/Makefile > create mode 100644 drivers/firmware/firmware-uclass.c > create mode 100644 drivers/firmware/psci.c > create mode 100644 drivers/sysreset/sysreset_psci.c
Reviewed-by: Simon Glass <s...@chromium.org> It is not clear what the firmware uclass is for. Also there are no operations. How about adding a header file with at least a comment in it? - Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot