[bpi-r64] support pstore/ramoops

dangowrt · April 15, 2024, 9:52pm

Yes, linux-dt. We have no influence on U-Boot dt on third-party loaders.

frank-w · April 16, 2024, 5:49am

I talk about these patches where you add dt nodes to uboot (and cause dtc warning for mt7622),but @moore does not and sets options in config

github.com

openwrt/openwrt/blob/main/package/boot/uboot-mediatek/patches/050-mt7622-enable-pstore.patch

--- a/arch/arm/dts/mt7622.dtsi
+++ b/arch/arm/dts/mt7622.dtsi
@@ -37,6 +37,30 @@
 		};
 	};
 
+	psci {
+		compatible  = "arm,psci-1.0";
+		method      = "smc";
+	};
+
+	reserved-memory {
+		#address-cells = <2>;
+		#size-cells = <2>;
+		ranges;
+
+		/* 64 KiB reserved for ramoops/pstore */
+		ramoops@42ff0000 {
+			compatible = "ramoops";
+			reg = <0 0x42ff0000 0 0x10000>;

This file has been truncated. show original

github.com

openwrt/openwrt/blob/main/package/boot/uboot-mediatek/patches/051-mt7986-enable-pstore.patch

--- a/arch/arm/dts/mt7986.dtsi
+++ b/arch/arm/dts/mt7986.dtsi
@@ -50,6 +50,35 @@
 		};
 	};
 
+	psci {
+		compatible  = "arm,psci-0.2";
+		method      = "smc";
+	};
+
+	reserved-memory {
+		#address-cells = <2>;
+		#size-cells = <2>;
+		ranges;
+
+		/* 64 KiB reserved for ramoops/pstore */
+		ramoops@42ff0000 {
+			compatible = "ramoops";
+			reg = <0 0x42ff0000 0 0x10000>;

This file has been truncated. show original

Vs.

CONFIG_CMD_PSTORE=y
CONFIG_CMD_PSTORE_MEM_ADDR=0x42ff0000
CONFIG_CMD_PSTORE_RECORD_SIZE=0x2000
CONFIG_CMD_PSTORE_CONSOLE_SIZE=0x2000
CONFIG_CMD_PSTORE_FTRACE_SIZE=0x0
CONFIG_CMD_PSTORE_PMSG_SIZE=0x2000

dangowrt · April 20, 2024, 2:34am

Probably just two ways to do the same thing – no idea which would be preferred. To me, doing it in DT looks better, but that might be just my taste.

frank-w · July 16, 2024, 8:54am

Next hang try to show the pstore in linux or uboot.

You find commands here (just ignore that it was not working for me at this time):