Remove features to rely on those generated by optional dependencies

Author: ithinuel

Diff for: .github/workflows/doctests.yml

@@ -12,7 +12,7 @@ jobs:
     strategy:
       matrix:
         include:
-          - features: thumbv6
+          - features: cortex-m
             nodefault: "--no-default-features"
           - features: ""
             nodefault: ""

Diff for: .github/workflows/embedded-builds.yml

@@ -14,7 +14,7 @@ jobs:
         include:
           - features: ""
             target: thumbv7em-none-eabihf
-          - feature: thumbv6
+          - feature: cortex-m
             target: thumbv6m-none-eabi
 
     steps:

Diff for: core/Cargo.toml

@@ -21,9 +21,5 @@ cortex-m = { version = "0.6.0", optional = true }
 version = "0.3.0"
 optional = true
 
-[features]
-thumbv6 = ["cortex-m"]
-defmt_0_3 = ["defmt"]
-
 [package.metadata.docs.rs]
 all-features = true

Diff for: core/src/bbbuffer.rs

@@ -63,7 +63,7 @@ impl<'a, const N: usize> BBBuffer<N> {
     /// is placed at `static` scope within the `.bss` region, the explicit initialization
     /// will be elided (as it is already performed as part of memory initialization)
     ///
-    /// NOTE:  If the `thumbv6` feature is selected, this function takes a short critical section
+    /// NOTE:  If the `cortex-m` feature is selected, this function takes a short critical section
     /// while splitting.
     ///
     /// ```rust
@@ -81,7 +81,7 @@ impl<'a, const N: usize> BBBuffer<N> {
     /// # }
     /// #
     /// # fn main() {
-    /// # #[cfg(not(feature = "thumbv6"))]
+    /// # #[cfg(not(feature = "cortex-m"))]
     /// # bbqtest();
     /// # }
     /// ```
@@ -123,7 +123,7 @@ impl<'a, const N: usize> BBBuffer<N> {
     /// is placed at `static` scope within the `.bss` region, the explicit initialization
     /// will be elided (as it is already performed as part of memory initialization)
     ///
-    /// NOTE:  If the `thumbv6` feature is selected, this function takes a short critical
+    /// NOTE:  If the `cortex-m` feature is selected, this function takes a short critical
     /// section while splitting.
     pub fn try_split_framed(&'a self) -> Result<(FrameProducer<'a, N>, FrameConsumer<'a, N>)> {
         let (producer, consumer) = self.try_split()?;
@@ -159,7 +159,7 @@ impl<'a, const N: usize> BBBuffer<N> {
     /// # }
     /// #
     /// # fn main() {
-    /// # #[cfg(not(feature = "thumbv6"))]
+    /// # #[cfg(not(feature = "cortex-m"))]
     /// # bbqtest();
     /// # }
     /// ```
@@ -345,7 +345,7 @@ impl<'a, const N: usize> Producer<'a, N> {
     /// # }
     /// #
     /// # fn main() {
-    /// # #[cfg(not(feature = "thumbv6"))]
+    /// # #[cfg(not(feature = "cortex-m"))]
     /// # bbqtest();
     /// # }
     /// ```
@@ -443,7 +443,7 @@ impl<'a, const N: usize> Producer<'a, N> {
     /// # }
     /// #
     /// # fn main() {
-    /// # #[cfg(not(feature = "thumbv6"))]
+    /// # #[cfg(not(feature = "cortex-m"))]
     /// # bbqtest();
     /// # }
     /// ```
@@ -548,7 +548,7 @@ impl<'a, const N: usize> Consumer<'a, N> {
     /// # }
     /// #
     /// # fn main() {
-    /// # #[cfg(not(feature = "thumbv6"))]
+    /// # #[cfg(not(feature = "cortex-m"))]
     /// # bbqtest();
     /// # }
     /// ```
@@ -674,7 +674,7 @@ impl<const N: usize> BBBuffer<N> {
     /// # }
     /// #
     /// # fn main() {
-    /// # #[cfg(not(feature = "thumbv6"))]
+    /// # #[cfg(not(feature = "cortex-m"))]
     /// # bbqtest();
     /// # }
     /// ```
@@ -691,7 +691,7 @@ impl<const N: usize> BBBuffer<N> {
 /// automatically be committed with `to_commit()`, then no bytes
 /// will be comitted for writing.
 ///
-/// If the `thumbv6` feature is selected, dropping the grant
+/// If the `cortex-m` feature is selected, dropping the grant
 /// without committing it takes a short critical section,
 #[derive(Debug, PartialEq)]
 pub struct GrantW<'a, const N: usize> {
@@ -712,7 +712,7 @@ unsafe impl<'a, const N: usize> Send for GrantW<'a, N> {}
 /// as read.
 ///
 ///
-/// If the `thumbv6` feature is selected, dropping the grant
+/// If the `cortex-m` feature is selected, dropping the grant
 /// without releasing it takes a short critical section,
 #[derive(Debug, PartialEq)]
 pub struct GrantR<'a, const N: usize> {
@@ -744,7 +744,7 @@ impl<'a, const N: usize> GrantW<'a, N> {
     /// If `used` is larger than the given grant, the maximum amount will
     /// be commited
     ///
-    /// NOTE:  If the `thumbv6` feature is selected, this function takes a short critical
+    /// NOTE:  If the `cortex-m` feature is selected, this function takes a short critical
     /// section while committing.
     pub fn commit(mut self, used: usize) {
         self.commit_inner(used);
@@ -770,7 +770,7 @@ impl<'a, const N: usize> GrantW<'a, N> {
     /// # }
     /// #
     /// # fn main() {
-    /// # #[cfg(not(feature = "thumbv6"))]
+    /// # #[cfg(not(feature = "cortex-m"))]
     /// # bbqtest();
     /// # }
     /// ```
@@ -861,7 +861,7 @@ impl<'a, const N: usize> GrantR<'a, N> {
     /// If `used` is larger than the given grant, the full grant will
     /// be released.
     ///
-    /// NOTE:  If the `thumbv6` feature is selected, this function takes a short critical
+    /// NOTE:  If the `cortex-m` feature is selected, this function takes a short critical
     /// section while releasing.
     pub fn release(mut self, used: usize) {
         // Saturate the grant release
@@ -903,7 +903,7 @@ impl<'a, const N: usize> GrantR<'a, N> {
     /// # }
     /// #
     /// # fn main() {
-    /// # #[cfg(not(feature = "thumbv6"))]
+    /// # #[cfg(not(feature = "cortex-m"))]
     /// # bbqtest();
     /// # }
     /// ```
@@ -969,7 +969,7 @@ impl<'a, const N: usize> SplitGrantR<'a, N> {
     /// If `used` is larger than the given grant, the full grant will
     /// be released.
     ///
-    /// NOTE:  If the `thumbv6` feature is selected, this function takes a short critical
+    /// NOTE:  If the `cortex-m` feature is selected, this function takes a short critical
     /// section while releasing.
     pub fn release(mut self, used: usize) {
         // Saturate the grant release
@@ -1004,7 +1004,7 @@ impl<'a, const N: usize> SplitGrantR<'a, N> {
     /// # }
     /// #
     /// # fn main() {
-    /// # #[cfg(not(feature = "thumbv6"))]
+    /// # #[cfg(not(feature = "cortex-m"))]
     /// # bbqtest();
     /// # }
     /// ```
@@ -1102,7 +1102,7 @@ impl<'a, const N: usize> DerefMut for GrantR<'a, N> {
     }
 }
 
-#[cfg(feature = "thumbv6")]
+#[cfg(feature = "cortex-m")]
 mod atomic {
     use core::sync::atomic::{
         AtomicBool, AtomicUsize,
@@ -1138,7 +1138,7 @@ mod atomic {
     }
 }
 
-#[cfg(not(feature = "thumbv6"))]
+#[cfg(not(feature = "cortex-m"))]
 mod atomic {
     use core::sync::atomic::{AtomicBool, AtomicUsize, Ordering};
 

Diff for: core/src/framed.rs

@@ -34,7 +34,7 @@
 //! # }
 //! #
 //! # fn main() {
-//! # #[cfg(not(feature = "thumbv6"))]
+//! # #[cfg(not(feature = "cortex-m"))]
 //! # bbqtest();
 //! # }
 //! ```

Diff for: core/src/lib.rs

@@ -90,7 +90,7 @@
 //! (mostly embedded) platforms atomic support is limited and with the default features you will get
 //! a compiler error about missing atomic methods.
 //!
-//! This crate contains special support for Cortex-M0(+) targets with the `thumbv6` feature. By
+//! This crate contains special support for Cortex-M0(+) targets with the `cortex-m` feature. By
 //! enabling the feature, unsupported atomic operations will be replaced with critical sections
 //! implemented by disabling interrupts. The critical sections are very short, a few instructions at
 //! most, so they should make no difference to most applications.