pw_digital_io_mcuxpresso: Fix bazel build
Remove no-op Enable method on interrupt controller as well, which fixes
build warning.
Change-Id: I94fd53ac0729cf654a427b9ff0e47332cb726fc3
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/154970
Presubmit-Verified: CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com>
Commit-Queue: Auto-Submit <auto-submit@pigweed-service-accounts.iam.gserviceaccount.com>
Pigweed-Auto-Submit: Austin Foxley <afoxley@google.com>
Reviewed-by: Ted Pudlik <tpudlik@google.com>
diff --git a/pw_digital_io_mcuxpresso/BUILD.bazel b/pw_digital_io_mcuxpresso/BUILD.bazel
index 6c505cc..44d4084 100644
--- a/pw_digital_io_mcuxpresso/BUILD.bazel
+++ b/pw_digital_io_mcuxpresso/BUILD.bazel
@@ -34,12 +34,14 @@
"public/pw_digital_io_mcuxpresso/interrupt_controller.h",
"public/pw_digital_io_mcuxpresso/interrupt_line.h",
],
+ includes = ["public"],
target_compatible_with = [
"//pw_build/constraints/board:mimxrt595_evk",
],
deps = [
- "//pw_digital_io:facade",
+ "//pw_digital_io",
"//pw_preprocessor",
+ "//pw_sync:borrow",
"@pigweed_config//:mcuxpresso_sdk",
],
)
diff --git a/pw_digital_io_mcuxpresso/interrupt_controller.cc b/pw_digital_io_mcuxpresso/interrupt_controller.cc
index 4b84022..0bd2f38 100644
--- a/pw_digital_io_mcuxpresso/interrupt_controller.cc
+++ b/pw_digital_io_mcuxpresso/interrupt_controller.cc
@@ -83,12 +83,6 @@
return pw::OkStatus();
}
-pw::Status McuxpressoInterruptController::Enable(pint_pin_int_t /*pin*/) {
- // Can not enabled at individual line level. Only at controller level, which
- // is always enabled.
- return pw::OkStatus();
-}
-
pw::Status McuxpressoInterruptController::EnableHandler(pint_pin_int_t pin,
bool enable) {
if (enable) {
diff --git a/pw_digital_io_mcuxpresso/interrupt_line.cc b/pw_digital_io_mcuxpresso/interrupt_line.cc
index 47ef00b..a4fc4d3 100644
--- a/pw_digital_io_mcuxpresso/interrupt_line.cc
+++ b/pw_digital_io_mcuxpresso/interrupt_line.cc
@@ -27,8 +27,10 @@
pint_pin_int_t pin)
: controller_(controller), pin_(pin) {}
-pw::Status McuxpressoDigitalInInterrupt::DoEnable(bool enable) {
- return controller_.acquire()->Enable(pin_);
+pw::Status McuxpressoDigitalInInterrupt::DoEnable(bool) {
+ // Can not enabled at individual line level. Only at controller level, which
+ // is always enabled.
+ return pw::OkStatus();
}
pw::Result<pw::digital_io::State> McuxpressoDigitalInInterrupt::DoGetState() {
diff --git a/pw_digital_io_mcuxpresso/public/pw_digital_io_mcuxpresso/interrupt_controller.h b/pw_digital_io_mcuxpresso/public/pw_digital_io_mcuxpresso/interrupt_controller.h
index 8b065f7..33b7e2d 100644
--- a/pw_digital_io_mcuxpresso/public/pw_digital_io_mcuxpresso/interrupt_controller.h
+++ b/pw_digital_io_mcuxpresso/public/pw_digital_io_mcuxpresso/interrupt_controller.h
@@ -32,7 +32,6 @@
pw::Status Config(pint_pin_int_t pin,
pw::digital_io::InterruptTrigger trigger,
pw::digital_io::InterruptHandler&& handler);
- pw::Status Enable(pint_pin_int_t pin);
pw::Status EnableHandler(pint_pin_int_t pin, bool enable);
pw::Result<pw::digital_io::State> GetState(pint_pin_int_t pin);