-
Notifications
You must be signed in to change notification settings - Fork 25
arm64: dts: qcom: common dtsi for Xiaomi SDM660-family (upstream prep) #166
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
arm64: dts: qcom: common dtsi for Xiaomi SDM660-family (upstream prep) #166
Conversation
|
there are over 10000 commits, how can we review these changes? |
nvm, there are last commits I see |
|
I'll drop some nitpicks:
instead of Except this small improvements, looks pretty good. |
d1d4031 to
94987e4
Compare
b2134b9 to
0c3ae74
Compare
setotau
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
There was a problem hiding this 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
0c3ae74 to
fdc6544
Compare
|
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 |
fdc6544 to
feaec76
Compare
feaec76 to
3beb0a2
Compare
…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>
3beb0a2 to
0910450
Compare
| * by individual device dts | ||
| */ | ||
| format = "a8r8g8b8"; | ||
| status = "disabled"; |
There was a problem hiding this comment.
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>; |
There was a problem hiding this comment.
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) */
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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"
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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.
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: for platina also Angelo was the original author, so: |
|
Also your every device's commit message says "..USB support for debugging." but it's a normal USB, not just for debugging? 😺 |

Per discussion in #161 and #165, this prepares the common dtsi for upstream submission.
Based on
torvalds/master(mainline).Patchset:
sdm660-xiaomi-common.dtsi, refactor lavender to use itValidation:
checkpatch.pl --strict: 0 errorsmake CHECK_DTBS=y: passedPlease review before I send to linux-arm-msm@vger.kernel.org.