[coreboot-gerrit] New patch to review for coreboot: e0b14b1 drivers/ioapic: DEVICE_NOOP some stub function callbacks

Edward O'Callaghan (eocallaghan@alterapraxis.com) gerrit at coreboot.org
Fri Nov 7 03:31:08 CET 2014


Edward O'Callaghan (eocallaghan at alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/7350

-gerrit

commit e0b14b1764e14dfff9be60dbd7b434a09da95721
Author: Edward O'Callaghan <eocallaghan at alterapraxis.com>
Date:   Fri Nov 7 12:43:49 2014 +1100

    drivers/ioapic: DEVICE_NOOP some stub function callbacks
    
    Just when you thought you found them all..
    Reduces loc and makes NOP's explicit.
    
    Change-Id: I416e0468b7f2f462c940daae695d67fb409aa4c6
    Signed-off-by: Edward O'Callaghan <eocallaghan at alterapraxis.com>
---
 src/drivers/generic/ioapic/ioapic.c | 17 +++++------------
 1 file changed, 5 insertions(+), 12 deletions(-)

diff --git a/src/drivers/generic/ioapic/ioapic.c b/src/drivers/generic/ioapic/ioapic.c
index 9c4bc7d..463474a 100644
--- a/src/drivers/generic/ioapic/ioapic.c
+++ b/src/drivers/generic/ioapic/ioapic.c
@@ -20,6 +20,7 @@ static void ioapic_init(struct device *dev)
 	u32 i, ioapic_interrupts;
 	u32 ioapic_base;
 	u8 ioapic_id;
+
 	if (!dev->enabled || !config)
 		return;
 
@@ -86,14 +87,6 @@ static void ioapic_init(struct device *dev)
 	}
 }
 
-static void ioapic_enable_resources(struct device *dev)
-{
-}
-
-static void ioapic_nop(struct device *dummy)
-{
-}
-
 static void ioapic_read_resources(struct device *dev)
 {
 	struct drivers_generic_ioapic_config *config = (struct drivers_generic_ioapic_config *)dev->chip_info;
@@ -106,10 +99,10 @@ static void ioapic_read_resources(struct device *dev)
 }
 
 static struct device_operations ioapic_operations = {
-        .read_resources   = ioapic_read_resources,
-        .set_resources    = ioapic_nop,
-        .enable_resources = ioapic_enable_resources,
-        .init             = ioapic_init,
+	.read_resources   = ioapic_read_resources,
+	.set_resources    = DEVICE_NOOP,
+	.enable_resources = DEVICE_NOOP,
+	.init             = ioapic_init,
 };
 
 static void enable_dev(struct device *dev)



More information about the coreboot-gerrit mailing list