Re: [PATCH v3 23/23] arm64: dts: qcom: starqltechn: add new features

Konrad Dybcio
2024-06-18T14:12:18Z

On 6/18/24 15:59, Dzmitry Sankouski wrote:
> Add support for new features:
> - sound (headphones and mics only)
> - gpu
> - panel
> - buttons
> - MAX77705 MFD:
>    - charger
>    - fuelgauge
>    - haptic
>    - led
>
> Signed-off-by: Dzmitry Sankouski <dsankouski@gmail.com>
> ---

Please split this up

> .../boot/dts/qcom/sdm845-samsung-starqltechn.dts | 593 ++++++++++++++++++++-
> 1 file changed, 592 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/sdm845-samsung-starqltechn.dts b/arch/arm64/boot/dts/qcom/sdm845-samsung-starqltechn.dts
> index bad154b1e894..28a5210e22fb 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845-samsung-starqltechn.dts
> +++ b/arch/arm64/boot/dts/qcom/sdm845-samsung-starqltechn.dts
> @@ -7,15 +7,40 @@
>
> /dts-v1/;
>
> +#include <dt-bindings/input/linux-event-codes.h>
> #include <dt-bindings/gpio/gpio.h>
> #include <dt-bindings/regulator/qcom,rpmh-regulator.h>
> +#include <dt-bindings/sound/qcom,q6afe.h>
> +#include <dt-bindings/sound/qcom,q6asm.h>
> +
> #include "sdm845.dtsi"
> +#include "pm8998.dtsi"
> +#include "sdm845-wcd9340.dtsi"
> +
> +/delete-node/ &rmtfs_mem;
> +/delete-node/ &adsp_mem;
> +/delete-node/ &slpi_mem;
> +/delete-node/ &spss_mem;
>
> / {
> chassis-type = "handset";
> model = "Samsung Galaxy S9 SM-G9600";
> compatible = "samsung,starqltechn", "qcom,sdm845";
>
> + aliases {
> + serial0 = &uart9;
> + };
> +
> + battery: battery {
> + compatible = "simple-battery";
> + constant-charge-current-max-microamp = <2150000>;
> + charge-full-design-microamp-hours = <3000000>;
> +
> + over-voltage-threshold-microvolt = <4500000>;
> + voltage-min-design-microvolt = <3400000>;
> + voltage-max-design-microvolt = <4350000>;
> + };
> +
> chosen {
> #address-cells = <2>;
> #size-cells = <2>;
> @@ -59,9 +84,199 @@ memory@a1300000 {
> ftrace-size = <0x40000>;
> pmsg-size = <0x40000>;
> };
> +
> + /* The rmtfs_mem needs to be guarded due to "XPU limitations"
> + * it is otherwise possible for an allocation adjacent to the
> + * rmtfs_mem region to trigger an XPU violation, causing a crash.
> + */
> + rmtfs_lower_guard: memory@fde00000 {
> + no-map;
> + reg = <0 0xfde00000 0 0x1000>;
> + };

qcom,use-guard-pages instead

> +
> + rmtfs_mem: rmtfs-mem@fde01000 {
> + compatible = "qcom,rmtfs-mem";
> + reg = <0 0xfde01000 0 0x200000>;
> + no-map;
> +
> + qcom,client-id = <1>;
> + qcom,vmid = <15>;

QCOM_SCM_VMID_MSS_MSA

[...]

> + /*
> + * It seems like reserving the old rmtfs_mem region is also needed to prevent
> + * random crashes which are most likely modem related, more testing needed.
> + */
> + removed_region: removed-region@88f00000 {
> + no-map;
> + reg = <0 0x88f00000 0 0x1c00000>;

Please keep no-map below reg for consistency

[...]

> +
> +&gmu {
> + status = "okay";
> +};

Drop this and remove the disablement in 845.dtsi, gmu is only probed when
GPU is

[...]

> gpio-reserved-ranges = <0 4>, <27 4>, <81 4>, <85 4>;

Do you know what these are for?

Konrad