AMiRo-OS

commit a23878ddb85bb8fd069f02042f15ad4be2a0d709

Author: Marc Rothmann <mrothmann@techfak.uni-bielefeld.de>

Date:   Mon Sep 17 11:40:39 2018 +0200

    Added QEI driver to HAL.

diff --git a/os/hal/hal.mk b/os/hal/hal.mk

index f177a3f..64d96d9 100644

--- a/os/hal/hal.mk

+++ b/os/hal/hal.mk

@@ -41,6 +41,9 @@ endif

 ifneq ($(findstring HAL_USE_ICU TRUE,$(HALCONF)),)

 HALSRC += $(CHIBIOS)/os/hal/src/hal_icu.c

 endif

+ifneq ($(findstring HAL_USE_QEI TRUE,$(HALCONF)),)

+HALSRC += $(CHIBIOS)/os/hal/src/hal_qei.c

+endif

 ifneq ($(findstring HAL_USE_MAC TRUE,$(HALCONF)),)

 HALSRC += $(CHIBIOS)/os/hal/src/hal_mac.c

 endif

@@ -94,6 +97,7 @@ HALSRC = $(CHIBIOS)/os/hal/src/hal.c \

          $(CHIBIOS)/os/hal/src/hal_i2c.c \

          $(CHIBIOS)/os/hal/src/hal_i2s.c \

          $(CHIBIOS)/os/hal/src/hal_icu.c \

+         $(CHIBIOS)/os/hal/src/hal_qei.c \

          $(CHIBIOS)/os/hal/src/hal_mac.c \

          $(CHIBIOS)/os/hal/src/hal_mmc_spi.c \

          $(CHIBIOS)/os/hal/src/hal_pal.c \

diff --git a/os/hal/include/hal_qei.h b/os/hal/include/hal_qei.h

new file mode 100644

index 0000000..aef5e62

--- /dev/null

+++ b/os/hal/include/hal_qei.h

@@ -0,0 +1,148 @@

+/*

+AMiRo-OS is an operating system designed for the Autonomous Mini Robot (AMiRo) platform.

+Copyright (C) 2016..2018  Thomas Schöpping et al.

+

+This program is free software: you can redistribute it and/or modify

+it under the terms of the GNU General Public License as published by

+the Free Software Foundation, either version 3 of the License, or

+(at your option) any later version.

+

+This program is distributed in the hope that it will be useful,

+but WITHOUT ANY WARRANTY; without even the implied warranty of

+MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the

+GNU General Public License for more details.

+

+You should have received a copy of the GNU General Public License

+along with this program.  If not, see <http://www.gnu.org/licenses/>.

+*/

+

+/**

+ * @file    hal_qei.h

+ * @brief   QEI Driver macros and structures.

+ *

+ * @addtogroup QEI

+ * @{

+ */

+

+#ifndef _HAL_QEI_H_

+#define _HAL_QEI_H_

+

+#if HAL_USE_QEI || defined(__DOXYGEN__)

+

+/*===========================================================================*/

+/* Driver constants.                                                         */

+/*===========================================================================*/

+

+/*===========================================================================*/

+/* Driver pre-compile time settings.                                         */

+/*===========================================================================*/

+

+/*===========================================================================*/

+/* Derived constants and error checks.                                       */

+/*===========================================================================*/

+

+/*===========================================================================*/

+/* Driver data structures and types.                                         */

+/*===========================================================================*/

+

+/**

+ * @brief   Driver state machine possible states.

+ */

+typedef enum {

+  QEI_UNINIT = 0,                   /**< Not initialized.                   */

+  QEI_STOP = 1,                     /**< Stopped.                           */

+  QEI_READY = 2,                    /**< Ready.                             */

+  QEI_ACTIVE = 4,                   /**< Active.                            */

+} qeistate_t;

+

+/**

+ * @brief   Type of a structure representing an QEI driver.

+ */

+typedef struct QEIDriver QEIDriver;

+

+#include "hal_qei_lld.h"

+

+/*===========================================================================*/

+/* Driver macros.                                                            */

+/*===========================================================================*/

+

+/**

+ * @name    Macro Functions

+ * @{

+ */

+/**

+ * @brief   Enables the quadrature encoder.

+ *

+ * @param[in] qeip      pointer to the @p QEIDriver object

+ *

+ * @iclass

+ */

+#define qeiEnableI(qeip) qei_lld_enable(qeip)

+

+/**

+ * @brief   Disables the quadrature encoder.

+ *

+ * @param[in] qeip      pointer to the @p QEIDriver object

+ *

+ * @iclass

+ */

+#define qeiDisableI(qeip) qei_lld_disable(qeip)

+

+/**

+ * @brief   Returns the direction of the last transition.

+ * @details The direction is defined as boolean and is

+ *          calculated at each transition on any input.

+ *

+ * @param[in] qeip      pointer to the @p QEIDriver object

+ * @return              The request direction.

+ * @retval FALSE        Position counted up.

+ * @retval TRUE         Position counted down.

+ * @iclass

+ */

+#define qeiGetDirectionI(qeip) qei_lld_get_direction(qeip)

+

+/**

+ * @brief   Returns the position of the encoder.

+ * @details The position is defined as number of pulses since last reset.

+ *

+ * @param[in] qeip      pointer to the @p QEIDriver object

+ * @return              The number of pulses.

+ *

+ * @iclass

+ */

+#define qeiGetPositionI(qeip) qei_lld_get_position(qeip)

+

+/**

+ * @brief   Returns the range of the encoder.

+ * @details The range is defined as number of maximum pulse count.

+ *

+ * @param[in] qeip      pointer to the @p QEIDriver object

+ * @return              The number of pulses.

+ *

+ * @iclass

+ */

+#define qeiGetRangeI(qeip) qei_lld_get_range(qeip)

+/** @} */

+

+/*===========================================================================*/

+/* External declarations.                                                    */

+/*===========================================================================*/

+

+#ifdef __cplusplus

+extern "C" {

+#endif

+  void qeiInit(void);

+  void qeiObjectInit(QEIDriver *qeip);

+  void qeiStart(QEIDriver *qeip, const QEIConfig *config);

+  void qeiStop(QEIDriver *qeip);

+  void qeiEnable(QEIDriver *qeip);

+  void qeiDisable(QEIDriver *qeip);

+#ifdef __cplusplus

+}

+#endif

+

+#endif /* HAL_USE_QEI */

+

+#endif /* _HAL_QEI_H_ */

+

+/** @} */

diff --git a/os/hal/ports/STM32/LLD/I2Cv1/hal_i2c_lld.c b/os/hal/ports/STM32/LLD/I2Cv1/hal_i2c_lld.c

index 6ade226..96c9da0 100644

--- a/os/hal/ports/STM32/LLD/I2Cv1/hal_i2c_lld.c

+++ b/os/hal/ports/STM32/LLD/I2Cv1/hal_i2c_lld.c

@@ -34,6 +34,7 @@

 /* Driver local definitions.                                                 */

 /*===========================================================================*/

+#if STM32_I2C_I2C1_USE_DMA

 #define I2C1_RX_DMA_CHANNEL                                                 \

   STM32_DMA_GETCHANNEL(STM32_I2C_I2C1_RX_DMA_STREAM,                        \

                        STM32_I2C1_RX_DMA_CHN)

@@ -41,7 +42,9 @@

 #define I2C1_TX_DMA_CHANNEL                                                 \

   STM32_DMA_GETCHANNEL(STM32_I2C_I2C1_TX_DMA_STREAM,                        \

                        STM32_I2C1_TX_DMA_CHN)

+#endif

+#if STM32_I2C_I2C2_USE_DMA

 #define I2C2_RX_DMA_CHANNEL                                                 \

   STM32_DMA_GETCHANNEL(STM32_I2C_I2C2_RX_DMA_STREAM,                        \

                        STM32_I2C2_RX_DMA_CHN)

@@ -49,7 +52,9 @@

 #define I2C2_TX_DMA_CHANNEL                                                 \

   STM32_DMA_GETCHANNEL(STM32_I2C_I2C2_TX_DMA_STREAM,                        \

                        STM32_I2C2_TX_DMA_CHN)

+#endif

+#if STM32_I2C_I2C3_USE_DMA

 #define I2C3_RX_DMA_CHANNEL                                                 \

   STM32_DMA_GETCHANNEL(STM32_I2C_I2C3_RX_DMA_STREAM,                        \

                        STM32_I2C3_RX_DMA_CHN)

@@ -57,6 +62,7 @@

 #define I2C3_TX_DMA_CHANNEL                                                 \

   STM32_DMA_GETCHANNEL(STM32_I2C_I2C3_TX_DMA_STREAM,                        \

                        STM32_I2C3_TX_DMA_CHN)

+#endif

 /*===========================================================================*/

 /* Driver constants.                                                         */

@@ -72,6 +78,20 @@

 #define I2C_EV6_MASTER_REC_MODE_SELECTED                                    \

   ((uint32_t)(((I2C_SR2_MSL | I2C_SR2_BUSY)<< 16) | I2C_SR1_ADDR))

+#define I2C_EV7_MASTER_REC_BYTE_RECEIVED                                    \

+  ((uint32_t)(((I2C_SR2_MSL | I2C_SR2_BUSY)<< 16) | I2C_SR1_RXNE))

+

+#define I2C_EV7_MASTER_REC_BYTE_RECEIVED_STOP                               \

+  ((uint32_t)(I2C_SR1_RXNE))

+

+#define I2C_EV7_2_EV7_3_MASTER_REC_BYTE_QUEUED                              \

+  ((uint32_t)(((I2C_SR2_MSL | I2C_SR2_BUSY)<< 16) |                         \

+              I2C_SR1_BTF | I2C_SR1_RXNE))

+

+#define I2C_EV8_MASTER_BYTE_TRANSMITTING                                    \

+  ((uint32_t)(((I2C_SR2_MSL | I2C_SR2_BUSY | I2C_SR2_TRA)<< 16) |           \

+              I2C_SR1_TXE))

+

 #define I2C_EV8_2_MASTER_BYTE_TRANSMITTED                                   \

   ((uint32_t)(((I2C_SR2_MSL | I2C_SR2_BUSY | I2C_SR2_TRA) << 16) |          \

               I2C_SR1_BTF | I2C_SR1_TXE))

@@ -129,8 +149,24 @@ static void i2c_lld_abort_operation(I2CDriver *i2cp) {

   dp->SR1 = 0;

   /* Stops the associated DMA streams.*/

-  dmaStreamDisable(i2cp->dmatx);

-  dmaStreamDisable(i2cp->dmarx);

+#if STM32_I2C_USE_I2C1 && STM32_I2C_I2C1_USE_DMA

+  if (&I2CD1 == i2cp) {

+    dmaStreamDisable(i2cp->dmatx);

+    dmaStreamDisable(i2cp->dmarx);

+  }

+#endif

+#if STM32_I2C_USE_I2C2 && STM32_I2C_I2C2_USE_DMA

+  if (&I2CD2 == i2cp) {

+    dmaStreamDisable(i2cp->dmatx);

+    dmaStreamDisable(i2cp->dmarx);

+  }

+#endif

+#if STM32_I2C_USE_I2C3 && STM32_I2C_I2C3_USE_DMA

+  if (&I2CD3 == i2cp) {

+    dmaStreamDisable(i2cp->dmatx);

+    dmaStreamDisable(i2cp->dmarx);

+  }

+#endif

 }

 /**

@@ -236,13 +272,17 @@ static void i2c_lld_set_opmode(I2CDriver *i2cp) {

 }

 /**

- * @brief   I2C shared ISR code.

+ * @brief   I2C shared ISR code for DMA access.

  *

  * @param[in] i2cp      pointer to the @p I2CDriver object

  *

  * @notapi

  */

-static void i2c_lld_serve_event_interrupt(I2CDriver *i2cp) {

+#if (STM32_I2C_USE_I2C1 && STM32_I2C_I2C1_USE_DMA) ||                       \

+    (STM32_I2C_USE_I2C2 && STM32_I2C_I2C2_USE_DMA) ||                       \

+    (STM32_I2C_USE_I2C3 && STM32_I2C_I2C3_USE_DMA) ||                       \

+    defined(__DOXYGEN__)

+static void i2c_lld_serve_event_interrupt_dma(I2CDriver *i2cp) {

   I2C_TypeDef *dp = i2cp->i2c;

   uint32_t regSR2 = dp->SR2;

   uint32_t event = dp->SR1;

@@ -252,7 +292,7 @@ static void i2c_lld_serve_event_interrupt(I2CDriver *i2cp) {

      done by the DMA.*/

   switch (I2C_EV_MASK & (event | (regSR2 << 16))) {

   case I2C_EV5_MASTER_MODE_SELECT:

-    if ((i2cp->addr >> 8) > 0) { 

+    if ((i2cp->addr >> 8) > 0) {

       /* 10-bit address: 1 1 1 1 0 X X R/W */

       dp->DR = 0xF0 | (0x6 & (i2cp->addr >> 8)) | (0x1 & i2cp->addr);

     } else {