From 42c711a5b23c716bfd61a77254fabbf782fa21b2 Mon Sep 17 00:00:00 2001
From: Peter Barker <pbarker@barker.dropbear.id.au>
Date: Tue, 21 Aug 2018 21:40:16 +1000
Subject: [PATCH] AP_HAL_Linux: remove pointless attach_interrupt override

---
 libraries/AP_HAL_Linux/GPIO_BBB.cpp   | 8 --------
 libraries/AP_HAL_Linux/GPIO_BBB.h     | 5 -----
 libraries/AP_HAL_Linux/GPIO_RPI.cpp   | 8 --------
 libraries/AP_HAL_Linux/GPIO_RPI.h     | 5 -----
 libraries/AP_HAL_Linux/GPIO_Sysfs.cpp | 7 -------
 libraries/AP_HAL_Linux/GPIO_Sysfs.h   | 7 -------
 6 files changed, 40 deletions(-)

diff --git a/libraries/AP_HAL_Linux/GPIO_BBB.cpp b/libraries/AP_HAL_Linux/GPIO_BBB.cpp
index cff3b90161..3c5830b1c2 100644
--- a/libraries/AP_HAL_Linux/GPIO_BBB.cpp
+++ b/libraries/AP_HAL_Linux/GPIO_BBB.cpp
@@ -112,14 +112,6 @@ AP_HAL::DigitalSource* GPIO_BBB::channel(uint16_t n) {
     return new DigitalSource(n);
 }
 
-/* Interrupt interface: */
-bool GPIO_BBB::attach_interrupt(uint8_t interrupt_num,
-                                irq_handler_fn_t fn,
-                                INTERRUPT_TRIGGER_TYPE mode)
-{
-    return true;
-}
-
 bool GPIO_BBB::usb_connected(void)
 {
     return false;
diff --git a/libraries/AP_HAL_Linux/GPIO_BBB.h b/libraries/AP_HAL_Linux/GPIO_BBB.h
index f88ca12609..6244e5222a 100644
--- a/libraries/AP_HAL_Linux/GPIO_BBB.h
+++ b/libraries/AP_HAL_Linux/GPIO_BBB.h
@@ -129,11 +129,6 @@ public:
     /* Alternative interface: */
     AP_HAL::DigitalSource* channel(uint16_t n);
 
-    /* Interrupt interface: */
-    bool    attach_interrupt(uint8_t interrupt_num,
-                             irq_handler_fn_t fn,
-                             INTERRUPT_TRIGGER_TYPE mode) override;
-
     /* return true if USB cable is connected */
     bool    usb_connected(void);
 };
diff --git a/libraries/AP_HAL_Linux/GPIO_RPI.cpp b/libraries/AP_HAL_Linux/GPIO_RPI.cpp
index db5cd8f11b..b3b626465b 100644
--- a/libraries/AP_HAL_Linux/GPIO_RPI.cpp
+++ b/libraries/AP_HAL_Linux/GPIO_RPI.cpp
@@ -121,14 +121,6 @@ AP_HAL::DigitalSource* GPIO_RPI::channel(uint16_t n)
     return new DigitalSource(n);
 }
 
-/* Interrupt interface: */
-bool GPIO_RPI::attach_interrupt(uint8_t interrupt_num,
-                                irq_handler_fn_t fn,
-                                INTERRUPT_TRIGGER_TYPE mode)
-{
-    return true;
-}
-
 bool GPIO_RPI::usb_connected(void)
 {
     return false;
diff --git a/libraries/AP_HAL_Linux/GPIO_RPI.h b/libraries/AP_HAL_Linux/GPIO_RPI.h
index e0480f5298..7521bf9ef2 100644
--- a/libraries/AP_HAL_Linux/GPIO_RPI.h
+++ b/libraries/AP_HAL_Linux/GPIO_RPI.h
@@ -56,11 +56,6 @@ public:
     /* Alternative interface: */
     AP_HAL::DigitalSource* channel(uint16_t n);
 
-    /* Interrupt interface: */
-    bool    attach_interrupt(uint8_t interrupt_num,
-                             irq_handler_fn_t fn,
-                             INTERRUPT_TRIGGER_TYPE mode);
-
     /* return true if USB cable is connected */
     bool    usb_connected(void);
 
diff --git a/libraries/AP_HAL_Linux/GPIO_Sysfs.cpp b/libraries/AP_HAL_Linux/GPIO_Sysfs.cpp
index d08d30d282..dbf5af6ed4 100644
--- a/libraries/AP_HAL_Linux/GPIO_Sysfs.cpp
+++ b/libraries/AP_HAL_Linux/GPIO_Sysfs.cpp
@@ -188,13 +188,6 @@ AP_HAL::DigitalSource* GPIO_Sysfs::channel(uint16_t vpin)
     return new DigitalSource_Sysfs(pin, value_fd);
 }
 
-bool GPIO_Sysfs::attach_interrupt(uint8_t interrupt_num,
-                                  irq_handler_fn_t p,
-                                  INTERRUPT_TRIGGER_TYPE mode)
-{
-    return false;
-}
-
 bool GPIO_Sysfs::usb_connected(void)
 {
     return false;
diff --git a/libraries/AP_HAL_Linux/GPIO_Sysfs.h b/libraries/AP_HAL_Linux/GPIO_Sysfs.h
index a51c8a7c8a..d0315e8e4f 100644
--- a/libraries/AP_HAL_Linux/GPIO_Sysfs.h
+++ b/libraries/AP_HAL_Linux/GPIO_Sysfs.h
@@ -49,13 +49,6 @@ public:
      */
     AP_HAL::DigitalSource *channel(uint16_t vpin) override;
 
-    /*
-     * Currently this function always returns false.
-     */
-    bool attach_interrupt(uint8_t interrupt_num,
-                          irq_handler_fn_t p,
-                          INTERRUPT_TRIGGER_TYPE mode) override;
-
     /*
      * Currently this function always returns false.
      */