Skip to content

Mekotronics R58X-Pro (vendor): use mainline u-boot for vendor; use new LCD driver#9836

Merged
igorpecovnik merged 2 commits into
armbian:mainfrom
rpardini:pr/Mekotronics-R58X-Pro-vendor-use-mainline-u-boot-for-vendor-use-new-LCD-driver
May 17, 2026
Merged

Mekotronics R58X-Pro (vendor): use mainline u-boot for vendor; use new LCD driver#9836
igorpecovnik merged 2 commits into
armbian:mainfrom
rpardini:pr/Mekotronics-R58X-Pro-vendor-use-mainline-u-boot-for-vendor-use-new-LCD-driver

Conversation

@rpardini
Copy link
Copy Markdown
Member

@rpardini rpardini commented May 16, 2026

Mekotronics R58X-Pro (vendor): use mainline u-boot for vendor; use new LCD driver

Summary by CodeRabbit

  • New Features

    • Enabled Bluetooth hardware support for compatible boards.
    • Added display driver support for auxiliary display configurations.
  • Bug Fixes

    • Improved device configuration handling for legacy vendor branches.
    • Enhanced USB gadget functionality availability across different firmware versions.
    • Refined boot and kernel command-line parameters for better system stability.

Review Change Stack

rpardini added 2 commits May 16, 2026 11:08
- new LCD driver for the Meko-R58X-Pro as a module at armbian/linux-rockchip#482
- old always-built-in `lcd_vk2c21` is gone
…or branch

- this unifies board config; vendor branch only gets a different DTB name
  and bluetooth stuff extra on top of edge
- drop usage of shared vendor/mekotronics config fragments
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 16, 2026

📝 Walkthrough

Walkthrough

This pull request updates board and kernel configuration for the Mekotronics R58X Pro board and broader RK35XX platform. It refactors the vendor/legacy branch setup to apply configuration inline rather than through external sourcing, adds bluetooth and kernel command-line parameters, ensures u-boot device tree patching applies consistently, and enables auxiliary display driver support in the kernel.

Changes

Mekotronics R58X Pro Vendor Configuration

Layer / File(s) Summary
Vendor/legacy boot parameters and UMS device tree patching
config/boards/mekotronics-r58x-pro.csc
The vendor/legacy branch no longer sources the shared vendor config or exits early; instead vendor setup including BOOT_FDT_FILE, BLUETOOTH_HCIATTACH_PARAMS, and SRC_CMDLINE is applied inline. The pre_config_uboot_target__meko_r58x_pro_patch_uboot_dtsi_for_ums function removes its branch guard, so UMS dtsi patching now applies to vendor builds.

RK35XX Kernel Display Configuration

Layer / File(s) Summary
Auxiliary display and VK2C21 LCD module configuration
config/kernel/linux-rk35xx-vendor.config
Kernel configuration enables CONFIG_AUXDISPLAY as a built-in feature and CONFIG_LCD_VK2C21 as a loadable module for VK2C21 LCD display controller support.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A rabbit hops through config land,
With boot flags held in fuzzy hand,
A display blooms where paths once crossed,
No early returns—just gains, no loss!
Device trees patch and modules load,
Building bright along the mekotronics road. 🎉

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the two main changes: switching to mainline u-boot for the vendor branch and enabling the new LCD driver, matching the actual modifications to the board configuration and kernel config files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


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

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions Bot added size/small PR with less then 50 lines 05 Milestone: Second quarter release Needs review Seeking for review Hardware Hardware related like kernel, U-Boot, ... labels May 16, 2026
@igorpecovnik igorpecovnik added 08 Milestone: Third quarter release and removed 05 Milestone: Second quarter release labels May 16, 2026
@rpardini rpardini marked this pull request as ready for review May 16, 2026 18:23
@github-actions github-actions Bot added the Ready to merge Reviewed, tested and ready for merge label May 16, 2026
@github-actions
Copy link
Copy Markdown
Contributor

✅ This PR has been reviewed and approved — all set for merge!

@github-actions github-actions Bot removed the Needs review Seeking for review label May 16, 2026
Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@config/kernel/linux-rk35xx-vendor.config`:
- Around line 2331-2332: The CONFIG_LCD_VK2C21 option is not present in the
rk35xx-vendor kernel 6.1 so CONFIG_LCD_VK2C21=m will be ignored; either backport
the VK2C21 driver patches from rockchip64-7.0/7.1 into the rk35xx-vendor-6.1
tree (ensuring you also pull in required Kconfig changes and dependencies like
GPIOLIB and OF), or remove CONFIG_LCD_VK2C21 and CONFIG_AUXDISPLAY from this
config to avoid misleading/unused entries; locate the entries by searching for
CONFIG_LCD_VK2C21 and CONFIG_AUXDISPLAY in the config/kernel file and apply the
chosen change (port patches + add Kconfig and drivers, or delete both config
lines).
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 08f6b50f-adc3-48ec-abff-04c2970ca49f

📥 Commits

Reviewing files that changed from the base of the PR and between 5e60c01 and 980d08c.

📒 Files selected for processing (2)
  • config/boards/mekotronics-r58x-pro.csc
  • config/kernel/linux-rk35xx-vendor.config

Comment thread config/kernel/linux-rk35xx-vendor.config
@igorpecovnik igorpecovnik merged commit 98914fb into armbian:main May 17, 2026
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

08 Milestone: Third quarter release Hardware Hardware related like kernel, U-Boot, ... Ready to merge Reviewed, tested and ready for merge size/small PR with less then 50 lines

Development

Successfully merging this pull request may close these issues.

3 participants