6 comments:
File src/ec/google/chromeec/ec.c:
I don't really care either way, but if you want to use Nicolas' variant please make sure to do […]
Ack
Patch Set #4, Line 747: #ifndef __PRE_RAM__
I'm actually not sure why this is __PRE_RAM__ scoped, that might just be wrong or waaaayy outdated. […]
Thanks for the explanation.
File src/ec/google/chromeec/ec_commands.h:
Patch Set #5, Line 4824: #define RESET_FLAG_AP_WATCHDOG (1 << 18) /* AP experienced a watchdog reset */
Is there a corresponding EC patch for this? Please always update ec_commands. […]
I have uploaded two CLs for ec_commands.h changes:
https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1520574
https://review.coreboot.org/c/coreboot/+/31885
File src/vendorcode/google/chromeos/Kconfig:
Patch Set #5, Line 93: bool "Use the AP watchdog flag stored in EC"
This option should not be user-selectable, so don't put a naming string here (just plain 'bool').
Done
Even though it's not user-selectable, please add a small 'help' paragraph explaining what it does (a […]
Done
File src/vendorcode/google/chromeos/watchdog.c:
Patch Set #5, Line 51: if (!CONFIG(CHROMEOS_USE_EC_WATCHDOG_FLAG)) {
I assume you're just trying to prevent the MTK driver from doing this? I don't think it makes sense […]
It seems like currently MTK WDT driver is the only one who calls mark_watchdog_tombstone...
For Kukui, actually it won't call mark_watchdog_tombstone during watchdog initialization (instead, AP thinks it were doing cold reboot).
I just wanted to reduce useless actions (as we're not reading the tombstone in this case).
If we check that in MTK WDT driver, then we're still changing their code, which is against your earlier comment...?
When you say "check !REGION_SIZE(watchdog_tombstone) here", do you mean to abort or to return directly? (There is already an assertion here)
And, I am not sure if we should remove the tombstone region from Kukui.
(As it only takes 4 bytes, removing it also requires changing MT8183 memlayout.ld)
To view, visit change 31834. To unsubscribe, or for help writing mail filters, visit settings.