Hi Nandor, On Thu, 10 Jun 2021 at 07:57, Nandor Han <[email protected]> wrote: > > RTC devices could provide battery-backed memory that can be used for > storing the reboot mode magic value. > > Add a new reboot-mode back-end that uses RTC to store the reboot-mode > magic value. The driver also supports both endianness modes. > > Signed-off-by: Nandor Han <[email protected]> > --- > arch/sandbox/dts/test.dts | 10 ++ > configs/sandbox_defconfig | 1 + > .../reboot-mode/reboot-mode-rtc.txt | 22 +++ > drivers/reboot-mode/Kconfig | 9 ++ > drivers/reboot-mode/Makefile | 1 + > drivers/reboot-mode/reboot-mode-rtc.c | 127 ++++++++++++++++++ > include/reboot-mode/reboot-mode-rtc.h | 16 +++ > test/dm/reboot-mode.c | 29 ++++ > 8 files changed, 215 insertions(+) > create mode 100644 doc/device-tree-bindings/reboot-mode/reboot-mode-rtc.txt > create mode 100644 drivers/reboot-mode/reboot-mode-rtc.c > create mode 100644 include/reboot-mode/reboot-mode-rtc.h >
Reviewed-by: Simon Glass <[email protected]> But I think it would be better to put this device as a child of the rtc, so it is more obvious that they are related. It also avoids the phandle. What do you think?
