[riot-notifications] [RIOT-OS/RIOT] drivers: support for NXP PCA9685 I2C 16-channel, 12-bit PWM controller (#10556)

benpicco notifications at github.com
Fri Sep 6 17:14:08 CEST 2019


benpicco commented on this pull request.



> + */
+typedef enum {
+    PCA9685_OFF = 0,      /**< If /OE pin is HIGH, outputs are LOW */
+    PCA9685_OUT_DRV = 1,  /**< Outputs depend on the output driver mode
+                               pca9685_params_t::out_drv. If /OE pin is HIGH,
+                               it is high-impedance for PCA9685_OPEN_DRAIN,
+                               and HIGH for PCA9685_TOTEM_POLE */
+    PCA9685_HIHGH_Z = 2,  /**< If /OE pin is HIGH, outputs are high-impedance */
+} pca9685_out_ne_t;
+
+/**
+ * @brief   PCA9685 device initialization parameters
+ */
+typedef struct {
+
+    unsigned i2c_dev;   /**< I2C device, default I2C_DEV(0) */

```suggestion
    i2c_t i2c_dev;   /**< I2C device, default I2C_DEV(0) */
```

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/10556#pullrequestreview-284929384
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190906/08db7e7b/attachment.htm>


More information about the notifications mailing list