summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRichard Spiegel <richard.spiegel@amd.corp-partner.google.com>2017-09-11 15:21:14 -0700
committerAaron Durbin <adurbin@chromium.org>2017-09-15 19:28:11 +0000
commitc661c8eab1dab5b19d0528e264d5205641035d61 (patch)
tree5896e69a6e97bf86588e8c716a475f670c64fa42
parentea4649f65fece2e14a3f2b0d1b4f5835a76a1141 (diff)
soc/amd/stoneyridge: Improve code quality
Remove empty functions. In function pointer structure "device_operations", replace the 0 equality by NULL equality. Files: hda.c, sata.c and usb.c Change-Id: I9f8dc7681ab2e651872e69a8b2e990e59ebe80c9 Signed-off-by: Richard Spiegel <richard.spiegel@amd.corp-partner.google.com> Reviewed-on: https://review.coreboot.org/21522 Reviewed-by: Aaron Durbin <adurbin@chromium.org> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
-rw-r--r--src/soc/amd/stoneyridge/hda.c8
-rw-r--r--src/soc/amd/stoneyridge/sata.c2
-rw-r--r--src/soc/amd/stoneyridge/usb.c8
3 files changed, 5 insertions, 13 deletions
diff --git a/src/soc/amd/stoneyridge/hda.c b/src/soc/amd/stoneyridge/hda.c
index b41b999925..1774a4abc6 100644
--- a/src/soc/amd/stoneyridge/hda.c
+++ b/src/soc/amd/stoneyridge/hda.c
@@ -28,10 +28,6 @@ static const unsigned short pci_device_ids[] = {
0
};
-static void hda_init(struct device *dev)
-{
-}
-
static struct pci_operations lops_pci = {
.set_subsystem = pci_dev_set_subsystem,
};
@@ -40,8 +36,8 @@ static struct device_operations hda_audio_ops = {
.read_resources = pci_dev_read_resources,
.set_resources = pci_dev_set_resources,
.enable_resources = pci_dev_enable_resources,
- .init = hda_init,
- .scan_bus = 0,
+ .init = NULL,
+ .scan_bus = NULL,
.ops_pci = &lops_pci,
};
diff --git a/src/soc/amd/stoneyridge/sata.c b/src/soc/amd/stoneyridge/sata.c
index e2de3e6fdf..c1e54937bf 100644
--- a/src/soc/amd/stoneyridge/sata.c
+++ b/src/soc/amd/stoneyridge/sata.c
@@ -63,7 +63,7 @@ static struct device_operations sata_ops = {
.set_resources = pci_dev_set_resources,
.enable_resources = pci_dev_enable_resources,
.init = sata_init,
- .scan_bus = 0,
+ .scan_bus = NULL,
.ops_pci = &lops_pci,
};
diff --git a/src/soc/amd/stoneyridge/usb.c b/src/soc/amd/stoneyridge/usb.c
index c1caf7db4c..82984b9b82 100644
--- a/src/soc/amd/stoneyridge/usb.c
+++ b/src/soc/amd/stoneyridge/usb.c
@@ -26,16 +26,12 @@ static struct pci_operations lops_pci = {
.set_subsystem = pci_dev_set_subsystem,
};
-static void usb_init(struct device *dev)
-{
-}
-
static struct device_operations usb_ops = {
.read_resources = pci_ehci_read_resources,
.set_resources = pci_dev_set_resources,
.enable_resources = pci_dev_enable_resources,
- .init = usb_init,
- .scan_bus = 0,
+ .init = NULL,
+ .scan_bus = NULL,
.ops_pci = &lops_pci,
};