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 <dsankou...@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

Reply via email to