Hi Oleksandr, > On 15 Sep 2021, at 8:54 am, Oleksandr Andrushchenko > <[email protected]> wrote: > > Hi, Rahul! >>>> static void pci_config_write(pci_sbdf_t sbdf, unsigned int reg, >>>> unsigned int len, uint32_t val) >>>> { >>>> + struct pci_host_bridge *bridge = pci_find_host_bridge(sbdf.seg, >>>> sbdf.bus); >>>> + >>>> + if ( unlikely(!bridge) ) >>>> + { >>>> + printk(XENLOG_ERR "Unable to find bridge for "PRI_pci"\n", >>>> + sbdf.seg, sbdf.bus, sbdf.dev, sbdf.fn); >>> same here >> Yes I am printing with “PRI_pci". > > vPCI and the rest are widely using > > printk("%pp\n", &sbdf); > So, I think if we have SBDF then it is better to use %pp instead of trying to > unfold it manually.
Ok. I will use the %pp for printing the SBDF. Regards, Rahul
