Google Git
Sign in
gbmc/hothd/23ae9c8bec4afe6ddd90233cb24c8747d0e29b03
commit
23ae9c8bec4afe6ddd90233cb24c8747d0e29b03
[log]
author
Hao Jiang <jianghao@google.com>
Tue Oct 07 18:52:04 2025 +0000
committer
Hao Jiang <jianghao@google.com>
Thu Oct 16 09:56:37 2025 -0700
tree
21a63c8baad2c7713bc17cdc117d7f34dd911b4f
parent
8342a7d14decaf4024efd7596aeacb1dd747e47d [diff]
use-reset: listen to the chassis signal instead of host

Inherited from the dbus-sensors, the `powerMatch` for PowerState::on was
incorrectly pointed to the Host status.

That is a mismatch for the definition of x86-power-control:
https://github.com/openbmc/x86-power-control/blob/master/src/power_control.cpp#L515-L537

More specifically, `PowerState::checkForWarmReset` should be considered
as part of PowerState::On, instead of PowerState::Off.

The ChassisState of x86-power-control is more close to reflect the
desired hardware signal (PowerOk signal).
https://github.com/openbmc/x86-power-control/blob/master/src/power_control.cpp#L538-L559

The only delta is the `waitForSIOPowerGood` which is still considered as
Chassis.PowerState.Off, which happens after the `Event::powerOKAssert`.
https://github.com/openbmc/x86-power-control/blob/master/src/power_control.cpp#L1781

However, the SIO is not enabled on the target platform

Google-Bug-Id: 316465377
Tested: tested against a warmreboot machine and confirmed no usb reset
Change-Id: If441bdfb79f5262bbbc60235f3447ab3e0aae816
Signed-off-by: Hao Jiang <jianghao@google.com>
1 file changed
tree: 21a63c8baad2c7713bc17cdc117d7f34dd911b4f
  1. google3/
  2. subprojects/
  3. test/
  4. tools/
  5. usb_reset/
  6. xyz/
  7. yaml/
  8. .clang-tidy
  9. .clang-tidy-ignore
  10. 70-hothd-spi.rules
  11. 70-hothd-usb.rules
  12. 70-hothd.rules
  13. activation.hpp
  14. asset.cpp
  15. asset.hpp
  16. ec_util.cpp
  17. ec_util.hpp
  18. ec_util_interface.hpp
  19. firmware_mtd_updater.cpp
  20. firmware_mtd_updater.hpp
  21. firmware_spi_updater.cpp
  22. firmware_spi_updater.hpp
  23. firmware_updater_interface.hpp
  24. fs.cpp
  25. fs.hpp
  26. host_command.cpp
  27. host_command.hpp
  28. host_command_interface.hpp
  29. hoth.cpp
  30. hoth.hpp
  31. hoth_state.cpp
  32. hoth_state.hpp
  33. hothd-usb@.service.in
  34. hothd.conf
  35. hothd.service.in
  36. hothd_usb.sh.in
  37. libhoth_spi.cpp
  38. libhoth_spi.hpp
  39. libhoth_usb.cpp
  40. libhoth_usb.hpp
  41. libusb.cpp
  42. libusb.hpp
  43. libusb_impl.cpp
  44. libusb_impl.hpp
  45. LICENSE
  46. log_collector_util.cpp
  47. log_collector_util.hpp
  48. main.cpp
  49. MAINTAINERS
  50. meson.build
  51. meson_options.txt
  52. message_file.cpp
  53. message_file.hpp
  54. message_hoth_usb.cpp
  55. message_hoth_usb.hpp
  56. message_intf.hpp
  57. message_reinit.hpp
  58. message_spi.cpp
  59. message_spi.hpp
  60. message_util.cpp
  61. message_util.hpp
  62. mtd_util.cpp
  63. mtd_util.hpp
  64. mtd_util_interface.hpp
  65. payload_update.cpp
  66. payload_update.hpp
  67. payload_update_interface.hpp
  68. README.md
  69. sys.cpp
  70. sys.hpp
  71. sys_interface.hpp
  72. usb_util.cpp
  73. usb_util.hpp
  74. version.cpp
  75. version.hpp
README.md

gbmc-hothd

NOTE: This branch has been scurbbed so it can be upstream or share.