{
        struct zpci_dev *zdev = get_zdev(to_pci_dev(dev));
 
-       sprintf(buf, "0x%08x\n", zdev->fid);
-       return strlen(buf);
+       return sprintf(buf, "0x%08x\n", zdev->fid);
 }
 static DEVICE_ATTR(function_id, S_IRUGO, show_fid, NULL);
 
 {
        struct zpci_dev *zdev = get_zdev(to_pci_dev(dev));
 
-       sprintf(buf, "0x%08x\n", zdev->fh);
-       return strlen(buf);
+       return sprintf(buf, "0x%08x\n", zdev->fh);
 }
 static DEVICE_ATTR(function_handle, S_IRUGO, show_fh, NULL);
 
 {
        struct zpci_dev *zdev = get_zdev(to_pci_dev(dev));
 
-       sprintf(buf, "0x%04x\n", zdev->pchid);
-       return strlen(buf);
+       return sprintf(buf, "0x%04x\n", zdev->pchid);
 }
 static DEVICE_ATTR(pchid, S_IRUGO, show_pchid, NULL);
 
 {
        struct zpci_dev *zdev = get_zdev(to_pci_dev(dev));
 
-       sprintf(buf, "0x%02x\n", zdev->pfgid);
-       return strlen(buf);
+       return sprintf(buf, "0x%02x\n", zdev->pfgid);
 }
 static DEVICE_ATTR(pfgid, S_IRUGO, show_pfgid, NULL);