<p>Philipp Deppenwiese has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/24994">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">lib/cbfs: Remove cbfs_boot_map_optionrom from cbfs impl<br><br>* Move cbfs_boot_map_optionrom to device/pci_rom.c<br><br>Change-Id: I57a35d3ce95850d22f79a1045a30b7c6b011bb68<br>Signed-off-by: zaolin <zaolin@das-labor.org><br>---<br>M src/device/pci_rom.c<br>M src/include/cbfs.h<br>M src/lib/cbfs.c<br>3 files changed, 25 insertions(+), 26 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/94/24994/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/device/pci_rom.c b/src/device/pci_rom.c</span><br><span>index a411f06..e2a62cb 100644</span><br><span>--- a/src/device/pci_rom.c</span><br><span>+++ b/src/device/pci_rom.c</span><br><span>@@ -24,11 +24,35 @@</span><br><span> #include <device/pci_ids.h></span><br><span> #include <device/pci_ops.h></span><br><span> #include <string.h></span><br><span style="color: hsl(0, 100%, 40%);">-#include <cbfs.h></span><br><span style="color: hsl(120, 100%, 40%);">+#include <blob_provider.h></span><br><span> </span><br><span> /* Rmodules don't like weak symbols. */</span><br><span> u32 __attribute__((weak)) map_oprom_vendev(u32 vendev) { return vendev; }</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+static inline int tohex4(unsigned int c)</span><br><span style="color: hsl(120, 100%, 40%);">+{</span><br><span style="color: hsl(120, 100%, 40%);">+    return (c <= 9) ? (c + '0') : (c - 10 + 'a');</span><br><span style="color: hsl(120, 100%, 40%);">+}</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+static void tohex16(unsigned int val, char *dest)</span><br><span style="color: hsl(120, 100%, 40%);">+{</span><br><span style="color: hsl(120, 100%, 40%);">+   dest[0] = tohex4(val >> 12);</span><br><span style="color: hsl(120, 100%, 40%);">+    dest[1] = tohex4((val >> 8) & 0xf);</span><br><span style="color: hsl(120, 100%, 40%);">+ dest[2] = tohex4((val >> 4) & 0xf);</span><br><span style="color: hsl(120, 100%, 40%);">+ dest[3] = tohex4(val & 0xf);</span><br><span style="color: hsl(120, 100%, 40%);">+}</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+static void *cbfs_boot_map_optionrom(uint16_t vendor, uint16_t device)</span><br><span style="color: hsl(120, 100%, 40%);">+{</span><br><span style="color: hsl(120, 100%, 40%);">+      char name[17] = "pciXXXX,XXXX.rom";</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+       tohex16(vendor, name + 3);</span><br><span style="color: hsl(120, 100%, 40%);">+    tohex16(device, name + 8);</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+  return blob_read_arbitrary_map(COREBOOT_REGION, name,</span><br><span style="color: hsl(120, 100%, 40%);">+                                CBFS_TYPE_OPTIONROM, NULL);</span><br><span style="color: hsl(120, 100%, 40%);">+}</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> struct rom_header *pci_rom_probe(struct device *dev)</span><br><span> {</span><br><span>         struct rom_header *rom_header;</span><br><span>diff --git a/src/include/cbfs.h b/src/include/cbfs.h</span><br><span>index 1dac504..0dcd0cd 100644</span><br><span>--- a/src/include/cbfs.h</span><br><span>+++ b/src/include/cbfs.h</span><br><span>@@ -23,8 +23,6 @@</span><br><span>  * Perform CBFS operations on the boot device. *</span><br><span>  ***********************************************/</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-/* Return mapping of option ROM found in boot device. NULL on error. */</span><br><span style="color: hsl(0, 100%, 40%);">-void *cbfs_boot_map_optionrom(uint16_t vendor, uint16_t device);</span><br><span> /* Load stage by name into memory. Returns entry address on success. NULL on</span><br><span>  * failure. */</span><br><span> void *cbfs_boot_load_stage_by_name(const char *name);</span><br><span>diff --git a/src/lib/cbfs.c b/src/lib/cbfs.c</span><br><span>index 596abc5..709a9ee 100644</span><br><span>--- a/src/lib/cbfs.c</span><br><span>+++ b/src/lib/cbfs.c</span><br><span>@@ -141,29 +141,6 @@</span><br><span>   }</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static inline int tohex4(unsigned int c)</span><br><span style="color: hsl(0, 100%, 40%);">-{</span><br><span style="color: hsl(0, 100%, 40%);">-     return (c <= 9) ? (c + '0') : (c - 10 + 'a');</span><br><span style="color: hsl(0, 100%, 40%);">-}</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">-static void tohex16(unsigned int val, char *dest)</span><br><span style="color: hsl(0, 100%, 40%);">-{</span><br><span style="color: hsl(0, 100%, 40%);">-     dest[0] = tohex4(val>>12);</span><br><span style="color: hsl(0, 100%, 40%);">-        dest[1] = tohex4((val>>8) & 0xf);</span><br><span style="color: hsl(0, 100%, 40%);">-     dest[2] = tohex4((val>>4) & 0xf);</span><br><span style="color: hsl(0, 100%, 40%);">-     dest[3] = tohex4(val & 0xf);</span><br><span style="color: hsl(0, 100%, 40%);">-}</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">-void *cbfs_boot_map_optionrom(uint16_t vendor, uint16_t device)</span><br><span style="color: hsl(0, 100%, 40%);">-{</span><br><span style="color: hsl(0, 100%, 40%);">-       char name[17] = "pciXXXX,XXXX.rom";</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">-   tohex16(vendor, name+3);</span><br><span style="color: hsl(0, 100%, 40%);">-        tohex16(device, name+8);</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">-        return cbfs_boot_map_with_leak(name, CBFS_TYPE_OPTIONROM, NULL);</span><br><span style="color: hsl(0, 100%, 40%);">-}</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span> void *cbfs_boot_load_stage_by_name(const char *name)</span><br><span> {</span><br><span>       struct cbfsf fh;</span><br><span></span><br></pre><p>To view, visit <a href="https://review.coreboot.org/24994">change 24994</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/24994"/><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-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: I57a35d3ce95850d22f79a1045a30b7c6b011bb68 </div>
<div style="display:none"> Gerrit-Change-Number: 24994 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Philipp Deppenwiese <zaolin.daisuki@gmail.com> </div>