ARM: amba: Make driver_override output consistent with other buses
authorGeert Uytterhoeven <geert+renesas@glider.be>
Tue, 10 Apr 2018 13:21:43 +0000 (15:21 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 26 Apr 2018 08:32:51 +0000 (10:32 +0200)
For AMBA devices with unconfigured driver override, the
"driver_override" sysfs virtual file is empty, while it contains
"(null)" for platform and PCI devices.

Make AMBA consistent with other buses by dropping the test for a NULL
pointer.

Note that contrary to popular belief, sprintf() handles NULL pointers
fine; they are printed as "(null)".

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Cc: stable <stable@vger.kernel.org>
Reviewed-by: Todd Kjos <tkjos@google.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/amba/bus.c

index fac8e36de11eaec7073c6d5dc4f4d140b65d5d97..f8c01fbef64d6e7bd341073f2dccc4d0de15b764 100644 (file)
@@ -70,9 +70,6 @@ static ssize_t driver_override_show(struct device *_dev,
 {
        struct amba_device *dev = to_amba_device(_dev);
 
-       if (!dev->driver_override)
-               return 0;
-
        return sprintf(buf, "%s\n", dev->driver_override);
 }