Printing PTR_ERR(NULL) is not useful so I have updated the error messages a little. Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> --- v2: completely different diff --git a/drivers/platform/x86/intel_ips.c b/drivers/platform/x86/intel_ips.c index 18dcb58..e5c115d 100644 --- a/drivers/platform/x86/intel_ips.c +++ b/drivers/platform/x86/intel_ips.c @@ -1329,9 +1329,7 @@ static void ips_debugfs_init(struct ips_driver *ips) ips->debug_root = debugfs_create_dir("ips", NULL); if (!ips->debug_root) { - dev_err(&ips->dev->dev, - "failed to create debugfs entries: %ld\n", - PTR_ERR(ips->debug_root)); + dev_err(&ips->dev->dev, "failed to create debugfs entries\n"); return; } @@ -1345,8 +1343,8 @@ static void ips_debugfs_init(struct ips_driver *ips) &ips_debugfs_ops); if (!ent) { dev_err(&ips->dev->dev, - "failed to create debug file: %ld\n", - PTR_ERR(ent)); + "failed to create debug file: %s\n", + node->name); goto err_cleanup; } } -- To unsubscribe from this list: send the line "unsubscribe platform-driver-x86" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html