Get rid of the strlen calls, use the return value of sprintf instead.
Reviewed-by: Gerald Schaefer <gerald.schaefer@de.ibm.com>
Signed-off-by: Sebastian Ott <sebott@linux.vnet.ibm.com>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
{
struct zpci_dev *zdev = get_zdev(to_pci_dev(dev));
{
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);
}
static DEVICE_ATTR(function_id, S_IRUGO, show_fid, NULL);
{
struct zpci_dev *zdev = get_zdev(to_pci_dev(dev));
{
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);
}
static DEVICE_ATTR(function_handle, S_IRUGO, show_fh, NULL);
{
struct zpci_dev *zdev = get_zdev(to_pci_dev(dev));
{
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);
}
static DEVICE_ATTR(pchid, S_IRUGO, show_pchid, NULL);
{
struct zpci_dev *zdev = get_zdev(to_pci_dev(dev));
{
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);
}
static DEVICE_ATTR(pfgid, S_IRUGO, show_pfgid, NULL);