Re: [PATCH 4/6] arm64: dts: qcom: Add sm6115-motorola-common

[Val Packett <val@packett.cool>]

On 11/9/26 7:14 PM, Val Packett wrote:
> Add the shared dtsi for sm6115/sm4250-based Motorola smartphones,
> with support for:
>
> * Simple framebuffer from bootloader
> * Power, volume, and assistant buttons (PMI632)
> * Charging LED, haptics/vibrator (PMI632)
> * USB Type-C with dual-role USB 2.0 (PMI632)
> * eMMC onboard storage and microSD card slot
> * Wi-Fi, Bluetooth (WCN3990)
>
> Signed-off-by: Val Packett <val@packett.cool>
> ---
>
> yes on both phones I've tested, only msm-id is required, not board-id.
>
> ---
>   .../boot/dts/qcom/sm6115-motorola-common.dtsi | 541 ++++++++++++++++++
>   1 file changed, 541 insertions(+)
>   create mode 100644 arch/arm64/boot/dts/qcom/sm6115-motorola-common.dtsi
>
> diff --git a/arch/arm64/boot/dts/qcom/sm6115-motorola-common.dtsi b/arch/arm64/boot/dts/qcom/sm6115-motorola-common.dtsi
> new file mode 100644
> index 000000000000..6a5110319bc3
> --- /dev/null
> +++ b/arch/arm64/boot/dts/qcom/sm6115-motorola-common.dtsi
> @@ -0,0 +1,541 @@
> +// SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause)
> +/*
> + * Copyright (c) 2026, Val Packett <val@packett.cool>
> + */
> +
> +#include <dt-bindings/arm/qcom,ids.h>
> +#include <dt-bindings/leds/common.h>
> +#include <dt-bindings/pinctrl/qcom,pmic-gpio.h>
> +#include <dt-bindings/usb/pd.h>
> +#include "pm6125.dtsi"
> +#include "pmi632.dtsi"
> +
> +/ {
> +	chassis-type = "handset";
> +
> +	/* required for bootloader to select correct board */
> +	qcom,msm-id = <QCOM_ID_SM6115 0x10000>;
> +
> +	aliases {
> +		bluetooth0 = &bluetooth;
> +		hsuart0 = &uart3;
> +		mmc0 = &sdhc_1;
> +		mmc1 = &sdhc_2;
> +		serial0 = &uart4;
> +		wifi0 = &wifi;
> +	};
> +
> +	chosen {
> +		#address-cells = <2>;
> +		#size-cells = <2>;
> +		ranges;
> +
> +		framebuffer0: framebuffer@5c000000 {
> +			compatible = "simple-framebuffer";
> +			memory-region = <&cont_splash_memory>;
> +			width = <720>;
> +			height = <1600>;
> +			stride = <(720 * 4)>;
> +			format = "a8r8g8b8";
> +
> +			clocks = <&gcc GCC_DISP_AHB_CLK>,
> +				 <&gcc GCC_DISP_HF_AXI_CLK>;
> +			vddio-supply = <&vreg_l9a>;
s/vreg_l9a/pm6125_l9a/ of course, last-second copy-paste fail, sorry
> [..]