Re: [PATCH v1 0/6] misc: add reboot mode driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




Hi Alexandre:

On 2015年12月23日 00:47, Alexandre Belloni wrote:
Hi,

On 22/12/2015 at 17:02:29 +0800, Andy Yan wrote :
This driver parse the reboot commands like "reboot loader"
and "reboot recovery" to get a boot mode described in the
device tree , then call the vendor specific write interfae
to store the boot mode in some place like special register
or sram , which can be read by the bootloader after system
reboot.

This is commonly done on Android based devices, in order to
reboot the device into fastboot or recovery mode.

Before this patch , I have try some hack on[0], and then found
John Stultz also doing the same work[1].

As John is busy these days, I go on with this work.

[0]https://patchwork.kernel.org/patch/7647751/
[1]https://patchwork.kernel.org/patch/7802391/

Changes in v1:
- fix the embarrassed compile warning
- correct the maskrom magic number
- check for the normal reboot
- correct the maskrom magic number
- use macro defined in rockchip_boot-mode.h for reboot-mode DT node

Andy Yan (6):
   dt-bindings: misc: add document for reboot-mode driver
   dt-bindings: soc: add document for rockchip reboot-mode driver
   misc: add reboot mode driver
   soc: rockchip: add reboot mode driver
   ARM: dts: rockchip: add reboot-mode node
   ARM64: dts: rockchip: add reboot-mode node

This seems nice and useful, some further ideas:

  .../devicetree/bindings/misc/reboot-mode.txt       | 41 +++++++++
  .../bindings/soc/rockchip/rockchip,reboot-mode.txt | 39 +++++++++
  arch/arm/boot/dts/rk3288.dtsi                      | 26 ++++++
  arch/arm/boot/dts/rk3xxx.dtsi                      | 26 ++++++
  arch/arm64/boot/dts/rockchip/rk3368.dtsi           | 26 ++++++
  drivers/misc/Kconfig                               |  7 ++
  drivers/misc/Makefile                              |  1 +
  drivers/misc/reboot_mode.c                         | 96 ++++++++++++++++++++++
I think this actually belongs to drivers/power/reset/ instead of misc

   I also have the idea to put is in drivers/power/reset,  But considering
this driver have not bind anything about power, so I put it in driver/misc
   at last. So I hope if some people can give more suggestions here.

  drivers/soc/rockchip/Kconfig                       |  9 ++
  drivers/soc/rockchip/Makefile                      |  1 +
  drivers/soc/rockchip/reboot.c                      | 68 +++++++++++++++
And maybe that part could be made generic instead of rockchip specific.
It simply uses a regmap to do the accesses, I guess a lot of other
platforms will do that. We have syscon-reboot and syscon-poweroff for example.

I think then we can extend the "framework" by having generic drivers to
store the value in eeprom or nvram for example.


I also hope the write interface can be generic. But I found some platform use different hardware to store the value. For example, John's patch use
   SRAM on qcom apq8064 to store value for nexus7. It seems there also have
some platform use dram or nvram to store it. And these different hardware use
   different write method. I don't have a generic way to handle this.

   I have a idea to handle it like this:

+static const struct of_device_id reboot_mode_dt_match[] = {
+ { .compatible = "linux,reboot-mode-sfr", /*for magic value stored in special function register, which can be accessed by regmap*/
+                .data = (void *)&reboot-mode-sfr },
+ { .compatible = "linux,reboot-mode-sram", /*for magic value stored in
+                .data = (void *)&reboot-mode-sram },
+        { .compatible = "linux,reboot-mode-sdram",
+ .data = (void *)&reboot-mode-sdram }, /*for magic value stored
+        { .compatible = "rockchip,reboot-mode-nvram",
+                .data = (void *)&reboot-mode-nvram },
+        {},
+};

   the data point to different hardware access method.

  Hope to see more suggestions from you.

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux