diff options
author | Olof Johansson <olofj@chromium.org> | 2011-07-15 16:14:14 -0700 |
---|---|---|
committer | Simon Glass <sjg@chromium.org> | 2011-08-29 10:58:59 -0700 |
commit | 3d3cc33366ecf081776f5580106b06da80a7c149 (patch) | |
tree | c47c10b048a2ac7817c32b35d02311d37081eb0b /lib | |
parent | 5b8557f7dc44fa7cb78626a90da3a24ef8aee988 (diff) |
CHROMIUM: crossystem_data bugfix
Well, this is embarrassing. add break statements to the switches. :)
BUG=chromium-os:17654
TEST=none
Change-Id: Ie8ba460e873fd8977b2e2d9016722925cc73a31c
Signed-off-by: Olof Johansson <olofj@chromium.org>
Reviewed-on: http://gerrit.chromium.org/gerrit/4211
Reviewed-by: Doug Anderson <dianders@chromium.org>
Reviewed-by: Simon Glass <sjg@chromium.org>
Diffstat (limited to 'lib')
-rw-r--r-- | lib/chromeos/crossystem_data.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/lib/chromeos/crossystem_data.c b/lib/chromeos/crossystem_data.c index b0f2817d29..c1f641d26c 100644 --- a/lib/chromeos/crossystem_data.c +++ b/lib/chromeos/crossystem_data.c @@ -214,17 +214,22 @@ int crossystem_data_embed_into_fdt(crossystem_data_t *cdata, void *fdt, switch (cdata->binf[2]) { case 0: err |= set_conststring_prop("active-ec-firmware", "RO"); + break; case 1: err |= set_conststring_prop("active-ec-firmware", "RW"); + break; } switch (cdata->binf[3]) { case RECOVERY_TYPE: err |= set_conststring_prop("firmware-type", "recovery"); + break; case NORMAL_TYPE: err |= set_conststring_prop("firmware-type", "normal"); + break; case DEVELOPER_TYPE: err |= set_conststring_prop("firmware-type", "developer"); + break; } err |= set_scalar_prop("recovery-reason", binf[4]); |