<p>Patrick Georgi <strong>merged</strong> this change.</p><p><a href="https://review.coreboot.org/c/coreboot/+/29767">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  build bot (Jenkins): Verified
  Jonathan Neuschäfer: Looks good to me, approved

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">riscv: fix bug of sifive-gpt.py<br><br>The GPT version must be "00 00 01 00" and the little endian should be<br>represented as 0x10000.<br><br>Please refer to: https://en.wikipedia.org/wiki/GUID_Partition_Table<br><br>Change-Id: Ib025197fc96f32823e687a89de0cee51c952b031<br>Signed-off-by: Xiang Wang <wxjstz@126.com><br>Reviewed-on: https://review.coreboot.org/c/29767<br>Reviewed-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net><br>Tested-by: build bot (Jenkins) <no-reply@coreboot.org><br>---<br>M util/riscv/sifive-gpt.py<br>1 file changed, 1 insertion(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/util/riscv/sifive-gpt.py b/util/riscv/sifive-gpt.py</span><br><span>index 7f522d9..f1e4fa8 100755</span><br><span>--- a/util/riscv/sifive-gpt.py</span><br><span>+++ b/util/riscv/sifive-gpt.py</span><br><span>@@ -88,7 +88,7 @@</span><br><span>     def pack_with_crc(self, crc):</span><br><span>         header_size = 92</span><br><span>         header = struct.pack('<8sIIIIQQQQ16sQIII',</span><br><span style="color: hsl(0, 100%, 40%);">-                b'EFI PART', 0x100, header_size, crc, 0,</span><br><span style="color: hsl(120, 100%, 40%);">+                b'EFI PART', 0x10000, header_size, crc, 0,</span><br><span>                 self.current_lba, self.backup_lba, self.first_usable_lba,</span><br><span>                 self.last_usable_lba, self.uniq.get_bytes(),</span><br><span>                 self.part_entries_lba, self.part_entries_number,</span><br><span></span><br></pre><p>To view, visit <a href="https://review.coreboot.org/c/coreboot/+/29767">change 29767</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://review.coreboot.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://review.coreboot.org/c/coreboot/+/29767"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: coreboot </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Ib025197fc96f32823e687a89de0cee51c952b031 </div>
<div style="display:none"> Gerrit-Change-Number: 29767 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: Xiang Wang <wxjstz@126.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jonathan Neuschäfer <j.neuschaefer@gmx.net> </div>
<div style="display:none"> Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: Philipp Hug <philipp@hug.cx> </div>
<div style="display:none"> Gerrit-Reviewer: Shawn Chang <citypw@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Xiang Wang <wxjstz@126.com> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-CC: Patrick Rudolph <siro@das-labor.org> </div>
<div style="display:none"> Gerrit-MessageType: merged </div>