On Mon, Mar 09, 2015 at 02:27:46PM +0800, Chen, Tiejun wrote:
[...]
> >>+
> >>+ FILE *f = fopen(pci_device_vendor_path, "r");
> >>+ if (!f) {
> >>+ LOGE(ERROR,
> >>+ "pci device "PCI_BDF" does not have vendor attribute",
> >>+ pcidev->domain, pcidev->bus, pcidev->dev, pcidev->func);
> >>+ return 0xffff;
> >>+ }
> >>+ read_items = fscanf(f, "0x%lx\n", &pci_device_vendor);
>
> we have to refactor this as well,
>
> read_items = fscanf(f, "0x%hx\n", &pci_device_vendor);
>
> Right?
>
Sure. Just eliminate any warnings / errors.
Wei.
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
