On Sat, May 18, 2019 at 09:58:59PM +0300, Sam Eiderman wrote:
> I wanted it to look more consistent with a dprintf just a few lines above:
> 
> dprintf(1, "Intel IGD OpRegion enabled at 0x%08x, size %dKB, dev "
>         "%02x:%02x.%x\n", (u32)addr, opregion->size >> 10,
>         pci_bdf_to_bus(bdf), pci_bdf_to_dev(bdf), pci_bdf_to_fn(bdf));
> 
> What do you think?

I'd use the dprintf(1, "%pP", dev) form instead.

-Kevin
_______________________________________________
SeaBIOS mailing list -- seabios@seabios.org
To unsubscribe send an email to seabios-le...@seabios.org

Reply via email to