leandrof/u-boot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

binman: Adjust size of test SPL binary

Browse Source 
This is only 3 bytes long which is not enough to hold two symbol values,
needed to test the binman symbols feature. Increase it to 15 bytes.

Using very small regions is useful since we can easily compare them in
tests and errors are fairly easy to diagnose.

Signed-off-by: Simon Glass <sjg@chromium.org>
This commit is contained in:
Simon Glass 2017-11-13 18:54:59 -07:00
parent 5cfcf7e0fd
commit f689890d8e
5 changed files with 14 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
tools/binman/ftest.py
View File

@ -28,7 +28,7 @@ import tout
# Contents of test files, corresponding to different entry types
U_BOOT_DATA = '1234'
U_BOOT_IMG_DATA = 'img'
U_BOOT_SPL_DATA = '567'
U_BOOT_SPL_DATA = '56780123456789abcde'
BLOB_DATA = '89'
ME_DATA = '0abcd'
VGA_DATA = 'vga'
@ -565,7 +565,7 @@ class TestFunctional(unittest.TestCase):
def testImagePadByte(self):
"""Test that the image pad byte can be specified"""
data = self._DoReadFile('21_image_pad.dts')
self.assertEqual(U_BOOT_SPL_DATA + (chr(0xff) * 9) + U_BOOT_DATA, data)
self.assertEqual(U_BOOT_SPL_DATA + (chr(0xff) * 1) + U_BOOT_DATA, data)
def testImageName(self):
"""Test that image files can be named"""
@ -587,7 +587,7 @@ class TestFunctional(unittest.TestCase):
def testPackSorted(self):
"""Test that entries can be sorted"""
data = self._DoReadFile('24_sorted.dts')
self.assertEqual(chr(0) * 5 + U_BOOT_SPL_DATA + chr(0) * 2 +
self.assertEqual(chr(0) * 1 + U_BOOT_SPL_DATA + chr(0) * 2 +
U_BOOT_DATA, data)
def testPackZeroPosition(self):
@ -615,14 +615,14 @@ class TestFunctional(unittest.TestCase):
with self.assertRaises(ValueError) as e:
self._DoTestFile('28_pack_4gb_outside.dts')
self.assertIn("Node '/binman/u-boot': Position 0x0 (0) is outside "
"the image starting at 0xfffffff0 (4294967280)",
"the image starting at 0xffffffe0 (4294967264)",
str(e.exception))
def testPackX86Rom(self):
"""Test that a basic x86 ROM can be created"""
data = self._DoReadFile('29_x86-rom.dts')
self.assertEqual(U_BOOT_DATA + chr(0) * 3 + U_BOOT_SPL_DATA +
chr(0) * 6, data)
self.assertEqual(U_BOOT_DATA + chr(0) * 7 + U_BOOT_SPL_DATA +
chr(0) * 2, data)
def testPackX86RomMeNoDesc(self):
"""Test that an invalid Intel descriptor entry is detected"""

2
tools/binman/test/21_image_pad.dts
View File

@ -10,7 +10,7 @@
};
u-boot {
pos = <12>;
pos = <20>;
};
};
};

4
tools/binman/test/24_sorted.dts
View File

@ -7,11 +7,11 @@
binman {
sort-by-pos;
u-boot {
pos = <10>;
pos = <22>;
};
u-boot-spl {
pos = <5>;
pos = <1>;
};
};
};

4
tools/binman/test/28_pack_4gb_outside.dts
View File

@ -7,13 +7,13 @@
binman {
sort-by-pos;
end-at-4gb;
size = <16>;
size = <32>;
u-boot {
pos = <0>;
};
u-boot-spl {
pos = <0xfffffff7>;
pos = <0xffffffeb>;
};
};
};

6
tools/binman/test/29_x86-rom.dts
View File

@ -7,13 +7,13 @@
binman {
sort-by-pos;
end-at-4gb;
size = <16>;
size = <32>;
u-boot {
pos = <0xfffffff0>;
pos = <0xffffffe0>;
};
u-boot-spl {
pos = <0xfffffff7>;
pos = <0xffffffeb>;
};
};
};