diff options
author | Rob Barnes <robbarnes@google.com> | 2020-05-12 18:30:10 +0000 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2020-06-09 04:18:37 +0000 |
commit | 687a4244e4d734ee8402cea7d796a1d49dc0b990 (patch) | |
tree | 242c7a5985d95d3a0f2b94b86246aa4ec7bd7a71 | |
parent | d5a4570063abd5883559f40af9f8f5192a143ee7 (diff) | |
download | vboot-687a4244e4d734ee8402cea7d796a1d49dc0b990.tar.gz |
Revert "futility: Adds platform check quirk for zork"
This reverts commit 8dac1697aef0114584d737e6a3027b55abdae5f0.
Reason for revert: Case sensitivity was turned off in https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/2192537. This quirk is no longer needed. It could result in accidental overwriting with incompatible firmware.
Original change's description:
> futility: Adds platform check quirk for zork
>
> Adds a quirk for futility on zork boards.
> Zork boards before 13073 used lowercase for the firmware
> names which causes the compatible platform check fail.
>
> This adds the disable_compatible_platform_check quirk and
> enables it by default for zork boards.
>
> BUG=b:156119908, b:155941790
> TEST=flashed Google_trembyle.13066.0.0 using servo
> chromeos-firmwareupdater --force
> rebooted and confirmed Google_Trembyle.13073.0.0 was flashed
> BRANCH=none
>
> Change-Id: I6fc6bf5bb42b725b5e7c9d0166f945b9c123bab4
> Signed-off-by: Rob Barnes <robbarnes@google.com>
> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/2191089
> Reviewed-by: Edward Hill <ecgh@chromium.org>
> Reviewed-by: Eric Peers <epeers@google.com>
> Commit-Queue: Edward Hill <ecgh@chromium.org>
Bug: b:156119908, b:155941790
Change-Id: I4688a380dbcd722e8875bf0b5331ecf797efd6a1
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/2197277
Tested-by: Rob Barnes <robbarnes@google.com>
Tested-by: Bhanu Prakash Maiya <bhanumaiya@chromium.org>
Reviewed-by: Rob Barnes <robbarnes@google.com>
Commit-Queue: Bhanu Prakash Maiya <bhanumaiya@chromium.org>
Commit-Queue: Rob Barnes <robbarnes@google.com>
Auto-Submit: Rob Barnes <robbarnes@google.com>
-rw-r--r-- | futility/updater.c | 3 | ||||
-rw-r--r-- | futility/updater.h | 1 | ||||
-rw-r--r-- | futility/updater_quirks.c | 29 |
3 files changed, 0 insertions, 33 deletions
diff --git a/futility/updater.c b/futility/updater.c index 336ec6d7..49a94b8e 100644 --- a/futility/updater.c +++ b/futility/updater.c @@ -1162,9 +1162,6 @@ enum updater_error_codes update_firmware(struct updater_config *cfg) image_to->file_name, image_to->ro_version, image_to->rw_version_a, image_to->rw_version_b); - if (try_apply_quirk(QUIRK_DISABLE_COMPATIBLE_PLATFORM_CHECK, cfg)) - return UPDATE_ERR_PLATFORM; - if (try_apply_quirk(QUIRK_MIN_PLATFORM_VERSION, cfg)) return UPDATE_ERR_PLATFORM; diff --git a/futility/updater.h b/futility/updater.h index 5013a789..e11cc745 100644 --- a/futility/updater.h +++ b/futility/updater.h @@ -44,7 +44,6 @@ enum quirk_types { QUIRK_ALLOW_EMPTY_WLTAG, QUIRK_EC_PARTIAL_RECOVERY, QUIRK_OVERRIDE_SIGNATURE_ID, - QUIRK_DISABLE_COMPATIBLE_PLATFORM_CHECK, QUIRK_MAX, }; diff --git a/futility/updater_quirks.c b/futility/updater_quirks.c index 22543de8..457a79b0 100644 --- a/futility/updater_quirks.c +++ b/futility/updater_quirks.c @@ -59,18 +59,6 @@ static const struct quirks_record quirks_records[] = { { .match = "Google_Wizpig.", .quirks = "allow_empty_wltag" }, { .match = "Google_Phaser.", .quirks = "override_signature_id" }, - - /* Zork Boards */ - { .match = "Google_Berknip.", - .quirks = "disable_compatible_platform_check" }, - { .match = "Google_Dalboz.", - .quirks = "disable_compatible_platform_check" }, - { .match = "Google_Ezkinil.", - .quirks = "disable_compatible_platform_check" }, - { .match = "Google_Morphius.", - .quirks = "disable_compatible_platform_check" }, - { .match = "Google_Trembyle.", - .quirks = "disable_compatible_platform_check" }, }; /* Preserves meta data and reload image contents from given file path. */ @@ -393,16 +381,6 @@ static int quirk_ec_partial_recovery(struct updater_config *cfg) } /* - * Disables compatible platform check. - * The compatible platform check ensures the current platform and update - * start with the same name. - */ -static int quirk_disable_compatible_platform_check(struct updater_config *cfg) { - cfg->check_platform = 0; - return 0; -} - -/* * Registers known quirks to a updater_config object. */ void updater_register_quirks(struct updater_config *cfg) @@ -455,13 +433,6 @@ void updater_register_quirks(struct updater_config *cfg) quirks->help = "chromium/146876241; override signature id for " "devices shipped with different root key."; quirks->apply = NULL; /* Simple config. */ - - quirks = &cfg->quirks[QUIRK_DISABLE_COMPATIBLE_PLATFORM_CHECK]; - quirks->name = "disable_compatible_platform_check"; - quirks->help = "b/155941790; Disables compatible platform check." - "The compatible platform check ensures the current" - " platform and update start with the same name."; - quirks->apply = quirk_disable_compatible_platform_check; } /* |