From b8da04cd148547852447cf643af2a8e9caa9a058 Mon Sep 17 00:00:00 2001 From: Madhurima Paruchuri Date: Wed, 15 Feb 2023 09:41:24 +0000 Subject: binman: Add CBI section to fmap CBI section is added to fmap. The CBI section will be filled with 0xFF in binary. The size and offset could be overridden with device tree config in chip/program/project. A property named "preserve" is added, to determine the CBI portion should not be updated during flashing of chip. BRANCH=None BUG=b:266972341 TEST=dump_fmap && manually checking ec.bin content Change-Id: I43647285502917bf557b2fb4ce174c24538fb363 Signed-off-by: Madhurima Paruchuri Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/4253328 Reviewed-by: Wai-Hong Tam Reviewed-by: Simon Glass --- zephyr/dts/bindings/cros_flash/cros-ec,flash-layout.yaml | 6 ++++++ zephyr/include/cros/binman.dtsi | 6 ++++++ 2 files changed, 12 insertions(+) diff --git a/zephyr/dts/bindings/cros_flash/cros-ec,flash-layout.yaml b/zephyr/dts/bindings/cros_flash/cros-ec,flash-layout.yaml index 0ec9bdfbb0..afc716296d 100644 --- a/zephyr/dts/bindings/cros_flash/cros-ec,flash-layout.yaml +++ b/zephyr/dts/bindings/cros_flash/cros-ec,flash-layout.yaml @@ -37,3 +37,9 @@ properties: required: false description: >- Set if the section should be read-only. + preserve: + type: boolean + required: false + description: >- + Set if the section should be preserved when updating firmware. This is + just a signal. Enforcement of this is upto the updater. diff --git a/zephyr/include/cros/binman.dtsi b/zephyr/include/cros/binman.dtsi index 1ba7a3bde6..dcf494843f 100644 --- a/zephyr/include/cros/binman.dtsi +++ b/zephyr/include/cros/binman.dtsi @@ -28,6 +28,12 @@ size = <32>; text-label = "version"; }; + cbi { + type = "fill"; + size = <0>; + fill-byte = [ff]; + preserve; + }; }; }; ec-rw { -- cgit v1.2.1