Implement Tulip Creative Computer 4r11

[NellowTCS]

This pull request adds support for the Tulip Creative Computer 4 (rev R11) hardware platform, including device configuration, driver bindings, and hardware-specific adjustments. The changes introduce a new device definition, update driver modules to support the Tulip 4 R11's peripherals, and enhance the GT911 touch controller driver for greater board compatibility.

Driver and binding improvements

• Updated the dummy I2S amplifier driver and bindings to support the TI PCM5101A DAC used on Tulip 4 R11, including YAML schema, header, and driver registration.
• Enhanced the GT911 touch controller driver and YAML schema to support board-specific reset polarity, reset pulse count, and coordinate calibration (offsets and scaling), necessary for correct operation on Tulip 4 R11.

Summary by CodeRabbit

• New Features
  • Added support for the Tulip Creative Computer 4R11, including its display, touch input, audio, battery sensing, Bluetooth, and USB host features.
  • Added support for 8-bit RGB332 displays.
  • Added touch calibration with configurable reset behavior, scaling, offsets, and persisted settings.
  • Added compatibility for PCM5101A-based audio output.
• Bug Fixes
  • Improved touch accuracy through calibration and boundary clamping.
  • Added safer handling for display conversion buffer allocation failures.

[NellowTCS]

There is no proper fix for the display being yellow-green and the icons being a shade lighter, as that's one of the drawbacks of RGB332

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 84c11ddd-0135-4345-9a02-d1e92c9490b6

📥 Commits

Reviewing files that changed from the base of the PR and between 471bf13 and 66396fe.

📒 Files selected for processing (1)

• Drivers/gt911-module/bindings/goodix,gt911.yaml

💤 Files with no reviewable changes (1)

• Drivers/gt911-module/bindings/goodix,gt911.yaml

---

📝 Walkthrough

Walkthrough

Adds the Tulip 4R11 device component and hardware description. Extends the GT911 driver with configurable reset and coordinate calibration. Adds RGB565-to-RGB332 conversion for 8-bit displays. Adds PCM5101A compatibility to the dummy I2S amplifier driver. Applies valid saved touch calibration settings during LVGL startup.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 46.67% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and concisely identifies the main change: implementing support for the Tulip Creative Computer 4 R11.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[coderabbitai]

Actionable comments posted: 1

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

Drivers/rgb-display-module/source/rgb_display.cpp (1)

240-270: 🩺 Stability & Availability | 🟠 Major | ⚡ Quick win

Free rgb332_scratch on later startup failures.

If semaphore creation or callback registration fails, the function frees internal but leaks the RGB332 scratch buffer. A retry can leak one full-frame buffer per failed start.

Proposed fix

     internal->frame_complete_semaphore = xSemaphoreCreateBinary();
     if (internal->frame_complete_semaphore == nullptr) {
+        heap_caps_free(internal->rgb332_scratch);
         esp_lcd_panel_del(internal->panel_handle);
         free(internal);
         return ERROR_OUT_OF_MEMORY;
@@
         LOG_E(TAG, "Failed to register panel event callbacks");
         vSemaphoreDelete(internal->frame_complete_semaphore);
+        heap_caps_free(internal->rgb332_scratch);
         esp_lcd_panel_del(internal->panel_handle);
         free(internal);
         return ERROR_RESOURCE;

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: f1d48502-bd33-4702-b25f-d8ea6fa44b12

📥 Commits

Reviewing files that changed from the base of the PR and between d2c69ee and 471bf13.

⛔ Files ignored due to path filters (9)

• partitions-16mb-no-sd-dev.csv is excluded by !**/*.csv
• partitions-16mb-no-sd.csv is excluded by !**/*.csv
• partitions-16mb-with-sd.csv is excluded by !**/*.csv
• partitions-32mb-no-sd-dev.csv is excluded by !**/*.csv
• partitions-32mb-no-sd.csv is excluded by !**/*.csv
• partitions-4mb-with-sd.csv is excluded by !**/*.csv
• partitions-8mb-no-sd-dev.csv is excluded by !**/*.csv
• partitions-8mb-no-sd.csv is excluded by !**/*.csv
• partitions-8mb-with-sd.csv is excluded by !**/*.csv

📒 Files selected for processing (14)

• Devices/tulip-4r11/CMakeLists.txt
• Devices/tulip-4r11/LICENSE-Apache-2.0.md
• Devices/tulip-4r11/device.properties
• Devices/tulip-4r11/devicetree.yaml
• Devices/tulip-4r11/source/module.cpp
• Devices/tulip-4r11/tulip-4r11.dts
• Drivers/dummy-i2s-amp-module/bindings/ti,pcm5101a.yaml
• Drivers/dummy-i2s-amp-module/include/bindings/dummy_i2s_amp.h
• Drivers/dummy-i2s-amp-module/source/dummy_i2s_amp.cpp
• Drivers/gt911-module/bindings/goodix,gt911.yaml
• Drivers/gt911-module/include/drivers/gt911.h
• Drivers/gt911-module/source/gt911.cpp
• Drivers/rgb-display-module/source/rgb_display.cpp
• Tactility/Source/Tactility.cpp