Skip to content

Remove HW CS SPI configuration from Device tree configuration#60

Merged
vmanchala-dev merged 6 commits intocomcard_v2_cifrom
vj/remove_hw_cs_spi_config
Jan 28, 2026
Merged

Remove HW CS SPI configuration from Device tree configuration#60
vmanchala-dev merged 6 commits intocomcard_v2_cifrom
vj/remove_hw_cs_spi_config

Conversation

@vmanchala-dev
Copy link

@vmanchala-dev vmanchala-dev commented Jan 27, 2026

Description

  • Removed HW CS configuration in device tree and CS is configured from user space application
  • Updated RESET pin.

Test

  • Verified with MOSI test example and able to led toggling.

Note

Disables hardware chip select on SPI1 in the BeagleBone device tree and shifts CS handling to userspace; aligns Elixir TFT code accordingly.

  • Device tree (BB-SPIDEV1-00A0_zola.dts): comments out spi1_cs0 (P9.28) from exclusive-use, pinmux disable, and pinctrl entries; retains spi1 with spidev channels 1.0 and 1.1 enabled
  • Elixir (tft_id.ex, tft_spi_ili9488.ex): updates reset GPIO to 74, parameterizes SPI device via @spi_device, switches to import Bitwise; TFTTest adds register reads and rotation set; Ili9488 removes old init path and uses new_init_display exclusively

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

@cursor
Copy link

cursor bot commented Jan 27, 2026

You have run out of free Bugbot PR reviews for this billing cycle. This will reset on February 19.

To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

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 2 potential issues.

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

@vmanchala-dev vmanchala-dev merged commit 187de49 into comcard_v2_ci Jan 28, 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