Comment 4 for bug 1054422

Indeed, there's an error in v3 definition which disallows samsung_bl1_start field.

Fix should be simple:

$ bzr diff
=== modified file 'linaro_image_tools/hwpack/hwpack_fields.py'
--- linaro_image_tools/hwpack/hwpack_fields.py 2012-08-28 06:02:43 +0000
+++ linaro_image_tools/hwpack/hwpack_fields.py 2012-09-24 08:02:09 +0000
@@ -128,7 +128,7 @@
     BOOT_MIN_SIZE_FIELD: None,
     ROOT_MIN_SIZE_FIELD: None,
     LOADER_MIN_SIZE_FIELD: None,
- SAMSUNG_BL1_LEN_FIELD: None,
+ SAMSUNG_BL1_START_FIELD: None,
     SAMSUNG_BL1_LEN_FIELD: None,
     SAMSUNG_ENV_LEN_FIELD: None,
     SAMSUNG_BL2_LEN_FIELD: None,

Can you please try with that and let us know if it works? If so, we'll re-roll 2012.09.1 with the fix included.

Milo, can you please ensure we've got a test that covers samsung fields as well?