diff options
| author | Vincent Palatin <vpalatin@chromium.org> | 2017-03-22 15:30:59 +0100 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2017-03-24 09:32:53 -0700 |
| commit | d9afaba9b4043a9d953b0d4c4e846520b04a0597 (patch) | |
| tree | 3dbb4cfce11e82158926c1d28f665c3f2a5b89a3 /include/usb_descriptor.h | |
| parent | e43ba03ebf3c921f2abfd4e9153725e87f28da62 (diff) | |
| download | chrome-ec-d9afaba9b4043a9d953b0d4c4e846520b04a0597.tar.gz | |
flash: ensure proper pstate alignment
The pstate structure is written using the low-level
flash_physical_write() function. As a consequence, it is supposed to
meet the CONFIG_FLASH_WRITE_SIZE alignment constraint.
Add a build-time assertion to avoid silent failures.
Slightly decrease the maximum size of the serial number string, so the
structure has a natural 32-byte alignment which is compatible with a
large number of platforms (including STM32L4 which requires 64-bit
alignment).
Of course, this change is not fully backward-compatible.
Signed-off-by: Vincent Palatin <vpalatin@chromium.org>
BRANCH=none
BUG=chromium:571477 b:35648258
TEST=on STM32L442, build and run 'flashrom --wp-enable' without failure.
Change-Id: Ia8f82790a61a6c7d2cf9bfeb95bfdaf7b8c52d11
Reviewed-on: https://chromium-review.googlesource.com/458201
Commit-Ready: Vincent Palatin <vpalatin@chromium.org>
Tested-by: Vincent Palatin <vpalatin@chromium.org>
Reviewed-by: Randall Spangler <rspangler@chromium.org>
Reviewed-by: Nick Sanders <nsanders@chromium.org>
Diffstat (limited to 'include/usb_descriptor.h')
| -rw-r--r-- | include/usb_descriptor.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/usb_descriptor.h b/include/usb_descriptor.h index 3c5352e452..1343e0d777 100644 --- a/include/usb_descriptor.h +++ b/include/usb_descriptor.h @@ -258,7 +258,7 @@ struct usb_setup_packet { #ifdef CONFIG_USB_SERIALNO /* String Descriptor for USB, for editable strings. */ -#define USB_STRING_LEN 30 +#define USB_STRING_LEN 28 struct usb_string_desc { uint8_t _len; uint8_t _type; |