Merge pull request #15 from m5stack/develop

0.0.7

Author: lovyan03

library.json

@@ -10,7 +10,7 @@
     "type": "git",
     "url": "https://github.com/m5stack/M5GFX.git"
   },
-  "version": "0.0.6",
+  "version": "0.0.7",
   "framework": "arduino",
   "platforms": "espressif32"
 }

library.properties

@@ -1,5 +1,5 @@
 name=M5GFX
-version=0.0.6
+version=0.0.7
 author=M5Stack
 maintainer=M5Stack
 sentence=Library for M5Stack All Display

src/M5GFX.cpp

@@ -130,9 +130,12 @@ namespace m5gfx
 
       if (_cfg.pin_int >= 0)
       {
-        lgfx::pinMode(_cfg.pin_int, lgfx::pin_mode_t::input_pullup);
+        lgfx::pinMode(_cfg.pin_int, lgfx::pin_mode_t::input);
       }
       _inited = lgfx::i2c::init(_cfg.i2c_port, _cfg.pin_sda, _cfg.pin_scl).has_value();
+      static constexpr uint8_t irq_modechange_cmd[] = { 0x5a, 0x5a };  /// (INT mode change)
+      lgfx::i2c::transactionWrite(_cfg.i2c_port, _cfg.i2c_addr, irq_modechange_cmd, 2);
+
       return _inited;
     }
 
@@ -141,10 +144,10 @@ namespace m5gfx
       if (tp) tp->size = 0;
       if (!_inited || count == 0) return 0;
       if (count > 2) count = 2; // max 2 point.
-      // if (_cfg.pin_int >= 0)
-      // {
-      //   Serial.printf("tp:%d \r\n", gpio_in(_cfg.pin_int));
-      // }
+      if (_cfg.pin_int >= 0)
+      {
+        if (gpio_in(_cfg.pin_int)) return 0;
+      }
 
       std::size_t len = 3 + count * 6;
       std::uint8_t buf[2][len];

src/M5GFX.h

@@ -144,6 +144,8 @@ namespace m5gfx
     , board_M5Paper
     , board_M5Tough
     , board_M5ATOM
+    , board_M5Camera
+    , board_M5TimerCam
     };
   }
   using board_t = boards::board_t;

src/lgfx/v1/Bus.hpp

@@ -42,53 +42,53 @@ namespace lgfx
 
     virtual bus_type_t busType(void) const = 0;
 
-    /// ペリフェラルの準備を行う。
+    /// ペリフェラルの準備を行う。;
     virtual bool init(void) = 0;
 
-    /// ペリフェラルを解放する。
+    /// ペリフェラルを解放する。;
     virtual void release(void) = 0;
 
-    /// 通信トランザクションを開始する。（ペリフェラルを占有する）
+    /// 通信トランザクションを開始する。（ペリフェラルを占有する）;
     virtual void beginTransaction(void) = 0;
 
-    /// 通信トランザクションを終了する。（ペリフェラルの占有を終了する）
+    /// 通信トランザクションを終了する。（ペリフェラルの占有を終了する）;
     virtual void endTransaction(void) = 0;
 
-    /// 通信が完了するのを待機する
+    /// 通信が完了するのを待機する;
     virtual void wait(void) = 0;
 
-    /// 現在通信中か否かを返す。true:通信中
+    /// 現在通信中か否かを返す。true:通信中;
     virtual bool busy(void) const = 0;
 
-    /// DMA転送に必要なペリフェラルの準備を行う。
+    /// DMA転送に必要なペリフェラルの準備を行う。;
     virtual void initDMA(void) = 0;
 
-    /// DMA転送キューを追加する。
+    /// DMA転送キューを追加する。;
     virtual void addDMAQueue(const uint8_t* data, uint32_t length) = 0; // { writeBytes(data, length, true); }
 
-    /// 蓄積したDMA転送キューの送信を実行する。
+    /// 蓄積したDMA転送キューの送信を実行する。;
     virtual void execDMAQueue(void) = 0;
 
-    /// DMA用のバッファを取得する。バスの実装によっては内部的には2個のバッファを交互に使用する。
-    /// 繰返し実行した場合は前回と異なるポインタを得るが、前々回と同じになる場合がある点に注意すること。
+    /// DMA用のバッファを取得する。バスの実装によっては内部的には2個のバッファを交互に使用する。;
+    /// 繰返し実行した場合は前回と異なるポインタを得るが、前々回と同じになる場合がある点に注意すること。;
     virtual uint8_t* getDMABuffer(uint32_t length) = 0;
 
-    /// 未送信のデータがあれば送信を開始する。
+    /// 未送信のデータがあれば送信を開始する。;
     virtual void flush(void) = 0;
 
-    /// D/Cピンをlowにしてデータを送信する。
+    /// D/Cピンをlowにしてデータを送信する。;
     virtual bool writeCommand(uint32_t data, uint_fast8_t bit_length) = 0;
 
-    /// D/Cピンをhighにしてデータを送信する。
+    /// D/Cピンをhighにしてデータを送信する。;
     virtual void writeData(uint32_t data, uint_fast8_t bit_length) = 0;
 
-    /// D/Cピンをhighにして指定回数繰り返しデータを送信する。
+    /// D/Cピンをhighにして指定回数繰り返しデータを送信する。;
     virtual void writeDataRepeat(uint32_t data, uint_fast8_t bit_length, uint32_t count) = 0;
 
-    /// pixelcopy構造体を利用してピクセルデータを送信する。
+    /// pixelcopy構造体を利用してピクセルデータを送信する。;
     virtual void writePixels(pixelcopy_t* pc, uint32_t length) = 0;
 
-    /// 引数のバイト列を送信する。
+    /// 引数のバイト列を送信する。;
     virtual void writeBytes(const uint8_t* data, uint32_t length, bool dc, bool use_dma) = 0;
 
     virtual void beginRead(void) = 0;

src/lgfx/v1/LGFXBase.cpp

@@ -31,6 +31,13 @@ Original Source:
 #include <cmath>
 #include <list>
 
+#ifdef min
+#undef min
+#endif
+#ifdef max
+#undef max
+#endif
+
 namespace lgfx
 {
  inline namespace v1
@@ -1095,31 +1102,31 @@ namespace lgfx
     result[5] =  dst_y - src_x * result[3] - src_y * result[4];
   }
 
-  static bool make_invert_affine32(int32_t* __restrict__ result, const float* __restrict__ matrix)
+  static bool make_invert_affine32(int32_t* __restrict result, const float* __restrict matrix)
   {
     float det = matrix[0] * matrix[4] - matrix[1] * matrix[3];
     if (det == 0.0f) return false;
     det = (1 << FP_SCALE) / det;
-    result[0] = roundf(det *  matrix[4]);
-    result[1] = roundf(det * -matrix[1]);
-    result[2] = roundf(det * (matrix[1] * matrix[5] - matrix[2] * matrix[4]));
-    result[3] = roundf(det * -matrix[3]);
-    result[4] = roundf(det *  matrix[0]);
-    result[5] = roundf(det * (matrix[2] * matrix[3] - matrix[0] * matrix[5]));
+    result[0] = (int32_t)roundf(det *  matrix[4]);
+    result[1] = (int32_t)roundf(det * -matrix[1]);
+    result[2] = (int32_t)roundf(det * (matrix[1] * matrix[5] - matrix[2] * matrix[4]));
+    result[3] = (int32_t)roundf(det * -matrix[3]);
+    result[4] = (int32_t)roundf(det *  matrix[0]);
+    result[5] = (int32_t)roundf(det * (matrix[2] * matrix[3] - matrix[0] * matrix[5]));
     return true;
   }
 
   void LGFXBase::push_image_rotate_zoom(float dst_x, float dst_y, float src_x, float src_y, float angle, float zoom_x, float zoom_y, int32_t w, int32_t h, pixelcopy_t* pc)
   {
     float matrix[6];
-    make_rotation_matrix(matrix, dst_x + 0.5, dst_y + 0.5, src_x + 0.5, src_y + 0.5, angle, zoom_x, zoom_y);
+    make_rotation_matrix(matrix, dst_x + 0.5f, dst_y + 0.5f, src_x + 0.5f, src_y + 0.5f, angle, zoom_x, zoom_y);
     push_image_affine(matrix, w, h, pc);
   }
 
   void LGFXBase::push_image_rotate_zoom_aa(float dst_x, float dst_y, float src_x, float src_y, float angle, float zoom_x, float zoom_y, int32_t w, int32_t h, pixelcopy_t* pc)
   {
     float matrix[6];
-    make_rotation_matrix(matrix, dst_x + 0.5, dst_y + 0.5, src_x + 0.5, src_y + 0.5, angle, zoom_x, zoom_y);
+    make_rotation_matrix(matrix, dst_x + 0.5f, dst_y + 0.5f, src_x + 0.5f, src_y + 0.5f, angle, zoom_x, zoom_y);
     push_image_affine_aa(matrix, w, h, pc);
   }
 
@@ -1485,11 +1492,11 @@ namespace lgfx
     p.src_bits = _read_conv.depth & color_depth_t::bit_mask;
     switch (_read_conv.depth)
     {
-    case color_depth_t::rgb888_3Byte: p.fp_copy = pixelcopy_t::compare_rgb_fast<bgr888_t>;  break;
-    case color_depth_t::rgb666_3Byte: p.fp_copy = pixelcopy_t::compare_rgb_fast<bgr666_t>;  break;
-    case color_depth_t::rgb565_2Byte: p.fp_copy = pixelcopy_t::compare_rgb_fast<swap565_t>; break;
-    case color_depth_t::rgb332_1Byte: p.fp_copy = pixelcopy_t::compare_rgb_fast<rgb332_t>;  break;
-    default: p.fp_copy = pixelcopy_t::compare_bit_fast;
+    case color_depth_t::rgb888_3Byte: p.fp_copy = pixelcopy_t::compare_rgb_affine<bgr888_t>;  break;
+    case color_depth_t::rgb666_3Byte: p.fp_copy = pixelcopy_t::compare_rgb_affine<bgr666_t>;  break;
+    case color_depth_t::rgb565_2Byte: p.fp_copy = pixelcopy_t::compare_rgb_affine<swap565_t>; break;
+    case color_depth_t::rgb332_1Byte: p.fp_copy = pixelcopy_t::compare_rgb_affine<rgb332_t>;  break;
+    default: p.fp_copy = pixelcopy_t::compare_bit_affine;
       p.src_mask = (1 << p.src_bits) - 1;
       p.transp &= p.src_mask;
       break;
@@ -1529,6 +1536,8 @@ namespace lgfx
           it = points.begin();
 
           bufY[0] = it->y;
+          p.src_x32_add = 1 << FP_SCALE;
+          p.src_y32_add = 0;
           _panel->readRect(cl, it->y, w, 1, linebufs[0], &p);
         }
         else
@@ -1569,6 +1578,8 @@ namespace lgfx
         if (bidx == 3) {
           for (bidx = 0; bidx < 2 && (abs(bufY[bidx] - ly) <= 1); ++bidx);
           bufY[bidx] = newy;
+          p.src_x32_add = 1 << FP_SCALE;
+          p.src_y32_add = 0;
           _panel->readRect(cl, newy, w, 1, linebufs[bidx], &p);
         }
         auto linebuf = &linebufs[bidx][- cl];
@@ -1610,10 +1621,10 @@ namespace lgfx
 
   static char* floatToStr(double number, char* buf, size_t /*buflen*/, uint8_t digits)
   {
-    if (std::isnan(number))    { return strcpy(buf, "nan"); }
-    if (std::isinf(number))    { return strcpy(buf, "inf"); }
-    if (number > 4294967040.0) { return strcpy(buf, "ovf"); } // constant determined empirically
-    if (number <-4294967040.0) { return strcpy(buf, "ovf"); } // constant determined empirically
+    if (std::isnan(number))    { return (char*)memcpy(buf, "nan\0", 4); }
+    if (std::isinf(number))    { return (char*)memcpy(buf, "inf\0", 4); }
+    if (number > 4294967040.0) { return (char*)memcpy(buf, "ovf\0", 4); } // constant determined empirically
+    if (number <-4294967040.0) { return (char*)memcpy(buf, "ovf\0", 4); } // constant determined empirically
 
     char* dst = buf;
     // Handle negative numbers
@@ -1996,7 +2007,7 @@ namespace lgfx
   }
 
 #if !defined (ARDUINO)
-  size_t LGFXBase::printf(const char * __restrict__ format, ...) 
+  size_t LGFXBase::printf(const char * __restrict format, ...) 
   {
     va_list arg;
     va_start(arg, format);
@@ -2007,7 +2018,7 @@ namespace lgfx
   }
 #endif
 
-  size_t LGFXBase::vprintf(const char* __restrict__ format, va_list arg)
+  size_t LGFXBase::vprintf(const char* __restrict format, va_list arg)
   {
     char loc_buf[64];
     char * temp = loc_buf;
@@ -2967,7 +2978,7 @@ namespace lgfx
       uint_fast8_t panel_offsetrot = panel()->config().offset_rotation;
       uint_fast8_t touch_offsetrot = touch()->config().offset_rotation;
 
-      // 回転オフセットをキャンセルしてタッチデバイスのデフォルトの向きに合わせる
+      // 回転オフセットをキャンセルしてタッチデバイスのデフォルトの向きに合わせる;
       setRotation(( (touch_offsetrot ^ panel_offsetrot) & 4)
                  |(-(touch_offsetrot + panel_offsetrot) & 3));