On Sun, 2013-09-01 at 22:37 -0300, Fabio Estevam wrote: > On Sun, Sep 1, 2013 at 9:39 PM, Ben Hutchings <b...@decadent.org.uk> wrote: > > > pci_addr_phys = pdev->resource[0].start; > > - dev_info(&pdev->dev, "memory at 0x%08X\n", > > - (unsigned int)pci_addr_phys); > > + dev_info(&pdev->dev, "memory at 0x%08llX\n", > > + (unsigned long long)pci_addr_phys); > > You could use %pa instead for printing phys_addr_t: > > dev_info(&pdev->dev, "memory at %pa\n", &pci_addr_phys);
Could do, but I'm not sure it's clearer. And all these %p formats defeat type-checking anyway. Ben. -- Ben Hutchings If you seem to know what you are doing, you'll be given more to do.
signature.asc
Description: This is a digitally signed message part
_______________________________________________ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
