Skip to content

Conversation

@M0Rf30
Copy link

@M0Rf30 M0Rf30 commented Jan 14, 2026

Per discussion in #161 and #165, this prepares the common dtsi for upstream submission.

Based on torvalds/master (mainline).

Patchset:

  1. Create sdm660-xiaomi-common.dtsi, refactor lavender to use it
  2. dt-bindings for tulip, whyred, jasmine, platina
  3. Add tulip DTS (Redmi Note 6 Pro)
  4. Add whyred DTS (Redmi Note 5 Pro)
  5. Add jasmine DTS (Mi A2)
  6. Add platina DTS (Mi 8 Lite)

Validation:

  • checkpatch.pl --strict: 0 errors
  • make CHECK_DTBS=y: passed
  • All DTBs build

Please review before I send to linux-arm-msm@vger.kernel.org.

@setotau
Copy link

setotau commented Jan 14, 2026

there are over 10000 commits, how can we review these changes?

@setotau
Copy link

setotau commented Jan 14, 2026

there are over 10000 commits, how can we review these changes?

nvm, there are last commits I see

@M0Rf30 M0Rf30 changed the base branch from qcom-sdm660-6.18.y to master January 14, 2026 21:15
@okias
Copy link

okias commented Jan 14, 2026

I'll drop some nitpicks:

chosen > framebuffer - use

memory-region = <&framebufer_mem>;

instead of reg =

Except this small improvements, looks pretty good.

@M0Rf30 M0Rf30 force-pushed the upstreaming/xiaomi-common-v1 branch 2 times, most recently from d1d4031 to 94987e4 Compare January 14, 2026 21:47
@M0Rf30 M0Rf30 force-pushed the upstreaming/xiaomi-common-v1 branch 2 times, most recently from b2134b9 to 0c3ae74 Compare January 14, 2026 22:25
@M0Rf30 M0Rf30 requested review from minlexx, okias and setotau January 15, 2026 13:08
Copy link

@setotau setotau left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link

@okias okias left a comment

Choose a reason for hiding this comment

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

LGTM, last nitpick:
arm64: dts: qcom: add sdm660-xiaomi-common.dtsi for Xiaomi SDM660-family devices

should more like rather than "add" say something like "extract common parts into xiaomi-common". you can omit .dtsi in the commit msg

@M0Rf30 M0Rf30 force-pushed the upstreaming/xiaomi-common-v1 branch from 0c3ae74 to fdc6544 Compare January 15, 2026 13:53
@minlexx
Copy link
Member

minlexx commented Jan 15, 2026

And I think SoB chain is wrong in most commit messages where several authors are involved (missing co-developed-by?)

At least commit messages for whyred and platina

@M0Rf30 M0Rf30 force-pushed the upstreaming/xiaomi-common-v1 branch from fdc6544 to feaec76 Compare January 15, 2026 14:06
@M0Rf30 M0Rf30 force-pushed the upstreaming/xiaomi-common-v1 branch from feaec76 to 3beb0a2 Compare January 15, 2026 14:15
…omi-common

Create a shared device tree include file for Xiaomi smartphones and tablets
based on Qualcomm SDM636/SDM660 SoCs. This reduces code duplication when
adding new Xiaomi devices and ensures consistent configuration across the
family.

The common dtsi includes:
- PM660/PM660L PMIC configuration
- Serial console (blsp1_uart2)
- Volume and power buttons (gpio-keys, pon_pwrkey, pon_resin)
- USB (qusb2phy0, usb3) in peripheral mode
- SD card (sdhc_2) with card detect on GPIO 54
- eMMC (sdhc_1) with HS400 support
- Backlight (pm660l_wled)
- Framebuffer memory reservation
- TLMM GPIO reserved ranges

Refactor sdm660-xiaomi-lavender.dts to include the common file while
keeping device-specific parts (ramoops, regulators) in the board DTS.

Signed-off-by: Gianluca Boiano <morf3089@gmail.com>
Add device tree bindings for:
- Xiaomi Redmi Note 6 Pro (tulip) - SDM636
- Xiaomi Redmi Note 5 Pro (whyred) - SDM636
- Xiaomi Mi A2 (jasmine) - SDM660
- Xiaomi Mi 8 Lite (platina) - SDM660

These are Android-based smartphones that can boot mainline Linux
with simple-framebuffer and USB support.

Signed-off-by: Gianluca Boiano <morf3089@gmail.com>
Add initial device tree support for the Xiaomi Redmi Note 6 Pro
(codename: tulip), a smartphone based on Qualcomm SDM636 SoC with
4GB RAM and a 6.26" 1080x2280 display.

This enables booting to a framebuffer console with USB support
for debugging.

Signed-off-by: Gianluca Boiano <morf3089@gmail.com>
Add initial device tree support for the Xiaomi Redmi Note 5 Pro
(codename: whyred), a smartphone based on Qualcomm SDM636 SoC with
4/6GB RAM and a 5.99" 1080x2160 display.

This enables booting to a framebuffer console with USB support
for debugging.

Co-developed-by: Alexey Minnekhanov <alexeymin@postmarketos.org>
Signed-off-by: Alexey Minnekhanov <alexeymin@postmarketos.org>
Signed-off-by: Mathesh Velayudan <123v.mathesh@gmail.com>
Signed-off-by: Gianluca Boiano <morf3089@gmail.com>
Add initial device tree support for the Xiaomi Mi A2
(codename: jasmine), a smartphone based on Qualcomm SDM660 SoC with
4/6GB RAM and a 5.99" 1080x2160 display.

This enables booting to a framebuffer console with USB support
for debugging.

Signed-off-by: Joe Mason <buddyjojo06@outlook.com>
Signed-off-by: Gianluca Boiano <morf3089@gmail.com>
Add initial device tree support for the Xiaomi Mi 8 Lite
(codename: platina), a smartphone based on Qualcomm SDM660 SoC with
4/6GB RAM and a 6.26" 1080x2280 display.

This enables booting to a framebuffer console with USB support
for debugging.

Co-developed-by: Molly Sophia <mollysophia379@gmail.com>
Signed-off-by: Molly Sophia <mollysophia379@gmail.com>
Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@somainline.org>
Signed-off-by: Gianluca Boiano <morf3089@gmail.com>
@M0Rf30 M0Rf30 force-pushed the upstreaming/xiaomi-common-v1 branch from 3beb0a2 to 0910450 Compare January 15, 2026 14:24
@M0Rf30 M0Rf30 requested a review from minlexx January 15, 2026 14:26
@M0Rf30 M0Rf30 requested a review from okias January 15, 2026 14:32
* by individual device dts
*/
format = "a8r8g8b8";
status = "disabled";
Copy link

Choose a reason for hiding this comment

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

newline before status

};

&tlmm {
gpio-reserved-ranges = <8 4>;
Copy link

Choose a reason for hiding this comment

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

while we at it, mainline developers wants these days inline comment, what these GPIOs are reserved to.

Sample of recently merged Pixel 3 DT:

        gpio-reserved-ranges = < 0 4>, /* SPI (Intel MNH Pixel Visual Core) */
                               <81 4>; /* SPI (most likely Fingerprint Cards FPC1075) */
                

Copy link
Member

@minlexx minlexx Jan 15, 2026

Choose a reason for hiding this comment

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

So for platina GPIOs 8..11 are for fingerprint SPI.

image

And looks like GPIOs 0..3 should also be reserved for eSE (embedded Secure Element) ? But we can't test platina, so.. or maybe not, it's the same for lavender and it boots fine without reserving 0..3.

Copy link

Choose a reason for hiding this comment

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

Is this secure element related to NFC? Afaik platina doesn't have it.

Copy link
Member

Choose a reason for hiding this comment

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

For whyred, jasmine and lavender it's the same - gpios 8..11 are "Fingerprint SPI"

Copy link
Member

Choose a reason for hiding this comment

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

Is this secure element related to NFC? Afaik platina doesn't have it.

Perhaps

Copy link

Choose a reason for hiding this comment

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

I've googled. It doesn't relate to NFC.

@minlexx
Copy link
Member

minlexx commented Jan 15, 2026

And I think SoB chain is wrong in most commit messages where several authors are involved (missing co-developed-by?)

At least commit messages for whyred and platina

I think in commit messages SoB chain should go top-to-bottom in chronological order: https://docs.kernel.org/process/submitting-patches.html#when-to-use-acked-by-cc-and-co-developed-by

So, for whyred:

Co-developed-by: Mathesh Velayudan <123v.mathesh@gmail.com>
Signed-off-by: Mathesh Velayudan <123v.mathesh@gmail.com>
Co-developed-by: Alexey Minnekhanov <alexeymin@postmarketos.org>
Signed-off-by: Alexey Minnekhanov <alexeymin@postmarketos.org>
Signed-off-by: Gianluca Boiano <morf3089@gmail.com>

for platina also Angelo was the original author, so:

Co-developed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@somainline.org>
Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@somainline.org>
Co-developed-by: Molly Sophia <mollysophia379@gmail.com>
Signed-off-by: Molly Sophia <mollysophia379@gmail.com>
Signed-off-by: Gianluca Boiano <morf3089@gmail.com>

@minlexx
Copy link
Member

minlexx commented Jan 15, 2026

Also your every device's commit message says "..USB support for debugging." but it's a normal USB, not just for debugging? 😺

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants