Skip to content

Fix SPI1 dts configuration to resolve TFT MISO read and touch issues on V2 board#64

Merged
vmanchala-dev merged 1 commit intocomcard_v2_cifrom
vj/fix_miso_pin_config_ci
Feb 25, 2026
Merged

Fix SPI1 dts configuration to resolve TFT MISO read and touch issues on V2 board#64
vmanchala-dev merged 1 commit intocomcard_v2_cifrom
vj/fix_miso_pin_config_ci

Conversation

@vmanchala-dev
Copy link

@vmanchala-dev vmanchala-dev commented Feb 24, 2026

Description

Fixed SPI1 communication issues on the V2 board for LCD display Id read and touch issue, now ILI9488 display register reads
ILI9488 display register reads
Added below changes

0x190 0x2B   /* spi1_sclk  - RXACTIVE=1, pull disabled | MODE3 */
0x194 0x2B   /* spi1_d0    - RXACTIVE=1, pull disabled | MODE3 */
0x198 0x13   /* spi1_d1    - output, MODE3 */

Disabled internal pull resistors on SCLK and MISO
Enabled RXACTIVE on input signals

Verification

  • Display ID read correctly: 0x4033
  • SPI register reads stable
  • Touch controller issue fixed
  • Verified using Saleae logic analyzer

Note

Low Risk
Small device-tree pinmux tweak limited to SPI1 signal electrical configuration; main risk is unintended SPI timing/electrical side effects on boards relying on the previous pull-up settings.

Overview
Updates the BB-SPIDEV1-00A0_zola.dts SPI1 pinmux to enable RXACTIVE and disable internal pulls on spi1_sclk and spi1_d0 (MISO), replacing the prior pull-up configurations.

This is intended to stabilize SPI read behavior (e.g., TFT register reads / touch controller reads) by changing the electrical characteristics of the clock and input data lines while leaving the rest of the SPI1 overlay unchanged.

Written by Cursor Bugbot for commit 2adba18. This will update automatically on new commits. Configure here.

@vmanchala-dev vmanchala-dev requested a review from vs-pp February 24, 2026 12:54
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

@vmanchala-dev vmanchala-dev merged commit 851050c into comcard_v2_ci Feb 25, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants