imx93: fix kernel-imx93-current build — drop redundant blk-ctrl.c hunk (6.18.42)

[igorpecovnik]

Fixes the kernel-imx93-current failure (run 30825508798):

Failed to apply patch 0001-arm64-dts-freescale-add-MBa93xxLA-MINI-modifications.patch
  Hunk #2 FAILED at 301 (drivers/pmdomain/imx/imx93-blk-ctrl.c)
  1 out of 2 hunks FAILED → failed_apply → build abort

Root cause

The MBa93xxLA-MINI patch adds devm_of_platform_populate() + #include <linux/of_platform.h> to imx93_blk_ctrl_probe() (to probe the added LCDIF sub-node). That change is already upstream — mainline imx93-blk-ctrl.c has the include (line 10) and devm_of_platform_populate(dev) (line 360). On the 6.18.42 base the code is already present, so the hunk conflicts.

Fix

Drop the blk-ctrl.c changes from the patch; keep the DTS additions (imx91_93_common.dtsi LCDIF node + imx93.dtsi), which are still needed and applied cleanly. CI will confirm the patch now applies.

Summary by CodeRabbit

• New Features

  • Enabled LCD display controller support on compatible i.MX93 boards.
  • Added display bridge connectivity, clock configuration, power management, and endpoint wiring for LCD and LVDS output.

• Refactor

  • Simplified display support by removing automatic sub-device population and related cleanup logic.

[coderabbitai]

📝 Walkthrough

Walkthrough

The patch removes the imx93-blk-ctrl.c change and adds i.MX93 LCDIF and LVDS device-tree configuration, including clocks, power domains, and display endpoints.

Changes

i.MX93 display configuration

Layer / File(s)	Summary
LCDIF and LVDS device-tree wiring patch/kernel/archive/imx93-6.18/0001-arm64-dts-freescale-add-MBa93xxLA-MINI-modifications.patch	The patch adds an LCDIF node with resources, configures LCDIF and media block clocks, adds an LVDS bridge sub-device, and defines display endpoints. Patch metadata removes the previous imx93-blk-ctrl.c modification.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers: schmiedelm

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
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.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly identifies the build fix and the removal of the redundant blk-ctrl.c hunk.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch imx93-drop-redundant-blkctrl-hunk

---

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.}